Skip to content

ci: add pnpm audit job - #4010

Merged
amilz merged 1 commit into
mainfrom
ci/TOO-1010-pnpm-audit
Oct 26, 2026
Merged

amilz merged 1 commit into
mainfrom
ci/TOO-1010-pnpm-audit

Conversation

@amilz

@amilz amilz commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator
  • Add an audit job to CI that runs pnpm audit against the lockfile. It has no install step, so it finishes quickly.
  • If an advisory has no fix yet, add its GHSA to auditConfig.ignoreGhsas in pnpm-workspace.yaml with a link to the upstream issue.
  • AI tooling was used. Tool and extent: Claude Code ran the dependency audit, wrote the CI job and drafted this PR; I reviewed it.
  • Closes TOO-1010

@amilz amilz self-assigned this Oct 26, 2026
@linear

linear Bot commented Oct 26, 2026

Copy link
Copy Markdown

TOO-1010

Refs: TOO-1010
@amilz
amilz force-pushed the ci/TOO-1010-pnpm-audit branch from 4019cc7 to 881f8c5 Compare October 26, 2026 22:02
@greptile-apps

greptile-apps Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 4/5

[Low impact] The change reaches the repository's CI workflow, not application behavior.

Set the promised high-only audit threshold before merging.

Findings

  1. P1 Lower-severity advisories fail CI ▶

Summary

Adds a standalone dependency audit job to CI.

  • CI runs a dependency audit in its own job.

Reviews (1) · Last reviewed commit: "ci: add pnpm audit job" · Reviewed by Greptile

Comment thread .github/workflows/ci.yml Resolved

@dev-jodee dev-jodee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Adding pnpm audit job to CI. All checks pass.

@amilz
amilz merged commit d675f6f into main Oct 26, 2026
14 checks passed
@amilz
amilz deleted the ci/TOO-1010-pnpm-audit branch October 26, 2026 22:17
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