@kody-ade/kody-engine 0.4.539 → 0.4.541

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.539",
+      version: "0.4.541",
       description: "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative implementation profiles.",
       license: "MIT",
       type: "module",
@@ -17869,113 +17869,6 @@ var init_runFlow = __esm({
   }
 });
 
-// src/scripts/syncFlow.ts
-import { execFileSync as execFileSync16 } from "child_process";
-function restoreDone(prNumber, cwd) {
-  try {
-    setKodyLabel(prNumber, DONE2, cwd);
-  } catch {
-  }
-}
-function bail2(ctx, prNumber, reason) {
-  ctx.skipAgent = true;
-  ctx.output.exitCode = 1;
-  ctx.output.reason = reason;
-  const runUrl = getRunUrl();
-  const runSuffix = runUrl ? ` ([logs](${runUrl}))` : "";
-  tryPostPr3(prNumber, `\u274C kody sync could not complete${runSuffix}: ${reason}`, ctx.cwd);
-}
-function revParseHead(cwd) {
-  try {
-    return execFileSync16("git", ["rev-parse", "HEAD"], { cwd, encoding: "utf-8", stdio: ["ignore", "pipe", "pipe"] }).toString().trim();
-  } catch {
-    return "";
-  }
-}
-function pushBranch(branch, cwd) {
-  const result = pushWithRetry({ cwd: cwd ?? process.cwd(), branch, setUpstream: true });
-  if (!result.ok) {
-    throw new Error(result.reason);
-  }
-}
-function tryPostPr3(prNumber, body, cwd) {
-  try {
-    postPrReviewComment(prNumber, body, cwd);
-  } catch {
-  }
-}
-var DONE2, syncFlow;
-var init_syncFlow = __esm({
-  "src/scripts/syncFlow.ts"() {
-    "use strict";
-    init_branch();
-    init_gha();
-    init_issue();
-    init_lifecycleLabels();
-    init_pushWithRetry();
-    DONE2 = {
-      label: "kody:done",
-      color: "0e8a16",
-      description: "kody: PR ready for human review/merge"
-    };
-    syncFlow = async (ctx, _profile, args) => {
-      const announceOnSuccess = Boolean(args?.announceOnSuccess);
-      const prNumber = ctx.args.pr;
-      const pr = getPr(prNumber, ctx.cwd);
-      if (pr.state !== "OPEN") {
-        bail2(ctx, prNumber, `PR #${prNumber} is not OPEN (state: ${pr.state})`);
-        return;
-      }
-      ctx.data.pr = pr;
-      if (announceOnSuccess) {
-        ctx.data.commentTargetType = "pr";
-        ctx.data.commentTargetNumber = prNumber;
-      }
-      checkoutPrBranch(prNumber, ctx.cwd);
-      ctx.data.branch = getCurrentBranch(ctx.cwd);
-      const baseBranch = pr.baseRefName || ctx.config.git.defaultBranch;
-      ctx.data.baseBranch = baseBranch;
-      const headBefore = revParseHead(ctx.cwd);
-      const mergeStatus = mergeBase(baseBranch, ctx.cwd);
-      if (mergeStatus === "error") {
-        bail2(ctx, prNumber, `failed to merge origin/${baseBranch} (non-conflict error); see runner log`);
-        return;
-      }
-      if (mergeStatus === "conflict") {
-        bail2(
-          ctx,
-          prNumber,
-          `merge from origin/${baseBranch} produced conflicts \u2014 run \`@kody resolve\` to let kody resolve them`
-        );
-        return;
-      }
-      const headAfter = revParseHead(ctx.cwd);
-      if (headAfter === headBefore) {
-        ctx.data.syncResult = "noop";
-        if (announceOnSuccess) {
-          ctx.output.exitCode = 0;
-          ctx.output.reason = `already up to date with origin/${baseBranch}`;
-          restoreDone(prNumber, ctx.cwd);
-        }
-        return;
-      }
-      try {
-        pushBranch(ctx.data.branch, ctx.cwd);
-      } catch (err) {
-        const msg = err instanceof Error ? err.message : String(err);
-        bail2(ctx, prNumber, `merge succeeded but push failed: ${msg}`);
-        return;
-      }
-      ctx.data.syncResult = "merged";
-      if (announceOnSuccess) {
-        ctx.output.exitCode = 0;
-        ctx.output.reason = `merged origin/${baseBranch} into ${ctx.data.branch}`;
-        restoreDone(prNumber, ctx.cwd);
-      }
-    };
-  }
-});
-
 // src/scripts/prepareCapabilityDelivery.ts
 var prepareCapabilityDelivery;
 var init_prepareCapabilityDelivery = __esm({
@@ -17983,7 +17876,6 @@ var init_prepareCapabilityDelivery = __esm({
     "use strict";
     init_capabilityDelivery();
     init_runFlow();
-    init_syncFlow();
     prepareCapabilityDelivery = async (ctx, profile) => {
       const target = capabilityDeliveryTarget(ctx.data.capabilityInput);
       if (!target) {
@@ -17996,7 +17888,6 @@ var init_prepareCapabilityDelivery = __esm({
       }
       ctx.data.commentTargetType = "pr";
       ctx.data.commentTargetNumber = target.number;
-      await syncFlow(ctx, profile);
     };
   }
 });
@@ -18352,7 +18243,7 @@ var init_publishReport = __esm({
 });
 
 // src/scripts/recordClassification.ts
-import { execFileSync as execFileSync17 } from "child_process";
+import { execFileSync as execFileSync16 } from "child_process";
 function parseClassification(prSummary) {
   if (!prSummary) return null;
   const classMatch = prSummary.match(/classification:\s*(feature|bug|spec|chore)\b/i);
@@ -18364,7 +18255,7 @@ function parseClassification(prSummary) {
 }
 function tryAuditComment(issueNumber, body, cwd) {
   try {
-    execFileSync17("gh", ["issue", "comment", String(issueNumber), "--body", body], {
+    execFileSync16("gh", ["issue", "comment", String(issueNumber), "--body", body], {
       cwd,
       timeout: API_TIMEOUT_MS5,
       stdio: ["ignore", "pipe", "pipe"]
@@ -18461,7 +18352,16 @@ var init_requireDeliveryArtifacts = __esm({
       if (ctx.data.agentDone !== true) return;
       const commitResult = ctx.data.commitResult;
       const hasCommits = ctx.data.hasCommitsAhead === true;
-      if (!commitResult?.committed && !hasCommits) return;
+      if (ctx.data.jobDelivery === "pull-request" && (!commitResult?.committed || !commitResult.pushed)) {
+        const reason2 = "pull-request delivery produced no commit";
+        ctx.data.agentFailureReason = reason2;
+        ctx.output.exitCode = 4;
+        ctx.output.reason = reason2;
+        return;
+      }
+      if (!commitResult?.committed && !hasCommits) {
+        return;
+      }
       const commitMessage = String(ctx.data.commitMessage ?? "").trim();
       const prSummary = String(ctx.data.prSummary ?? "").trim();
       const missing = [];
@@ -18584,7 +18484,7 @@ var init_resolveArtifacts = __esm({
 });
 
 // src/scripts/resolveFlow.ts
-import { execFileSync as execFileSync18 } from "child_process";
+import { execFileSync as execFileSync17 } from "child_process";
 function buildPreferBlock(prefer, baseBranch) {
   if (prefer !== "ours" && prefer !== "theirs") return "";
   const keepSide = prefer === "ours" ? "HEAD (this PR branch)" : `origin/${baseBranch} (base branch)`;
@@ -18604,7 +18504,7 @@ function buildPreferBlock(prefer, baseBranch) {
 }
 function getConflictedFiles(cwd) {
   try {
-    const out = execFileSync18("git", ["diff", "--name-only", "--diff-filter=U"], {
+    const out = execFileSync17("git", ["diff", "--name-only", "--diff-filter=U"], {
       encoding: "utf-8",
       cwd,
       env: { ...process.env, HUSKY: "0" }
@@ -18619,7 +18519,7 @@ function getConflictMarkersPreview(files, cwd, maxBytes = CONFLICT_DIFF_MAX_BYTE
   let total = 0;
   for (const f of files) {
     try {
-      const content = execFileSync18("cat", [f], { encoding: "utf-8", cwd }).toString();
+      const content = execFileSync17("cat", [f], { encoding: "utf-8", cwd }).toString();
       const snippet = `### ${f}
 
 \`\`\`
@@ -18634,7 +18534,7 @@ ${content.slice(0, 6e3)}
   }
   return chunks.join("\n");
 }
-function tryPostPr4(prNumber, body, cwd) {
+function tryPostPr3(prNumber, body, cwd) {
   try {
     postPrReviewComment(prNumber, body, cwd);
   } catch {
@@ -18643,12 +18543,12 @@ function tryPostPr4(prNumber, body, cwd) {
 function pushEmptyCommit(branch, cwd) {
   const env = { ...process.env, HUSKY: "0", SKIP_HOOKS: "1" };
   try {
-    execFileSync18(
+    execFileSync17(
       "git",
       ["commit", "--allow-empty", "-m", "chore: kody resolve refresh \u2014 empty commit to recompute mergeable status"],
       { cwd, env, stdio: ["ignore", "pipe", "pipe"] }
     );
-    execFileSync18("git", ["push", "-u", "origin", branch], {
+    execFileSync17("git", ["push", "-u", "origin", branch], {
       cwd,
       env,
       stdio: ["ignore", "pipe", "pipe"]
@@ -18686,14 +18586,14 @@ var init_resolveFlow = __esm({
         ctx.output.exitCode = 0;
         ctx.output.reason = `PR #${prNumber} is mergeable (no conflicts) \u2014 nothing to resolve`;
         ctx.skipAgent = true;
-        tryPostPr4(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr3(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       if (ghStatus.status === "BLOCKED") {
         ctx.output.exitCode = 0;
         ctx.output.reason = `PR #${prNumber} is mergeable but blocked by checks/reviews (mergeStateStatus=${ghStatus.mergeStateStatus}) \u2014 nothing for resolve to do`;
         ctx.skipAgent = true;
-        tryPostPr4(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr3(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       checkoutPrBranch(prNumber, ctx.cwd);
@@ -18705,20 +18605,20 @@ var init_resolveFlow = __esm({
           ctx.output.exitCode = 0;
           ctx.output.reason = pushed ? `local merge clean despite GitHub reporting CONFLICTING \u2014 pushed empty commit to force re-evaluation` : `local merge clean despite GitHub reporting CONFLICTING \u2014 couldn't refresh GitHub cache (push failed)`;
           ctx.skipAgent = true;
-          tryPostPr4(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
+          tryPostPr3(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
           return;
         }
         ctx.output.exitCode = 0;
         ctx.output.reason = `already up to date with origin/${baseBranch} \u2014 nothing to resolve`;
         ctx.skipAgent = true;
-        tryPostPr4(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr3(prNumber, `\u2139\uFE0F kody resolve: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       if (mergeStatus === "error") {
         ctx.output.exitCode = 99;
         ctx.output.reason = `failed to merge origin/${baseBranch} (non-conflict error); see runner log`;
         ctx.skipAgent = true;
-        tryPostPr4(prNumber, `\u26A0\uFE0F kody resolve FAILED: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr3(prNumber, `\u26A0\uFE0F kody resolve FAILED: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       const conflictedFiles = getConflictedFiles(ctx.cwd);
@@ -18733,7 +18633,7 @@ var init_resolveFlow = __esm({
       ctx.data.preferBlock = buildPreferBlock(ctx.args.prefer, baseBranch);
       const runUrl = getRunUrl();
       const runSuffix = runUrl ? `, run ${runUrl}` : "";
-      tryPostPr4(
+      tryPostPr3(
         prNumber,
         `\u2699\uFE0F kody resolve started on \`${ctx.data.branch}\`${runSuffix} \u2014 ${conflictedFiles.length} conflicted file(s)`,
         ctx.cwd
@@ -18834,10 +18734,10 @@ var init_resolvePreviewUrl = __esm({
 });
 
 // src/scripts/resolveQaUrl.ts
-import { execFileSync as execFileSync19 } from "child_process";
+import { execFileSync as execFileSync18 } from "child_process";
 function ghQuery(args, cwd) {
   try {
-    const out = execFileSync19("gh", args, {
+    const out = execFileSync18("gh", args, {
       cwd,
       stdio: ["ignore", "pipe", "pipe"],
       encoding: "utf-8",
@@ -18914,7 +18814,7 @@ var init_resolveQaUrl = __esm({
 });
 
 // src/scripts/revertFlow.ts
-import { execFileSync as execFileSync20 } from "child_process";
+import { execFileSync as execFileSync19 } from "child_process";
 function buildCommitMessage(resolved) {
   if (resolved.length === 1) {
     const { full, subject } = resolved[0];
@@ -18927,7 +18827,7 @@ function buildPrSummary(resolved) {
   return resolved.map((r) => `- Reverted \`${r.full.slice(0, 7)}\`${r.subject ? ` \u2014 ${r.subject}` : ""}`).join("\n");
 }
 function git4(args, cwd) {
-  return execFileSync20("git", args, {
+  return execFileSync19("git", args, {
     encoding: "utf-8",
     timeout: 3e4,
     cwd,
@@ -18937,7 +18837,7 @@ function git4(args, cwd) {
 }
 function isAncestorOfHead(sha, cwd) {
   try {
-    execFileSync20("git", ["merge-base", "--is-ancestor", sha, "HEAD"], {
+    execFileSync19("git", ["merge-base", "--is-ancestor", sha, "HEAD"], {
       cwd,
       env: { ...process.env, HUSKY: "0", SKIP_HOOKS: "1" },
       stdio: ["ignore", "ignore", "ignore"]
@@ -18947,7 +18847,7 @@ function isAncestorOfHead(sha, cwd) {
     return false;
   }
 }
-function tryPostPr5(prNumber, body, cwd) {
+function tryPostPr4(prNumber, body, cwd) {
   try {
     postPrReviewComment(prNumber, body, cwd);
   } catch (err) {
@@ -18983,7 +18883,7 @@ var init_revertFlow = __esm({
         ctx.output.exitCode = 64;
         ctx.output.reason = "no commit SHAs provided \u2014 usage: @kody revert <sha> [<sha> \u2026]";
         ctx.skipAgent = true;
-        tryPostPr5(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr4(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       const requested = shasArg.split(/\s+/).filter((s) => s.length > 0);
@@ -18992,7 +18892,7 @@ var init_revertFlow = __esm({
         ctx.output.exitCode = 64;
         ctx.output.reason = `not valid SHA-shaped tokens: ${bad.join(", ")}`;
         ctx.skipAgent = true;
-        tryPostPr5(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr4(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       const resolved = [];
@@ -19020,7 +18920,7 @@ var init_revertFlow = __esm({
         ctx.output.exitCode = 64;
         ctx.output.reason = `commit(s) not found in this PR branch: ${unreachable.join(", ")}`;
         ctx.skipAgent = true;
-        tryPostPr5(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
+        tryPostPr4(prNumber, `\u26A0\uFE0F kody revert FAILED: ${ctx.output.reason}`, ctx.cwd);
         return;
       }
       ctx.args.shas = resolved.map((r) => r.full).join(" ");
@@ -19030,13 +18930,13 @@ var init_revertFlow = __esm({
       const runUrl = getRunUrl();
       const runSuffix = runUrl ? `, run ${runUrl}` : "";
       const shaList = resolved.map((r) => `\`${r.full.slice(0, 7)}\``).join(", ");
-      tryPostPr5(prNumber, `\u2699\uFE0F kody revert started on \`${ctx.data.branch}\`${runSuffix} \u2014 reverting ${shaList}`, ctx.cwd);
+      tryPostPr4(prNumber, `\u2699\uFE0F kody revert started on \`${ctx.data.branch}\`${runSuffix} \u2014 reverting ${shaList}`, ctx.cwd);
     };
   }
 });
 
 // src/scripts/reviewFlow.ts
-function tryPostPr6(prNumber, body, cwd) {
+function tryPostPr5(prNumber, body, cwd) {
   try {
     postPrReviewComment(prNumber, body, cwd);
   } catch {
@@ -19066,7 +18966,7 @@ var init_reviewFlow = __esm({
       ctx.data.prDiff = getPrDiff(prNumber, ctx.cwd);
       const runUrl = getRunUrl();
       const runSuffix = runUrl ? `, run ${runUrl}` : "";
-      tryPostPr6(prNumber, `\u{1F440} kody review started on PR #${prNumber}${runSuffix}`, ctx.cwd);
+      tryPostPr5(prNumber, `\u{1F440} kody review started on PR #${prNumber}${runSuffix}`, ctx.cwd);
     };
   }
 });
@@ -19922,7 +19822,7 @@ var init_skipAgent = __esm({
 });
 
 // src/scripts/stageMergeConflicts.ts
-import { execFileSync as execFileSync21 } from "child_process";
+import { execFileSync as execFileSync20 } from "child_process";
 var stageMergeConflicts;
 var init_stageMergeConflicts = __esm({
   "src/scripts/stageMergeConflicts.ts"() {
@@ -19930,7 +19830,7 @@ var init_stageMergeConflicts = __esm({
     stageMergeConflicts = async (ctx) => {
       if (ctx.data.agentDone === false) return;
       try {
-        execFileSync21("git", ["add", "-A"], {
+        execFileSync20("git", ["add", "-A"], {
           cwd: ctx.cwd,
           env: { ...process.env, HUSKY: "0", SKIP_HOOKS: "1" },
           stdio: "pipe"
@@ -19983,6 +19883,113 @@ var init_startFlow = __esm({
   }
 });
 
+// src/scripts/syncFlow.ts
+import { execFileSync as execFileSync21 } from "child_process";
+function restoreDone(prNumber, cwd) {
+  try {
+    setKodyLabel(prNumber, DONE2, cwd);
+  } catch {
+  }
+}
+function bail2(ctx, prNumber, reason) {
+  ctx.skipAgent = true;
+  ctx.output.exitCode = 1;
+  ctx.output.reason = reason;
+  const runUrl = getRunUrl();
+  const runSuffix = runUrl ? ` ([logs](${runUrl}))` : "";
+  tryPostPr6(prNumber, `\u274C kody sync could not complete${runSuffix}: ${reason}`, ctx.cwd);
+}
+function revParseHead(cwd) {
+  try {
+    return execFileSync21("git", ["rev-parse", "HEAD"], { cwd, encoding: "utf-8", stdio: ["ignore", "pipe", "pipe"] }).toString().trim();
+  } catch {
+    return "";
+  }
+}
+function pushBranch(branch, cwd) {
+  const result = pushWithRetry({ cwd: cwd ?? process.cwd(), branch, setUpstream: true });
+  if (!result.ok) {
+    throw new Error(result.reason);
+  }
+}
+function tryPostPr6(prNumber, body, cwd) {
+  try {
+    postPrReviewComment(prNumber, body, cwd);
+  } catch {
+  }
+}
+var DONE2, syncFlow;
+var init_syncFlow = __esm({
+  "src/scripts/syncFlow.ts"() {
+    "use strict";
+    init_branch();
+    init_gha();
+    init_issue();
+    init_lifecycleLabels();
+    init_pushWithRetry();
+    DONE2 = {
+      label: "kody:done",
+      color: "0e8a16",
+      description: "kody: PR ready for human review/merge"
+    };
+    syncFlow = async (ctx, _profile, args) => {
+      const announceOnSuccess = Boolean(args?.announceOnSuccess);
+      const prNumber = ctx.args.pr;
+      const pr = getPr(prNumber, ctx.cwd);
+      if (pr.state !== "OPEN") {
+        bail2(ctx, prNumber, `PR #${prNumber} is not OPEN (state: ${pr.state})`);
+        return;
+      }
+      ctx.data.pr = pr;
+      if (announceOnSuccess) {
+        ctx.data.commentTargetType = "pr";
+        ctx.data.commentTargetNumber = prNumber;
+      }
+      checkoutPrBranch(prNumber, ctx.cwd);
+      ctx.data.branch = getCurrentBranch(ctx.cwd);
+      const baseBranch = pr.baseRefName || ctx.config.git.defaultBranch;
+      ctx.data.baseBranch = baseBranch;
+      const headBefore = revParseHead(ctx.cwd);
+      const mergeStatus = mergeBase(baseBranch, ctx.cwd);
+      if (mergeStatus === "error") {
+        bail2(ctx, prNumber, `failed to merge origin/${baseBranch} (non-conflict error); see runner log`);
+        return;
+      }
+      if (mergeStatus === "conflict") {
+        bail2(
+          ctx,
+          prNumber,
+          `merge from origin/${baseBranch} produced conflicts \u2014 run \`@kody resolve\` to let kody resolve them`
+        );
+        return;
+      }
+      const headAfter = revParseHead(ctx.cwd);
+      if (headAfter === headBefore) {
+        ctx.data.syncResult = "noop";
+        if (announceOnSuccess) {
+          ctx.output.exitCode = 0;
+          ctx.output.reason = `already up to date with origin/${baseBranch}`;
+          restoreDone(prNumber, ctx.cwd);
+        }
+        return;
+      }
+      try {
+        pushBranch(ctx.data.branch, ctx.cwd);
+      } catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        bail2(ctx, prNumber, `merge succeeded but push failed: ${msg}`);
+        return;
+      }
+      ctx.data.syncResult = "merged";
+      if (announceOnSuccess) {
+        ctx.output.exitCode = 0;
+        ctx.output.reason = `merged origin/${baseBranch} into ${ctx.data.branch}`;
+        restoreDone(prNumber, ctx.cwd);
+      }
+    };
+  }
+});
+
 // src/scripts/validateAgencyModelProposal.ts
 import * as path46 from "path";
 function validateModelBundle(bundle, expectedKind, options = {}) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine",
-  "version": "0.4.539",
+  "version": "0.4.541",
   "description": "kody \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative implementation profiles.",
   "license": "MIT",
   "type": "module",