Files

T

Jeffery feb294ee9a feat(code-review-resolve): 改採新版 findings wrapper 格式並新增 --issue 議題模式（B0 讀議題、B5 回寫並全解決時關閉）

2026-07-20 14:18:07 +08:00

36 KiB

Raw Blame History

name, description, argument-hint

name	description	argument-hint
code-review-resolve	解決新版 AI Code Review 的 findings（`.gitea/ai-review/findings/` 目錄下每次 review 一個 wrapper 物件 JSON，含 `findings`/`excluded` 陣列、中文 severity），或指定 Gitea 議題編號（建問題模式，從議題標題/描述/留言解析嚴重問題為 findings）；先同步 git（fetch → 檢查目前分支的遠端分支是否存在；存在則留在目前分支 pull 更新，不存在則切換到 develop／master 後 pull 更新；只有來源分支與 PR 目標分支相同時才建立新的工作分支；必要時嘗試解衝突），再依嚴重度（嚴重/警告/建議）逐條修復；可判斷為誤報者寫入 `.gitea/ai-review/exclusions.json`，已解決或已登記為誤報的 finding 逐檔就地自來源 wrapper 移除；議題模式修復後於議題留言進度、全解決時關閉議題；接著可將工作區變更依 conventional commit 類型分類提交、push 當前分支，並透過 Gitea API 對指定目標分支開 PR。當使用者說解決 findings、處理 AI review 問題、依議題編號修 code review 問題、修掉 `.gitea/ai-review` 問題、依嚴重度修復後分類提交，或要分類 commit 並 push/開 PR 時觸發。不適用於：產生 findings、單純 code review 不修改、只保存裁決紀錄，或只要不分類的單一 commit。	[--issue <議題編號>] [--findings <findings 檔或目錄路徑>] [--target <目標分支>] [--pr-desc <full\|simple\|自訂文字>] [--no-commit] [--no-pr] [--yes]

code-review-resolve — 解決 AI review findings、分類提交、push 並開 PR

五階段 skill：先執行 git 同步（git fetch → 確認目前分支的遠端分支是否存在；存在則留在目前分支 git pull 更新，不存在則切換到 develop，再退而 master 後 git pull 更新；只有來源分支與 PR 目標分支相同時才建立新的工作分支；必要時告知並嘗試解衝突），再逐條處理問題來源——預設讀 .gitea/ai-review/findings/ 目錄下每次 review 產生的 wrapper 物件 JSON（取其 findings 陣列），或帶 --issue <議題編號> 時從 Gitea 議題標題／描述／留言解析嚴重問題；成立問題修復後自來源 wrapper 逐檔就地移除，可判斷為誤報者寫入 .gitea/ai-review/exclusions.json 後也可移除（議題模式改在議題留言進度、全解決時關閉議題），接著必須盤點並處理工作區的所有變更，再依 conventional commit 類型分門別類 commit，然後 push 當前分支，最後透過 Gitea API 發 PR。

階段	動作
A. Git 同步	`git fetch` → 當前分支不在遠端則切換 develop（再退而 master）→ `git pull`／必要時解衝突 → 只有來源分支與 PR 目標分支相同時才建立新工作分支
B. 解決問題	讀 `.gitea/ai-review/findings/.json`（wrapper 物件，取 `findings`）或 `--issue` 議題留言 / `exclusions.json` → 依等級 🔴嚴重→🟠警告→🔵建議逐條修復或判斷誤報 → 已解決者逐檔就地自來源 wrapper 移除，誤報寫入 `exclusions.json`；議題模式回寫議題（留言＋全解決關閉）；無待處理問題則跳到階段 D*
C. 分類提交	必須分析並涵蓋工作區所有變更（已修改／新增／刪除／改名，含已暫存與未暫存、未追蹤檔、findings/exclusions 更新）→ 依 feat/fix/docs/style/refactor/perf/test/chore/revert 分組 → 各組一個 commit
D. Push 當前分支	認證管理器 → 失敗改 token → 再失敗詢問使用者；無 commit 可 push 則跳到階段 E
E. 發出 PR	確定目標分支（不明必問）→ 選 PR 描述（完整／簡單／自訂）→ token 呼叫 Gitea API 建 PR → 通知並清除內文

共用規範（generic plugin，必要前置）

執行本 skill 前，先以 Skill 工具載入下列共用規範並全程遵守；任一載入不到（generic plugin 未安裝）時，先詢問使用者是否安裝 generic plugin（https://gitea.jsc.idv.tw/plugins/generic.git），使用者不安裝則直接中斷本 skill，不得只憑下方一行摘要繼續執行：

