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

2026-06-23 03:33:17 +00:00
parent e492ea5447
commit 980f45308d
1 changed files with 42 additions and 1 deletions
@@ -1 +1,42 @@
-[]
+[
+  {
+    "level": "critical",
+    "role": "Assassin",
+    "location": "app/resolve.js:180",
+    "problem": "在 `reconcileConversations` 函式中，從外部 Gitea comment 取得的 `c.path`（檔案路徑）未經額外驗證或淨化，直接傳遞給了 `getFileContent`（即 `getFileContentAtRef`）。由於 `getFileContentAtRef` 存在路徑穿越漏洞，攻擊者可以透過在 PR 中建立惡意檔案名稱，並在該檔案上留言，來觸發路徑穿越，讀取伺服器上的任意檔案。",
+    "suggestion": "在將 `c.path` 傳遞給 `getFileContent` 之前，必須對其進行嚴格的白名單驗證，確保它只包含預期的檔案名稱字元，且不包含任何路徑穿越序列（例如 `..` 或 `/`）。或者，確保 `getFileContentAtRef` 的路徑處理是絕對安全的，不允許任何形式的路徑穿越。",
+    "is_new": true
+  },
+  {
+    "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
+  },
+  {
+    "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
+  },
+  {
+    "level": "warning",
+    "role": "Leo",
+    "location": "app/resolve.js:207",
+    "problem": "函式 `normalizeKey` 對建議內容進行了非常積極的正規化，移除了所有標點符號、符號和空白字元。雖然這有助於避免行號漂移和微小措辭差異造成的重複判斷，但過度正規化可能會導致不同但語意相近的建議被視為相同，進而影響問題追蹤的精確性。",
+    "suggestion": "請評估這種積極正規化是否會導致誤判。如果發現有不同建議被錯誤合併的情況，可以考慮放寬正規化規則，例如只移除空白字元和部分標點符號，或加入其他判斷維度（如關鍵字比對）來提高精確度。",
+    "is_new": true
+  },
+  {
+    "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
+  }
+]