skills/code-review-resolve/SKILL.md

@@ -10,10 +10,10 @@ argument-hint: "[--findings <findings.json 路徑>] [--target <目標分支>] [-
 
 | 階段 | 動作 |
 | --- | --- |
-| A. Git 同步 | `git fetch` → `git pull` → 若有衝突則告知並嘗試解衝突 |
-| B. 解決問題 | 讀 `findings.json` / `exclusions.json` → 依等級 🔴→🟠→🟡→🔵 逐條修復或判斷誤報 → 已解決者自 `findings.json` 移除，誤報寫入 `exclusions.json` 後也可移除 |
+| A. Git 同步 | `git fetch` → 當前分支不在遠端則切換 develop（再退而 master）→ `git pull` → 若有衝突則告知並嘗試解衝突 |
+| B. 解決問題 | 讀 `findings.json` / `exclusions.json` → 依等級 🔴→🟠→🟡→🔵 逐條修復或判斷誤報 → 已解決者自 `findings.json` 移除，誤報寫入 `exclusions.json` 後也可移除；**無待處理問題則跳到階段 D** |
 | C. 分類提交 | 分析工作區所有變更 → 依 feat/fix/docs/style/refactor/perf/test/chore/revert 分組 → 各組一個 commit |
-| D. Push 當前分支 | 認證管理器 → 失敗改 token → 再失敗詢問使用者 |
+| D. Push 當前分支 | 認證管理器 → 失敗改 token → 再失敗詢問使用者；**無 commit 可 push 則跳到階段 E** |
 | E. 發出 PR | 確定目標分支（不明必問）→ 選 PR 描述（完整／簡單／自訂）→ token 呼叫 Gitea API 建 PR → 通知並清除內文 |
 
 ---
@@ -68,7 +68,36 @@ git fetch --all --prune
 - fetch 失敗時回報錯誤並停止，不進入 findings 修復。
 - 若錯誤訊息可能含 credential / token，輸出前必須遮蔽。
 
-### A3. Pull 當前分支
+### A3. 確認當前分支存在於遠端，必要時切換（develop → master）
+
+fetch 後，確認當前分支在遠端是否有對應分支：
+
+```bash
+current_branch="$(git rev-parse --abbrev-ref HEAD)"
+git rev-parse --verify --quiet "origin/${current_branch}"
+```
+
+- **`origin/<current_branch>` 存在** → 維持當前分支，直接進入 A4 更新到最新。
+- **`origin/<current_branch>` 不存在**（當前分支為本地獨有，遠端無對應）→ 依序嘗試切換到後備分支：
+  1. 切換前先確認工作區可安全切換（承接 A1 結果）。若有未提交變更導致切換失敗，停止並回報，請使用者先處理未提交變更；不可強制丟棄。
+  2. 若 `origin/develop` 存在 → 切換到 `develop`：
+
+     ```bash
+     git switch develop 2>/dev/null || git switch -c develop --track origin/develop
+     ```
+
+  3. 否則若 `origin/master` 存在 → 切換到 `master`：
+
+     ```bash
+     git switch master 2>/dev/null || git switch -c master --track origin/master
+     ```
+
+  4. **`develop` 與 `master` 在遠端都不存在** → 回報「當前分支不在遠端，且找不到 develop／master 後備分支」並停止，不臆測其他分支。
+- 切換完成後，後續階段（含 push 與開 PR 的來源分支）都以切換後的分支為準；切換到後備分支屬不可忽略的狀態變更，需在輸出中明確告知使用者已從原分支切換到哪一個分支。
+
+### A4. Pull（更新到最新）
+
+對當前分支（可能已於 A3 切換為 develop／master）拉取最新：
 
 ```bash
 git pull
@@ -76,9 +105,9 @@ git pull
 
 - pull 成功後進入階段 B。
 - 若顯示需要指定 merge / rebase 策略，先回報原因；未帶 `--yes` 時詢問使用者要採用哪種策略，不可自行猜測。
-- 若 pull 產生衝突，立即告知使用者發生衝突，接著依階段 A4 嘗試解衝突。
+- 若 pull 產生衝突，立即告知使用者發生衝突，接著依階段 A5 嘗試解衝突。
 
-### A4. 必要時嘗試解衝突
+### A5. 必要時嘗試解衝突
 
 當 `git pull` 後出現衝突：
 
@@ -132,7 +161,7 @@ git pull
 
   若既有 exclusions 使用 `suggestion`、`title`、`note` 等欄位，讀取時可容錯；寫回時仍要維持 top-level array，並盡量採用上述 canonical 欄位。
 
-- **檔案不存在、內容為 `[]` 或空白** → 視為「無問題待解決」，輸出告知並**直接跳到階段 C**（仍會把工作區既有變更分類提交，除非 `--no-commit`）。
+- **檔案不存在、內容為 `[]` 或空白** → 視為「無問題待解決」，輸出告知。`--no-commit` 時就此結束；否則**跳過階段 C 直接進入階段 D**（無待處理問題代表本 skill 未產生需要分類提交的修復變更）。
 - **JSON 解析失敗** → 回報錯誤與檔案路徑並停止，不臆測內容、不亂改檔。
 - `exclusions.json` 不存在時視為空陣列，必要時建立 `.gitea/ai-review/exclusions.json` 並寫入 `[]` 或新增後的排除項目。
 - 若 `exclusions.json` 是舊 wrapper 格式（例如 `{ "exclusions": [...] }` 或 `{ "excluded_findings": [...] }`），先正規化為 top-level array 再寫回。
@@ -190,7 +219,7 @@ git diff            # 已追蹤檔的未暫存變更
 git diff --staged   # 已暫存變更
 ```
 
