@@ -170,31 +170,29 @@ Include findings that cannot be pinpointed to a single line (cross-cutting
170170concerns, architectural issues, dependency problems) in the review body:
171171
172172``` markdown
173- ## Staff Engineer Review
174-
175- ### Summary
173+ ## Summary
176174
177175<1-2 sentence overview of the changes and overall assessment>
178176
179- ### Findings
177+ ## Findings
180178
181- #### P0 — Blockers
179+ ### P0 — Blockers
182180
183181- ** Title** : description (file: line )
184182
185- #### P1 — High
183+ ### P1 — High
186184
187185- ** Title** : description (file: line )
188186
189- #### P2 — Medium
187+ ### P2 — Medium
190188
191189- ** Title** : description
192190
193- #### P3 — Low
191+ ### P3 — Low
194192
195193- ** Title** : description
196194
197- ### CI Status
195+ ## CI Status
198196
199197- fmt: PASS/FAIL
200198- clippy: PASS/FAIL
@@ -251,8 +249,9 @@ Output:
251249- Don't nitpick style that ` cargo fmt ` handles — focus on substance.
252250- Don't flag things that are correct but unfamiliar — verify before flagging.
253251- Cross-reference findings: if an issue appears in multiple places, group them.
254- - Do not include any byline, "Generated with" footer, or ` Co-Authored-By `
255- trailer in review comments.
252+ - Do not include any byline, "Generated with" footer, ` Co-Authored-By `
253+ trailer, or self-referential titles (e.g., "Staff Engineer Review") in
254+ review comments or the review body.
256255- If the diff is very large (>50 files), prioritize ` crates/common/ ` changes
257256 and new files over mechanical changes (Cargo.lock, generated code).
258257- Never submit a review without explicit user approval of the findings.
0 commit comments