automata-cli 0.6.0-develop.225 → 0.6.0-develop.230

package/dist/{ConfigWizard-5B5AOGUU.js → ConfigWizard-J67TY22Z.js}

@@ -130,8 +130,8 @@ function ConfigWizard() {
   const initialTechIndex = TECHNIQUE_OPTIONS.findIndex((o) => o.value === existing.issueDiscoveryTechnique);
   const [screen, setScreen] = useState("main");
   const [mainMenuIndex, setMainMenuIndex] = useState(0);
-  const [selectedRemoteIndex, setSelectedRemoteIndex] = useState(initialRemoteIndex >= 0 ? initialRemoteIndex : 0);
-  const [selectedTechIndex, setSelectedTechIndex] = useState(initialTechIndex >= 0 ? initialTechIndex : 0);
+  const [selectedRemoteIndex, setSelectedRemoteIndex] = useState(Math.max(initialRemoteIndex, 0));
+  const [selectedTechIndex, setSelectedTechIndex] = useState(Math.max(initialTechIndex, 0));
   const [discoveryValue, setDiscoveryValue] = useState(existing.issueDiscoveryValue ?? "");
   const [systemPrompt, setSystemPrompt] = useState(existing.claudeSystemPrompt ?? "");
   const [promptsMenuIndex, setPromptsMenuIndex] = useState(0);

package/dist/index.js

@@ -194,7 +194,7 @@ var configCommand = new Command("config").description("Configure automata settin
   const [{ render }, React, { ConfigWizard }] = await Promise.all([
     import("ink"),
     import("react"),
-    import("./ConfigWizard-5B5AOGUU.js")
+    import("./ConfigWizard-J67TY22Z.js")
   ]);
   const { waitUntilExit } = render(React.createElement(ConfigWizard));
   await waitUntilExit();
@@ -276,15 +276,15 @@ function parseOwnerRepo() {
   const { stdout, status } = run2("git", ["remote", "get-url", "origin"]);
   if (status !== 0) return null;
   const url = stdout.trim();
-  const https = url.match(/github\.com\/([^/]+\/[^/]+?)(?:\.git)?$/);
+  const https = /github\.com\/([^/]+\/[^/]+?)(?:\.git)?$/.exec(url);
   if (https) return https[1];
-  const ssh = url.match(/github\.com:([^/]+\/[^/]+?)(?:\.git)?$/);
+  const ssh = /github\.com:([^/]+\/[^/]+?)(?:\.git)?$/.exec(url);
   if (ssh) return ssh[1];
   return null;
 }
 function extractLastMarkdownUrl(markdown) {
   const matches = [...markdown.matchAll(/\]\((https?:\/\/[^)]+)\)/g)];
-  return matches.length > 0 ? matches[matches.length - 1][1] ?? null : null;
+  return matches.at(-1)?.[1] ?? null;
 }
 function fetchCheckRunOutputs(ownerRepo, sha) {
   const { stdout, status } = run2("gh", [
@@ -366,12 +366,14 @@ function resolveSonarPath(componentKey, components) {
   const path = componentKey.slice(separatorIndex + 1).trim();
   return path || void 0;
 }
-function mapSonarIssuesPage(data, targetIssues, components, rules) {
+function collectSonarComponents(data, components) {
   for (const component of data.components ?? []) {
     if (component.key && component.path) {
       components.set(component.key, component.path);
     }
   }
+}
+function collectSonarRules(data, rules) {
   for (const rule of data.rules ?? []) {
     if (!rule.key) continue;
     const explanation = normalizeText(rule.htmlDesc) ?? normalizeText(rule.htmlNote) ?? normalizeText(rule.name);
@@ -379,6 +381,10 @@ function mapSonarIssuesPage(data, targetIssues, components, rules) {
       rules.set(rule.key, explanation);
     }
   }
+}
+function mapSonarIssuesPage(data, targetIssues, components, rules) {
+  collectSonarComponents(data, components);
+  collectSonarRules(data, rules);
   for (const issue of data.issues ?? []) {
     if (!issue.key || !issue.message) continue;
     const ruleKey = issue.rule;
@@ -396,11 +402,7 @@ function mapSonarIssuesPage(data, targetIssues, components, rules) {
   }
 }
 function mapSonarHotspotsPage(data, targetHotspots, components) {
-  for (const component of data.components ?? []) {
-    if (component.key && component.path) {
-      components.set(component.key, component.path);
-    }
-  }
+  collectSonarComponents(data, components);
   targetHotspots.push(...data.hotspots ?? []);
 }
 function mapSonarHotspot(hotspot, components, detail) {
@@ -625,6 +627,32 @@ async function fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl) {
     issueNote
   };
 }
+async function describeSonarCheck(sonarCheck, prNumber) {
+  const sonarcloudUrl = resolveSonarPullRequestUrl(sonarCheck.detailsUrl, prNumber);
+  const projectKey = extractSonarProjectKey(sonarcloudUrl);
+  if (!projectKey) {
+    return {
+      sonarcloudUrl,
+      sonarNewIssues: null,
+      sonarNewIssuesNote: "Could not determine the SonarCloud project key from the check URL."
+    };
+  }
+  if (sonarCheck.conclusion !== null && SONAR_FAIL_CONCLUSIONS.has(sonarCheck.conclusion)) {
+    const sonarSummary = await fetchSonarFailureSummary(projectKey, prNumber, sonarcloudUrl);
+    return {
+      sonarcloudUrl,
+      sonarNewIssues: sonarSummary.issueTotal,
+      sonarNewIssuesNote: sonarSummary.issueNote,
+      sonarFailures: sonarSummary.summary
+    };
+  }
+  const sonarNewIssuesResult = await fetchSonarNewIssues(projectKey, prNumber, sonarcloudUrl);
+  return {
+    sonarcloudUrl,
+    sonarNewIssues: sonarNewIssuesResult.total,
+    sonarNewIssuesNote: sonarNewIssuesResult.note
+  };
+}
 async function getPrInfoGh(branch) {
   const { stdout, stderr, status } = run2("gh", [
     "pr",
@@ -660,37 +688,19 @@ async function getPrInfoGh(branch) {
     };
   });
   const sonarCheck = checks.find((c) => isSonarUrl(c.detailsUrl));
-  let sonarcloudUrl;
-  let sonarNewIssues;
-  let sonarNewIssuesNote;
-  let sonarFailures;
-  if (sonarCheck) {
-    sonarcloudUrl = resolveSonarPullRequestUrl(sonarCheck.detailsUrl, raw.number);
-    const projectKey = extractSonarProjectKey(sonarcloudUrl);
-    if (projectKey) {
-      if (sonarCheck.conclusion !== null && SONAR_FAIL_CONCLUSIONS.has(sonarCheck.conclusion)) {
-        const sonarSummary = await fetchSonarFailureSummary(projectKey, raw.number, sonarcloudUrl);
-        sonarFailures = sonarSummary.summary;
-        sonarNewIssues = sonarSummary.issueTotal;
-        sonarNewIssuesNote = sonarSummary.issueNote;
-      } else {
-        const sonarNewIssuesResult = await fetchSonarNewIssues(projectKey, raw.number, sonarcloudUrl);
-        sonarNewIssues = sonarNewIssuesResult.total;
-        sonarNewIssuesNote = sonarNewIssuesResult.note;
-      }
-    } else {
-      sonarNewIssues = null;
-      sonarNewIssuesNote = "Could not determine the SonarCloud project key from the check URL.";
-    }
-  }
+  const sonar = sonarCheck === void 0 ? void 0 : await describeSonarCheck(sonarCheck, raw.number);
   return {
     number: raw.number,
     title: raw.title,
     state: raw.state,
     url: raw.url,
     checks,
-    ...sonarcloudUrl === void 0 ? {} : { sonarcloudUrl, sonarNewIssues, sonarNewIssuesNote },
-    ...sonarFailures === void 0 ? {} : { sonarFailures }
+    ...sonar === void 0 ? {} : {
+      sonarcloudUrl: sonar.sonarcloudUrl,
+      sonarNewIssues: sonar.sonarNewIssues,
+      sonarNewIssuesNote: sonar.sonarNewIssuesNote
+    },
+    ...sonar?.sonarFailures === void 0 ? {} : { sonarFailures: sonar.sonarFailures }
   };
 }
 async function getPrInfo2(branch) {
@@ -1059,7 +1069,6 @@ When checks fail, details are printed in a trailing FailedChecks section.
 See docs/git.md for full output reference.`
 ).action(async (options) => {
   let branch;
-  let pr = null;
   try {
     branch = getCurrentBranch();
   } catch (err) {
@@ -1067,33 +1076,7 @@ See docs/git.md for full output reference.`
 `);
     process.exit(1);
   }
-  if (options.waitFinishChecks) {
-    while (true) {
-      try {
-        pr = await getPrInfo2(branch);
-      } catch (err) {
-        process.stderr.write(`Error: ${err.message}
-`);
-        process.exit(1);
-      }
-      if (pr === null) {
-        break;
-      }
-      const running = pr.checks.filter((c) => c.status !== "COMPLETED");
-      if (running.length === 0) break;
-      process.stdout.write(`Waiting for ${running.length} check(s) to complete...
-`);
-      await sleep(POLL_INTERVAL_MS);
-    }
-  } else {
-    try {
-      pr = await getPrInfo2(branch);
-    } catch (err) {
-      process.stderr.write(`Error: ${err.message}
-`);
-      process.exit(1);
-    }
-  }
+  const pr = options.waitFinishChecks ? await pollUntilChecksComplete(branch) : await readPrInfoOrExit(branch);
   if (pr === null) {
     process.stdout.write(`No pull request found for branch: ${branch}
 `);
@@ -1102,36 +1085,61 @@ See docs/git.md for full output reference.`
   if (options.json) {
     process.stdout.write(JSON.stringify(pr, null, 2) + "\n");
   } else {
-    const failed = pr.checks.filter((c) => c.conclusion !== null && FAIL_CONCLUSIONS.has(c.conclusion));
-    process.stdout.write(`PR:    #${pr.number}
-Title: ${pr.title}
-State: ${pr.state}
-URL:   ${pr.url}
+    printPrStatus(pr);
+  }
+});
+async function readPrInfoOrExit(branch) {
+  try {
+    return await getPrInfo2(branch);
+  } catch (err) {
+    process.stderr.write(`Error: ${err.message}
 `);
-    if (pr.sonarcloudUrl !== void 0) {
-      process.stdout.write(`Sonar: ${pr.sonarcloudUrl}
+    process.exit(1);
+  }
+}
+async function pollUntilChecksComplete(branch) {
+  for (; ; ) {
+    const pr = await readPrInfoOrExit(branch);
+    if (pr === null) return null;
+    const running = pr.checks.filter((c) => c.status !== "COMPLETED");
+    if (running.length === 0) return pr;
+    process.stdout.write(`Waiting for ${running.length} check(s) to complete...
 `);
-      const issueStr = pr.sonarNewIssues === null || pr.sonarNewIssues === void 0 ? "unavailable" : String(pr.sonarNewIssues);
-      process.stdout.write(`Sonar New Issues: ${issueStr}
+    await sleep(POLL_INTERVAL_MS);
+  }
+}
+function printSonarStatus(pr) {
+  if (pr.sonarcloudUrl === void 0) return;
+  process.stdout.write(`Sonar: ${pr.sonarcloudUrl}
 `);
-      const failureNote = sonarFailureNote(pr.sonarFailures);
-      if (pr.sonarNewIssuesNote && pr.sonarNewIssuesNote !== failureNote) {
-        process.stdout.write(`Sonar Note: ${sanitizeText(pr.sonarNewIssuesNote)}
+  const issueStr = pr.sonarNewIssues === null || pr.sonarNewIssues === void 0 ? "unavailable" : String(pr.sonarNewIssues);
+  process.stdout.write(`Sonar New Issues: ${issueStr}
+`);
+  const failureNote = sonarFailureNote(pr.sonarFailures);
+  if (pr.sonarNewIssuesNote && pr.sonarNewIssuesNote !== failureNote) {
+    process.stdout.write(`Sonar Note: ${sanitizeText(pr.sonarNewIssuesNote)}
 `);
-      }
-    }
-    process.stdout.write(formatCheckSummary(pr.checks));
-    process.stdout.write(formatChecks(pr.checks));
-    if (failed.length > 0) {
-      process.stdout.write(formatFailedChecks(failed));
-    }
-    if (pr.sonarFailures !== void 0) {
-      process.stdout.write(formatSonarFailures(pr.sonarFailures, pr.sonarcloudUrl));
-    }
   }
-});
-var ANSI_ESCAPE_RE = new RegExp("\x1B(?:[@-Z\\\\-_]|\\[[0-?]*[ -/]*[@-~])", "g");
-var CONTROL_CHARS_RE = new RegExp("[\0-\b\v\f-\x7F]", "g");
+}
+function printPrStatus(pr) {
+  const failed = pr.checks.filter((c) => c.conclusion !== null && FAIL_CONCLUSIONS.has(c.conclusion));
+  process.stdout.write(`PR:    #${pr.number}
+Title: ${pr.title}
+State: ${pr.state}
+URL:   ${pr.url}
+`);
+  printSonarStatus(pr);
+  process.stdout.write(formatCheckSummary(pr.checks));
+  process.stdout.write(formatChecks(pr.checks));
+  if (failed.length > 0) {
+    process.stdout.write(formatFailedChecks(failed));
+  }
+  if (pr.sonarFailures !== void 0) {
+    process.stdout.write(formatSonarFailures(pr.sonarFailures, pr.sonarcloudUrl));
+  }
+}
+var ANSI_ESCAPE_RE = /\x1b(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])/g;
+var CONTROL_CHARS_RE = /[\x00-\x08\x0b\x0c\x0e-\x1f\x7f]/g;
 function sanitizeText(text) {
   return text.replace(ANSI_ESCAPE_RE, "").replace(CONTROL_CHARS_RE, "");
 }
@@ -1666,65 +1674,78 @@ function invokeClaudeCodeVerbose(prompt, yolo, model) {
     });
   });
 }
