[#569] Disable automatic CodeRabbit issue comments - #573
GodlyDonuts wants to merge 1 commit into
Conversation
|
I'm not sure these are the choices we want to make @GodlyDonuts Can you explain your reasoning for each of them? I made this issue because the AI code review kept spamming edit suggestions, ai prompt suggestions, and pull request summaries, among other things. Honestly I already tried having ai silence coderabbit. It worked a bit, but not perfectly. So I made the issue to look at it more in depth when I have time. Happy to look through this PR if you are willing to explain more in depth, but quite frankly I don't appreciate AI slop PRs without a real person describing the change produced. |
|
Also, @GodlyDonuts cosign your commits with the AI agent used in the future. |
|
@alexanderpaolini Thats my fault, I told codex to ignore the agents.md a while back and it seemed to have followed through with that command even now when creating this commit, including omitting the AI co-author trailer. Okay so this is my justification for everything done:
My goal was to retain one quiet automatic code review when a PR leaves draft, while removing everything that creates unsolicited issue comments, repeated reviews, prompts, and extra status noise. |
|
@alexanderpaolini I realized that the solution could be a lot simpler than that, we could just do the following: issue_enrichment:
auto_enrich:
enabled: false
planning:
enabled: true
auto_planning:
enabled: falseThis would stop unsolicited comments and automatic coding plans like the one linked in #569, while preserving manually requested The existing configuration already uses the quiet review profile and disables PR summaries and AI-agent prompts, so this change should address the newly introduced issue spam without weakening the useful parts of CodeRabbit’s reviews. If that scope sounds right, I’ll amend the draft PR accordingly :) |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
set -euo pipefail
if [ -f .coderabbit.yml ]; then
nl -ba .coderabbit.yml | sed -n '1,260p'
else
fd -HI '^\.coderabbit\.ya?ml$' .
fiLength of output: 11817 🏁 Script executed: #!/bin/bash
set -euo pipefail
nl -ba .coderabbit.yml | sed -n '249,380p'Length of output: 4406 🏁 Script executed: #!/bin/bash
set -euo pipefail
rg -n -C 8 '^\s*issue_enrichment:|^\s*(auto_enrich|planning|auto_planning):|^\s*enabled:\s*(true|false)\s*$' .coderabbit.yml | tail -n 150Length of output: 3948 🤖 web_search executed:
Length of output: 7469
issue_enrichment:
auto_enrich:
enabled: false
planning:
enabled: true
auto_planning:
enabled: falseThis keeps manual ✏️ Learnings added
You are interacting with an AI system. |
yeah bro I know |
mm lets add this as well so we can get rid of that: knowledge_base:
opt_out: true |
|
@GodlyDonuts IMO a lot of these changes are unnecessary but I like the more narrow version that you just put out. Personally, these are my critiques and what doesn't need to be changed or altered that much unless everyone says otherwise:
Rest of it seems fine with me to reduce some AI clutter, especially with the enrichment changes |
Keep manual issue planning available while disabling unsolicited enrichment and label-triggered plans. Co-authored-by: Codex <codex@openai.com>
97b93c3 to
b515408
Compare
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: QUIET Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe CodeRabbit configuration disables Priority: ➖ Normal Severity of issue fixed: Medium Merge Risk: ⚪ Minimal · up to The change limits unsolicited automation while preserving manual planning and review workflows. 🚥 Pre-merge checks | ✅ 8✅ Passed checks (8 passed)
✨ 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 |
Why
CodeRabbit automatically enriches issues and generates full coding plans when common labels are applied, adding unsolicited noise to issue discussions. PR reviews and on-demand planning should remain available.
What
Closes: #569
@coderabbitai planfunctionality.Test Plan
pnpm exec prettier --check .coderabbit.yml.coderabbit.ymlagainst CodeRabbit’s liveschema.v2.jsonChecklist