next-issue 2.4.1 → 2.4.3

package/README.md

@@ -98,32 +98,32 @@ when:
 - the check budget or the review budget runs out;
 - a fixer round adds no commit and gives no reason, so there is no progress;
 - the fixer reports that the checks fail for a reason the change did not cause,
-   such as a test already broken on the base branch or one that fails only under
-   load. It changes nothing and the reason reaches the pull request, rather than
-   the fixer spending its whole budget on a failure it cannot fix;
+  such as a test already broken on the base branch or one that fails only under
+  load. It changes nothing and the reason reaches the pull request, rather than
+  the fixer spending its whole budget on a failure it cannot fix;
 - a fixer disputes the findings and leaves a change behind, either in the
-   worktree or in a commit of its own. A dispute carries a reason, not a diff,
-   so the harness throws none of it away: an unclean worktree stays on the
-   disk, and a commit of the fixer stays on the local branch. Neither is
-   pushed, so the pull request shows no such change. It holds the reason of the
-   dispute, and the change is on the machine that ran the harness;
+  worktree or in a commit of its own. A dispute carries a reason, not a diff,
+  so the harness throws none of it away: an unclean worktree stays on the
+  disk, and a commit of the fixer stays on the local branch. Neither is
+  pushed, so the pull request shows no such change. It holds the reason of the
+  dispute, and the change is on the machine that ran the harness;
 - the reviewer repeats a finding set from an earlier round, which shows a
-   ping-pong between the fixer and the reviewer, or a finding that the fixer
-   disputed and the reviewer holds to. The person then decides;
+  ping-pong between the fixer and the reviewer, or a finding that the fixer
+  disputed and the reviewer holds to. The person then decides;
 - the reviewer gives no verdict, or the checks do not finish in time;
 - with `--merge`, the merge of an approved pull request fails, for example
-   because branch protection asks for a human approval;
+  because branch protection asks for a human approval;
 - with `--merge`, a queued merge does not finish in `checkTimeoutMinutes`, or
-   the pull request closes without a merge;
+  the pull request closes without a merge;
 - with `--merge`, the checks on the base branch fail after the merge and the
-   fixer cannot fix them, or they do not finish in time. The base branch is
-   then broken, so the run stops after this issue;
+  fixer cannot fix them, or they do not finish in time. The base branch is
+  then broken, so the run stops after this issue;
 - an agent does not finish in `agentTimeoutMinutes`;
 - the setup command fails, or does not finish in `setupTimeoutMinutes`;
 - a step of the run fails with an error. A `gh` call that is safe to repeat,
-   such as a read or a label edit, is tried up to three times first when the
-   network fails. A `gh pr create` or `gh pr comment` is tried once, so a lost
-   reply never makes a duplicate.
+  such as a read or a label edit, is tried up to three times first when the
+  network fails. A `gh pr create` or `gh pr comment` is tried once, so a lost
+  reply never makes a duplicate.
 
 The fixer sees the findings of all earlier rounds, not only the last one, with
 its own disputes next to them. From round two, the reviewer judges only the
@@ -213,11 +213,11 @@ jq -r 'select(.kind=="usage") | [.issue,.role,.turns,.total] | @tsv' \
 
 Three levels of console output:
 
-| Level | Shows |
-| --- | --- |
-| `--quiet` | Run and issue milestones, verdicts, hand-overs, errors |
-| default | The above plus each step end, each tool call and failed commands |
-| `--verbose` | The above plus every command and all agent text |
+| Level       | Shows                                                            |
+| ----------- | ---------------------------------------------------------------- |
+| `--quiet`   | Run and issue milestones, verdicts, hand-overs, errors           |
+| default     | The above plus each step end, each tool call and failed commands |
+| `--verbose` | The above plus every command and all agent text                  |
 
 ## Requirements
 
