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

.gitea/ai-review/exclusions.json

@@ -0,0 +1,44 @@
+[
+  {
+    "location": "src/config.js:7",
+    "role": "Assassin",
+    "original_finding": "這裡把 `NODE_TLS_REJECT_UNAUTHORIZED` 全域設為 `0`，等於讓整個 Node 程序放棄 TLS 憑證驗證。攻擊者只要能站到 runner 與 Gitea／LLM／任何 HTTPS API 之間，就能用偽造憑證攔截或竄改 diff、review 結果、token 驗證流程，甚至偷走 Authorization header。",
+    "reason": "內部自架 Gitea／自簽憑證環境的刻意設計（見 config.js 註解）。如需強化可改用 NODE_EXTRA_CA_CERTS，屬人工決策而非誤判。"
+  },
+  {
+    "location": "src/config.js:53",
+    "role": "Assassin",
+    "original_finding": "這個 helper 直接建立 `rejectUnauthorized: false` 的 HTTPS agent，後續 Gitea API 與 preflight 都會用它。攻擊者若能進行中間人攻擊，就能假冒 Gitea 回傳惡意 diff、偽造 comment/review API 回應，或攔截寫入用 token。",
+    "reason": "同 config.js:7，為相容內部自簽憑證的刻意設計；預設不驗證憑證僅限受信任內網使用。"
+  },
+  {
+    "location": "src/main.js:2",
+    "role": "Bard",
+    "original_finding": "這一行 import 把大量設定常數擠成長長一串，讀起來像沒有換氣的樂句，與後續同檔案多個長 import 一起讓檔案開頭難以掃描。",
+    "reason": "多行 import 排版為風格偏好，非缺陷；本專案採現行單行分組匯入風格。"
+  },
+  {
+    "location": "src/findings.js:4",
+    "role": "Bard",
+    "original_finding": "這行把四個 prompt/role helper 壓在同一行，與檔案中龐大的流程函式相比，開頭的依賴清單先失了拍，降低可讀性。",
+    "reason": "多行 import 排版為風格偏好，非缺陷。"
+  },
+  {
+    "location": "src/gitea.js:2",
+    "role": "Bard",
+    "original_finding": "Gitea 設定匯入一口氣列出八個名稱，行寬過長，讓讀者難以快速分辨這個模組真正依賴哪些環境值。",
+    "reason": "多行 import 排版為風格偏好，非缺陷。"
+  },
+  {
+    "location": "src/comments.js:11",
+    "role": "Bard",
+    "original_finding": "大量私有輔助函式都配上篇幅很長的 JSDoc，許多內容只是重述程式碼表面行為，註解的聲量蓋過了旋律本身。",
+    "reason": "詳盡 JSDoc 為本專案 doc-funcs 流程的刻意文件化風格，非過度註解。"
+  },
+  {
+    "location": "src/main.js:18",
+    "role": "Bard",
+    "original_finding": "main() 前的 JSDoc 幾乎把整條 pipeline 逐步重寫一次，和函式內 Step 註解重複，維護時很容易變成兩份會走調的文件。",
+    "reason": "main() 的 pipeline 概述 JSDoc 為刻意文件化風格，與 Step 註解並存屬設計選擇。"
+  }
+]

.gitea/ai-review/findings.json