+function formatAssistantEvent(event, turnCount) {
+  const message = event["message"];
+  const content = message?.["content"];
+  if (!content) return;
+  for (const block of content) {
+    const line = formatContentBlock(block);
+    if (line !== null) {
+      process.stderr.write(`  [step ${turnCount + 1}] ${line}
+`);
+    }
+  }
+}
+function formatContentBlock(block) {
+  if (block["type"] === "tool_use") {
+    const toolName = block["name"];
+    const input = block["input"];
+    return summarizeTool(toolName, input);
+  }
+  if (block["type"] === "text") {
+    const text = block["text"] ?? "";
+    if (text.length === 0) return null;
+    const preview = text.length > 120 ? text.slice(0, 120) + "..." : text;
+    return preview.split("\n")[0] ?? null;
+  }
+  return null;
+}
+function formatResultEvent(event) {
+  const result = event["result"];
+  const cost = event["cost_usd"];
+  const duration = event["duration_ms"];
+  const turns = event["num_turns"];
+  process.stderr.write("\n--- Result ---\n");
+  const parts = [];
+  if (turns !== void 0) parts.push(`${turns} turns`);
+  if (duration !== void 0) parts.push(`${(duration / 1e3).toFixed(1)}s`);
+  if (cost !== void 0) parts.push(`$${cost.toFixed(4)}`);
+  if (parts.length > 0) {
+    process.stderr.write(`  [info] ${parts.join(" | ")}
+`);
+  }
+  if (result) {
+    process.stdout.write(result + "\n");
+  }
+}
 function formatEvent(event, turnCount) {
   const type = event["type"];
   if (type === "assistant") {
-    const message = event["message"];
-    const content = message?.["content"];
-    if (!content) return;
-    for (const block of content) {
-      if (block["type"] === "tool_use") {
-        const toolName = block["name"];
-        const input = block["input"];
-        const summary = summarizeTool(toolName, input);
-        process.stderr.write(`  [step ${turnCount + 1}] ${summary}
-`);
-      } else if (block["type"] === "text") {
-        const text = block["text"] ?? "";
-        if (text.length > 0) {
-          const preview = text.length > 120 ? text.slice(0, 120) + "..." : text;
-          const firstLine = preview.split("\n")[0];
-          process.stderr.write(`  [step ${turnCount + 1}] ${firstLine}
-`);
-        }
-      }
-    }
+    formatAssistantEvent(event, turnCount);
   } else if (type === "result") {
-    const result = event["result"];
-    const cost = event["cost_usd"];
-    const duration = event["duration_ms"];
-    const turns = event["num_turns"];
-    process.stderr.write("\n--- Result ---\n");
-    if (cost !== void 0 || duration !== void 0 || turns !== void 0) {
-      const parts = [];
-      if (turns !== void 0) parts.push(`${turns} turns`);
-      if (duration !== void 0) parts.push(`${(duration / 1e3).toFixed(1)}s`);
-      if (cost !== void 0) parts.push(`$${cost.toFixed(4)}`);
-      process.stderr.write(`  [info] ${parts.join(" | ")}
-`);
-    }
-    if (result) {
-      process.stdout.write(result + "\n");
-    }
+    formatResultEvent(event);
   }
 }
