From 4fc33d4a69b5b6ca9e7f1debc8f8693514ec5d54 Mon Sep 17 00:00:00 2001 From: Jeffery Date: Mon, 20 Jul 2026 18:14:57 +0800 Subject: [PATCH 1/2] =?UTF-8?q?feat(code-review-resolve):=20findings=20?= =?UTF-8?q?=E6=AA=94=E4=B8=80=E5=BE=8B=E8=AE=80=E5=8F=96=E4=B8=A6=E8=88=87?= =?UTF-8?q?=20--issue=EF=BC=88=E7=84=A1/=E5=96=AE=E9=81=B8/=E5=A4=9A?= =?UTF-8?q?=E9=81=B8/=E5=85=A8=E9=83=A8=EF=BC=89=E8=AD=B0=E9=A1=8C?= =?UTF-8?q?=E5=90=88=E4=BD=B5=E5=8E=BB=E9=87=8D=EF=BC=8CStage=20B=20?= =?UTF-8?q?=E6=AD=A5=E9=A9=9F=E9=87=8D=E6=96=B0=E7=B7=A8=E8=99=9F=E7=82=BA?= =?UTF-8?q?=20B1=E2=80=93B6?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- skills/code-review-resolve/SKILL.md | 119 ++++++++++++++++------------ 1 file changed, 69 insertions(+), 50 deletions(-) diff --git a/skills/code-review-resolve/SKILL.md b/skills/code-review-resolve/SKILL.md index 9d132bd..1dc1e80 100644 --- a/skills/code-review-resolve/SKILL.md +++ b/skills/code-review-resolve/SKILL.md @@ -1,17 +1,17 @@ --- name: code-review-resolve -description: 解決新版 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 移除；議題模式修復後於議題留言進度、把待人工處理問題寫回 findings 目錄後關閉議題；接著可將工作區變更依 conventional commit 類型分類提交、push 當前分支，並透過 Gitea API 對指定目標分支開 PR。當使用者說解決 findings、處理 AI review 問題、依議題編號修 code review 問題、修掉 `.gitea/ai-review` 問題、依嚴重度修復後分類提交，或要分類 commit 並 push/開 PR 時觸發。不適用於：產生 findings、單純 code review 不修改、只保存裁決紀錄，或只要不分類的單一 commit。 -argument-hint: "[--issue <議題編號>] [--findings ] [--target <目標分支>] [--pr-desc ] [--no-commit] [--no-pr] [--yes]" +description: 解決新版 AI Code Review 的 findings——一律讀取專案內 `.gitea/ai-review/findings/` 目錄下的 wrapper 物件 JSON（含 `findings`/`excluded` 陣列、中文 severity）與 `findings.json`，並可額外用 `--issue`（無/單選/多選/全部）從 Gitea 議題（建問題模式）解析嚴重問題留言為 findings，兩來源合併去重後逐條解決；先同步 git（fetch → 檢查目前分支的遠端分支是否存在；存在則留在目前分支 pull 更新，不存在則切換到 develop／master 後 pull 更新；只有來源分支與 PR 目標分支相同時才建立新的工作分支；必要時嘗試解衝突），再依嚴重度（嚴重/警告/建議）逐條修復；可判斷為誤報者寫入 `.gitea/ai-review/exclusions.json`，已解決或已登記為誤報的 finding 逐檔就地自來源 wrapper 移除；處理過的議題於留言進度、把只來自議題的待人工處理問題寫回 findings 目錄後關閉；接著可將工作區變更依 conventional commit 類型分類提交、push 當前分支，並透過 Gitea API 對指定目標分支開 PR。當使用者說解決 findings、處理 AI review 問題、依議題編號修 code review 問題、修掉 `.gitea/ai-review` 問題、依嚴重度修復後分類提交，或要分類 commit 並 push/開 PR 時觸發。不適用於：產生 findings、單純 code review 不修改、只保存裁決紀錄，或只要不分類的單一 commit。 +argument-hint: "[--issue <編號|編號清單|all>] [--findings ] [--target <目標分支>] [--pr-desc ] [--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` 後也可移除（議題模式改在議題留言進度、把待人工處理問題寫回 findings 目錄後關閉議題），接著**必須盤點並處理工作區的所有變更**，再依 conventional commit 類型分門別類 commit，然後 **push 當前分支**，最後**透過 Gitea API 發 PR**。 +五階段 skill：先執行 **git 同步**（`git fetch` → 確認目前分支的遠端分支是否存在；存在則留在目前分支 `git pull` 更新，不存在則切換到 `develop`，再退而 `master` 後 `git pull` 更新；只有來源分支與 PR 目標分支相同時才建立新的工作分支；必要時告知並嘗試解衝突），再**蒐集問題來源**——**一律讀** `.gitea/ai-review/findings/` 目錄下的 wrapper 物件 JSON（取其 `findings` 陣列）與 `findings.json`，並依 `--issue`（無／單選／多選／全部）**額外**從 Gitea 議題留言解析嚴重問題，兩來源**合併去重**後逐條處理；成立問題修復後自來源 wrapper 逐檔就地移除，可判斷為誤報者寫入 `.gitea/ai-review/exclusions.json` 後也可移除（處理過的議題於留言進度、把只來自議題的待人工處理問題寫回 findings 目錄後關閉），接著**必須盤點並處理工作區的所有變更**，再依 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`；議題模式回寫議題（留言＋待人工處理寫回 findings 後關閉）；**無待處理問題則跳到階段 D** | +| B. 解決問題 | 一律讀 `.gitea/ai-review/findings/*.json`＋`findings.json`（wrapper，取 `findings`），依 `--issue`（無/單選/多選/全部）加讀議題留言 → 合併去重 → 依等級 🔴嚴重→🟠警告→🔵建議 逐條修復或判斷誤報 → 已解決者逐檔就地自來源 wrapper 移除，誤報寫入 `exclusions.json`；處理過的議題回寫（留言＋只來自議題的待人工寫回 findings 後關閉）；**無待處理問題則跳到階段 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 → 通知並清除內文 | @@ -37,10 +37,10 @@ argument-hint: "[--issue <議題編號>] [--findings ] [--findings <路徑>] [--target <目標分支>] [--pr-desc ] [--no-commit] [--no-pr] [--yes]` +格式：`[--issue <編號|編號清單|all>] [--findings <路徑>] [--target <目標分支>] [--pr-desc ] [--no-commit] [--no-pr] [--yes]` -- `--issue <議題編號>`：**議題模式**——問題來源改為指定的 Gitea 議題（對應 code-review 的建問題模式）。帶此參數時走階段 B0：讀議題標題／描述／全部留言，把符合嚴重問題留言格式的內容解析成 findings，逐條修復；修復後於階段 B5 在議題留言進度、把待人工處理問題寫回 `findings/` 目錄後關閉議題。此模式**不讀 `findings/` 目錄**（除非同時明確帶 `--findings`）。需要 `GITEA_TOKEN` 與可解析的 `origin` repo 座標。 -- `--findings <路徑>`：findings 來源路徑。**省略時預設讀目錄 `.gitea/ai-review/findings/` 下所有 `*.json`（每個檔為新版 wrapper 物件，取其 `findings` 陣列合併）**（相對於工作目錄根）。指定路徑時：目錄則讀該目錄所有 `*.json`；單一檔則讀該檔（同為 wrapper 物件）。與 `--issue` 併用時，兩者的問題都納入處理。 +- `--issue <無 / 編號 / 編號清單 / all>`：**額外**從 Gitea 議題（對應 code-review 建問題模式）撈問題，支援四種——**無**（省略此參數，不讀議題）、**單選**（`--issue 12`）、**多選**（`--issue 12,15,20` 逗號分隔）、**全部**（`--issue all`，讀 repo 內所有 open 議題）。帶此參數時走 B1b 解析議題嚴重問題留言為 findings；修復後於 B6 對每個處理過的議題留言進度、把「只來自議題」的待人工處理問題寫回 `findings/` 目錄後關閉該議題。需要 `GITEA_TOKEN` 與可解析的 `origin` repo 座標。**注意：不論有無此參數，findings 檔一律照讀（見 `--findings`），議題只是額外來源。** +- `--findings <路徑>`：findings 檔來源路徑。**不論是否帶 `--issue`，都一律讀取**；省略時預設讀目錄 `.gitea/ai-review/findings/` 下所有 `*.json` **與**（若存在）單一檔 `.gitea/ai-review/findings.json`（每個檔為新版 wrapper 物件，取其 `findings` 陣列）（相對於工作目錄根）。指定路徑時：目錄則讀該目錄所有 `*.json`；單一檔則讀該檔（同為 wrapper 物件）。findings 檔與 `--issue` 議題的問題會在 B1c 合併去重後一起處理。 - `--target <目標分支>`：PR 的目標分支。**省略時必須詢問使用者，不可猜測**；若需要避免把同名來源分支誤推到目標分支，可能會在階段 D 先詢問並於階段 E 沿用。 - `--pr-desc `：PR 描述形式。`full`＝完整版（重新分析 diff 總結）、`simple`＝簡單版（逐條列 commit 訊息）、或直接給自訂文字；省略時預設 `full`，不要為描述形式中斷詢問。 - `--no-commit`：只做階段 A/B（git 同步 + 修復 + 更新 findings/exclusions），**不**執行階段 C/D/E（不提交、不推送、不開 PR）。 @@ -140,7 +140,7 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)" --- -## 階段 B：讀取問題（findings 檔或 Gitea 議題）、依嚴重度逐條解決或登記誤報 +## 階段 B：蒐集問題（findings 檔 ＋ Gitea 議題）合併去重、依嚴重度逐條解決或登記誤報 新版 AI Code Review 的產出格式（對齊 `https://gitea.jsc.idv.tw/actions/code-review`，`src/index.js` / `src/lib/review.js`）： @@ -188,25 +188,44 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)" } ``` -- **建問題模式**：某些 review 不寫 findings 檔，改開 Gitea 議題、把嚴重問題**逐則發成議題留言**（見 B0）。 +- **建問題模式**：某些 review 不寫 findings 檔，改開 Gitea 議題、把嚴重問題**逐則發成議題留言**（見 B1）。 > 本 skill 只支援上述**新版格式**；讀到非 wrapper 物件（例如舊版頂層陣列 findings）時，回報格式不符並停止，不臆測欄位。 -### B0. 議題模式：從 Gitea 議題讀取問題（帶 `--issue` 時） +### B1. 蒐集問題來源並合併去重 -帶 `--issue <議題編號>` 時，**問題來源為該議題**（不讀 `findings/` 目錄，除非同時明確帶 `--findings`）： +**不論是否處理議題，都必須先讀取專案內的 findings 檔**；若有指定議題，再加上議題來源，兩者合併去重後才進入 B3。 -1. 解析 repo 座標（同 E3，從 `git remote get-url origin` 取 host／owner／repo），token 由 `GITEA_TOKEN` 提供。 -2. 讀議題本文與全部留言（分頁全取；輸出時遮蔽 token）： +**B1a. 讀 findings 檔（一律執行）** + +- 讀目錄 `.gitea/ai-review/findings/` 下所有 `*.json`（含子目錄、依檔名排序）**與**（若存在）單一檔 `.gitea/ai-review/findings.json`；每個檔皆為上述 **wrapper 物件**，取其 `findings` 陣列。 +- 指定 `--findings <路徑>` 時改讀該路徑（目錄則讀其所有 `*.json`；單一檔則讀該檔），仍須為 wrapper 物件。 +- 逐筆記住各 finding 的**來源檔路徑與所屬 wrapper**，供 B5 逐檔就地寫回；此步不得更動任何來源檔。 +- JSON 解析失敗或非 wrapper 物件 → 回報該檔路徑並停止，不臆測、不亂改檔。 + +**B1b. 讀議題（依 `--issue` 選擇）** + +`--issue` 支援四種形式： + +| 形式 | 寫法 | 行為 | +| --- | --- | --- | +| 無 | 省略 `--issue` | 不讀議題，只用 findings 檔 | +| 單選 | `--issue 12` | 讀議題 #12 | +| 多選 | `--issue 12,15,20` | 讀多個議題（逗號分隔） | +| 全部 | `--issue all` | 讀 repo 內**所有 open 議題** | + +1. 解析 repo 座標（同 E3），token 由 `GITEA_TOKEN` 提供。 +2. `all` 時先取所有 open 議題（分頁全取）：`GET /repos///issues?state=open&type=issues`。 +3. 對每個選定議題讀本文與全部留言（分頁全取；輸出遮蔽 token）： ```bash curl -sS -H "Authorization: token ${GITEA_TOKEN}" \ - "https:///api/v1/repos///issues/<議題編號>" + "https:///api/v1/repos///issues/<編號>" curl -sS -H "Authorization: token ${GITEA_TOKEN}" \ - "https:///api/v1/repos///issues/<議題編號>/comments" + "https:///api/v1/repos///issues/<編號>/comments" ``` -3. 解析每則符合「嚴重問題留言」格式（action `severeCommentBody` 產出）的留言為一條 finding： +4. 解析每則符合「嚴重問題留言」格式（action `severeCommentBody` 產出）的留言為一條 finding： ``` ### ｜ @@ -219,28 +238,28 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)" ``` - 逐則映射為 `{ reviewer, severity, file, startLine, endLine, problem, suggestion, suggestedCode }`，並記住來源留言 id（供 B5 回報）；議題標題／描述作為整體背景脈絡納入判斷。 -4. 解析不到任何嚴重問題留言 → 回報並停止議題模式，不臆測。 -5. 議題模式下沒有現成 findings 檔可更新；解析出的問題直接進入 B2/B3，處理結果於 B5 回寫議題（留言＋把待人工處理問題寫回 `findings/` 目錄後關閉議題），誤報仍寫入 `exclusions.json`。 + 映射為 `{ reviewer, severity, file, startLine, endLine, problem, suggestion, suggestedCode }`，並記住**來源議題編號與留言 id**（供 B6 回寫）；議題標題／描述作為整體背景脈絡納入判斷。 +5. 某議題解析不到任何嚴重問題留言 → 記錄並略過該議題（`all`／多選時不因單一議題無問題而中斷整批），不臆測。 -### B1. 讀取 findings 與 exclusions（檔案／目錄來源） +**B1c. 合併去重** -**未帶 `--issue` 時**，依序定位 findings 來源： +- 把 B1a（findings 檔）與 B1b（議題）的所有 finding 合併成單一待處理清單。 +- **去重鍵**：`file` + `startLine` + `endLine` + `reviewer` + `problem`（正規化空白後比對；行號缺失時以 `file` ＋ `reviewer` ＋ `problem` 為準）。 +- 重複的 finding **合併其來源清單**（同一問題可能同時來自某 findings 檔與某議題），只列入一次；後續解決後要**同時回寫所有來源**（B5 移除該 finding 檔項目、B6 計入相關議題的關閉與留言）。 +- 合併去重後即為待處理 finding 清單，進入 B3 排序。 -1. **省略 `--findings`**：讀目錄 `.gitea/ai-review/findings/` 下所有 `*.json`（含子目錄、依檔名排序）；每個檔皆為上述 **wrapper 物件**，取其 `findings` 陣列合併為待處理清單。 -2. **指定 `--findings <路徑>`**：路徑為目錄則比照第 1 點；為單一檔則讀該檔（同為 wrapper 物件），取其 `findings`。 -3. **合併時逐筆記住各 finding 的來源檔路徑與其所屬 wrapper**，供 B4 逐檔就地寫回；此步不得更動任何來源檔。 -4. **目錄不存在／無 `*.json`／各檔 `findings` 皆空** → 視為「無問題待解決」，輸出告知。`--no-commit` 時就此結束；否則**跳過階段 C 直接進入階段 D**。 -5. **JSON 解析失敗或非 wrapper 物件** → 回報錯誤與檔案路徑並停止，不臆測、不亂改檔。 +**無任何 finding**（findings 檔皆空／不存在，且未選議題或選到的議題都無問題）→ 視為「無問題待解決」，輸出告知。`--no-commit` 時就此結束；否則**跳過階段 C 直接進入階段 D**。 -`exclusions.json` 不存在時視為空陣列（頂層陣列）；解析失敗或非陣列時回報，並在本次不寫入誤報（避免破壞既有內容、需人工確認）。 +### B2. 讀取 exclusions -### B2. 依嚴重等級排序 +`.gitea/ai-review/exclusions.json` 不存在時視為空陣列（頂層陣列）；解析失敗或非陣列時回報，並在本次不寫入誤報（避免破壞既有內容、需人工確認）。 + +### B3. 依嚴重等級排序 把所有 finding 依等級由高到低排序：**🔴 嚴重 → 🟠 警告 → 🔵 建議**。 `severity` 為中文 `嚴重` / `警告` / `建議`；容錯對應：`嚴重`/`critical`/`high`/`blocker`→🔴、`警告`/`warning`/`major`→🟠、`建議`/`info`/`low`/`minor`→🔵；無法辨識者**排在最後**並標「等級未知」。排序後先處理高等級。 -### B3. 逐條處理（高等級先） +### B4. 逐條處理（高等級先） 輸出一張「處理計畫」表，除非使用者要求確認或遇到必要決策，否則**依排序由上而下逐條**處理： @@ -251,30 +270,30 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)" 1. 讀取對應檔案與其脈絡（依 `file` 與 `startLine`–`endLine`）。 2. 先判斷是否為誤報或不適用：例如程式碼脈絡證明指控不成立、已有等價防護、命中 `exclusions.json` 已知排除、CI/CD 必要權限、或 finding 對非本次變更做不合理要求。 -3. **可判斷為誤報** → 不改程式碼；將排除條目 append 到 `.gitea/ai-review/exclusions.json`（新版欄位，見 B4），保留原始問題文字與語意；若已有等價 exclusion（同 `file`＋`reviewer`＋`problem` 高度相似）則不重複新增。 +3. **可判斷為誤報** → 不改程式碼；將排除條目 append 到 `.gitea/ai-review/exclusions.json`（新版欄位，見 B5），保留原始問題文字與語意；若已有等價 exclusion（同 `file`＋`reviewer`＋`problem` 高度相似）則不重複新增。 4. **確認為真問題且可安全修復** → 依 `suggestion`／`suggestedCode`／`problem` 在程式碼中實作最小合理修正；建議含糊或與現況不符時，依原始碼脈絡做最小且合理的修正。 5. **無法安全自動修復或無法確認真偽**（例如需求不明、牽涉設計取捨、檔案不存在）→ **不硬改**，標記為「待人工處理」並記錄原因，繼續下一條。 -6. 逐條完成後，記錄該條結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因）；議題模式另記來源留言 id。 +6. 逐條完成後，記錄該條結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因），並保留該 finding 依 B1c 記錄的**所有來源**（findings 檔路徑 / 議題編號＋留言 id）供 B5／B6 回寫。 > 一次只處理一條、修完再處理下一條，避免互相干擾；同檔多條問題可合併讀取但仍逐條套用修正。 -### B4. 更新 findings 檔與 exclusions.json（檔案／目錄來源） +### B5. 更新 findings 檔與 exclusions.json -**所有條目處理完畢後**依結果更新（**議題模式無 findings 檔，跳過本節 findings 寫回，改走 B5**）： +**所有條目處理完畢後**，依每條 finding 於 B1c 記錄的來源回寫。**只來自議題、無任何 findings 檔來源的 finding 不在本節處理**（交由 B6 回寫議題與新建 findings 檔）；有 findings 檔來源者（含同時來自議題與檔案的）依下列更新其來源檔： -- **只有確認已解決，或已確認為誤報並成功寫入 `exclusions.json` 的 finding，才可從來源 findings 檔移除**；待人工處理、無法確認真偽、修復未驗證成功者一律保留。 -- **逐檔就地更新（wrapper 物件）**：依 B1 記錄的來源檔，把該檔 `findings` 陣列中要保留的 finding 留下、其餘移除，**只改該檔的 `findings` 陣列**；wrapper 其他欄位（`generatedAt`、`commitSha`、`prNumber`、`tool`、`excluded`）原樣保留，**不得**把多個來源檔合併成單一檔。某檔 `findings` 全數已解決／誤報時，把該檔的 `findings` 寫成 `[]`（保留檔案與 wrapper、維持目錄結構，不刪檔）。 +- **只有確認已解決，或已確認為誤報並成功寫入 `exclusions.json` 的 finding，才可從來源 findings 檔移除**；待人工處理、無法確認真偽、修復未驗證成功者一律保留於來源檔。 +- **逐檔就地更新（wrapper 物件）**：依 B1a 記錄的來源檔，把該檔 `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` 時） +### B6. 議題收尾：回寫並關閉每個處理過的議題（B1b 有讀到議題時） -議題模式所有問題處理完畢後： +**對每個處理過的議題**（單選／多選／全部各自處理），依該議題涵蓋的 finding 結果收尾： -1. **在議題留言回報進度**（Gitea API `POST /repos///issues/<編號>/comments`，body 以 UTF-8 JSON 檔帶入、token 不 echo）：以表格逐條列出結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因）。 -2. **把「待人工處理」的問題寫回 findings**：若有任何 ⏭️ 待人工處理 的問題，將它們寫成一個**新的 wrapper 物件檔**到目錄 `.gitea/ai-review/findings/`（檔名用台北時間戳，如 `2026-07-20-14:30:15.json`），格式同本階段開頭的 findings wrapper，讓後續 review／`resolve` 可追蹤： +1. **在該議題留言回報進度**（Gitea API `POST /repos///issues/<編號>/comments`，body 以 UTF-8 JSON 檔帶入、token 不 echo）：以表格逐條列出該議題各問題結果（✅ 已解決 / 🚫 誤報已寫入 exclusions / ⏭️ 待人工處理 + 原因）；同時來自 findings 檔的問題，一併註明已於 B5 更新來源檔。 +2. **把「待人工處理」且只來自議題的問題寫回 findings**：收集所有 ⏭️ 待人工處理 且**沒有任何 findings 檔來源**的問題（已在 findings 檔的靠 B5 保留，不重複寫），寫成一個**新的 wrapper 物件檔**到目錄 `.gitea/ai-review/findings/`（檔名用台北時間戳，如 `2026-07-20-14:30:15.json`），格式同本階段開頭的 findings wrapper： ```json { @@ -287,20 +306,20 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)" } ``` - - `findings` 帶入所有待人工處理的問題（欄位沿用 B0 從議題留言解析出的 `reviewer`／`severity`／`file`／`startLine`／`endLine`／`problem`／`suggestion`／`suggestedCode`，並補流水號 `id`）；`excluded` 固定為 `[]`。 - - 無待人工處理時**不建立**此檔。 + - `findings` 帶入這些待人工處理問題（欄位沿用 B1b 從議題留言解析出的欄位，並補流水號 `id`）；`excluded` 固定為 `[]`。多個議題可合寫成同一個檔。 + - 沒有「只來自議題」的待人工處理問題時**不建立**此檔。 - 此新檔屬工作區變更，會在階段 C 一併分類提交（歸 `chore`）。UTF-8（不含 BOM）、結尾保留一個換行。 -3. **關閉議題（一律關閉）**：問題已全部分流為 ✅ 已修復 / 🚫 誤報寫入 exclusions / ⏭️ 待人工處理已寫回 findings 追蹤，因此收尾時關閉該議題： +3. **關閉該議題（一律關閉）**：該議題的問題已全部分流為 ✅ 已修復 / 🚫 誤報寫入 exclusions / ⏭️ 待人工處理已寫回 findings（B5 保留或 B6 新建）追蹤，因此收尾時關閉： ```bash curl -sS -X PATCH -H "Authorization: token ${GITEA_TOKEN}" \ -H "Content-Type: application/json" \ - "https:///api/v1/repos///issues/<議題編號>" \ + "https:///api/v1/repos///issues/<編號>" \ --data '{"state":"closed"}' ``` - 關閉前先在步驟 1 的留言中標明各待人工處理項已寫回哪個 findings 檔與原因，確保關閉議題不會遺失待辦。 -4. 誤報仍寫入 `exclusions.json`（同 B4），讓後續 review 沿用。 + 關閉前先在步驟 1 的留言中標明各待人工處理項寫回哪個 findings 檔與原因，確保關閉議題不會遺失待辦。 +4. 誤報仍寫入 `exclusions.json`（同 B5），讓後續 review 沿用。 --- @@ -515,7 +534,7 @@ curl -sS -X POST \ 各階段執行後輸出： - **階段 A**：git fetch / pull 是否成功、當前分支是否存在於遠端（不存在時切到 develop／master 的結果）、若來源分支與目標分支同名是否已建立新工作分支、是否發生衝突、衝突是否已解決或仍需人工處理。 -- **階段 B**：問題來源（findings 目錄或議題 `#N`）、已解決 N 條（依等級分佈）、誤報寫入 exclusions M 條、待人工處理 K 條（列出原因）、來源 findings 檔保留／移除筆數與 `exclusions.json` 新增筆數；議題模式另報已在議題留言、待人工處理問題寫回哪個 findings 檔、以及已關閉議題；若無待處理問題，說明已跳過階段 C 直接進入階段 D。 +- **階段 B**：問題來源（findings 檔筆數＋議題來源：無／`#N`／多個／全部）、合併去重後總數、已解決 N 條（依等級分佈）、誤報寫入 exclusions M 條、待人工處理 K 條（列出原因）、來源 findings 檔保留／移除筆數與 `exclusions.json` 新增筆數；有處理議題時另報各議題已留言、只來自議題的待人工問題寫回哪個 findings 檔、以及各議題已關閉；若無待處理問題，說明已跳過階段 C 直接進入階段 D。 - **階段 C**：建立了哪幾個 commit（type＋訊息＋檔數），或為何略過（`--no-commit` / 無變更）。 - **階段 D**：push 成功與否、用了哪種方式（認證管理器／token／使用者指定），遠端分支名；若無 commit 可 push，說明已跳過 push 直接進入階段 E。 - **階段 E**：PR 連結／編號、目標分支、描述形式；並提醒已（或請使用者）清除對話內文以防 token 外洩。 @@ -525,10 +544,10 @@ curl -sS -X POST \ ## 呼叫方式 格式：`[--findings <路徑>] [--target <目標分支>] [--pr-desc ] [--no-commit] [--no-pr] [--yes]` — -全部可省略（findings 預設讀目錄 `.gitea/ai-review/findings/` 下所有 wrapper `*.json`；帶 `--issue <編號>` 改由 Gitea 議題取問題；目標分支省略時必問、不猜測）。token 一律由環境變數（如 `GITEA_TOKEN`）提供。 +除 `--target` 外皆可省略（findings 一律讀目錄 `.gitea/ai-review/findings/` 下所有 wrapper `*.json` 與 `findings.json`；`--issue` 無/單選/多選/全部 額外從 Gitea 議題取問題，與 findings 合併去重；目標分支省略時必問、不猜測）。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 問題」）自動觸發 | +| Claude Code / Antigravity | `/jsc:code-review-resolve`，或 `/jsc:code-review-resolve --target develop --pr-desc full --yes`、`/jsc:code-review-resolve --issue 12,15 --target master`、`/jsc:code-review-resolve --issue all --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 all --target master`，或用 `/skills` 選單 | +| OpenCode | 描述需求（如「讀 .gitea/ai-review/findings/ 目錄與 findings.json 的 wrapper findings，再併入議題 #12、#15 的 code review 問題去重後依嚴重度逐條處理，更新 findings/exclusions，把工作區變更依 conventional commit 分類提交，push 後對 develop 發 PR（完整版描述）」）自動觸發 | -- 2.53.0 From 8547f0af410b16791e063147af5df8b21be552fe Mon Sep 17 00:00:00 2001 From: Jeffery Date: Mon, 20 Jul 2026 18:14:57 +0800 Subject: [PATCH 2/2] =?UTF-8?q?chore(plugin=20=E7=89=88=E6=9C=AC):=20?= =?UTF-8?q?=E7=89=88=E6=9C=AC=E8=99=9F=20bump=20=E8=87=B3=200.0.9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .claude-plugin/plugin.json | 2 +- .codex-plugin/plugin.json | 2 +- plugin.json | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.claude-plugin/plugin.json b/.claude-plugin/plugin.json index d14b449..8daef6b 100644 --- a/.claude-plugin/plugin.json +++ b/.claude-plugin/plugin.json @@ -1,6 +1,6 @@ { "name": "jsc", - "version": "0.0.8", + "version": "0.0.9", "description": "JSC 程式碼工作流 plugin（Claude Code / Codex / Antigravity / OpenCode），提供 Gitea AI review findings 修復/PR 流程、Gitea 議題 TODO 彙整與逐項實作、C# / .NET NuGet 套件更新、Gitea 專案批次同步，以及 action / Dockerfile 標準化流程；於 Claude Code 以 /jsc: 前綴呼叫。", "skills": "./skills", "author": { diff --git a/.codex-plugin/plugin.json b/.codex-plugin/plugin.json index 7bf652e..5e92f36 100644 --- a/.codex-plugin/plugin.json +++ b/.codex-plugin/plugin.json @@ -1,6 +1,6 @@ { "name": "jsc", - "version": "0.0.8", + "version": "0.0.9", "description": "JSC 程式碼工作流 plugin，提供 Gitea AI review findings 修復/PR 流程、Gitea 議題 TODO 彙整與逐項實作、C# / .NET NuGet 套件更新、Gitea 專案批次同步，以及 action / Dockerfile 標準化流程。", "skills": "./skills" } diff --git a/plugin.json b/plugin.json index bc1c91d..c1d3a04 100644 --- a/plugin.json +++ b/plugin.json @@ -1,6 +1,6 @@ { "name": "jsc", - "version": "0.0.8", + "version": "0.0.9", "description": "JSC 程式碼工作流 plugin，提供 Gitea AI review findings 修復/PR 流程、Gitea 議題 TODO 彙整與逐項實作、C# / .NET NuGet 套件更新、Gitea 專案批次同步，以及 action / Dockerfile 標準化流程；於 Antigravity 以 /jsc: 前綴呼叫。", "skills": "./skills/" } -- 2.53.0