@kody-ade/kody-engine 0.4.183 → 0.4.184

package/dist/bin/kody.js

@@ -1309,7 +1309,7 @@ var init_loadPriorArt = __esm({
 // package.json
 var package_default = {
   name: "@kody-ade/kody-engine",
-  version: "0.4.183",
+  version: "0.4.184",
   description: "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   license: "MIT",
   type: "module",
@@ -1348,7 +1348,6 @@ var package_default = {
     "@biomejs/biome": "^2.4.12",
     "@types/node": "^22.5.4",
     "@vitest/coverage-v8": "^4.1.4",
-    playwright: "^1.60.0",
     tsup: "^8.5.1",
     tsx: "^4.21.0",
     typescript: "~5.7.0",
@@ -1409,10 +1408,10 @@ var HttpSink = class {
         signal: AbortSignal.timeout(5e3)
       });
       if (!res.ok) {
-        this.logger.warn(`HttpSink POST ${url} \u2192 ${res.status}`);
+        this.logger.warn(`HttpSink POST ${redactUrl(url)} \u2192 ${res.status}`);
       }
     } catch (err) {
-      this.logger.warn(`HttpSink POST ${url} failed: ${err instanceof Error ? err.message : String(err)}`);
+      this.logger.warn(`HttpSink POST ${redactUrl(url)} failed: ${err instanceof Error ? err.message : String(err)}`);
     }
   }
 };
@@ -1425,6 +1424,9 @@ var TeeSink = class {
     await Promise.all(this.sinks.map((s) => s.emit(event)));
   }
 };
