fix: add claude.yml template + checkout audit check#63
Merged
Conversation
Root cause: the recent org-wide PRs added checkout only to the claude-issue job, leaving the claude job (PR reviews / @claude mentions) without one. claude-code-action reads CLAUDE.md and AGENTS.md from the working tree; without checkout it errors on every PR-triggered run. Changes: - standards/workflows/claude.yml: canonical copy-paste template with checkout in both jobs, matching the other templates in standards/workflows/. Both checkout steps are annotated as REQUIRED to prevent silent removal. - scripts/compliance-audit.sh: new check_claude_workflow_checkout() detects any repo whose claude or claude-issue job is missing checkout and raises an error finding. Wired into the main audit loop so weekly scans surface affected repos automatically. - standards/ci-standards.md: added a visible callout that both jobs need checkout and a pointer to the new template file. Closes #33 Co-authored-by: don-petry <don-petry@users.noreply.github.com>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
standards/workflows/claude.yml— canonical copy-paste template for all repos, withactions/checkoutin bothclaudeandclaude-issuejobs andREQUIREDannotations so the step is never silently removed.check_claude_workflow_checkout()toscripts/compliance-audit.sh— detects repos where either job is missing checkout and raises anerrorfinding. Wired into the main audit loop so the weekly scan auto-surfaces affected repos.standards/ci-standards.md— prominent callout that both jobs need checkout, pointer to the new template.Root Cause
The recent org-wide PRs (broodly#70, markets#54, TalkTerm#49, ContentTwin#31) added a
Checkout repositorystep only to theclaude-issuejob. Theclaudejob (PR reviews /@claudementions) was left without one.claude-code-actionreadsCLAUDE.mdandAGENTS.mdfrom the working tree; without checkout it errors on every PR-triggered run.Closes #33
Generated with Claude Code