Repository navigation
Compliance: required-claude-check-broken #97
Description
Compliance Finding
Category: rulesets
Severity: error
Check: required-claude-check-broken
Detail
Required-status-check ruleset includes claude, which is incompatible with workflow-modifying PRs. claude-code-action's GitHub App refuses to mint an OAuth token for any PR whose diff includes a workflow file, so the check fails on every workflow PR and the merge gate becomes a deadlock. Remove claude from required status checks — do NOT rename it. The Claude review check still runs on normal PRs and surfaces feedback without being a merge gate. See scripts/apply-rulesets.sh (post combsmith/.github#94) for the canonical required-checks list.
Standard Reference
standards/ci-standards.md#centralization-tiers
Remediation
Please review the linked standard and bring this repository into compliance.
See the full standards documentation for implementation guidance.
This issue was automatically created by the weekly compliance audit.
Activity
don-petry commentedon Apr 9, 2026
Weekly Compliance Audit (2026-04-09)
This finding is still open.
Detail: Required-status-check ruleset includes
claude, which is incompatible with workflow-modifying PRs. claude-code-action's GitHub App refuses to mint an OAuth token for any PR whose diff includes a workflow file, so the check fails on every workflow PR and the merge gate becomes a deadlock. Removeclaudefrom required status checks — do NOT rename it. The Claude review check still runs on normal PRs and surfaces feedback without being a merge gate. Seescripts/apply-rulesets.sh(post combsmith/.github#94) for the canonical required-checks list.Standard: standards/ci-standards.md#centralization-tiers
don-petry commentedon Apr 9, 2026
Claude encountered an error —— View job
I'll analyze this and get back to you.
don-petry commentedon Apr 10, 2026
Weekly Compliance Audit (2026-04-10)
This finding is still open.
Detail: Required-status-check ruleset includes
claude, which is incompatible with workflow-modifying PRs. claude-code-action's GitHub App refuses to mint an OAuth token for any PR whose diff includes a workflow file, so the check fails on every workflow PR and the merge gate becomes a deadlock. Removeclaudefrom required status checks — do NOT rename it. The Claude review check still runs on normal PRs and surfaces feedback without being a merge gate. Seescripts/apply-rulesets.sh(post combsmith/.github#94) for the canonical required-checks list.Standard: standards/ci-standards.md#centralization-tiers
don-petry commentedon Apr 14, 2026
Claude finished @don-petry's task in 1m 57s —— View job
Fixed: removed
claudefrom required status checksscripts/apply-rulesets.shclaudefrom required status checks in thecode-qualityrulesetWhat was done
The
code-qualitybranch ruleset on this repository includedclaudeas a required status check. This caused a deadlock on any PR that modified.github/workflows/files because claude-code-action's GitHub App refuses to mint an OAuth token for workflow-touching PRs, making the check permanently fail.The fix was applied directly via the GitHub API —
claudehas been removed from the required status checks. The updated required checks now match the canonical list frompetry-projects/.github/scripts/apply-rulesets.sh(which explicitly notes thatclaudemust NOT be a required check):SonarCloudAnalyze (javascript-typescript)Analyze (actions)LintFormat checkType checkTestsclaudeThe Claude review check still runs on normal PRs and provides feedback — it just no longer blocks merges.
don-petry commentedon Apr 17, 2026
Resolved! This check is now passing as of 2026-04-17. Closing automatically.