+function asText(value, fallback) {
+  return typeof value === "string" ? value : fallback;
+}
 function summarizeTool(name, input) {
   if (!input) return `tool: ${name}`;
   switch (name) {
     case "Read":
-      return `reading ${input["file_path"] ?? "file"}`;
+      return `reading ${asText(input["file_path"], "file")}`;
     case "Write":
-      return `writing ${input["file_path"] ?? "file"}`;
+      return `writing ${asText(input["file_path"], "file")}`;
     case "Edit":
-      return `editing ${input["file_path"] ?? "file"}`;
+      return `editing ${asText(input["file_path"], "file")}`;
     case "Bash":
-      return `running: ${truncate(String(input["command"] ?? ""), 80)}`;
+      return `running: ${truncate(asText(input["command"], ""), 80)}`;
     case "Glob":
-      return `searching files: ${input["pattern"] ?? ""}`;
+      return `searching files: ${asText(input["pattern"], "")}`;
     case "Grep":
-      return `searching content: ${truncate(String(input["pattern"] ?? ""), 60)}`;
+      return `searching content: ${truncate(asText(input["pattern"], ""), 60)}`;
     case "Agent":
-      return `spawning agent: ${input["description"] ?? name}`;
+      return `spawning agent: ${asText(input["description"], name)}`;
     default:
       return `tool: ${name}`;
   }
@@ -1889,16 +1910,7 @@ ${issue.body}
   if (options.queryOnly) {
     process.exit(0);
   }
-  let executor;
-  if (options.claude !== false) {
-    const requestedExecutor = options.with.toLowerCase();
-    if (requestedExecutor !== "claude" && requestedExecutor !== "codex") {
-      process.stderr.write(`Error: --with must be 'claude' or 'codex', got '${options.with}'.
-`);
-      process.exit(1);
-    }
-    executor = requestedExecutor;
-  }
+  const executor = options.claude === false ? void 0 : resolveExecutor(options.with);
   let commentUrl;
   try {
     commentUrl = postComment(issue.number, "working");
@@ -1923,37 +1935,49 @@ ${issue.body}`;
       await invokeClaudeCode(prompt, { yolo: options.yolo, verbose: !options.silent, model: options.model });
     }
   }
-  try {
-    const pr = getCurrentBranchPr();
-    if (pr) {
-      if (commentUrl) {
-        try {
-          editComment(commentUrl, `Working on this in PR #${pr.number} \u2014 ${pr.url}`);
-        } catch (err) {
-          process.stderr.write(`Warning: could not update issue comment: ${err.message}
-`);
-        }
-      }
-      try {
-        addClosesRefToPr(pr.number, issue.number);
-      } catch (err) {
-        process.stderr.write(`Warning: could not add Closes #${issue.number} to PR: ${err.message}
+  linkPrToIssue(issue.number, commentUrl, options.askCopilotReview === true);
+});
+function resolveExecutor(requested) {
+  const executor = requested.toLowerCase();
+  if (executor !== "claude" && executor !== "codex") {
+    process.stderr.write(`Error: --with must be 'claude' or 'codex', got '${requested}'.
 `);
-      }
-      if (options.askCopilotReview) {
-        try {
-          addCopilotReviewer(pr.number);
-        } catch (err) {
-          process.stderr.write(`Warning: could not request Copilot review: ${err.message}
+    process.exit(1);
+  }
+  return executor;
+}
+function warnOnFailure(what, action) {
+  try {
+    action();
+  } catch (err) {
+    process.stderr.write(`Warning: could not ${what}: ${err.message}
 `);
-        }
-      }
-    }
+  }
+}
+function linkPrToIssue(issueNumber, commentUrl, askCopilotReview) {
+  let pr;
+  try {
+    pr = getCurrentBranchPr();
   } catch (err) {
     process.stderr.write(`Warning: could not detect current branch PR: ${err.message}
 `);
+    return;
   }
-});
+  if (!pr) return;
+  if (commentUrl) {
+    warnOnFailure("update issue comment", () => {
+      editComment(commentUrl, `Working on this in PR #${pr.number} \u2014 ${pr.url}`);
+    });
+  }
+  warnOnFailure(`add Closes #${String(issueNumber)} to PR`, () => {
+    addClosesRefToPr(pr.number, issueNumber);
+  });
+  if (askCopilotReview) {
+    warnOnFailure("request Copilot review", () => {
+      addCopilotReviewer(pr.number);
+    });
+  }
+}
 
 // src/commands/execute.ts
 import { readFileSync as readFileSync2 } from "fs";