@@ -281,26 +281,26 @@ does not know all stop the run, so a typo cannot pass without a word.
 }
 ```
 
-| Field | Default | Effect |
-| --- | --- | --- |
-| `remote` | `origin` | The git remote for the repository and the branches |
-| `issueLimit` | `100` | The maximum number of open issues to read. With `labels.ready` set, only issues with the ready, in-progress or in-review label count |
-| `maxCiFixes` | `5` | The budget for check fixes per issue |
-| `maxCommitFixes` | `10` | The budget for fixes of a commit that a git hook stopped, per issue |
-| `maxReviewRounds` | `5` | The budget for review rounds per issue |
-| `checkIntervalSeconds` | `15` | The time between two check states |
-| `checkGraceSeconds` | `60` | The time to wait for the first check to show |
-| `checkTimeoutMinutes` | `60` | The limit for one check wait |
-| `agentTimeoutMinutes` | `30` | The limit for one agent call |
-| `setupTimeoutMinutes` | `15` | The limit for the setup command |
-| `commandTimeoutMinutes` | `10` | The limit for one `git` or `gh` command |
-| `logMaxChars` | `20000` | The maximum length of the failed job logs, shared between the failed jobs |
-| `diffMaxChars` | `60000` | The maximum length of the diff that the reviewer reads |
-| `keepRunDays` | `7` | The number of days to keep the run files; `0` keeps them all |
-| `draftPullRequest` | `true` | Open the pull request as a draft, until the review approves |
-| `setupCommand` | from the lock file | A shell command to run in a new worktree, before the implementer |
-| `models` | `null` per role | The pi model per agent role, as `provider/model`: `implementer`, `reviewer`, `fixer` |
-| `labels` | see above | The names of the labels that the harness reads and sets |
+| Field                   | Default            | Effect                                                                                                                               |
+| ----------------------- | ------------------ | ------------------------------------------------------------------------------------------------------------------------------------ |
+| `remote`                | `origin`           | The git remote for the repository and the branches                                                                                   |
+| `issueLimit`            | `100`              | The maximum number of open issues to read. With `labels.ready` set, only issues with the ready, in-progress or in-review label count |
+| `maxCiFixes`            | `5`                | The budget for check fixes per issue                                                                                                 |
+| `maxCommitFixes`        | `10`               | The budget for fixes of a commit that a git hook stopped, per issue                                                                  |
+| `maxReviewRounds`       | `5`                | The budget for review rounds per issue                                                                                               |
+| `checkIntervalSeconds`  | `15`               | The time between two check states                                                                                                    |
+| `checkGraceSeconds`     | `60`               | The time to wait for the first check to show                                                                                         |
+| `checkTimeoutMinutes`   | `60`               | The limit for one check wait                                                                                                         |
+| `agentTimeoutMinutes`   | `30`               | The limit for one agent call                                                                                                         |
+| `setupTimeoutMinutes`   | `15`               | The limit for the setup command                                                                                                      |
+| `commandTimeoutMinutes` | `10`               | The limit for one `git` or `gh` command                                                                                              |
+| `logMaxChars`           | `20000`            | The maximum length of the failed job logs, shared between the failed jobs                                                            |
+| `diffMaxChars`          | `60000`            | The maximum length of the diff that the reviewer reads                                                                               |
+| `keepRunDays`           | `7`                | The number of days to keep the run files; `0` keeps them all                                                                         |
+| `draftPullRequest`      | `true`             | Open the pull request as a draft, until the review approves                                                                          |
+| `setupCommand`          | from the lock file | A shell command to run in a new worktree, before the implementer                                                                     |
+| `models`                | `null` per role    | The pi model per agent role, as `provider/model`: `implementer`, `reviewer`, `fixer`                                                 |
+| `labels`                | see above          | The names of the labels that the harness reads and sets                                                                              |
 
 An empty `labels.ready` turns the ready requirement off. The harness then
 claims every open issue that no other label holds back.
@@ -314,17 +314,17 @@ your pi settings.
 Without a `setupCommand`, the harness looks in the new worktree for a lock file
 and runs the install command that goes with it:
 
-| Lock file | Command |
-| --- | --- |
-| `pnpm-lock.yaml` | `pnpm install --frozen-lockfile` |
-| `package-lock.json` | `npm ci` |
-| `yarn.lock` | `yarn install --frozen-lockfile`, or `yarn install --immutable` with a `.yarnrc.yml` |
-| `bun.lock`, `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` |
+| Lock file               | Command                                                                              |
+| ----------------------- | ------------------------------------------------------------------------------------ |
+| `pnpm-lock.yaml`        | `pnpm install --frozen-lockfile`                                                     |
+| `package-lock.json`     | `npm ci`                                                                             |
+| `yarn.lock`             | `yarn install --frozen-lockfile`, or `yarn install --immutable` with a `.yarnrc.yml` |
+| `bun.lock`, `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`                                                                     |
 
 The first match in that order wins. A project with no lock file in the list
 runs no setup, and a `setupCommand` in the config file always wins. Thus the
@@ -353,21 +353,21 @@ that a label still holds back stays as it was.
 
 ## Commands and options
 
-| Argument | Effect |
-| --- | --- |
-| `init` | Write the config file with the defaults |
-| `serve` | Run the worker that pi extensions talk to, see [Worker](#worker) |
-| `stop` | Stop the worker, see [Worker](#worker) |
-| `--force` | Replace an existing config file, with `init` |
-| `--once` | Stop after the first handled issue |
-| `--max <n>` | Handle at most n issues |
-| `--merge` | Squash-merge the pull request after the approval, delete the remote branch and fix a failure of the checks on the base branch |
-| `--watch` | Keep running, and read the open issues again after each pass |
-| `--reset` | Drop the saved budgets and findings, and keep the rest of the state |
-| `--json` | Print the machine summary instead of the text one |
-| `--verbose` | Show every command and all agent output |
-| `--quiet` | Show only the milestones and the summary |
-| `--help` | Show the option list |
+| Argument    | Effect                                                                                                                        |
+| ----------- | ----------------------------------------------------------------------------------------------------------------------------- |
+| `init`      | Write the config file with the defaults                                                                                       |
+| `serve`     | Run the worker that pi extensions talk to, see [Worker](#worker)                                                              |
+| `stop`      | Stop the worker, see [Worker](#worker)                                                                                        |
+| `--force`   | Replace an existing config file, with `init`                                                                                  |
+| `--once`    | Stop after the first handled issue                                                                                            |
+| `--max <n>` | Handle at most n issues                                                                                                       |
+| `--merge`   | Squash-merge the pull request after the approval, delete the remote branch and fix a failure of the checks on the base branch |
+| `--watch`   | Keep running, and read the open issues again after each pass                                                                  |
+| `--reset`   | Drop the saved budgets and findings, and keep the rest of the state                                                           |
+| `--json`    | Print the machine summary instead of the text one                                                                             |
+| `--verbose` | Show every command and all agent output                                                                                       |
+| `--quiet`   | Show only the milestones and the summary                                                                                      |
+| `--help`    | Show the option list                                                                                                          |
 
 The exit code is 1 when an issue needs a person or failed, and 0 when it does
 not, so a cron job or a CI step can read the result of the run.
@@ -378,7 +378,8 @@ A run without `--watch` ends when no unseen issue is left. With `--watch`, the
 harness then waits, reads the open issues again and runs another pass, so it
 picks up an issue that someone opens during the run, or one whose blocker
 merges. Each pass tries every open issue again, also one that an earlier pass
-skipped.
+skipped. An issue that failed in an earlier pass is not tried again until the
+next run.
 
 The waits follow a fixed back-off in minutes: 1, 2, 3, 5, 10, then 10 from then
 on. A pass that handles an issue, with any outcome other than `skipped`, resets

package/dist/agents.mjs

@@ -1,36 +1,36 @@
-import { join, sep } from "node:path";
-import { createAgentSession, DefaultResourceLoader, defineTool, getAgentDir, ModelRuntime, SessionManager, } from "@earendil-works/pi-coding-agent";
-import { onStop } from "./exec.mjs";
-import { emptyUsage } from "./observe.mjs";
+import { join, sep } from 'node:path';
+import { createAgentSession, DefaultResourceLoader, defineTool, getAgentDir, ModelRuntime, SessionManager, } from '@earendil-works/pi-coding-agent';
+import { onStop } from './exec.mjs';
+import { emptyUsage } from './observe.mjs';
 const REVIEWING = {
-    tools: ["read", "bash", "grep", "find", "ls"],
+    tools: ['read', 'bash', 'grep', 'find', 'ls'],
 };
 const CODING = {
-    tools: ["read", "bash", "edit", "write", "grep", "find", "ls"],
+    tools: ['read', 'bash', 'edit', 'write', 'grep', 'find', 'ls'],
 };
 export { CODING, REVIEWING };
-export const SUBMIT_TOOL = "submit_result";
+export const SUBMIT_TOOL = 'submit_result';
 export function isOwnExtension(path) {
-    return path.split(sep).includes("pi-next-issue");
+    return path.split(sep).includes('pi-next-issue');
 }
 export async function runAgent(recorder, request) {
     const log = recorder.scope({ role: request.name });
     const runtime = request.runtime ?? {};
     const agentDir = runtime.agentDir ?? getAgentDir();
     const started = Date.now();
-    log.event("agent.start", { model: request.model, timeoutMs: request.timeoutMs }, "normal");
+    log.event('agent.start', { model: request.model, timeoutMs: request.timeoutMs }, 'normal');
     let structured;
     const customTools = [];
     if (request.outputSchema !== undefined) {
         customTools.push(defineTool({
             name: SUBMIT_TOOL,
-            label: "Submit result",
-            description: "Give the result. Call it once, as your last action.",
+            label: 'Submit result',
+            description: 'Give the result. Call it once, as your last action.',
             parameters: request.outputSchema,
             async execute(_toolCallId, params) {
                 structured = params;
                 return {
-                    content: [{ type: "text", text: "Result saved." }],
+                    content: [{ type: 'text', text: 'Result saved.' }],
                     details: {},
                     terminate: true,
                 };
@@ -46,10 +46,11 @@ export async function runAgent(recorder, request) {
         }),
     });
     await resourceLoader.reload();
-    const modelRuntime = runtime.modelRuntime ?? (await ModelRuntime.create({
-        authPath: join(agentDir, "auth.json"),
-        modelsPath: join(agentDir, "models.json"),
-    }));
+    const modelRuntime = runtime.modelRuntime ??
+        (await ModelRuntime.create({
+            authPath: join(agentDir, 'auth.json'),
+            modelsPath: join(agentDir, 'models.json'),
+        }));
     const model = request.model === undefined ? undefined : await findModel(modelRuntime, request.model);
     const { session, modelFallbackMessage } = await createAgentSession({
         cwd: request.cwd,
@@ -63,7 +64,7 @@ export async function runAgent(recorder, request) {
         ...(model === undefined ? {} : { model }),
     });
     if (modelFallbackMessage !== undefined) {
-        log.event("agent.model", { note: modelFallbackMessage }, "quiet");
+        log.event('agent.model', { note: modelFallbackMessage }, 'quiet');
     }
     let timedOut = false;
     const timer = request.timeoutMs === undefined
@@ -80,9 +81,9 @@ export async function runAgent(recorder, request) {
         if (timedOut) {
             throw new Error(`The ${request.name} agent did not finish in ${request.timeoutMs} ms.`);
         }
-        const last = session.messages.findLast((message) => message.role === "assistant");
-        if (last !== undefined && "stopReason" in last && last.stopReason === "error") {
-            throw new Error(`The ${request.name} agent failed: ${last.errorMessage ?? "no reason given"}`);
+        const last = session.messages.findLast((message) => message.role === 'assistant');
+        if (last !== undefined && 'stopReason' in last && last.stopReason === 'error') {
+            throw new Error(`The ${request.name} agent failed: ${last.errorMessage ?? 'no reason given'}`);
         }
         const stats = session.getSessionStats();
         const usage = { ...emptyUsage(), ...stats.tokens };
@@ -96,7 +97,7 @@ export async function runAgent(recorder, request) {
             costUsd: stats.cost,
         });
         return {
-            text: (session.getLastAssistantText() ?? "").trim(),
+            text: (session.getLastAssistantText() ?? '').trim(),
             usage,
             turns: stats.assistantMessages,
             toolCalls: stats.toolCalls,
@@ -115,7 +116,7 @@ export async function runAgent(recorder, request) {
     }
 }
 async function findModel(modelRuntime, name) {
-    const slash = name.indexOf("/");
+    const slash = name.indexOf('/');
     if (slash <= 0) {
         throw new Error(`The model ${name} is not in the form provider/model.`);
     }
@@ -126,22 +127,22 @@ async function findModel(modelRuntime, name) {
     return found;
 }
 function observe(log, event) {
-    if (event.type === "tool_execution_start") {
-        log.event("tool", toolFields(event.toolName, event.args));
+    if (event.type === 'tool_execution_start') {
+        log.event('tool', toolFields(event.toolName, event.args));
     }
-    else if (event.type === "tool_execution_end" && event.isError) {
-        log.event("tool.error", { tool: event.toolName }, "quiet");
+    else if (event.type === 'tool_execution_end' && event.isError) {
+        log.event('tool.error', { tool: event.toolName }, 'quiet');
     }
-    else if (event.type === "message_update" &&
-        event.assistantMessageEvent.type === "text_delta" &&
-        log.level === "verbose") {
+    else if (event.type === 'message_update' &&
+        event.assistantMessageEvent.type === 'text_delta' &&
+        log.level === 'verbose') {
         process.stderr.write(event.assistantMessageEvent.delta);
     }
 }
 export function toolFields(name, input) {
-    if (name !== "bash" || typeof input !== "object" || input === null) {
+    if (name !== 'bash' || typeof input !== 'object' || input === null) {
         return { tool: name };
     }
     const { command } = input;
-    return typeof command === "string" ? { tool: name, command } : { tool: name };
+    return typeof command === 'string' ? { tool: name, command } : { tool: name };
 }

package/dist/args.mjs

@@ -1,4 +1,4 @@
-import { CONFIG_FILE } from "./config.mjs";
+import { CONFIG_FILE } from './config.mjs';
 export const USAGE = `next-issue [command] [options]
 
   init         write ${CONFIG_FILE} with the defaults
