Repository navigation
fix(git): upgrade just-git to 1.10.0 - #265
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
There was a problem hiding this comment.
No issues found across 5 files
You’re at about 90% of the monthly reviewed-line limit. You may want to disable incremental reviews to conserve quota. Reviews will continue until that limit is exceeded. If you need help avoiding interruptions, please contact contact@cubic.dev.
Tip: Get a Slack DM when cubic finishes reviewing your PRs. Connect Slack
View guided diff | Turn on auto-fix | Re-trigger cubic
Why
just-git 1.10.0 includes the four upstream fixes carried by #256. Use the published release rather than vendoring its bundled JavaScript. This PR supersedes #256.
What changed
^1.10.0and resolve the lockfile to 1.10.0.ls-tree, and--summary.Scope
Blast Radius
The upgrade affects sandbox git commands. Existing unit, integration, HTTP/MCP, and live SDK checks pass with the published package.
Verification
pnpm checkpassed, including 2,313 core tests, 154 TypeScript SDK tests, and 174 Python SDK tests. The eight preserved regressions also passed directly.pnpm verify:localpassed with 265 integration tests, 92 HTTP/MCP contract checks, and 102 live SDK checks.pnpm lint:fixpassed.pnpm security:auditpassed its high-severity threshold and reports one existing moderate sprintf-js advisory, GHSA-hp3w-g68c-fv3c.Summary by cubic
Upgrades
just-gitfrom^1.9.1to^1.10.0to pick up upstream fixes for mode-only changes,ls-tree, and--summaryoutput instead of vendoring the fix bundle from #256.src/api/tests/unit/git-command.regressions.test.ts.src/api/commands/git-command.ts; wrapper mode repair and index-mode handling are unchanged.sql-fs-api.Verification
pnpm checkpassed: 2,313 core tests, 154 TypeScript SDK tests, 174 Python SDK tests, plus the eight new regressions.pnpm verify:localpassed: 265 integration tests, 92 HTTP/MCP contract checks, 102 live SDK checks.pnpm lint:fixandpnpm security:auditpassed; audit reports one existing moderatesprintf-jsadvisory.Written for commit 679a06c. Summary will update on new commits.