chore(deps): bump @inquirer/core to 12.0.0 and @changesets/cli to 3.0.3 - #1954
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: Fission-AI/OpenSpec/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe package manifest updates ChangesDependency update
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to No merge-blocking issue was confirmed; run the normal Nix validation before release. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Out of Scope Changes checkExplanation The whole-PR diff directly changes
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
No PR-relevant drift confirmed.
|
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
alfred-openspec
left a comment
There was a problem hiding this comment.
Reviewed the dependency and lockfile changes. Node engine compatibility is preserved, the Nix hash is updated, and the full cross-platform, security, and Nix checks pass.
Replaces Dependabot #1930 and #1931 with one PR.
Status
Not yet — the
flake.nixpnpmDepshash still needs its real value (see Notes). Everything else is done and verified.Why this is needed
Both Dependabot PRs rewrite the same
flake.nixpnpmDepshash, so whichever lands first makes the other stale. Dependabot also discards commits pushed to its own branches on rebase, so the hash cannot be fixed in place. Consolidating means the hash is computed once against the final lockfile.What it does
@inquirer/core(#1931)^11.2.1^12.0.0package.json+ lockfile@changesets/cli(#1930)^3.0.1, matching #1930)Proof it works
artifact-workflow"creates skills for Cursor tool" andconfig-profile"confirmed project apply…") fail identically on cleanmain— verified by running them onorigin/mainwith a fresh install. Pre-existing, unrelated.@inquirer/coremajor was previously rejected in chore(deps): bump @inquirer/core from 10.3.2 to 11.2.1 #1422 because@inquirer/prompts@7bundled core 10 and the Windows init prompt test hung. That no longer applies: the repo is on@inquirer/prompts@8, and chore(deps): bump @inquirer/core from 11.2.1 to 12.0.0 #1931's CI passed on linux, macOS and Windows.brace-expansion,postcss,js-yaml(x2),nanoid,fflateare all still in the lockfile'soverridesblock, andtest/pnpm-workspace-config.test.tspasses.Notes
pnpmDepshash inflake.nixis still the old value. No nix or running docker daemon locally, so Nix Flake Validation will fail once and report the realgot:hash, which then lands as a one-line fix.pnpm@10.34.5(the pinnedpackageManager) for any lockfile work here.pnpm@9silently drops theoverridesblock, because the overrides live inpnpm-workspace.yaml, which only pnpm 10+ reads.🤖 Generated with Claude Code
Summary by CodeRabbit
@inquirer/coredependency to version 12.0.0.