feat(建問題模式): PR 相依追蹤 issue、標籤建立時帶入、議題問題逐條可回覆
整併議題 #7 的建問題模式流程調整與多項 findings 修復（index.js、gitea.js 交錯同檔，整檔歸此）。 - addIssueDependency：建立追蹤 issue 後讓 PR 相依於該 issue，issue 關閉前無法合併（需 repo 啟用問題相依）。 - 🔵 #10（Rogue/efficiency）：標籤改於建立 issue 前先 selectLabels、createIssue 時一次帶入，移除事後 addLabelsToIssue 補掛的多餘 API 往返（addLabelsToIssue 保留為通用工具）。 - 使用者補充：建問題模式警告＋建議改逐條發到 issue（postOthersToIssue），每條問題可個別回覆。 - 🔴 #1（Mage/logic）＋使用者補充：resolveOldComments 延後到審查成功產生結果、發布問題留言前才執行，並以 currentRunCommentIds 排除本回合新留言，避免前置步驟失敗時舊結果被清卻無新結果。 - 🔵 #9（Bard/style）：修正「無保留問題」段落註解與實作（靜默通過、不留言）一致。 - 同步更新相關 JSDoc（含 gitea.js listLabels／createIssue／addLabelsToIssue 使用情境）。 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jeffery
co-authored by
Claude Opus 4.8
Claude Opus 4.8
parent
35887f1a68
commit
e44a8ed0cb
+69
-36
@@ -90,6 +90,7 @@ function saveFindings({ cwd, ctx, tool, kept, excluded }) {
|
||||
* 依模式組出 filesToCommit（一般模式：findings 檔＋有變更時的 exclusions.json；
|
||||
* 建問題模式：只有 exclusions.json）後呼叫本函式；另在步驟 4 判定無可審查變更且非建問題模式時，
|
||||
* 也會以 result: 'success' 提交空 findings。
|
||||
* 推送以 `ctx.pushToken`（PAT）優先，使結果 commit 再觸發 CI、由步驟 1 快速回報；
|
||||
* 注意 commit 訊息與模組常數 `BOT_COMMIT_PREFIX` 耦合，修改前綴會使步驟 1 的快速回報失效。
|
||||
*/
|
||||
function commitFindings({ cwd, ctx, files, result }) {
|
||||
@@ -100,6 +101,7 @@ function commitFindings({ cwd, ctx, files, result }) {
|
||||
message: `${BOT_COMMIT_PREFIX}[${result}]`,
|
||||
files,
|
||||
token: ctx.token,
|
||||
pushToken: ctx.pushToken,
|
||||
serverUrl: ctx.serverUrl,
|
||||
repository: ctx.repository,
|
||||
});
|
||||
@@ -119,7 +121,9 @@ function commitFindings({ cwd, ctx, files, result }) {
|
||||
*
|
||||
* 流程概要（步驟 2～10 描述一般模式；建問題模式差異見末段）：
|
||||
* 1. 快速回報 — 最新 commit 若為 ai-review-bot 的結果 commit（[success]/[failure]），直接回報 0/1 不重審；
|
||||
* 2. 將 PR 既有舊留言標記為解決（跳過本回合留言；建問題模式不執行此步）；
|
||||
* 2. 將 PR 既有舊留言標記為解決（跳過本回合留言；建問題模式不執行此步）——
|
||||
* 此步延後到「本回合審查已成功產生結果、即將發布問題留言前」才執行，避免工具偵測／diff／
|
||||
* 攻防裁決任一失敗時舊結果先被清掉卻沒有新結果（一般模式）；
|
||||
* 3. 偵測 AI 工具（antigravity／codex／claude）並留言；
|
||||
* 4. 讀 .reviewignore、整理 git diff 並留言（無可審查變更時：留言＋保存空 findings，
|
||||
* 一般模式 commit success、建問題模式略過 commit，回傳 0）；
|
||||
@@ -129,9 +133,11 @@ function commitFindings({ cwd, ctx, files, result }) {
|
||||
* 9. 嚴重問題逐條掛在程式碼行上留言；
|
||||
* 10. 警告＋建議彙整為單一表格留言；
|
||||
* 建問題模式（input: create-issue）：不執行步驟 2、不觸碰 PR 既有留言；步驟 3～10 的所有留言
|
||||
* 改發到追蹤 issue（工具／diff／角色留言先暫存，確定有保留問題後才建立 issue 並一次寫入，
|
||||
* 嚴重問題與警告＋建議亦發到該 issue）；無保留問題或無可審查變更則不建 issue、PR 也完全不留言（靜默通過）；
|
||||
* 收束時依保留問題補掛 issue 標籤，並在 PR 回貼 issue 連結形成雙向關聯；
|
||||
* 改發到追蹤 issue（工具／diff／角色留言先暫存，確定有保留問題後先挑好標籤、連同標籤一次建立 issue
|
||||
* 並寫入暫存留言，嚴重問題與警告＋建議再逐條發到該 issue，讓每條問題都能被個別回覆）；
|
||||
* 無保留問題或無可審查變更則不建 issue、PR 也完全不留言（靜默通過）；
|
||||
* 收束時在 PR 回貼 issue 連結形成雙向關聯，
|
||||
* 並讓 PR 相依於該 issue（addIssueDependency，issue 完成／關閉前 PR 無法合併；需 repo 啟用問題相依功能）；
|
||||
* 收尾：組 filesToCommit —— 一般模式 commit findings 檔（＋有變更的 exclusions.json）、
|
||||
* 建問題模式只 commit exclusions.json、無檔案可 commit 時略過；
|
||||
* commit 訊息帶結果標記（success＝無嚴重問題、failure＝有嚴重問題）。
|
||||
@@ -199,17 +205,19 @@ async function main() {
|
||||
return created;
|
||||
};
|
||||
/**
|
||||
* 建問題模式：建立追蹤 issue（標題＝PR 標題、本文＝PR 描述＋回溯 PR 的引言，先不掛標籤），
|
||||
* 建問題模式：建立追蹤 issue（標題＝PR 標題、本文＝PR 描述＋回溯 PR 的引言，連同挑好的標籤一次建立），
|
||||
* 並把 `issueBuffer` 內暫存的情境留言依序寫入 issue；設定閉包變數 `issue` 供後續留言直接發到 issue。
|
||||
* 僅於「確定有保留問題」時呼叫一次。
|
||||
* 僅於「確定有保留問題」時呼叫一次。標籤於建立時一次帶入，省去「先建空標籤 issue 再補掛」的多餘 API 往返。
|
||||
*
|
||||
* @param {number[]} [labelIds] - 建立 issue 時要一併掛上的標籤 id 陣列（由 `review.selectLabels` 事先挑選）；
|
||||
* 空陣列或省略時不掛任何標籤（`gitea.createIssue` 對空陣列不帶 labels 欄位）。
|
||||
* @returns {Promise<void>} 無回傳值；結果反映在閉包變數 `issue` 與 issue 留言。
|
||||
*/
|
||||
const ensureIssueCreated = async () => {
|
||||
const ensureIssueCreated = 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 }),
|
||||
labels: [],
|
||||
labels: labelIds,
|
||||
});
|
||||
log('建問題', 'INF', `已建立追蹤 issue #${issue.number}，寫入 ${issueBuffer.length} 則情境留言。`);
|
||||
for (const body of issueBuffer) {
|
||||
@@ -218,12 +226,12 @@ async function main() {
|
||||
issueBuffer.length = 0;
|
||||
};
|
||||
|
||||
// ── 步驟 2：將 PR 既有留言標記為解決（本回合留言除外）───────────────────
|
||||
// 僅一般模式執行；建問題模式不觸碰 PR 既有留言（審查內容改發到 issue）。
|
||||
// 早於偵測工具與所有本回合留言：先把上一回合的 bot 留言標為過時（此時尚無本回合留言）。
|
||||
if (!ctx.createIssue) {
|
||||
await review.resolveOldComments({ ctx, gitea, currentRunCommentIds });
|
||||
}
|
||||
// ── 步驟 2：延後執行 ───────────────────────────────────────────────────
|
||||
// 「將 PR 既有留言標記為解決」原本在此執行，但若工具偵測／diff／攻防裁決任一失敗，
|
||||
// 舊結果會先被清掉卻沒有新結果。故延後到「本回合審查已成功產生結果、發布問題留言前」
|
||||
// 才呼叫 review.resolveOldComments（見下方步驟 4 空變更路徑與步驟 9 前）；
|
||||
// 屆時本回合的工具／diff／角色留言已登錄於 currentRunCommentIds，不會被誤標為過時。
|
||||
// 建問題模式全程不觸碰 PR 既有留言（審查內容改發到 issue）。
|
||||
|
||||
// ── 步驟 3：偵測 AI agent 工具並留言 ──────────────────────────────────
|
||||
const tool = agents.detectTool();
|
||||
@@ -259,6 +267,8 @@ async function main() {
|
||||
log('步驟4', 'INF', '建問題模式且無可審查變更：靜默通過（不建 issue、PR 不留言）。');
|
||||
} else {
|
||||
await postComment(templates.nothingToReviewComment(ignoredCount));
|
||||
// 已成功產生本回合結果留言（無可審查變更），此時才把舊留言標為過時（本回合留言已排除）。
|
||||
await review.resolveOldComments({ ctx, gitea, currentRunCommentIds });
|
||||
}
|
||||
const relativePath = saveFindings({ cwd, ctx, tool, kept: [], excluded: [] });
|
||||
if (ctx.createIssue) {
|
||||
@@ -301,17 +311,42 @@ async function main() {
|
||||
log('步驟8', 'INF', `分組結果：嚴重 ${severe.length} 條、警告＋建議 ${others.length} 條。`);
|
||||
|
||||
// ── 建問題模式：確定有保留問題才建立 issue，並把暫存的情境留言一次寫入；
|
||||
// 無保留問題則不建 issue，改在 PR 留一則審查通過提示。 ──────────────────
|
||||
// 無保留問題則不建 issue、PR 也完全不留言（靜默通過，暫存的情境留言捨棄）。 ──────
|
||||
if (ctx.createIssue) {
|
||||
if (kept.length > 0) {
|
||||
await ensureIssueCreated();
|
||||
// 先依保留問題挑好標籤，於建立 issue 時一次帶入（省去「先建空標籤 issue 再補掛」的多餘 API 往返）；
|
||||
// 標籤挑選失敗一律降級為不掛標籤，不阻斷建 issue 流程。
|
||||
let labelIds = [];
|
||||
try {
|
||||
const labels = await gitea.listLabels(ctx);
|
||||
labelIds = await review.selectLabels({
|
||||
tool,
|
||||
model: ctx.model,
|
||||
cwd,
|
||||
labels,
|
||||
prTitle: ctx.prTitle,
|
||||
prBody: ctx.prBody,
|
||||
findings: kept,
|
||||
});
|
||||
} catch (err) {
|
||||
log('建問題', 'WRN', `標籤挑選失敗（${err.message}），issue 不掛標籤。`);
|
||||
}
|
||||
await ensureIssueCreated(labelIds);
|
||||
} else {
|
||||
// 無保留問題 → 不建 issue、PR 也不留言（靜默通過，暫存的情境留言捨棄）。
|
||||
log('建問題', 'INF', '沒有保留的問題：靜默通過（不建 issue、PR 不留言）。');
|
||||
}
|
||||
}
|
||||
|
||||
// ── 步驟 9：嚴重問題留言（一般模式掛在 PR 程式碼行上；建問題模式發到 issue）─
|
||||
// ── 步驟 2（延後執行，一般模式）：審查已成功產生結果，發布問題留言前才把舊留言標為過時 ─
|
||||
// 延後到此可避免工具偵測／diff／攻防裁決任一失敗時舊結果先被清掉卻無新結果；
|
||||
// 本回合的工具／diff／角色留言已登錄於 currentRunCommentIds，不會被誤標為過時；
|
||||
// 嚴重／其他問題留言於本步驟之後才發布，同樣不受影響。
|
||||
if (!ctx.createIssue) {
|
||||
await review.resolveOldComments({ ctx, gitea, currentRunCommentIds });
|
||||
}
|
||||
|
||||
// ── 步驟 9：嚴重問題留言（一般模式掛在 PR 程式碼行上；建問題模式逐條發到 issue）─
|
||||
if (severe.length > 0) {
|
||||
if (ctx.createIssue) {
|
||||
await review.postSevereToIssue({ ctx, gitea, issueNumber: issue.number, severe });
|
||||
@@ -320,29 +355,20 @@ async function main() {
|
||||
}
|
||||
}
|
||||
|
||||
// ── 步驟 10：警告＋建議彙整為單一表格留言（去向由 postComment 依模式決定）──
|
||||
// ── 步驟 10：警告＋建議——一般模式彙整為單一表格留言到 PR；
|
||||
// 建問題模式逐條發到 issue，讓每條問題都能被個別回覆。 ──
|
||||
if (others.length > 0) {
|
||||
await postComment(templates.othersComment(others));
|
||||
log('步驟10', 'INF', `警告＋建議表格留言已發布（${others.length} 條）。`);
|
||||
if (ctx.createIssue) {
|
||||
await review.postOthersToIssue({ ctx, gitea, issueNumber: issue.number, others });
|
||||
} else {
|
||||
await postComment(templates.othersComment(others));
|
||||
log('步驟10', 'INF', `警告＋建議表格留言已發布（${others.length} 條）。`);
|
||||
}
|
||||
}
|
||||
|
||||
// ── 建問題模式收束：依保留問題補掛 issue 標籤，並在 PR 回貼 issue 連結（雙向關聯）─
|
||||
// ── 建問題模式收束：在 PR 回貼 issue 連結（雙向關聯），並讓 PR 相依於該 issue ─
|
||||
// 標籤已於建立 issue 時一次帶入（見上方 selectLabels → ensureIssueCreated），此處不再補掛。
|
||||
if (ctx.createIssue && issue) {
|
||||
try {
|
||||
const labels = await gitea.listLabels(ctx);
|
||||
const labelIds = await review.selectLabels({
|
||||
tool,
|
||||
model: ctx.model,
|
||||
cwd,
|
||||
labels,
|
||||
prTitle: ctx.prTitle,
|
||||
prBody: ctx.prBody,
|
||||
findings: kept,
|
||||
});
|
||||
await gitea.addLabelsToIssue(ctx, issue.number, labelIds);
|
||||
} catch (err) {
|
||||
log('建問題', 'WRN', `補掛標籤失敗（${err.message}），issue 不掛標籤。`);
|
||||
}
|
||||
await gitea.createIssueComment(
|
||||
ctx,
|
||||
templates.issueLinkComment({
|
||||
@@ -352,6 +378,13 @@ async function main() {
|
||||
otherCount: others.length,
|
||||
}),
|
||||
);
|
||||
// 讓 PR 相依於此追蹤 issue：issue 完成／關閉前 PR 無法合併（需 repo 啟用「問題相依」功能）。
|
||||
try {
|
||||
await gitea.addIssueDependency(ctx, ctx.prNumber, issue.number);
|
||||
log('建問題', 'INF', `已將 PR #${ctx.prNumber} 設為相依於 issue #${issue.number}：issue 關閉前無法合併。`);
|
||||
} catch (err) {
|
||||
log('建問題', 'WRN', `設定 PR 相依失敗（可能未啟用「問題相依」功能）：${err.message}。`);
|
||||
}
|
||||
log('建問題', 'INF', `issue #${issue.number} 已寫入審查內容，並在 PR 回貼連結。`);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user