Skip to content

fix(ci): switch qodo from /review to /agentic_review#2568

Merged
ValentaTomas merged 2 commits into
mainfrom
fix/qodo-agentic-review
May 5, 2026
Merged

fix(ci): switch qodo from /review to /agentic_review#2568
ValentaTomas merged 2 commits into
mainfrom
fix/qodo-agentic-review

Conversation

@ValentaTomas
Copy link
Copy Markdown
Member

@ValentaTomas ValentaTomas commented May 5, 2026

Switch qodo PR-Agent from /review (removed after 2026-05-31) to /agentic_review, and migrate the style guidance from [pr_reviewer] to [review_agent] (issues_user_guidelines + compliance_user_guidelines) — the old section keys don't apply to the new command.

/review is deprecated by qodo and removed after 2026-05-31.
@cursor
Copy link
Copy Markdown

cursor Bot commented May 5, 2026

PR Summary

Low Risk
Low risk because it only changes PR-Agent configuration and review output guidelines; the main risk is misconfiguration causing PR automation to stop running or to post no comments.

Overview
Switches Qodo PR-Agent from /review to /agentic_review and replaces the old [pr_reviewer] settings with [review_agent] guidelines that suppress summaries and enforce terse, no-output-when-clean behavior, which may result in fewer/no automated review comments if the configuration doesn’t match expected keys.

Reviewed by Cursor Bugbot for commit 156af91. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix prepared a fix for the issue found in the latest run.

  • ✅ Fixed: Configuration section mismatch after command switch
    • Migrated configuration from [pr_reviewer] section to [review_agent] section with extra_instructions split into issues_user_guidelines and compliance_user_guidelines per Qodo documentation.

Create PR

Or push these changes by commenting:

@cursor push 4cf2c31e6f
Preview (4cf2c31e6f)
diff --git a/.pr_agent.toml b/.pr_agent.toml
--- a/.pr_agent.toml
+++ b/.pr_agent.toml
@@ -8,21 +8,8 @@
 # /review is deprecated by qodo and removed after 2026-05-31.
 pr_commands = ["/agentic_review"]
 
-[pr_reviewer]
-require_score_review = false
-require_tests_review = false
-require_estimate_effort_to_review = false
-require_can_be_split_review = false
-require_security_review = false
-require_ticket_analysis_review = false
-enable_review_labels_security = false
-enable_review_labels_effort = false
-inline_code_comments = true
-num_code_suggestions = 3
-persistent_comment = true
-final_update_message = false
-enable_help_text = false
-extra_instructions = """
+[review_agent]
+issues_user_guidelines = """
 Output style rules (strict):
 - Be terse. One short paragraph per finding, no preamble.
 - Only report concrete bugs, correctness issues, or rule violations. Do not summarize the PR.
