@kody-ade/kody-engine 0.4.656 → 0.4.658

package/dist/bin/kody.js

@@ -15,7 +15,7 @@ var init_package = __esm({
   "package.json"() {
     package_default = {
       name: "@kody-ade/kody-engine",
-      version: "0.4.656",
+      version: "0.4.658",
       description: "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative implementation profiles.",
       license: "MIT",
       repository: {
@@ -5023,6 +5023,12 @@ async function runAgent(opts) {
       outcomeKind = "model_error";
       errorMessage2 = "Claude Code reported it is not logged in; refusing to mark agent run successful";
     }
+    const providerApiError = outcome === "completed" && tokens.output === 0 && /^API Error:\s*\d+\b/i.test(finalText.trim());
+    if (providerApiError) {
+      outcome = "failed";
+      outcomeKind = "model_error";
+      errorMessage2 = finalText.trim();
+    }
     if (outcome === "completed" && !sawMutatingTool) {
       const hasUsefulSuccess = tokens.output > 0 || finalText !== "" || Boolean(getSubmitted?.());
       if (!hasUsefulSuccess) {
@@ -9519,12 +9525,12 @@ function isSafeConfigActivationChange(before, after) {
   const beforeCompanyRest = { ...beforeCompany };
   const afterCompanyRest = { ...afterCompany };
   for (const field of ACTIVATION_FIELDS) {
-    const previous = beforeCompany[field];
+    const previous = beforeCompany[field] === void 0 ? [] : beforeCompany[field];
     const next = afterCompany[field];
     delete beforeCompanyRest[field];
     delete afterCompanyRest[field];
-    if (previous === void 0 && next === void 0) continue;
-    if (!Array.isArray(previous ?? []) || !Array.isArray(next)) return false;
+    if (beforeCompany[field] === void 0 && next === void 0) continue;
+    if (!Array.isArray(previous) || !Array.isArray(next)) return false;
     if (!next.every((value) => typeof value === "string")) return false;
     if (!previous.every((value) => typeof value === "string")) return false;
     const nextValues = new Set(next);
@@ -9563,6 +9569,9 @@ function isTrustedConfigActivationChange(filePath, deliveryPathAllowlist, delive
   }
 }
 function listChangedFiles(cwd) {
+  return [...new Set(listChangedPathGroups(cwd).flat())];
+}
+function listChangedPathGroups(cwd) {
   const raw = execFileSync6("git", ["status", "--porcelain=v1", "-z", "--untracked-files=all"], {
     encoding: "utf-8",
     maxBuffer: GIT_MAX_BUFFER_BYTES,
@@ -9571,8 +9580,19 @@ function listChangedFiles(cwd) {
     stdio: ["pipe", "pipe", "pipe"]
   });
   if (!raw) return [];
-  const entries = raw.split("\0").filter((e) => e.length > 0);
-  return entries.map((e) => e.slice(3)).filter(Boolean);
+  const entries = raw.split("\0");
+  const groups = [];
+  for (let index = 0; index < entries.length - 1; index++) {
+    const entry = entries[index];
+    const paths = [entry.slice(3)];
+    if (/[RC]/.test(entry.slice(0, 2))) {
+      const source = entries[++index];
+      if (!source) throw new Error("git status returned an incomplete rename/copy record");
+      paths.push(source);
+    }
+    groups.push(paths);
+  }
+  return groups;
 }
 function listAllowlistedIgnoredFiles(deliveryPathAllowlist, cwd) {
   if (deliveryPathAllowlist.length === 0) return [];
@@ -9611,27 +9631,26 @@ function normalizeCommitMessage(raw) {
 function commitAndPush(branch, agentMessage, cwd, deliveryPathAllowlist = [], deliveryConfigAllowlist = {}) {
   const ignoredAllowedFiles = listAllowlistedIgnoredFiles(deliveryPathAllowlist, cwd);
   const ignoredAllowedSet = new Set(ignoredAllowedFiles);
-  const allChanged = [.../* @__PURE__ */ new Set([...listChangedFiles(cwd), ...ignoredAllowedFiles])];
-  const allowedFiles = allChanged.filter(
-    (f) => !isForbiddenPath(f, deliveryPathAllowlist) || isTrustedConfigActivationChange(f, deliveryPathAllowlist, deliveryConfigAllowlist, cwd)
-  );
-  const forbiddenFiles = allChanged.filter(
-    (f) => isForbiddenPath(f, deliveryPathAllowlist) && !isTrustedConfigActivationChange(f, deliveryPathAllowlist, deliveryConfigAllowlist, cwd)
+  const pathGroups = [...listChangedPathGroups(cwd), ...ignoredAllowedFiles.map((file) => [file])];
+  const blockedGroups = pathGroups.filter(
+    (group) => group.some(
+      (f) => isForbiddenPath(f, deliveryPathAllowlist) && !isTrustedConfigActivationChange(f, deliveryPathAllowlist, deliveryConfigAllowlist, cwd)
+    )
   );
+  const forbiddenFiles = [...new Set(blockedGroups.flat())];
+  const forbiddenSet = new Set(forbiddenFiles);
+  const allowedFiles = [...new Set(pathGroups.flat())].filter((file) => !forbiddenSet.has(file));
   const omittedFiles = forbiddenFiles.filter(isReportableDeliveryOmission);
   const mergeHeadExists = fs30.existsSync(path28.join(cwd ?? process.cwd(), ".git", "MERGE_HEAD"));
+  for (const f of forbiddenFiles) {
+    git(["--literal-pathspecs", "reset", "-q", "--", f], cwd);
+  }
   if (allowedFiles.length === 0 && !mergeHeadExists) {
     return { committed: false, pushed: false, sha: "", message: "", omittedFiles };
   }
-  for (const f of forbiddenFiles) {
-    try {
-      git(["reset", "-q", "--", f], cwd);
-    } catch {
-    }
-  }
   for (const f of allowedFiles) {
     try {
-      git(["add", ...ignoredAllowedSet.has(f) ? ["--force"] : [], "--", f], cwd);
+      git(["--literal-pathspecs", "add", ...ignoredAllowedSet.has(f) ? ["--force"] : [], "--", f], cwd);
     } catch {
     }
   }
@@ -9646,8 +9665,8 @@ function commitAndPush(branch, agentMessage, cwd, deliveryPathAllowlist = [], de
     }
     throw err;
   }
-  const sha = git(["rev-parse", "HEAD"], cwd).slice(0, 7);
   const pushResult = pushWithRetry({ cwd, branch, setUpstream: true });
+  const sha = git(["rev-parse", "HEAD"], cwd).slice(0, 7);
   if (pushResult.ok) {
     return { committed: true, pushed: true, sha, message, omittedFiles };
   }
@@ -25169,6 +25188,9 @@ function shouldRunCapabilityWorkflow(job, workflow, capabilityIdentity, selected
   return requestedImplementation === selectedImplementation || requestedImplementation === capabilityIdentity || requestedImplementation === job.action;
 }
 async function runCapabilityWorkflow(parent, workflow, capability, base, checkpoint) {
+  if (parent.workflowState?.status === "done") {
+    return { exitCode: 0, workflowState: structuredClone(parent.workflowState), usage: parent.workflowState.usage };
+  }
   const invalid = workflowError(workflow, base);
   if (invalid) {
     if (isGraphWorkflow(workflow)) {
@@ -25401,6 +25423,20 @@ async function runGraphCapabilityWorkflow(parent, workflow, capability, base, ch
       return { ...result, exitCode: 64, reason, workflowState: state };
     }
     const label = step.action ?? step.capability;
+    if (!shouldRunWorkflowStep(step, chainData)) {
+      process.stdout.write(
+        `\u2192 kody: workflow ${capability.slug} step ${index + 1}/${workflow.steps.length} \u2192 ${label} (skipped)
+
+`
+      );
+      result = { exitCode: 0 };
+      delete chainData.workflowLastResult;
+      delete chainData.workflowLastOutput;
+      delete chainData.workflowLastOutcome;
+      const terminal2 = await advanceGraphWorkflow(step, capability, state, result, chainData, checkpoint, true);
+      if (terminal2) return terminal2;
+      continue;
+    }
     if (step.approval === "required") {
       const approvalState = requireWorkflowStepApproval(state, step.id);
       state.status = approvalState.status;
@@ -25494,42 +25530,47 @@ async function runGraphCapabilityWorkflow(parent, workflow, capability, base, ch
       await checkpoint?.(state);
       return withWorkflowBoundaryEval(capability, { ...result, workflowState: state });
     }
-    if (!step.next || step.next.length === 0) {
-      return completeWorkflowAtTerminal(capability, state, result, checkpoint);
-    }
-    const resultConditionPaths = workflowResultConditionPaths(step.next);
-    if (resultConditionPaths.length > 0 && !result.capabilityResults?.at(-1)) {
-      const reason = `workflow step ${step.id} did not emit the structured result required by its conditions: ${resultConditionPaths.join(", ")}`;
-      state.status = "blocked";
-      state.blocker = reason;
-      await checkpoint?.(state);
-      return { ...result, exitCode: 64, reason, workflowState: state };
-    }
-    const transition = selectWorkflowTransition(step, chainData, state.transitionCounts);
-    if (!transition) {
-      const exhausted = exhaustedWorkflowTransitions(step, chainData, state.transitionCounts);
-      const reason = exhausted.length > 0 ? `workflow step ${step.id} reached iteration limit: ${exhausted.join(", ")}` : `workflow step ${step.id} has no available connection`;
-      state.status = "blocked";
-      state.blocker = reason;
-      await checkpoint?.(state);
-      return { ...result, exitCode: 64, reason, workflowState: state };
-    }
-    if (transition.maxIterations !== void 0) {
-      const key = `${step.id}->${transition.to}`;
-      state.transitionCounts[key] = (state.transitionCounts[key] ?? 0) + 1;
-    }
-    if (transition.to === "$end") {
-      return completeWorkflowAtTerminal(capability, state, result, checkpoint);
-    }
-    state.currentStepId = transition.to;
-    state.status = "running";
-    delete state.blocker;
-    await checkpoint?.(state);
+    const terminal = await advanceGraphWorkflow(step, capability, state, result, chainData, checkpoint);
+    if (terminal) return terminal;
   }
   state.status = "done";
   await checkpoint?.(state);
   return withWorkflowBoundaryEval(capability, { ...result, workflowState: state });
 }
+async function advanceGraphWorkflow(step, capability, state, result, chainData, checkpoint, skipped = false) {
+  if (!step.next || step.next.length === 0) {
+    return completeWorkflowAtTerminal(capability, state, result, checkpoint);
+  }
+  const resultConditionPaths = workflowResultConditionPaths(step.next);
+  if (!skipped && resultConditionPaths.length > 0 && !result.capabilityResults?.at(-1)) {
+    const reason = `workflow step ${step.id} did not emit the structured result required by its conditions: ${resultConditionPaths.join(", ")}`;
+    state.status = "blocked";
+    state.blocker = reason;
+    await checkpoint?.(state);
+    return { ...result, exitCode: 64, reason, workflowState: state };
+  }
+  const transition = selectWorkflowTransition(step, chainData, state.transitionCounts);
+  if (!transition) {
+    const exhausted = exhaustedWorkflowTransitions(step, chainData, state.transitionCounts);
+    const reason = exhausted.length > 0 ? `workflow step ${step.id} reached iteration limit: ${exhausted.join(", ")}` : `workflow step ${step.id} has no available connection`;
+    state.status = "blocked";
+    state.blocker = reason;
+    await checkpoint?.(state);
+    return { ...result, exitCode: 64, reason, workflowState: state };
+  }
+  if (transition.maxIterations !== void 0) {
+    const key = `${step.id}->${transition.to}`;
+    state.transitionCounts[key] = (state.transitionCounts[key] ?? 0) + 1;
+  }
+  if (transition.to === "$end") {
+    return completeWorkflowAtTerminal(capability, state, result, checkpoint);
+  }
+  state.currentStepId = transition.to;
+  state.status = "running";
+  delete state.blocker;
+  await checkpoint?.(state);
+  return null;
+}
 async function completeWorkflowAtTerminal(capability, state, output, checkpoint) {
   const result = output.capabilityResults?.at(-1);
   const customStatus = output.capabilityOutput && typeof output.capabilityOutput === "object" && !Array.isArray(output.capabilityOutput) && typeof output.capabilityOutput.status === "string" ? output.capabilityOutput.status : void 0;
@@ -31719,6 +31760,10 @@ function buildServer2(opts) {
         sendJson3(res, 400, { error: parsed.error });
         return;
       }
+      if (busy) {
+        sendJson3(res, 409, { error: "runner busy" });
+        return;
+      }
       busy = true;
       sendJson3(res, 202, { ok: true, jobId: parsed.job.jobId, started: true });
       void runJob2(parsed.job).catch((err) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.4.656",
+  "version": "0.4.658",
   "description": "kody — autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative implementation profiles.",
   "license": "MIT",
   "repository": {