node-actions/ai-code-review
2
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Releases 14 Activity

feat(ai-code-review): 建問題模式導向 issue，並整併流程調整、文件與 CI #6

Merged
admin merged 79 commits from ai-review-resolve/develop-20260717-185330 into develop 2026-07-21 09:39:48 +00:00
Conversation 25 Commits 79 Files Changed 30
+10 -14
1 changed files with 10 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 491c170d36 - Show all commits
src/lib/review.js
+10 -14
View File
@@ -41,10 +41,9 @@ function redactSecrets(text) {
* *
* 安全考量：AI CLI 失敗時可能在 stderr／stdout 回顯提示內容、環境資訊、token、PII 或 * 安全考量：AI CLI 失敗時可能在 stderr／stdout 回顯提示內容、環境資訊、token、PII 或
* 原始碼祕密，這些會被長期保存並供多人讀取的 CI log 收錄。權衡「可除錯性」後：本函式 * 原始碼祕密，這些會被長期保存並供多人讀取的 CI log 收錄。權衡「可除錯性」後：本函式
* 於失敗時**預設**附上經 {@link redactSecrets} 遮罩且去除控制字元的 **stderr** 片段 * 於失敗時**預設**附上經 {@link redactSecrets} 遮罩且去除控制字元的 **stderr 與 stdout**
* （先截去過長輸入再取前 500 字）——只印 exit code 幾乎無從判斷 CLI * 片段（各先截去過長輸入再取前 500 字）——只印 exit code 幾乎無從判斷 CLI 為何失敗，
* 為何失敗；**另在明確開啟 Actions step debug**（環境變數 `ACTIONS_STEP_DEBUG=true`）時， * 且部分 CLI（如 claude-code 的 `-p` 模式）將錯誤寫到 stdout 而非 stderr。純函式、不拋例外。
* 再附上同樣遮罩的 **stdout** 片段作為第二診斷來源。純函式、不拋例外。
* *
* @param {{error: (Error & {code?: number|string, signal?: string, killed?: boolean})|null, stderr?: string, output?: string}} res * @param {{error: (Error & {code?: number|string, signal?: string, killed?: boolean})|null, stderr?: string, output?: string}} res
* `runAgent` 的回傳物件。 * `runAgent` 的回傳物件。
@@ -66,17 +65,14 @@ function agentFailureDetail(res) {
} }
// 先截去過長輸入再遮罩，避免對數 MB 的失敗輸出跑整份 O(k×n) 正規掃描； // 先截去過長輸入再遮罩，避免對數 MB 的失敗輸出跑整份 O(k×n) 正規掃描；
// 2000 字上限已足以涵蓋跨界機密樣式，最終仍截為 500 字。 // 2000 字上限已足以涵蓋跨界機密樣式，最終仍截為 500 字。
const DEBUG_INPUT_LIMIT = 2_000; const INPUT_LIMIT = 2_000;
// 預設即附上「經 redactSecrets 遮罩＋去控制字元＋限長」的 stderr 片段——CLI 失敗時若只印 // 預設即附上「經 redactSecrets 遮罩＋去控制字元＋限長」的 stderr 與 stdout 片段——CLI 失敗時
// exit code 幾乎無從除錯（見 test-claude 秒失敗案例）；遮罩為盡力防線，仍以此權衡可除錯性。 // 只印 exit code 幾乎無從除錯（見 test-claude 秒失敗案例）；且部分 CLI（如 claude-code 的
const stderr = redactSecrets(String((res && res.stderr) || '').slice(0, DEBUG_INPUT_LIMIT)); // -p 模式）會把錯誤寫到 stdout 而非 stderr，故兩者都輸出。redactSecrets 為盡力防線。
const stderr = redactSecrets(String((res && res.stderr) || '').slice(0, INPUT_LIMIT));
if (stderr) parts.push(`stderr：${stderr.slice(0, 500)}`); if (stderr) parts.push(`stderr：${stderr.slice(0, 500)}`);
// 明確開啟 Actions step debug 時，再附上 stdout 片段作為第二診斷來源。 const stdout = redactSecrets(String((res && res.output) || '').slice(0, INPUT_LIMIT));
const verbose = String(process.env.ACTIONS_STEP_DEBUG || '').trim().toLowerCase() === 'true'; if (stdout) parts.push(`stdout：${stdout.slice(0, 500)}`);
if (verbose) {
const stdout = redactSecrets(String((res && res.output) || '').slice(0, DEBUG_INPUT_LIMIT));
if (stdout) parts.push(`stdout：${stdout.slice(0, 500)}`);
}
if (parts.length === 0) { if (parts.length === 0) {
parts.push((err && err.message && redactSecrets(err.message)) || 'AI CLI 執行失敗（無診斷輸出）'); parts.push((err && err.message && redactSecrets(err.message)) || 'AI CLI 執行失敗（無診斷輸出）');
} }