+function redactUrl(url) {
+  return url.replace(/([?&]token=)[^&]+/gi, "$1***");
+}
 function withSessionParam(baseUrl, sessionId) {
   const joiner = baseUrl.includes("?") ? "&" : "?";
   return `${baseUrl}${joiner}sessionId=${encodeURIComponent(sessionId)}`;
@@ -1654,7 +1656,11 @@ function loadConfig(projectDir = process.cwd()) {
     issueContext: parseIssueContext(raw.issueContext),
     testRequirements: parseTestRequirements(raw.testRequirements),
     defaultExecutable: typeof raw.defaultExecutable === "string" && raw.defaultExecutable.length > 0 ? raw.defaultExecutable : "run",
-    defaultPrExecutable: typeof raw.defaultPrExecutable === "string" && raw.defaultPrExecutable.length > 0 ? raw.defaultPrExecutable : "fix",
+    // No bare-PR default ships with the engine (there is no `fix` executable),
+    // so leave this unset unless a consumer configures a real one. Dispatch
+    // treats a missing/unknown default as "unrecognized" rather than routing
+    // to a phantom executable that would crash the executor at load time.
+    defaultPrExecutable: typeof raw.defaultPrExecutable === "string" && raw.defaultPrExecutable.length > 0 ? raw.defaultPrExecutable : void 0,
     aliases: mergeAliases(raw.aliases),
     classify: parseClassifyConfig(raw.classify),
     release: parseReleaseConfig(raw.release),
@@ -1717,9 +1723,7 @@ function parseJobsConfig(raw) {
   return Object.keys(out).length > 0 ? out : void 0;
 }
 var BUILTIN_ALIASES = {
-  build: "run",
-  orchestrate: "bug",
-  orchestrator: "bug"
+  build: "run"
 };
 function mergeAliases(raw) {
   const out = { ...BUILTIN_ALIASES };
@@ -1969,6 +1973,7 @@ async function runAgent(opts) {
     let ndjsonWriteFailed = false;
     let ndjsonWriteError;
     let sawMutatingTool = false;
+    let sawTerminalSuccess = false;
     try {
       const queryOptions = {
         model: opts.model.model,
@@ -2180,6 +2185,7 @@ async function runAgent(opts) {
           if (m.subtype === "success") {
             outcome = "completed";
             outcomeKind = "ok";
+            sawTerminalSuccess = true;
             const text = (typeof m.result === "string" ? m.result : "").trim();
             if (text) resultTexts.push(text);
           } else {
@@ -2190,9 +2196,13 @@ async function runAgent(opts) {
         }
       }
     } catch (e) {
-      outcome = "failed";
-      outcomeKind = "model_error";
-      errorMessage = e instanceof Error ? e.message : String(e);
+      if (sawTerminalSuccess) {
+        errorMessage = e instanceof Error ? e.message : String(e);
+      } else {
+        outcome = "failed";
+        outcomeKind = "model_error";
+        errorMessage = e instanceof Error ? e.message : String(e);
+      }
     } finally {
       try {
         fullLog.end();
@@ -3296,7 +3306,7 @@ function autoDispatch(opts) {
     }
   }
   if (!executable && !firstToken) {
-    executable = isPr ? opts?.config?.defaultPrExecutable ?? "fix" : opts?.config?.defaultExecutable ?? null;
+    executable = isPr ? opts?.config?.defaultPrExecutable ?? null : opts?.config?.defaultExecutable ?? null;
   }
   if (isBotAuthor && !consumedFirstToken) {
     process.stderr.write(
@@ -4533,7 +4543,7 @@ var FORBIDDEN_PATH_PREFIXES = [
   "build/"
 ];
 var ALLOWED_PATH_PREFIXES = [".kody/memory/", ".kody/tasks/"];
-var FORBIDDEN_PATH_EXACT = /* @__PURE__ */ new Set([".env", ".kody-pip-requirements.txt"]);
+var FORBIDDEN_PATH_EXACT = /* @__PURE__ */ new Set([".env", ".kody-pip-requirements.txt", "kody.config.json"]);
 var FORBIDDEN_PATH_SUFFIXES = [".log"];
 var CONVENTIONAL_PREFIXES = [
   "feat:",
@@ -7546,13 +7556,33 @@ var ContentsApiBackend = class {
     };
     if (typeof loaded.handle === "string") payload.sha = loaded.handle;
     ensureStateBranch(this.owner, this.repo, this.cwd);
-    gh(["api", "--method", "PUT", `/repos/${this.owner}/${this.repo}/contents/${loaded.path}`, "--input", "-"], {
+    try {
+      this.put(loaded.path, payload);
+    } catch (err) {
+      if (!isShaConflict(err)) throw err;
+      const current = this.load(slug);
+      if (!current.created && isStateUnchanged(current.state, next)) return false;
+      if (typeof current.handle === "string") payload.sha = current.handle;
+      else delete payload.sha;
+      process.stderr.write(
+        `[kody] jobState: concurrent write detected for ${slug}; reloaded SHA and retrying (last-write-wins)
+`
+      );
+      this.put(loaded.path, payload);
+    }
+    return true;
+  }
+  put(filePath, payload) {
+    gh(["api", "--method", "PUT", `/repos/${this.owner}/${this.repo}/contents/${filePath}`, "--input", "-"], {
       cwd: this.cwd,
       input: JSON.stringify(payload)
     });
-    return true;
   }
 };
+function isShaConflict(err) {
+  const msg = err instanceof Error ? err.message : String(err);
+  return /HTTP 409/i.test(msg) || /HTTP 422/i.test(msg) || /does not match|is at|but expected/i.test(msg);
+}
 
 // src/scripts/jobState/localFileBackend.ts
 import * as fs28 from "fs";
@@ -7655,7 +7685,9 @@ var LocalFileBackend = class {
     const absPath = path27.join(this.cwd, loaded.path);
     fs28.mkdirSync(path27.dirname(absPath), { recursive: true });
     const body = JSON.stringify(next, null, 2) + "\n";
-    fs28.writeFileSync(absPath, body, "utf-8");
+    const tmpPath = `${absPath}.${process.pid}.tmp`;
+    fs28.writeFileSync(tmpPath, body, "utf-8");
+    fs28.renameSync(tmpPath, absPath);
     return true;
   }
   cacheKeyPrefix() {
@@ -12858,8 +12890,6 @@ var saveTaskState = async (ctx, profile) => {
   if (!target || !number || !state) return;
   const executable = profile.name;
   const action = ctx.data.action ?? synthesizeAction(ctx);
-  if (ctx.output.prUrl && !state.core.prUrl) state.core.prUrl = ctx.output.prUrl;
-  if (typeof ctx.data.runUrl === "string") state.core.runUrl = ctx.data.runUrl;
   const next = reduce(state, executable, action, profile.phase);
   if (ctx.output.prUrl) next.core.prUrl = ctx.output.prUrl;
   if (typeof ctx.data.runUrl === "string") next.core.runUrl = ctx.data.runUrl;

package/dist/executables/goal-scheduler/profile.json

@@ -29,6 +29,16 @@
       "verify": "gh auth status",
       "usage": "",
       "allowedUses": ["api", "issue", "pr"]
+    },
+    {
+      "name": "python3",
+      "install": {
+        "required": true,
+        "checkCommand": "command -v python3"
+      },
+      "verify": "python3 --version",
+      "usage": "scheduler.sh parses each goal's state.json to read its 'state' field",
+      "allowedUses": []
     }
   ],
   "inputArtifacts": [],

package/dist/executables/goal-scheduler/scheduler.sh

@@ -11,6 +11,15 @@ set -euo pipefail
 
 goals_dir=".kody/goals"
 
+# python3 parses each goal's state.json below. It is declared as a required
+# cliTool in profile.json, but guard here too: without this check a missing
+# interpreter silently makes EVERY goal read state="" → treated as inactive →
+# nothing ticks, reported as a misleading success. Fail loud instead.
+if ! command -v python3 >/dev/null 2>&1; then
+  echo "[goal-scheduler] FATAL: python3 not found on PATH (required to read goal state)" >&2
+  exit 1
+fi
+
 # Goal state lives on the dedicated `kody-state` branch, not the default branch
 # (keeps `chore(goals): …` churn out of code history). Materialize it into the
 # working tree so the glob below sees current state. Best-effort: `kody-state`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.4.183",
+  "version": "0.4.184",
   "description": "kody — autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   "license": "MIT",
   "type": "module",
@@ -12,24 +12,6 @@
     "templates",
     "kody.config.schema.json"
   ],
-  "scripts": {
-    "kody:run": "tsx bin/kody.ts",
-    "serve": "tsx bin/kody.ts serve",
-    "serve:vscode": "tsx bin/kody.ts serve vscode",
-    "serve:claude": "tsx bin/kody.ts serve claude",
-    "build": "tsup && node scripts/copy-assets.cjs",
-    "check:modularity": "tsx scripts/check-script-modularity.ts",
-    "pretest": "pnpm check:modularity",
-    "test": "vitest run tests/unit tests/int --coverage",
-    "test:smoke": "vitest run tests/smoke --no-coverage",
-    "test:e2e": "vitest run tests/e2e --no-coverage",
-    "test:all": "vitest run tests --no-coverage",
-    "typecheck": "tsc --noEmit",
-    "lint": "biome check",
-    "lint:fix": "biome check --write",
-    "format": "biome format --write",
-    "prepublishOnly": "pnpm build"
-  },
   "dependencies": {
     "@actions/cache": "^6.0.0",
     "@anthropic-ai/claude-agent-sdk": "0.2.119",
@@ -39,7 +21,6 @@
     "@biomejs/biome": "^2.4.12",
     "@types/node": "^22.5.4",
     "@vitest/coverage-v8": "^4.1.4",
-    "playwright": "^1.60.0",
     "tsup": "^8.5.1",
     "tsx": "^4.21.0",
     "typescript": "~5.7.0",
@@ -53,5 +34,22 @@
     "url": "git+https://github.com/aharonyaircohen/kody-engine.git"
   },
   "homepage": "https://github.com/aharonyaircohen/kody-engine",
-  "bugs": "https://github.com/aharonyaircohen/kody-engine/issues"
-}
+  "bugs": "https://github.com/aharonyaircohen/kody-engine/issues",
+  "scripts": {
+    "kody:run": "tsx bin/kody.ts",
+    "serve": "tsx bin/kody.ts serve",
+    "serve:vscode": "tsx bin/kody.ts serve vscode",
+    "serve:claude": "tsx bin/kody.ts serve claude",
+    "build": "tsup && node scripts/copy-assets.cjs",
+    "check:modularity": "tsx scripts/check-script-modularity.ts",
+    "pretest": "pnpm check:modularity",
+    "test": "vitest run tests/unit tests/int --coverage",
+    "test:smoke": "vitest run tests/smoke --no-coverage",
+    "test:e2e": "vitest run tests/e2e --no-coverage",
+    "test:all": "vitest run tests --no-coverage",
+    "typecheck": "tsc --noEmit",
+    "lint": "biome check",
+    "lint:fix": "biome check --write",
+    "format": "biome format --write"
+  }
+}