chore(deps): refresh lockfile within existing ranges - #142
Merged
Conversation
Routine dependency refresh, replacing the bot PRs that #127 put into conflict (#131 from Dependabot, and the npm half of #128 from Renovate). Doing it directly avoids a rebase round-trip against branches Dependabot auto-closes on external pushes, and it lands the same bumps deterministically. Lockfile only -- server/package.json is untouched. Every change is a version the existing caret ranges already authorized, so this is a re-resolution rather than a policy change. The Node pin is unaffected: @types/node stays 24.13.3 and engines.node stays ">=24 <25". Notable direct bumps: fastify 5.8.5 -> 5.11.0, @fastify/cors 11.2.0 -> 11.3.0, @fastify/helmet 13.0.2 -> 13.1.0, better-sqlite3 12.10.0 -> 12.11.1, bonjour-service 1.3.0 -> 1.4.4, @playwright/test 1.60.0 -> 1.62.1, vitest and @vitest/coverage-v8 4.1.6 -> 4.1.10, fast-check 4.8.0 -> 4.9.0, tsx 4.22.4 -> 4.23.1. 100 packages moved in total, which is wider than the direct list. Three entries look alarming and were each traced before committing; all are upstream decisions reached through in-range bumps, not choices made here: @emnapi/core 1.11.1 -> 2.0.0-alpha.3 pinned exactly by @rolldown/binding-wasm32-wasi. An optional wasm32-wasi platform binding, not installed or loaded on the platforms this project builds for. fastify-plugin 5.1.0 -> 6.0.0 required at ^6.0.0 by @fastify/cors, fast-json-stringify 6.3.0 -> 7.0.1 @fastify/helmet, @fastify/static and fastify itself after their minor bumps. fsevents 2.3.3 -> 2.3.2 darwin-only optional dependency, pinned exactly elsewhere in the tree. Verified under node:24, with a boot check rather than tests alone since fastify moved three minors and fastify-plugin went major: npm ci clean, check:node-pin consistent, typecheck + typecheck:scripts + build exit 0, audit reports 0 vulnerabilities, 1670 tests pass (11 skipped). Server boots with no plugin registration errors and serves /, /js/app.js, /health and /fixtures as 200 with traversal still 404. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
thewrz
marked this pull request as ready for review
August 2, 2026 05:47
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.
This was written agentically; verify its assertions and edit accordingly:
Why
#127 put both routine dependency PRs into conflict — #131 (Dependabot's npm minor/patch group) and the npm half of #128 (Renovate). Rather than run a rebase round-trip against branches Dependabot auto-closes on external pushes, this lands the same bumps directly and deterministically.
What
Lockfile only —
server/package.jsonis untouched. Every change is a version the existing caret ranges already authorized, so this is a re-resolution, not a policy change. The Node pin is unaffected:@types/nodestays24.13.3,engines.nodestays">=24 <25", andcheck:node-pinpasses.Notable direct bumps:
fastify@fastify/cors@fastify/helmetbetter-sqlite3bonjour-service@playwright/testvitest,@vitest/coverage-v8fast-checktsxThree entries that look alarming in the diff
100 packages moved, which is wider than the direct list. Three deserve a callout — each was traced before committing, and all are upstream decisions reached through in-range bumps, not choices made here:
@emnapi/core1.11.1 →2.0.0-alpha.3— an alpha, but pinned exactly by@rolldown/binding-wasm32-wasi("@emnapi/core": "2.0.0-alpha.3"). It is an optionalwasm32-wasiplatform binding, not installed or loaded on the platforms this project builds for.fastify-plugin5.1.0 → 6.0.0 andfast-json-stringify6.3.0 → 7.0.1 — transitive majors, required at^6.0.0/^7.0.0by@fastify/cors,@fastify/helmet,@fastify/staticandfastifyitself after their minor bumps.fsevents2.3.3 → 2.3.2 — a downgrade, butfseventsis a darwin-only optional dependency pinned exactly elsewhere in the tree.Testing
Verified in a real
node:24container. Becausefastifymoved three minors andfastify-pluginwent major, this was booted, not just unit-tested:npm ci— clean, lockfile in syncnpm run check:node-pin— consistent, Node 24 across 4 declarationsnpm run typecheckandnpm run typecheck:scripts— exit 0npm run build— exit 0npm audit --audit-level=high— 0 vulnerabilities/,/js/app.js,/health,/fixturesall 200; path traversal still 404Supersedes
Closes the npm side of the bot backlog: #131 and the npm portion of #128 (already closed). #123/#137 (GitHub Actions bumps) are separate and not covered here.
🤖 Co-authored by Claude Opus 5 (1M context).