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

.gitea/ai-review/findings.json

@@ -5,38 +5,105 @@
     "location": "app/resolve.js:180",
     "problem": "在 `reconcileConversations` 函式中，從外部 Gitea comment 取得的 `c.path`（檔案路徑）未經額外驗證或淨化，直接傳遞給了 `getFileContent`（即 `getFileContentAtRef`）。由於 `getFileContentAtRef` 存在路徑穿越漏洞，攻擊者可以透過在 PR 中建立惡意檔案名稱，並在該檔案上留言，來觸發路徑穿越，讀取伺服器上的任意檔案。",
     "suggestion": "在將 `c.path` 傳遞給 `getFileContent` 之前，必須對其進行嚴格的白名單驗證，確保它只包含預期的檔案名稱字元，且不包含任何路徑穿越序列（例如 `..` 或 `/`）。或者，確保 `getFileContentAtRef` 的路徑處理是絕對安全的，不允許任何形式的路徑穿越。",
-    "is_new": true
+    "is_new": false
   },
   {
     "level": "critical",
     "role": "Assassin",
     "location": "app/resolve.js:145",
-    "problem": "在 `judgeConversationsResolved` 函式中，`thread`（來自 Gitea comment 內容）和 `code`（來自 PR 檔案內容）被直接拼接進傳給 LLM 的 `payload` 中。如果攻擊者能夠控制這些內容，他們可以透過注入惡意指令來劫持 LLM 的行為，例如使其始終將特定問題判斷為已解決，或嘗試從 LLM 獲取敏感資訊（提示詞注入）。",
-    "suggestion": "對所有傳遞給 LLM 的外部輸入（如 `thread` 和 `code`）進行嚴格的淨化和隔離。考慮使用結構化輸入而非直接拼接字串，並在 LLM 提示詞中明確指示其忽略任何試圖改變其行為的指令。對於敏感操作，應建立多層驗證機制，不單純依賴 LLM 的判斷。",
-    "is_new": true
+    "problem": "LLM 提示詞注入風險：在 `judgeConversationsResolved` 函式中，外部來源的 `thread` 和 `code` 被直接拼接進傳給 LLM 的 `payload` 中，攻擊者可能注入惡意指令來劫持 LLM 行為。",
+    "suggestion": "對所有傳遞給 LLM 的外部輸入進行嚴格的淨化和隔離。使用結構化輸入而非直接拼接字串，並在提示詞中明確指示 AI 忽略任何試圖下達指令的內容，僅對邏輯進行判斷。對於敏感操作，應建立多層驗證機制。"
+  },
+  {
+    "level": "critical",
+    "role": "Mage",
+    "location": "app/resolve.js:77",
+    "problem": "在 `judgeConversationsResolved` 函式中，對 `chatFn` 的結果結構缺乏足夠的嚴格檢查。若回傳結構不符合預期，可能導致所有對話被錯誤判定為「未解決」。",
+    "suggestion": "增加對 `result` 結構的嚴格檢查。如果 `result` 不是預期的陣列結構，應拋出例外或進行更謹慎的錯誤處理，而不是默默地將所有對話視為未解決。"
   },
   {
     "level": "warning",
     "role": "Assassin",
     "location": "app/resolve.js:52",
-    "problem": "在 `parseBotReviewComment` 函式中，從 Gitea comment 內文解析出的 `problem` 和 `suggestion` 欄位，若包含惡意 HTML 或 JavaScript 程式碼，且這些內容在後續的處理或顯示中未經適當的輸出編碼，可能導致跨網站指令碼（XSS）攻擊。",
-    "suggestion": "確保所有從外部來源解析出的字串（特別是 `problem` 和 `suggestion`）在任何將其渲染到網頁或其他使用者介面的地方，都必須經過嚴格的上下文相關輸出編碼（例如 HTML 實體編碼、JavaScript 字串編碼等），以防止 XSS 攻擊。",
-    "is_new": true
+    "problem": "在 `parseBotReviewComment` 函式中，從 Gitea comment 內文解析出的 `problem` 和 `suggestion` 欄位若包含惡意內容且未經適當輸出編碼，可能導致 XSS 攻擊。",
+    "suggestion": "確保所有從外部來源解析出的字串在渲染到任何介面時，都必須經過嚴格的上下文相關輸出編碼（例如 HTML 實體編碼），以防止 XSS 攻擊。"
   },
   {
     "level": "warning",
     "role": "Leo",
     "location": "app/resolve.js:207",
-    "problem": "函式 `normalizeKey` 對建議內容進行了非常積極的正規化，移除了所有標點符號、符號和空白字元。雖然這有助於避免行號漂移和微小措辭差異造成的重複判斷，但過度正規化可能會導致不同但語意相近的建議被視為相同，進而影響問題追蹤的精確性。",
-    "suggestion": "請評估這種積極正規化是否會導致誤判。如果發現有不同建議被錯誤合併的情況，可以考慮放寬正規化規則，例如只移除空白字元和部分標點符號，或加入其他判斷維度（如關鍵字比對）來提高精確度。",
+    "problem": "正規化邏輯（`normalizeKey` 等）過於激進且未快取，既可能導致語意相近建議被誤判為相同，也在頻繁比較時造成效能浪費。",
+    "suggestion": "請評估目前的正規化規則，若發現誤判，放寬規則或加入關鍵字比對。將簽章產生邏輯抽離為獨立 Helper 函式，並在產生時進行快取（Memoize）以提升效能。"
+  },
+  {
+    "level": "warning",
+    "role": "Maya",
+    "location": "app/resolve.js:144",
+    "problem": "缺少關鍵邊界條件與異常路徑的測試案例。包含 `judge` 拋出錯誤、`chatFn` 解析異常、`levelRaw` 或 `suggestion` 空值、`getFileContent` 失敗以及混合正確/錯誤的判斷數據等場景。",
+    "suggestion": "請在 `app/resolve.test.js` 中新增這些邊界條件的測試案例，確保系統在面對 AI 異常輸出、API 失敗、或輸入欄位缺失時，仍能穩健處理並符合預期行為。"
+  },
+  {
+    "level": "warning",
+    "role": "Assassin",
+    "location": "app/resolve.js:18",
+    "problem": "函式 `parseBotReviewComment` 動態產生正規表達式，且輸入來源 `body` 為外部輸入，存在 Regex Injection 風險。",
+    "suggestion": "將正規表達式改為靜態定義，並透過 `String.raw` 或更安全的字串處理方式來匹配標籤，確保輸入不包含特殊 regex 字元。"
+  },
+  {
+    "level": "warning",
+    "role": "Bard",
+    "location": "app/resolve.js",
+    "problem": "`judgeConversationsResolved` 內的 `systemPrompt` 硬編碼在函式中，過於冗長且干擾邏輯。",
+    "suggestion": "將此 `systemPrompt` 抽離為檔案層級的常數，提升程式碼結構美與清晰度。"
+  },
+  {
+    "level": "warning",
+    "role": "Leo",
+    "location": "app/resolve.js:77",
+    "problem": "程式碼片段定位邏輯（如字串拼接行號）與上下文擷取策略（如 radius）寫死在函式內，擴展性與維護性不足。",
+    "suggestion": "建立明確的 `Location` 物件封裝定位資訊，並將 `radius` 或擷取策略抽離為配置參數或常數。"
+  },
+  {
+    "level": "warning",
+    "role": "Mage",
+    "location": "app/resolve.js:187",
+    "problem": "在 `reconcileConversations` 函式中，並行（`Promise.all`）呼叫 `resolveComment`，即使個別呼叫失敗，也僅在 `settled` 中記錄為 `rejected` 並印出 `warn`。然而，若 `resolveComment` 失敗是因為 `Authorization` token 過期或權限不足，後續所有的 `resolve` 呼叫都會失敗，此時程式碼沒有對這些特定的錯誤進行分類處理。",
+    "suggestion": "應判斷 `outcome.reason` 的錯誤類型。若是連線/權限相關的嚴重錯誤，應立即停止後續的 `resolve` 嘗試，避免在已知無法成功的情況下發出無效請求。",
     "is_new": true
   },
