Skip to content

replace retired pnpm audit with OSV-Scanner gate - #69

Merged
jeesunikim merged 4 commits into
mainfrom
osv
Jul 17, 2026
Merged

replace retired pnpm audit with OSV-Scanner gate#69
jeesunikim merged 4 commits into
mainfrom
osv

Conversation

@jeesunikim

Copy link
Copy Markdown
Contributor

npm retired the legacy quick-audit endpoint (410), which pnpm audit still calls, so the CI audit step failed hard. This PR replaces it with a blocking OSV-Scanner step that reads pnpm-lock.yaml directly.

  • I also added osv-scanner.toml to ignore low/med severity packages. By default, OSV-Scanner fails on any severity.
  • I bumped protobufjs, recommended by dependabot

@jeesunikim
jeesunikim requested review from Copilot and marcelosalloum and removed request for Copilot July 17, 2026 18:24
@stellar-jenkins-ci

Copy link
Copy Markdown

Copilot AI review requested due to automatic review settings July 17, 2026 18:34
@socket-security

socket-security Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedesbuild@​0.28.0 ⏵ 0.28.191100 +17388100

View full report

@stellar-jenkins-ci

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Replaces the failing CI dependency audit with OSV-Scanner and updates vulnerable dependencies.

Changes:

  • Adds a blocking OSV-Scanner lockfile scan with an advisory exception.
  • Updates esbuild and protobufjs overrides and lockfile resolutions.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/test.yml Replaces pnpm audit with OSV-Scanner.
osv-scanner.toml Configures the temporary advisory exclusion.
package.json Updates dependency overrides.
pnpm-lock.yaml Records updated dependency resolutions.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +52 to +57
- name: Audit dependencies (OSV-Scanner)
uses: google/osv-scanner-action/osv-scanner-action@9a498708959aeaef5ef730655706c5a1df1edbc2 # v2.3.8
with:
scan-args: |-
--config=osv-scanner.toml
--lockfile=pnpm-lock.yaml
@stellar-jenkins-ci

Copy link
Copy Markdown

@jeesunikim
jeesunikim merged commit 75b08cb into main Jul 17, 2026
8 checks passed
@jeesunikim
jeesunikim deleted the osv branch July 17, 2026 23:21
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.

3 participants