nightralph 0.0.21 → 0.0.23

package/dist/meta.json

@@ -1,7 +1,7 @@
 {
   "inputs": {
     "src/setup.ts": {
-      "bytes": 4981,
+      "bytes": 5004,
       "imports": [
         {
           "path": "node:fs",
@@ -27,7 +27,7 @@
       "format": "esm"
     },
     "src/worktree.ts": {
-      "bytes": 4846,
+      "bytes": 5108,
       "imports": [
         {
           "path": "node:child_process",
@@ -116,7 +116,7 @@
       "format": "esm"
     },
     "src/testcmd.ts": {
-      "bytes": 2195,
+      "bytes": 2463,
       "imports": [
         {
           "path": "node:child_process",
@@ -142,7 +142,7 @@
       "format": "esm"
     },
     "src/orchestrator.ts": {
-      "bytes": 56085,
+      "bytes": 57999,
       "imports": [
         {
           "path": "node:child_process",
@@ -305,7 +305,7 @@
       "imports": [],
       "exports": [],
       "inputs": {},
-      "bytes": 161652
+      "bytes": 165335
     },
     "dist/index.js": {
       "imports": [
@@ -465,13 +465,13 @@
           "bytesInOutput": 6896
         },
         "src/setup.ts": {
-          "bytesInOutput": 4318
+          "bytesInOutput": 4339
         },
         "src/orchestrator.ts": {
-          "bytesInOutput": 39685
+          "bytesInOutput": 40993
         },
         "src/worktree.ts": {
-          "bytesInOutput": 3537
+          "bytesInOutput": 3788
         },
         "src/progress.ts": {
           "bytesInOutput": 4419
@@ -483,13 +483,13 @@
           "bytesInOutput": 7877
         },
         "src/testcmd.ts": {
-          "bytesInOutput": 1620
+          "bytesInOutput": 1911
         },
         "src/resolve.ts": {
           "bytesInOutput": 1937
         }
       },
-      "bytes": 75730
+      "bytes": 77601
     }
   }
 }

package/dist/orchestrator.js

@@ -219,6 +219,7 @@ function renderMergePrompt(specBody, ticket) {
     "3. Commit the resolved files.",
     "",
     "Do NOT re-implement the ticket from scratch.",
+    "Do NOT read or modify files outside your current directory (the worktree).",
     "Do NOT run git push."
   ].join("\n");
 }
@@ -600,6 +601,7 @@ function spawnAgent(opts) {
     __name(finish, "finish");
     proc.on("exit", (code) => {
       graceTimer = setTimeout(() => {
+        killProcessGroup(proc.pid);
         proc.stdout.destroy();
         proc.stderr.destroy();
         finish(code ?? 1);
@@ -710,7 +712,7 @@ Test command: ${opts.testCmd ?? "(none)"}`
     `Retries:      ${opts.retries ?? DEFAULT_RETRIES}`
   );
   console.log(
-    `Retry delay:  ${opts.retryDelay ?? DEFAULT_RETRY_DELAY}s`
+    `Retry delay:  ${opts.retryDelay ?? 0}s`
   );
   console.log(
     `
@@ -723,7 +725,8 @@ async function runTestGate(opts) {
   const result = await runTestCmd({
     cmd: opts.testCmd,
     cwd: opts.worktreePath,
-    timeout: opts.timeout
+    timeout: opts.timeout,
+    registry: opts.registry
   });
   const testLogPath = attemptLogPath(
     opts.logsDir,
@@ -799,7 +802,8 @@ async function respawnAndRetryMerge(opts) {
       worktreePath: opts.worktreePath,
       timeout: opts.timeout,
       logsDir: opts.logsDir,
-      log
+      log,
+      registry: runningAgents
     });
     if (!passed) {
       log(
@@ -915,7 +919,12 @@ async function runTicketAttempt(opts) {
     opts.baseBranch,
     worktree.branchName
   )) {
-    return { kind: "no-commits" };
+    await removeWorktreeQuietly(worktree.worktreePath, d);
+    return {
+      kind: "no-commits",
+      worktreePath: worktree.worktreePath,
+      branchName: worktree.branchName
+    };
   }
   if (opts.testCmd) {
     const passed = await runTestGate({
@@ -925,7 +934,8 @@ async function runTicketAttempt(opts) {
       timeout: opts.timeout,
       logsDir: opts.logsDir,
       attempt: opts.attempt,
-      log: /* @__PURE__ */ __name((msg) => d.log(msg), "log")
+      log: /* @__PURE__ */ __name((msg) => d.log(msg), "log"),
+      registry: runningAgents
     });
     if (!passed) {
       await removeWorktreeQuietly(
@@ -954,7 +964,11 @@ async function runTicketAttempts(opts) {
   const maxAttempts = opts.retries + 1;
   const isPi = basename(opts.agentCmd) === "pi";
   let thinking = opts.thinking;
-  let outcome = { kind: "no-commits" };
+  let outcome = {
+    kind: "no-commits",
+    worktreePath: "",
+    branchName: ""
+  };
   for (let attempt = 1; attempt <= maxAttempts; attempt++) {
     if (attempt === 1) {
       d.log(`  Starting ${ticket.filename}`);
@@ -1215,6 +1229,18 @@ async function runWavesIsolated(opts) {
         d.log(
           `  ${ticket.filename}: failed to create worktree: ${formatErrorMessage(outcome.error)}`
         );
+        checkTicket(progress, ticket.num, {
+          branch: "",
+          exitCode: -1,
+          done: false
+        });
+        await saveProgress(
+          repoRoot,
+          progressPath,
+          progress,
+          `nightralph: ${ticket.filename} worktree error`,
+          d
+        );
         continue;
       }
       if (outcome.kind === "error") {
@@ -1223,6 +1249,18 @@ async function runWavesIsolated(opts) {
         d.log(
           `  ${ticket.filename}: attempt error: ` + formatErrorMessage(outcome.error)
         );
+        checkTicket(progress, ticket.num, {
+          branch: outcome.branchName ?? "",
+          exitCode: -1,
+          done: false
+        });
+        await saveProgress(
+          repoRoot,
+          progressPath,
+          progress,
+          `nightralph: ${ticket.filename} error`,
+          d
+        );
         continue;
       }
       if (outcome.kind === "no-commits") {
@@ -1231,6 +1269,18 @@ async function runWavesIsolated(opts) {
         d.log(
           `  ${ticket.filename}: agent exited 0 but made no changes; leaving ticket ready-for-agent`
         );
+        checkTicket(progress, ticket.num, {
+          branch: outcome.branchName,
+          exitCode: 0,
+          done: false
+        });
+        await saveProgress(
+          repoRoot,
+          progressPath,
+          progress,
+          `nightralph: ${ticket.filename} no commits`,
+          d
+        );
         continue;
       }
       if (outcome.kind === "failed") {