對 HEAD 與使用者提供的固定點之間的 diff 做雙軸審查:
- 規範(軸)——程式碼是否符合此 repo 記錄的編碼規範?
- 規格(軸)——程式碼是否忠實實作原始 issue / 規格?
兩個軸都以平行子代理執行,讓它們不污染彼此的上下文,然後本技能匯總它們的發現。
Issue 追蹤器應該已經提供給你——如果 docs/agents/issue-tracker.md 缺失,執行 /setup-matt-pocock-skills。
流程
1. 釘住固定點
無論使用者說的固定點是什麼——一個 commit SHA、分支名稱、標籤、main、HEAD~5 等。如果他們沒指定,就問。
把 diff 指令記下來一次:git diff <fixed-point>...HEAD(三點,這樣比較對象是合併基點)。也要用 git log <fixed-point>..HEAD --oneline 記下 commit 清單。
進一步之前,確認固定點能解析(git rev-parse <fixed-point>)而且 diff 非空。壞的 ref 或空的 diff 應該在這裡失敗——而不是在兩個平行子代理內部。
2. 找出規格來源
依此順序尋找原始規格:
- commit 訊息中的 issue 引用(
#123、Closes #45、GitLab!67等)——透過docs/agents/issue-tracker.md中的工作流程擷取。 - 使用者以參數傳入的路徑。
docs/、specs/或.scratch/下符合分支名稱或功能的規格檔。- 如果什麼都沒找到,問使用者規格在哪。如果他們說沒有,規格子代理會跳過並回報「no spec available」。
3. 找出規範來源
repo 中任何記錄程式碼應如何撰寫的東西,例如 CODING_STANDARDS.md 或 CONTRIBUTING.md。
在 repo 記錄的任何東西之上,規範軸永遠攜帶下面這份壞味道基線——一組固定的 Fowler 壞味道(Refactoring,第 3 章),即使 repo 什麼都沒記錄也適用。兩條規則綁定它:
- repo 優先。 記錄在案的 repo 標準永遠勝出;當它背書基線會標記的某個東西時,抑制該壞味道。
- 永遠是判斷。 每個壞味道都是帶標籤的啟發式(「可能是 Feature Envy」),從來不是硬性違規——而且像這裡的任何標準一樣,跳過任何工具已經強制的東西。
每個壞味道讀成它是什麼 → 如何修;把它對上 diff:
- 神祕名稱(Mysterious Name)——函式、變數或型別的名字看不出它做什麼或存什麼。→ 重新命名;如果想不出誠實的名字,設計很模糊。
- 重複程式碼(Duplicated Code)——相同的邏輯形狀在變更中出現於多個 hunk 或檔案。→ 抽取共享形狀,兩邊都呼叫它。
- 特徵依附(Feature Envy)——方法觸及另一個物件的資料比自己的多。→ 把方法移到它所羨慕的資料上。
- 資料叢集(Data Clumps)——相同的幾個欄位或參數一直結伴同行(一個想出生的型別)。→ 把它們捆成一個型別,傳那個。
- 基本型別執著(Primitive Obsession)——用基本型別或字串代替某個值得擁有自己型別的領域概念。→ 給那個概念一個自己的小型別。
- 重複 switch(Repeated Switches)——同一型別上相同的
switch/if級聯在變更中重複出現。→ 用多型取代,或一個兩處共享的 map。 - 散彈槍手術(Shotgun Surgery)——一個邏輯變更迫使 diff 中許多檔案被散落編輯。→ 把一起變更的東西收攏到一個模組。
- 發散式變更(Divergent Change)——一個檔案或模組因幾個無關的理由被編輯。→ 拆分,讓每個模組只為一個理由變更。
- 投機性一般化(Speculative Generality)——為規格沒有的需求加入的抽象、參數或鉤子。→ 刪掉它;在真實需求出現前內嵌回去。
- 訊息鏈(Message Chains)——長
a.b().c().d()導覽,呼叫者不該依賴它。→ 把這趟走訪藏到第一個物件上的單一方法後面。 - 中間人(Middle Man)——類別或函式多半只是轉手往下委派。→ 砍掉它,直接呼叫真正的目標。
- 拒絕遺贈(Refused Bequest)——子類別或實作者忽略或覆寫它繼承的大部分東西。→ 放棄繼承,改用組合。
4. 平行啟動兩個子代理
發送單一訊息、兩個 Agent 工具呼叫。兩者都用 general-purpose 子代理。
規範子代理提示——包含:
- 完整的 diff 指令與 commit 清單。
- 你在第 3 步找到的規範來源檔案清單,加上第 3 步的壞味道基線完整貼上——子代理沒有其他途徑取得它。
- 簡報:「回報——在相關處逐檔案 / hunk——(a) diff 違反記錄在案標準的每一處:引用標準(檔案 + 規則);以及 (b) 你注意到的任何基線壞味道:指出名字並引用 hunk。區分硬性違規與判斷——記錄在案的標準違規可以是硬性的,但基線壞味道永遠是判斷,而且記錄在案的 repo 標準覆蓋基線。跳過任何工具已強制的東西。400 字以內。」
規格子代理提示——包含:
- diff 指令與 commit 清單。
- 規格的路徑或擷取的內容。
- 簡報:「回報:(a) 規格要求但缺失或部分實作的需求;(b) diff 中沒有被要求卻存在的行為(範圍蔓延);(c) 看起來已實作但實作看起來錯誤的需求。每個發現引用規格的原文。400 字以內。」
如果規格缺失,跳過規格子代理並在最終報告中註明。
5. 匯總
在 ## Standards 與 ## Spec 標題下呈現兩份報告,原樣或輕度清理。不要合併或重新排序發現——兩個軸刻意分開(見《為什麼兩軸》)。
以一行摘要結尾:每個軸的發現總數,以及每個軸內最嚴重的問題(如果有的話)。不要跨軸選一個總冠軍——那正是分離所要防止的重新排序。
為什麼兩軸
一個變更可以過一軸而掛另一軸:
- 遵循所有標準但實作錯誤的東西 → 規範過、規格掛。
- 確切做了 issue 要求但破壞專案慣例的程式碼 → 規格過、規範掛。
分開回報能阻止一個軸掩蓋另一個。