next-issue 0.5.1 → 0.6.0

package/README.md

@@ -56,6 +56,17 @@ when:
 The fixer sees the findings of all earlier rounds, not only the last one. From
 round two, the reviewer judges only the earlier findings and any regression.
 
+## Stop after the current issue
+
+Run `touch .next-issue/stop` in a second terminal. The harness reads the file
+only between two issues, finishes the issue that it holds, deletes the file and
+then writes the summary as usual. A file from an earlier run is dropped at the
+start, so it stops nothing.
+
+`Ctrl-C` is different: the terminal signals every child too, so the run dies in
+the middle of a step. The saved state lets the next run continue the issue, but
+the issue keeps `status:in-progress` and the run writes no summary.
+
 Every status label that the harness puts on the issue goes on the pull request
 too, from the first review round. Thus a draft pull request shows work that the
 harness has not finished, and a `status:needs-human` pull request shows work

package/dist/args.mjs

@@ -0,0 +1,73 @@
+import { CONFIG_FILE } from "./config.mjs";
+export const USAGE = `next-issue [command] [options]
+
+  init         write ${CONFIG_FILE} with the defaults
+
+  --issue <n>  handle one issue only
+  --once       stop after the first handled issue
+  --max <n>    handle at most n issues
+  --reset      drop the saved budgets and findings first
+  --json       print the machine summary instead of the text one
+  --force      replace an existing config file, with init
+  --verbose    show every command and all agent output
+  --quiet      show only the milestones and the summary
+  --help       show this text
+`;
+export function parseArgs(argv) {
+    const args = {
+        command: "run",
+        once: false,
+        help: false,
+        json: false,
+        reset: false,
+        force: false,
+        level: "normal",
+    };
+    for (let index = 0; index < argv.length; index += 1) {
+        const arg = argv[index];
+        if (index === 0 && !arg.startsWith("-")) {
+            if (arg !== "init") {
+                throw new Error(`Unknown command: ${arg}`);
+            }
+            args.command = "init";
+        }
+        else if (arg === "--force") {
+            args.force = true;
+        }
+        else if (arg === "--json") {
+            args.json = true;
+        }
+        else if (arg === "--help") {
+            args.help = true;
+        }
+        else if (arg === "--reset") {
+            args.reset = true;
+        }
+        else if (arg === "--once") {
+            args.once = true;
+        }
+        else if (arg === "--verbose") {
+            args.level = "verbose";
+        }
+        else if (arg === "--quiet") {
+            args.level = "quiet";
+        }
+        else if (arg === "--issue") {
+            args.issue = whole(argv[++index], arg);
+        }
+        else if (arg === "--max") {
+            args.max = whole(argv[++index], arg);
+        }
+        else {
+            throw new Error(`Unknown argument: ${arg}`);
+        }
+    }
+    return args;
+}
+function whole(value, option) {
+    const parsed = Number(value);
+    if (value === undefined || !Number.isInteger(parsed) || parsed < 1) {
+        throw new Error(`${option} needs a whole number of 1 or more`);
+    }
+    return parsed;
+}

package/dist/cli.mjs

@@ -1,86 +1,15 @@
 #!/usr/bin/env node
 import { writeFile } from "node:fs/promises";
 import { join } from "node:path";
-import { CONFIG_FILE, loadConfig, writeDefaultConfig } from "./config.mjs";
+import { parseArgs, USAGE } from "./args.mjs";
+import { loadConfig, writeDefaultConfig } from "./config.mjs";
 import { setCommandObserver } from "./exec.mjs";
 import { detectRepo, ensureIgnored, repoRoot } from "./git.mjs";
 import { currentLogin, defaultBranch, openIssues } from "./github.mjs";
 import { message, Recorder } from "./observe.mjs";
 import { processIssue } from "./pipeline.mjs";
-import { clearState, STATE_DIR } from "./state.mjs";
+import { clearState, STATE_DIR, takeStop } from "./state.mjs";
 import { formatSummary } from "./summary.mjs";
