Commit 070bfe2101a2

Vincent Demeester <vincent@sbr.pm>
2026-02-17 22:43:42
feat(pi): omit priority tags when posting reviews
Added instruction to the review rubric to keep P0-P3 tags for local output only and drop them when posting to GitHub, using natural prose for severity instead.
main
1 parent f9c7986
Changed files (1)
dots
pi
agent
extensions
review.ts
dots/pi/agent/extensions/review.ts
@@ -448,7 +448,11 @@ Provide your findings in a clear, structured format:
 5. Ignore trivial style issues unless they obscure meaning or violate documented standards.
 6. Do not generate a full PR fix — only flag issues and optionally provide short suggestion blocks.
 
-Output all findings the author would fix if they knew about them. If there are no qualifying findings, explicitly state the code looks good. Don't stop at the first finding - list every qualifying issue.`;
+Output all findings the author would fix if they knew about them. If there are no qualifying findings, explicitly state the code looks good. Don't stop at the first finding - list every qualifying issue.
+
+## Posting reviews to GitHub
+
+Priority tags ([P0]-[P3]) are for local consumption only. When posting a review to GitHub (via \`gh pr review\` or similar), **omit the priority tags** and describe severity naturally in prose. Keep the posted review concise and conversational — GitHub comments should read like a human reviewer, not a structured report.`;
 
 async function loadProjectReviewGuidelines(cwd: string): Promise<string | null> {
 	let currentDir = path.resolve(cwd);