chore: update ai-review findings [skip ci]

.gitea/ai-review/findings.json

@@ -1,23 +1,16 @@
 [
   {
-    "level": "info",
+    "level": "warning",
     "role": "Rex",
-    "location": "app/gitea.js:19",
+    "location": "app/gitea.js:10",
-    "suggestion": "將 `filterDiff` 函數中的 diff 區塊過濾邏輯從正則表達式改為 `startsWith` 是一個重要的安全改進。這可以有效防止潛在的正則表達式注入攻擊，即使 `excludePrefixes` 參數未來可能受到外部控制，也能確保過濾邏輯的安全性。",
+    "suggestion": "`getPRDiff` 函數現在回傳未經過濾的原始 Git Diff 內容。雖然 `main.js` 中已立即呼叫 `filterDiff` 進行過濾，但這種設計模式將過濾的責任完全推給呼叫端。這增加了未來開發者在其他地方呼叫 `getPRDiff` 時，可能忘記或錯誤地應用過濾，導致 `.gitea/` 等敏感路徑的內容（可能包含工作流程設定或機密資訊）被意外傳送給 AI 或其他不應接收的組件，造成資訊洩漏風險。建議考慮將過濾邏輯保留在 `getPRDiff` 內部，或提供一個明確的 `getFilteredPRDiff` 函數，以降低誤用的風險。",
-    "is_new": false
+    "is_new": true
   },
   {
-    "level": "info",
+    "level": "warning",
-    "role": "Rex",
+    "role": "Zara",
-    "location": "app/main.js:46",
+    "location": "app/git.js, app/main.js",
-    "suggestion": "在將 Git Diff 內容傳遞給 AI 進行分析之前，明確呼叫 `filterDiff` 函數以排除 `.gitea/` 等敏感路徑，是一個良好的安全實踐。這有助於避免 AI 分析到不必要的或包含敏感配置的非業務程式碼，降低潛在的資訊洩漏風險。",
+    "suggestion": "在 `app/main.js` 中，`commitAndPush` 函數內部會再次呼叫 `cloneRepo`。然而，`main.js` 在此之前已經呼叫過 `cloneRepo` 以取得 `repoDir`。這導致了重複的 `git fetch` 和 `git checkout` 操作，即使 `cloneRepo` 內部有檢查機制，仍會造成不必要的資源消耗和時間延遲。建議修改 `commitAndPush` 函數，使其接收已存在的 `repoDir` 作為參數，避免重複執行 `cloneRepo`。",
-    "is_new": false
+    "is_new": true
-  },
-  {
-    "level": "info",
-    "role": "Rex",
-    "location": "app/main.js:98",
-    "suggestion": "新增對 `findings.json` 和 `exclusions.json` 檔案進行 JSON 格式驗證的步驟，並在格式錯誤時嘗試重置和備份，這是一個重要的健壯性與安全措施。它能防止因檔案損壞或惡意修改導致的服務中斷或行為異常，確保系統的穩定性和資料的完整性。",
-    "is_new": false
   }
 ]