Pr Review Resolver

PR Review Resolver

互動式解決 PR review comment 中的未解決問題。

重要原則：

1. 永遠使用繁體中文（zh-TW） 與使用者溝通
2. 逐一處理每個問題，不要批次處理
3. 由使用者決定處理方式，不要自行決定

使用時機

當需要：

• 處理 PR review 中的未解決問題
• 解決先前 review session 的回饋
• 在 merge 前完成剩餘任務
• 討論如何處理特定的 review 發現

工作流程

步驟 1：取得 PR Review Comment（使用快取）

首先取得 PR 編號（使用 branch-map 快取）：

PR_NUMBER=$("${CLAUDE_PLUGIN_ROOT}/scripts/get-pr-number.sh")

此指令使用 branch-to-PR-number 快取（TTL 1 小時），快取未命中時才呼叫 GitHub API。

若無 PR，通知使用者並停止。

接著取得 review comment（使用本地快取）：

REVIEW_CONTENT=$(${CLAUDE_PLUGIN_ROOT}/scripts/cache-read-comment.sh "$PR_NUMBER")

此指令會先檢查本地快取，若快取不存在則從 GitHub 取得並建立快取。

若找不到 review comment（exit code 2），通知使用者先執行 pr-review-and-document skill。

步驟 2：解析未解決項目

從 review comment 中識別所有未解決項目：

未解決指標：

• ⚠️ 需要注意的項目（待處理）
• 🔴 阻擋性問題（必須在 merge 前解決）
• [Codex] 或 [Gemini] 來源的 ⚠️ / 🔴 details
• 表格中的 ⚠️ Pending 狀態
• 沒有 ✅ 或 ⏭️ 前綴的 details summary
• 「Action Plan」中沒有完成標記 [x] 的任務

已解決指標：

• ✅ 問題已解決（程式碼變更已完成）
• ⏭️ 刻意延後或不適用（標記為 Deferred / N/A / Kept）
• [x] 已勾選的 checkbox

來源判斷：

• [Codex] = Codex review pass 產生的 issue
• [Gemini] = Gemini Code Assist integration 產生的 issue
• 無來源 prefix 的 issue 視為 Claude issue（相容既有 comment）

步驟 3：逐一處理每個項目（最重要！）

關鍵要求：一次討論一個項目，但修復以背景子任務並行執行，不阻塞討論流程。

對於 每一個 未解決項目，依序執行：

3.1 清楚呈現問題

用繁體中文向使用者說明：

• 引用 review 中的問題描述
• 顯示 file:line 參考位置
• 解釋問題及其影響

3.2 閱讀實際原始碼

• 使用 Read 工具檢視被參考的檔案
• 驗證問題是否仍然存在
• 檢查是否已在最近的 commit 中修復

3.3 與使用者討論（必要！）

這是最重要的步驟。必須：

• 用繁體中文解釋可能的解決方案
• 如果有多種方法，以選項形式呈現
• 使用 AskUserQuestion 工具或 <options> 讓使用者選擇
• 等待使用者決定後才能繼續

選項範例：

這個問題有幾種處理方式：

1. **修復** - [說明修復方法]
2. **延後** - [說明為何可以延後]
3. **標記為 N/A** - [說明為何不適用]

<options>
<option>修復這個問題</option>
<option>延後到下個 PR</option>
<option>標記為不適用 (N/A)</option>
</options>

3.4 依使用者決定執行

修復路徑：

1. 確認目標檔案清單（從問題描述與原始碼分析中取得）
2. 檢查檔案衝突——目標檔案是否與進行中的背景子任務重疊：
   • 無衝突 → 立即啟動背景子任務
   • 有衝突 → 通知使用者：「⏳ 目標檔案 {file} 正在被子任務 {task_id}（{問題名稱}）修改中，等待完成後再啟動。」使用 TaskOutput 等待衝突子任務完成，然後再啟動新子任務
3. 用 Task tool（run_in_background: true）啟動背景子任務，prompt 必須包含：
   • 完整的問題描述（從 review comment 引用）
   • 目標檔案路徑與行號
   • 使用者選擇的修復方案
   • 足夠的 context 讓子任務可獨立執行（不依賴主對話）
4. 將子任務加入追蹤清單（在記憶體中維護）：{ task_id, 問題名稱, 目標檔案列表 }
5. 記錄暫定狀態 🔧 修復中

延後 / N/A 路徑（inline 處理，不需子任務）：

• 若使用者選擇 延後：記錄狀態為 ⏭️ Deferred，加上原因
• 若使用者選擇 標記 N/A：記錄狀態為 ⏭️ N/A，加上說明

