Skip to content

fix(deps): resolve pnpm audit vulnerabilities#26

Merged
mcalthrop merged 1 commit intomainfrom
fix/security-vulnerabilities
Apr 17, 2026
Merged

fix(deps): resolve pnpm audit vulnerabilities#26
mcalthrop merged 1 commit intomainfrom
fix/security-vulnerabilities

Conversation

@mcalthrop
Copy link
Copy Markdown
Owner

@mcalthrop mcalthrop commented Apr 17, 2026

Summary

Fixes 2 vulnerabilities identified by pnpm audit, resolved by adding overrides to pnpm.overrides in package.json and running pnpm install.

Vulnerabilities fixed

Package Severity Vulnerable versions Resolution
protobufjs Critical <7.5.5 Override to >=7.5.5 via "protobufjs@<7.5.5": ">=7.5.5" in pnpm.overrides
dompurify Moderate <=3.3.3 Override to >=3.4.0 via "dompurify@<=3.3.3": ">=3.4.0" in pnpm.overrides

Both packages are transitive dependencies of packages__openapi > @redocly/cli.

Test plan

  • pnpm audit reports no known vulnerabilities after fix
  • CI passes

🤖 Generated with Claude Code

@mcalthrop mcalthrop marked this pull request as ready for review April 17, 2026 17:32
Copilot AI review requested due to automatic review settings April 17, 2026 17:32
Copy link
Copy Markdown

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 (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pnpm-lock.yaml
Comment thread pnpm-lock.yaml
@mcalthrop mcalthrop force-pushed the fix/security-vulnerabilities branch from a1f8acc to a47d0d9 Compare April 17, 2026 17:42
@mcalthrop mcalthrop merged commit e44c7e2 into main Apr 17, 2026
5 checks passed
@mcalthrop mcalthrop deleted the fix/security-vulnerabilities branch April 17, 2026 17:44
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