Commit 780a578
fix(ci): hypatia-scan.yml -- --exit-zero + GITHUB_TOKEN (hyperpolymath/hypatia#213) (#18)
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 f6db41b commit 780a578
1 file changed
Lines changed: 5 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
53 | 53 | | |
54 | 54 | | |
55 | 55 | | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
56 | 59 | | |
57 | 60 | | |
58 | 61 | | |
59 | 62 | | |
60 | | - | |
| 63 | + | |
61 | 64 | | |
62 | 65 | | |
63 | 66 | | |
| |||
79 | 82 | | |
80 | 83 | | |
81 | 84 | | |
82 | | - | |
| 85 | + | |
83 | 86 | | |
84 | 87 | | |
85 | 88 | | |
| |||
0 commit comments