Skip to content

build(deps): bump @fastify/swagger from 9.6.1 to 9.7.0#7535

Merged
n-lark merged 2 commits into
mainfrom
dependabot/npm_and_yarn/fastify/swagger-9.7.0
Jun 17, 2026
Merged

build(deps): bump @fastify/swagger from 9.6.1 to 9.7.0#7535
n-lark merged 2 commits into
mainfrom
dependabot/npm_and_yarn/fastify/swagger-9.7.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jun 16, 2026

Copy link
Copy Markdown
Contributor

Bumps @fastify/swagger from 9.6.1 to 9.7.0.

Release notes

Sourced from @​fastify/swagger's releases.

v9.7.0

What's Changed

New Contributors

Full Changelog: fastify/fastify-swagger@v9.6.1...v9.7.0

Commits
  • 4e44d70 Bumped v9.7.0
  • ab6c5df fix(openapi): keep required query flag for params (#910)
  • 99b9b43 chore: ignore local AI assistant files (#909)
  • a65d96a fix(openapi): always set requestBody.required to true when schema.body exists...
  • d35da58 chore: remove tests-checker workflow (#908)
  • efcb074 chore(deps-dev): bump @​types/node from 24.10.4 to 25.0.3 (#907)
  • 746069d chore: faster checks (#905)
  • See full diff in compare view

@dependabot dependabot Bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Jun 16, 2026
@hardillb

Copy link
Copy Markdown
Contributor

@n-lark It looks like this change updates the typescript definitions. Is the correct thing to do, pull this branch and run the update script manually and push back?

@hardillb

Copy link
Copy Markdown
Contributor

Having just pulled this

  • I deleted local node_modules
  • ran npm ci to ensure it used the version this updates
  • ran npm run build
  • ran npm run generate:types

But there were no file changes, so I'm not sure how this test failed?

(only difference is I ran everything locally with NodeJS 20 not 24)

@dependabot dependabot Bot force-pushed the dependabot/npm_and_yarn/fastify/swagger-9.7.0 branch from 57b5db2 to 4598f87 Compare June 16, 2026 14:26
@hardillb

Copy link
Copy Markdown
Contributor

@dependabot rebase

Bumps [@fastify/swagger](https://github.com/fastify/fastify-swagger) from 9.6.1 to 9.7.0.
- [Release notes](https://github.com/fastify/fastify-swagger/releases)
- [Commits](fastify/fastify-swagger@v9.6.1...v9.7.0)

---
updated-dependencies:
- dependency-name: "@fastify/swagger"
  dependency-version: 9.7.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot force-pushed the dependabot/npm_and_yarn/fastify/swagger-9.7.0 branch from 4598f87 to c8cdab7 Compare June 16, 2026 15:50
@n-lark

n-lark commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Hey @hardillb interesting - I would have run through the same steps as you did so we probably need to just allocate some time to poke at this more. I can make a ticket under Resolve Stale Dependabot PRs #7139 if you'd like.

@hardillb

Copy link
Copy Markdown
Contributor

@n-lark I have opened one, we can try and find (hopefully) 10mins at your convenience to poke at this together.

@n-lark n-lark left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Dependabot triage ✅

  • Package: @fastify/swagger 9.6.19.7.0 (minor)
  • Publish window: version published 2026-02-07T11:12:05Z, ≥12h elapsed ✅
  • Advisories: no open npm advisory or GitHub advisory against 9.7.0
  • Release notes: OpenAPI fixes (#903 requestBody.required, #910 query required flag) — no breaking changes affecting our usage ✅
  • CI: green ✅
  • Decision: approving for merge — minor bump; OpenAPI output changes required regenerating frontend/src/types/generated.ts (committed in 87a994e), diff is flag-flips only

@n-lark

n-lark commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Hey @hardillb so I ran npm run generate:types with a fresh npm ci and node 24 and got a diff that was matching the failures we saw in the ci. As long as the tests pass I think we are good to merge this.

Previously fastify-swagger emitted requestBody without required: true, so openapi-typescript generated it as an optional property (requestBody?). In 9.7.0, any route with a schema.body now gets requestBody.required: true in the OpenAPI spec, so the generator emits it as a required property. Every flip is a route in our API that defines a body schema, 49 of them.

@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.88%. Comparing base (6651f18) to head (87a994e).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7535   +/-   ##
=======================================
  Coverage   76.88%   76.88%           
=======================================
  Files         410      410           
  Lines       20939    20939           
  Branches     5098     5098           
=======================================
  Hits        16099    16099           
  Misses       4840     4840           
Flag Coverage Δ
backend 76.88% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@n-lark n-lark merged commit a5aebfe into main Jun 17, 2026
43 of 47 checks passed
@n-lark n-lark deleted the dependabot/npm_and_yarn/fastify/swagger-9.7.0 branch June 17, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants