automata-cli 0.1.0-feature-030-do-work.192 → 0.1.0-feature-030-do-work.194

package/README.md

@@ -129,14 +129,7 @@ See [docs/execute-prompt.md](docs/execute-prompt.md) for full details.
 
 ## `automata do-work`
 
-Run one tick of the autonomous loop: find the open issues whose newest message from an authorized account the agent has not answered — on the issue or on its pull request — and answer them. Designed to run from cron in a disposable VM or container.
-
-```bash
-automata do-work              # one tick
-automata do-work --dry-run    # show the work plan, change nothing
-```
-
-See [docs/do-work.md](docs/do-work.md) for the command reference, and the [wiki](docs/wiki/Home.md) for the process: the trust model, the issue lifecycle, setup and operation.
+One tick of the autonomous loop: answer the open issues whose newest message from an authorized account the agent has not answered. Reference: [docs/do-work.md](docs/do-work.md). Process, trust model and setup: [the wiki](docs/wiki/Home.md).
 
 ---
 

package/dist/{ConfigWizard-SS7OOS5T.js → ConfigWizard-GN5JC4XP.js}

@@ -164,7 +164,7 @@ function ConfigWizard() {
       } else if (key.backspace || key.delete) {
         setDoWorkMaxRuns((v) => v.slice(0, -1));
       } else if (key.escape) {
-        setScreen("do-work-executor");
+        setScreen("do-work-codex-model");
       } else if (key.ctrl && input === "c") {
         exit();
       } else if (input && !key.ctrl && !key.meta) {

package/dist/index.js

@@ -183,7 +183,7 @@ var configCommand = new Command("config").description("Configure automata settin
   const [{ render }, React, { ConfigWizard }] = await Promise.all([
     import("ink"),
     import("react"),
-    import("./ConfigWizard-SS7OOS5T.js")
+    import("./ConfigWizard-GN5JC4XP.js")
   ]);
   const { waitUntilExit } = render(React.createElement(ConfigWizard));
   await waitUntilExit();
@@ -1532,6 +1532,31 @@ function buildClaudeArgs(prompt, options = {}) {
   args.push("-p", prompt);
   return args;
 }
+var activeChildren = /* @__PURE__ */ new Set();
+function terminateActiveClaudeProcesses(timeoutMs = 1e4) {
+  const children = [...activeChildren];
+  if (children.length === 0) return Promise.resolve();
+  const exits = children.map(
+    (child) => new Promise((resolve2) => {
+      if (child.exitCode !== null || child.signalCode !== null) {
+        resolve2();
+        return;
+      }
+      child.once("exit", () => resolve2());
+      child.kill("SIGTERM");
+    })
+  );
+  return Promise.race([
+    Promise.all(exits).then(() => void 0),
+    new Promise((resolve2) => {
+      const timer = setTimeout(() => {
+        for (const child of children) child.kill("SIGKILL");
+        resolve2();
+      }, timeoutMs);
+      timer.unref();
+    })
+  ]);
+}
 function invokeClaudeCode(prompt, options = {}) {
   if (options.verbose) {
     return invokeClaudeCodeVerbose(prompt, options.yolo ?? false, options.model);
@@ -1550,6 +1575,7 @@ function invokeClaudeCodeVerbose(prompt, yolo, model) {
     const claudeBin = resolveCommand("claude");
     const args = buildClaudeArgs(prompt, { yolo, model, verbose: true });
     const child = spawn(claudeBin, args, { stdio: ["inherit", "pipe", "inherit"] });
+    activeChildren.add(child);
     const rl = createInterface({ input: child.stdout });
     let turnCount = 0;
     child.on("error", (err) => {
@@ -1564,6 +1590,7 @@ function invokeClaudeCodeVerbose(prompt, yolo, model) {
       }
     });
     child.on("close", (code) => {
+      activeChildren.delete(child);
       handleExitCode(code, "Claude Code");
       resolve2();
     });
@@ -2311,42 +2338,64 @@ function getIssueSurface(issueNumber) {
   };
 }
 var LINK_MAP_QUERY = `
-query($owner:String!,$repo:String!){
+query($owner:String!,$repo:String!,$cursor:String){
   repository(owner:$owner,name:$repo){
-    pullRequests(states:OPEN, first:100, orderBy:{field:UPDATED_AT, direction:DESC}){
+    pullRequests(states:OPEN, first:100, after:$cursor, orderBy:{field:UPDATED_AT, direction:DESC}){
+      pageInfo{ hasNextPage endCursor }
       nodes{
         number url title headRefName isDraft updatedAt
-        closingIssuesReferences(first:10){ nodes{ number } }
+        closingIssuesReferences(first:50){
+          pageInfo{ hasNextPage }
+          nodes{ number }
+        }
       }
     }
   }
 }`.trim();
+var MAX_LINK_MAP_PAGES = 50;
 function getOpenPrLinkMap() {
   const { owner, repo } = getRepoSlug();
-  const response = ghJson(
-    ["api", "graphql", "-f", `query=${LINK_MAP_QUERY}`, "-f", `owner=${owner}`, "-f", `repo=${repo}`],
-    "query open pull requests"
-  );
   const map = /* @__PURE__ */ new Map();
-  for (const node of response.data.repository.pullRequests.nodes) {
-    const ref = {
-      number: node.number,
-      url: node.url,
-      title: node.title,
-      headRefName: node.headRefName,
-      state: "OPEN",
-      isDraft: node.isDraft,
-      updatedAt: node.updatedAt
-    };
-    for (const issue of node.closingIssuesReferences.nodes) {
-      const existing = map.get(issue.number);
-      if (existing) {
-        existing.push(ref);
-      } else {
-        map.set(issue.number, [ref]);
+  let cursor = null;
+  for (let page = 0; page < MAX_LINK_MAP_PAGES; page++) {
+    const args = ["api", "graphql", "-f", `query=${LINK_MAP_QUERY}`, "-f", `owner=${owner}`, "-f", `repo=${repo}`];
+    if (cursor !== null) args.push("-f", `cursor=${cursor}`);
+    const response = ghJson(args, "query open pull requests");
+    const connection = response.data.repository.pullRequests;
+    for (const node of connection.nodes) {
+      const ref = {
+        number: node.number,
+        url: node.url,
+        title: node.title,
+        headRefName: node.headRefName,
+        state: "OPEN",
+        isDraft: node.isDraft,
+        updatedAt: node.updatedAt
+      };
+      if (node.closingIssuesReferences.pageInfo?.hasNextPage) {
+        process.stderr.write(
+          `Warning: pull request #${String(node.number)} closes more than 50 issues; some links were not read.
+`
+        );
+      }
+      for (const issue of node.closingIssuesReferences.nodes) {
+        const existing = map.get(issue.number);
+        if (existing) {
+          existing.push(ref);
+        } else {
+          map.set(issue.number, [ref]);
+        }
       }
     }
+    if (!connection.pageInfo?.hasNextPage || connection.pageInfo.endCursor === null) {
+      return map;
+    }
+    cursor = connection.pageInfo.endCursor;
   }
+  process.stderr.write(
+    `Warning: stopped paginating open pull requests after ${String(MAX_LINK_MAP_PAGES)} pages; the issue-to-pull-request map may be incomplete.
+`
+  );
   return map;
 }
 var REVIEW_THREADS_QUERY2 = `
@@ -2501,9 +2550,19 @@ function isAssignedToAgent(assignees, agentUser) {
   return assignees.some((name) => name.toLowerCase() === agent);
 }
 function findActionableThreads(threads, p) {
-  return threads.filter(
-    (thread) => !thread.isResolved && lastAuthorClass(thread.comments, p) === "authorized"
-  );
+  const actionable = [];
+  for (const thread of threads) {
+    if (thread.isResolved) continue;
+    const comments = thread.comments.filter((comment) => classifyForThread(comment.author, p) !== "other");
+    if (lastAuthorClass(comments, p) !== "authorized") continue;
+    actionable.push({ ...thread, comments });
+  }
+  return actionable;
+}
+function classifyForThread(author, p) {
+  const login2 = author.toLowerCase();
+  if (login2 === p.agentUser.toLowerCase()) return "agent";
+  return p.allowedUsers.some((user) => user.toLowerCase() === login2) ? "authorized" : "other";
 }
 function newestPr(prs) {
   return [...prs].sort((a, b) => b.updatedAt.localeCompare(a.updatedAt))[0];
@@ -2710,6 +2769,7 @@ function preparePrBranch(headRefName) {
 
 // src/run/runLock.ts
 import { writeFileSync, readFileSync as readFileSync3, unlinkSync, mkdirSync } from "fs";
+import { randomUUID } from "crypto";
 import { hostname } from "os";
 import { join as join2 } from "path";
 var LOCK_DIR = ".automata";
@@ -2733,7 +2793,8 @@ function readOwner(path) {
       pid: parsed.pid,
       startedAt: parsed.startedAt,
       host: parsed.host ?? "unknown",
-      command: parsed.command ?? "unknown"
+      command: parsed.command ?? "unknown",
+      token: parsed.token ?? ""
     };
   } catch {
     return null;
@@ -2741,17 +2802,23 @@ function readOwner(path) {
 }
 function isStale(owner, staleMinutes) {
   if (owner === null) return true;
-  if (owner.host === hostname() && !isAlive(owner.pid)) return true;
+  if (owner.host === hostname()) {
+    return !isAlive(owner.pid);
+  }
   const startedAt = Date.parse(owner.startedAt);
   if (Number.isNaN(startedAt)) return true;
   return Date.now() - startedAt > staleMinutes * 60 * 1e3;
 }
-function makeHandle(path) {
+function makeHandle(path, token) {
   let released = false;
   return {
     release() {
       if (released) return;
       released = true;
+      const current = readOwner(path);
+      if (current !== null && current.token !== token) {
+        return;
+      }
       try {
         unlinkSync(path);
       } catch {
@@ -2759,21 +2826,23 @@ function makeHandle(path) {
     }
   };
 }
-function write(path, command) {
+function write(path, command, token) {
   const owner = {
     pid: process.pid,
     startedAt: (/* @__PURE__ */ new Date()).toISOString(),
     host: hostname(),
-    command
+    command,
+    token
   };
   writeFileSync(path, JSON.stringify(owner, null, 2) + "\n", { encoding: "utf8", flag: "wx" });
 }
 function acquireRunLock(command, staleMinutes) {
   const path = lockPath();
+  const token = randomUUID();
   mkdirSync(join2(process.cwd(), LOCK_DIR), { recursive: true });
   try {
-    write(path, command);
-    return { ok: true, handle: makeHandle(path) };
+    write(path, command, token);
+    return { ok: true, handle: makeHandle(path, token) };
   } catch (err) {
     if (err.code !== "EEXIST") throw err;
   }
@@ -2786,14 +2855,20 @@ function acquireRunLock(command, staleMinutes) {
   } catch {
   }
   try {
-    write(path, command);
-    return { ok: true, handle: makeHandle(path) };
+    write(path, command, token);
+    return { ok: true, handle: makeHandle(path, token) };
   } catch (err) {
     if (err.code !== "EEXIST") throw err;
     const winner = readOwner(path);
     return {
       ok: false,
-      heldBy: winner ?? { pid: 0, startedAt: (/* @__PURE__ */ new Date()).toISOString(), host: "unknown", command: "unknown" }
+      heldBy: winner ?? {
+        pid: 0,
+        startedAt: (/* @__PURE__ */ new Date()).toISOString(),
+        host: "unknown",
+        command: "unknown",
+        token: ""
+      }
     };
   }
 }
@@ -2886,9 +2961,8 @@ function checkAuthenticatedIdentity(agentUser, allowedUsers) {
       `\`gh\` is authenticated as "${login2}", which is listed in allowedUsers. Everything do-work posts would be attributed to an account that is allowed to instruct the agent, so its own marker comment would look like a new instruction and each tick would answer the previous tick forever. Authenticate \`gh\` as the agent account (${agentUser}) in this environment, or correct \`agentUser\`.`
     );
   }
-  progress(
-    `Warning: \`gh\` is authenticated as "${login2}" but agentUser is "${agentUser}". The agent will not recognise its own messages and may repeat itself. Authenticate as the agent account, or correct \`agentUser\`.
-`
+  fail(
+    `\`gh\` is authenticated as "${login2}" but agentUser is "${agentUser}". Comments posted under that identity are neither the agent's nor an authorized user's, so they are filtered out of the conversation: the answer boundary would never advance and the same message would start a run on every tick. Authenticate \`gh\` as the agent account (${agentUser}) in this environment, or correct \`agentUser\`.`
   );
 }
 function planRun(item, settings, silent) {
@@ -3016,8 +3090,14 @@ async function invokeExecutor(prompt, settings, silent) {
   }
   await invokeClaudeCode(prompt, { yolo: true, verbose: !silent, model: settings.model });
 }
-function repairIssueLink(item) {
+function repairIssueLink(item, baseBranch) {
   try {
+    const branch = getCurrentBranch();
+    if (branch === baseBranch) {
+      progress(`  issue #${String(item.issue.number)} is still in discussion (no branch was created).
+`);
+      return;
+    }
     const pr = getCurrentBranchPr();
     if (!pr) {
       progress(`  issue #${String(item.issue.number)} is still in discussion (no pull request).
@@ -3088,7 +3168,7 @@ async function processItem(item, settings, silent) {
   progress(`  ${reconciled.detail}
 `);
   if (item.turn === "issue-discuss") {
-    repairIssueLink(item);
+    repairIssueLink(item, settings.baseBranch);
   }
   return { ...base, outcome: reconciled.outcome, detail: reconciled.detail };
 }
@@ -3119,9 +3199,15 @@ var doWorkCommand = new Command6("do-work").description(
     return;
   }
   const handle = lock.handle;
+  let shuttingDown = false;
   const onSignal = () => {
-    handle.release();
-    process.exit(130);
+    if (shuttingDown) return;
+    shuttingDown = true;
+    progress("\nInterrupted: stopping the executor before releasing the run lock\u2026\n");
+    void terminateActiveClaudeProcesses().then(() => {
+      handle.release();
+      process.exit(130);
+    });
   };
   process.once("SIGINT", onSignal);
   process.once("SIGTERM", onSignal);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "automata-cli",
-  "version": "0.1.0-feature-030-do-work.192",
+  "version": "0.1.0-feature-030-do-work.194",
   "description": "Automata CLI tool",
   "type": "module",
   "bin": {