3.5 在原始碼中記錄決策

對於標記為 N/A 或 Deferred 的項目，在相關原始碼中加入註解：

• 註解格式：// Design Decision: [完整原因，直接在註解中說明清楚]
• 不要使用 @see PR #N 或任何外部參考——所有脈絡必須自包含在註解內
• 這可防止相同問題在未來的 review 中再次出現，且不依賴外部連結

3.6 自動繼續下一個項目

決策記錄完成後，不需等待修復子任務完成，直接告知使用者：

「✓ 已記錄。接下來看下一個問題（{current}/{total}）。」

然後立即進入下一個項目的 3.1 步驟。

3.7 等待並驗收所有子任務

所有項目討論完畢後，執行驗收流程：

1. 顯示進度總覽：

   📊 處理總覽：
   - 🔧 修復中：N 個（背景子任務進行中）
   - ⏭️ 延後：N 個
   - ⏭️ N/A：N 個
   

2. 逐一收集子任務結果：

   • 使用 TaskOutput 讀取每個背景子任務的輸出
   • 成功 → 更新狀態為 ✅ Fixed
   • 失敗 → 向使用者報告錯誤，提供選項：

     <options>
     <option>重試修復</option>
     <option>延後到下個 PR</option>
     <option>標記為不適用 (N/A)</option>
     </options>
     

3. 驗證無非預期重疊修改：

   git diff --name-only
   

   確認修改的檔案都在預期範圍內，若有非預期檔案變更，通知使用者確認。

4. 呈現最終結果表格：

   | # | 問題 | 決策 | 狀態 |
   |---|------|------|------|
   | 1 | Silent Cart Failure | 修復 | ✅ Fixed |
   | 2 | Missing Error Boundary | 延後 | ⏭️ Deferred |
   | 3 | Unused Import | N/A | ⏭️ N/A |
   

步驟 4：更新 Review Comment

前提條件：步驟 3.7 驗收完成後才執行此步驟。

⚠️ 警告：必須使用快取腳本！

更新 PR review comment 時，務必使用 cache-write-comment.sh 腳本，絕對不要直接使用 gh api 指令。 呼叫時不要使用 --local-only 旗標，否則只會更新本地快取而不同步 GitHub。

直接使用 gh api 會導致本地快取與 GitHub 永久不同步——comment 內容快取沒有 TTL（與 branch-map 快取的 1 小時 TTL 不同），不會自動過期，所有後續 cache-read-comment.sh 讀取都會拿到過期資料，直到下一次 cache-write-comment.sh 或 cache-read-comment.sh --force-refresh 才會修正。

更新 PR review comment：

1. 準備更新後的內容：

   • 更新各項目的狀態指標
   • 更新 Summary 表格的計數
   • 更新 metadata 中的 updated_at 和 issues 計數
   • 寫入前用 shared helper 正規化 metadata：

     METADATA_JSON=$(printf '%s\n' "$REVIEW_CONTENT" | ${CLAUDE_PLUGIN_ROOT}/scripts/review-metadata-upgrade.sh --stdin --last-writer pr-review-resolver)
     

   • 更新 metadata JSON 後（例如透過 jq 編輯 $METADATA_JSON），用 shared helper 寫回 hidden block，避免改動 issue sections：

     # 為修改後的 metadata JSON 建立暫存檔。
     # （若此 code block 已有自己的 trap，請擴充既有 trap 而非新增一行。）
     METADATA_FILE=$(mktemp)
     trap 'rm -f "$METADATA_FILE"' EXIT
     
     # 將編輯後的 metadata JSON 寫入暫存檔。
     printf '%s' "$METADATA_JSON" > "$METADATA_FILE"
     
     # 替換 comment 中的 metadata block。
     UPDATED_CONTENT=$(printf '%s\n' "$REVIEW_CONTENT" | ${CLAUDE_PLUGIN_ROOT}/scripts/review-metadata-replace.sh --stdin --metadata-file "$METADATA_FILE")
     

   • 保留 review_sources、[Codex] issues、[Gemini] issues，以及無 prefix 的 Claude issues
   • 保持既有 review_round 不變（resolver 只更新狀態，不是 review producer）
   • 將 Status 更新為適當狀態

2. 透過 --stdin 將更新內容直接寫入本地快取並同步至 GitHub：

EXPECTED_CONTENT_HASH=$(jq -r '.content_hash // ""' ".pr-review-cache/pr-${PR_NUMBER}.json" 2>/dev/null || echo "")

