chore: dependency maintenance — minor/patch batch + @fastify/static v10#173
chore: dependency maintenance — minor/patch batch + @fastify/static v10#173jaredwray wants to merge 3 commits into
Conversation
Update dependencies within existing semver ranges: Dependencies: - @fastify/cookie 11.0.2 -> 11.1.1 - @fastify/helmet 13.0.2 -> 13.1.0 - @fastify/rate-limit 11.0.0 -> 11.1.0 - @fastify/static 9.1.3 -> 9.3.0 - @fastify/swagger 9.7.0 -> 9.8.0 - @fastify/swagger-ui 6.0.0 -> 6.1.0 - @scalar/api-reference 1.59.3 -> 1.62.5 - fastify 5.8.5 -> 5.10.0 - hookified 3.0.0 -> 3.0.1 Dev dependencies: - @biomejs/biome 2.4.16 -> 2.5.3 - @swc/core 1.15.41 -> 1.15.43 - tsdown 0.22.2 -> 0.22.4 - tsx 4.22.4 -> 4.23.0 Verified with build + full test suite (473 tests passing, coverage unchanged). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NdfeGrhVzDqajGuC7eRo3R
Update @fastify/static from 9.3.0 to 10.1.0. The only breaking change in v10 is that the `setHeaders` option now receives a FastifyReply instead of a Node.js response object. This project does not use `setHeaders` (it only registers the plugin with `root`, `prefix`, and `decorateReply`), so no code changes are required. Verified with build + full test suite (473 tests passing, coverage unchanged). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NdfeGrhVzDqajGuC7eRo3R
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Code Review
This pull request updates several dependencies and devDependencies in package.json, including @biomejs/biome, @swc/core, tsdown, tsx, various @fastify ecosystem packages, @scalar/api-reference, fastify, hookified, and pino-pretty to their newer versions. There are no review comments, and I have no additional feedback to provide.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #173 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 41 41
Lines 1310 1310
Branches 259 261 +2
=========================================
Hits 1310 1310 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c15098773d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "rimraf": "^6.1.3", | ||
| "tsdown": "^0.22.2", | ||
| "tsx": "^4.22.4", | ||
| "tsdown": "^0.22.4", |
There was a problem hiding this comment.
Keep tsdown behind the release-age gate
This repo explicitly sets minimumReleaseAge: 10080 and minimumReleaseAgeStrict: true in pnpm-workspace.yaml, and pnpm 11 re-applies that supply-chain gate during installs. tsdown@0.22.4 was published only hours before this update, so a fresh install with the configured package manager can reject the lockfile or bypass the repo's 7-day maturity policy until the version ages; keep a mature 0.22.x version or add a deliberate exemption if this release must be used immediately.
Useful? React with 👍 / 👎.
| "rimraf": "^6.1.3", | ||
| "tsdown": "^0.22.2", | ||
| "tsx": "^4.22.4", | ||
| "tsdown": "^0.22.4", |
There was a problem hiding this comment.
Align the Node engine with tsdown
When users build or publish on Node 24.0 through 24.10, this package still advertises support via engines.node: >=22.18.0, but tsdown@0.22.4 declares support only for ^22.18.0 || >=24.11.0. That makes the build/prepublishOnly toolchain unsupported or rejected under engine-strict installs for a Node range the package claims is valid; either keep a tsdown version that supports all allowed Node 24 releases or tighten this package's engine range.
Useful? React with 👍 / 👎.
tsdown 0.22.4 narrows its supported Node range to `^22.18.0 || >=24.11.0`, dropping Node 24.0-24.10, while this package advertises `engines.node: >=22.18.0`. tsdown is a build-only devDependency and there is no engine-strict, so this only surfaces as a warning for contributors building on those Node versions -- but keeping tsdown at 0.22.2 (engines `^22.18.0 || >=24.0.0`) avoids narrowing the build toolchain's Node support surface. The two skipped patch releases carry negligible value for a build tool. Verified with build + test:ci (473 tests passing, coverage unchanged). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NdfeGrhVzDqajGuC7eRo3R
Please check if the PR fulfills these requirements
What kind of change does this PR introduce?
Chore — automated dependency maintenance. No source code changes.
Dependency Updates
Minor / patch (within existing semver ranges) — commit
9449fdcDependencies
Dev dependencies
Major — commit
c150987The only breaking change in
@fastify/staticv10 is that thesetHeadersoption now receives aFastifyReplyinstead of a Node.js response object. This project does not usesetHeaders(it only registers the plugin withroot,prefix, anddecorateReply), so no code changes were required. Kept as a separate commit so it can be reviewed/reverted independently of the minor/patch batch.Held back on purpose
19a7b7b)^22.18.0 || >=24.11.0, dropping Node 24.0–24.10, whereas this package advertisesengines.node: >=22.18.0. It is a build-only devDependency with noengine-strict, so it is only a warning — but holding at 0.22.2 (^22.18.0 || >=24.0.0) avoids narrowing the build toolchain's Node support surface for a negligible gain. (Addresses reviewer feedback on the initial push.)build+ full test suite pass, but the build toolchain warns: "TypeScript 7.0 does not yet have a stable API and is experimental. Some options will be unavailable." The dependency is also exact-pinned (no caret), signalling deliberate version control. Best handled in its own PR once TS 7 stabilizes.Verification
Verified with the CI-equivalent commands (
pnpm build+pnpm test:ci) after each change:Automated by the
project-maintenanceskill.