.gitea/ai-review/exclusions.json

@@ -392,5 +392,11 @@
         "role":  "Assassin",
         "original_finding":  "此測試明確證實了 `OPENCODE_SKIP_TLS_VERIFY` 環境變數的寬鬆判斷邏輯，導致 OpenCode LLM 連線的 TLS 驗證容易被關閉。這是「關閉 TLS 驗證」的不安全預設，極大地增加了中間人攻擊的風險。",
         "reason":  "誤判／既有設計。OpenCode server 目前支援自簽或內部服務情境，action input 與 README 均明確標示 OPENCODE_SKIP_TLS_VERIFY 預設跳過 TLS 驗證；本 PR 只補測試與 Review comment 內容，未新增或放寬此安全行為。"
+    },
+    {
+        "location": "app/comments.test.js:30",
+        "role": "Leo",
+        "original_finding": "將 `REVIEW_SEVERITY_LABELS`、`REVIEW_SEVERITY_PATTERN` 和 `reviewSeverityLabel` 這些與評論格式相關的常數與函式，提取到一個獨立的共用模組中（例如 `app/utils/reviewComments.js`），並讓測試檔案和任何需要用到它們的應用程式邏輯都從該模組匯入。這樣能確保「評論格式」的定義只有一個來源，提升可維護性。",
+        "reason": "誤判。這些常數與 `reviewSeverityLabel` 只用於 `app/comments.test.js` 內部驗證 review comment body 格式，production code 沒有使用同一段解析邏輯；抽成共用模組會把測試專用輔助程式提升為正式 API，增加不必要的維護負擔。"
     }
 ]

.gitea/ai-review/findings.json

@@ -1,18 +1 @@
-[
+[]
-  {
-    "level": "warning",
-    "role": "Leo",
-    "location": "app/comments.test.js:30",
-    "problem": "常數 `REVIEW_SEVERITY_LABELS`、`REVIEW_SEVERITY_PATTERN` 以及輔助函式 `reviewSeverityLabel` 被定義在測試檔案的 `describe` 區塊內。如果實際的應用程式邏輯（例如產生或處理這些評論的程式碼）也需要用到這些資訊，那麼這會造成邏輯重複或知識分散。未來若評論格式或嚴重等級標籤有變動，將需要同時修改多處，增加維護成本與出錯風險。",
-    "suggestion": "將 `REVIEW_SEVERITY_LABELS`、`REVIEW_SEVERITY_PATTERN` 和 `reviewSeverityLabel` 這些與評論格式相關的常數與函式，提取到一個獨立的共用模組中（例如 `app/utils/reviewComments.js`），並讓測試檔案和任何需要用到它們的應用程式邏輯都從該模組匯入。這樣能確保「評論格式」的定義只有一個來源，提升可維護性。",
-    "is_new": true
-  },
-  {
-    "level": "info",
-    "role": "Bard",
-    "location": "app/comments.test.js:192",
-    "problem": "此處新增的 `reviewSeverityLabel` 函式，雖其意圖在上下文脈絡中尚稱清晰，但若能為其添上一筆簡潔的 JSDoc 註解，闡明其參數與回傳值的語義，將使這段樂章更臻完善，即便在測試檔案中，亦能提升未來維護者的閱讀體驗，使程式碼的旋律更加和諧。",
-    "suggestion": "建議為 `reviewSeverityLabel` 函式加上 JSDoc 註解，例如：\n```javascript\n  /**\n   * 從評論物件中提取嚴重等級標籤。\n   * @param {object | null | undefined} comment - 評論物件，預期包含 `body` 屬性。\n   * @returns {string | undefined} 嚴重等級標籤字串（如 '🔴 嚴重'），若無匹配或輸入無效則回傳 undefined。\n   */\n  function reviewSeverityLabel(comment) {\n    return comment?.body?.match(REVIEW_SEVERITY_PATTERN)?.[1];\n  }\n```",
-    "is_new": true
-  }
-]