Skip to content

fix: resolve 3 high-severity dependency vulnerabilities (sharp, js-yaml, postcss) - #51

Merged
farizanjum merged 1 commit into
mainfrom
fix/dep-vulnerabilities-sharp-js-yaml-postcss
Jul 31, 2026
Merged

fix: resolve 3 high-severity dependency vulnerabilities (sharp, js-yaml, postcss)#51
farizanjum merged 1 commit into
mainfrom
fix/dep-vulnerabilities-sharp-js-yaml-postcss

Conversation

@farizanjum

@farizanjum farizanjum commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes remaining high-severity postcss vulnerability reported by pnpm audit. After rebasing on latest main (which already fixed sharp and js-yaml), the only remaining fix is the postcss override range.

Vulnerability Fixed

Package CVE Vulnerable Range Fix
postcss GHSA-r28c-9q8g-f849 (path traversal in sourceMappingURL auto-loading) <=8.5.17 Bump override ^8.5.10 -> ^8.5.18

Already fixed upstream (commit d705ba6)

  • sharp: bumped to ^0.35.3 (CVE-2026-33327/33328/35590/35591)
  • js-yaml: gray-matter replaced with yaml ^2.9.0, js-yaml override removed (GHSA-pm4m-ph32-ghv5)

Changes

pnpm-lock.yaml: postcss 8.5.23 -> 8.5.25 (lockfile reflects the ^8.5.18 override range floor)

Verification

  • pnpm audit: No known vulnerabilities found (0 issues)
  • pnpm test --run: 22 passed, 13 skipped (e2e suite pre-existing failure on main - missing root build script, unrelated to this change)
  • tsx scripts/validate-pr.ts: All 64 skills valid
  • tsx scripts/validate-skill-descriptions.ts: OK - all SKILL.md descriptions populated
  • pnpm build:plugin: Plugin generated successfully (64 skills)

Testing

pnpm install
pnpm audit              # 0 vulnerabilities
pnpm test -- --run      # 22 pass, 13 skip (e2e pre-existing)
pnpm build:plugin       # OK
npx tsx scripts/validate-pr.ts                  # All 64 skills valid
npx tsx scripts/validate-skill-descriptions.ts  # OK

Copilot AI review requested due to automatic review settings July 31, 2026 08:18

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

…ml, postcss)

- sharp: bump range ^0.34.5 -> ^0.35.0 (CVE-2026-33327/33328/35590/35591)
- postcss: bump override ^8.5.10 -> ^8.5.18 (path traversal in sourceMappingURL)
- js-yaml: constrain override >=3.15.0 -> >=3.15.0 <4.0.0 (DoS in flow collections)
  The previous override >=3.15.0 was too broad, resolving to 5.x which
  removed safeLoad() and broke gray-matter. Constrained to 3.x which is
  not in the vulnerable range (CVE only affects >=5.0.0 <=5.2.1).
@farizanjum
farizanjum force-pushed the fix/dep-vulnerabilities-sharp-js-yaml-postcss branch from 5b487b1 to b06e078 Compare July 31, 2026 08:21
@farizanjum
farizanjum merged commit bfaae54 into main Jul 31, 2026
2 checks passed
@farizanjum
farizanjum deleted the fix/dep-vulnerabilities-sharp-js-yaml-postcss branch July 31, 2026 08:27
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