












@@ -169,8 +169,9 @@ Output only qualifying candidates, with: ref, surface, proof, cause, fix sketch,
169169- Start every PR review with 1-3 plain sentences explaining what the change does and why it matters. Put this before `Findings`.
170170- Then list findings first. If none, say `No blocking findings` or `No findings`.
171171- Always answer: bug/behavior being fixed, PR/issue URL and affected surface, provenance for regressions when traceable, and best-fix verdict.
172-- For bug/regression fixes, include a compact `Provenance:` line after cause/root-cause when a bounded history pass can identify it. Use `git log -S/-G`, `git blame`, linked PRs/issues, and tests; separate author, committer/merger, and current PR author when they differ.
173-- For any confirmed bug, run `git blame` on the implicated line(s) after identifying the root cause. Report who broke it and when by naming the merging maintainer for the blamed PR, not the commit author. Include the PR number; if no PR is traceable, include the blamed commit SHA/date instead of guessing a merger.
172+- For bug/regression fixes, include a compact `Provenance:` line after cause/root-cause when a bounded history pass can identify it. Use `git log -S/-G`, `git blame`, linked PRs/issues, and tests.
173+- Provenance must separate roles when they differ: blamed code author username, blamed PR merger/committer username, current PR author username, PR number, and date. Do not collapse them into one "introduced by" actor.
174+- For any confirmed bug, run `git blame` on the implicated line(s) after identifying the root cause. Report who broke it as the blamed PR merger/committer, and also name the blamed code author. Include the PR number; if no PR is traceable, include the blamed commit SHA/date and author username instead of guessing a merger.
174175- Phrase provenance as `introduced by`, `made visible by`, or `carried forward by`, with confidence (`clear`, `likely`, `unknown`). If unclear, say what evidence is missing instead of guessing. For features, docs, and refactors, use `Provenance: N/A` or omit it when no broken behavior is being fixed.
175176- Keep summaries compact, but include enough proof that the verdict is auditable without rereading the PR.
176177此內容由慣性聚合(RSS閱讀器)自動聚合整理,僅供閱讀參考。 原文來自 — 版權歸原作者所有。