Skip to content

chore: add .pr_agent.toml to configure Qodo reviews#12264

Merged
zkochan merged 1 commit into
pnpm:mainfrom
gvago:add-pr-agent-config
Jun 8, 2026
Merged

chore: add .pr_agent.toml to configure Qodo reviews#12264
zkochan merged 1 commit into
pnpm:mainfrom
gvago:add-pr-agent-config

Conversation

@gvago

@gvago gvago commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

As discussed with a maintainer, this adds a .pr_agent.toml so Qodo runs an agentic review in addition to the summary it already posts.

What it configures:

  • On PR open: /agentic_describe + /agentic_review
  • On push: /agentic_review (enabled via handle_push_trigger)
  • Skips bot-authored PRs (dependabot, renovate) to cut noise. Qodo auto-detects most bots; these are listed explicitly to be safe.
  • Commented-out ignore_pr_title and review_agent user-guideline keys so maintainers can tweak later.

Summary by CodeRabbit

Release Notes

  • Chores
    • Added PR automation configuration file with settings for automated commands on PR open, push trigger handling, and bot account exclusions.

@gvago gvago requested a review from zkochan as a code owner June 8, 2026 11:53
@qodo-free-for-open-source-projects

qodo-free-for-open-source-projects Bot commented Jun 8, 2026

Copy link
Copy Markdown

Code Review by Qodo

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

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Qodo Logo

@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

A new .pr_agent.toml configuration file is added to automate PR workflows. The file configures commands to execute on PR open and push events, filters out bot-authored PRs, and includes placeholder review agent guidelines.

Changes

PR Agent Configuration

Layer / File(s) Summary
PR automation configuration
.pr_agent.toml
Defines PR automation triggers: /agentic_describe and /agentic_review commands run on PR open, and /agentic_review reruns on push via handle_push_trigger = true. Bot authors dependabot[bot] and renovate[bot] are excluded. Review agent guideline placeholders are present as commented-out keys.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

A config file hops into place,
With bot-filters and PR embrace,
Commands dance on open and push,
Automation flows in a gentle gush,
The workflow now runs with newfound grace! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a .pr_agent.toml configuration file to set up Qodo reviews, which is the primary objective of this pull request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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

Copy link
Copy Markdown

Review Summary by Qodo

Configure Qodo agentic PR reviews

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Add Qodo PR review commands
• Enable review reruns on pushes
• Ignore common bot authors
• Leave optional rules commented
Diagram
flowchart LR
  A["Pull request opened"] -- "run" --> B["/agentic_describe"]
  A -- "run" --> C["/agentic_review"]
  D["New push to PR"] -- "rerun" --> C
  E["Bot-authored PRs"] -- "skip" --> F["No automated review"]

Loading

Grey Divider

File Changes

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

Add Qodo review automation settings

• Configures automatic agentic_describe and agentic_review on PR open
• Enables handle_push_trigger so pushes rerun agentic_review
• Skips reviews for dependabot[bot] and renovate[bot]
• Leaves optional title and agent guideline overrides commented out

.pr_agent.toml


Grey Divider

Qodo Logo

@coderabbitai coderabbitai 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.

🧹 Nitpick comments (1)
.pr_agent.toml (1)

35-35: ⚡ Quick win

Consider adding a trailing newline.

The file appears to end without a trailing newline character. Many tools and style guides expect text files to end with a newline.

📝 Add trailing newline

Ensure the file ends with a newline character (this is typically handled automatically by most editors when configured properly).

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.pr_agent.toml at line 35, Add a trailing newline at the end of the
.pr_agent.toml file by ensuring the file ends with a single newline character
(update the file in your editor or save with a newline at EOF); optionally
enable your editor’s "insert final newline" setting or add a .gitattributes rule
to enforce it so future edits include the newline.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In @.pr_agent.toml:
- Line 35: Add a trailing newline at the end of the .pr_agent.toml file by
ensuring the file ends with a single newline character (update the file in your
editor or save with a newline at EOF); optionally enable your editor’s "insert
final newline" setting or add a .gitattributes rule to enforce it so future
edits include the newline.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 46bf1cf0-d258-4ca0-a9dc-1c65a19ad73d

📥 Commits

Reviewing files that changed from the base of the PR and between 29a496a and 90ab308.

📒 Files selected for processing (1)
  • .pr_agent.toml
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: Compile & Lint
  • GitHub Check: Analyze (javascript)
