From 242717a141f9224b3524bdbc326724a631d7983d Mon Sep 17 00:00:00 2001 From: Danny Kim Date: Sun, 13 Sep 2026 23:56:05 +0900 Subject: [PATCH] Verdict is the first line --- prompt.md | 20 ++++++++++-------- run.ts | 61 ++++++++++++++++++++++++++----------------------------- 2 files changed, 40 insertions(+), 41 deletions(-) diff --git a/prompt.md b/prompt.md index 67a8d7f..ee4cbad 100644 --- a/prompt.md +++ b/prompt.md @@ -19,19 +19,21 @@ against the PR's base and head, and make its complete output your final response instead of the short comment style above. Check the whole repository against the code rules at the end of this prompt, not only the diff; a violation is at least Important even when the diff did not cause it. Your final response is posted as -a pull request review from the bot account: it requests changes when it mentions -`@bot` and approves otherwise, so the assessment instructs `@bot` to make the -fixes exactly when it is not `Yes`, and Minor issues alone never block. For UI +a pull request review from the bot account, and its first line must be exactly +the template's verdict and nothing else: `Yes`, `No`, or `With fixes`. `Yes` +approves and the other two request changes; any other first line is posted as a +plain comment. Minor issues alone never block. When the verdict is not `Yes`, +the second line is a one-line instruction to `@bot` to make the fixes. For UI changes, check that the result is aligned, clean, and pixel-perfect, and that included screenshots prove the intended result was achieved. The review must read at a glance: everything outside `
` blocks totals -under 512 bytes. Only core information stays visible: the assessment line, the -section headings, and a one-line `@bot` instruction. Anything verbose goes into -a `
` block whose `` is a few words, such as the `file:line` -and title of an issue with the what, why, and how inside; the same for each -strength, each recommendation, the reasoning, and any compliance notes. Details -blocks are top-level, never inside a list item, because Gitea breaks them there. +under 512 bytes. Only core information stays visible: the verdict, the `@bot` +line, and the section headings. Anything verbose goes into a `
` block +whose `` is a few words, such as the `file:line` and title of an issue +with the what, why, and how inside; the same for each strength, each +recommendation, the reasoning, and any compliance notes. Details blocks are +top-level, never inside a list item, because Gitea breaks them there. For an `issue_comment` or `pull_request_review_comment` event, treat the `body` in the triggering comment payload below as the user's exact instruction. diff --git a/run.ts b/run.ts index 98aea80..0850f86 100644 --- a/run.ts +++ b/run.ts @@ -52,29 +52,21 @@ async function postComment(body: string): Promise { }); } -// A review must read at a glance: when the agent left more than VISIBLE_BYTES -// outside
blocks, the whole review folds under its verdict line. -const VISIBLE_BYTES = 512; -function fold(body: string): string { - const visible = body.replace(/
[\s\S]*?<\/details>/g, ""); - if (new TextEncoder().encode(visible).length <= VISIBLE_BYTES) return body; - const verdict = body.match(/\*\*Ready to merge\?\*\*.*/)?.[0] ?? "Review"; - return `${verdict}\n\n
\nDetails\n\n${body}\n\n
`; -} +// A pull request event is a review request, so the response becomes a review. +// Its first line is the verdict; anything unexpected only comments, never +// approves. +const VERDICTS: Record = { + Yes: "APPROVED", + No: "REQUEST_CHANGES", + "With fixes": "REQUEST_CHANGES", +}; -// A pull request event is a review request, so the response becomes a review: -// changes are requested when the agent asked @bot to fix something, a failed -// run only comments, and anything else approves. async function postResult(body: string): Promise { if (EVENT !== "pull_request") return postComment(body); - const event = body.includes("@bot") - ? "REQUEST_CHANGES" - : body.startsWith("Bot failed:") - ? "COMMENT" - : "APPROVED"; + const [verdict] = body.split("\n", 1); await gitea(REVIEWER_TOKEN, `repos/${REPO}/pulls/${INDEX}/reviews`, { - body: fold(stripAnsi(body)), - event, + body: stripAnsi(body), + event: VERDICTS[verdict.trim()] ?? "COMMENT", }); } @@ -132,10 +124,9 @@ async function renderPrompt(): Promise { async function runClaude(prompt: string): Promise { const token = Deno.env.get("BOT_TOKEN"); if (!token) { - await postComment( + throw new Error( "Run `claude setup-token` locally and set the `bot-token` action input.", ); - Deno.exit(1); } await installSuperpowers("claude"); const claude = new Deno.Command("claude", { @@ -154,7 +145,7 @@ async function runClaude(prompt: string): Promise { clearEnv: true, stdout: "piped", }).spawn(); - let result = "Bot failed: no result"; + let result: { result?: string; subtype: string } | undefined; // Print events as they stream so the runner does not kill the job as a zombie. const lines = claude.stdout .pipeThrough(new TextDecoderStream()) @@ -166,12 +157,13 @@ async function runClaude(prompt: string): Promise { const text = part.thinking ?? part.text ?? part.name; if (text) console.log(text); } - if (event.type === "result") { - result = event.result ?? `Bot failed: ${event.subtype}`; - } + if (event.type === "result") result = event; } await claude.status; - return result; + if (result?.result === undefined) { + throw new Error(`claude ended with ${result?.subtype ?? "no result"}`); + } + return result.result; } // Posts the device code so a human can finish the login on the persisted home. @@ -222,9 +214,14 @@ async function runCodex(prompt: string): Promise { return await Deno.readTextFile(file); } -await configureGitAuthor(); -const prompt = await renderPrompt(); -const result = env("BOT_TYPE") === "claude" - ? await runClaude(prompt) - : await runCodex(prompt); -await postResult(result); +try { + await configureGitAuthor(); + const prompt = await renderPrompt(); + const result = env("BOT_TYPE") === "claude" + ? await runClaude(prompt) + : await runCodex(prompt); + await postResult(result); +} catch (error) { + await postComment(`Bot failed: ${error}`); + throw error; +}