release: 將 AI 程式碼審查 action 發布到 master #2

Merged

admin merged 32 commits from develop into master 2026-07-03 10:07:51 +00:00

Conversation 0 Commits 32 Files Changed 457

+79547 -319

jiantw83 commented 2026-07-03 10:07:10 +00:00

Member

Copy Link

Copy Source

變更摘要

將 develop 上完整的 AI 程式碼審查 Gitea Node action 發布到 master（領先 32 個 commit）。此為首個正式版本：一個以多角色 LLM 審查 PR diff、產生 findings、發布 Gitea Review、並具記憶／收斂能力的 action。

主要內容

• 核心管線（src/main.js，Step1–Step11）：前置驗證 → 自動提交檢查 → PR 對話收斂 → 6 角色分析產生 findings → 合併去重 → 排除規則＋防守方誤報裁決 → 發布 Review → JSON 驗證 → 記憶區 commit/push → 嚴重問題把關。
• 參數自動接線（config.js）：INPUT_* 優先，PR 參數自 runner 事件 payload 帶入；使用端只需 with: token。
• LLM 抽象（llm.js）：自動偵測 codex / claude / antigravity / opencode CLI；mapWithConcurrency 讓角色分析、補行號、誤報裁決以並行 sub-agent 執行（預設不限併發，AI_ASSISTANT_CONCURRENCY 可限流）。
• diff 過濾（.reviewignore + gitea.js）：外部化排除清單，並強制排除任意深度 node_modules/，避免超過 LLM 輸入上限。
• 前置模型檢查（preflight.js）：Step2 向 codex 模型清單端點確認 auth 有效且 model 可用，提早分辨「認證失效」與「模型無權限」。
• 測試：13 個模組 + main 管線整合測試，共約 300+ 筆，全數通過。

影響範圍

• 依賴採 vendored node_modules（隨 repo 進版控，runner 不需 npm install）。
• CI：test（AI 審查）只在目標為 develop 時執行；目標為 master 時跳過（本 release PR 即不會觸發自我審查）。

風險 / 注意事項

• config.js 全域關閉 TLS 憑證驗證（NODE_TLS_REJECT_UNAUTHORIZED=0）以相容內部自簽憑證，僅限受信任內網。
• exclusions.json 中登記了數個「已知的安全設計待辦」（信任邊界：.reviewignore／exclusions 信任 PR head、可偽造的 bot-skip marker、resolve 全留言）——這些是經決策暫時接受、尚未修復的真實問題，合併到 master 後仍為後續強化項目。

驗證

• 全套件 309 pass / 0 fail。
• 已於本機煙霧測試確認 node src/main.js 會實際啟動 pipeline（修復先前 node action runtime 空跑的回歸）。

## 變更摘要 將 `develop` 上完整的 **AI 程式碼審查 Gitea Node action** 發布到 `master`（領先 32 個 commit）。此為首個正式版本：一個以多角色 LLM 審查 PR diff、產生 findings、發布 Gitea Review、並具記憶／收斂能力的 action。 ## 主要內容 - **核心管線（`src/main.js`，Step1–Step11）**：前置驗證 → 自動提交檢查 → PR 對話收斂 → 6 角色分析產生 findings → 合併去重 → 排除規則＋防守方誤報裁決 → 發布 Review → JSON 驗證 → 記憶區 commit/push → 嚴重問題把關。 - **參數自動接線（`config.js`）**：`INPUT_*` 優先，PR 參數自 runner 事件 payload 帶入；使用端只需 `with: token`。 - **LLM 抽象（`llm.js`）**：自動偵測 codex / claude / antigravity / opencode CLI；`mapWithConcurrency` 讓角色分析、補行號、誤報裁決以並行 sub-agent 執行（預設不限併發，`AI_ASSISTANT_CONCURRENCY` 可限流）。 - **diff 過濾（`.reviewignore` + `gitea.js`）**：外部化排除清單，並強制排除任意深度 `node_modules/`，避免超過 LLM 輸入上限。 - **前置模型檢查（`preflight.js`）**：Step2 向 codex 模型清單端點確認 auth 有效且 model 可用，提早分辨「認證失效」與「模型無權限」。 - **測試**：13 個模組 + `main` 管線整合測試，共約 **300+ 筆**，全數通過。 ## 影響範圍 - 依賴採 vendored `node_modules`（隨 repo 進版控，runner 不需 `npm install`）。 - CI：`test`（AI 審查）只在**目標為 develop** 時執行；目標為 master 時跳過（本 release PR 即不會觸發自我審查）。 ## 風險 / 注意事項 - `config.js` 全域關閉 TLS 憑證驗證（`NODE_TLS_REJECT_UNAUTHORIZED=0`）以相容內部自簽憑證，僅限受信任內網。 - **`exclusions.json` 中登記了數個「已知的安全設計待辦」**（信任邊界：`.reviewignore`／`exclusions` 信任 PR head、可偽造的 bot-skip marker、resolve 全留言）——這些是**經決策暫時接受、尚未修復**的真實問題，合併到 master 後仍為後續強化項目。 ## 驗證 - 全套件 **309 pass / 0 fail**。 - 已於本機煙霧測試確認 `node src/main.js` 會實際啟動 pipeline（修復先前 node action runtime 空跑的回歸）。

