Skip to content

Audit production dependencies in pull requests - #308

Open
PathGao wants to merge 12 commits into
alecdotdev:masterfrom
PathGao:ci/enforce-production-dependency-audit
Open

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

Conversation

@PathGao

@PathGao PathGao commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

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
Contributor 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.

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

1 participant