Skip to content

Configure AI PR reviewers: CodeRabbit primary, Codex secondary, Copilot off #338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
brianmhunt wants to merge 4 commits into from
Closed

Configure AI PR reviewers: CodeRabbit primary, Codex secondary, Copilot off #338

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
758baf9
Pin AI PR reviewer config: CodeRabbit primary, Codex secondary, Copil…
Apr 20, 2026
f8ce701
AGENTS.md: trim AI PR review section to one-line pointer
Apr 20, 2026
5fa64f3
Simplify .coderabbit.yaml: prune defaults, fix glob overlap
Apr 20, 2026
d517543
Drop path_instructions: don't pre-filter the reviewer
Apr 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions .coderabbit.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
# CodeRabbit configuration — primary AI reviewer for TKO PRs.
# Codex (chatgpt-codex-connector) is kept as a secondary, adversarial
# reviewer for deep reviews on demand. GitHub Copilot auto-review is
# disabled at the repo level.
#
# Schema: https://docs.coderabbit.ai/guides/configure-coderabbit
# Anything not set here uses CodeRabbit's default.

reviews:
poem: false
5 changes: 5 additions & 0 deletions AGENTS.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,11 @@ Significant changes should have a plan file in `plans/` before implementation
begins. Plans document the context, approach, and verification steps. Review
existing plans in that directory for format examples.

## AI PR review

CodeRabbit (primary) + Codex (secondary). Copilot disabled. Config and
rationale in `.coderabbit.yaml`.
Comment on lines +156 to +159
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Apr 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor

Clarify that Copilot disabling is an external repo setting.

Line 158 says Copilot is disabled, but the PR notes that this is a post-merge GitHub UI action. Consider wording this as an intended/required repository setting so the docs don’t overstate what this commit enforces.

Proposed wording 
 ## AI PR review
 
-CodeRabbit (primary) + Codex (secondary). Copilot disabled. Config and
-rationale in `.coderabbit.yaml`.
+CodeRabbit (primary) + Codex (secondary). Copilot auto-review should remain
+disabled via repository settings. Config and rationale live in `.coderabbit.yaml`.
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
## AI PR review
CodeRabbit (primary) + Codex (secondary). Copilot disabled. Config and
rationale in `.coderabbit.yaml`.
## AI PR review
CodeRabbit (primary) + Codex (secondary). Copilot auto-review should remain
disabled via repository settings. Config and rationale live in `.coderabbit.yaml`.
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@AGENTS.md` around lines 156 - 159, The document currently states "Copilot
disabled" under the "AI PR review" section which implies the commit enforces
this; update the wording around that phrase to clarify that disabling GitHub
Copilot is an external repository-level setting (a post-merge GitHub UI action)
rather than something this commit enforces—e.g., change the line to indicate
"Copilot should be disabled at the repository level (GitHub setting); this is an
external repo configuration, not enforced by this commit" so readers understand
it’s a required repo setting, not enforced by code changes.


## Agent-First Documentation

AI coding agents are first-class citizens of TKO. The docs site serves both
Expand Down
Loading