Skip to content

Commit 4cb276f

Browse files
fix(ci): adopt canonical hypatia-scan.yml (env.HOME/scanner-layout + Comment-step gate) (#7)
1 parent 2ca6164 commit 4cb276f

1 file changed

Lines changed: 25 additions & 3 deletions

File tree

.github/workflows/hypatia-scan.yml

Lines changed: 25 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,26 @@ on:
1010
schedule:
1111
- cron: '0 0 * * 0' # Weekly on Sunday
1212
workflow_dispatch:
13+
# Estate guardrail: cancel superseded runs so re-pushes don't pile up
14+
# queued runs across the estate. Safe here because this workflow only
15+
# performs read-only checks/lint/test/scan with no publish or mutation.
16+
concurrency:
17+
group: ${{ github.workflow }}-${{ github.ref }}
18+
cancel-in-progress: true
1319

1420
permissions:
1521
contents: read
1622
# security-events: read lets the built-in GITHUB_TOKEN query this
17-
# repo\'s own Dependabot alerts via the Hypatia DependabotAlerts rule.
23+
# repo's own Dependabot alerts via the Hypatia DependabotAlerts rule
24+
# (DA001-DA004). Without this, `scan_from_path` gets HTTP 403 and
25+
# the rule silently returns no findings.
26+
# See 007-lang/audits/audit-dependabot-automation-gap-2026-04-17.md.
1827
security-events: read
28+
# pull-requests: write lets the advisory "Comment on PR with findings"
29+
# step post its summary. Without it the built-in GITHUB_TOKEN gets
30+
# "Resource not accessible by integration" and (absent continue-on-error)
31+
# hard-fails the scan — exactly what the gate-decoupling design forbids.
32+
pull-requests: write
1933

2034
jobs:
2135
scan:
@@ -41,8 +55,8 @@ jobs:
4155
fi
4256
4357
- name: Build Hypatia scanner (if needed)
44-
working-directory: /home/runner/hypatia
4558
run: |
59+
cd "$HOME/hypatia"
4660
if [ ! -f hypatia ]; then
4761
echo "Building hypatia scanner..."
4862
mix deps.get
@@ -52,7 +66,10 @@ jobs:
5266
- name: Run Hypatia scan
5367
id: scan
5468
env:
55-
# Suppress the Dependabot "GITHUB_TOKEN not set" warning.
69+
# Pass the built-in Actions token through to Hypatia so the
70+
# DependabotAlerts rule can query this repo's own alerts.
71+
# For cross-repo scanning (fleet-coordinator scan-supervised),
72+
# a PAT with `security_events` scope is required instead.
5673
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
5774
run: |
5875
echo "Scanning repository: ${{ github.repository }}"
@@ -199,6 +216,11 @@ jobs:
199216
200217
- name: Comment on PR with findings
201218
if: github.event_name == 'pull_request' && steps.scan.outputs.findings_count > 0
219+
# Advisory only — posting findings as a PR comment must never gate
220+
# the scan (hypatia#213 gate decoupling). Belt-and-braces alongside
221+
# the pull-requests: write permission above: a token/API hiccup or
222+
# a fork PR (read-only token) skips the comment, not the check.
223+
continue-on-error: true
202224
uses: actions/github-script@ed597411d8f924073f98dfc5c65a23a2325f34cd # v7
203225
with:
204226
script: |

0 commit comments

Comments
 (0)