From 14c7ac4ed062e9bc0ab71ff5e2c7f67d1767fc90 Mon Sep 17 00:00:00 2001 From: Jeffery Date: Fri, 26 Jun 2026 14:05:08 +0800 Subject: [PATCH] =?UTF-8?q?chore(ai-review=20=E7=8B=80=E6=85=8B):=20?= =?UTF-8?q?=E6=9B=B4=E6=96=B0=20findings.json=20=E8=88=87=20exclusions.jso?= =?UTF-8?q?n?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 已解決 5 條（critical x1、warning x2、info x2：多位元組截斷、repository 驗證與命名、邊界測試、串流攔截 DRY），1 條判定為誤報寫入 exclusions（warning：logger 改 DI 屬過度設計），findings 清空。 Co-Authored-By: Claude Opus 4.8 (1M context) --- .gitea/ai-review/exclusions.json | 6 +++++ .gitea/ai-review/findings.json | 46 +------------------------------- 2 files changed, 7 insertions(+), 45 deletions(-) diff --git a/.gitea/ai-review/exclusions.json b/.gitea/ai-review/exclusions.json index 7a5fd72..10abb06 100644 --- a/.gitea/ai-review/exclusions.json +++ b/.gitea/ai-review/exclusions.json @@ -64,5 +64,11 @@ "role": "Leo", "original_finding": "withFetch 為全域 fetch 的通用封裝工具，目前定義在特定測試檔案內。建議提取至獨立的測試工具檔案（例如 app/test/test-utils.js）以提升重用性。", "reason": "目前僅 releases.test.js 單一測試檔使用 withFetch（logger.test.js 使用的是不同的 stdout/stderr 攔截輔助），尚無第二個消費者；為單一用途提前抽出共用模組屬過度設計。" + }, + { + "location": "app/test/logger.test.js:8", + "role": "Leo", + "original_finding": "在測試中使用 monkey-patch 直接覆蓋 process.stdout.write 或 process.stderr.write 是極度危險的模式，可能導致狀態污染；建議重構 logger 使其支援依賴注入。", + "reason": "攔截已於 finally 還原原本的 write，無狀態污染，且為 Node 測試 stdout/stderr 輸出的標準作法；logger 僅是寫入 stdout/stderr 的薄封裝，為其引入依賴注入框架屬過度設計。已將兩個攔截輔助合併為單一 captureStream。" } ] diff --git a/.gitea/ai-review/findings.json b/.gitea/ai-review/findings.json index ef308d5..fe51488 100644 --- a/.gitea/ai-review/findings.json +++ b/.gitea/ai-review/findings.json @@ -1,45 +1 @@ -[ - { - "level": "critical", - "role": "Maya", - "location": "app/releases.js:68", - "problem": "新增的異常處理邏輯截斷了 `text` 為 200 字元，但在 `text` 為極大值或無法轉為字串時未進行防禦性檢查，且截斷點可能將多位元組字元拆分導致亂碼。", - "suggestion": "請確保 `text` 是字串後再截斷，並使用正確處理多位元組字元的方式（如 `Array.from(text).slice(0, 200).join('')`）以避免亂碼。", - "is_new": true - }, - { - "level": "warning", - "role": "Maya", - "location": "app/config.js:61", - "problem": "新增的 `assertRepository` 函式缺乏對 owner/repo 字串長度的邊界檢查，且變數名稱 `valid` 過於通用，語義不夠明確。", - "suggestion": "建議在 `assertRepository` 中加入長度限制檢查，並將 `valid` 更名為 `isRepoFormatValid` 或類似具備明確語義的命名。" - }, - { - "level": "warning", - "role": "Leo", - "location": "app/test/logger.test.js:8", - "problem": "在測試中使用 monkey-patch 直接覆蓋 `process.stdout.write` 或 `process.stderr.write` 是極度危險的模式，可能導致狀態污染；且針對 `logger.error` 的測試不夠全面。", - "suggestion": "建議重構 `logger` 使其支援依賴注入；並增加針對錯誤內容包含情況及非字串參數的斷言測試。" - }, - { - "level": "warning", - "role": "Maya", - "location": "app/test/config.test.js:84", - "problem": "缺少針對 `assertRepository` 邊界條件（如空字串、異常格式）的測試。", - "suggestion": "補上 `assert.throws` 測試案例以涵蓋上述邊界條件，確保驗證邏輯完整。" - }, - { - "level": "info", - "role": "Bard", - "location": "app/releases.js:67", - "problem": "將截斷字串直接串接在 Error 訊息中若處理不當易造成混淆，變數名稱 `snippet` 語義不明確。", - "suggestion": "建議在錯誤訊息中對片段內容進行適當標記，並將變數更名為 `contentSnippet`。" - }, - { - "level": "info", - "role": "Bard", - "location": "app/test/logger.test.js:10", - "problem": "測試工具函式 `captureStdout` 與 `captureStderr` 邏輯高度重複，不符合 DRY 原則。", - "suggestion": "建議抽取出通用的 `captureStream(stream, fn)` 函式以提升可維護性。" - } -] +[]