Skip to content

Added Qodo configuration - #43668

Merged
lukeheath merged 2 commits into
mainfrom
victor/qodo-config
Apr 16, 2026
Merged

Added Qodo configuration#43668
lukeheath merged 2 commits into
mainfrom
victor/qodo-config

Conversation

@getvictor

@getvictor getvictor commented Apr 16, 2026

Copy link
Copy Markdown
Member

Disable Qodo by default

Summary by CodeRabbit

  • Chores
    • Updated development tooling configuration for code review processes.

@getvictor

Copy link
Copy Markdown
Member Author

/agentic_review

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Apr 16, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider


Advisory comments

1. Ambiguous “disabled by default” 🐞 Bug ⚙ Maintainability
Description
The PR description says “Disable Qodo by default”, but the added config disables automatic triggers
while keeping the review agent enabled for manual comment commands, which can be interpreted as not
actually disabling Qodo. This ambiguity can lead to incorrect expectations about whether Qodo is
fully off vs. only auto-run is off.
Code

.pr_agent.toml[R4-14]

+# Do not auto-run anything when a PR is opened
+pr_commands = []
+
+# Do not auto-run anything on new commits / pushes
+handle_push_trigger = false
+push_commands = []
+
+# Keep the review tool available for manual comment commands
+[review_agent]
+enabled = true
+publish_output = true
Evidence
The config explicitly disables automatic execution on PR open/push (no pr_commands, push trigger
disabled), while simultaneously keeping the review agent enabled, meaning Qodo can still run when
manually invoked.

.pr_agent.toml[4-9]
.pr_agent.toml[11-14]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The PR description implies Qodo is disabled, but the configuration actually disables auto-triggers while leaving the agent enabled for manual runs.

## Issue Context
This is primarily a clarity/expectation issue: maintainers may read the PR description and assume Qodo is completely disabled.

## Fix Focus Areas
- .pr_agent.toml[11-14]

## Suggested fix
- If the goal is to fully disable Qodo, set `[review_agent].enabled = false`.
- If the goal is only to disable auto-run, keep the config as-is but update the PR title/description (or add an explicit clarifying comment) to say "Disable Qodo auto-run by default (manual commands still enabled)".

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

@getvictor
getvictor marked this pull request as ready for review April 16, 2026 15:48
Copilot AI review requested due to automatic review settings April 16, 2026 15:48

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@qodo-free-for-open-source-projects

Copy link
Copy Markdown

Review Summary by Qodo

Add Qodo PR agent configuration with auto-run disabled

Grey Divider

Walkthroughs

Description
• Add .pr_agent.toml configuration file for Qodo
• Disable auto-run on PR open and push events
• Keep review tool available for manual comment commands
Diagram
flowchart LR
  A[".pr_agent.toml"] -- "github_app config" --> B["pr_commands = []"]
  A -- "github_app config" --> C["handle_push_trigger = false"]
  A -- "review_agent config" --> D["enabled = true"]
  A -- "review_agent config" --> E["publish_output = true"]
Loading

Grey Divider

File Changes

1. .pr_agent.toml ⚙️ Configuration changes +12/-0

Add Qodo config disabling auto-run, enabling manual reviews

• New configuration file for Qodo code review tool
• Disables automatic PR review on open and push events by setting pr_commands = [] and
 handle_push_trigger = false
• Keeps the review agent enabled for manual comment-triggered reviews with output publishing enabled

.pr_agent.toml


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Apr 16, 2026

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider


Advisory comments

1. Ambiguous “disabled by default” 🐞 Bug ⚙ Maintainability
Description
The PR description says “Disable Qodo by default”, but the added config disables automatic triggers
while keeping the review agent enabled for manual comment commands, which can be interpreted as not
actually disabling Qodo. This ambiguity can lead to incorrect expectations about whether Qodo is
fully off vs. only auto-run is off.
Code

.pr_agent.toml[R4-14]

+# Do not auto-run anything when a PR is opened
+pr_commands = []
+
+# Do not auto-run anything on new commits / pushes
+handle_push_trigger = false
+push_commands = []
+
+# Keep the review tool available for manual comment commands
+[review_agent]
+enabled = true
+publish_output = true
Evidence
The config explicitly disables automatic execution on PR open/push (no pr_commands, push trigger
disabled), while simultaneously keeping the review agent enabled, meaning Qodo can still run when
manually invoked.

.pr_agent.toml[4-9]
.pr_agent.toml[11-14]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The PR description implies Qodo is disabled, but the configuration actually disables auto-triggers while leaving the agent enabled for manual runs.
## Issue Context
This is primarily a clarity/expectation issue: maintainers may read the PR description and assume Qodo is completely disabled.
## Fix Focus Areas
- .pr_agent.toml[11-14]
## Suggested fix
- If the goal is to fully disable Qodo, set `[review_agent].enabled = false`.
- If the goal is only to disable auto-run, keep the config as-is but update the PR title/description (or add an explicit clarifying comment) to say "Disable Qodo auto-run by default (manual commands still enabled)".

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a .pr_agent.toml configuration file to control Qodo (PR-Agent) behavior, aiming to prevent automatic runs while keeping the tool available when explicitly invoked.

Changes:

  • Introduces .pr_agent.toml with GitHub App settings to avoid auto-running on PR open.
  • Disables handling of push triggers (handle_push_trigger = false).
  • Keeps the review agent enabled and publishing output.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .pr_agent.toml
@coderabbitai

coderabbitai Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 487ba959-e545-47fd-984c-165ccfb08c9d

📥 Commits

Reviewing files that changed from the base of the PR and between 62c41fa and b0e9a15.

📒 Files selected for processing (1)
  • .pr_agent.toml

Walkthrough

A new .pr_agent.toml configuration file is added for the Qodo code review tool. The file contains two configuration sections: a github_app section that disables automatic PR commands by setting pr_commands to an empty list and prevents trigger events on push by setting handle_push_trigger to false; and a review_agent section that enables the review agent with enabled = true and configures output publishing with publish_output = true. The change adds 12 lines of configuration without modifying any existing code logic.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is minimal and vague. It states 'Disable Qodo by default' but lacks context, motivation, or any of the template structure required by the repository. Expand the description to explain why this configuration change is needed, what behavior changes, and any relevant context or related issues (e.g., Resolves #issue_number).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Added Qodo configuration' accurately describes the main change—adding a new .pr_agent.toml configuration file for Qodo.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch victor/qodo-config

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@lukeheath
lukeheath merged commit e6894eb into main Apr 16, 2026
8 checks passed
@lukeheath
lukeheath deleted the victor/qodo-config branch April 16, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants