chore(deps): update execa 10, esbuild 0.28, vitest 5 and TypeScript 7 - #7
Merged
Merged
Conversation
zod stays on 3.x: zod 4 made each bundled hook about 5x larger and added 10-20 ms to every hook start, for no functional gain in Junto's schemas. The Windows fake reviewer now forwards %* to a Node script like the shims npm installs, because execa 10 quotes every argument passed to .cmd files and the old batch-only fake parsed %1 itself.
TypeScript 7 no longer includes @types packages implicitly, so the base config lists node types explicitly. A full forced build of all project references takes about 1.2 s.
Merged
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
1 participant
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.
Updates runtime and build dependencies after the Node.js 22.12 baseline made them installable:
execa9.6 → 10.0 (runtime, bundled)esbuild0.24 → 0.28,vitest3 → 5 andtypescript5.9 → 7.0 (dev)zodfloor raised to^3.25.76; zod 4 was evaluated and deliberately not adoptedWhy not zod 4
It type-checks and passes all tests, but it bloats every bundled hook, and hooks run on every Write/Edit (the guard twice):
Junto only uses zod for internal schema validation, so zod 4 has no functional benefit here.
execa 10 and the Windows test fake
execa 10 wraps every argument in quotes when it runs a
.cmdfile. npm shims forward%*unchanged, so realocr/pnpmshims receive identical argv under execa 9 and 10 (verified with an npm-style shim). The test's batch-only fake parsed%~1itself and broke; it now forwards%*to a Node script, the same shape as the shims npm installs.TypeScript 7
TypeScript 7 no longer includes
@typespackages implicitly, sotsconfig.base.jsonlists"types": ["node"]. No source changes were needed; a forced build of all project references takes about 1.2 s. Nothing in the repository uses thetypescriptJS API.Verification
tsc -b --forcewith TypeScript 7.0.2 — 0 errorscorepack pnpm test— 378 passed, 4 platform/optional skipscorepack pnpm build— bundles rebuilt; sizes unchanged (guard 136 KB, server ~1.0 MB); the bundled MCP server answersinitializeas 0.6.0