

























@@ -134,6 +134,22 @@ Loop:
134134135135Output only qualifying candidates, with: ref, surface, proof, cause, fix sketch, why small, expected test/gate. If none qualify, say so; do not pad.
136136137+## Structure PR review output
138+139+- Start every PR review with 1-3 plain sentences explaining what the change does and why it matters. Put this before `Findings`.
140+- Then list findings first. If none, say `No blocking findings` or `No findings`.
141+- Always answer: bug/behavior being fixed, PR/issue URL and affected surface, and best-fix verdict.
142+- Keep summaries compact, but include enough proof that the verdict is auditable without rereading the PR.
143+144+## Read beyond the diff
145+146+- Review the surrounding code path, not just changed lines. Open the caller, callee, data contracts, adjacent tests, and owner module.
147+- For large-codebase PRs, sample enough related files to understand the runtime boundary before deciding. Default to more code reading when the change touches agents, gateway, plugins, auth, sessions, process, config, or provider/runtime seams.
148+- Compare the PR against current `origin/main` behavior. Check whether recent main already changed the same surface.
149+- Dependency-backed behavior: MUST read upstream docs/source/types before judging API use, defaults, output shapes, errors, timeouts, memory behavior, or compatibility. Do not assume dependency contracts from memory or PR text.
150+- Judge solution quality, not only correctness. Ask whether the PR is the clean owner-boundary fix or a wart/workaround that should be replaced by a small refactor, moved seam, contract change, or deletion of duplicate logic.
151+- Mention the main files read when the verdict depends on code-path evidence.
152+137153## Enforce the bug-fix evidence bar
138154139155- Never merge a bug-fix PR based only on issue text, PR text, or AI rationale.
此内容由惯性聚合(RSS阅读器)自动聚合整理,仅供阅读参考。 原文来自 — 版权归原作者所有。