Skip to content

Audit production dependencies in pull requests - #308

Merged
alecdotdev merged 12 commits into
sftwrdotdev:masterfrom
PathGao:ci/enforce-production-dependency-audit
Aug 1, 2026
Merged

Audit production dependencies in pull requests#308
alecdotdev merged 12 commits into
sftwrdotdev:masterfrom
PathGao:ci/enforce-production-dependency-audit

Conversation

@PathGao

@PathGao PathGao commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Fixes #307

Adds npm audit --omit=dev directly after the locked dependency install in the PR test workflow. This rejects known runtime dependency vulnerabilities while deliberately excluding development-only findings.

The workflow behavior is covered by a regression test.

Depends on #306. Merge order: #286#287#288#293#296#298#300#301#302#304#306 → this PR.

Validated with npm ci, npm run check, npm test (136 tests), npm audit --omit=dev (0 vulnerabilities), and cargo test (21 Rust tests).

@PathGao

PathGao commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator Author

@alecdotdev This follows #306 and turns its production audit result into a PR gate. It deliberately audits only runtime dependencies, so development-tooling advisories do not block application changes. Please merge after #306.

@alecdotdev
alecdotdev merged commit 2ae2f44 into sftwrdotdev:master Aug 1, 2026
5 checks passed
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.

CI does not enforce the production dependency audit

2 participants