Skip to content

fix(cli): fail closed on scan block and clarify CI empty diffs#39

Merged
sairintechnologycom merged 1 commit into
mainfrom
fix/scan-fail-on-and-ci-clarity
Jul 13, 2026
Merged

fix(cli): fail closed on scan block and clarify CI empty diffs#39
sairintechnologycom merged 1 commit into
mainfrom
fix/scan-fail-on-and-ci-clarity

Conversation

@sairintechnologycom

Copy link
Copy Markdown
Owner

Summary

  • Scan commands exit non-zero under --mode block (or --fail-on) so shell scripts cannot install after a BLOCK decision
  • CI human summary warns when changed-only scans zero packages (false green)
  • Public-boundary check falls back to find+grep when ripgrep is missing

Test plan

  • go test ./...
  • make build + doctor + corpus
  • Local fixtures: block mode exit 1 on curl-pipe-sh, exit 0 on safe-package
  • CI scan NOTICE for zero changed packages
  • scripts/check-public-boundary.sh without rg
  • CI workflow green
  • Tag beta release after merge

Scan commands exit non-zero under --mode block (or --fail-on) so shell
scripts cannot install after a BLOCK decision. CI human summaries now
warn when changed-only scans zero packages, and the public-boundary
check falls back to grep when ripgrep is missing.
@github-actions

Copy link
Copy Markdown

PkgSafe Dependency Gate

Decision: ALLOW
Mode: WARN
Fail On: BLOCK
Workflow Result: passes
Ecosystem: npm
Lockfile: testdata/npm/self-scan/package-lock.json
Changed Only: false
Baseline: main
Packages Scanned: 1

Counts

Allow Warn Review Required Block Unknown Vulnerabilities
1 0 0 0 0 0

Warn / Review / Block Findings

No blocked, review-required, or warning dependencies found.

Recommended Action

All scanned packages are allowed by policy.

Governance Evidence

  • Evidence pack generated: pkgsafe-evidence-pack.zip

@sairintechnologycom
sairintechnologycom merged commit 26a3ae2 into main Jul 13, 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