nightralph 0.0.35 → 0.0.37

package/dist/meta.json

@@ -142,7 +142,7 @@
       "format": "esm"
     },
     "src/orchestrator.ts": {
-      "bytes": 65884,
+      "bytes": 65558,
       "imports": [
         {
           "path": "node:child_process",
@@ -234,7 +234,7 @@
       "format": "esm"
     },
     "src/index.ts": {
-      "bytes": 10237,
+      "bytes": 10105,
       "imports": [
         {
           "path": "node:fs",
@@ -305,7 +305,7 @@
       "imports": [],
       "exports": [],
       "inputs": {},
-      "bytes": 176450
+      "bytes": 175862
     },
     "dist/index.js": {
       "imports": [
@@ -462,13 +462,13 @@
       "entryPoint": "src/index.ts",
       "inputs": {
         "src/index.ts": {
-          "bytesInOutput": 6851
+          "bytesInOutput": 6800
         },
         "src/setup.ts": {
           "bytesInOutput": 4265
         },
         "src/orchestrator.ts": {
-          "bytesInOutput": 45245
+          "bytesInOutput": 45107
         },
         "src/worktree.ts": {
           "bytesInOutput": 3788
@@ -489,7 +489,7 @@
           "bytesInOutput": 1937
         }
       },
-      "bytes": 81788
+      "bytes": 81599
     }
   }
 }

package/dist/orchestrator.js

@@ -251,11 +251,7 @@ function stripSpecSection(specBody, heading) {
   return out.join("\n");
 }
 __name(stripSpecSection, "stripSpecSection");
-function renderPrompt(specBody, ticket, opts = {}) {
-  const testRules = opts.testCmd ? [
-    "- Work test-first. For each checklist item, write a failing test at the seams named in the spec's Testing Decisions section, then write only enough code to make it pass.",
-    `- Run \`${opts.testCmd}\` before committing. Do not commit with failing tests. The orchestrator runs the same command after you exit and rejects the ticket if it fails.`
-  ] : [];
+function renderPrompt(specBody, ticket, opts) {
   return [
     "---- SPEC CONTEXT ----",
     specBody,
@@ -265,12 +261,13 @@ function renderPrompt(specBody, ticket, opts = {}) {
     ticket.body,
     "",
     "---- AGENT INSTRUCTIONS ----",
-    "You are an autonomous coding agent. Implement the ticket.",
+    "You are an autonomous coding agent. Implement the ticket using /tdd.",
     "",
     "Follow these rules:",
     "- Read the files directly relevant to the ticket before writing code.",
     "- Do NOT research external APIs or services via web search. Use the spec and ticket body as your sole reference for API shapes.",
-    ...testRules,
+    "- Use /tdd for every checklist item. For each item, write a failing test at the seams named in the spec's Testing Decisions section, then write only enough code to make it pass, then refactor.",
+    `- Run \`${opts.testCmd}\` before committing. Do not commit with failing tests. The orchestrator runs the same command after you exit and rejects the ticket if it fails.`,
     "- Do NOT read or modify files outside your current directory (the worktree).",
     "- Do NOT run git push.",
     "- Commit your work when finished.",
@@ -775,7 +772,7 @@ function spawnAgent(opts) {
   });
 }
 __name(spawnAgent, "spawnAgent");
-function dryRun(tickets, specBody) {
+function dryRun(tickets, specBody, testCmd) {
   listTickets(tickets);
   console.log("\nSimulated wave order:");
   for (const { wave, ready } of simulateWaves(tickets)) {
@@ -797,7 +794,11 @@ Prompt for first ready ticket (${firstReady[0].filename}):
 `
     );
     console.log(
-      renderPrompt(specBody, firstReady[0])
+      renderPrompt(
+        specBody,
+        firstReady[0],
+        { testCmd }
+      )
     );
   }
 }
@@ -843,7 +844,7 @@ Prompt for first ready ticket (${firstReady[0].filename}):
   const strategyDesc = opts.conflictStrategy === "respawn" ? "respawn (re-run agent on conflict)" : "stop (stop merging on conflict)";
   console.log(
     `
-Test command: ${opts.testCmd ?? "(none)"}`
+Test command: ${opts.testCmd}`
   );
   console.log(
     `Effort:       ${opts.effort ?? "(provider default)"}`
@@ -935,22 +936,20 @@ async function respawnAndRetryMerge(opts) {
     }
     return false;
   }
-  if (opts.testCmd) {
-    const passed = await runTestGate({
-      testCmd: opts.testCmd,
-      ticket: opts.ticket,
-      worktreePath: opts.worktreePath,
-      timeout: opts.timeout,
-      logsDir: opts.logsDir,
-      log,
-      registry: runningAgents
-    });
-    if (!passed) {
-      log(
-        `  ${opts.ticket.filename}: tests failed after conflict respawn; not merging`
-      );
-      return false;
-    }
+  const passed = await runTestGate({
+    testCmd: opts.testCmd,
+    ticket: opts.ticket,
+    worktreePath: opts.worktreePath,
+    timeout: opts.timeout,
+    logsDir: opts.logsDir,
+    log,
+    registry: runningAgents
+  });
+  if (!passed) {
+    log(
+      `  ${opts.ticket.filename}: tests failed after conflict respawn; not merging`
+    );
+    return false;
   }
   const mergeResult = await mergeBranch(
     opts.repoRoot,
@@ -1066,30 +1065,28 @@ async function runTicketAttempt(opts) {
       branchName: worktree.branchName
     };
   }
-  if (opts.testCmd) {
-    const passed = await runTestGate({
-      testCmd: opts.testCmd,
-      ticket,
-      worktreePath: worktree.worktreePath,
-      timeout: opts.timeout,
-      logsDir: opts.logsDir,
-      attempt: opts.attempt,
-      log: /* @__PURE__ */ __name((msg) => d.log(msg), "log"),
-      registry: runningAgents
-    });
-    if (!passed) {
-      await removeWorktreeQuietly(
-        worktree.worktreePath,
-        d
-      );
-      return {
-        kind: "failed",
-        reason: "tests",
-        branchName: worktree.branchName,
-        exitCode: 1,
-        completedItems: agentResult.completedItems
-      };
-    }
+  const passed = await runTestGate({
+    testCmd: opts.testCmd,
+    ticket,
+    worktreePath: worktree.worktreePath,
+    timeout: opts.timeout,
+    logsDir: opts.logsDir,
+    attempt: opts.attempt,
+    log: /* @__PURE__ */ __name((msg) => d.log(msg), "log"),
+    registry: runningAgents
+  });
+  if (!passed) {
+    await removeWorktreeQuietly(
+      worktree.worktreePath,
+      d
+    );
+    return {
+      kind: "failed",
+      reason: "tests",
+      branchName: worktree.branchName,
+      exitCode: 1,
+      completedItems: agentResult.completedItems
+    };
   }
   return {
     kind: "ready",
@@ -1163,7 +1160,7 @@ function runProgress(tickets) {
   };
 }
 __name(runProgress, "runProgress");
-async function runWaves(tickets, specBody, agentCmd, model, timeout, logsDir, display, maxTurns, effort, retries, retryDelay) {
+async function runWaves(tickets, specBody, agentCmd, model, timeout, logsDir, testCmd, display, maxTurns, effort, retries, retryDelay) {
   mkdirSync(logsDir, { recursive: true });
   const d = display ?? createDisplay();
   const maxAttempts = (retries ?? DEFAULT_RETRIES) + 1;
@@ -1200,7 +1197,11 @@ async function runWaves(tickets, specBody, agentCmd, model, timeout, logsDir, di
     );
     const results = await Promise.allSettled(
       ready.map(async (t) => {
-        const prompt = renderPrompt(specBody, t);
+        const prompt = renderPrompt(
+          specBody,
+          t,
+          { testCmd }
+        );
         const label = t.filename.replace(
           /\.md$/,
           ""
@@ -1327,9 +1328,6 @@ async function runWavesIsolated(opts) {
     `nightralph: start ${opts.featureName}`,
     d
   );
-  if (!opts.testCmd) {
-    d.log("  Test gate disabled: no test command");
-  }
   let totalCompleted = 0;
   let waveNum = 0;
   const failedNums = /* @__PURE__ */ new Set();