feat: Update wizard to 0.42.0 - #639
Merged
Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
The optional dependency was pinned at `^0.38.1`. Because npm's caret operator only permits patch-level changes below 1.0.0, that range resolves to `>=0.38.1 <0.39.0`, so `npx @seamapi/cli wizard` installed a wizard four minor versions behind the published 0.42.0. Lockfile regenerated with the pinned npm@11.19.0 so the wizard's optional peer subtree (ink, react, and their peers) is preserved. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
chmaltsp
force-pushed
the
fix/update-wizard-0.42.0
branch
from
August 27, 2026 23:56
6984fef to
975902a
Compare
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
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.
Problem
@seamapi/clideclares the wizard as an optional dependency:npm's caret operator only permits patch-level changes for versions below
1.0.0, so^0.38.1resolves to>=0.38.1 <0.39.0.@seamapi/wizardis published at 0.42.0, which falls outside that range. Anyone runningnpx @seamapi/cli wizardgets a wizard four minor versions stale, and no warning is printed.Fix
Move the range to
^0.42.0and refresh the lockfile — the same shape as #637 (^0.28.0→^0.38.1).The dependency stays optional, which is deliberate: packaging sources that disallow non-commercial licenses (the Claude SDK) omit it, and
runWizardalready handles that with a dedicatedUsageError.Lockfile note for reviewers
The lockfile diff is deliberately minimal — 5 insertions, 5 deletions: the range, plus the wizard's
version/resolved/integrity. No other entry is added or removed.Two traps worth recording, since both silently produce a lockfile that breaks CI:
@seamapi:registry=https://npm.pkg.github.com/in~/.npmrc(common for Seam employees) gets a lockfile whose wizardresolvedURL points at GitHub Packages. CI'snpm cicannot authenticate there, and because the dependency is optional the failure is swallowed — surfacing later asTS2307: Cannot find module '@seamapi/wizard'rather than an install error. Regenerate with--@seamapi:registry=https://registry.npmjs.org/.packageManager(npm@11.19.0) drops the wizard's optional peer subtree —ink,react,@anthropic-ai/sdk,@modelcontextprotocol/sdk,express,hono— 11 entries in total.This branch was regenerated with
npm@11.19.0against the public registry, and its package set is identical tomain's.Verification
npm ciwith GitHub auth removed installs the wizard at 0.42.0 withink/reactpresent.npm run typecheck— clean, on Node 22 and 24 in CI.npm run lint— clean, Prettier included.Follow-up worth considering
Because a caret cannot cross a minor below
1.0.0, this range goes stale again on the wizard's next minor release. Options: widen to>=0.38.1 <1, or add an explicit Dependabot rule so the bump is automated rather than manual.🤖 Generated with Claude Code