@@ -0,0 +1,186 @@
+[
+  {
+    "level": "critical",
+    "role": "Assassin",
+    "location": "src/resolve.js:253",
+    "problem": "這裡會把 PR 上所有未解決的 review comment ID 全部送去 resolve，而不是只處理 AI Review bot 自己建立的 thread。攻擊者只要開 PR 觸發這個 action，就可能讓 bot 關閉人類審查者留下的安全疑慮或阻擋性對話，繞過人工審查流程。",
+    "suggestion": "只 resolve 可證明由本 bot 建立且格式符合預期的 comment，例如檢查作者、固定 marker、review body 簽章或 botFinding 解析結果；人類留言與未知格式留言不得自動關閉。",
+    "is_new": true
+  },
+  {
+    "level": "critical",
+    "role": "Mage",
+    "location": "src/findings.js:471",
+    "problem": "當排除條目有 location 或 role 時，這裡直接把文字比對結果短路成 true。最小重現：exclusions.json 只有 `{ \"location\": \"app/a.js:10\", \"original_finding\": \"誤報 A\" }`，新的 finding 是 `app/a.js:99` 且 suggestion 完全不同，仍會因同檔案而被排除，導致真問題被靜默丟掉。",
+    "suggestion": "不要用 `exPath || ex.role ? true : textMatches` 跳過文字比對；應至少要求位置精確匹配到同一行，或在同檔/同角色時仍必須通過 `textMatches`，例如 `return locationMatches && roleMatches && textMatches`，並明確定義 suggestion 空白時才是萬用規則。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Assassin",
+    "location": "src/findings.js:14",
+    "problem": "這裡把未信任的 Git diff 直接送進 LLM。攻擊者可以在新增程式碼或註解中塞入提示詞注入內容，例如要求模型忽略安全問題、回傳空陣列或偽造低風險 findings，藉此讓自動安全審查失明。",
+    "suggestion": "在分析 prompt 中明確標示 diff 是不可信資料，要求模型忽略 diff 內任何指令；同時加入結構化封裝、輸出 schema 驗證與必要的規則式安全檢查，避免完全依賴可被 prompt injection 操控的 LLM 判斷。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Leo",
+    "location": "src/main.js:36",
+    "problem": "`main()` 把前置驗證、bot commit 判斷、對話收斂、角色分析、合併去重、排除、發布、JSON 驗證、commit/push 與 gate 全部塞在同一個 190 行左右的函式裡，且中間散落多個 `process.exit()`。六個月後要改其中任一步驟時，很難隔離副作用，也不容易針對單一階段寫單元測試。",
+    "suggestion": "將每個 Step 拆成可注入相依、回傳明確結果的函式，例如 `runAnalysisStep()`、`runFilteringStep()`、`runPublishStep()`；最外層再統一把結果轉成 exit code，讓流程控制與業務邏輯分離。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Leo",
+    "location": "src/findings.js:381",
+    "problem": "`loadExclusions()` 同時負責讀檔、解析多種格式、正規化、去重、記錄 repo 狀態、改寫原檔、鏡像寫入與建立 AI prompt 摘要。這個函式的職責過多，之後只要調整 exclusions 格式或同步策略，就很容易牽動不相關行為。",
+    "suggestion": "拆成 `readExclusionsFile()`、`normalizeExclusionsData()`、`canonicalizeExclusionsFile()`、`logExclusionMetadata()` 等小函式，讓讀取、轉換、寫回與診斷各自可測。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Mage",
+    "location": "src/findings.js:466",
+    "problem": "這裡優先使用 `ex.textKey`，但 `textKey` 是由 `toKeyText()` 產生的無分隔且未轉小寫文字，而 findingText 是 `normalizeText()` 產生的小寫、以空白分隔文字。最小重現：排除文字 `Update tests` 會變成 `Updatetests`，finding suggestion 會變成 `update tests`，兩邊互相 `includes` 都不成立，導致純文字排除規則失效。",
+    "suggestion": "排除條目與 finding 應使用同一套正規化函式比對；例如改存並使用 `normalizeText(ex.text || ex.suggestion || ex.title || '')`，或讓 finding 也轉成同樣的 compact/lowercase key。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "src/main.js:118",
+    "problem": "Step5 的角色分析與流程分支是整個 action 的核心，但目前測試沒有覆蓋 main orchestrator：例如所有角色分析都失敗時應 exit 1、部分角色失敗時仍繼續、diff 為空時 exit 0、critical finding 最後應讓 workflow 失敗。這些行為沒有被驗證，等於 pipeline 成敗判斷還沒通過試煉。",
+    "suggestion": "補上 main 流程層級測試，透過 mock getPRDiff、loadRoles、analyzeWithRole、postFindingsReview、process.exit 等相依，至少覆蓋：diff 空、全部分析失敗、部分分析失敗但繼續、產生 critical 後 exit 1、無 critical 後正常通過。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "src/main.js:166",
+    "problem": "Step7 會把 reconcile.excludedFindings 追加到 exclusions，接著再載入並套用排除規則，但目前缺少整合測試驗證「誤報對話 → 寫入 exclusions → 後續 findings 被排除」這條關鍵路徑。若 append/load/apply 任一環節接錯 workspace 或 mirror，單元測試不一定會抓到。",
+    "suggestion": "補一個接近流程層級的測試，mock reconcileConversations 回傳 excludedFindings，準備一筆會被排除的新 finding，驗證 appendExclusions 寫入的檔案被 loadExclusions 讀到，且最後 save/post 的 filtered findings 不含該誤報。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "src/findings.js:416",
+    "problem": "applyExclusions 的核心比對支援「只有文字、沒有路徑/角色」的排除規則，但現有測試多半靠相同檔案路徑命中，沒有驗證純文字排除、空文字排除、大小寫/標點差異等邊界。這條排除規則的最脆弱分支還沒被測到。",
+    "suggestion": "補上 applyExclusions 的邊界測試：只有 suggestion/title 文字沒有 location 的 exclusion 應如何比對；空文字 exclusion 不應意外排除全部；標點、空白、大小寫正規化後相同的文字應依預期排除。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "src/gitea.js:95",
+    "problem": "shouldSkipBotCommit 目前只看到命中 bot marker 的測試，缺少「commit API 失敗、分支查詢失敗、sha/branch 都沒有 marker」時應回 false 的失敗與保守路徑驗證。這是避免 workflow 誤跳過審查的關鍵判斷，不能只測快樂路徑。",
+    "suggestion": "新增測試讓 getCommitMessageBySha / getBranchHeadCommitMessage 對應的 axios 呼叫拋錯或回一般 commit message，斷言 shouldSkipBotCommit 回 false，且不會把查詢失敗誤判成 bot commit。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "src/llm.js:91",
+    "problem": "runAssistantCLI 有 timeout 與 maxBuffer 兩條重要失敗路徑，但目前測試只覆蓋 CLI 非零退出，沒有驗證逾時會 kill 子程序並拒絕、輸出超過限制會中止且不產生未處理的重複 reject。這些是 CI 上最常見的失敗情境。",
+    "suggestion": "新增 llm 測試：用假的 CLI sleep 超過 AI_ASSISTANT_TIMEOUT_MS，斷言錯誤訊息包含逾時；再用大量 stdout/stderr 超過 AI_ASSISTANT_MAX_BUFFER，斷言錯誤訊息正確且測試過程沒有 unhandled rejection。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Rogue",
+    "location": "src/main.js:132",
+    "problem": "這裡把每個角色的 LLM 分析逐一 await，6 個角色就把總耗時堆成約 6 倍單次模型延遲；這些分析彼此獨立，CPU 沒偷到時間，反而把整條 pipeline 卡在序列網路/CLI 呼叫上。",
+    "suggestion": "改用 Promise.allSettled 平行執行 roles.map(role => analyzeWithRole(role, diff))，再彙整 fulfilled 結果與 warning；保留 fulfilledAnalyses 的判斷即可。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Rogue",
+    "location": "src/gitea.js:239",
+    "problem": "這裡逐一 await 每個 review 的 comments，PR review 一多就變成 N 次遠端呼叫的線性延遲累加；例如 30 個 review 就是 30 個 round-trip 排隊等，時間都被網路空轉偷走。",
+    "suggestion": "把 reviews.map(review => getPullReviewComments(review.id).catch(...)) 丟進 Promise.all 或 Promise.allSettled 平行抓取，再 flat 結果；單筆失敗仍可記 warn 後略過。",
+    "is_new": true
+  },
+  {
+    "level": "warning",
+    "role": "Rogue",
+    "location": "src/findings.js:430",
+    "problem": "applyExclusions 在 findings × exclusions 的巢狀比對裡，每遇到一條 exclusion 就重算同一個 finding 的 normalizeText；F 筆 finding、E 條 exclusion 會做最多 F×E 次正規化與正則替換，這是很明顯的 CPU 浪費。",
+    "suggestion": "先把 findings 預處理成含 fPath、normalizedFindingText 的陣列，exclusions 也先補齊 normalizedExclusionText，再做比對；同一筆文字只正規化一次。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Bard",
+    "location": "src/findings.js:138",
+    "problem": "註解中留下「不確定」這種未定案語氣，像樂譜上的猶豫記號；讀者無法判斷這是刻意設計、待辦事項，還是審查遺留。",
+    "suggestion": "若是刻意差異，改寫成明確理由；若待確認，改成可追蹤的 TODO 並標明決策者或議題。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Bard",
+    "location": "src/git.js:226",
+    "problem": "_sourceRoot` 的參數文件寫著「不確定，待確認」，讓公開函式簽名帶著未完成的旁白，破壞 API 文件的一致與可信度。",
+    "suggestion": "若參數已不使用，移除它；若為相容性保留，明確寫成 deprecated/compatibility note，不要留下模糊語句。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Bard",
+    "location": "src/config.js:4",
+    "problem": "註解說「需要內部服務相容時才使用 getInsecureHttpsAgent()」，下一行卻在模組載入時全域設定 TLS 環境變數，文件與程式碼唱了不同旋律。",
+    "suggestion": "讓註解忠實描述目前行為，或把全域設定移到明確命名的初始化函式；至少避免文件暗示這是選擇性使用。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Leo",
+    "location": "src/json.js:13",
+    "problem": "`stripCodeFence()` 與 `src/llm.js` 內的 `stripOuterFence()` 幾乎是同一個功能，未來如果要支援更多 fence 格式或修 bug，兩邊需要同步修改，容易產生行為漂移。",
+    "suggestion": "抽成共用的 JSON/text utility，例如 `src/text.js` 或 `src/json.js` 匯出單一 fence 清理函式，讓 LLM JSON 解析與 JSON repair 共用同一套邏輯。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Leo",
+    "location": "src/findings.js:104",
+    "problem": "文字正規化邏輯分散在 `normalizeText()`、`toKeyText()`，而 `src/resolve.js` 也有另一套 `normalizeKey()`。這些函式對大小寫、標點與空白的處理不完全一致，長期會讓 finding 去重、排除與對話收斂出現難追的差異。",
+    "suggestion": "建立單一 normalization 模組，明確定義 `normalizeForDisplayMatch`、`normalizeForSignature` 等用途，再讓 findings、resolve、exclusions 共用，並補上跨模組測試鎖定語意。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Leo",
+    "location": "src/roles.js:7",
+    "problem": "`ROLES_DIR` 用 `fileURLToPath(import.meta.url)` 直接接 `..` 來推目錄，雖然目前可運作，但語意上把檔案路徑當目錄路徑處理，未來搬檔或重構時不直覺。",
+    "suggestion": "先用 `path.dirname(fileURLToPath(import.meta.url))` 取得目前模組目錄，再組 `prompts/roles`，讓路徑意圖清楚且不依賴 `..` 抵銷檔名的技巧。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Maya",
+    "location": "src/comments.js:25",
+    "problem": "Markdown 表格列直接嵌入 role、location、suggestion，但測試沒有覆蓋 suggestion 含 `|`、換行或 Markdown 特殊字元時的輸出。這不是要求現在一定要改格式，而是目前缺少案例確認表格在真實 LLM 輸出下不會被破壞。",
+    "suggestion": "補一個 comment body 格式測試，輸入 suggestion 含 pipe、換行與粗體符號，斷言輸出的 Markdown 結構符合預期；若目前行為會破表，應先定義轉義或替換規則再測。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Rogue",
+    "location": "src/comments.js:126",
+    "problem": "countBy 用 filter(predicate).length 只為了計數卻配置中間陣列；formatFindingsStats/formatFindingsStatsLine 每列又重複掃多次，雖然 findings 通常不大，但這是在白白丟記憶體與掃描週期。",
+    "suggestion": "改成單趟 reduce 統計 new/old × level 的計數表，或讓 countBy 用 for-of 累加數字、不建立 filter 結果陣列。",
+    "is_new": true
+  },
+  {
+    "level": "info",
+    "role": "Rogue",
+    "location": "src/findings.js:382",
+    "problem": "loadExclusions 前面已經 normalizeExclusionEntry + dedupeExclusions，這裡又呼叫 buildExclusionContext(exclusions) 重新 normalize、dedupe、group 一輪，只為了 log groups 數；排除規則多時會多跑一趟 O(e log e) 的整理成本。",
+    "suggestion": "讓 buildExclusionContext 可接受已正規化/已去重的 exclusions，或直接在 loadExclusions 重用現有 exclusions 進行 group 統計，避免重複正規化與排序。",
+    "is_new": true
+  }
+]

