chore(deps): bump tsx to 4.23.13 to silence Node 26 DEP0205 - #2379
Conversation
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Warning Review limit reachedNext included review available in 16 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
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 |
|
@coderabbitai review |
|
Which Linear task belongs to this PR?
None — dependency bump, covered by the
trivialcarve-out in the ticket-linkage metric (.agents/context.md).Why did I implement it this way?
Running any
bunx tsxscript on Node 26 prints:The call is not ours —
git grep module.registeracross the repo returns nothing. It comes fromtsx4.21.0's ESM loader; Node 26 deprecatedmodule.register()as DEP0205.tsx4.23.13 no longer uses it, so the warning is gone.Blast radius is local only. CI never saw this warning: the workflows run under bun, and the two that use Node pin
node-version: 20(jsonChecker.yml,updateLouperViem.yml), where DEP0205 does not exist. This is purely noise reduction for local script runs and agent sessions.Transitive change:
tsxwidened itsesbuildrange from~0.27.0to~0.28.0, so the lockfile also movesesbuild0.27.7 → 0.28.2 plus its 26 platform-specific optional packages. Nothing else inbun.lockchanges — the diff is exactlytsx,esbuild, and those 26@esbuild/*entries.Verification
tsx4.21.0 and confirmed its absence on 4.23.13, same script, same Node 26.7.0.script/utils/deploymentHelpers.ts(pulls in viem) under both versions — identical behaviour, no transpile drift.tasks/buildClearSigningProposal.ts) under 4.23.13; it executed and hit its ownforge buildprecondition guard as expected.bun install --lockfile-onlyso the sharednode_modulessymlink was left untouched.Checklist before requesting a review
Checklist for reviewer (DO NOT DEPLOY and contracts BEFORE CHECKING THIS!!!)
🤖 Generated with Claude Code