executant 1.4.2 → 1.4.4

package/dist/index.js

@@ -1292,6 +1292,73 @@ async function runPass3Judge(description, workflow2) {
     return { pass: true, feedback: "", skipped: true };
   }
 }
+function isNumericSequence(arr) {
+  return arr.every((item, i) => item === String(i + 1));
+}
+function parseSeqCommand(cmd) {
+  const t = cmd.trim();
+  const shorthand = t.match(/^seq\s+(\d+)$/);
+  if (shorthand) return parseInt(shorthand[1], 10);
+  const explicit = t.match(/^seq\s+1\s+(\d+)$/);
+  if (explicit) return parseInt(explicit[1], 10);
+  return null;
+}
+function extractCountFromName(name) {
+  const m = name.match(/_of_(\d+)/);
+  return m ? parseInt(m[1], 10) : null;
+}
+function normalizeWorkflow(workflow2) {
+  const steps = workflow2.steps.map((step) => {
+    if (Array.isArray(step.forEach) && isNumericSequence(step.forEach)) {
+      const { forEach, ...rest } = step;
+      return { ...rest, repeat: forEach.length };
+    }
+    if (typeof step.forEach === "string") {
+      const n = parseSeqCommand(step.forEach);
+      if (n !== null) {
+        const { forEach, ...rest } = step;
+        return { ...rest, repeat: n };
+      }
+    }
+    const prompt = typeof step.prompt === "string" ? step.prompt : "";
+    if (prompt.includes("{{item}}") && step.forEach === void 0 && step.repeat === void 0) {
+      const n = extractCountFromName(step.name);
+      if (n !== null) return { ...step, repeat: n };
+    }
+    return step;
+  });
+  return { ...workflow2, steps: collapseSequentialSteps(steps) };
+}
+function collapseSequentialSteps(steps) {
+  const result = [];
+  let i = 0;
+  while (i < steps.length) {
+    const step = steps[i];
+    if (step.forEach !== void 0 || step.repeat !== void 0) {
+      result.push(step);
+      i++;
+      continue;
+    }
+    const m = step.name.match(/^(.+?)_1$/);
+    if (!m) {
+      result.push(step);
+      i++;
+      continue;
+    }
+    const prefix = m[1];
+    let n = 1;
+    while (i + n < steps.length && steps[i + n].name === `${prefix}_${n + 1}`) n++;
+    if (n < 2) {
+      result.push(step);
+      i++;
+      continue;
+    }
+    const { name, ...rest } = step;
+    result.push({ ...rest, name: `${prefix}_{{item}}`, repeat: n });
+    i += n;
+  }
+  return result;
+}
 async function* streamPlan(args) {
   const { description, taskFile } = args;
   const skipResearch = args.fast || isSimpleRequest(description);
@@ -1356,7 +1423,7 @@ async function* streamPlan(args) {
 ${basePrompt}` : basePrompt,
       allowedTools: [],
       permissionMode: "bypassPermissions",
-      model: "opus",
+      model: skipResearch ? "sonnet" : "opus",
       appendSystemPrompt: PLAN_SYSTEM_RULES,
       jsonSchema: WORKFLOW_JSON_SCHEMA
     };
@@ -1414,7 +1481,7 @@ ${issues}` };
     if (!judgeResult.pass) {
       yield { type: "plan:warn", message: `Judge rejected plan but retries exhausted: ${judgeResult.feedback}` };
     }
-    const { goal, vars, steps, ...rest } = zodResult.data;
+    const { goal, vars, steps, ...rest } = normalizeWorkflow(zodResult.data);
     const ordered = { goal, ...vars && { vars }, steps, ...rest };
     const yamlContent = dumpYaml(ordered, {
       lineWidth: -1,

package/dist/prompts/plan-decompose.txt

@@ -97,6 +97,11 @@ Before finalising your JSON, scan every `prompt` and `command` field you wrote.
 For each field, ask: "Does this contain a file path or directory path as a string literal?"
 If yes, extract it to `vars` and replace with `{{var_name}}`.
 
+**Pre-Output Self-Review — Repeat (MANDATORY):**
+Scan every `forEach` field you wrote.
+Ask: "Is this array just sequential numbers like `["1","2","3"]` with no meaningful items?"
+If yes, replace the entire `forEach` with `repeat: N` where N is the count. Sequential-number forEach arrays are ALWAYS wrong — they are a misuse of forEach and must be converted to `repeat: N`.
+
 ## When to Use Each Step Type
 
 **Use `prompt` steps (AI-assisted) for:**

package/dist/prompts/plan-judge.txt

@@ -59,6 +59,15 @@ Are all file paths declared in `vars`?
 - Do any `prompt` or `command` fields contain hardcoded file paths or directory paths?
 - Hardcoded paths in steps (not in `vars`) are a violation
 
+### 5. Repeat Misuse (if applicable)
+If the user's goal mentions "N times", "repeat N", "N iterations", or "N passes":
+
+- Does any step use `forEach` with a sequential numeric array like `["1","2","3","4","5"]`?
+- This is always wrong — `repeat: N` must be used instead of a numeric forEach array
+- Reject and require the offending step be converted to `repeat: N`
+- Does any single step's `prompt` describe doing something "N times" or "across N passes" inline, instead of using `repeat: N`? A step that says "do this 10 times" or "perform N passes" inside its prompt text rather than setting `repeat: N` is wrong — reject it and require it to be restructured as a single-pass prompt with `repeat: N` on the step
+- Are there N consecutive steps with names like `step_1`, `step_2`, `step_3`? Sequential named steps are always wrong when they do the same thing — reject and require a single step with `repeat: N`
+
 ## Output Format
 
 Respond with ONLY a JSON object in this exact shape:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "executant",
-  "version": "1.4.2",
+  "version": "1.4.4",
   "description": "Harness for YAML-defined workflows that enables stepping through Claude sessions and bash commands",
   "repository": {
     "type": "git",