Skip to content

ci: bump Hypatia scan reusable pin to GITHUB_TOKEN fix - #541

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/hypatia-token-setup-uobc42
Jun 26, 2026
Merged

ci: bump Hypatia scan reusable pin to GITHUB_TOKEN fix#541
hyperpolymath merged 1 commit into
mainfrom
claude/hypatia-token-setup-uobc42

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

What

Bumps the scan wrapper's reusable-workflow pin:

hypatia-scan-reusable.yml@d135b05…  →  @abd93c3…

abd93c3 is the standards-side commit (see hyperpolymath/standards PR for claude/hypatia-token-setup-uobc42) that sets GITHUB_TOKEN: ${{ secrets.HYPATIA_SCAN_PAT || secrets.GITHUB_TOKEN }} in the scan step.

Why

  • Stops the three GITHUB_TOKEN not set warnings (Dependabot / code-scanning / secret-scanning) in this repo's scan.
  • The reusable at this pin also carries --exit-zero + if: always() on the artifact upload, so hypatia-findings.json is published even when the severity gate fails — making the real findings reviewable.

Notes

  • This does not make CI green: the scan still exits non-zero on medium+ findings (currently 22 medium / 0 high).
  • Once the standards PR merges, this pin can be advanced to the merge commit; abd93c3 remains a valid reachable SHA in the meantime.
  • The org secret HYPATIA_SCAN_PAT must exist for the Dependabot check to actually resolve (read-only fine-grained PAT).

🤖 Generated with Claude Code


Generated by Claude Code

Point the scan wrapper at the standards reusable commit that sets
GITHUB_TOKEN (HYPATIA_SCAN_PAT || GITHUB_TOKEN) in the scan step, so the
Dependabot / code-scanning / secret-scanning alert checks stop emitting
"GITHUB_TOKEN not set" warnings. The reusable at this pin also keeps the
--exit-zero + always() artifact upload, so findings are published even
when the severity gate fails.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V13sGvHnaeCY5A9fF2QsDm
@hyperpolymath
hyperpolymath marked this pull request as ready for review June 26, 2026 18:34
@hyperpolymath
hyperpolymath merged commit ce80471 into main Jun 26, 2026
30 of 32 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.

2 participants