next-issue 2.4.1 → 2.4.3

package/dist/prompts.mjs

@@ -1,35 +1,35 @@
 const COMMIT_RULES = [
-    "Do not run git, gh or any push command. The harness commits for you.",
-    "End your final message with one line in the form `commit: <conventional commit subject>`.",
-].join("\n");
+    'Do not run git, gh or any push command. The harness commits for you.',
+    'End your final message with one line in the form `commit: <conventional commit subject>`.',
+].join('\n');
 function issueBlock(issue, comments) {
     return [
         `# Issue #${issue.number}: ${issue.title}`,
-        issue.body.length > 0 ? issue.body : "(no description)",
-        comments.length > 0 ? `## Comments\n${comments.join("\n\n")}` : "",
+        issue.body.length > 0 ? issue.body : '(no description)',
+        comments.length > 0 ? `## Comments\n${comments.join('\n\n')}` : '',
     ]
         .filter((part) => part.length > 0)
-        .join("\n\n");
+        .join('\n\n');
 }
 export function implementPrompt(issue, comments) {
     return [
-        "You are the implementer. Solve the issue below in this worktree.",
+        'You are the implementer. Solve the issue below in this worktree.',
         issueBlock(issue, comments),
-        "## Rules",
-        "Read the repository conventions first. Keep the change minimal and focused on the issue.",
-        "Add or update tests when the repository has a test suite.",
+        '## Rules',
+        'Read the repository conventions first. Keep the change minimal and focused on the issue.',
+        'Add or update tests when the repository has a test suite.',
         COMMIT_RULES,
-    ].join("\n\n");
+    ].join('\n\n');
 }
 export function hookPrompt(issue, comments, role, output) {
     return [
         `You are the ${role}. The harness could not commit your work for the issue below, because a git hook failed.`,
         issueBlock(issue, comments),
         `## Hook output\n${output}`,
-        "## Rules",
-        "Fix what the hook reports and keep the rest of the work in this worktree. Do not skip or weaken the hook.",
+        '## Rules',
+        'Fix what the hook reports and keep the rest of the work in this worktree. Do not skip or weaken the hook.',
         COMMIT_RULES,
-    ].join("\n\n");
+    ].join('\n\n');
 }
 export function capDiff(diff, maxChars) {
     return diff.length <= maxChars
@@ -38,42 +38,42 @@ export function capDiff(diff, maxChars) {
 }
 export function reviewPrompt(issue, comments, diff, earlier) {
     const scope = earlier.length === 0
-        ? "Check correctness, scope, repository conventions and test cover."
+        ? 'Check correctness, scope, repository conventions and test cover.'
         : [
-            "This is a later round. Judge two things only:",
-            "whether the earlier findings below are now fixed, and whether the new commits broke something.",
-            "Do not raise a new point of taste. Do not repeat a finding that is fixed.",
-            "A `The fixer disputed every finding of this round` line answers all of the findings of that round, not one of them.",
-            "Withdraw every finding of that round if the dispute is correct. Repeat them all if the dispute is wrong.",
-            `## Earlier findings\n${earlier.join("\n")}`,
-        ].join("\n");
+            'This is a later round. Judge two things only:',
+            'whether the earlier findings below are now fixed, and whether the new commits broke something.',
+            'Do not raise a new point of taste. Do not repeat a finding that is fixed.',
+            'A `The fixer disputed every finding of this round` line answers all of the findings of that round, not one of them.',
+            'Withdraw every finding of that round if the dispute is correct. Repeat them all if the dispute is wrong.',
+            `## Earlier findings\n${earlier.join('\n')}`,
+        ].join('\n');
     return [
-        "You are the reviewer. Judge whether the diff solves the issue correctly.",
+        'You are the reviewer. Judge whether the diff solves the issue correctly.',
         issueBlock(issue, comments),
-        "## Diff",
-        "```diff",
+        '## Diff',
+        '```diff',
         diff,
-        "```",
-        "## Rules",
-        "Read files and run commands to prove what you say: the tests, the type check or the linter.",
-        "The worktree is ready to use. Do not change a file. Do not write with git or gh.",
-        "The harness discards each change that you make, so a change is only lost work.",
+        '```',
+        '## Rules',
+        'Read files and run commands to prove what you say: the tests, the type check or the linter.',
+        'The worktree is ready to use. Do not change a file. Do not write with git or gh.',
+        'The harness discards each change that you make, so a change is only lost work.',
         scope,
-        "Mark a finding blocking only for a wrong result, a missing part of the issue, a regression or a broken convention.",
-        "Give the result with the submit_result tool: a short summary and one entry per finding. The findings decide the outcome, so a problem that you leave out counts as no problem.",
-    ].join("\n\n");
+        'Mark a finding blocking only for a wrong result, a missing part of the issue, a regression or a broken convention.',
+        'Give the result with the submit_result tool: a short summary and one entry per finding. The findings decide the outcome, so a problem that you leave out counts as no problem.',
+    ].join('\n\n');
 }
 const UNRELATED_RULE = [
-    "The checks can fail for a reason this change did not cause: a test that is already broken on the base branch, one that exercises code the diff does not touch, or one that fails only under load.",
-    "Establish that before you fix anything. If the failure is not yours, change nothing and end your final message with one line in the form `unrelated: <one sentence on what fails and why it is not this change>`.",
-    "A person then reads it. Do not weaken, skip or re-record a test to make the checks pass.",
-].join("\n");
+    'The checks can fail for a reason this change did not cause: a test that is already broken on the base branch, one that exercises code the diff does not touch, or one that fails only under load.',
+    'Establish that before you fix anything. If the failure is not yours, change nothing and end your final message with one line in the form `unrelated: <one sentence on what fails and why it is not this change>`.',
+    'A person then reads it. Do not weaken, skip or re-record a test to make the checks pass.',
+].join('\n');
 const DISPUTE_RULE = [
-    "A finding can be wrong: it can miss a convention of this repository, ask for work that the issue does not want, or describe behaviour that the code does not have.",
-    "Establish that before you change anything. If every finding is wrong, change nothing and end your final message with one line in the form `unrelated: <one sentence on why the findings are wrong>`.",
-    "If only some of the findings are wrong, fix the correct ones and change nothing for the wrong ones. Write no `unrelated` line in that round. The next round holds only the findings that are left, and you can dispute them there.",
-    "The reviewer reads that line in the next round. Do not make a change that you hold to be wrong only to end the round.",
-].join("\n");
+    'A finding can be wrong: it can miss a convention of this repository, ask for work that the issue does not want, or describe behaviour that the code does not have.',
+    'Establish that before you change anything. If every finding is wrong, change nothing and end your final message with one line in the form `unrelated: <one sentence on why the findings are wrong>`.',
+    'If only some of the findings are wrong, fix the correct ones and change nothing for the wrong ones. Write no `unrelated` line in that round. The next round holds only the findings that are left, and you can dispute them there.',
+    'The reviewer reads that line in the next round. Do not make a change that you hold to be wrong only to end the round.',
+].join('\n');
 export function fixPrompt(issue, reason, detail, earlier, kind) {
     return [
         `You are the fixer. Correct the work for issue #${issue.number}: ${issue.title}.`,
@@ -81,18 +81,18 @@ export function fixPrompt(issue, reason, detail, earlier, kind) {
         `## Detail\n${detail}`,
         earlier.length > 0
             ? [
-                "## Earlier rounds",
-                earlier.join("\n"),
-                "Keep those earlier fixes. Do not undo work that a later round did not question.",
-            ].join("\n")
-            : "",
-        "## Rules",
-        "Fix the cause, not the symptom. Keep the change minimal.",
-        kind === "checks" ? UNRELATED_RULE : DISPUTE_RULE,
+                '## Earlier rounds',
+                earlier.join('\n'),
+                'Keep those earlier fixes. Do not undo work that a later round did not question.',
+            ].join('\n')
+            : '',
+        '## Rules',
+        'Fix the cause, not the symptom. Keep the change minimal.',
+        kind === 'checks' ? UNRELATED_RULE : DISPUTE_RULE,
         COMMIT_RULES,
     ]
         .filter((part) => part.length > 0)
-        .join("\n\n");
+        .join('\n\n');
 }
 export function parseUnrelated(text) {
     const match = [...text.matchAll(/^unrelated:\s*(.+)$/gim)].at(-1);
@@ -103,7 +103,7 @@ export function parseCommitSubject(text, fallback) {
     const match = [...text.matchAll(/^commit:\s*(.+)$/gim)].at(-1);
     const subject = match?.[1]
         ?.trim()
-        .replace(/^`+|`+$/g, "")
+        .replace(/^`+|`+$/g, '')
         .trim();
     return subject && subject.length > 0 ? subject : fallback;
 }

package/dist/run.mjs

@@ -1,27 +1,27 @@
-import { writeFile } from "node:fs/promises";
-import { join, sep } from "node:path";
-import { loadConfig } from "./config.mjs";
-import { setCommandObserver, setDefaultTimeout } from "./exec.mjs";
-import { detectRepo } from "./git.mjs";
-import { defaultBranch } from "./github.mjs";
-import { exitCode, runIssues, watchIssues } from "./loop.mjs";
-import { message, pruneRuns, Recorder } from "./observe.mjs";
-import { pendingIssues, PORTS, processIssue } from "./pipeline.mjs";
-import { STATE_DIR } from "./state.mjs";
-import { packageVersion } from "./version.mjs";
+import { writeFile } from 'node:fs/promises';
+import { join, sep } from 'node:path';
+import { loadConfig } from './config.mjs';
+import { setCommandObserver, setDefaultTimeout } from './exec.mjs';
+import { detectRepo } from './git.mjs';
+import { defaultBranch } from './github.mjs';
+import { exitCode, runIssues, watchIssues } from './loop.mjs';
+import { message, pruneRuns, Recorder } from './observe.mjs';
+import { pendingIssues, PORTS, processIssue } from './pipeline.mjs';
+import { STATE_DIR } from './state.mjs';
+import { packageVersion } from './version.mjs';
 const recorders = new Map();
 setCommandObserver((record) => {
     for (const [root, recorder] of recorders) {
         if (record.cwd !== undefined && record.cwd !== root && !record.cwd.startsWith(root + sep)) {
             continue;
         }
-        recorder.event(record.code === 0 ? "command" : "command.fail", {
-            cmd: `${record.command} ${record.args.join(" ")}`,
+        recorder.event(record.code === 0 ? 'command' : 'command.fail', {
+            cmd: `${record.command} ${record.args.join(' ')}`,
             code: record.code,
             ms: record.ms,
             stderr: record.stderr.length > 0 ? record.stderr : undefined,
             timedOut: record.timedOut ? true : undefined,
-        }, record.code === 0 ? "verbose" : "quiet");
+        }, record.code === 0 ? 'verbose' : 'quiet');
     }
 });
 export async function runRepo(root, args, controls) {
@@ -31,7 +31,7 @@ export async function runRepo(root, args, controls) {
     const recorder = await Recorder.create(repo.root, args.level, controls.onEvent);
     const pruned = await pruneRuns(repo.root, config.keepRunDays);
     if (pruned > 0) {
-        recorder.event("runs.pruned", { files: pruned, keepDays: config.keepRunDays }, "verbose");
+        recorder.event('runs.pruned', { files: pruned, keepDays: config.keepRunDays }, 'verbose');
     }
     recorders.set(repo.root, recorder);
     try {
@@ -55,16 +55,16 @@ async function handle(args, config, repo, recorder, controls) {
         recorder,
         ports: PORTS,
     };
-    recorder.event("run.start", {
+    recorder.event('run.start', {
         version: await packageVersion(),
         repo: `${repo.owner}/${repo.name}`,
         base,
         log: recorder.logFile,
-    }, "quiet");
+    }, 'quiet');
     const loop = {
         issues: async () => {
             const issues = await pendingIssues(context);
-            recorder.event("issues", { open: issues.length }, "quiet");
+            recorder.event('issues', { open: issues.length }, 'quiet');
             return issues;
         },
         process: (issue) => processIssue(context, issue),
@@ -76,7 +76,7 @@ async function handle(args, config, repo, recorder, controls) {
         ? await watchIssues(args, {
             ...loop,
             wait: (minutes) => {
-                recorder.event("watch.wait", { minutes }, "quiet");
+                recorder.event('watch.wait', { minutes }, 'quiet');
                 passStart = recorder.summary();
                 return interrupts.sleep(minutes * 60_000);
             },
@@ -84,20 +84,20 @@ async function handle(args, config, repo, recorder, controls) {
         })
         : await runIssues(args, loop);
     if (stoppedAt !== undefined) {
-        recorder.event("run.stop", { issue: stoppedAt }, "quiet");
+        recorder.event('run.stop', { issue: stoppedAt }, 'quiet');
     }
     if (error !== undefined) {
-        recorder.event("issues.fail", { error: message(error) }, "quiet");
+        recorder.event('issues.fail', { error: message(error) }, 'quiet');
     }
-    recorder.event("run.end", {
-        done: count(reports, "done"),
-        needsHuman: count(reports, "needs-human"),
-        failed: count(reports, "error"),
-        skipped: count(reports, "skipped"),
+    recorder.event('run.end', {
+        done: count(reports, 'done'),
+        needsHuman: count(reports, 'needs-human'),
+        failed: count(reports, 'error'),
+        skipped: count(reports, 'skipped'),
         tokens: recorder.summary().total.total,
-    }, "quiet");
+    }, 'quiet');
     const summary = { ...recorder.summary(), issues: reports };
-    const summaryFile = join(repo.root, STATE_DIR, "runs", `${recorder.runId}.summary.json`);
+    const summaryFile = join(repo.root, STATE_DIR, 'runs', `${recorder.runId}.summary.json`);
     await writeFile(summaryFile, `${JSON.stringify(summary, null, 2)}\n`);
     if (!args.watch) {
         await print(summary);

package/dist/serve.mjs

@@ -1,24 +1,24 @@
-import { rmSync } from "node:fs";
-import { chmod, readFile, rm, writeFile } from "node:fs/promises";
-import { createServer } from "node:http";
-import { join } from "node:path";
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { WebSocketServer } from "ws";
-import { parseArgs } from "./args.mjs";
-import { stopChildren } from "./exec.mjs";
-import { message } from "./observe.mjs";
-import { applyEvent, startProgress } from "./progress.mjs";
-import { runRepo } from "./run.mjs";
-import { packageVersion } from "./version.mjs";
+import { rmSync } from 'node:fs';
+import { chmod, readFile, rm, writeFile } from 'node:fs/promises';
+import { createServer } from 'node:http';
+import { join } from 'node:path';
+import { getAgentDir } from '@earendil-works/pi-coding-agent';
+import { WebSocketServer } from 'ws';
+import { parseArgs } from './args.mjs';
+import { stopChildren } from './exec.mjs';
+import { message } from './observe.mjs';
+import { applyEvent, startProgress } from './progress.mjs';
+import { runRepo } from './run.mjs';
+import { packageVersion } from './version.mjs';
 export function socketPaths(agentDir = getAgentDir()) {
     return {
-        socketPath: join(agentDir, "next-issue.sock"),
-        lockPath: join(agentDir, "next-issue.lock"),
+        socketPath: join(agentDir, 'next-issue.sock'),
+        lockPath: join(agentDir, 'next-issue.lock'),
     };
 }
 export function isNewer(candidate, current) {
-    const left = candidate.split(".").map(Number);
-    const right = current.split(".").map(Number);
+    const left = candidate.split('.').map(Number);
+    const right = current.split('.').map(Number);
     for (let index = 0; index < Math.max(left.length, right.length); index += 1) {
         const difference = (left[index] ?? 0) - (right[index] ?? 0);
         if (difference !== 0) {
@@ -33,28 +33,28 @@ function alive(pid) {
         return true;
     }
     catch (error) {
-        return error.code === "EPERM";
+        return error.code === 'EPERM';
     }
 }
 async function holder(lockPath) {
-    const pid = Number((await readFile(lockPath, "utf8").catch(() => "")).trim());
+    const pid = Number((await readFile(lockPath, 'utf8').catch(() => '')).trim());
     return pid > 0 && alive(pid) ? pid : undefined;
 }
 export async function stopWorker(lockPath) {
     const pid = await holder(lockPath);
     if (pid !== undefined) {
-        process.kill(pid, "SIGTERM");
+        process.kill(pid, 'SIGTERM');
     }
     return pid;
 }
 async function lock(path) {
     for (let attempt = 0; attempt < 2; attempt += 1) {
         try {
-            await writeFile(path, `${process.pid}\n`, { flag: "wx", mode: 0o600 });
+            await writeFile(path, `${process.pid}\n`, { flag: 'wx', mode: 0o600 });
             return true;
         }
         catch (error) {
-            if (error.code !== "EEXIST") {
+            if (error.code !== 'EEXIST') {
                 throw error;
             }
             if ((await holder(path)) !== undefined) {
@@ -79,7 +79,7 @@ export async function startWorker(options) {
     const server = createServer();
     const sockets = new WebSocketServer({ server });
     const snapshot = () => ({
-        type: "runs",
+        type: 'runs',
         runs: Object.fromEntries([...runs].map(([root, run]) => [root, run.progress])),
     });
     const send = (socket, value) => {
@@ -91,7 +91,7 @@ export async function startWorker(options) {
             send(socket, value);
         }
     };
-    const busy = () => [...runs.values()].some((run) => run.progress.status === "running" || run.progress.status === "stopping");
+    const busy = () => [...runs.values()].some((run) => run.progress.status === 'running' || run.progress.status === 'stopping');
     const close = async () => {
         for (const socket of sockets.clients) {
             socket.terminate();
@@ -109,12 +109,12 @@ export async function startWorker(options) {
     };
     const start = (socket, root, merge) => {
         const current = runs.get(root);
-        if (current?.progress.status === "running" || current?.progress.status === "stopping") {
-            send(socket, { type: "error", root, message: "a run is going in this repository already" });
+        if (current?.progress.status === 'running' || current?.progress.status === 'stopping') {
+            send(socket, { type: 'error', root, message: 'a run is going in this repository already' });
             return;
         }
         if (restart) {
-            send(socket, { type: "error", root, message: "the worker is restarting, try again soon" });
+            send(socket, { type: 'error', root, message: 'the worker is restarting, try again soon' });
             return;
         }
         const run = { progress: startProgress(), stop: false };
@@ -145,25 +145,29 @@ export async function startWorker(options) {
             onEvent: (record) => update(applyEvent(run.progress, record)),
         })
             .then(() => {
-            const status = run.stop ? "stopped" : run.progress.error === undefined ? "finished" : "failed";
+            const status = run.stop
+                ? 'stopped'
+                : run.progress.error === undefined
+                    ? 'finished'
+                    : 'failed';
             update({ ...run.progress, status });
-        }, (error) => update({ ...run.progress, status: "failed", error: message(error) }))
+        }, (error) => update({ ...run.progress, status: 'failed', error: message(error) }))
             .finally(restartWhenIdle);
     };
     const stop = (root) => {
         const run = runs.get(root);
-        if (run === undefined || run.progress.status !== "running") {
+        if (run === undefined || run.progress.status !== 'running') {
             return;
         }
         run.stop = true;
         run.wake?.();
-        run.progress = { ...run.progress, status: "stopping" };
+        run.progress = { ...run.progress, status: 'stopping' };
         broadcast();
     };
-    sockets.on("connection", (socket) => {
-        send(socket, { type: "hello", version: options.version });
+    sockets.on('connection', (socket) => {
+        send(socket, { type: 'hello', version: options.version });
         send(socket, snapshot());
-        socket.on("message", (data) => {
+        socket.on('message', (data) => {
             let value;
             try {
                 value = JSON.parse(String(data));
@@ -171,24 +175,24 @@ export async function startWorker(options) {
             catch {
                 return;
             }
-            if (value.type === "hello") {
+            if (value.type === 'hello') {
                 if (isNewer(value.version, options.version)) {
                     restart = true;
                     restartWhenIdle();
                 }
             }
-            else if (value.type === "start") {
+            else if (value.type === 'start') {
                 start(socket, value.root, value.merge === true);
             }
-            else if (value.type === "stop") {
+            else if (value.type === 'stop') {
                 stop(value.root);
             }
         });
     });
     await new Promise((resolve, reject) => {
-        server.once("error", reject);
+        server.once('error', reject);
         server.listen(options.socketPath, () => {
-            server.off("error", reject);
+            server.off('error', reject);
             resolve();
         });
     });
@@ -203,20 +207,20 @@ export async function serve() {
         runRepo,
     });
     if (worker === undefined) {
-        process.stderr.write("next-issue: another worker is running\n");
+        process.stderr.write('next-issue: another worker is running\n');
         return 1;
     }
     const end = (signal) => {
-        process.off("SIGTERM", onTerminate);
-        process.off("SIGINT", onInterrupt);
+        process.off('SIGTERM', onTerminate);
+        process.off('SIGINT', onInterrupt);
         rmSync(paths.socketPath, { force: true });
         rmSync(paths.lockPath, { force: true });
         stopChildren(signal);
     };
-    const onTerminate = () => end("SIGTERM");
-    const onInterrupt = () => end("SIGINT");
-    process.on("SIGTERM", onTerminate);
-    process.on("SIGINT", onInterrupt);
+    const onTerminate = () => end('SIGTERM');
+    const onInterrupt = () => end('SIGINT');
+    process.on('SIGTERM', onTerminate);
+    process.on('SIGINT', onInterrupt);
     process.stderr.write(`next-issue: worker listening on ${paths.socketPath}\n`);
     await worker.closed;
     return 0;

package/dist/setup.mjs

@@ -1,17 +1,17 @@
-import { access } from "node:fs/promises";
-import { join } from "node:path";
+import { access } from 'node:fs/promises';
+import { join } from 'node:path';
 const MARKERS = [
-    ["pnpm-lock.yaml", "pnpm install --frozen-lockfile"],
-    ["package-lock.json", "npm ci"],
-    [".yarnrc.yml", "yarn install --immutable"],
-    ["yarn.lock", "yarn install --frozen-lockfile"],
-    ["bun.lock", "bun install --frozen-lockfile"],
-    ["bun.lockb", "bun install --frozen-lockfile"],
-    ["Cargo.lock", "cargo fetch"],
-    ["go.sum", "go mod download"],
-    ["uv.lock", "uv sync"],
-    ["poetry.lock", "poetry install"],
-    ["Gemfile.lock", "bundle install"],
+    ['pnpm-lock.yaml', 'pnpm install --frozen-lockfile'],
+    ['package-lock.json', 'npm ci'],
+    ['.yarnrc.yml', 'yarn install --immutable'],
+    ['yarn.lock', 'yarn install --frozen-lockfile'],
+    ['bun.lock', 'bun install --frozen-lockfile'],
+    ['bun.lockb', 'bun install --frozen-lockfile'],
+    ['Cargo.lock', 'cargo fetch'],
+    ['go.sum', 'go mod download'],
+    ['uv.lock', 'uv sync'],
+    ['poetry.lock', 'poetry install'],
+    ['Gemfile.lock', 'bundle install'],
 ];
 export async function detectSetupCommand(root) {
     for (const [file, command] of MARKERS) {

package/dist/state.mjs

@@ -1,6 +1,6 @@
-import { mkdir, readdir, readFile, rm, writeFile } from "node:fs/promises";
-import { join } from "node:path";
-export const STATE_DIR = ".next-issue";
+import { mkdir, readdir, readFile, rm, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
+export const STATE_DIR = '.next-issue';
 function dir(repo) {
     return join(repo.root, STATE_DIR);
 }
@@ -11,10 +11,10 @@ export async function readState(repo, issue) {
     const path = file(repo, issue);
     let text;
     try {
-        text = await readFile(path, "utf8");
+        text = await readFile(path, 'utf8');
     }
     catch (error) {
-        if (error.code === "ENOENT") {
+        if (error.code === 'ENOENT') {
             return undefined;
         }
         throw error;
@@ -31,34 +31,34 @@ export async function readState(repo, issue) {
     }
     return parsed;
 }
-const PHASES = ["claimed", "implemented", "review"];
+const PHASES = ['claimed', 'implemented', 'review'];
 function isRecord(value) {
-    return typeof value === "object" && value !== null;
+    return typeof value === 'object' && value !== null;
 }
 function isOptional(value, type) {
     return value === undefined || typeof value === type;
 }
 function isBaseFix(value) {
-    return isRecord(value) && typeof value.sha === "string" && typeof value.head === "string";
+    return isRecord(value) && typeof value.sha === 'string' && typeof value.head === 'string';
 }
 function isReviewRound(value) {
     return (isRecord(value) &&
-        typeof value.round === "number" &&
-        typeof value.fingerprint === "string" &&
-        typeof value.findings === "string" &&
-        isOptional(value.dispute, "string"));
+        typeof value.round === 'number' &&
+        typeof value.fingerprint === 'string' &&
+        typeof value.findings === 'string' &&
+        isOptional(value.dispute, 'string'));
 }
 function isState(value) {
     return (isRecord(value) &&
-        typeof value.issue === "number" &&
+        typeof value.issue === 'number' &&
         PHASES.includes(value.phase) &&
-        typeof value.branch === "string" &&
-        isOptional(value.pr, "number") &&
-        isOptional(value.title, "string") &&
-        typeof value.ciFixes === "number" &&
-        isOptional(value.commitFixes, "number") &&
+        typeof value.branch === 'string' &&
+        isOptional(value.pr, 'number') &&
+        isOptional(value.title, 'string') &&
+        typeof value.ciFixes === 'number' &&
+        isOptional(value.commitFixes, 'number') &&
         (value.baseFix === undefined || isBaseFix(value.baseFix)) &&
-        typeof value.reviewRounds === "number" &&
+        typeof value.reviewRounds === 'number' &&
         Array.isArray(value.reviewLog) &&
         value.reviewLog.every(isReviewRound));
 }
@@ -68,7 +68,7 @@ export async function savedIssues(repo) {
         names = await readdir(dir(repo));
     }
     catch (error) {
-        if (error.code === "ENOENT") {
+        if (error.code === 'ENOENT') {
             return [];
         }
         throw error;

package/dist/summary.mjs

@@ -1,16 +1,16 @@
-const OUTCOMES = ["done", "needs-human", "error", "skipped", "stopped"];
+const OUTCOMES = ['done', 'needs-human', 'error', 'skipped', 'stopped'];
 export function formatSummary(summary) {
-    const tally = OUTCOMES.map((outcome) => `${summary.issues.filter((report) => report.outcome === outcome).length} ${outcome}`).join(", ");
+    const tally = OUTCOMES.map((outcome) => `${summary.issues.filter((report) => report.outcome === outcome).length} ${outcome}`).join(', ');
     const lines = [`run ${summary.runId}`, `issues ${summary.issues.length}: ${tally}`];
     for (const report of summary.issues) {
         lines.push(`  ${issueLine(report)}`);
     }
     const roles = Object.entries(summary.roles)
         .map(([role, totals]) => `${role} ${tokens(totals.total)} in ${totals.calls}`)
-        .join(", ");
-    lines.push(`tokens ${tokens(summary.total.total)}${roles.length > 0 ? ` (${roles})` : ""}`);
+        .join(', ');
+    lines.push(`tokens ${tokens(summary.total.total)}${roles.length > 0 ? ` (${roles})` : ''}`);
     lines.push(`log ${summary.logFile}`);
-    return `${lines.join("\n")}\n`;
+    return `${lines.join('\n')}\n`;
 }
 function issueLine(report) {
     const parts = [`#${report.issue} ${report.outcome.padEnd(11)}`];
@@ -29,12 +29,12 @@ function issueLine(report) {
     if (report.reason !== undefined) {
         parts.push(`(${report.reason})`);
     }
-    return parts.join("  ").trimEnd();
+    return parts.join('  ').trimEnd();
 }
 function duration(ms) {
     const seconds = Math.round(ms / 1000);
     const minutes = Math.floor(seconds / 60);
-    return minutes === 0 ? `${seconds}s` : `${minutes}m${String(seconds % 60).padStart(2, "0")}s`;
+    return minutes === 0 ? `${seconds}s` : `${minutes}m${String(seconds % 60).padStart(2, '0')}s`;
 }
 function tokens(value) {
     if (value < 1000) {