fix(setup): headless/CI mode + fork type-staleness warning + doc drift#13
Merged
Conversation
Addresses a second clean-environment evaluation. - TOC: fix stale "#quick-setup-with-claude-code" anchor to match the renamed "Quick Setup (Automated)" heading; drop the remaining "don't have Claude Code" framing from the Manual Setup intro. - Project Structure: add scripts/setup-bootstrap.mjs (the actual entry point `pnpm setup` runs) to the diagram. - Headless/CI: add a non-interactive mode (-y/--yes/--non-interactive), also auto-enabled when stdin is not a TTY, so @inquirer/prompts no longer throws in a pipeline. It keeps existing .env.local values, auto-generates missing secrets, and applies defaults; all step 2/4/5/6 prompts are guarded. Documented under Quick Setup. - Fork type-staleness footgun: when type generation fails and committed models are used as a fallback, the warning now loudly states the committed models come from a reference tenant and may not match the user's instance, and to run `pnpm mp:generate:models` before relying on them. README's committed-models note expanded to say the same. (The reported "v18+ / Claude Code" GitHub web view was stale CDN cache; the clone is already correct — confirmed no v18 references remain.) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Next.js 16 (Turbopack) regenerates next-env.d.ts to import from ./.next/dev/types/routes.d.ts. Committing the regenerated reference so the tracked file matches the installed Next version and the working tree stays clean (this file is generated, not hand-edited). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promotes the second-evaluation fixes from
devtomain. Follow-up to #11 and #12.Changes
Headless / CI mode (
scripts/setup.ts)-y/--yes/--non-interactive, also auto-enabled when stdin isn't a TTY, so@inquirer/promptsno longer throws in a pipeline..env.localvalues, auto-generates missing secrets, applies defaults. All prompts in steps 2/4/5/6 are guarded.node scripts/setup-bootstrap.mjs --yes.Fork type-staleness footgun
pnpm mp:generate:modelsbefore relying on them. README's committed-models note expanded to match.Doc drift
#quick-setup-with-claude-code→#quick-setup-automated).scripts/setup-bootstrap.mjs(the realpnpm setupentry point) to the Project Structure diagram.Chore
next-env.d.tsto the Next 16.next/dev/typespath.Testing
tsc --noEmitclean;--helprenders the new flag; prompt-guard coverage reviewed across all interactive steps. A full live--yesbuild was not run (it would regeneratemodels/against the live tenant); non-interactive path verified by type-check + review.🤖 Generated with Claude Code