nightralph 0.0.34 → 0.0.36

package/README.md

@@ -318,4 +318,4 @@ caffeinate -i npx nightralph pi -m openrouter/z-ai/glm-5.3-flash
 
 ```sh
 caffeinate -i npx nightralph codex -m gpt-5.6-luna
-```
+```

package/dist/docs-templates/issue-tracker.md

@@ -6,18 +6,26 @@ Issues and specs for this repo live as markdown files in `.scratch/`.
 
 - One feature per directory: `.scratch/<feature-slug>/`
 - The spec is `.scratch/<feature-slug>/spec.md`
-- Implementation issues are one file per ticket at `.scratch/<feature-slug>/issues/<NN>-<slug>.md`, numbered from `01`, never a single combined tickets file
-- Triage state is recorded as a `Status:` line near the top of each issue file (see `triage-labels.md` for the role strings)
-- Dependencies are recorded as a `**Blocked by:**` line near the top of each issue file, listing the numbers of the tickets that must complete first, or `None`
-- Comments and conversation history append to the bottom of the file under a `## Comments` heading
+- Implementation issues are one file per ticket at
+  `.scratch/<feature-slug>/issues/<NN>-<slug>.md`, numbered from `01`,
+  never a single combined tickets file
+- Triage state is recorded as a `**Status:**` line near the top of each
+  issue file (see `triage-labels.md` for the role strings)
+- Dependencies are recorded as a `**Blocked by:**` line near the top of
+  each issue file, listing the numbers of the tickets that must complete
+  first, or `None`
+- Comments and conversation history append to the bottom of the file under
+  a `## Comments` heading
 
 ## When a skill says "publish to the issue tracker"
 
-Create a new file under `.scratch/<feature-slug>/` (creating the directory if needed).
+Create a new file under `.scratch/<feature-slug>/` (creating the directory
+if needed).
 
 ## When a skill says "fetch the relevant ticket"
 
-Read the file at the referenced path. The user will normally pass the path or the issue number directly.
+Read the file at the referenced path. The user will normally pass the path
+or the issue number directly.
 
 ## Tickets are executed within hours, not weeks
 

package/dist/index.js

@@ -1292,11 +1292,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,
@@ -1306,12 +1302,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.",
@@ -1816,7 +1813,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)) {
@@ -1838,7 +1835,11 @@ Prompt for first ready ticket (${firstReady[0].filename}):
 `
     );
     console.log(
-      renderPrompt(specBody, firstReady[0])
+      renderPrompt(
+        specBody,
+        firstReady[0],
+        { testCmd }
+      )
     );
   }
 }
@@ -1884,7 +1885,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)"}`
@@ -1976,22 +1977,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,
@@ -2107,30 +2106,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",
@@ -2204,7 +2201,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) {
   mkdirSync2(logsDir, { recursive: true });
   const d = display ?? createDisplay();
   const maxAttempts = (retries ?? DEFAULT_RETRIES) + 1;
@@ -2241,7 +2238,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$/,
           ""
@@ -2368,9 +2369,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();
@@ -2823,7 +2821,7 @@ yargs(hideBin(process.argv)).scriptName("nightralph").usage("$0 <command>").comm
     default: 3600
   }).option("test-cmd", {
     type: "string",
-    describe: 'Command run in each worktree after the agent exits; non-zero exit rejects the ticket. Defaults to "npm test" when package.json has a test script. Pass "" to disable.'
+    describe: 'Command run in each worktree after the agent exits; non-zero exit rejects the ticket. Defaults to "npm test".'
   }).option("X", {
     type: "boolean",
     describe: "Stop on merge conflict instead of re-spawning",
@@ -2918,9 +2916,9 @@ async function runExecute(argv) {
   }
   const logsDir = join6(dirname3(dir), "logs");
   const timeout = argv.timeout ?? 900;
-  const testCmd = argv.testCmd === void 0 ? detectTestCmd(
+  const testCmd = argv.testCmd || detectTestCmd(
     isGitRepo() ? getRepoRoot() : process.cwd()
-  ) : argv.testCmd || null;
+  ) || "npm test";
   const featureName = extractFeatureName(spec);
   const conflictStrategy = argv.stopOnConflict ? "stop" : "respawn";
   const progressPath = join6(
@@ -2942,7 +2940,7 @@ async function runExecute(argv) {
         retryDelay: argv.retryDelay
       });
     } else {
-      dryRun(tickets, specBody);
+      dryRun(tickets, specBody, testCmd);
     }
     return;
   }
@@ -2981,6 +2979,7 @@ async function runExecute(argv) {
       argv.model ?? null,
       timeout,
       logsDir,
+      testCmd,
       display,
       argv.maxTurns,
       argv.effort,