@@ -31,6 +18,15 @@
 - No Qodo branding, logos, dividers, or footer links.
 - If nothing to flag, post nothing.
 """
+compliance_user_guidelines = """
+Output style rules (strict):
+- Be terse. One short paragraph per finding, no preamble.
+- Only report concrete bugs, correctness issues, or rule violations. Do not summarize the PR.
+- No headers, no bullet lists, no tables, no diagrams, no code-fence wrapping unless quoting code.
+- No emojis. No severity badges. No tags like "Bug", "Enhancement", "Correctness", "Maintainability".
+- No Qodo branding, logos, dividers, or footer links.
+- If nothing to flag, post nothing.
+"""
 
 [pr_description]
 publish_description_as_comment = false

You can send follow-ups to the cloud agent here.

Reviewed by Cursor Bugbot for commit 4788668. Configure here.

Comment thread .pr_agent.toml
@codecov
Copy link
Copy Markdown

codecov Bot commented May 5, 2026

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
2355 4 2351 5
View the full list of 4 ❄️ flaky test(s)
github.com/e2b-dev/infra/tests/integration/internal/tests/api/sandboxes::TestUpdateNetworkConfig

Flake rate in main: 57.14% (Passed 6 times, Failed 8 times)

Stack Traces | 104s run time
=== RUN   TestUpdateNetworkConfig
=== PAUSE TestUpdateNetworkConfig
=== CONT  TestUpdateNetworkConfig
--- FAIL: TestUpdateNetworkConfig (103.91s)
github.com/e2b-dev/infra/tests/integration/internal/tests/api/sandboxes::TestUpdateNetworkConfig/pause_resume_preserves_allow_internet_access_false

Flake rate in main: 50.00% (Passed 6 times, Failed 6 times)

Stack Traces | 1.47s run time
=== RUN   TestUpdateNetworkConfig/pause_resume_preserves_allow_internet_access_false
Executing command curl in sandbox i93vq9hcg184u2g0ivnz8
    sandbox_network_update_test.go:372: Command [curl] output: event:{start:{pid:1365}}
    sandbox_network_update_test.go:372: Command [curl] output: event:{end:{exit_code:35 exited:true status:"exit status 35" error:"exit status 35"}}
Executing command curl in sandbox i93vq9hcg184u2g0ivnz8
    sandbox_network_update_test.go:372: Command [curl] output: event:{start:{pid:1366}}
    sandbox_network_update_test.go:372: Command [curl] output: event:{end:{exit_code:35 exited:true status:"exit status 35" error:"exit status 35"}}
    sandbox_network_update_test.go:391: Command [curl] output: event:{start:{pid:1367}}
    sandbox_network_update_test.go:391: Command [curl] output: event:{data:{stdout:"HTTP/2 302 \r\nx-content-type-options: nosniff\r\nlocation: https://dns.google/\r\ndate: Tue, 05 May 2026 02:02:03 GMT\r\ncontent-type: text/html; charset=UTF-8\r\nserver: HTTP server (unknown)\r\ncontent-length: 216\r\nx-xss-protection: 0\r\nx-frame-options: SAMEORIGIN\r\nalt-svc: h3=\":443\"; ma=2592000,h3-29=\":443\"; ma=2592000\r\n\r\n"}}
    sandbox_network_update_test.go:391: Command [curl] output: event:{end:{exited:true status:"exit status 0"}}
    sandbox_network_update_test.go:391: Command [curl] completed successfully in sandbox i93vq9hcg184u2g0ivnz8
    sandbox_network_update_test.go:391: 
        	Error Trace:	.../api/sandboxes/sandbox_network_out_test.go:74
        	            				.../api/sandboxes/sandbox_network_update_test.go:60
        	            				.../api/sandboxes/sandbox_network_update_test.go:391
        	Error:      	An error is expected but got nil.
        	Test:       	TestUpdateNetworkConfig/pause_resume_preserves_allow_internet_access_false
        	Messages:   	https://8.8.8.8 should be blocked
--- FAIL: TestUpdateNetworkConfig/pause_resume_preserves_allow_internet_access_false (1.47s)
github.com/e2b-dev/infra/tests/integration/internal/tests/api/templates::TestSandboxCreateWithTag

Flake rate in main: 66.67% (Passed 1 times, Failed 2 times)

Stack Traces | 301s run time
=== RUN   TestSandboxCreateWithTag
=== PAUSE TestSandboxCreateWithTag
=== CONT  TestSandboxCreateWithTag
    template_tags_test.go:116: Build timeout exceeded
--- FAIL: TestSandboxCreateWithTag (300.92s)
github.com/e2b-dev/infra/tests/integration/internal/tests/api/templates::TestTemplateBuildCOPY

Flake rate in main: 66.67% (Passed 1 times, Failed 2 times)

Stack Traces | 301s run time
=== RUN   TestTemplateBuildCOPY
=== PAUSE TestTemplateBuildCOPY
=== CONT  TestTemplateBuildCOPY
    build_template_test.go:134: test-ubuntu-copy: [info] Building template r3fyy2gtjph3879ujd1w/a88f3ae1-686f-4d38-87e8-feaa0b0d7d91
    build_template_test.go:134: test-ubuntu-copy: [info] [base] FROM ubuntu:24.04 [33c3d1f55ae33f402733ec22d91cfbce7572f860c9bb4d15b497c19d978167b5]
    build_template_test.go:134: test-ubuntu-copy: [info] Base Docker image size: 30 MB
    build_template_test.go:134: test-ubuntu-copy: [info] Creating file system and pulling Docker image
    build_template_test.go:134: test-ubuntu-copy: [info] Uncompressing layer sha256:b40150c1c2717d324cdb17278c8efdfa4dfcd2ffe083e976f0bcedf31115f081 30 MB
    build_template_test.go:134: test-ubuntu-copy: [info] Uncompressing layer sha256:4d3ad5cc0bafe8511ed3591e0680169281e407c1e38e5d58bb35e756f954e1e2 12 MB
    build_template_test.go:134: test-ubuntu-copy: [info] Uncompressing layer sha256:8c4b1b28875140ed3abacaf16ad0d696f6bef912f52d2148f261a23e3349465b 168 B
    build_template_test.go:134: test-ubuntu-copy: [info] Layers extracted
    build_template_test.go:134: test-ubuntu-copy: [info] Root filesystem structure: bin, boot, dev, etc, home, lib, lib64, media, mnt, opt, proc, root, run, sbin, srv, sys, tmp, usr, var
    build_template_test.go:134: test-ubuntu-copy: [info] Provisioning sandbox template
    build_template_test.go:1153: 
        	Error Trace:	.../api/templates/build_template_test.go:97
        	            				.../api/templates/build_template_test.go:1153
        	Error:      	Received unexpected error:
        	            	Get "http://localhost:.../builds/a88f3ae1-686f-4d38-87e8-feaa0b0d7d91/status?level=info&logsOffset=10": context deadline exceeded
        	Test:       	TestTemplateBuildCOPY
--- FAIL: TestTemplateBuildCOPY (300.59s)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

The .pr_agent.toml configuration was updated to replace the deprecated /review command with /agentic_review. I have no feedback to provide.

@ValentaTomas ValentaTomas marked this pull request as ready for review May 5, 2026 01:50
@ValentaTomas ValentaTomas enabled auto-merge (squash) May 5, 2026 01:50
@qodo-code-review
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Warning

/review is deprecated. Use /agentic_review instead (removed after 2026-05-31).

Here are some key observations to aid the review process:

⚡ No major issues detected

The old [pr_reviewer] section keys (require_*, num_code_suggestions,
extra_instructions, etc.) only apply to the deprecated /review command.
/agentic_review reads its style guidance from [review_agent] using
issues_user_guidelines (Issues Agent) and compliance_user_guidelines
(Compliance Agent).
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, reopen this pull request to trigger a review.

@ValentaTomas ValentaTomas merged commit d59770a into main May 5, 2026
47 checks passed
@ValentaTomas ValentaTomas deleted the fix/qodo-agentic-review branch May 5, 2026 02:25
@ValentaTomas ValentaTomas requested review from djeebus and matthewlouisbrockman and removed request for jakubno May 5, 2026 02:25
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.

2 participants