feat: make pr-review-mention an org standard#237
Conversation
- Extract all logic from pr-review-mention.yml into pr-review-mention-reusable.yml (org single source of truth) - Slim pr-review-mention.yml down to a thin caller stub (local ref pattern, matching auto-rebase.yml) - Add standards/workflows/pr-review-mention.yml canonical template for other repos (@v1 reference) - Add pr-review-mention.yml to REQUIRED_WORKFLOWS and centralized stub checks in compliance-audit.sh - Document in ci-standards.md: template table, required-workflow count (6→7), and §10 with full spec - Add scripts/deploy-standard-workflows.sh to push standard stubs to all org repos in one command Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, 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 the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. 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 (7)
✨ 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 |
…eowners-standard Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces the pr-review-mention.yml workflow, which triggers a review agent when a specific bot is mentioned or assigned. The changes include updating the compliance audit script and documentation to reflect this new requirement, as well as adding a new deploy-standard-workflows.sh script to automate the distribution of workflow stubs across the organization. Feedback on the new deployment script identifies a naming mismatch for the Claude workflow, portability issues with the base64 command on macOS, and redundant API calls. Additionally, the reviewer noted that the repository limit should be increased for scalability and that several initialized counter variables remain unused.
There was a problem hiding this comment.
Pull request overview
This PR centralizes the PR Review Mention trigger into an org-level reusable workflow and formalizes it as a required org standard, aligning it with existing “thin stub + reusable” patterns (auto-rebase, claude, agent-shield, etc.). It also adds automation to deploy the standard workflow stubs org-wide and updates compliance auditing + documentation accordingly.
Changes:
- Added
pr-review-mention-reusable.ymlas the single source of truth and converted repo workflow(s) into thin caller stubs. - Updated standards + compliance audit to treat
pr-review-mention.ymlas an org-required, centralized stubbed workflow. - Introduced
deploy-standard-workflows.shto push standard stub workflows into org repositories via the Contents API.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
standards/workflows/pr-review-mention.yml |
Adds the canonical Tier-1 thin stub template for downstream repos. |
standards/ci-standards.md |
Documents the new standard workflow, increases required workflow count, and adds a full spec section. |
scripts/deploy-standard-workflows.sh |
New org-wide deployment script to upsert standard workflow stubs across repos. |
scripts/compliance-audit.sh |
Makes pr-review-mention.yml required and enforces it as a centralized stub. |
.github/workflows/pr-review-mention.yml |
Converts this repo’s workflow into a thin stub pointing at the local reusable. |
.github/workflows/pr-review-mention-reusable.yml |
Introduces the reusable implementation containing the review-dispatch logic. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Fix claude.yml compliance check: derive uses: from template (not stem-reusable heuristic), so the claude→claude-code-reusable name exception is handled automatically - Combine two API calls (SHA + content) into one fetch_existing call with tab-split output - Fix base64 portability: try -w 0 (GNU), fall back to -b 0 (BSD/macOS) - Increase repo list limit to 500 for larger orgs - Remove unused counter variables (already fixed in prior commit; this replaces the old approach) Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
- Declare GH_PAT_WORKFLOWS in workflow_call secrets block (matching other reusables) - Clarify fork-PR guard docs: only review_requested path excludes forks; comment triggers are base-repo-only by GitHub's event model, protected by trust check - Fix 'SHA' → 'tag' in standards/workflows/pr-review-mention.yml header comment - Add --no-archived to gh repo list in deploy script - Switch --field to --raw-field for content/sha/message to avoid form-encoding issues with base64's + and / characters Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Summary
pr-review-mention.ymlinto a newpr-review-mention-reusable.yml(org single source of truth), following the same pattern asauto-rebase,claude,agent-shield, etc.pr-review-mention.ymlin this repo to a thin caller stub using a local./ref (matching theauto-rebase.ymlpattern).standards/workflows/pr-review-mention.yml— the canonical@v1template for other repos to copy.pr-review-mention.ymltoREQUIRED_WORKFLOWSand thecentralizedstub-check array incompliance-audit.shso missing or inline copies surface as compliance findings.ci-standards.md: template table row, required-workflow count (6→7), and new §10 with full spec.scripts/deploy-standard-workflows.sh— deploys standard stubs to all org repos in one command (--dry-run,--workflow,--repo,--forceflags).Test plan
pr-review-mention-reusable.ymlparses cleanly (actionlint)deploy-standard-workflows.shpasses shellcheck@petry-review-boton this PR to confirm the workflow fires end-to-end🤖 Generated with Claude Code