Repository navigation
fix(#3656): skip review agent for Renovate bot PRs - #3657
fullsend-ai-coder[bot] wants to merge 1 commit into
Conversation
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>
|
Important Review skippedBot user detected. To trigger a single review, invoke the
|
PR Summary by QodoSkip review agent dispatch for Renovate PRs
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
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 reviewPowered by Qodo |
|
🤖 Review · Commit: |
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
<!-- fullsend:risk-assessment --> DetailsMinimal-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. |
|
Looks good to me <!-- fullsend:review-findings-v2:eyJ2ZXJzaW9uIjoyLCJmaW5kaW5ncyI6W119 --> |
What:
Add
REVIEW_SKIP_AUTHORS: "renovate[bot]"to both theenv.runnerandenv.sandboxsections of.fullsend/customized/harness/review.yaml.Why:
Renovate bot PRs in this repo only touch
go.mod/go.sumfiles, 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_AUTHORSmechanism (implemented in fullsend-ai/agents#368) to bypass review dispatch for Renovate-authored PRs, matching the pattern already established byREVIEW_FINDING_SEVERITY_THRESHOLDtuning in this file.Tickets:
Closes #3656
Post-script verification
agent/3656-skip-renovate-reviews)c376db9d5f1a68668bc911ff78aad02a4e673ab5..HEAD)