@@ -18,7 +18,7 @@ export const USAGE = `next-issue [command] [options]
 `;
 export function parseArgs(argv) {
     const args = {
-        command: "run",
+        command: 'run',
         once: false,
         help: false,
         json: false,
@@ -26,44 +26,44 @@ export function parseArgs(argv) {
         force: false,
         merge: false,
         watch: false,
-        level: "normal",
+        level: 'normal',
     };
     for (let index = 0; index < argv.length; index += 1) {
         const arg = argv[index];
-        if (index === 0 && !arg.startsWith("-")) {
-            if (arg !== "init" && arg !== "serve" && arg !== "stop") {
+        if (index === 0 && !arg.startsWith('-')) {
+            if (arg !== 'init' && arg !== 'serve' && arg !== 'stop') {
                 throw new Error(`Unknown command: ${arg}`);
             }
             args.command = arg;
         }
-        else if (arg === "--force") {
+        else if (arg === '--force') {
             args.force = true;
         }
-        else if (arg === "--json") {
+        else if (arg === '--json') {
             args.json = true;
         }
-        else if (arg === "--help") {
+        else if (arg === '--help') {
             args.help = true;
         }
-        else if (arg === "--merge") {
+        else if (arg === '--merge') {
             args.merge = true;
         }
-        else if (arg === "--watch") {
+        else if (arg === '--watch') {
             args.watch = true;
         }
-        else if (arg === "--reset") {
+        else if (arg === '--reset') {
             args.reset = true;
         }
-        else if (arg === "--once") {
+        else if (arg === '--once') {
             args.once = true;
         }
-        else if (arg === "--verbose") {
-            args.level = "verbose";
+        else if (arg === '--verbose') {
+            args.level = 'verbose';
         }
-        else if (arg === "--quiet") {
-            args.level = "quiet";
+        else if (arg === '--quiet') {
+            args.level = 'quiet';
         }
-        else if (arg === "--max") {
+        else if (arg === '--max') {
             args.max = whole(argv[++index], arg);
         }
         else {

package/dist/cli.mjs

@@ -1,16 +1,16 @@
 #!/usr/bin/env node
-import { parseArgs, USAGE } from "./args.mjs";
-import { writeDefaultConfig } from "./config.mjs";
-import { ensureIgnored, repoRoot, WORKTREE_DIR } from "./git.mjs";
-import { listenForInterrupts } from "./interrupt.mjs";
-import { message } from "./observe.mjs";
-import { runRepo } from "./run.mjs";
-import { serve, socketPaths, stopWorker } from "./serve.mjs";
-import { STATE_DIR } from "./state.mjs";
-import { formatSummary } from "./summary.mjs";
+import { parseArgs, USAGE } from './args.mjs';
+import { writeDefaultConfig } from './config.mjs';
+import { ensureIgnored, repoRoot, WORKTREE_DIR } from './git.mjs';
+import { listenForInterrupts } from './interrupt.mjs';
+import { message } from './observe.mjs';
+import { runRepo } from './run.mjs';
+import { serve, socketPaths, stopWorker } from './serve.mjs';
+import { STATE_DIR } from './state.mjs';
+import { formatSummary } from './summary.mjs';
 function write(stream, text) {
     return new Promise((resolve, reject) => {
-        stream.write(text, (error) => (error === null || error === undefined ? resolve() : reject(error)));
+        stream.write(text, (error) => error === null || error === undefined ? resolve() : reject(error));
     });
 }
 async function main() {
@@ -19,16 +19,18 @@ async function main() {
         await write(process.stdout, USAGE);
         return 0;
     }
-    if (args.command === "serve") {
+    if (args.command === 'serve') {
         return serve();
     }
-    if (args.command === "stop") {
+    if (args.command === 'stop') {
         const pid = await stopWorker(socketPaths().lockPath);
-        await write(process.stdout, pid === undefined ? "next-issue: no worker is running\n" : `next-issue: stopped the worker with process id ${pid}\n`);
+        await write(process.stdout, pid === undefined
+            ? 'next-issue: no worker is running\n'
+            : `next-issue: stopped the worker with process id ${pid}\n`);
         return 0;
     }
     const root = await repoRoot(process.cwd());
-    if (args.command === "init") {
+    if (args.command === 'init') {
         return init(root, args.force);
     }
     const interrupts = listenForInterrupts((text) => void write(process.stderr, `${text}\n`));

package/dist/config.mjs

@@ -1,8 +1,8 @@
-import { readFile, writeFile } from "node:fs/promises";
-import { join } from "node:path";
-import { message } from "./observe.mjs";
+import { readFile, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
+import { message } from './observe.mjs';
 const DEFAULTS = {
-    remote: "origin",
+    remote: 'origin',
     issueLimit: 100,
     maxCiFixes: 5,
     maxCommitFixes: 10,
@@ -19,49 +19,49 @@ const DEFAULTS = {
     draftPullRequest: true,
     models: {},
     labels: {
-        ready: "status:todo",
-        inProgress: "status:in-progress",
-        inReview: "status:in-review",
-        done: "status:done",
-        needsHuman: "status:needs-human",
-        skip: "status:blocked",
+        ready: 'status:todo',
+        inProgress: 'status:in-progress',
+        inReview: 'status:in-review',
+        done: 'status:done',
+        needsHuman: 'status:needs-human',
+        skip: 'status:blocked',
     },
 };
 function defaults() {
     return { ...DEFAULTS, models: { ...DEFAULTS.models }, labels: { ...DEFAULTS.labels } };
 }
-const FIELDS = [...Object.keys(DEFAULTS), "setupCommand"];
+const FIELDS = [...Object.keys(DEFAULTS), 'setupCommand'];
 const WHOLE_FIELDS = [
-    ["issueLimit", 1],
-    ["maxCiFixes", 0],
-    ["maxCommitFixes", 0],
-    ["maxReviewRounds", 1],
-    ["checkIntervalSeconds", 1],
-    ["checkGraceSeconds", 0],
-    ["checkTimeoutMinutes", 1],
-    ["agentTimeoutMinutes", 1],
-    ["setupTimeoutMinutes", 1],
-    ["commandTimeoutMinutes", 1],
-    ["logMaxChars", 1],
-    ["diffMaxChars", 1],
-    ["keepRunDays", 0],
+    ['issueLimit', 1],
+    ['maxCiFixes', 0],
+    ['maxCommitFixes', 0],
+    ['maxReviewRounds', 1],
+    ['checkIntervalSeconds', 1],
+    ['checkGraceSeconds', 0],
+    ['checkTimeoutMinutes', 1],
+    ['agentTimeoutMinutes', 1],
+    ['setupTimeoutMinutes', 1],
+    ['commandTimeoutMinutes', 1],
+    ['logMaxChars', 1],
+    ['diffMaxChars', 1],
+    ['keepRunDays', 0],
 ];
-const MODEL_ROLES = ["implementer", "reviewer", "fixer"];
-const LABEL_NAMES = ["ready", "inProgress", "inReview", "done", "needsHuman", "skip"];
+const MODEL_ROLES = ['implementer', 'reviewer', 'fixer'];
+const LABEL_NAMES = ['ready', 'inProgress', 'inReview', 'done', 'needsHuman', 'skip'];
 const FILE_DEFAULTS = {
     ...defaults(),
     models: Object.fromEntries(MODEL_ROLES.map((role) => [role, null])),
 };
-export const CONFIG_FILE = "next-issue.config.json";
+export const CONFIG_FILE = 'next-issue.config.json';
 export async function writeDefaultConfig(root, force) {
     const path = join(root, CONFIG_FILE);
     try {
         await writeFile(path, `${JSON.stringify(FILE_DEFAULTS, null, 2)}\n`, {
-            flag: force ? "w" : "wx",
+            flag: force ? 'w' : 'wx',
         });
     }
     catch (error) {
-        if (error.code === "EEXIST") {
+        if (error.code === 'EEXIST') {
             throw new Error(`${CONFIG_FILE} exists already. Use --force to replace it.`);
         }
         throw error;
@@ -71,10 +71,10 @@ export async function writeDefaultConfig(root, force) {
 export async function loadConfig(root) {
     let raw;
     try {
-        raw = await readFile(join(root, CONFIG_FILE), "utf8");
+        raw = await readFile(join(root, CONFIG_FILE), 'utf8');
     }
     catch (error) {
-        if (error.code === "ENOENT") {
+        if (error.code === 'ENOENT') {
             return defaults();
         }
         throw new Error(`${CONFIG_FILE} cannot be read: ${message(error)}`);
@@ -89,7 +89,7 @@ export async function loadConfig(root) {
     return parseConfig(parsed);
 }
 function parseConfig(value) {
-    const file = record(value, "the file");
+    const file = record(value, 'the file');
     for (const key of Object.keys(file)) {
         if (!FIELDS.includes(key)) {
             throw new Error(`${CONFIG_FILE} holds an unknown field: ${key}`);
@@ -97,7 +97,7 @@ function parseConfig(value) {
     }
     const config = defaults();
     if (file.remote !== undefined) {
-        config.remote = text(file.remote, "remote", false);
+        config.remote = text(file.remote, 'remote', false);
     }
     for (const [field, least] of WHOLE_FIELDS) {
         if (file[field] !== undefined) {
@@ -105,16 +105,16 @@ function parseConfig(value) {
         }
     }
     if (file.draftPullRequest !== undefined) {
-        if (typeof file.draftPullRequest !== "boolean") {
+        if (typeof file.draftPullRequest !== 'boolean') {
             throw new Error(`${CONFIG_FILE}: draftPullRequest must be true or false.`);
         }
         config.draftPullRequest = file.draftPullRequest;
     }
     if (file.setupCommand !== undefined) {
-        config.setupCommand = text(file.setupCommand, "setupCommand", false);
+        config.setupCommand = text(file.setupCommand, 'setupCommand', false);
     }
     if (file.models !== undefined) {
-        const models = record(file.models, "models");
+        const models = record(file.models, 'models');
         for (const key of Object.keys(models)) {
             if (!MODEL_ROLES.includes(key)) {
                 throw new Error(`${CONFIG_FILE} holds an unknown model role: ${key}`);
@@ -127,7 +127,7 @@ function parseConfig(value) {
         }
     }
     if (file.labels !== undefined) {
-        const labels = record(file.labels, "labels");
+        const labels = record(file.labels, 'labels');
         for (const key of Object.keys(labels)) {
             if (!LABEL_NAMES.includes(key)) {
                 throw new Error(`${CONFIG_FILE} holds an unknown label name: ${key}`);
@@ -135,27 +135,27 @@ function parseConfig(value) {
         }
         for (const name of LABEL_NAMES) {
             if (labels[name] !== undefined) {
-                config.labels[name] = text(labels[name], `labels.${name}`, name === "ready");
+                config.labels[name] = text(labels[name], `labels.${name}`, name === 'ready');
             }
         }
     }
     return config;
 }
 function record(value, what) {
-    if (typeof value !== "object" || value === null || Array.isArray(value)) {
+    if (typeof value !== 'object' || value === null || Array.isArray(value)) {
         throw new Error(`${CONFIG_FILE}: ${what} must be an object.`);
     }
     return value;
 }
 function whole(value, field, least) {
-    if (typeof value !== "number" || !Number.isInteger(value) || value < least) {
+    if (typeof value !== 'number' || !Number.isInteger(value) || value < least) {
         throw new Error(`${CONFIG_FILE}: ${field} must be a whole number of ${least} or more.`);
     }
     return value;
 }
 function text(value, field, allowEmpty) {
-    if (typeof value !== "string" || (!allowEmpty && value.length === 0)) {
-        throw new Error(`${CONFIG_FILE}: ${field} must be ${allowEmpty ? "text" : "text that is not empty"}.`);
+    if (typeof value !== 'string' || (!allowEmpty && value.length === 0)) {
+        throw new Error(`${CONFIG_FILE}: ${field} must be ${allowEmpty ? 'text' : 'text that is not empty'}.`);
     }
     return value;
 }