jiantw83 added 32 commits 2026-07-03 10:07:10 +00:00

刪除 README.md 18314b88a3

feat(ai-code-review): 新增 AI 程式碼審查主程式、模組與測試 f46dfe6258

fix(action 進入點): 將 main 指向 src/main.js 2cf45db210

chore(專案設定): 更新 CI workflow 並移除範本 package.json e15f3f41f3

fix(config): comment token 缺省時回退至 token 7c35dc0601

fix(ci): 串接 build→test 的 VERSION 輸出 64af9e9e92

feat(preflight): Step2 前置驗證加入 codex 模型清單檢查 1118606e97

test(preflight): 補 codex 模型檢查與 auth 失效測試 51e9568ccf

fix(llm): 錯誤訊息改抽尾端錯誤，避免被 codex banner 洗掉 fa9be791ee

test(llm): 補 extractMeaningfulError 測試 9e70bb2245

fix(gitea): diff 過濾排除 node_modules 與 lock 檔，避免超出 LLM 輸入上限 6868dbdc8f

test(gitea): 補 filterDiff 排除 node_modules／lock 測試 3393e43877

chore: update ai-review findings [ai-review-bot][failure] fc5baf32db

fix(審查流程): 修復 AI 去重超量、clone 失敗誤持久化、留言行號漏 new_position 2c0ac71c08

chore(ai-review 狀態): 移除已修與誤報 findings、登記 exclusions 1a9a6bce5c

chore: update ai-review findings [ai-review-bot][failure] 4b4b934cda

feat(diff 過濾): 改讀 .reviewignore 外部化 diff 排除清單 47aa199e5e

test(gitea): 補 .reviewignore 解析與載入測試 1279cae575

chore: update ai-review findings [ai-review-bot][failure] 94d86809d5

perf(LLM 併發): 角色分析與誤報/補行號裁決改為並行 sub-agent（預設不限併發） c521451b66

test(llm): 補 mapWithConcurrency 測試 93142e2250

chore: update ai-review findings [ai-review-bot][failure] 0be25e667c

chore(workflows): CI 目標非 develop 跳過 test job、CD 加印 gitea context 與 fetch-tags 11205d5b82

chore: update ai-review findings [ai-review-bot][failure] fda3331dd4

refactor(main): 匯出 main 並守衛 auto-run 以利測試載入 a7824cff29

test(main): 新增 main pipeline 各分支整合測試 2f294845c8

chore(test 設定): 啟用 test module mocks flag e54c3f5fbe

chore(ci): 抽出 IS_BETA 統一 prerelease 與 test 判斷 69122dca3e

chore(ai-review 狀態): 解決 critical #2、其餘 findings 轉入 exclusions d1293bb950

fix(main): 改用 AI_REVIEW_SKIP_MAIN 守衛 auto-run，修復 node action runtime 不執行 main 的回歸 5fbfec3aaf

test(main): 設 AI_REVIEW_SKIP_MAIN 略過自動執行 5c2c37f9d1

Merge pull request 'feat: 導入 AI 程式碼審查 action 並修正進入點與參數接線' (#1) from ai-review-resolve/develop-20260702-160700 into develop ... c2f41b16eb

Reviewed-on: #1

admin merged commit ec61b282fa into master 2026-07-03 10:07:51 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No labels

Milestone

No items

No Milestone

Assignees

Clear assignees

admin (系統管理員) jiantw83 (Jeffery)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: actions/ai-code-review#2