diff --git a/src/index.js b/src/index.js index 15f2be6..8131148 100644 --- a/src/index.js +++ b/src/index.js @@ -198,7 +198,7 @@ async function main() { * 使用情境:只在 `main()` 內部使用,處理工具資訊、diff 摘要、角色登場與 * 警告/建議彙整等留言。若 Gitea API 失敗,例外會往上拋出並由主流程頂層 catch 收斂。 */ - const postComment = async (body) => { + const queueOrPostComment = async (body) => { if (ctx.createIssue) { if (issue) return gitea.createCommentOnIssue(ctx, issue.number, body); issueBuffer.push(body); @@ -217,7 +217,7 @@ async function main() { * 空陣列或省略時不掛任何標籤(`gitea.createIssue` 對空陣列不帶 labels 欄位)。 * @returns {Promise} 無回傳值;結果反映在閉包變數 `issue` 與 issue 留言。 */ - const ensureIssueCreated = async (labelIds = []) => { + const createIssueAndFlushBufferedComments = async (labelIds = []) => { issue = await gitea.createIssue(ctx, { title: ctx.prTitle || `AI Code Review:PR #${ctx.prNumber}`, body: templates.issueBody({ prNumber: ctx.prNumber, prBody: ctx.prBody }), @@ -245,7 +245,7 @@ async function main() { } log('步驟3', 'INF', `選用工具:${tool.name}(${tool.version})。`); const runLink = `${ctx.serverUrl}/${ctx.repository}/actions/runs/${ctx.runId}`; - await postComment( + await queueOrPostComment( templates.toolComment({ toolName: tool.name, version: tool.version, @@ -270,7 +270,7 @@ async function main() { if (ctx.createIssue) { log('步驟4', 'INF', '建問題模式且無可審查變更:靜默通過(不建 issue、PR 不留言)。'); } else { - await postComment(templates.nothingToReviewComment(ignoredCount)); + await queueOrPostComment(templates.nothingToReviewComment(ignoredCount)); // 已成功產生本回合結果留言(無可審查變更),此時才把舊留言標為過時(本回合留言已排除)。 await review.resolveOldComments({ ctx, gitea, currentRunCommentIds }); } @@ -286,20 +286,20 @@ async function main() { const diffRows = review.collectDiffRows({ cwd, files, base, gitrepo }); await review.fillPurposes({ tool, model: ctx.model, cwd, diffRows }); - await postComment(templates.diffComment(diffRows, ignoredCount)); + await queueOrPostComment(templates.diffComment(diffRows, ignoredCount)); // ── 步驟 5:攻擊方角色登場留言 ───────────────────────────────────────── const roles = loadRoles(path.join(ctx.actionPath, 'src', 'prompts', 'roles')); const attackers = attackersOf(roles); const defenders = defendersOf(roles); log('步驟5', 'INF', `攻擊方 ${attackers.length} 位、防守方 ${defenders.length} 位。`); - await postComment(templates.rolesComment({ title: '⚔️ 攻擊方登場', roles: attackers })); + await queueOrPostComment(templates.rolesComment({ title: '⚔️ 攻擊方登場', roles: attackers })); // ── 步驟 6:每個攻擊方一個 sub agent 並行分析,合併問題列表 ──────────── const findings = await review.runAttackers({ tool, model: ctx.model, cwd, attackers, diffRows }); // ── 步驟 7:防守方角色登場留言 ───────────────────────────────────────── - await postComment(templates.rolesComment({ title: '🛡️ 防守方登場', roles: defenders })); + await queueOrPostComment(templates.rolesComment({ title: '🛡️ 防守方登場', roles: defenders })); // ── 步驟 8:防守方裁決 → 排除 → 排序 → 保存 findings ────────────────── const { kept, excluded } = await review.runDefenders({ tool, model: ctx.model, cwd, defenders, findings }); @@ -335,7 +335,7 @@ async function main() { } catch (err) { log('建問題', 'WRN', `標籤挑選失敗(${err.message}),issue 不掛標籤。`); } - await ensureIssueCreated(labelIds); + await createIssueAndFlushBufferedComments(labelIds); } else { // 無保留問題 → 不建 issue、PR 也不留言(靜默通過,暫存的情境留言捨棄)。 log('建問題', 'INF', '沒有保留的問題:靜默通過(不建 issue、PR 不留言)。'); @@ -365,13 +365,13 @@ async function main() { if (ctx.createIssue) { await review.postOthersToIssue({ ctx, gitea, issueNumber: issue.number, others }); } else { - await postComment(templates.othersComment(others)); + await queueOrPostComment(templates.othersComment(others)); log('步驟10', 'INF', `警告+建議表格留言已發布(${others.length} 條)。`); } } // ── 建問題模式收束:在 PR 回貼 issue 連結(雙向關聯);僅在有嚴重問題時才讓 PR 相依於該 issue ─ - // 標籤已於建立 issue 時一次帶入(見上方 selectLabels → ensureIssueCreated),此處不再補掛。 + // 標籤已於建立 issue 時一次帶入(見上方 selectLabels → createIssueAndFlushBufferedComments),此處不再補掛。 if (ctx.createIssue && issue) { await gitea.createIssueComment( ctx, diff --git a/src/lib/review.js b/src/lib/review.js index 0125a22..f348117 100644 --- a/src/lib/review.js +++ b/src/lib/review.js @@ -66,13 +66,15 @@ function agentFailureDetail(res) { // 先截去過長輸入再遮罩,避免對數 MB 的失敗輸出跑整份 O(k×n) 正規掃描; // 2000 字上限已足以涵蓋跨界機密樣式,最終仍截為 500 字。 const INPUT_LIMIT = 2_000; + // 每段診斷片段(stderr/stdout)寫入日誌的字元上限,兩段共用同一政策,抽為具名常數避免兩處各寫一個魔術數字。 + const AGENT_DIAGNOSTIC_OUTPUT_LIMIT = 500; // 預設即附上「經 redactSecrets 遮罩+去控制字元+限長」的 stderr 與 stdout 片段——CLI 失敗時 // 只印 exit code 幾乎無從除錯(見 test-claude 秒失敗案例);且部分 CLI(如 claude-code 的 // -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, AGENT_DIAGNOSTIC_OUTPUT_LIMIT)}`); const stdout = redactSecrets(String((res && res.output) || '').slice(0, INPUT_LIMIT)); - if (stdout) parts.push(`stdout:${stdout.slice(0, 500)}`); + if (stdout) parts.push(`stdout:${stdout.slice(0, AGENT_DIAGNOSTIC_OUTPUT_LIMIT)}`); if (parts.length === 0) { parts.push((err && err.message && redactSecrets(err.message)) || 'AI CLI 執行失敗(無診斷輸出)'); }