@kody-ade/kody-engine 0.2.33 → 0.2.34

package/dist/bin/kody2.js

@@ -3,7 +3,7 @@
 // package.json
 var package_default = {
   name: "@kody-ade/kody-engine",
-  version: "0.2.33",
+  version: "0.2.34",
   description: "kody2 \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   license: "MIT",
   type: "module",
@@ -393,9 +393,6 @@ async function runAgent(opts) {
     if (typeof opts.maxTurns === "number" && opts.maxTurns > 0) {
       queryOptions.maxTurns = opts.maxTurns;
     }
-    if (typeof opts.maxThinkingTokens === "number" && opts.maxThinkingTokens > 0) {
-      queryOptions.maxThinkingTokens = opts.maxThinkingTokens;
-    }
     if (typeof opts.systemPromptAppend === "string" && opts.systemPromptAppend.length > 0) {
       queryOptions.systemPrompt = { type: "preset", preset: "claude_code", append: opts.systemPromptAppend };
     }
@@ -856,7 +853,6 @@ function parseClaudeCode(p, raw) {
     model: typeof r.model === "string" ? r.model : "inherit",
     permissionMode,
     maxTurns: typeof r.maxTurns === "number" ? r.maxTurns : null,
-    maxThinkingTokens: typeof r.maxThinkingTokens === "number" ? r.maxThinkingTokens : null,
     systemPromptAppend: typeof r.systemPromptAppend === "string" ? r.systemPromptAppend : null,
     tools,
     hooks: Array.isArray(r.hooks) ? r.hooks : [],
@@ -3219,31 +3215,17 @@ ${truncate2(r.stderr, 2e3)}
 
 // src/scripts/requireFeedbackActions.ts
 var MIN_ITEMS = 1;
-var ACTIONABLE_HEADINGS = /^#{1,6}\s+(Concerns|Suggestions|Bugs)\b/i;
-var NEXT_HEADING = /^#{1,6}\s+/;
 var requireFeedbackActions = async (ctx, profile) => {
   if (!ctx.data.agentDone) return;
   const actions = String(ctx.data.feedbackActions ?? "").trim();
   const items = countActionItems(actions);
+  ctx.data.feedbackAgentItemCount = items;
   if (items < MIN_ITEMS) {
     fail(
       ctx,
       profile,
       actions.length === 0 ? "agent omitted required FEEDBACK_ACTIONS block \u2014 cannot verify that review feedback was addressed" : "agent FEEDBACK_ACTIONS block listed no items \u2014 cannot verify that review feedback was addressed"
     );
-    return;
-  }
-  const reviewBody = String(ctx.data.feedback ?? "");
-  const expectedItems = countActionableReviewBullets(reviewBody);
-  ctx.data.feedbackReviewItemCount = expectedItems;
-  ctx.data.feedbackAgentItemCount = items;
-  if (expectedItems > 0 && items < expectedItems) {
-    fail(
-      ctx,
-      profile,
-      `agent FEEDBACK_ACTIONS listed ${items} item(s) but the review has ${expectedItems} actionable bullet(s) under ### Concerns / ### Suggestions / ### Bugs \u2014 every review item must be accounted for`
-    );
-    return;
   }
 };
 function fail(ctx, profile, reason) {
@@ -3265,25 +3247,6 @@ function countActionItems(block) {
   }
   return count;
 }
-function countActionableReviewBullets(reviewBody) {
-  if (!reviewBody.trim()) return 0;
-  const lines = reviewBody.split("\n");
-  let count = 0;
-  let insideActionable = false;
-  for (const raw of lines) {
-    if (ACTIONABLE_HEADINGS.test(raw)) {
-      insideActionable = true;
-      continue;
-    }
-    if (insideActionable && NEXT_HEADING.test(raw)) {
-      insideActionable = false;
-      continue;
-    }
-    if (!insideActionable) continue;
-    if (/^[-*]\s+\S/.test(raw)) count++;
-  }
-  return count;
-}
 
 // src/scripts/requirePlanDeviations.ts
 var requirePlanDeviations = async (ctx, profile) => {
@@ -3703,6 +3666,44 @@ function summarizeFeedbackActions(block) {
   }
   return summary;
 }
+function extractReviewFileRefs(reviewBody) {
+  if (!reviewBody) return [];
+  const found = /* @__PURE__ */ new Set();
+  const backtick = /`([^`\s]+\.[a-zA-Z]{1,5})(?::\d+(?:-\d+)?)?`/g;
+  let m;
+  while ((m = backtick.exec(reviewBody)) !== null) {
+    const raw = m[1];
+    if (isPlausibleSourcePath(raw)) found.add(raw);
+  }
+  const bare = /(?<![A-Za-z0-9/_.-])((?:[A-Za-z0-9_./-]+\/)+[A-Za-z0-9_.-]+\.[a-zA-Z]{1,5})(?::\d+(?:-\d+)?)?/g;
+  while ((m = bare.exec(reviewBody)) !== null) {
+    const raw = m[1];
+    if (isPlausibleSourcePath(raw)) found.add(raw);
+  }
+  return Array.from(found);
+}
+function isPlausibleSourcePath(p) {
+  if (p.startsWith("http://") || p.startsWith("https://")) return false;
+  if (p.startsWith("//")) return false;
+  if (p.startsWith("/")) return false;
+  if (!p.includes("/")) return false;
+  if (/\.(md|rst|txt|png|jpg|jpeg|gif|svg|pdf)$/i.test(p)) return false;
+  const firstSeg = p.slice(0, p.indexOf("/"));
+  if (firstSeg.includes(".")) return false;
+  return true;
+}
+function declinedFileRefs(feedbackActions, refs) {
+  if (!feedbackActions.trim() || refs.length === 0) return /* @__PURE__ */ new Set();
+  const declined = /* @__PURE__ */ new Set();
+  for (const raw of feedbackActions.split("\n")) {
+    if (!/^\s*[-*]\s+/.test(raw)) continue;
+    if (!/\bdeclined\s*:/i.test(raw)) continue;
+    for (const ref of refs) {
+      if (raw.includes(ref)) declined.add(ref);
+    }
+  }
+  return declined;
+}
 function makeAction2(type, payload) {
   return { type, payload, timestamp: (/* @__PURE__ */ new Date()).toISOString() };
 }
@@ -3715,24 +3716,32 @@ var verifyFixAlignment = async (ctx, profile) => {
   ctx.data.feedbackActionsSummary = summary;
   const committed = Boolean(ctx.data.commitResult?.committed);
   if (summary.totalItems === 0) {
-    ctx.output.exitCode = 1;
-    ctx.output.reason = "fix produced no FEEDBACK_ACTIONS items";
-    ctx.data.agentDone = false;
-    ctx.data.action = makeAction2("FIX_FAILED", {
-      reason: ctx.output.reason,
-      feedbackActionsSummary: summary
-    });
-    return;
+    return failOnce(ctx, "FIX_FAILED", "fix produced no FEEDBACK_ACTIONS items", summary);
   }
   if (summary.fixedItems > 0 && !committed) {
-    ctx.output.exitCode = 1;
-    ctx.output.reason = `fix claimed ${summary.fixedItems} fixed item(s) but produced no commit`;
-    ctx.data.agentDone = false;
-    ctx.data.action = makeAction2("FIX_FAILED", {
-      reason: ctx.output.reason,
-      feedbackActionsSummary: summary
-    });
-    return;
+    return failOnce(
+      ctx,
+      "FIX_FAILED",
+      `fix claimed ${summary.fixedItems} fixed item(s) but produced no commit`,
+      summary
+    );
+  }
+  const reviewBody = ctx.data.feedback ?? "";
+  const refs = extractReviewFileRefs(reviewBody);
+  const changedFiles = (ctx.data.changedFiles ?? []).map((f) => f.trim()).filter(Boolean);
+  ctx.data.reviewFileRefs = refs;
+  if (refs.length > 0 && committed) {
+    const declined = declinedFileRefs(feedbackActions, refs);
+    const missing = refs.filter((r) => !declined.has(r) && !changedFiles.some((f) => filesMatch(f, r)));
+    if (missing.length > 0) {
+      return failOnce(
+        ctx,
+        "FIX_FAILED",
+        `fix did not touch review-named file(s): ${missing.join(", ")} \u2014 address them or mark declined with a reason`,
+        summary,
+        { missingFiles: missing, declinedFiles: Array.from(declined), changedFiles }
+      );
+    }
   }
   if (summary.fixedItems === 0 && summary.declinedItems > 0 && !committed) {
     ctx.data.action = makeAction2("FIX_DECLINED", {
@@ -3741,6 +3750,25 @@ var verifyFixAlignment = async (ctx, profile) => {
     });
   }
 };
+function failOnce(ctx, type, reason, summary, extra) {
+  ctx.output.exitCode = 1;
+  ctx.output.reason = reason;
+  ctx.data.agentDone = false;
+  ctx.data.action = makeAction2(type, {
+    reason,
+    feedbackActionsSummary: summary,
+    ...extra ?? {}
+  });
+}
+function filesMatch(changedPath, reviewRef) {
+  if (changedPath === reviewRef) return true;
+  if (changedPath.endsWith("/" + reviewRef)) return true;
+  if (reviewRef.endsWith("/" + changedPath)) return true;
+  const a = changedPath.split("/");
+  const b = reviewRef.split("/");
+  if (a[a.length - 1] !== b[b.length - 1]) return false;
+  return a.length >= 2 && b.length >= 2 && a[a.length - 2] === b[b.length - 2];
+}
 
 // src/scripts/watchStalePrsFlow.ts
 function readWatchConfig(ctx) {
@@ -4000,7 +4028,6 @@ async function runExecutable(profileName, input) {
       mcpServers: profile.claudeCode.mcpServers,
       pluginPaths: pluginPaths.length > 0 ? pluginPaths : void 0,
       maxTurns: profile.claudeCode.maxTurns,
-      maxThinkingTokens: profile.claudeCode.maxThinkingTokens,
       systemPromptAppend: profile.claudeCode.systemPromptAppend,
       settingSources: profile.claudeCode.settingSources
     });

package/dist/executables/types.ts

@@ -87,8 +87,6 @@ export interface ClaudeCodeSpec {
   permissionMode: "default" | "acceptEdits" | "plan" | "bypassPermissions"
   /** null = unbounded. */
   maxTurns: number | null
-  /** Extended-thinking token budget. null = SDK default. */
-  maxThinkingTokens: number | null
   /** Text appended on top of Claude Code's baseline system prompt. */
   systemPromptAppend: string | null
   /** SDK built-in tools this executable is allowed to use (capability pack). */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.2.33",
+  "version": "0.2.34",
   "description": "kody2 — autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   "license": "MIT",
   "type": "module",
@@ -12,18 +12,6 @@
     "templates",
     "kody.config.schema.json"
   ],
-  "scripts": {
-    "kody2": "tsx bin/kody2.ts",
-    "build": "tsup && node scripts/copy-assets.cjs",
-    "test": "vitest run tests/unit tests/int --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": {
     "@anthropic-ai/claude-agent-sdk": "0.2.92"
   },
@@ -43,5 +31,16 @@
     "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": {
+    "kody2": "tsx bin/kody2.ts",
+    "build": "tsup && node scripts/copy-assets.cjs",
+    "test": "vitest run tests/unit tests/int --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"
+  }
+}