Skip to content

Ci/npm audit fix - #165

Merged
aneshlya merged 3 commits into
mainfrom
ci/npm-audit-fix
Oct 7, 2026
Merged

aneshlya merged 3 commits into
mainfrom
ci/npm-audit-fix

Conversation

@aneshlya

@aneshlya aneshlya commented Oct 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

aneshlya and others added 3 commits October 7, 2026 13:39
Runs weekly (and on demand) in client-vscode, applies in-range fixes
via npm audit fix, verifies install and compile, and opens a PR for
vulnerabilities Dependabot did not update.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Fail on invalid or errored npm audit reports instead of skipping
- Guard jq against missing data; fall back to 'via' for transitive entries
- Open a PR only when the vulnerability count drops
- Split into a read-only audit/verify job and a PR job that runs no
  project code; no persisted checkout credentials
- Create the PR with AUDIT_FIX_TOKEN so build.yml runs on it
- Surface npm audit fix errors, queue runs instead of cancelling,
  fence audit output in the step summary

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aneshlya
aneshlya merged commit 4725a3c into main Oct 7, 2026
6 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.

1 participant