Skip to content

Update vulnerable npm lockfile dependencies#1415

Merged
stephentoub merged 1 commit into
mainfrom
stephentoub/fix-vulnerabilities
May 25, 2026
Merged

Update vulnerable npm lockfile dependencies#1415
stephentoub merged 1 commit into
mainfrom
stephentoub/fix-vulnerabilities

Conversation

@stephentoub
Copy link
Copy Markdown
Collaborator

This updates the npm lockfiles flagged by Dependabot so the resolved transitive dependencies move to patched versions. The existing package ranges already allow the safe versions, so no manifest dependency ranges needed to change.

Summary

  • Refresh nodejs/package-lock.json to resolve patched lodash and vite versions.
  • Refresh test/harness/package-lock.json to resolve patched @hono/node-server, fast-uri, hono, qs, and vite versions.
  • Keep the change lockfile-only to minimize package surface area.

Validation

  • npm audit --omit=optional --audit-level=low in nodejs
  • npm audit --omit=optional --audit-level=low in test/harness
  • npm run typecheck in nodejs
  • npm test in test/harness

Note: a full local nodejs test run hit an unrelated E2E hang/failure path and generated snapshot diffs, which were reverted.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 25, 2026 14:21
@stephentoub stephentoub requested a review from a team as a code owner May 25, 2026 14:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (2)
  • nodejs/package-lock.json: Language not supported
  • test/harness/package-lock.json: Language not supported

@github-actions
Copy link
Copy Markdown
Contributor

Cross-SDK Consistency Review ✅

This PR only updates npm lockfile files (nodejs/package-lock.json and test/harness/package-lock.json) to resolve security vulnerabilities in transitive dependencies. No SDK API changes were made, so there are no cross-SDK consistency concerns.

These lockfile updates are Node.js/npm-specific artifacts with no equivalent in the Python, Go, .NET, or Java SDKs.

Generated by SDK Consistency Review Agent for issue #1415 · ● 875.4K ·

@stephentoub stephentoub merged commit 89fd30c into main May 25, 2026
42 of 44 checks passed
@stephentoub stephentoub deleted the stephentoub/fix-vulnerabilities branch May 25, 2026 15:03
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