fix(compliance): add CLAUDE.md and dependabot.yml#152
Conversation
|
Warning Review limit reached
Your plan currently allows 1 review/hour. Refill in 59 minutes and 49 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAdded ChangesRepository Development Guidance for Claude Code
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request introduces a Dependabot configuration for GitHub Actions and a CLAUDE.md file to provide context and guidelines for Claude Code. The review feedback suggests bundling Dependabot updates into groups to reduce noise, expanding the repository purpose section to include missing directories, and adding a commands section for common tasks like shell script linting.
| - **`agents/`** — Copilot custom agent profiles (org-wide, invocable from GitHub.com, VS Code, JetBrains) | ||
| - **`scripts/`** — Shell orchestration for GitHub Actions (PR review, health checks) | ||
| - **`.github/workflows/`** — Scheduled automation (PR review, health checks, dependency audit) |
There was a problem hiding this comment.
The "Repository Purpose" section is missing the prompts/ and frameworks/ directories, which are key components of this repository as described in the README.md and AGENTS.md. Including them provides essential context for the agent's understanding of the project structure.
| - **`agents/`** — Copilot custom agent profiles (org-wide, invocable from GitHub.com, VS Code, JetBrains) | |
| - **`scripts/`** — Shell orchestration for GitHub Actions (PR review, health checks) | |
| - **`.github/workflows/`** — Scheduled automation (PR review, health checks, dependency audit) | |
| - **`agents/`** — Copilot custom agent profiles (org-wide, invocable from GitHub.com, VS Code, JetBrains) | |
| - **`prompts/`** — Prompt libraries used by workflows | |
| - **`scripts/`** — Shell orchestration for GitHub Actions (PR review, health checks) | |
| - **`frameworks/`** — Installed agentic frameworks (git subtree) | |
| - **`.github/workflows/`** — Scheduled automation (PR review, health checks, dependency audit) |
| ## Key Guidelines | ||
|
|
||
| - This repo contains shell scripts executed by GitHub Actions — test changes locally with `shellcheck` | ||
| - Workflow files here are non-stub (not thin callers) — they contain org-private automation logic | ||
| - SHAs for action pinning must be looked up via the GitHub API — never guessed |
There was a problem hiding this comment.
CLAUDE.md files are most effective for Claude Code when they include a "Commands" section for common tasks like linting. This allows the agent to verify its work autonomously using the tools mentioned in the guidelines.
| ## Key Guidelines | |
| - This repo contains shell scripts executed by GitHub Actions — test changes locally with `shellcheck` | |
| - Workflow files here are non-stub (not thin callers) — they contain org-private automation logic | |
| - SHAs for action pinning must be looked up via the GitHub API — never guessed | |
| ## Key Guidelines | |
| - This repo contains shell scripts executed by GitHub Actions — test changes locally with `shellcheck` | |
| - Workflow files here are non-stub (not thin callers) — they contain org-private automation logic | |
| - SHAs for action pinning must be looked up via the GitHub API — never guessed | |
| ## Commands | |
| - Lint shell scripts: `shellcheck scripts/*.sh` |
There was a problem hiding this comment.
Pull request overview
Adds missing compliance artifacts for this org-level .github-private repository: Claude Code instructions and Dependabot configuration for GitHub Actions updates.
Changes:
- Add
CLAUDE.mdwith a reference toAGENTS.mdand repository-specific guidance. - Add
.github/dependabot.ymlto enable weekly GitHub Actions dependency updates withsecurityanddependencieslabels.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
CLAUDE.md |
Adds Claude Code repo instructions and points contributors to AGENTS.md standards. |
.github/dependabot.yml |
Enables Dependabot for the github-actions ecosystem with standard labels. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Auto-rebase failed — merge conflict — this branch has conflicts with Please resolve the conflicts and push: |
f89bd49 to
2557ab4
Compare
Superseded by automated re-review at
|
Superseded by automated re-review at
|
Review — fix requested (cycle 3/3)The automated review identified the following issues. Please address each one: Findings to fix[Findings would be inserted here] Additional tasks
The review cascade will automatically re-review after new commits are pushed. |
4ade6da to
1268491
Compare
- Add groups bundling to dependabot.yml for GitHub Actions updates - Add prompts/ and frameworks/ directories to CLAUDE.md Repository Purpose - Correct inaccurate "non-stub" claim; clarify which workflows are thin callers - Add Commands section with shellcheck lint command Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with Claude will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
|
@dev-lead please re-check this PR — re-triggering after the dev-lead/pr-review workflows were briefly disabled. |
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
don-petry
left a comment
There was a problem hiding this comment.
PR Review — fix(compliance): add CLAUDE.md and dependabot.yml
Overview
This PR adds two compliance artifacts for the .github-private repo:
CLAUDE.md(new) — Claude Code context file.github/dependabot.yml(updated) — restructured and extended with required labels and grouping
Both changes are focused and purposeful, addressing four documented compliance findings.
CLAUDE.md — New File ✅
- Correctly references
AGENTS.md— satisfies theclaude-md-missing-agents-reffinding - Repository purpose section is complete — lists
agents/,prompts/,scripts/,frameworks/, and.github/workflows/, consistent withAGENTS.mdandREADME.md - Key guidelines accurately distinguish thin-caller stubs from org-private workflows — the bullet correctly names
claude.yml,agent-shield.yml,auto-rebase.yml,dependabot-automerge.ymlas stubs and notes that others contain private logic. This matches the note inAGENTS.mdabout not modifyingagent-shield.ymland thedev-lead.ymldistinction - Commands section includes
shellcheck scripts/*.sh— appropriate, matches the project's linting practice - SHA pinning note is consistent with the project's policy against guessing SHAs
One observation: CLAUDE.md lists frameworks/ as "Installed agentic frameworks (git subtree)" but the current AGENTS.md also lists bmad-method, spec-kit, gsd by name. Not a blocker — the CLAUDE.md description is accurate enough for Claude Code's purposes.
.github/dependabot.yml — Restructured ✅
Key improvements over the previous version:
version: 2moved to the top — the old file had it at the bottom, which is technically valid but non-canonical; top placement is the dependabot spec convention- YAML indentation regularized — consistent 2-space indentation under
updates: securityanddependencieslabels added — fixesmissing-security-labelandmissing-dependencies-labelfindingsgroups:block added — bundles all action updates into a single PR, reducing CI noise; this addresses the Gemini reviewer feedback from earlier in the threadignore:forgithub/gh-aw-actions/*preserved — the lock comment explaining the gh-aw compiler version-lock is retained
No functional regressions: the github-actions ecosystem, / directory, weekly schedule, and 10 PR limit are all preserved.
Previous Reviewer Feedback — Status
| Reviewer | Comment | Status |
|---|---|---|
| Gemini | Add groups: to bundle action updates |
✅ Addressed — groups.actions with "*" pattern present |
| Gemini | List prompts/ and frameworks/ in CLAUDE.md |
✅ Addressed — both are listed |
| Gemini | Add Commands section to CLAUDE.md |
✅ Addressed — shellcheck scripts/*.sh present |
| Copilot | Clarify thin-caller vs. non-stub distinction | ✅ Addressed — current wording correctly names stubs and distinguishes from private logic workflows |
| CodeRabbit | — | ✅ Approved (2026-06-06) |
Test / Lint Verification
- shellcheck on
scripts/*.sh: only pre-existingSC1091(info, source-following) andSC2016(info, single-quote expansion) notices — none introduced by this PR, no errors or warnings - No shell scripts modified; both changed files are YAML and Markdown — no additional test surface
Verdict
Looks good. Both files are correct, complete, and consistent with AGENTS.md standards. All four compliance findings are addressed, all previous reviewer feedback has been incorporated, and no regressions are introduced. The remaining findings listed in the PR description (workflow files, SHA pinning, repo settings, secret scanning) appropriately require elevated permissions and are correctly deferred to human action.
Note: Cannot self-approve — this PR was opened by the same bot account. A human reviewer from
org-leadsshould provide the required approval.
Dev-Lead — on-mention (no-changes)Engine ran but made no changes. |
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |
|
Automated review — human attention neededThis PR has been through 3 automated review cycles (cap: 3) without converging on an approval-and-merge state. Further automated review has been paused to avoid infinite loops. Please take a look manually, or close this PR if it's no longer needed. Once a human review resolves the situation, remove the Posted by the donpetry-bot PR-review cascade. |
Dev-Lead — fix-bot-comment (no-changes)Agent reasoning |



Summary
CLAUDE.mdwith requiredAGENTS.mdreference (fixesclaude-md-missing-agents-ref).github/dependabot.ymlwithgithub-actionsecosystem,securityanddependencieslabels (fixesmissing-github-actions-ecosystem,missing-security-label,missing-dependencies-label)Compliance findings addressed
claude-md-missing-agents-refmissing-github-actions-ecosystemmissing-security-labelmissing-dependencies-labelRemaining findings requiring human action
The following findings for
.github-privaterequire elevated permissions not available to this agent:missing-ci.yml,missing-pr-review-mention.yml,non-stub-auto-rebase.yml,non-stub-dependabot-automerge.yml,non-stub-agent-shield.yml) — requireworkflowswrite permissionunpinned-actions-*) — requireworkflowswrite permissionallow_auto_merge,delete_branch_on_merge,check-suite-auto-trigger-*) — runbash scripts/apply-repo-settings.sh .github-privatewith admin credentialsbash scripts/apply-repo-settings.sh .github-privatePart of petry-projects/.github#261
Generated with Claude Code
Summary by CodeRabbit