-const USAGE = `next-issue [command] [options]
-
-  init         write ${CONFIG_FILE} with the defaults
-
-  --issue <n>  handle one issue only
-  --once       stop after the first handled issue
-  --max <n>    handle at most n issues
-  --reset      drop the saved budgets and findings first
-  --json       print the machine summary instead of the text one
-  --force      replace an existing config file, with init
-  --verbose    show every command and all agent output
-  --quiet      show only the milestones and the summary
-  --help       show this text
-`;
-function parseArgs(argv) {
-    const args = {
-        command: "run",
-        once: false,
-        help: false,
-        json: false,
-        reset: false,
-        force: false,
-        level: "normal",
-    };
-    for (let index = 0; index < argv.length; index += 1) {
-        const arg = argv[index];
-        if (index === 0 && !arg.startsWith("-")) {
-            if (arg !== "init") {
-                throw new Error(`Unknown command: ${arg}`);
-            }
-            args.command = "init";
-        }
-        else if (arg === "--force") {
-            args.force = true;
-        }
-        else if (arg === "--json") {
-            args.json = true;
-        }
-        else if (arg === "--help") {
-            args.help = true;
-        }
-        else if (arg === "--reset") {
-            args.reset = true;
-        }
-        else if (arg === "--once") {
-            args.once = true;
-        }
-        else if (arg === "--verbose") {
-            args.level = "verbose";
-        }
-        else if (arg === "--quiet") {
-            args.level = "quiet";
-        }
-        else if (arg === "--issue") {
-            args.issue = whole(argv[++index], arg);
-        }
-        else if (arg === "--max") {
-            args.max = whole(argv[++index], arg);
-        }
-        else {
-            throw new Error(`Unknown argument: ${arg}`);
-        }
-    }
-    return args;
-}
-function whole(value, option) {
-    const parsed = Number(value);
-    if (value === undefined || !Number.isInteger(parsed) || parsed < 1) {
-        throw new Error(`${option} needs a whole number of 1 or more`);
-    }
-    return parsed;
-}
 function write(stream, text) {
     return new Promise((resolve, reject) => {
         stream.write(text, (error) => (error === null || error === undefined ? resolve() : reject(error)));
@@ -128,6 +57,7 @@ async function handle(args, config, repo, recorder) {
     recorder.event("run.start", { repo: `${repo.owner}/${repo.name}`, base, login, log: recorder.logFile }, "quiet");
     const issues = (await openIssues(repo, config.issueLimit)).filter((issue) => args.issue === undefined || issue.number === args.issue);
     recorder.event("issues", { open: issues.length }, "quiet");
+    await takeStop(repo);
     const reports = [];
     for (const issue of issues) {
         const handled = reports.filter((report) => report.outcome !== "skipped").length;
@@ -139,6 +69,10 @@ async function handle(args, config, repo, recorder) {
         }
         const report = await processIssue(context, issue);
         reports.push(report);
+        if (await takeStop(repo)) {
+            recorder.event("run.stop", { issue: issue.number }, "quiet");
+            break;
+        }
         if (args.once && report.outcome !== "skipped") {
             break;
         }

package/dist/pipeline.mjs

@@ -1,7 +1,7 @@
 import { CODING, READ_ONLY, runAgent } from "./agents.mjs";
 import { managedLabels } from "./config.mjs";
 import { run } from "./exec.mjs";
-import { addWorktree, branchName, commitAll, diff, push, removeWorktree, worktreePath, } from "./git.mjs";
+import { addWorktree, branchName, commitAll, diff, hasWorktree, push, removeWorktree, worktreePath, } from "./git.mjs";
 import { assignIssue, commentOnPr, createPr, failedCheckLogs, findPr, issueComments, markPrReady, setLabel, waitForChecks, } from "./github.mjs";
 import { message } from "./observe.mjs";
 import { fixPrompt, implementPrompt, parseCommitSubject, reviewPrompt } from "./prompts.mjs";
@@ -55,6 +55,7 @@ export async function processIssue(context, issue) {
         catch (failure) {
             log.event("hand-over.error", { error: message(failure) }, "quiet");
         }
+        await cleanUp(context, log, issue.number);
         return {
             issue: issue.number,
             outcome: "error",
@@ -79,15 +80,18 @@ async function work(context, issue, log, state) {
     const worktree = await log.step("worktree", { branch }, () => addWorktree(repo, issue.number, context.base, config.remote));
     const job = { context, issue, log, state, branch, worktree, comments };
     const escalate = (reason) => handOver(context, log, issue.number, state, reason);
-    const finish = (outcome, reason) => ({
-        issue: issue.number,
-        outcome,
-        reason,
-        pr: state.pr,
-        ciFixes: state.ciFixes,
-        reviewRounds: state.reviewRounds,
-        ms: Date.now() - started,
-    });
+    const finish = async (outcome, reason) => {
+        await cleanUp(context, log, issue.number);
+        return {
+            issue: issue.number,
+            outcome,
+            reason,
+            pr: state.pr,
+            ciFixes: state.ciFixes,
+            reviewRounds: state.reviewRounds,
+            ms: Date.now() - started,
+        };
+    };
     if (config.setupCommand !== undefined) {
         const setup = await log.step("setup", { cmd: config.setupCommand }, () => run("bash", ["-lc", config.setupCommand], { cwd: worktree }));
         if (setup.code !== 0) {
@@ -182,9 +186,6 @@ async function work(context, issue, log, state) {
             await setStatus(context, issue.number, state.pr, config.labels.done);
             state.phase = "done";
             await writeState(repo, state);
-            if (!(await removeWorktree(repo, issue.number))) {
-                log.event("worktree.kept", { path: worktreePath(repo, issue.number) }, "quiet");
-            }
             return finish("done");
         }
         const mark = fingerprint(blocking);
@@ -220,6 +221,20 @@ async function fixRound(job, reason, detail, earlier = []) {
     await push(worktree, context.config.remote, branch);
     return true;
 }
+async function cleanUp(context, log, issue) {
+    const path = worktreePath(context.repo, issue);
+    try {
+        if (!(await hasWorktree(context.repo, path))) {
+            return;
+        }
+        if (!(await removeWorktree(context.repo, issue))) {
+            log.event("worktree.kept", { path }, "quiet");
+        }
+    }
+    catch (error) {
+        log.event("worktree.error", { error: message(error) }, "quiet");
+    }
+}
 async function handOver(context, log, issue, state, reason) {
     log.event("hand-over", { reason }, "quiet");
     state.phase = "needs-human";

package/dist/state.mjs

@@ -22,3 +22,12 @@ export async function writeState(repo, state) {
     await mkdir(dir(repo), { recursive: true });
     await writeFile(file(repo, state.issue), `${JSON.stringify(state, null, 2)}\n`);
 }
+export async function takeStop(repo) {
+    try {
+        await rm(join(dir(repo), "stop"));
+        return true;
+    }
+    catch {
+        return false;
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "0.5.1",
+  "version": "0.6.0",
   "description": "Autonomous issue harness driven by the Claude Agent SDK",
   "keywords": [
     "github",