Commit 9d2c4ae
fix(ci): hypatia-scan.yml -- --exit-zero + GITHUB_TOKEN (hyperpolymath/hypatia#213) (#19)
Mirrors hyperpolymath/hypatia#228 in this consumer repo.
## What was actually broken in `Hypatia Security Scan`
The scanner halts with `System.halt(1)` whenever findings exist at or
above the severity threshold (`lib/hypatia/cli.ex:158-160` pre-#228).
Under GitHub Actions' default `set -e`, that exit-1 short-circuits the
workflow step before `jq` aggregation, `actions/upload-artifact`, the PR
comment, AND the explicit "Check for critical or high-severity issues"
step.
The previous `actions/upload-artifact` SHA-bump sweep across the estate
(41 PRs) was based on a wrong diagnosis -- the failing runs were not at
action-resolve time. See hyperpolymath/hypatia#213 for the full
root-cause writeup.
## Changes in this PR
- **Pass `GITHUB_TOKEN`** to the scan step env so the Dependabot rule
can query alerts (and stops emitting `Warning: Dependabot alerts
unavailable: GITHUB_TOKEN not set`).
- **Append `--exit-zero`** to the `hypatia-cli.sh scan .` invocation so
findings-at-severity no longer short-circuits the step. The downstream
"Check for critical or high-severity issues" step (already in this
workflow) remains the explicit gate.
- **Pin `actions/upload-artifact` to v4.6.2**
(`ea165f8d65b6e75b540449e92b4886f43607fa02`) to match the estate-wide
pin.
## Notes
- `--exit-zero` was added in hyperpolymath/hypatia#228 and is silently
ignored by pre-#228 versions of the scanner (OptionParser strict mode
places unknown flags in `invalid` and the CLI discards that), so this PR
is safe to merge in either order relative to #228.
- This change does not affect non-CI usage of the scanner; the default
`exit 1` on findings is unchanged for shell / pre-commit users.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>1 parent 544a0b0 commit 9d2c4ae
1 file changed
Lines changed: 5 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
52 | 55 | | |
53 | 56 | | |
54 | 57 | | |
55 | 58 | | |
56 | | - | |
| 59 | + | |
57 | 60 | | |
58 | 61 | | |
59 | 62 | | |
| |||
75 | 78 | | |
76 | 79 | | |
77 | 80 | | |
78 | | - | |
| 81 | + | |
79 | 82 | | |
80 | 83 | | |
81 | 84 | | |
| |||
0 commit comments