Skip to content

fix(#3656): skip review agent for Renovate bot PRs - #3657

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/3656-skip-renovate-reviews
Open

fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/3656-skip-renovate-reviews

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

What:

Add REVIEW_SKIP_AUTHORS: "renovate[bot]" to both the env.runner and env.sandbox sections of .fullsend/customized/harness/review.yaml.

Why:

Renovate bot PRs in this repo only touch go.mod/go.sum files, and the review agent consistently produces the same informational findings on every rebase. PR #3564 was rebased 8+ times over 7 weeks, triggering 8 full opus/high-effort reviews costing ~$29 total with identical non-actionable findings each time. CI checks (build, test, lint) already validate these dependency changes independently.

This uses the existing REVIEW_SKIP_AUTHORS mechanism (implemented in fullsend-ai/agents#368) to bypass review dispatch for Renovate-authored PRs, matching the pattern already established by REVIEW_FINDING_SEVERITY_THRESHOLD tuning in this file.

Tickets:


Closes #3656

Post-script verification

  • Branch is not main/master (agent/3656-skip-renovate-reviews)
  • Secret scan passed (gitleaks — c376db9d5f1a68668bc911ff78aad02a4e673ab5..HEAD)
  • PR body secret scan passed (gitleaks — no-git)
  • Final PR description secret scan passed (gitleaks — no-git)

Add REVIEW_SKIP_AUTHORS: "renovate[bot]" to both the runner and
sandbox env sections of the review harness config. This uses the
existing pre-review skip mechanism to bypass review dispatch for
Renovate-authored PRs, which only touch go.mod/go.sum and
consistently produce the same informational findings.

PR #3564 (Renovate dependency update) was rebased 8+ times over
7 weeks, triggering 8 full opus/high-effort reviews costing ~$29
total -- all producing identical non-actionable findings. CI checks
(build, test, lint) already validate these changes independently.

Note: pre-commit hooks could not run (infrastructure failure:
certificate verification). The only configured hook
(check-commit-message) validates commit messages, not file content.

Closes #3656

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@fullsend-ai-coder fullsend-ai-coder Bot added the ready-for-review Triggers review agent dispatch label Oct 29, 2026
@fullsend-ai-coder
fullsend-ai-coder Bot requested a review from a team as a code owner October 29, 2026 12:33
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 5a4af1e8-7cc1-4206-8464-f3b8c4b70c4b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

@qodo-for-conforma

Copy link
Copy Markdown

PR Summary by Qodo

Skip review agent dispatch for Renovate PRs

🐞 Bug fix ⚙️ Configuration changes 🕐 Less than 5 minutes


AI Description

• Skip review dispatch for Renovate-authored PRs to avoid repeated, non-actionable reviews.
• Set the existing author-skip option in both runner and sandbox environments.
Diagram

graph TD
C["Review config"] --> R["Runner env"] --> G{"Author skipped?"}
C --> S["Sandbox env"] --> G
G -->|"Renovate bot"| X["No review"]
G -->|"Other authors"| A["Review agent"]
Loading
High-Level Assessment

Use the existing author-skip setting as proposed. It avoids adding review logic or changing CI for dependency PRs.

Files changed (1) +2 / -0

Other (1) +2 / -0
review.yamlExclude Renovate from review dispatch +2/-0

Exclude Renovate from review dispatch

• Adds REVIEW_SKIP_AUTHORS for renovate[bot] to both runner and sandbox environments. This uses the existing skip mechanism to prevent repeated reviews of Renovate PRs.

.fullsend/customized/harness/review.yaml

@qodo-for-conforma

Copy link
Copy Markdown

Code Review by Qodo

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

Great, no issues found!

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

Powered by Qodo

@fullsend-ai-review

Copy link
Copy Markdown

🤖 Review · ⚠️ Cancelled · Ended 12:35 PM UTC

Commit: a2df401 · View workflow run →

@codecov

codecov Bot commented Oct 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Flag Coverage Δ
acceptance 54.61% <ø> (ø)
generative 12.22% <ø> (ø)
integration 23.62% <ø> (ø)
unit 72.29% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fullsend-ai-review fullsend-ai-review Bot added the risk/low PR risk: low label Oct 29, 2026
@fullsend-ai-review

Copy link
Copy Markdown

<!-- fullsend:risk-assessment -->
Risk Assessment: low (1/5)

Details

Minimal-risk config-only change: 1 file, 2 lines added by a bot to skip Renovate reviews, with a well-scoped linked issue, no security or protected-path impact, and no source code touched.

@fullsend-ai-review

Copy link
Copy Markdown

Looks good to me

<!-- fullsend:review-findings-v2:eyJ2ZXJzaW9uIjoyLCJmaW5kaW5ncyI6W119 -->

@fullsend-ai-review fullsend-ai-review Bot added the requires-manual-review Review requires human judgment label Oct 29, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review Triggers review agent dispatch requires-manual-review Review requires human judgment risk/low PR risk: low size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configure REVIEW_SKIP_AUTHORS to skip Renovate bot reviews

0 participants