harnessed 3.9.24 → 3.9.26

package/dist/cli.mjs

@@ -1231,7 +1231,7 @@ var init_auto_install = __esm({
 
 // package.json
 var package_default = {
-  version: "3.9.24"};
+  version: "3.9.26"};
 
 // src/manifest/errors.ts
 function instancePathToKeyPath(instancePath) {
@@ -4027,11 +4027,91 @@ async function loadRolePrompts(workflowsDir) {
   const doc = parse(raw);
   return doc?.prompts ?? {};
 }
-function generateCommandFile(name, prompt, _capabilities, _installedPlugins, _installedUserSkills) {
-  const isMaster = prompt.is_master === true;
-  const argHint = isMaster ? "[task description]" : "[requirement text or omit]";
+var INTERACTIVE_COMMANDS = /* @__PURE__ */ new Set([
+  "discuss",
+  "discuss-strategic",
+  "discuss-phase",
+  "discuss-subtask",
+  "task-clarify"
+]);
+var HYBRID_COMMANDS = /* @__PURE__ */ new Set(["auto", "task"]);
+var MARKER = `<!-- harnessed-generated:v3.4.4 -->`;
+function buildInteractiveBody(name, prompt) {
+  return [
+    `# /${name}`,
+    ``,
+    prompt.description,
+    ``,
+    `## How to run (interactive \u2014 in THIS session)`,
+    ``,
+    `Clarification requires real user dialogue. Spawned subagents cannot ask the user questions, so run this stage directly in this session \u2014 do NOT spawn \`harnessed run ${name}\`.`,
+    ``,
+    `1. Evaluate the clarification criteria for "$ARGUMENTS":`,
+    `   - Strategic layer: new feature / new milestone / unclear business scope \u2192 run gstack \`/office-hours\` + \`/plan-ceo-review\``,
+    `   - Phase layer: \u22652 open implementation decisions / unclear cross-phase API contract \u2192 run GSD \`/gsd-discuss-phase\``,
+    `   - Subtask layer: \u22652 distinct approaches / core algorithm / API contract design / high error cost \u2192 run superpowers brainstorming`,
+    `2. For each layer that fires, hold the dialogue with the user (use AskUserQuestion for option-style decisions). Lock every open decision.`,
+    `3. Skip layers that don't fire \u2014 state which were skipped and why (transparent skip).`,
+    `4. Persist locked decisions to \`.planning/\` via planning-with-files (\`findings.md\` / \`task_plan.md\`).`,
+    ``,
+    `Output: a locked spec the execution stages can consume without further user input.`,
+    ``,
+    `## Notes`,
+    ``,
+    `- This file is generated by \`harnessed setup\`. Re-run \`harnessed setup\` after a harnessed upgrade to refresh.`,
+    `- The downstream execution command (\`/plan\` \u2192 \`/task\` \u2192 \`/verify\`) embeds the locked spec in its task text.`,
+    ``,
+    MARKER,
+    ``
+  ].join("\n");
+}
+function buildHybridBody(name, prompt) {
+  const chain = name === "auto" ? [
+    `3. Then run the execution stages in order via the Bash tool (each stage's output informs the next):`,
+    ``,
+    "```bash",
+    `echo "<locked spec + $ARGUMENTS>" | harnessed run plan --task-stdin`,
+    `echo "<locked spec + plan output>" | harnessed run task --task-stdin --skip-sub clarify`,
+    `echo "<locked spec + task output>" | harnessed run verify --task-stdin`,
+    `echo "<summary of all stages>" | harnessed run retro --task-stdin`,
+    "```",
+    ``,
+    `   Do NOT run \`harnessed run auto\` \u2014 that would re-spawn the discuss stage headlessly, defeating the interactive clarification you just did.`,
+    `   \`--skip-sub clarify\` tells the task master that clarification is already done.`
+  ] : [
+    `3. Then spawn the execution subs via the Bash tool:`,
+    ``,
+    "```bash",
+    `echo "<locked spec + $ARGUMENTS>" | harnessed run task --task-stdin --skip-sub clarify`,
+    "```",
+    ``,
+    `   \`--skip-sub clarify\` tells the task master that clarification is already done in this session.`
+  ];
+  return [
+    `# /${name}`,
+    ``,
+    prompt.description,
+    ``,
+    `## How to run (hybrid \u2014 clarify in THIS session, then spawn execution)`,
+    ``,
+    `1. FIRST clarify interactively in this session (spawned subagents cannot ask the user questions):`,
+    `   - Evaluate the subtask clarification criteria for "$ARGUMENTS": \u22652 distinct approaches / core algorithm / API contract design / high error cost \u2192 brainstorm with the user (AskUserQuestion for option-style decisions) and lock every open decision.`,
+    `   - If criteria don't fire (CRUD / known pattern / single obvious implementation), skip with a one-line transparent declaration.`,
+    `2. Embed the locked decisions into the task text passed downstream.`,
+    ...chain,
+    ``,
+    `## Notes`,
+    ``,
+    `- This file is generated by \`harnessed setup\`. Re-run \`harnessed setup\` after a harnessed upgrade to refresh.`,
+    `- Execution stages (code / test / deliver / verify) spawn headless subagents \u2014 appropriate since they need no user dialogue.`,
+    ``,
+    MARKER,
+    ``
+  ].join("\n");
+}
+function buildSpawnBody(name, prompt) {
   const stagedNote = name === "auto" ? "\n- For stage-by-stage review, append `--staged` (pauses between stages for user review)." : "";
-  const body = [
+  return [
     `# /${name}`,
     ``,
     prompt.description,
@@ -4054,9 +4134,21 @@ function generateCommandFile(name, prompt, _capabilities, _installedPlugins, _in
     `- The sister \`~/.claude/skills/${name}/SKILL.md\` is the Skill-tool entry point (Claude loads it when triggers match \`trigger_phrases:\`). Both files invoke the same \`harnessed run ${name}\` Bash command.${stagedNote}`,
     `- Workflow runtime: \`src/workflow/run.ts\` walks \`workflows/${nameToYamlHintPath(name)}\` with disciplines + judgments + master orchestration applied per the yaml \`delegates_to[]\` + \`gate\` clauses.`,
     ``,
-    `<!-- harnessed-generated:v3.4.4 -->`,
+    MARKER,
     ``
   ].join("\n");
+}
+function generateCommandFile(name, prompt, _capabilities, _installedPlugins, _installedUserSkills) {
+  const isMaster = prompt.is_master === true;
+  const argHint = isMaster ? "[task description]" : "[requirement text or omit]";
+  let body;
+  if (INTERACTIVE_COMMANDS.has(name)) {
+    body = buildInteractiveBody(name, prompt);
+  } else if (HYBRID_COMMANDS.has(name)) {
+    body = buildHybridBody(name, prompt);
+  } else {
+    body = buildSpawnBody(name, prompt);
+  }
   const warnings = [];
   const frontmatter = [
     "---",
@@ -5042,8 +5134,17 @@ async function runMasterOrchestrator(masterName, context, packageRoot, spawnDriv
     }
     effectiveOpts = pre.opts;
   }
+  const skipSubs = new Set(Array.isArray(context.skip_subs) ? context.skip_subs : []);
   const gateEvalled = [];
   for (const clause of master.delegates_to) {
+    if (skipSubs.has(clause.sub)) {
+      gateEvalled.push({
+        clause,
+        passes: false,
+        reason: `skipped via skip_subs (done interactively in main session)`
+      });
+      continue;
+    }
     if (!clause.gate) {
       gateEvalled.push({ clause, passes: true });
       continue;
@@ -5449,7 +5550,10 @@ function registerRun(program2) {
   ).option("--model <model>", "subagent model: 'haiku' | 'sonnet' | 'opus'").option(
     "--dry-run",
     "validate yaml load + walk runtime without spawning (Phase 1 default for verification)"
-  ).option("--staged", "/auto super-master: pause between stages for user review").option("--list", "print all known workflow names and exit").action(async (name, raw) => {
+  ).option("--staged", "/auto super-master: pause between stages for user review").option(
+    "--skip-sub <names>",
+    "comma-separated sub names to skip (work already done interactively in main session, e.g. clarify)"
+  ).option("--list", "print all known workflow names and exit").action(async (name, raw) => {
     if (raw.list) {
       const names = await listWorkflowNames(WORKFLOWS_DIR);
       for (const n of names) console.log(n);
@@ -5541,6 +5645,10 @@ function registerRun(program2) {
       ...raw.model ? { modelOverride: raw.model } : {},
       ...raw.maxIterations ? { maxIterations: raw.maxIterations } : {},
       ...raw.staged ? { staged: true } : {},
+      // v3.9.26 Option A — skip subs done interactively in main session.
+      ...typeof raw.skipSub === "string" && raw.skipSub.length > 0 ? {
+        skip_subs: raw.skipSub.split(",").map((s) => s.trim()).filter((s) => s.length > 0)
+      } : {},
       ...matchedTriggers.length > 0 ? { user_overrides: matchedTriggers } : {}
     };
     if (raw.dryRun) {
@@ -6279,20 +6387,19 @@ function printGrouped(b, prefix = "") {
   const GROUP_ORDER = ["mcp-tool", "command", "cli-binary", "other"];
   const groupOf = (n) => b.componentTypes[n] ?? "other";
   const buckets = {};
+  const push = (group, entry) => {
+    if (!buckets[group]) buckets[group] = [];
+    buckets[group].push(entry);
+  };
   for (const n of b.failed) {
     const m = n.match(/^([^:]+):/);
     const baseName = m?.[1] ?? n;
-    (buckets[groupOf(baseName)] ??= []).push({ status: "failed", name: n });
+    push(groupOf(baseName), { status: "failed", name: n });
   }
-  for (const n of b.installed) (buckets[groupOf(n)] ??= []).push({ status: "installed", name: n });
+  for (const n of b.installed) push(groupOf(n), { status: "installed", name: n });
   for (const s of b.skipped)
-    (buckets[groupOf(s.name)] ??= []).push({
-      status: "skipped",
-      name: s.name,
-      suffix: ` \u2014 ${s.reason}`
-    });
-  for (const n of b.alreadyInstalled)
-    (buckets[groupOf(n)] ??= []).push({ status: "already-installed", name: n });
+    push(groupOf(s.name), { status: "skipped", name: s.name, suffix: ` \u2014 ${s.reason}` });
+  for (const n of b.alreadyInstalled) push(groupOf(n), { status: "already-installed", name: n });
   for (const g of GROUP_ORDER) {
     const entries = buckets[g];
     if (!entries || entries.length === 0) continue;
@@ -6788,7 +6895,8 @@ async function removeSettingsEnv(settingsPath3) {
   if (!changed) return false;
   if (Object.keys(env).length === 0) delete data.env;
   else data.env = env;
-  await writeFile(settingsPath3, JSON.stringify(data, null, 2) + "\n", "utf8");
+  await writeFile(settingsPath3, `${JSON.stringify(data, null, 2)}
+`, "utf8");
   return true;
 }
 async function runUnifiedUninstall(home, dryRun) {