@@ -2132,7 +2156,7 @@ function pluralSuffix(count) {
 function addAiOptions(cmd) {
   return cmd.requiredOption("--with <executor>", "Executor to use: claude or codex").option("--model <string>", "Model identifier to pass to the executor").option("--silent", "Suppress step-by-step Claude output; show only the final summary").option("--push", "Append instruction to commit and push changes after the AI finishes");
 }
-function resolveExecutor(withOption) {
+function resolveExecutor2(withOption) {
   const executor = withOption.toLowerCase();
   if (executor !== "claude" && executor !== "codex") {
     process.stderr.write(`Error: --with must be 'claude' or 'codex', got '${withOption}'.
@@ -2153,7 +2177,7 @@ var executeSonarCmd = addAiOptions(
     "Check the current branch for a SonarCloud analysis and invoke the AI with the Sonar prompt and analysis URL"
   )
 ).action(async (options) => {
-  const executor = resolveExecutor(options.with);
+  const executor = resolveExecutor2(options.with);
   let branch;
   try {
     branch = getCurrentBranch();
@@ -2207,7 +2231,7 @@ var executeFixCommentsCmd = addAiOptions(
     "Fetch open review comments on the current PR and invoke the AI with the Fix-Comments prompt"
   )
 ).action(async (options) => {
-  const executor = resolveExecutor(options.with);
+  const executor = resolveExecutor2(options.with);
   const result = resolveCurrentBranchComments();
   if (!result.ok) {
     if (result.kind === "error") {
@@ -2251,7 +2275,7 @@ var executeCheckIssueCmd = addAiOptions(
     "Check a GitHub issue for a new message from an allowed user since the last agent run and invoke the AI with the issue conversation"
   ).argument("<issue-number>", "GitHub issue number to check")
 ).option("--force", "Skip the new-message check and invoke the AI directly").action(async (issueNumberArg, options) => {
-  const executor = resolveExecutor(options.with);
+  const executor = resolveExecutor2(options.with);
   const issueNumber = Number.parseInt(issueNumberArg, 10);
   if (Number.isNaN(issueNumber) || issueNumber <= 0) {
     process.stderr.write(`Error: <issue-number> must be a positive integer (got '${issueNumberArg}').
@@ -3401,26 +3425,28 @@ function validateDoWorkConfig(section) {
   validateOptionalString(section, "baseBranch", "doWork.baseBranch");
   validateOptionalInt(section, "maxRunsPerTick", "doWork.maxRunsPerTick", 0, "a non-negative integer (0 = unlimited)");
   validateOptionalInt(section, "lockStaleMinutes", "doWork.lockStaleMinutes", 1, "a positive integer");
-  const protectedBranches = section["protectedBranches"];
-  if (protectedBranches !== void 0 && protectedBranches !== null) {
-    if (!Array.isArray(protectedBranches) || protectedBranches.some((b) => typeof b !== "string" || b.trim().length === 0)) {
-      fail("doWork.protectedBranches must be an array of non-empty strings.");
-    }
+  validateProtectedBranches(section["protectedBranches"]);
+  validateSettingContainer(section["models"], "models", ["claude", "codex"]);
+  validateSettingContainer(section["prompts"], "prompts", ["issueDiscuss", "prWork"]);
+}
+function validateProtectedBranches(value) {
+  if (value === void 0 || value === null) return;
+  const isNonEmptyString = (b) => typeof b === "string" && b.trim().length > 0;
+  if (!Array.isArray(value) || !value.every(isNonEmptyString)) {
+    fail("doWork.protectedBranches must be an array of non-empty strings.");
   }
-  for (const container of ["models", "prompts"]) {
-    const value = section[container];
-    if (value === void 0 || value === null) continue;
-    if (!isPlainObject(value)) {
-      fail(`doWork.${container} must be an object, got ${JSON.stringify(value)}.`);
-    }
-    const keys = container === "models" ? ["claude", "codex"] : ["issueDiscuss", "prWork"];
-    for (const key of keys) {
-      validateOptionalString(value, key, `doWork.${container}.${key}`);
-    }
-    for (const key of Object.keys(value)) {
-      if (!keys.includes(key)) {
-        fail(`doWork.${container}.${key} is not a recognised setting; expected one of: ${keys.join(", ")}.`);
-      }
+}
+function validateSettingContainer(value, container, keys) {
+  if (value === void 0 || value === null) return;
+  if (!isPlainObject(value)) {
+    fail(`doWork.${container} must be an object, got ${JSON.stringify(value)}.`);
+  }
+  for (const key of keys) {
+    validateOptionalString(value, key, `doWork.${container}.${key}`);
+  }
+  for (const key of Object.keys(value)) {
+    if (!keys.includes(key)) {
+      fail(`doWork.${container}.${key} is not a recognised setting; expected one of: ${keys.join(", ")}.`);
     }
   }
 }
@@ -3736,10 +3762,9 @@ async function processItem(planned, settings, silent) {
     baseBranch: settings.baseBranch,
     frame: settings.prompts[item.turn]
   });
-  const MAX_PROMPT_BYTES = 96 * 1024;
-  const promptBytes = Buffer.byteLength(prompt, "utf8");
-  if (promptBytes > MAX_PROMPT_BYTES) {
-    const detail = `the composed prompt is ${String(Math.round(promptBytes / 1024))} KiB, over the ${String(MAX_PROMPT_BYTES / 1024)} KiB limit for a single command-line argument. The conversation is too long to hand to the executor this way.`;
+  const oversized = describeOversizedPrompt(prompt);
+  if (oversized !== null) {
+    const detail = oversized;
     progress(`  failed: ${detail}
 `);
     inFlightMarker = null;
@@ -3765,6 +3790,16 @@ async function processItem(planned, settings, silent) {
   const reconciled = reconcileMarker(item, marker, settings.participants, watermark, runError);
   progress(`  ${reconciled.detail}
 `);
+  const outcome = adjustOutcome(reconciled, item, settings, buriedByNote);
+  return { ...base, outcome: outcome.outcome, detail: outcome.detail, ranExecutor };
+}
+function describeOversizedPrompt(prompt) {
+  const MAX_PROMPT_BYTES = 96 * 1024;
+  const promptBytes = Buffer.byteLength(prompt, "utf8");
+  if (promptBytes <= MAX_PROMPT_BYTES) return null;
+  return `the composed prompt is ${String(Math.round(promptBytes / 1024))} KiB, over the ${String(MAX_PROMPT_BYTES / 1024)} KiB limit for a single command-line argument. The conversation is too long to hand to the executor this way.`;
+}
+function adjustOutcome(reconciled, item, settings, buriedByNote) {
   let outcome = reconciled;
   if (buriedByNote > 0 && outcome.outcome === "answered") {
     outcome = {
@@ -3773,18 +3808,17 @@ async function processItem(planned, settings, silent) {
       reason: "flagged"
     };
   }
-  if (item.turn === "issue-discuss") {
-    const linked = repairIssueLink(item, settings.baseBranch);
-    if (linked && outcome.reason === "no-answer") {
-      outcome = {
-        outcome: "answered",
-        detail: "opened a pull request (no issue comment)",
-        reason: "answered"
-      };
-      progress("  a pull request was opened, so the turn is counted as answered.\n");
-    }
+  if (item.turn !== "issue-discuss") return outcome;
+  const linked = repairIssueLink(item, settings.baseBranch);
+  if (linked && outcome.reason === "no-answer") {
+    progress("  a pull request was opened, so the turn is counted as answered.\n");
+    return {
+      outcome: "answered",
+      detail: "opened a pull request (no issue comment)",
+      reason: "answered"
+    };
   }
-  return { ...base, outcome: outcome.outcome, detail: outcome.detail, ranExecutor };
+  return outcome;
 }
 function summarize(reports) {
   out("\nTick summary:\n");
@@ -3811,25 +3845,7 @@ var doWorkCommand = new Command6("do-work").description(
   }
   const lock = acquireRunLock("do-work", settings.lockStaleMinutes);
   if (!lock.ok) {
-    const held = lock.heldBy;
-    const sentence = `Another automata instance is already running here (pid ${String(held.pid)} on ${held.host}, started ${held.startedAt}, command ${held.command}). Doing nothing.
-`;
-    const suspectSentence = lock.suspect ? `Warning: that lock has been held longer than ${String(settings.lockStaleMinutes)} minutes. If no tick is really running, its process id was probably reused; remove ${RUN_LOCK_RELATIVE_PATH} once you have confirmed that.
-` : "";
-    const exitCode2 = lock.suspect ? 2 : 0;
-    if (options.json === true) {
-      progress(sentence + suspectSentence);
-      out(
-        JSON.stringify(
-          { lockHeld: true, suspect: lock.suspect, heldBy: held, plan: [], items: [], exitCode: exitCode2 },
-          null,
-          2
-        ) + "\n"
-      );
-    } else {
-      out(sentence);
-      if (suspectSentence) progress(suspectSentence);
-    }
+    const exitCode2 = reportLockHeld(lock, settings, options);
     if (exitCode2 !== 0) process.exit(exitCode2);
     return;
   }
@@ -3839,18 +3855,7 @@ var doWorkCommand = new Command6("do-work").description(
     if (shuttingDown) return;
     shuttingDown = true;
     progress("\nInterrupted: stopping the executor before releasing the run lock\u2026\n");
-    const pending = inFlightMarker;
-    if (pending !== null) {
-      try {
-        updateMarker(
-          pending.marker,
-          `automata do-work: this run was interrupted before it finished, so no answer was produced. The branch \`${pending.item.branch}\` may have been changed. Reply here to have another attempt made.`
-        );
-      } catch (err) {
-        progress(`Warning: could not update the in-flight marker: ${err.message}
-`);
-      }
-    }
+    explainInterruptedMarker();
     void terminateTrackedChildren().then((allExited) => {
       if (allExited) {
         handle.release();
@@ -3878,6 +3883,37 @@ var doWorkCommand = new Command6("do-work").description(
   }
   if (exitCode !== 0) process.exit(exitCode);
 });
+function reportLockHeld(lock, settings, options) {
+  const held = lock.heldBy;
+  const sentence = `Another automata instance is already running here (pid ${String(held.pid)} on ${held.host}, started ${held.startedAt}, command ${held.command}). Doing nothing.
+`;
+  const suspectSentence = lock.suspect ? `Warning: that lock has been held longer than ${String(settings.lockStaleMinutes)} minutes. If no tick is really running, its process id was probably reused; remove ${RUN_LOCK_RELATIVE_PATH} once you have confirmed that.
+` : "";
+  const exitCode = lock.suspect ? 2 : 0;
+  if (options.json === true) {
+    progress(sentence + suspectSentence);
+    out(
+      JSON.stringify({ lockHeld: true, suspect: lock.suspect, heldBy: held, plan: [], items: [], exitCode }, null, 2) + "\n"
+    );
+  } else {
+    out(sentence);
+    if (suspectSentence) progress(suspectSentence);
+  }
+  return exitCode;
+}
+function explainInterruptedMarker() {
+  const pending = inFlightMarker;
+  if (pending === null) return;
+  try {
+    updateMarker(
+      pending.marker,
+      `automata do-work: this run was interrupted before it finished, so no answer was produced. The branch \`${pending.item.branch}\` may have been changed. Reply here to have another attempt made.`
+    );
+  } catch (err) {
+    progress(`Warning: could not update the in-flight marker: ${err.message}
+`);
+  }
+}
 async function runTick(settings, options) {
   const issues = discoverIssues(settings);
   const linkMap = getOpenPrLinkMap();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "automata-cli",
-  "version": "0.6.0-develop.225",
+  "version": "0.6.0-develop.230",
   "description": "Automata CLI tool",
   "type": "module",
   "bin": {
@@ -16,6 +16,7 @@
     "typecheck": "tsc --noEmit",
     "format": "prettier --check src/",
     "test": "npm run build && vitest run tests/unit",
+    "test:unit": "vitest run tests/unit",
     "test:integration": "npm run build && vitest run tests/integration"
   },
   "repository": {