.gitea/workflows/ci.yaml

@@ -8,6 +8,8 @@ jobs:
     runs-on: ubuntu
     env:
       VERSION: "0.0.0-beta.${{ gitea.run_number }}"
+    outputs:
+      version: ${{ env.VERSION }}
     steps:
     - name: Publishing Release
       uses: akkuman/gitea-release-action@${{ vars.ACTION_GITEA_RELEASE_VERSION }}
@@ -20,8 +22,8 @@ jobs:
     name: 2. TEST
     runs-on: ubuntu
     needs: [build]
-    outputs:
-      message: ${{ steps.composite-template.outputs.message }}
+    env:
+      VERSION: ${{ needs.build.outputs.version }}
     steps:
     - name: Setup LLM CLI
       uses: https://gitea.jsc.idv.tw/actions/setup-${{ vars.ACTION_SETUP_LLM_CLI }}

src/findings.js

@@ -420,13 +420,18 @@ export async function deduplicateWithAI(findings) {
 
   try {
     const result = await chatJSON(systemPrompt, JSON.stringify(toAIPayload(findings)));
-    if (Array.isArray(result) && result.length > 0) {
-      ok(`AI 去重: ${findings.length} -> ${result.length} 筆`);
-      // 以 location+suggestion 為 key，將原始 findings 的完整欄位（含 is_new）補回
-      const origMap = new Map(findings.map(f => [`${f.location}|${String(f.suggestion).slice(0, 50)}`, f]));
-      return result.map(r => origMap.get(`${r.location}|${String(r.suggestion).slice(0, 50)}`) ?? r);
+    // 去重結果數量不得超過輸入（避免 LLM 無中生有），且每筆都必須能對應回原始 finding。
+    if (Array.isArray(result) && result.length > 0 && result.length <= findings.length) {
+      const keyOf = f => `${f.location}|${String(f.suggestion).slice(0, 50)}`;
+      const origMap = new Map(findings.map(f => [keyOf(f), f]));
+      // 只保留能對應回原始 finding 的項目，丟棄無法對應（可能為幻覺）的結果
+      const mapped = result.map(r => origMap.get(keyOf(r))).filter(Boolean);
+      if (mapped.length > 0) {
+        ok(`AI 去重: ${findings.length} -> ${mapped.length} 筆`);
+        return mapped;
+      }
     }
-    throw new Error('AI 回傳空陣列');
+    throw new Error('AI 去重結果異常（空、超量或無法對應原始 findings）');
   } catch (e) {
     return fallback('AI 去重', findings, e);
   }

src/gitea.js

@@ -54,6 +54,9 @@ export async function getPRDiff() {
     '.github/',
     'README.md',
     'TODO.md',
+    'package-lock.json',
+    'src/package-lock.json',
+    'dist/',
   ]);
 }
 