+  {
+    "level": "warning",
+    "role": "Rogue",
+    "location": "app/resolve.js:77",
+    "problem": "大量使用字串拼接產生暫存物件，以及並行請求未限制數量，在高負載下可能導致 GC 壓力或觸發 API 限流。",
+    "suggestion": "對於大量 comments，考慮使用複合物件或分層 Map 結構。引入請求並行限制（如 `p-limit`）來確保系統穩定性。"
+  },
   {
     "level": "info",
     "role": "Mage",
-    "problem": "在 `groupConversations` 函式中，若行內 review comment 缺乏 `path` 或 `position`/`original_position` 資訊，它們將會被歸類到一個共同的 `key` (例如 `|0`)。這可能導致多個實際上不相關的、缺乏位置資訊的留言被錯誤地歸類為同一個對話群組。雖然這類留言通常不屬於「行內」評論，且 `parseBotReviewComment` 可能會將其視為非 bot 留言，但這種歸類方式可能與預期不符。",
-    "suggestion": "考慮是否應明確地過濾掉缺乏 `path` 或有效 `position` 的留言，或為這些留言提供一個更具區分性的預設 `key`，以避免不相關的留言被意外地歸併。例如，可以在迴圈開始時增加判斷：`if (!c?.path || (!c?.position && !c?.original_position)) continue;`。",
     "location": "app/resolve.js:70",
-    "is_new": false
+    "problem": "缺乏位置資訊的留言會被歸類到同一個預設 key，可能導致不相關留言被錯誤歸併。",
+    "suggestion": "明確過濾缺乏 `path` 或 `position` 的留言，或提供更具區分性的預設 key。"
+  },
+  {
+    "level": "info",
+    "role": "Bard",
+    "location": "app/resolve.js:8",
+    "problem": "RegExp 在函式內部重複建立，造成不必要的效能損耗。",
+    "suggestion": "將正則表達式移至函式外部宣告為常數。"
+  },
+  {
+    "level": "info",
+    "role": "Mage",
+    "location": "app/resolve.js:195",
+    "problem": "對 `botFinding` 的存取缺乏防禦性檢查。",
+    "suggestion": "在 `push` 之前增加防禦性檢查，確保物件完整性。"
+  },
+  {
+    "level": "info",
+    "role": "Rogue",
+    "location": "app/resolve.js:173",
+    "problem": "`Promise.allSettled` 的結果處理邏輯過於冗長，產生不必要的中間變數。",
+    "suggestion": "優化處理邏輯，直接在迴圈內處理或使用更緊湊的寫法。"
   }
 ]