if [ -n "$EXPECTED_CONTENT_HASH" ]; then
  printf '%s\n' "$UPDATED_CONTENT" | ${CLAUDE_PLUGIN_ROOT}/scripts/cache-write-comment.sh --stdin "$PR_NUMBER" --expected-content-hash "$EXPECTED_CONTENT_HASH"
else
  printf '%s\n' "$UPDATED_CONTENT" | ${CLAUDE_PLUGIN_ROOT}/scripts/cache-write-comment.sh --stdin "$PR_NUMBER"
fi

腳本會自動：

1. 將內容寫入本地快取（.pr-review-cache/pr-${PR_NUMBER}.json）
2. 嘗試同步至 GitHub（自動重試最多 3 次）
3. 成功後更新快取中的 comment ID
4. 使用 atomic write（先寫入 .tmp 再 mv）避免寫入中斷造成損壞

Exit codes：

• 0 = 成功（本地快取 + GitHub 都已更新）
• 1 = GitHub 同步失敗（本地快取是最新的，可用 /push-review-cache 重試）
• 2 = 本地錯誤（如找不到 PR、內容為空）
• 4 = cache hash mismatch，代表其他 tool 已更新 comment；重新讀取最新 comment，合併 resolver 狀態更新後重試一次

3. 若同步失敗，通知使用者：
   • 本地快取已安全保存（.pr-review-cache/pr-${PR_NUMBER}.json）
   • 使用者可稍後執行 /push-review-cache command 完成同步

步驟 5：知識萃取（所有項目完成後）

當所有項目都解決後，分析此次 session 的學習：

1. 識別模式：

   • 適用於多個檔案的決策
   • 新建立的慣例
   • 對現有指南的澄清

2. 檢查是否需要更新 CLAUDE.md：

   • 是否有決策澄清了模糊的指南？
   • 是否建立了應該記錄的新模式？
   • 是否發現了缺少的指南參考？

3. 必要時建立文件：

   • 將新指南加入適當的 docs/ 檔案
   • 用漸進式揭露方式更新 CLAUDE.md
   • 格式：CLAUDE.md 中放簡要摘要，docs/ 中放完整詳情

4. 更新 review comment：

   • 在最後加入「Knowledge Extracted」章節
   • 列出任何建立或更新的指南
   • 參考文件變更

決策記錄格式

標記項目為 N/A 或 Deferred 時，在原始碼中加入註解：

// Design Decision: [決策的完整原因，直接說明清楚]
// Rationale: [如需要，更詳細的說明]

範例：

// Design Decision: 為可讀性保留明確的 null 檢查
// Rationale: 抽象成 helper 會降低清晰度，收益不大；此處的明確檢查讓維護者一眼就能理解邊界條件

狀態指標

更新 review 時使用一致的狀態指標：

知識萃取標準

評估每個已解決項目是否需要知識萃取：

CLAUDE.md 更新格式

萃取知識時，遵循漸進式揭露：

在 CLAUDE.md 中（簡要參考）：

#### [類別] 指南

詳見 [`docs/[guideline-file].md`](docs/[guideline-file].md)。

**核心原則**：[一句話摘要]

**必須參考的情境：**
1. [觸發情境 1]
2. [觸發情境 2]

在 docs/ 中（完整詳情）：

# [指南標題]

## 背景

[為何需要此指南]

## 模式

[帶範例的詳細說明]

## 使用時機

[適用的情境]

## 例外

[不適用的情況]

驗證清單

完成 review 解決 session 前：

• 所有項目都有狀態（✅、⏭️ 或明確決策）
• N/A 和 Deferred 項目有原始碼註解
• PR review comment 已更新解決方案
• 已評估知識萃取
• 任何新指南已記錄並參考
• Review comment 中的 Summary 計數已更新

互動範例

完整互動範例（含並行背景子任務的正確流程）在 references/interaction-example.md。在開始逐一處理問題前，先讀取此範例以確保互動流程正確。

無 PR 或無 Review Comment 的處理

• 無 PR：通知使用者先建立 PR（gh pr create）
• 無 Review Comment：通知使用者先執行 pr-review-and-document skill 產生 review

常見錯誤

❌ 直接使用 gh api 更新 comment

# 錯誤做法 - 永遠不要這樣做！
gh api repos/{owner}/{repo}/issues/comments/${COMMENT_ID} -X PATCH -f body="..."

這會導致本地快取與 GitHub 永久不同步。正確做法及錯誤恢復方式請參考步驟 4：更新 Review Comment 中的警告。