/jsc:spec-output：繁體中文（台灣用語）、UTF-8（不含 BOM）無亂碼（含 commit 訊息、PR 標題／描述、寫檔／API body 實作要點與送出前自我檢查）。
/jsc:spec-execution：自動執行原則（必要決策才中斷）、不臆測／需人工確認。
/jsc:spec-gitea：GITEA_TOKEN 機密保護（不 echo、遮蔽 ***）、API body 以 UTF-8 JSON 檔帶入。
/jsc:spec-git-safety：不破壞既有工作、develop → master 後備、保守解衝突、新分支不覆蓋既有分支。

本 skill 特有補充：

必要決策（會中斷詢問）：目標分支缺失、pull 策略不明、無法安全解衝突、需人工判斷的設計取捨、push 憑證皆失敗。
涵蓋範圍：更新後的 findings/*.json（wrapper）/ exclusions.json、議題留言、commit 訊息、PR 標題／描述、終端訊息，與等級 emoji 🔴🟠🔵。
階段 E 完成後依規範清除對話內文（見 E6）。

參數

格式：[--issue <議題編號>] [--findings <路徑>] [--target <目標分支>] [--pr-desc <full|simple|自訂文字>] [--no-commit] [--no-pr] [--yes]

--issue <議題編號>：議題模式——問題來源改為指定的 Gitea 議題（對應 code-review 的建問題模式）。帶此參數時走階段 B0：讀議題標題／描述／全部留言，把符合嚴重問題留言格式的內容解析成 findings，逐條修復；修復後於階段 B5 在議題留言進度、全部解決時關閉議題。此模式不讀 findings/ 目錄（除非同時明確帶 --findings）。需要 GITEA_TOKEN 與可解析的 origin repo 座標。
--findings <路徑>：findings 來源路徑。省略時預設讀目錄 .gitea/ai-review/findings/ 下所有 *.json（每個檔為新版 wrapper 物件，取其 findings 陣列合併）（相對於工作目錄根）。指定路徑時：目錄則讀該目錄所有 *.json；單一檔則讀該檔（同為 wrapper 物件）。與 --issue 併用時，兩者的問題都納入處理。
--target <目標分支>：PR 的目標分支。省略時必須詢問使用者，不可猜測；若需要避免把同名來源分支誤推到目標分支，可能會在階段 D 先詢問並於階段 E 沿用。
--pr-desc <full|simple|自訂文字>：PR 描述形式。full＝完整版（重新分析 diff 總結）、simple＝簡單版（逐條列 commit 訊息）、或直接給自訂文字；省略時預設 full，不要為描述形式中斷詢問。
--no-commit：只做階段 A/B（git 同步 + 修復 + 更新 findings/exclusions），不執行階段 C/D/E（不提交、不推送、不開 PR）。
--no-pr：執行到階段 D（push）為止，不開 PR（階段 E 略過）。
--yes：明確要求全自動執行（修復、更新 findings/exclusions、commit、push、開 PR 一氣呵成）；即使未帶此參數，也依「自動執行原則」盡量不中斷，只有必要決策才詢問。

階段 A：Git 同步

A1. 檢查同步前狀態

執行同步前先盤點目前分支與工作區狀態：

git rev-parse --abbrev-ref HEAD
git status --porcelain

若工作區已有未提交變更：先告知使用者同步可能需要 merge / rebase 並可能與本地變更衝突；除非使用者要求先確認或目前狀態明顯容易覆蓋未提交工作，否則繼續同步。
若目前不在一般分支上（例如 detached HEAD）：回報狀態並停止，請使用者切到要處理的分支後再執行。

A2. Fetch 遠端更新

git fetch --all --prune

fetch 失敗時回報錯誤並停止，不進入 findings 修復。
若錯誤訊息可能含 credential / token，輸出前必須遮蔽。

A3. 確認當前分支存在於遠端，必要時切換（develop → master）

fetch 後，確認當前分支在遠端是否有對應分支：

current_branch="$(git rev-parse --abbrev-ref HEAD)"
git rev-parse --verify --quiet "origin/${current_branch}"

origin/<current_branch> 存在 → 維持當前分支，直接進入 A4 更新到最新；不要只因目前分支有遠端同名分支就建立新工作分支，後續只有來源分支與 PR 目標分支相同時才需要開新分支。
origin/<current_branch> 不存在（當前分支為本地獨有，遠端無對應）→ 依序嘗試切換到後備分支：
1. 切換前先確認工作區可安全切換（承接 A1 結果）。若有未提交變更導致切換失敗，停止並回報，請使用者先處理未提交變更；不可強制丟棄。
2. 若 origin/develop 存在 → 切換到 develop：
```
git switch develop 2>/dev/null || git switch -c develop --track origin/develop
```
3. 否則若 origin/master 存在 → 切換到 master：
```
git switch master 2>/dev/null || git switch -c master --track origin/master
```
4. develop 與 master 在遠端都不存在 → 回報「當前分支不在遠端，且找不到 develop／master 後備分支」並停止，不臆測其他分支。
切換完成後，A4 先更新該後備分支到最新；是否建立新的工作分支交由 A5 依來源分支與 PR 目標分支是否相同判斷。切換到後備分支屬不可忽略的狀態變更，需在輸出中明確告知使用者已從原分支切換到哪一個分支。

A4. Pull（更新到最新）

對當前分支（可能已於 A3 切換為 develop／master）拉取最新：

git pull

pull 成功後進入 A5，必要時建立新的工作分支，再進入階段 B。
若顯示需要指定 merge / rebase 策略，先回報原因；未帶 --yes 時詢問使用者要採用哪種策略，不可自行猜測。
若 pull 產生衝突，立即告知使用者發生衝突，接著依階段 A6 嘗試解衝突。

A5. 只有來源分支與 PR 目標分支相同時，建立新的工作分支

A4 更新完成後，取得目前來源分支與 PR 目標分支：

source_branch="$(git rev-parse --abbrev-ref HEAD)"
# target 來自 --target；若未提供且後續會建立 PR（未帶 --no-pr），需先依 E1 詢問目標分支。

若後續會建立 PR（未帶 --no-pr）且尚未知道目標分支 → 先依階段 E1 的規則詢問目標分支，避免後續修復 commit 落在與 PR 目標同名的來源分支上才發現 source/base 相同。若帶 --no-pr，此階段不因缺少目標分支而詢問。
若來源分支名稱與目標分支相同 → 不在該分支上直接修復／commit，也不可後續直接建立 head=base 的 PR。從目前已更新到最新的目標分支建立新的工作分支，後續階段（修復、commit、push、PR 的 head）都以新分支為準。
若來源分支名稱與目標分支不同 → 維持目前分支，不另開分支；即使目前分支存在 origin/<source_branch>，也照常在目前分支處理。

新分支名稱需可讀且避免覆蓋既有分支；預設格式：

work_branch="ai-review-resolve/${source_branch}-$(date +%Y%m%d-%H%M%S)"
git switch -c "${work_branch}"

建立前若本地或遠端已存在同名分支，換一個時間戳或短 hash，不可覆蓋既有分支。
若因未提交變更導致建立／切換新分支失敗，停止並回報，請使用者先處理未提交變更；不可強制丟棄。
建立新分支屬不可忽略的狀態變更，需在輸出中明確告知使用者「因來源分支與目標分支同名，已從 <source_branch> 建立並切換到 <work_branch>」。

A6. 必要時嘗試解衝突

當 git pull 後出現衝突，依 /jsc:spec-git-safety 的保守解衝突流程處理（定位衝突檔 → 最小合理整合 → 可安全解決者 git add 標記、無法安全判斷者停止並列出決策點）；全部衝突解完後，依 git 當前狀態完成 merge / rebase 的必要步驟，確認 git status --porcelain 沒有未解衝突，再回到 A5 判斷是否需要建立新工作分支。

階段 B：讀取問題（findings 檔或 Gitea 議題）、依嚴重度逐條解決或登記誤報

新版 AI Code Review 的產出格式（對齊 https://gitea.jsc.idv.tw/actions/code-review，src/index.js / src/lib/review.js）：

findings：每次 review 在目錄 .gitea/ai-review/findings/ 產生一個新檔（檔名為台北時間戳，如 2026-07-20-14:30:15.json），內容是頂層物件（wrapper），不是陣列：

{
  "generatedAt": "<台北時間>",
  "commitSha": "<head sha>",
  "prNumber": 9,
  "tool": { "name": "...", "version": "...", "model": "..." },
  "findings": [ { "<finding>": "..." } ],
  "excluded":  [ { "<finding>": "..." } ]
}

每個 <finding> 欄位：

欄位	語義
`id`	流水號（`F001`…）
`reviewer`	審查角色名稱
`focus`	審查面向代碼（如 `logic`）
`badge`	角色徽章（emoji，可空）
`severity`	嚴重等級（中文：`嚴重` / `警告` / `建議`）
`file`	問題所在檔案（repo 相對路徑）
`startLine` / `endLine`	問題起訖行號
`problem`	問題描述
`suggestion`	修改建議
`suggestedCode`	建議寫法（程式碼，可空）

exclusions：固定 .gitea/ai-review/exclusions.json，頂層陣列，每筆：

{
  "addedAt": "<台北時間>",
  "prNumber": 9,
  "reviewer": "<審查角色>",
  "severity": "嚴重|警告|建議",
  "file": "<檔案路徑>",
  "startLine": 10,
  "endLine": 12,
  "problem": "<問題描述>",
  "reason": "<判定誤報／重複的理由>"
}

建問題模式：某些 review 不寫 findings 檔，改開 Gitea 議題、把嚴重問題逐則發成議題留言（見 B0）。

本 skill 只支援上述新版格式；讀到非 wrapper 物件（例如舊版頂層陣列 findings）時，回報格式不符並停止，不臆測欄位。

B0. 議題模式：從 Gitea 議題讀取問題（帶 `--issue` 時）

帶 --issue <議題編號> 時，問題來源為該議題（不讀 findings/ 目錄，除非同時明確帶 --findings）：

解析 repo 座標（同 E3，從 git remote get-url origin 取 host／owner／repo），token 由 GITEA_TOKEN 提供。

讀議題本文與全部留言（分頁全取；輸出時遮蔽 token）：

curl -sS -H "Authorization: token ${GITEA_TOKEN}" \
  "https://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>"
curl -sS -H "Authorization: token ${GITEA_TOKEN}" \
  "https://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>/comments"

解析每則符合「嚴重問題留言」格式（action severeCommentBody 產出）的留言為一條 finding：
```
### <emoji> <severity>｜<badge> <reviewer>
**位置**：`<file>` 第 <startLine>–<endLine> 行
**問題**
<problem>
**修改建議**
<suggestion>
**建議寫法**（可選）
<suggestedCode>
```
逐則映射為 { reviewer, severity, file, startLine, endLine, problem, suggestion, suggestedCode }，並記住來源留言 id（供 B5 回報）；議題標題／描述作為整體背景脈絡納入判斷。
解析不到任何嚴重問題留言 → 回報並停止議題模式，不臆測。
議題模式下沒有 findings 檔可更新；解析出的問題直接進入 B2/B3，處理結果於 B5 回寫議題（留言＋全解決時關閉），誤報仍寫入 exclusions.json。

B1. 讀取 findings 與 exclusions（檔案／目錄來源）

未帶 --issue 時，依序定位 findings 來源：

省略 --findings：讀目錄 .gitea/ai-review/findings/ 下所有 *.json（含子目錄、依檔名排序）；每個檔皆為上述 wrapper 物件，取其 findings 陣列合併為待處理清單。
指定 --findings <路徑>：路徑為目錄則比照第 1 點；為單一檔則讀該檔（同為 wrapper 物件），取其 findings。
合併時逐筆記住各 finding 的來源檔路徑與其所屬 wrapper，供 B4 逐檔就地寫回；此步不得更動任何來源檔。
目錄不存在／無 *.json／各檔 findings 皆空 → 視為「無問題待解決」，輸出告知。--no-commit 時就此結束；否則跳過階段 C 直接進入階段 D。
JSON 解析失敗或非 wrapper 物件 → 回報錯誤與檔案路徑並停止，不臆測、不亂改檔。

exclusions.json 不存在時視為空陣列（頂層陣列）；解析失敗或非陣列時回報，並在本次不寫入誤報（避免破壞既有內容、需人工確認）。

B2. 依嚴重等級排序

把所有 finding 依等級由高到低排序：🔴 嚴重 → 🟠 警告 → 🔵 建議。 severity 為中文 嚴重 / 警告 / 建議；容錯對應：嚴重/critical/high/blocker→🔴、警告/warning/major→🟠、建議/info/low/minor→🔵；無法辨識者排在最後並標「等級未知」。排序後先處理高等級。

B3. 逐條處理（高等級先）

輸出一張「處理計畫」表，除非使用者要求確認或遇到必要決策，否則依排序由上而下逐條處理：

#	等級	審查員	問題	檔案	行數	處理方式

對每一條 finding：

讀取對應檔案與其脈絡（依 file 與 startLine–endLine）。
先判斷是否為誤報或不適用：例如程式碼脈絡證明指控不成立、已有等價防護、命中 exclusions.json 已知排除、CI/CD 必要權限、或 finding 對非本次變更做不合理要求。
可判斷為誤報 → 不改程式碼；將排除條目 append 到 .gitea/ai-review/exclusions.json（新版欄位，見 B4），保留原始問題文字與語意；若已有等價 exclusion（同 file＋reviewer＋problem 高度相似）則不重複新增。
確認為真問題且可安全修復 → 依 suggestion／suggestedCode／problem 在程式碼中實作最小合理修正；建議含糊或與現況不符時，依原始碼脈絡做最小且合理的修正。
無法安全自動修復或無法確認真偽（例如需求不明、牽涉設計取捨、檔案不存在）→ 不硬改，標記為「待人工處理」並記錄原因，繼續下一條。
逐條完成後，記錄該條結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因）；議題模式另記來源留言 id。

一次只處理一條、修完再處理下一條，避免互相干擾；同檔多條問題可合併讀取但仍逐條套用修正。

B4. 更新 findings 檔與 exclusions.json（檔案／目錄來源）

所有條目處理完畢後依結果更新（議題模式無 findings 檔，跳過本節 findings 寫回，改走 B5）：

只有確認已解決，或已確認為誤報並成功寫入 exclusions.json 的 finding，才可從來源 findings 檔移除；待人工處理、無法確認真偽、修復未驗證成功者一律保留。
逐檔就地更新（wrapper 物件）：依 B1 記錄的來源檔，把該檔 findings 陣列中要保留的 finding 留下、其餘移除，只改該檔的 findings 陣列；wrapper 其他欄位（generatedAt、commitSha、prNumber、tool、excluded）原樣保留，不得把多個來源檔合併成單一檔。某檔 findings 全數已解決／誤報時，把該檔的 findings 寫成 []（保留檔案與 wrapper、維持目錄結構，不刪檔）。
保留的 finding 維持其原 wrapper 內欄位（id、reviewer、severity、file、startLine、endLine、problem、suggestion、suggestedCode…）與排序。
誤報一律 append 到 .gitea/ai-review/exclusions.json（頂層陣列），欄位對齊新版：addedAt（當下台北時間，date 取得）、prNumber（已知則帶、未知可省略）、reviewer、severity、file、startLine、endLine、problem、reason（誤報理由）；append 後去重。
誤報移出 findings 前，必須先確認 exclusions.json 已成功寫入該條或已有等價條目；寫入失敗則該 finding 保留於來源檔。
所有寫檔以 UTF-8（不含 BOM）寫入，結尾保留一個換行。

B5. 議題模式收尾：回寫議題（帶 `--issue` 時）

議題模式所有問題處理完畢後：

在議題留言回報進度（Gitea API POST /repos/<owner>/<repo>/issues/<編號>/comments，body 以 UTF-8 JSON 檔帶入、token 不 echo）：以表格逐條列出結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因）。

全部問題都已解決或已登記誤報（無 ⏭️ 待人工處理）→ 關閉議題：

curl -sS -X PATCH -H "Authorization: token ${GITEA_TOKEN}" \
  -H "Content-Type: application/json" \
  "https://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>" \
  --data '{"state":"closed"}'

尚有待人工處理 → 只留言不關閉，於留言標明待處理項與原因。
誤報仍寫入 exclusions.json（同 B4），讓後續 review 沿用。

階段 C：分析變更並分類提交（`--no-commit` 時略過）

C1. 盤點工作區變更

git status --porcelain=v1 -uall
git diff            # 已追蹤檔的未暫存變更
git diff --staged   # 已暫存變更
git ls-files --others --exclude-standard   # 未追蹤檔

盤點時必須以 git status --porcelain=v1 -uall 為主，不可只看 git diff，因為那會漏掉未追蹤檔。盤點範圍必須包含所有變更：已修改檔、新增檔（?? 未追蹤檔）、刪除檔、改名檔，以及已暫存與未暫存變更。git diff、git diff --staged、git ls-files --others --exclude-standard 只作為輔助核對。無任何變更 → 回報「工作區無變更可提交」，跳過提交直接進入階段 D（D 會因無 commit 可 push 而跳到階段 E）。

C2. 依異動內容歸類 conventional commit 類型

逐一檢視每個變更檔的實際異動內容（不只看路徑），歸入下列其一；所有 ?? 未追蹤檔都必須納入分類，不能因為它們不在 git diff 裡就漏掉：

type	適用情境
`feat`	新增功能／新行為／新 API／新 skill
`fix`	修正錯誤、修掉 bug（階段 B 的 bug 修復多半歸此）
`docs`	只改文件（README、註解、`*.md`、說明）
`style`	不影響邏輯的格式調整（排版、空白、分號、命名一致化）
`refactor`	重構：不改外部行為的內部結構調整
`perf`	效能優化
`test`	新增或修改測試
`chore`	雜項：建置、設定、相依套件、版本號 bump、忽略檔等
`revert`	還原先前的提交

同一檔案橫跨多型 → 以該檔主要異動性質歸類；難以拆分時就近歸入影響最大的一類，並在總結註記。
階段 B 修復產生的變更：依其性質歸類（修 bug→fix、補功能→feat、改文件→docs…）。.gitea/ai-review/findings/ 目錄下 *.json（wrapper）與 exclusions.json 的問題狀態更新歸 chore。
未追蹤新檔：必須照實際內容歸入對應 type，必要時在提交前明確 git add -- <path>，不可因為是新檔就略過。

C3. 產出提交計畫

把變更檔依 type 分組，每個 type 一個 commit，輸出提交計畫；所有變更項目都必須被追蹤並納入計畫，不能遺漏任何 ?? 未追蹤檔。除非使用者要求確認或分組有不可忽略的取捨，否則直接提交：

順序	type(範圍)	commit 訊息	納入檔案

commit 訊息格式：type(範圍): 一句總結 —
- type：上表英文類型。
- 範圍（括號內）：必須是這組異動實際牽涉的功能／模組／元件名稱，而不是重述 type 的類別詞。取名規則：優先沿用程式碼／專案中既有的識別名（檔名、模組名、skill 名、功能名，可中可英、保持與原碼一致），讓人一眼看出「改到哪個東西」。
  - ✅ 對：feat(使用者登入)、fix(結帳流程)、perf(物件查詢)、docs(README)、refactor(訂單服務)、chore(plugin 版本)、feat(code-review-resolve)。
  - ❌ 錯（只是重述 type，禁止）：feat(新增功能)、fix(修正錯誤)、perf(優化效能)、docs(文件)、chore(雜項)。
  - 一組異動橫跨多個功能而無單一主體時，才退而取最貼近的上層範圍（例如多個 manifest → plugin 設定）。
- 一句總結：把這個 commit 內所有異動總結成一句繁體中文（簡短、聚焦做了什麼）。
- 範例：feat(使用者登入): 新增帳密登入與 token 簽發、fix(結帳流程): 修正空購物車導致的結帳例外、perf(物件查詢): 改用批次查詢降低 DB 往返、docs(README): 補上安裝與呼叫方式說明、chore(ai-review 狀態): 更新 findings 與 exclusions.json。
提交順序建議：fix/feat 等核心異動在前，docs/style/chore 在後（純屬建議，可依相依性調整）。
盤點要求：提交計畫必須完整對應 C1 盤點出的所有變更項目，若有新檔或未追蹤檔，必須明確列入對應 commit，不能只根據 git diff 下結論。

C4. 執行分類提交

對每組依序：

git add -- <該組檔案...>       # 僅暫存該組檔案，逐組精準 add
git commit -m "type(範圍): 一句總結"   # 範圍＝實際異動的功能／模組名

逐組 add／commit，確保每個 commit 只含該類異動；不要一次 git add -A 再混在一起。
改名／刪除檔一併納入對應組的 git add（git add -A -- <路徑> 或明確列出）。
所有 ?? 未追蹤檔都必須納入提交；若是新檔，必要時要明確執行 git add -- <path>，不可漏掉。
不可只根據 git diff 判斷變更；C1 盤點出的所有異動都必須反映到分類與提交，否則視為提交計畫不完整。
commit 完成後進入階段 D（push）。--no-commit 時不進入後續階段；若工作區無變更而沒有產生任何新 commit，仍進入階段 D，由 D 判斷無 commit 可 push 後跳到階段 E。

階段 D：Push 當前分支（`--no-commit` 時略過；無 commit 可 push 時跳到階段 E）

commit 完成後推送當前分支，依序嘗試三種方式，前者失敗才退到下一個：

推送前先記錄目前來源分支與其遠端基準，並確認是否有 commit 需要 push：

source_branch="$(git rev-parse --abbrev-ref HEAD)"
git rev-parse --verify "origin/${source_branch}"
git log --oneline "origin/${source_branch}..${source_branch}"   # 領先遠端的 commit

無 commit 可 push（來源分支已存在於遠端，且相對 origin/<source_branch> 沒有領先 commit）→ 跳過本階段 push，直接進入階段 E（仍可對既有遠端分支開 PR）；若帶 --no-pr，則就此結束。
若來源分支沒有對應的 origin/<source_branch>，視為需要 push 的新分支，先記錄「無遠端基準」並繼續；這種情況不需要切回 develop／master，因為 A3 已在同步階段處理「當前分支不存在於遠端」的後備切換。
若後續會建立 PR（未帶 --no-pr）且尚未知道目標分支，先依階段 E1 的規則詢問目標分支，這屬於不可忽略的必要決策。
若來源分支名稱與目標分支相同，不要 push 原來源分支；記下來源分支與遠端基準，直接進入階段 E，由 E2 建立新的 PR 來源分支、帶入 commit 後再 push 新分支。

認證管理器（優先）：直接用 git 既有的 credential helper（如 Windows 的 manager-core）：
```
git push -u origin "$(git rev-parse --abbrev-ref HEAD)"
```
失敗 → 改用 token push：從環境變數讀 token，組帶 token 的遠端 URL 推送。整個過程不可把含 token 的指令／URL 印出來（用變數帶入、輸出時遮蔽）：
```
# GITEA_TOKEN 來自環境變數；解析 origin 的 host/owner/repo
git push "https://oauth2:${GITEA_TOKEN}@<host>/<owner>/<repo>.git" \
  "$(git rev-parse --abbrev-ref HEAD)"
```
（token 用完即棄，不寫進 git remote 設定、不落地。）
再失敗 → 詢問使用者要如何 push：列出失敗原因（遮蔽 token），請使用者指示推送方式，不可自行猜測其他憑證或來源。

push 成功後記下遠端分支名，進入階段 E。若因來源分支與目標分支相同而延後 push，記下原因並進入階段 E2 建立新分支。

階段 E：透過 Gitea API 發出 PR（`--no-pr`／`--no-commit` 時略過）

E1. 確定目標分支（不可猜想）

帶 --target <分支> → 直接採用。
若階段 D 已為了避免 source/base 相同而詢問過目標分支，沿用該目標分支。
否則必須詢問使用者目標分支（可用 git branch -r 列出輔助選擇），嚴禁臆測或預設（不可自行假設 develop／main／master）。

E2. 若來源分支與目標分支相同，改建 PR 來源分支

先取得目前 PR 來源分支：

git rev-parse --abbrev-ref HEAD

若來源分支名稱與 --target 指定（或使用者選定）的目標分支相同，不可直接建立 head=base 的 PR，也不可先把原來源分支 push 到目標分支。改用下列流程建立新的 PR 來源分支，並把原來源分支的 commit 帶入後再開 PR：

先記錄原來源分支名稱與要帶入的 commit 清單：
```
source_branch="$(git rev-parse --abbrev-ref HEAD)"
git log --oneline "origin/${target}..${source_branch}"
```
- 若 origin/<target> 不存在，先回報並停止，不猜測替代 base。
- 若沒有任何 commit 可帶入，回報「來源分支沒有領先目標分支的 commit」，停止開 PR。
- 若階段 D 已記錄來源分支的遠端基準，使用該基準判斷要帶入的 commit，避免把原來源分支直接推進目標分支。
從目標分支的遠端基準建立新分支。新分支名稱需可讀且避免覆蓋既有分支，例如：
```
git switch -c "ai-review-resolve/<短時間戳>" "origin/${target}"
```
將原來源分支領先目標分支的 commit 依序帶入新分支：
```
git cherry-pick "origin/${target}..${source_branch}"
```
- cherry-pick 發生衝突時，先告知使用者，再依專案脈絡嘗試最小合理解衝突。
- 可安全解決的衝突：移除衝突標記、git add -- <檔案...>，再繼續 git cherry-pick --continue。
- 無法安全判斷的衝突：停止處理，列出衝突檔案與需要使用者決策的點；不要硬選任一邊。
將新分支 push 到遠端，使用階段 D 的同一套 push 憑證策略，並把後續 PR 的 head 改為這個新分支。

若來源分支與目標分支不同，直接以目前分支作為 PR 的 head。

E3. 解析 repo 座標

從 git remote get-url origin 解析出 host／owner／repo（例：https://gitea.jsc.idv.tw/plugins/code-review.git → host=gitea.jsc.idv.tw、owner=plugins、repo=code-review）。

E4. 決定 PR 描述形式（完整版／簡單版／使用者輸入）

依 --pr-desc 三選一；若未提供則預設使用 full，不要為描述形式中斷詢問：

完整版（full）：重新分析並總結 git diff <target>...<PR 來源分支>（比對 source 自分岔點以來的變更），整理成結構化繁體中文說明 —— 變更摘要、影響範圍、重點檔案／模組、風險或注意事項。不是貼原始 diff，而是「人讀得懂的總結」。
簡單版（simple）：直接把本分支領先 target 的 commit 訊息逐條列出：
```
git log --oneline "<target>..<PR 來源分支>"
```
以條列呈現每行 commit 訊息。
使用者輸入：採用使用者提供的描述文字。

PR 標題預設取一句總結（可用首個 feat/fix commit 或分支用途）；使用者另有指定則從之。

產生 PR body 時必須使用實際換行與 Markdown 內容，不可把跳脫字串當作正文送出。若用 shell / jq 組 JSON，應以 UTF-8 暫存檔、heredoc、printf 或 jq --rawfile 帶入內容；避免讓 PR 顯示成 ## Commit\n\n- ... 這類字面 \n。

E5. 呼叫 Gitea API 建立 PR（使用 token）

token 從環境變數讀取，呼叫 Gitea 建立 PR：

# 不可 echo 含 token 的指令；body 以 UTF-8 JSON 檔帶入，輸出時遮蔽 token
# PR 描述內的換行必須是實際換行，不可使用字面 \n
curl -sS -X POST \
  -H "Authorization: token ${GITEA_TOKEN}" \
  -H "Content-Type: application/json" \
  "https://<host>/api/v1/repos/<owner>/<repo>/pulls" \
  --data @body.json

成功：取回應中的 PR 連結／編號回報使用者。
失敗：顯示 API 回應的錯誤訊息供排查（先遮蔽 token）。常見錯誤：目標分支不存在、已有相同 head→base 的 PR、token 權限不足。

E6. 完成通知 + 清除對話內文（重要：可能含 token）

通知使用者：push 結果、PR 連結／編號、PR 描述採用哪種形式。
清除 AI 助理對話內文：因為 push／API 過程可能使對話內文殘留 gitea token，完成後務必清除對話內文／上下文以免外洩：
- Claude Code：提示使用者執行 /clear（或依當前助理對等指令清空對話）。
- 其他助理：執行各自清除對話／上下文的方式。
- 在清除前，請再次確認輸出與 log 中沒有任何明文 token。

總結

各階段執行後輸出：

階段 A：git fetch / pull 是否成功、當前分支是否存在於遠端（不存在時切到 develop／master 的結果）、若來源分支與目標分支同名是否已建立新工作分支、是否發生衝突、衝突是否已解決或仍需人工處理。
階段 B：問題來源（findings 目錄或議題 #N）、已解決 N 條（依等級分佈）、誤報寫入 exclusions M 條、待人工處理 K 條（列出原因）、來源 findings 檔保留／移除筆數與 exclusions.json 新增筆數；議題模式另報已在議題留言、是否關閉議題；若無待處理問題，說明已跳過階段 C 直接進入階段 D。
階段 C：建立了哪幾個 commit（type＋訊息＋檔數），或為何略過（--no-commit / 無變更）。
階段 D：push 成功與否、用了哪種方式（認證管理器／token／使用者指定），遠端分支名；若無 commit 可 push，說明已跳過 push 直接進入階段 E。
階段 E：PR 連結／編號、目標分支、描述形式；並提醒已（或請使用者）清除對話內文以防 token 外洩。

呼叫方式

格式：[--findings <路徑>] [--target <目標分支>] [--pr-desc <full|simple|自訂文字>] [--no-commit] [--no-pr] [--yes] — 全部可省略（findings 預設讀目錄 .gitea/ai-review/findings/ 下所有 wrapper *.json；帶 --issue <編號> 改由 Gitea 議題取問題；目標分支省略時必問、不猜測）。token 一律由環境變數（如 GITEA_TOKEN）提供。

助理	呼叫
Claude Code / Antigravity	`/jsc:code-review-resolve`，或 `/jsc:code-review-resolve --target develop --pr-desc full --yes`、`/jsc:code-review-resolve --issue 12 --target master`、`/jsc:code-review-resolve --no-pr`、`/jsc:code-review-resolve --no-commit`
Codex	`$code-review-resolve`，或 `$code-review-resolve --target develop --pr-desc full --yes`、`$code-review-resolve --issue 12 --target master`，或用 `/skills` 選單
OpenCode	描述需求（如「讀 .gitea/ai-review/findings/ 目錄下的 wrapper findings 依嚴重度逐條處理，更新 findings/exclusions，把工作區變更依 conventional commit 分類提交，push 後對 develop 發 PR（完整版描述）」；或「處理議題 #12 的 code review 問題」）自動觸發

36 KiB Raw Blame History Unescape Escape