-涵蓋**所有**變更：已修改、新增（未追蹤）、刪除、改名。**無任何變更** → 回報「工作區無變更可提交」並結束。
+涵蓋**所有**變更：已修改、新增（未追蹤）、刪除、改名。**無任何變更** → 回報「工作區無變更可提交」，跳過提交直接進入階段 D（D 會因無 commit 可 push 而跳到階段 E）。
 
 ### C2. 依異動內容歸類 conventional commit 類型
 
@@ -240,22 +269,24 @@ git commit -m "type(範圍): 一句總結"   # 範圍＝實際異動的功能／
 
 - **逐組 add／commit**，確保每個 commit 只含該類異動；不要一次 `git add -A` 再混在一起。
 - 改名／刪除檔一併納入對應組的 `git add`（`git add -A -- <路徑>` 或明確列出）。
-- commit 完成後進入階段 D（push）；`--no-commit` 或無新 commit 時不進入後續階段。
+- commit 完成後進入階段 D（push）。`--no-commit` 時不進入後續階段；若工作區無變更而沒有產生任何新 commit，仍進入階段 D，由 D 判斷無 commit 可 push 後跳到階段 E。
 
 ---
 
-## 階段 D：Push 當前分支（`--no-commit`／無新 commit 時略過）
+## 階段 D：Push 當前分支（`--no-commit` 時略過；無 commit 可 push 時跳到階段 E）
 
 commit 完成後推送**當前分支**，依序嘗試三種方式，前者失敗才退到下一個：
 
-推送前先記錄目前來源分支與其遠端基準：
+推送前先記錄目前來源分支與其遠端基準，並確認是否有 commit 需要 push：
 
 ```bash
 source_branch="$(git rev-parse --abbrev-ref HEAD)"
 git rev-parse --verify "origin/${source_branch}"
+git log --oneline "origin/${source_branch}..${source_branch}"   # 領先遠端的 commit
 ```
 
-- 若來源分支沒有對應的 `origin/<source_branch>`，先記錄「無遠端基準」並繼續；後續若 source/base 同名，階段 E2 仍必須以 `origin/<target>` 作為帶入 commit 的比較基準。
+- **無 commit 可 push**（來源分支已存在於遠端，且相對 `origin/<source_branch>` 沒有領先 commit）→ **跳過本階段 push，直接進入階段 E**（仍可對既有遠端分支開 PR）；若帶 `--no-pr`，則就此結束。
+- 若來源分支沒有對應的 `origin/<source_branch>`，視為需要 push 的新分支，先記錄「無遠端基準」並繼續；後續若 source/base 同名，階段 E2 仍必須以 `origin/<target>` 作為帶入 commit 的比較基準。
 - 若後續會建立 PR（未帶 `--no-pr`）且尚未知道目標分支，先依階段 E1 的規則詢問目標分支，這屬於不可忽略的必要決策，避免把 commit 直接推進目標分支後才發現 source/base 相同。
 - 若來源分支名稱與目標分支相同，**不要 push 原來源分支**；記下來源分支與遠端基準，直接進入階段 E，由 E2 建立新的 PR 來源分支、帶入 commit 後再 push 新分支。
 
@@ -383,10 +414,10 @@ curl -sS -X POST \
 
 各階段執行後輸出：
 
-- **階段 A**：git fetch / pull 是否成功、是否發生衝突、衝突是否已解決或仍需人工處理。
-- **階段 B**：已解決 N 條（依等級分佈）、誤報寫入 exclusions M 條、待人工處理 K 條（列出原因）、`findings.json` 保留／移除筆數與 `exclusions.json` 新增筆數。
+- **階段 A**：git fetch / pull 是否成功、當前分支是否存在於遠端（不存在時切到 develop／master 的結果）、是否發生衝突、衝突是否已解決或仍需人工處理。
+- **階段 B**：已解決 N 條（依等級分佈）、誤報寫入 exclusions M 條、待人工處理 K 條（列出原因）、`findings.json` 保留／移除筆數與 `exclusions.json` 新增筆數；若無待處理問題，說明已跳過階段 C 直接進入階段 D。
 - **階段 C**：建立了哪幾個 commit（type＋訊息＋檔數），或為何略過（`--no-commit` / 無變更）。
-- **階段 D**：push 成功與否、用了哪種方式（認證管理器／token／使用者指定），遠端分支名。
+- **階段 D**：push 成功與否、用了哪種方式（認證管理器／token／使用者指定），遠端分支名；若無 commit 可 push，說明已跳過 push 直接進入階段 E。
 - **階段 E**：PR 連結／編號、目標分支、描述形式；並提醒已（或請使用者）清除對話內文以防 token 外洩。
 
 ---