@kody-ade/kody-engine 0.3.51 → 0.3.53

package/dist/bin/kody.js

@@ -3,7 +3,7 @@
 // package.json
 var package_default = {
   name: "@kody-ade/kody-engine",
-  version: "0.3.51",
+  version: "0.3.53",
   description: "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   license: "MIT",
   type: "module",
@@ -1313,6 +1313,12 @@ function parseScriptList(p, key, raw) {
     if (r.with && typeof r.with === "object") {
       entry.with = r.with;
     }
+    if (typeof r.timeoutSec === "number" && r.timeoutSec > 0) {
+      if (!hasShell) {
+        throw new ProfileError(p, `scripts.${key}[${i}] "timeoutSec" only applies to shell entries`);
+      }
+      entry.timeoutSec = r.timeoutSec;
+    }
     out.push(entry);
   }
   return out;
@@ -1934,7 +1940,10 @@ function parseAgentResult(finalText) {
       priorArt: "",
       failureReason: "agent produced no final message"
     };
-  const failedMatch = text.match(/(?:^|\n)\s*FAILED\s*:\s*(.+?)\s*$/s);
+  const MARKDOWN_PREFIX = "[\\s>*_#`~\\-]*";
+  const FAILED_RE = new RegExp(`(?:^|\\n)${MARKDOWN_PREFIX}FAILED${MARKDOWN_PREFIX}\\s*:\\s*(.+?)\\s*$`, "is");
+  const DONE_RE = new RegExp(`(?:^|\\n)${MARKDOWN_PREFIX}DONE\\b`, "i");
+  const failedMatch = text.match(FAILED_RE);
   if (failedMatch) {
     return {
       done: false,
@@ -1943,11 +1952,11 @@ function parseAgentResult(finalText) {
       feedbackActions: "",
       planDeviations: "",
       priorArt: "",
-      failureReason: failedMatch[1].trim()
+      failureReason: stripMarkdownEmphasis(failedMatch[1])
     };
   }
-  const hasDoneMarker = /(^|\n)\s*DONE\b/i.test(text);
-  const hasCommitMsg = /^[ \t]*COMMIT_MSG\s*:/im.test(text);
+  const hasDoneMarker = DONE_RE.test(text);
+  const hasCommitMsg = /^[\s>*_#`~\-]*COMMIT_MSG\s*:/im.test(text);
   if (!hasDoneMarker && !hasCommitMsg) {
     return {
       done: false,
@@ -1959,8 +1968,8 @@ function parseAgentResult(finalText) {
       failureReason: "no DONE or FAILED marker in agent output"
     };
   }
-  const commitMatch = text.match(/^[ \t]*COMMIT_MSG\s*:\s*(.+)$/im);
-  const commitMessage = commitMatch ? commitMatch[1].trim() : "";
+  const commitMatch = text.match(/^[\s>*_#`~\-]*COMMIT_MSG[\s>*_#`~\-]*\s*:\s*(.+)$/im);
+  const commitMessage = commitMatch ? stripMarkdownEmphasis(commitMatch[1]) : "";
   const feedbackActions = extractBlock(
     text,
     /(?:^|\n)[ \t]*FEEDBACK_ACTIONS\s*:[ \t]*\n/i,
@@ -1986,6 +1995,9 @@ function parseAgentResult(finalText) {
   }
   return { done: true, commitMessage, prSummary, feedbackActions, planDeviations, priorArt, failureReason: "" };
 }
+function stripMarkdownEmphasis(s) {
+  return s.trim().replace(/^[*_`~]+|[*_`~]+$/g, "").trim();
+}
 function extractBlock(text, startMarker, endMarker) {
   const startIdx = text.search(startMarker);
   if (startIdx === -1) return "";
@@ -6725,7 +6737,17 @@ function finish(out) {
 `);
   return out;
 }
-var SHELL_TIMEOUT_MS = 3e5;
+var DEFAULT_SHELL_TIMEOUT_MS = 3e5;
+function resolveShellTimeoutMs(entry) {
+  if (typeof entry.timeoutSec === "number" && entry.timeoutSec > 0) {
+    return Math.floor(entry.timeoutSec * 1e3);
+  }
+  const envSec = Number(process.env.KODY_SHELL_TIMEOUT_SEC);
+  if (Number.isFinite(envSec) && envSec > 0) {
+    return Math.floor(envSec * 1e3);
+  }
+  return DEFAULT_SHELL_TIMEOUT_MS;
+}
 function runShellEntry(entry, ctx, profile) {
   const shellName = entry.shell;
   const shellPath = path21.join(profile.dir, shellName);
@@ -6744,12 +6766,13 @@ function runShellEntry(entry, ctx, profile) {
   for (const [k, v] of flattenConfig(ctx.config)) {
     env[`KODY_CFG_${k}`] = v;
   }
+  const timeoutMs = resolveShellTimeoutMs(entry);
   const r = spawnSync("bash", [shellPath, ...positional], {
     cwd: ctx.cwd,
     encoding: "utf-8",
     env,
     stdio: ["pipe", "pipe", "pipe"],
-    timeout: SHELL_TIMEOUT_MS
+    timeout: timeoutMs
   });
   const stdout = r.stdout ?? "";
   const stderr = r.stderr ?? "";
@@ -6763,6 +6786,18 @@ function runShellEntry(entry, ctx, profile) {
   if (prUrlMatch?.[1]) ctx.output.prUrl = prUrlMatch[1].trim();
   const reasonMatch = stdout.match(/^KODY_REASON=(.+)$/m);
   if (reasonMatch?.[1]) ctx.output.reason = reasonMatch[1].trim();
+  const timedOut = r.status === null && r.signal !== null;
+  if (timedOut) {
+    ctx.skipAgent = true;
+    const seconds = Math.round(timeoutMs / 1e3);
+    if (ctx.output.exitCode === void 0 || ctx.output.exitCode === 0) {
+      ctx.output.exitCode = 124;
+    }
+    if (!ctx.output.reason) {
+      ctx.output.reason = `shell '${shellName}' timed out after ${seconds}s (signal=${r.signal})`;
+    }
+    return;
+  }
   const exit = r.status ?? -1;
   if (exit !== 0) {
     ctx.skipAgent = true;

package/dist/executables/types.ts

@@ -185,6 +185,14 @@ export interface ScriptEntry {
    * dispatcher script can be reused with different `next` targets.
    */
   with?: Record<string, string | number | boolean>
+  /**
+   * Optional shell-script timeout in seconds. Only honored on `shell` entries.
+   * Falls back to `KODY_SHELL_TIMEOUT_SEC` env var, then the 300s default.
+   * Long-running shells (release publish, large repo verify) should declare
+   * a higher value rather than relying on the default and getting SIGKILLed
+   * with an opaque "exited -1".
+   */
+  timeoutSec?: number
 }
 
 export interface OutputContract {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.3.51",
+  "version": "0.3.53",
   "description": "kody — autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   "license": "MIT",
   "type": "module",