請求代碼審查
促進超能力:code-reviewer子代理在問題級聯發生之前發現問題。
**核心原則:**早複習、常複習。
何時請求審查
強迫的:
-
子代理程式驅動開發中的每個任務之後
-
完成主要功能後
-
合併到主程式之前
可選但有價值:
-
當卡住時(新視角)
-
重構前(基線檢查)
-
修復複雜的錯誤後
如何請求
1.獲取git SHA:
BASE_SHA=$(git rev-parse HEAD~1) # or origin/main HEAD_SHA=$(git rev-parse HEAD)
2.調度代碼審閱者子代理:
使用具有超能力的任務工具:代碼審閱者類型,填寫模板code-reviewer.md
佔位符:
-
{WHAT_WAS_IMPLEMENTED}
-
你剛剛建造的
-
{PLAN_OR_REQUIREMENTS}
-
它應該做什麼
-
{BASE_SHA}
-
開始提交
-
{HEAD_SHA}
-
結束提交
-
{DESCRIPTION}
-
簡要總結
3.根據反饋採取行動:
-
立即修復關鍵問題
-
在繼續之前修復重要問題
-
注意稍後的小問題
-
如果審稿人錯了就退回(有推理)
例子
[Just completed Task 2: Add verification function]
You: Let me request code review before proceeding.
BASE_SHA=$(git log --oneline | grep "Task 1" | head -1 | awk '{print $1}') HEAD_SHA=$(git rev-parse HEAD)
[Dispatch superpowers:code-reviewer subagent] WHAT_WAS_IMPLEMENTED: Verification and repair functions for conversation index PLAN_OR_REQUIREMENTS: Task 2 from docs/plans/deployment-plan.md BASE_SHA: a7981ec HEAD_SHA: 3df7661 DESCRIPTION: Added verifyIndex() and repairIndex() with 4 issue types
[Subagent returns]: Strengths: Clean architecture, real tests Issues: Important: Missing progress indicators Minor: Magic number (100) for reporting interval Assessment: Ready to proceed
You: [Fix progress indicators] [Continue to Task 3]
與工作流程集成
子代理驅動的開發:
-
每項任務後回顧
-
在問題複雜化之前發現問題
-
在進行下一個任務之前修復
執行計劃:
-
每批後回顧(3個任務)
-
獲取反饋、申請、繼續
臨時開發:
-
合併前審查
-
卡住時回顧
危險信號
絕不:
-
跳過評論,因為“很簡單”
-
忽略關鍵問題
-
繼續處理未解決的重要問題
-
用有效的技術回饋進行爭論
如果審稿人錯誤:
-
用技術推理進行反擊
-
顯示證明其有效的程式碼/測試
-
要求澄清
請參考模板:requesting-code-review/code-reviewer.md