Skip to content

build: configure dependabot and resolve open vulnerabilities - #459

Merged
ilteoood merged 1 commit into
mainfrom
chore/dependabot-config-and-vuln-fixes
Sep 6, 2026
Merged

build: configure dependabot and resolve open vulnerabilities#459
ilteoood merged 1 commit into
mainfrom
chore/dependabot-config-and-vuln-fixes

Conversation

@ilteoood

@ilteoood ilteoood commented Sep 6, 2026

Copy link
Copy Markdown
Member

What's in here

  • Adds .github/dependabot.yml matching the pattern used across the fastify org: weekly schedule, github-actions ecosystem for workflow bumps, and npm ecosystem covering the pnpm monorepo via directories: ['/'] with separate prod/dev grouping.
  • Resolves every open Dependabot advisory (69 → 0).

Approach

The vulns are almost all transitive — pulled in by direct deps that already had loose caret ranges — so the fix is to bump those direct deps rather than pin patches in pnpm.overrides:

Bumped dep What it pulls patched
fastify catalog ^5.8.5^5.12.1 fastify itself, fast-uri
@fastify/middie catalog ^9.3.2^9.3.4 find-my-way
vitest catalog ^4.1.9^5.0.0 jsdomform-data
@unhead/vue catalog ^3.0.0^3.4.0 valibot chain
eslint catalog ^10.2.0^10.10.0 @humanfs/node
eslint-plugin-vue ^10.8.0^10.11.0 postcss-selector-parser
react-router catalog ^7.15.1^7.18.2 react-router
@changesets/cli ^2.31.0^3.0.2 js-yaml
undici ^7.24.7^7.29.0 in e2e/react-next-mini undici
postcss ^8.5.3^8.5.23 in starters postcss, nanoid
postcss-preset-env ^10.1.5^11.5.1 in starters/react-base browserslist
mermaid ^11.15.0^11.17.2 in docs dompurify, uuid
fastify, postcss, @unhead/vue bumped in starters' package.json too (same)

pnpm.overrides (kept to a minimum)

Only three entries remain, two of which were pre-existing:

  • vite: '>=6.4.3' — new. Vitepress 1.6.4 pins vite 5.x; verified the docs site builds cleanly with vite 8.1.5 under this override.
  • @xmldom/xmldom: '>=0.9.12' — pre-existing. Transitive via mathjax-full / speech-rule-engine in docs; no direct dep to bump.
  • esbuild: '>=0.28.1' — pre-existing. Dev-only dep, kept.

Verification

  • pnpm auditNo known vulnerabilities found.
  • pnpm format:check → clean.
  • pnpm lint → no errors (warnings only, all pre-existing).
  • pnpm test → 103/103 passing across fastify-vite (86), fastify-react (0 tests via node --test, suite passes), fastify-vue (17).
  • pnpm build → both fastify-vite and fastify-vue compile.
  • vitepress build (docs site) → builds cleanly under the new vite override.

@changeset-bot

changeset-bot Bot commented Sep 6, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 9765e6b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ilteoood
ilteoood requested review from Eomm and Tony133 September 6, 2026 13:17

@Tony133 Tony133 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Adds a .github/dependabot.yml matching the pattern used across the
fastify org (weekly schedule, pnpm monorepo via directories, prod/dev
dependency groups, github-actions grouping).

Resolves every open Dependabot advisory (69 → 0) by bumping the direct
dependencies that pull them in, instead of pinning patches through
pnpm.overrides:

- fastify catalog ^5.8.5 → ^5.12.1
- @fastify/middie catalog ^9.3.2 → ^9.3.4 (pulls patched find-my-way)
- vitest catalog ^4.1.9 → ^5.0.0 (pulls patched jsdom/form-data)
- @unhead/vue catalog ^3.0.0 → ^3.4.0 (pulls patched valibot chain)
- eslint catalog ^10.2.0 → ^10.10.0 (pulls patched @humanfs/node)
- eslint-plugin-vue ^10.8.0 → ^10.11.0 (pulls patched
  postcss-selector-parser)
- react-router catalog ^7.15.1 → ^7.18.2
- @changesets/cli ^2.31.0 → ^3.0.2 (pulls patched js-yaml)
- undici ^7.24.7 → ^7.29.0 in e2e/react-next-mini
- postcss ^8.5.3 → ^8.5.23 in starters (pulls patched nanoid)
- postcss-preset-env ^10.1.5 → ^11.5.1 in starters/react-base
  (pulls patched browserslist)
- mermaid ^11.15.0 → ^11.17.2 in docs (pulls patched dompurify/uuid)
- fastify, postcss, @unhead/vue bumped in starters' package.json too

Only three pnpm.overrides remain (all pre-existing except vite):

- vite: '>=6.4.3' — vitepress 1.6.4 pins vite 5.x and is incompatible
  with vite 6+ peer ranges without the override; verified the docs
  site builds cleanly with vite 8.1.5.
- @xmldom/xmldom: '>=0.9.12' — already present, kept (transitive via
  mathjax-full/speech-rule-engine in docs, no direct dep to bump).
- esbuild: '>=0.28.1' — already present, kept.

Co-authored-by: Matteo Pietro Dazzi <matteo@iltteoood.com>
@ilteoood
ilteoood force-pushed the chore/dependabot-config-and-vuln-fixes branch from c254914 to 9765e6b Compare September 6, 2026 13:33
@ilteoood
ilteoood merged commit 70b98bf into main Sep 6, 2026
7 checks passed
@ilteoood
ilteoood deleted the chore/dependabot-config-and-vuln-fixes branch September 6, 2026 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants