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

.gitea/ai-review/findings.json

@@ -2,37 +2,46 @@
   {
     "level": "critical",
     "role": "Maya",
-    "location": "app/index.js:1, 14, 52, 58",
+    "location": "app/index.js:1, 14, 52, 58, 108, 132",
-    "problem": "核心邏輯缺乏自動化測試。包含 main 入口點、自動解衝突流程、以及 `createPull` 錯誤處理與 fallback 機制均未受測試驗證，難以保證建置與合併的正確性。",
+    "problem": "核心功能（包括入口、解衝突流程、錯誤處理、diff 截斷、分支名稱產生）嚴重缺乏自動化測試，難以保證在極端或失敗情境下的正確性與穩定性。",
-    "suggestion": "建立完整的單元與整合測試架構。Mock `Git`、`GiteaClient` 與 `OpenCode` 類別，針對 main() 覆蓋成功路徑與合併衝突的失敗路徑，並增強對 API 回傳錯誤的解析與處理。"
+    "suggestion": "建立完整的單元與整合測試架構。Mock 外部依賴（Git, Gitea, OpenCode），針對成功路徑、合併衝突、邊界條件（如極小長度）及錯誤路徑（如 API 失敗、參數異常）補全測試。"
   },
   {
     "level": "critical",
-    "role": "Maya/Mage",
+    "role": "Mage/Assassin",
-    "location": "app/lib/git.js:93, 106",
+    "location": "app/lib/git.js:17, 37, 93, 106",
-    "problem": "Git 衝突偵測與狀態復原邏輯缺乏測試。`detectConflict` 若行為預期外或 abort 失敗，將導致工作區殘留錯誤狀態，且未經測試驗證。",
+    "problem": "Git 操作存在嚴重安全性與穩定性隱患：命令列傳遞敏感 Token（竊取風險）、使用全域 Git 配置（污染環境/權限衝突）、衝突偵測邏輯缺乏測試且復原不健壯（工作區殘留狀態）。",
-    "suggestion": "針對 `Git.detectConflict` 補寫單元測試（模擬衝突場景）。在 `finally` 區塊中，增加檢查分支是否存在再進行刪除，確保環境乾淨。"
+    "suggestion": "改用 Git 憑證輔助工具或環境變數傳遞 Token；使用 --local 設定而非 --global；針對偵測與清理邏輯補寫單元測試，引入時間戳記/UUID 確保分支唯一性，並嚴謹處理資源清理（finally 區塊）。"
-  },
-  {
-    "level": "warning",
-    "role": "Maya",
-    "location": "app/index.js:114, 126, 154",
-    "problem": "diff 截斷邏輯、fallbackSummary 邊界處理缺乏測試。（分支名稱長度 Magic Number 已抽為具名常數 MAX_BRANCH_STEM_LENGTH。）",
-    "suggestion": "補上針對 truncateDiff 及 commitMessages 為空/null 時的測試案例。"
   },
   {
     "level": "warning",
     "role": "Assassin",
     "location": "Dockerfile:11",
     "problem": "Dockerfile 中使用全域 `npm install` 存在供應鏈風險。",
-    "suggestion": "使用 lockfile（如 `package-lock.json`）確保依賴版本一致性，並定期審查更新。"
+    "suggestion": "使用 lockfile（如 `package-lock.json`）確保依賴版本一致性，並定期審查更新。",
+    "is_new": false
   },
   {
     "level": "warning",
-    "role": "Maya",
+    "role": "Maya/Mage/Leo",
-    "location": "app/lib/opencode.js:130, app/lib/gitea.js:56",
+    "location": "app/index.js:52, 114, 126, 154, app/lib/opencode.js:141, app/lib/gitea.js:56, 84",
-    "problem": "fallback 機制與 Gitea API 狀態碼處理（422/409）缺乏測試，難以確保產出品質與正確行為。",
+    "problem": "fallback 機制、API 狀態碼處理（422/409）、diff 邊界處理與分支名稱生成邏輯缺乏測試，且在 fallback 資訊不足時產出的 PR 描述空洞無效。",
-    "suggestion": "補寫 OpenCode.summarize 與 GiteaClient.createPull 測試，Mock 不同情境回應，驗證系統邏輯。"
+    "suggestion": "補寫各模組的測試，Mock 不同 API 回應情境。增加對 fallback 輸出內容的檢查，確保分支名稱在極端情況下具唯一性，並優化錯誤處理邏輯。"
+  },
+  {
+    "level": "warning",
+    "role": "Leo/Rogue",
+    "location": "app/lib/git.js:148, 84",
+    "problem": "Git 提交策略不精確（全域 add 可能帶入非預期變更）、頻繁執行完整 checkout/merge 導致高效能消耗與高 I/O 成本。",
+    "suggestion": "使用更精確的 add 策略或在 commit 前檢查狀態；改用 `git rev-list --count` 或 `git merge-tree` 進行衝突預檢，減少對檔案系統的寫入。"
+  },
+  {
+    "level": "warning",
+    "role": "Bard",
+    "problem": "錯誤訊息處理中，對 `result.stderr` 進行截斷，可能導致切斷關鍵錯誤上下文，除錯困難。",
+    "suggestion": "改用 log.warn 輸出完整內容（遮蔽敏感資訊後），或將截斷訊息與「內容已截斷」提示並列。",
+    "location": "app/lib/opencode.js:154",
+    "is_new": false
   },
   {
     "level": "info",