Skip to content

fix: pass explicit config path to OSV scanner workflow#146

Merged
vredchenko merged 1 commit intomainfrom
fix/osv-scanner-config-path
Jan 28, 2026
Merged

fix: pass explicit config path to OSV scanner workflow#146
vredchenko merged 1 commit intomainfrom
fix/osv-scanner-config-path

Conversation

@vredchenko
Copy link
Collaborator

@vredchenko vredchenko commented Jan 28, 2026

Summary

  • Fix OSV scanner workflow failing to find ignore rules
  • Add --config flag via scan-args to both scanner jobs

Problem

The OSV scanner workflow fails because:

Solution

Pass --config ./osv-scanner.toml explicitly via scan-args to tell OSV scanner where to find the config file at repository root.

Test plan

  • OSV scanner workflow passes on this PR
  • Can verify manually: gh workflow run osv-scanner.yml after merge

OSV scanner looks for config files relative to lockfile location
(webui/package-lock.json), not the repo root. Since osv-scanner.toml
was moved to repo root in PR #145, the ignore rules weren't being found.

Add --config flag via scan-args to explicitly point to the config file
at repository root.
@github-actions github-actions bot added the devops CI/CD, deployment, infrastructure, or tooling work label Jan 28, 2026
@vredchenko vredchenko added security Security fixes, audits, or vulnerability remediation admin Project maintenance, dependency updates, or housekeeping smartem-devtools Developer tooling, documentation, and workspace configuration labels Jan 28, 2026
@vredchenko vredchenko merged commit 3ca40cd into main Jan 28, 2026
7 checks passed
@vredchenko vredchenko deleted the fix/osv-scanner-config-path branch January 28, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

admin Project maintenance, dependency updates, or housekeeping devops CI/CD, deployment, infrastructure, or tooling work security Security fixes, audits, or vulnerability remediation smartem-devtools Developer tooling, documentation, and workspace configuration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant