feat: route bot-authored PRs to per-repo owner#8
Conversation
Adds bot_pr_owners map keyed by repo name. When a PR author is in bot_authors (dependabot, renovate, claude, copilot review bot, listed external contributors), the workflow now requests the mapped owner as reviewer instead of exiting 0. Mapping: lisk-backend -> ishantiw lisk-web -> mmarinovic lisk-mobile -> 5heri lisk-infra -> Nazgolze lisk-contracts-> matjazv Repos without a bot_pr_owners entry retain the old skip behavior, so this is opt-in per repo. Implementation: - review-map.yml: add bot_pr_owners section + clarify bot_authors comment - auto-assign-reviewer.yml: hoist EXISTING fetch above bot path so the bot path can short-circuit on idempotency; replace skip with route + request + comment + soft-fail (mirrors the squad cascade's failure semantics). - validate-review-map.yml: assert every bot_pr_owners key is in enabled_repos; include bot_pr_owners values in the GitHub-user existence check. - docs/auto-reviewer.md: document the bot short-circuit, add the current owner table, update kill-switch list.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughAdds bot and external author routing to the auto-reviewer workflow. Introduces a bot_pr_owners mapping in the review configuration, validates it in CI, and implements a cascade bypass that routes bot-authored PRs to designated owners before falling back to manual assignment. Documentation is updated with kill switches and routing details. ChangesBot-Author Reviewer Routing
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Microsoft Presidio Analyzer (2.2.362).github/workflows/auto-assign-reviewer.ymlMicrosoft Presidio Analyzer failed to scan this file Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Comment |
Summary
Bot-authored PRs (dependabot, renovate, claude, copilot review bot, listed external contributors) now get routed to a single per-repo owner instead of being silently skipped.
Mapping
lisk-backend@ishantiwlisk-web@mmarinoviclisk-mobile@5herilisk-infra@Nazgolzelisk-contracts@matjazvWhy
Previously, anything in
bot_authorsexited 0 — dependabot bumps and external contributor PRs landed with no reviewer requested. The TL/owner of each repo should at least see them.Design choices
bot_pr_ownersis opt-in per repo — drop a repo to disable bot routing there without touching the workflow.EXISTINGfetch hoisted above the bot path so re-runs don't double-request.Files
review-map.yml— addsbot_pr_ownerssection..github/workflows/auto-assign-reviewer.yml— hoistsEXISTING, replaces bot-author skip with route → request → comment..github/workflows/validate-review-map.yml— asserts everybot_pr_ownerskey is inenabled_repos; includes mapped owners in the GitHub-user existence check.docs/auto-reviewer.md— documents the bot short-circuit; adds current owner table; updates kill-switch list.Test plan
validate-review-map.ymlCI passes on this PR (it triggers onreview-map.ymledits).lisk-backend→@ishantiwrequested.lisk-web→@mmarinovicrequested.bot_pr_ownersfrom a bot author → workflow exits 0 silently (current behavior preserved).Not in this PR
lisk-{backend,web,mobile,infra,contracts}is unchanged — they already invoke the reusable workflow.Summary by CodeRabbit
New Features
Documentation