next-issue 2.3.1 → 2.4.0

package/README.md

@@ -357,6 +357,7 @@ that a label still holds back stays as it was.
 | --- | --- |
 | `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 |
@@ -423,7 +424,10 @@ to it. The worker:
 - stops every agent and command and exits on `SIGTERM` or `SIGINT`, and
   removes its socket and lock files.
 
-Stop it by hand with `kill $(cat ~/.pi/agent/next-issue.lock)`.
+Stop it with `next-issue stop`. This sends `SIGTERM` to the process in the
+lock file, so the worker stops at once and a run that is going resumes from
+its saved state next time. When no worker is running, the command says so and
+exits with 0.
 
 ## Development
 

package/dist/args.mjs

@@ -3,6 +3,7 @@ export const USAGE = `next-issue [command] [options]
 
   init         write ${CONFIG_FILE} with the defaults
   serve        run the worker that pi extensions talk to
+  stop         stop the worker
 
   --once       stop after the first handled issue
   --max <n>    handle at most n issues
@@ -30,7 +31,7 @@ export function parseArgs(argv) {
     for (let index = 0; index < argv.length; index += 1) {
         const arg = argv[index];
         if (index === 0 && !arg.startsWith("-")) {
-            if (arg !== "init" && arg !== "serve") {
+            if (arg !== "init" && arg !== "serve" && arg !== "stop") {
                 throw new Error(`Unknown command: ${arg}`);
             }
             args.command = arg;

package/dist/cli.mjs

@@ -5,7 +5,7 @@ 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 } from "./serve.mjs";
+import { serve, socketPaths, stopWorker } from "./serve.mjs";
 import { STATE_DIR } from "./state.mjs";
 import { formatSummary } from "./summary.mjs";
 function write(stream, text) {
@@ -22,6 +22,11 @@ async function main() {
     if (args.command === "serve") {
         return serve();
     }
+    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`);
+        return 0;
+    }
     const root = await repoRoot(process.cwd());
     if (args.command === "init") {
         return init(root, args.force);

package/dist/serve.mjs

@@ -36,6 +36,17 @@ function alive(pid) {
         return error.code === "EPERM";
     }
 }
+async function holder(lockPath) {
+    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");
+    }
+    return pid;
+}
 async function lock(path) {
     for (let attempt = 0; attempt < 2; attempt += 1) {
         try {
@@ -46,8 +57,7 @@ async function lock(path) {
             if (error.code !== "EEXIST") {
                 throw error;
             }
-            const pid = Number((await readFile(path, "utf8").catch(() => "")).trim());
-            if (pid > 0 && alive(pid)) {
+            if ((await holder(path)) !== undefined) {
                 return false;
             }
             await rm(path, { force: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "2.3.1",
+  "version": "2.4.0",
   "description": "Autonomous issue harness driven by the pi SDK",
   "keywords": [
     "github",