Skip to content

fix(deps): bump postcss to safe version in lock file (GHSA-r28c-9q8g-f849) - #57

Merged
ferhimedamine merged 1 commit into
mainfrom
fix/security-audit-postcss-esbuild
Jul 31, 2026
Merged

fix(deps): bump postcss to safe version in lock file (GHSA-r28c-9q8g-f849)#57
ferhimedamine merged 1 commit into
mainfrom
fix/security-audit-postcss-esbuild

Conversation

@ferhimedamine

Copy link
Copy Markdown
Contributor

Summary

  • postcss ≤8.5.17 has a HIGH severity path traversal vulnerability (GHSA-r28c-9q8g-f849). It's a transitive dep — package.json unchanged, only package-lock.json updated via npm audit fix --package-lock-only.
  • This is the same CVE fixed in dakera-ai-sdk via PR#21.
  • esbuild LOW severity advisory (GHSA-g7r4-m6w7-qqqr) also remains but does NOT fail the --audit-level=high CI gate (exit 0 verified locally).

Why now

Dependabot PRs #54/#55/#56 all fail the security audit CI step on this CVE. Once this PR merges to main, a @dependabot rebase on each will pick up the fix and turn them green.

Test plan

…f849)

postcss ≤8.5.17 has a HIGH severity path traversal vulnerability via
sourceMappingURL auto-loading. Fixed via `npm audit fix --package-lock-only`
which updates the transitive dep in package-lock.json only.

Unblocks dependabot PRs #54/#55/#56 which all fail the `npm audit --audit-level=high` CI gate.

Co-Authored-By: Paperclip <noreply@paperclip.ing>
@ferhimedamine ferhimedamine added the auto-merge Auto-merge when CI passes label Jul 30, 2026
@ferhimedamine

Copy link
Copy Markdown
Contributor Author

Merge blocked — needs non-ferhimedamine review (same as #53)

This is a pure security fix (package-lock.json only, automated npm audit fix --package-lock-only). Once CI passes (running now), it just needs 1 approving review from a non-ferhimedamine account.

After this merges: @dependabot rebase on #54, #55, #56 → all three will turn green (both the security audit and the vectorWeight code fix will be on base).

@ferhimedamine
ferhimedamine merged commit 90876b0 into main Jul 31, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Auto-merge when CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant