Merge pull request 'feat(code-review-resolve): findings 一律讀取+議題無/單選/多選/全部合併去重、Stage B 重新編號(0.0.9)' (#12) from develop into master
Reviewed-on: #12 Reviewed-by: 系統管理員 <1+admin@noreply.localhost>
This commit was merged in pull request #12.
This commit is contained in:
@@ -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": {
|
||||
|
||||
@@ -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"
|
||||
}
|
||||
|
||||
+1
-1
@@ -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/"
|
||||
}
|
||||
|
||||
@@ -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 <findings 檔或目錄路徑>] [--target <目標分支>] [--pr-desc <full|simple|自訂文字>] [--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 <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` 後也可移除(議題模式改在議題留言進度、把待人工處理問題寫回 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 檔或目錄路
|
||||
|
||||
## 參數
|
||||
|
||||
格式:`[--issue <議題編號>] [--findings <路徑>] [--target <目標分支>] [--pr-desc <full|simple|自訂文字>] [--no-commit] [--no-pr] [--yes]`
|
||||
格式:`[--issue <編號|編號清單|all>] [--findings <路徑>] [--target <目標分支>] [--pr-desc <full|simple|自訂文字>] [--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 <full|simple|自訂文字>`: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/<owner>/<repo>/issues?state=open&type=issues`。
|
||||
3. 對每個選定議題讀本文與全部留言(分頁全取;輸出遮蔽 token):
|
||||
|
||||
```bash
|
||||
curl -sS -H "Authorization: token ${GITEA_TOKEN}" \
|
||||
"https://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>"
|
||||
"https://<host>/api/v1/repos/<owner>/<repo>/issues/<編號>"
|
||||
curl -sS -H "Authorization: token ${GITEA_TOKEN}" \
|
||||
"https://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>/comments"
|
||||
"https://<host>/api/v1/repos/<owner>/<repo>/issues/<編號>/comments"
|
||||
```
|
||||
|
||||
3. 解析每則符合「嚴重問題留言」格式(action `severeCommentBody` 產出)的留言為一條 finding:
|
||||
4. 解析每則符合「嚴重問題留言」格式(action `severeCommentBody` 產出)的留言為一條 finding:
|
||||
|
||||
```
|
||||
### <emoji> <severity>|<badge> <reviewer>
|
||||
@@ -219,28 +238,28 @@ source_branch="$(git rev-parse --abbrev-ref HEAD)"
|
||||
<suggestedCode>
|
||||
```
|
||||
|
||||
逐則映射為 `{ 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/<owner>/<repo>/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/<owner>/<repo>/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://<host>/api/v1/repos/<owner>/<repo>/issues/<議題編號>" \
|
||||
"https://<host>/api/v1/repos/<owner>/<repo>/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 <full|simple|自訂文字>] [--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(完整版描述)」)自動觸發 |
|
||||
|
||||
Reference in New Issue
Block a user