From 86f45c22741841db64937bc98b9a5d211340f2e4 Mon Sep 17 00:00:00 2001 From: Tam Nhu Tran Date: Sat, 28 Mar 2026 21:09:56 -0400 Subject: [PATCH] style(ai-review): add emojis to orchestrator output format sections --- .github/review-prompt.md | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/.github/review-prompt.md b/.github/review-prompt.md index 4fa3fb90..5cdce109 100644 --- a/.github/review-prompt.md +++ b/.github/review-prompt.md @@ -17,36 +17,36 @@ Your job is to merge their findings into a single, unified review comment. ## Output Format -### Summary +### 📋 Summary 2-3 sentences: what the PR does and overall assessment. -### Findings +### 🔍 Findings -**High** (must fix before merge): +**🔴 High** (must fix before merge): - [source] file:line — description -**Medium** (should fix): +**🟡 Medium** (should fix): - [source] file:line — description -**Low** (track for follow-up): +**🟢 Low** (track for follow-up): - [source] file:line — description -### Security Checklist +### 🔒 Security Checklist (From security reviewer — copy table directly) -### CCS Compliance +### 📊 CCS Compliance (From CCS reviewer — copy table directly) -### Informational +### 💡 Informational Non-blocking observations from quality reviewer. -### What's Done Well +### ✅ What's Done Well 2-3 items max. OPTIONAL — skip if nothing stands out. -### Overall Assessment +### 🎯 Overall Assessment -**APPROVED** — zero High, zero security Medium, all CCS rules respected, tests exist. -**APPROVED WITH NOTES** — zero High, only non-security Medium/Low remain. -**CHANGES REQUESTED** — ANY High, OR security Medium, OR CCS violation, OR missing tests/docs. +**✅ APPROVED** — zero High, zero security Medium, all CCS rules respected, tests exist. +**⚠️ APPROVED WITH NOTES** — zero High, only non-security Medium/Low remain. +**❌ CHANGES REQUESTED** — ANY High, OR security Medium, OR CCS violation, OR missing tests/docs. When in doubt, choose CHANGES REQUESTED.