@@ -126,13 +129,16 @@ export async function shouldSkipBotCommit({ sha = PR_HEAD_SHA || process.env.GIT
  * @param {string[]} excludePrefixes - 要排除的路徑前綴陣列（資料夾以 `/` 結尾，如 `.gitea/`）。
  * @returns {string} 過濾後重新接合的 diff 文字。
  */
-export function filterDiff(diff, excludePrefixes) {
+export function filterDiff(diff, excludePrefixes = []) {
   return diff.split(/(?=^diff --git )/m)
-    .filter(block => !excludePrefixes.some(p => {
-      const prefix = `diff --git a/${p}`;
-      const singleFile = `diff --git a/${p} b/${p}`;
-      return block.startsWith(prefix) || block.startsWith(singleFile);
-    }))
+    .filter(block => {
+      const m = block.match(/^diff --git a\/(.+?) b\//);
+      const path = m ? m[1] : '';
+      if (!path) return true;
+      // 一律排除任何深度的 node_modules：vendored 依賴不是審查對象，且會撐爆 LLM 輸入上限。
+      if (/(^|\/)node_modules\//.test(path)) return false;
+      return !excludePrefixes.some(p => path === p || path.startsWith(p));
+    })
     .join('');
 }
 

src/llm.js

@@ -39,10 +39,30 @@ function cliArgs({ provider, model, promptFile = null, prompt = null }) {
   throw new Error(`不支援的 AI 助理 CLI: ${provider}`);
 }
 
+/**
+ * 從 CLI 輸出中抽出「真正有意義的錯誤」。
+ *
+ * 像 codex 這類 CLI 會先印出一大段 banner（workdir/model/...）與回顯的 prompt，
+ * 真正的失敗原因（例如 401、token 失效、額度不足）通常落在**尾端**。直接取前段
+ * 會被 banner／prompt 洗掉，因此改為：先抽出看起來像錯誤的行；抽不到再退取尾段。
+ *
+ * @param {string} raw - CLI 的原始輸出（stderr 或 stdout）。
+ * @param {number} [limit=1000] - 回傳字串長度上限。
+ * @returns {string} 最能說明失敗原因的片段。
+ */
+export function extractMeaningfulError(raw, limit = 1000) {
+  const text = String(raw || '').trim();
+  const errorLines = text
+    .split('\n')
+    .filter(l => /\bERROR\b|error:|unauthorized|invalidated|revoked|forbidden|\b40[13]\b|rate.?limit|quota|insufficient/i.test(l));
+  const picked = (errorLines.length ? errorLines.join('\n') : text).trim();
+  return picked.length > limit ? picked.slice(-limit) : picked;
+}
+
 function summarizeCliError(e) {
   const stderr = String(e.stderr || '').trim();
   const stdout = String(e.stdout || '').trim();
-  return (stderr || stdout || e.message || String(e)).slice(0, 1000);
+  return extractMeaningfulError(stderr || stdout || e.message || String(e));
 }
 
 async function runAssistantCLI({ provider, command, model }, prompt) {

src/main.js

@@ -209,7 +209,13 @@ async function main() {
   step('Step10', '記憶區 Commit/Push');
   const reviewOutcome = filtered.some(f => f.level === 'critical') ? 'failure' : 'success';
   input(`review outcome=${reviewOutcome}`);
-  await commitAndPush(WORKSPACE, repoDir || WORKSPACE, undefined, undefined, reviewOutcome);
+  // clone 失敗（repoDir 為 undefined）時不可把 WORKSPACE（非來源分支 git repo）當 repoDir，
+  // 否則會在錯誤的工作目錄嘗試 commit/push，findings/exclusions 無法持久化到 PR 分支。
+  if (!repoDir) {
+    warn('來源分支 clone 失敗，略過 findings/exclusions 持久化（不以 WORKSPACE 當 repoDir）');
+  } else {
+    await commitAndPush(WORKSPACE, repoDir, undefined, undefined, reviewOutcome);
+  }
 
   // Step11 嚴重問題把關
   step('Step11', '嚴重問題把關');

src/preflight.js

@@ -1,4 +1,7 @@
 import axios from 'axios';
+import fs from 'fs';
+import os from 'os';
+import { join } from 'path';
 import {
   GITEA_TOKEN,
   GITEA_COMMENT_TOKEN,
@@ -11,6 +14,9 @@ import {
 import { verifyRemoteAccess } from './git.js';
 import { step, line, ok, error, result } from './log.js';
 
+// codex 內部用來取得帳號可用模型清單的端點；auth 失效時會回 HTTP 401。
+const CODEX_MODELS_ENDPOINT = 'https://chatgpt.com/backend-api/codex/models';
+
 const httpsAgent = getInsecureHttpsAgent();
 /**
  * 組出 Gitea REST API v1 的完整網址。
@@ -95,22 +101,87 @@ export async function verifyCommentToken(token = GITEA_COMMENT_TOKEN) {
   }
 }
 
+/**
+ * 讀取本機 codex 認證檔，向模型清單端點確認帳號目前可用的模型 slug。
+ *
+ * 用途：preflight 期即時分辨「auth 失效（HTTP 401）」與「模型無權限（不在清單）」，
+ * 不必等到 Step5 每個角色送 prompt 才神秘失敗。只讀清單、不送 prompt，不消耗生成額度。
+ * 所有錯誤都被攔截並轉為回傳值，不會 throw。
+ *
+ * @param {object} [deps] - 可注入相依，供測試避免真的讀檔/打網路。
+ * @param {typeof fetch} [deps.fetchImpl=fetch] - HTTP 取得函式。
+ * @param {string} [deps.authPath=~/.codex/auth.json] - codex 認證檔路徑。
+ * @param {string} [deps.clientVersion] - 帶給端點的 client_version 查詢參數。
+ * @returns {Promise<{ok: true, slugs: string[]}|{ok: false, error: string}>}
+ *   成功回傳可用模型 slug 陣列；失敗回傳格式化錯誤訊息。
+ */
+export async function fetchCodexModels({
+  fetchImpl = fetch,
+  authPath = join(os.homedir(), '.codex', 'auth.json'),
+  clientVersion = '0.142.5',
+} = {}) {
+  let auth;
+  try {
+    auth = JSON.parse(fs.readFileSync(authPath, 'utf8'));
+  } catch (e) {
+    return { ok: false, error: `無法讀取 codex 認證檔（${authPath}）: ${e.message}` };
+  }
+  const tokens = auth.tokens || {};
+  if (!tokens.access_token) return { ok: false, error: 'codex 認證檔缺少 tokens.access_token' };
+
+  const headers = { Authorization: `Bearer ${tokens.access_token}` };
+  if (tokens.account_id) headers['chatgpt-account-id'] = tokens.account_id;
+
+  let resp;
+  try {
+    resp = await fetchImpl(`${CODEX_MODELS_ENDPOINT}?client_version=${clientVersion}`, { headers });
+  } catch (e) {
+    return { ok: false, error: `codex 模型清單查詢連線錯誤: ${e.message}` };
+  }
+  if (resp.status === 401) {
+    return { ok: false, error: 'codex 認證失效（HTTP 401）——token 已被撤銷或過期，請重新登入 codex 並更新 LLM_OAUTH secret' };
+  }
+  if (!resp.ok) {
+    return { ok: false, error: `codex 模型清單查詢失敗（HTTP ${resp.status}）` };
+  }
+  let data;
+  try {
+    data = await resp.json();
+  } catch (e) {
+    return { ok: false, error: `codex 模型清單回應解析失敗: ${e.message}` };
+  }
+  const slugs = Array.isArray(data.models) ? data.models.map(m => m.slug).filter(Boolean) : [];
+  return { ok: true, slugs };
+}
+
 /**
  * 驗證 LLM（AI 助理 CLI）設定可用。
  *
- * 確認目前環境可偵測到支援的 CLI，且已解析出 model。實際模型可用性由 CLI
- * 在正式呼叫時回報；preflight 不主動送 prompt，避免額外消耗額度。
+ * 確認目前環境可偵測到支援的 CLI 且已解析出 model；provider 為 codex 時，
+ * 額外向模型清單端點確認 auth 有效且設定的 model 在可用清單內（不送 prompt）。
+ * @param {object} [deps] - 可注入相依，供測試。
+ * @param {Function} [deps.fetchCodexModelsFn=fetchCodexModels] - codex 模型清單取得函式。
  * @returns {Promise<
- *   {ok: true, provider: string, command: string, model: string} |
- *   {ok: false, provider?: string, error: string}
+ *   {ok: true, provider: string, command: string, model: string, models?: string[]} |
+ *   {ok: false, provider?: string, command?: string, model?: string, error: string}
  * >}
- *   通過時含 provider、command 與 model；未設定 provider 的失敗分支不含 provider 欄位。
+ *   通過時含 provider、command、model（codex 另含 models 清單）；未設定 provider 的失敗分支不含 provider。
  * @remarks 設定來源為 config.js 的 getLLMConfig()。
  */
-export async function verifyLLM() {
+export async function verifyLLM({ fetchCodexModelsFn = fetchCodexModels } = {}) {
   const { provider, command, model } = getLLMConfig();
   if (!provider || !command) return { ok: false, error: '未偵測到可用 AI 助理 CLI，請安裝 codex、claude、antigravity 或 opencode' };
   if (!model) return { ok: false, provider, error: '未設定 MODEL' };
+
+  if (provider === 'codex') {
+    const models = await fetchCodexModelsFn();
+    if (!models.ok) return { ok: false, provider, command, model, error: models.error };
+    if (!models.slugs.includes(model)) {
+      return { ok: false, provider, command, model, error: `模型 ${model} 不在 codex 可用清單: [${models.slugs.join(', ')}]` };
+    }
+    return { ok: true, provider, command, model, models: models.slugs };
+  }
+
   return { ok: true, provider, command, model };
 }
 
@@ -174,6 +245,7 @@ export async function runPreflight(workspace = process.env.GITHUB_WORKSPACE || '
     return false;
   }
   ok(`LLM CLI 可用（command=${llm.command}, provider=${llm.provider}, model=${llm.model}）`);
+  if (llm.models) line(`模型已確認在可用清單內（共 ${llm.models.length} 個可用模型）`);
 
   result(true, '前置驗證通過');
   return true;

src/resolve.js

@@ -75,7 +75,7 @@ export function groupConversations(comments) {
   for (const c of comments || []) {
     const filePath = typeof c?.path === 'string' ? c.path : '';
     if (!filePath) continue; // 無檔案路徑的留言無法定位，跳過以免併入共用群組
-    const lineNum = Number(c?.position) || Number(c?.original_position) || 0;
+    const lineNum = Number(c?.position) || Number(c?.new_position) || Number(c?.original_position) || 0;
     const key = `${filePath}|${lineNum}`;
     if (!groups.has(key)) {
       groups.set(key, { key, path: filePath, line: lineNum, commentIds: [], bodies: [], resolved: false, botFinding: null });

src/test/gitea.test.js

@@ -241,4 +241,21 @@ describe('filterDiff', () => {
   it('returns empty string for empty diff', () => {
     assert.equal(filterDiff('', ['.gitea/']), '');
   });
+
+  it('always drops node_modules blocks at any depth (avoids blowing the LLM input limit)', () => {
+    const diff = block('src/node_modules/axios/index.js')
+      + block('node_modules/js-yaml/lib.js')
+      + block('src/main.js');
+    const result = filterDiff(diff, []);
+    assert.ok(!result.includes('node_modules'));
+    assert.ok(result.includes('src/main.js'));
+  });
+
+  it('excludes lock files and dist via the getPRDiff prefix list', () => {
+    const diff = block('src/package-lock.json') + block('dist/index.js') + block('src/main.js');
+    const result = filterDiff(diff, ['package-lock.json', 'src/package-lock.json', 'dist/']);
+    assert.ok(!result.includes('package-lock.json'));
+    assert.ok(!result.includes('dist/'));
+    assert.ok(result.includes('src/main.js'));
+  });
 });

src/test/llm.test.js

@@ -3,7 +3,7 @@ import assert from 'node:assert/strict';
 import { mkdtemp, writeFile, chmod, rm, readFile } from 'fs/promises';
 import { tmpdir } from 'os';
 import { join } from 'path';
-import { extractBalancedJSON, extractJSONText } from '../llm.js';
+import { extractBalancedJSON, extractJSONText, extractMeaningfulError } from '../llm.js';
 
 const ENV_KEYS = [
   'AI_ASSISTANT_CLI', 'MODEL', 'OPENCODE_MODEL', 'PATH', 'AI_ASSISTANT_TIMEOUT_MS', 'AI_ASSISTANT_MAX_BUFFER',
@@ -237,3 +237,39 @@ describe('extractJSONText', () => {
     assert.equal(result, 'not json at all');
   });
 });
+
+describe('extractMeaningfulError', () => {
+  it('抽出尾端真正的錯誤，而非開頭的 codex banner／回顯 prompt', () => {
+    const raw = [
+      'OpenAI Codex v0.142.5',
+      '--------',
+      'workdir: /workspace/actions/ai-code-review',
+      'model: gpt-5.4-mini',
+      'reasoning effort: none',
+      '--------',
+      'user',
+      '請依照以下系統指示處理使用者內容，並只輸出要求的最終結果。',
+      'ERROR codex_api::endpoint::responses_websocket: failed to connect to websocket: HTTP error: 401 Unauthorized',
+      'ERROR: Your access token could not be refreshed because your refresh token was revoked. Please log out and sign in again.',
+    ].join('\n');
+
+    const result = extractMeaningfulError(raw);
+
+    assert.match(result, /401 Unauthorized/);
+    assert.match(result, /refresh token was revoked/);
+    assert.doesNotMatch(result, /workdir:/);
+    assert.doesNotMatch(result, /請依照以下系統指示/);
+  });
+
+  it('抽不到錯誤行時退取尾段（不取開頭）', () => {
+    const raw = 'A'.repeat(1200) + '\nTAIL-CONTENT';
+    const result = extractMeaningfulError(raw, 100);
+    assert.ok(result.length <= 100);
+    assert.match(result, /TAIL-CONTENT$/);
+  });
+
+  it('容錯處理空輸入', () => {
+    assert.equal(extractMeaningfulError(''), '');
+    assert.equal(extractMeaningfulError(null), '');
+  });
+});

src/test/preflight.test.js

@@ -4,7 +4,7 @@ import axios from 'axios';
 import { mkdtemp, writeFile, chmod, rm } from 'fs/promises';
 import { tmpdir } from 'os';
 import { join } from 'path';
-import { checkRequiredEnv, verifyGiteaToken, verifyCommentToken, verifyLLM, runPreflight } from '../preflight.js';
+import { checkRequiredEnv, verifyGiteaToken, verifyCommentToken, verifyLLM, fetchCodexModels, runPreflight } from '../preflight.js';
 
 const LLM_ENV_KEYS = [
   'AI_ASSISTANT_CLI', 'MODEL', 'OPENCODE_MODEL', 'PATH',
@@ -130,18 +130,52 @@ describe('verifyLLM', () => {
     assert.match(result.error, /AI 助理 CLI/);
   });
 
-  it('passes when a supported assistant CLI is detected', async () => {
+  it('passes when a supported assistant CLI is detected and the model is in the codex list', async () => {
     clearLLMEnv();
     await installFakeCLI('codex');
     process.env.AI_ASSISTANT_CLI = 'codex';
-    process.env.MODEL = 'gpt-5-mini';
+    process.env.MODEL = 'gpt-5.4-mini';
 
-    const result = await verifyLLM();
+    const result = await verifyLLM({
+      fetchCodexModelsFn: async () => ({ ok: true, slugs: ['gpt-5.5', 'gpt-5.4-mini'] }),
+    });
 
     assert.equal(result.ok, true);
     assert.equal(result.provider, 'codex');
     assert.equal(result.command, 'codex');
-    assert.equal(result.model, 'gpt-5-mini');
+    assert.equal(result.model, 'gpt-5.4-mini');
+    assert.deepEqual(result.models, ['gpt-5.5', 'gpt-5.4-mini']);
+  });
+
+  it('fails when codex auth is invalid (model list check reports 401)', async () => {
+    clearLLMEnv();
+    await installFakeCLI('codex');
+    process.env.AI_ASSISTANT_CLI = 'codex';
+    process.env.MODEL = 'gpt-5.4-mini';
+
+    const result = await verifyLLM({
+      fetchCodexModelsFn: async () => ({ ok: false, error: 'codex 認證失效（HTTP 401）——token 已被撤銷或過期，請重新登入 codex 並更新 LLM_OAUTH secret' }),
+    });
+
+    assert.equal(result.ok, false);
+    assert.equal(result.provider, 'codex');
+    assert.match(result.error, /HTTP 401/);
+    assert.match(result.error, /LLM_OAUTH/);
+  });
+
+  it('fails when the configured model is not in the codex available list', async () => {
+    clearLLMEnv();
+    await installFakeCLI('codex');
+    process.env.AI_ASSISTANT_CLI = 'codex';
+    process.env.MODEL = 'gpt-9-imaginary';
+
+    const result = await verifyLLM({
+      fetchCodexModelsFn: async () => ({ ok: true, slugs: ['gpt-5.5', 'gpt-5.4-mini'] }),
+    });
+
+    assert.equal(result.ok, false);
+    assert.match(result.error, /不在 codex 可用清單/);
+    assert.match(result.error, /gpt-5\.4-mini/);
   });
 
   it('fails when a requested CLI is not installed', async () => {
@@ -157,6 +191,59 @@ describe('verifyLLM', () => {
 
 });
 
+describe('fetchCodexModels', () => {
+  async function writeAuth(json) {
+    tempDir = await mkdtemp(join(tmpdir(), 'codex-auth-test-'));
+    const authPath = join(tempDir, 'auth.json');
+    await writeFile(authPath, JSON.stringify(json));
+    return authPath;
+  }
+
+  it('returns the model slugs on HTTP 200', async () => {
+    const authPath = await writeAuth({ tokens: { access_token: 'tok', account_id: 'acc' } });
+    let capturedUrl, capturedHeaders;
+    const result = await fetchCodexModels({
+      authPath,
+      fetchImpl: async (url, opts) => {
+        capturedUrl = url;
+        capturedHeaders = opts.headers;
+        return { status: 200, ok: true, json: async () => ({ models: [{ slug: 'gpt-5.5' }, { slug: 'gpt-5.4-mini' }] }) };
+      },
+    });
+    assert.deepEqual(result, { ok: true, slugs: ['gpt-5.5', 'gpt-5.4-mini'] });
+    assert.match(capturedUrl, /client_version=/);
+    assert.equal(capturedHeaders['Authorization'], 'Bearer tok');
+    assert.equal(capturedHeaders['chatgpt-account-id'], 'acc');
+  });
+
+  it('reports an auth failure on HTTP 401', async () => {
+    const authPath = await writeAuth({ tokens: { access_token: 'revoked' } });
+    const result = await fetchCodexModels({
+      authPath,
+      fetchImpl: async () => ({ status: 401, ok: false, json: async () => ({}) }),
+    });
+    assert.equal(result.ok, false);
+    assert.match(result.error, /HTTP 401/);
+    assert.match(result.error, /LLM_OAUTH/);
+  });
+
+  it('fails when the auth file cannot be read', async () => {
+    const result = await fetchCodexModels({
+      authPath: join(tmpdir(), 'definitely-missing-codex-auth-xyz.json'),
+      fetchImpl: async () => ({ status: 200, ok: true, json: async () => ({ models: [] }) }),
+    });
+    assert.equal(result.ok, false);
+    assert.match(result.error, /無法讀取 codex 認證檔/);
+  });
+
+  it('fails when the auth file lacks an access_token', async () => {
+    const authPath = await writeAuth({ tokens: {} });
+    const result = await fetchCodexModels({ authPath, fetchImpl: async () => ({ status: 200, ok: true, json: async () => ({}) }) });
+    assert.equal(result.ok, false);
+    assert.match(result.error, /缺少 tokens\.access_token/);
+  });
+});
+
 describe('runPreflight', () => {
   function makeDeps(overrides = {}) {
     return {