🧰 Additional context used
🧠 Learnings (7)
📓 Common learnings
Learnt from: CR
Repo: pnpm/pnpm PR: 0
File: pnpr/AGENTS.md:0-0
Timestamp: 2026-05-29T18:03:24.797Z
Learning: Read the monorepo-wide AGENTS.md (../AGENTS.md) first for GitHub PR workflow, signing agent-authored content, conventional commit messages, code-reuse philosophy, and test failure handling
Learnt from: zkochan
Repo: pnpm/pnpm PR: 11640
File: pacquet/AGENTS.md:1-6
Timestamp: 2026-05-14T17:38:39.470Z
Learning: In the pnpm/pnpm repository, `AGENTS.md` and `pacquet/AGENTS.md` are agent guides in the agentsmd.net convention — they contain instructions FOR AI coding tools (Claude Code, Codex, Cursor, etc.) working on the repo. They are NOT documentation about software agents defined by the project. Do not apply "document agent capabilities / inputs / outputs / integration points" review rules to these files; such rules are for agentic-framework documentation, not for repo-level AI-tool guides.
📚 Learning: 2026-05-29T18:03:24.797Z
Learnt from: CR
Repo: pnpm/pnpm PR: 0
File: pnpr/AGENTS.md:0-0
Timestamp: 2026-05-29T18:03:24.797Z
Learning: Read the monorepo-wide AGENTS.md (../AGENTS.md) first for GitHub PR workflow, signing agent-authored content, conventional commit messages, code-reuse philosophy, and test failure handling

Applied to files:

  • .pr_agent.toml
📚 Learning: 2026-05-14T17:38:39.470Z
Learnt from: zkochan
Repo: pnpm/pnpm PR: 11640
File: pacquet/AGENTS.md:1-6
Timestamp: 2026-05-14T17:38:39.470Z
Learning: In the pnpm/pnpm repository, `AGENTS.md` and `pacquet/AGENTS.md` are agent guides in the agentsmd.net convention — they contain instructions FOR AI coding tools (Claude Code, Codex, Cursor, etc.) working on the repo. They are NOT documentation about software agents defined by the project. Do not apply "document agent capabilities / inputs / outputs / integration points" review rules to these files; such rules are for agentic-framework documentation, not for repo-level AI-tool guides.

Applied to files:

  • .pr_agent.toml
📚 Learning: 2026-05-14T15:23:12.794Z
Learnt from: CR
Repo: pnpm/pnpm PR: 0
File: pacquet/GEMINI.md:0-0
Timestamp: 2026-05-14T15:23:12.794Z
Learning: Applies to pacquet/**/AGENTS.md : Document agent responsibilities, capabilities, and interactions in AGENTS.md file

Applied to files:

  • .pr_agent.toml
📚 Learning: 2026-05-14T15:23:09.244Z
Learnt from: CR
Repo: pnpm/pnpm PR: 0
File: pacquet/CLAUDE.md:0-0
Timestamp: 2026-05-14T15:23:09.244Z
Learning: Applies to pacquet/**/AGENTS.md : Document agent functionality and interactions in AGENTS.md

Applied to files:

  • .pr_agent.toml
📚 Learning: 2026-05-01T19:55:05.136Z
Learnt from: CR
Repo: pnpm/pnpm PR: 0
File: CLAUDE.md:0-0
Timestamp: 2026-05-01T19:55:05.136Z
Learning: Document agent functionality and capabilities in AGENTS.md

Applied to files:

  • .pr_agent.toml
📚 Learning: 2026-05-20T20:41:26.759Z
Learnt from: zkochan
Repo: pnpm/pnpm PR: 11773
File: pacquet/crates/resolving-tarball-resolver/src/tarball_resolver.rs:115-117
Timestamp: 2026-05-20T20:41:26.759Z
Learning: In pacquet's Rust pnpm port, keep the `is_http_url` behavior exactly aligned with upstream pnpm: when checking whether an input is an HTTP(S) URL, use `bare.starts_with("http:") || bare.starts_with("https:")` rather than `"http://"` / `"https://"`. Do not “fix” this to a more typical `http://` form; pacquet’s cardinal rule is to match pnpm semantics byte-for-byte. Non-URL/malformed inputs are later rejected downstream by `reqwest::Url::parse` (surfacing as a `ResolveError`), so this intentional prefix check is required for compatibility.

Applied to files:

  • .pr_agent.toml
🔇 Additional comments (3)
.pr_agent.toml (3)

16-27: LGTM!


29-34: LGTM!


1-14: Review: Qodo/PR-Agent automation keys look correct

  • pr_commands, handle_push_trigger, and push_commands are valid under the provider table (e.g., [github_app]); push_commands requires handle_push_trigger = true, which matches this config.

@zkochan zkochan merged commit 9777566 into pnpm:main Jun 8, 2026
7 checks passed
@welcome

welcome Bot commented Jun 8, 2026

Copy link
Copy Markdown

Congrats on merging your first pull request! 🎉🎉🎉

@zkochan

zkochan commented Jun 8, 2026

Copy link
Copy Markdown
Member

Thanks!

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