Skip to content

Commit 26f0fbf

Browse files
ValentaTomasValentaTomas
authored
ci: quiet down qodo and codex pr review noise (#2559)
Configures qodo-merge via `.pr_agent.toml` to skip the PR walkthrough/score/effort/labels and only post inline findings, and adds `AGENTS.md` to steer Codex toward terse, findings-only reviews. Goal: kill the auto-summary blocks and branding, keep the actual bug reports.
1 parent 92ea305 commit 26f0fbf

5 files changed

Lines changed: 109 additions & 2 deletions

File tree

‎.gemini/config.yaml‎

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
have_fun: false
2+
3+
code_review:
4+
disable: false
5+
# Only surface high-severity findings; drop nits and lower-severity suggestions.
6+
comment_severity_threshold: HIGH
7+
# Cap the number of review comments per PR to avoid noise floods.
8+
max_review_comments: 10
9+
pull_request_opened:
10+
help: false
11+
# Suppress the auto PR summary. We rely on the author's description.
12+
summary: false
13+
code_review: true
14+
include_drafts: true
15+
16+
ignore_patterns:
17+
- "**/*.gen.go"
18+
- "**/*.sql.go"
19+
- "**/*.pb.go"
20+
- "**/mocks/**"
21+
- "**/testdata/**"
22+
- "**/*.md"

‎.gemini/styleguide.md‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# Review style guide
2+
3+
Output style rules (strict, override defaults):
4+
5+
- Only report concrete bugs, regressions, or correctness issues. Never summarize the PR, the diff, or what changed — no opening paragraph, no "Code Review" header, no overview, no recap.
6+
- Skip the review entirely if there are no critical findings.
7+
- One short paragraph per finding. No preamble, no closing remark.
8+
- No headers, no bullet lists, no tables, no diagrams.
9+
- No emojis. No severity tags or labels.
10+
- No branding or footer lines.
11+
12+
## Scope
13+
14+
- Skip style/nit comments — `golangci-lint` covers those.
15+
- Skip test-coverage comments — Codecov covers those.
16+
- Focus on: race conditions, nil-deref, error handling, auth/authz, request routing, resource leaks, SQL/migration correctness, and gRPC/proto compatibility.

‎.pr_agent.toml‎

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
[config]
2+
publish_output = true
3+
publish_output_progress = false
4+
verbosity_level = 1
5+
6+
[github_app]
7+
# Only run /review. Drops the /describe walkthrough, diagram, and file-changes summary.
8+
pr_commands = ["/review"]
9+
10+
[pr_reviewer]
11+
require_score_review = false
12+
require_tests_review = false
13+
require_estimate_effort_to_review = false
14+
require_can_be_split_review = false
15+
require_security_review = false
16+
require_ticket_analysis_review = false
17+
enable_review_labels_security = false
18+
enable_review_labels_effort = false
19+
inline_code_comments = true
20+
num_code_suggestions = 3
21+
persistent_comment = true
22+
final_update_message = false
23+
enable_help_text = false
24+
extra_instructions = """
25+
Output style rules (strict):
26+
- Be terse. One short paragraph per finding, no preamble.
27+
- Only report concrete bugs, correctness issues, or rule violations. Do not summarize the PR.
28+
- No headers, no bullet lists, no tables, no diagrams, no code-fence wrapping unless quoting code.
29+
- No emojis. No severity badges. No tags like "Bug", "Enhancement", "Correctness", "Maintainability".
30+
- No Qodo branding, logos, dividers, or footer links.
31+
- If nothing to flag, post nothing.
32+
"""
33+
34+
[pr_description]
35+
publish_description_as_comment = false
36+
publish_labels = false
37+
enable_pr_type = false
38+
final_update_message = false
39+
enable_help_text = false

‎AGENTS.md‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
# AGENTS.md
2+
3+
Instructions for AI agents reviewing pull requests in this repository.
4+
5+
## Review guidelines
6+
7+
Output style rules (strict, override any defaults):
8+
9+
- Only report concrete bugs, regressions, or correctness issues. Do not summarize the PR, the diff, or what changed.
10+
- One short paragraph per finding. No preamble, no closing remark.
11+
- No headers, no bullet lists, no tables, no diagrams.
12+
- No emojis. No severity tags ("bug", "nit", "suggestion", "enhancement"). No labels.
13+
- No branding or footer lines.
14+
- If there are no real issues to flag, post no review at all.
15+
16+
## Scope
17+
18+
- Skip style/nit comments — `golangci-lint` covers those.
19+
- Skip test-coverage comments — Codecov covers those.
20+
- Focus on: race conditions, nil-deref, error handling, auth/authz, request routing, resource leaks, SQL/migration correctness, and gRPC/proto compatibility.

‎codecov.yml‎

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,22 @@
11
# Coverage signal only — never gates merges. Carries forward per-flag so
22
# shards that didn't run keep their last-known number instead of dropping to 0%.
3+
# Coverage info lives in the codecov/* status checks (click to expand).
4+
# The PR comment is kept (with minimal coverage layout) because Test
5+
# Analytics (failed + flaky tests) ships in the same comment.
36

47
coverage:
58
status:
69
project:
7-
default: { informational: true }
10+
default: { informational: true, only_pulls: true }
811
patch:
9-
default: { informational: true }
12+
default: { informational: true, only_pulls: true }
13+
14+
comment:
15+
layout: "condensed_header"
16+
hide_project_coverage: true
17+
require_changes: false
18+
require_base: false
19+
require_head: true
1020

1121
flag_management:
1222
default_rules:

0 commit comments

Comments
 (0)