Skip to content

Fix critical npm audit and Snyk findings - #2625

Merged
ravikiranvm merged 2 commits into
mainfrom
OPS-4971
Oct 22, 2026
Merged

ravikiranvm merged 2 commits into
mainfrom
OPS-4971

Conversation

@ravikiranvm

@ravikiranvm ravikiranvm commented Oct 22, 2026 •

Copy link
Copy Markdown
Contributor

Fixes OPS-4971.

npm audit and snyk test on main reported 1 and 2 unique critical advisories respectively. All have patch-level fixes, so this PR takes them together.

Package From To Advisory
storybook, @storybook/addon-docs, @storybook/react-vite (dev) 10.6.1 10.6.3 GHSA-f2ph-cmg5-wxc8 (critical, origin validation in preview)
undici (via @ai-sdk/provider-utils 4.x) 6.29.0 6.29.1 SNYK-JS-UNDICI-21181005 (critical, certificate validation)
undici (via @assistant-ui/react-ai-sdk > @ai-sdk/mcp 2.x) 7.30.0 7.30.1 SNYK-JS-UNDICI-21181005, SNYK-JS-UNDICI-21181009 (critical)
follow-redirects (via axios) 1.16.0 1.16.1 GHSA-q3jh-wrc6-jgq8, GHSA-8r9p-f939-6h3c (high)
qs 6.16.0 6.16.1 SNYK-JS-QS-20943482 (high)
zod 4.6.5 4.6.6 SNYK-JS-ZOD-20510278 (high)

The undici and follow-redirects changes are lockfile-only: the parent packages already allow the patched versions via caret ranges.

Bump dependencies with patch-level security fixes:
- storybook, @storybook/addon-docs, @storybook/react-vite 10.6.1 -> 10.6.3
  (GHSA-f2ph-cmg5-wxc8, missing origin validation in preview)
- undici 6.29.0 -> 6.29.1 and 7.30.0 -> 7.30.1 lock refresh
  (SNYK-JS-UNDICI-21181005 certificate validation, SNYK-JS-UNDICI-21181009)
- follow-redirects 1.16.0 -> 1.16.1 (GHSA-q3jh-wrc6-jgq8, GHSA-8r9p-f939-6h3c)
- qs 6.16.0 -> 6.16.1 (SNYK-JS-QS-20943482)
- zod 4.6.5 -> 4.6.6 (SNYK-JS-ZOD-20510278)

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 22, 2026 06:49
@linear

linear Bot commented Oct 22, 2026

Copy link
Copy Markdown

OPS-4971

Copilot AI 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.

🟡 Changes recommended

The stale license inventory will cause the CI license check to fail.

1 open issue
What changed in this PR

Updates direct and transitive dependencies to address critical and high security advisories without changing application code.

Changes:

  • Upgrades Storybook packages to 10.6.3, qs to 6.16.1, and zod to 4.6.6.
  • Refreshes locked undici and follow-redirects versions.

Blocking: Regenerate and commit the license inventory to pass CI.
Non-blocking: None.
Merge recommendation: Do not merge

File Description
package.json Updates direct dependency pins to security-fix releases.
package-lock.json Records updated direct and transitive dependency resolutions.

Review effort: Balanced

Comment thread package.json
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@ravikiranvm
ravikiranvm requested a review from pawel-oo October 22, 2026 07:25

@pawel-oo pawel-oo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

@ravikiranvm
ravikiranvm merged commit 6066ad9 into main Oct 22, 2026
23 checks passed
@ravikiranvm
ravikiranvm deleted the OPS-4971 branch October 22, 2026 09:28
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.

3 participants