Skip to content

fix: upgrade picomatch to 4.0.4, 3.0.2, 2.3.2 (CVE-2026-33671)#114940

Open
orbisai0security wants to merge 1 commit intogetsentry:masterfrom
orbisai0security:fix-cve-2026-33671-picomatch
Open

fix: upgrade picomatch to 4.0.4, 3.0.2, 2.3.2 (CVE-2026-33671)#114940
orbisai0security wants to merge 1 commit intogetsentry:masterfrom
orbisai0security:fix-cve-2026-33671-picomatch

Conversation

@orbisai0security
Copy link
Copy Markdown

Summary

Upgrade picomatch from 2.3.1 to 4.0.4, 3.0.2, 2.3.2 to fix CVE-2026-33671.

Vulnerability

Field Value
ID CVE-2026-33671
Severity HIGH
Scanner trivy
Rule CVE-2026-33671
File api-docs/pnpm-lock.yaml

Description: picomatch: Picomatch: Regular Expression Denial of Service via crafted extglob patterns

Changes

  • api-docs/package.json
  • api-docs/pnpm-lock.yaml

Verification

  • Build passes
  • Scanner re-scan confirms fix
  • LLM code review passed

Automated security fix by OrbisAI Security

Automated dependency upgrade by Orbis Security AI
@orbisai0security orbisai0security requested a review from a team as a code owner May 6, 2026 04:49
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 802e509. Configure here.

Comment thread api-docs/package.json
"json-diff": "^0.7.1",
"json-refs": "^3.0.15",
"openapi-examples-validator": "^6.0.2",
"picomatch": "2.3.2",
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.

Phantom direct dependency instead of pnpm override

Low Severity

picomatch is added as a direct production dependency, but no source file in api-docs actually imports or requires it — it's only a transitive dependency of anymatch and micromatch. The project already has a pnpm.overrides section used for exactly this purpose (e.g., lodash, form-data). Adding it to dependencies instead of pnpm.overrides is misleading and inconsistent with the existing pattern, and risks someone later removing it as "unused."

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 802e509. Configure here.

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.

1 participant