@kody-ade/kody-engine 0.2.39 → 0.2.41

package/dist/bin/kody2.js

@@ -3,7 +3,7 @@
 // package.json
 var package_default = {
   name: "@kody-ade/kody-engine",
-  version: "0.2.39",
+  version: "0.2.41",
   description: "kody2 \u2014 autonomous development engine. Single-session Claude Code agent behind a generic executor + declarative executable profiles.",
   license: "MIT",
   type: "module",
@@ -2855,6 +2855,24 @@ ${plan}
 Comment \`@kody2 run\` to execute this plan.`;
 }
 
+// src/scripts/postResearchComment.ts
+var postResearchComment = async (ctx) => {
+  if (!ctx.data.agentDone) return;
+  const targetType = ctx.data.commentTargetType;
+  const targetNumber = Number(ctx.data.commentTargetNumber ?? 0);
+  const body = ctx.data.prSummary?.trim();
+  if (targetType !== "issue" || !targetNumber || !body) return;
+  try {
+    postIssueComment(targetNumber, renderResearchComment(targetNumber, body), ctx.cwd);
+  } catch {
+  }
+};
+function renderResearchComment(issueNumber, body) {
+  return `## Research for issue #${issueNumber}
+
+${body}`;
+}
+
 // src/scripts/postReviewResult.ts
 function detectVerdict(body) {
   const m = body.match(/##\s*Verdict\s*:\s*(PASS|CONCERNS|FAIL)\b/i);
@@ -3654,138 +3672,6 @@ var verify = async (ctx) => {
   }
 };
 
-// src/scripts/verifyFixAlignment.ts
-function summarizeFeedbackActions(block) {
-  const summary = { totalItems: 0, fixedItems: 0, declinedItems: 0, unparsedLines: 0 };
-  if (!block.trim()) return summary;
-  for (const raw of block.split("\n")) {
-    if (!/^\s*[-*]\s+/.test(raw)) continue;
-    const line = raw.replace(/^\s*[-*]\s*/, "").trim();
-    summary.totalItems++;
-    if (/\bfixed\s*:/i.test(line)) summary.fixedItems++;
-    else if (/\bdeclined\s*:/i.test(line)) summary.declinedItems++;
-    else summary.unparsedLines++;
-  }
-  return summary;
-}
-var ACTIONABLE_HEADING = /^#{1,6}\s+(Concerns|Suggestions|Bugs)\b/i;
-var ANY_HEADING = /^#{1,6}\s+/;
-function actionableSections(reviewBody) {
-  const lines = reviewBody.split("\n");
-  const kept = [];
-  let inside = false;
-  for (const raw of lines) {
-    if (ACTIONABLE_HEADING.test(raw)) {
-      inside = true;
-      kept.push(raw);
-      continue;
-    }
-    if (inside && ANY_HEADING.test(raw)) {
-      inside = false;
-      continue;
-    }
-    if (inside) kept.push(raw);
-  }
-  return kept.join("\n");
-}
-function extractReviewFileRefs(reviewBody) {
-  if (!reviewBody) return [];
-  const scoped = actionableSections(reviewBody);
-  if (!scoped.trim()) return [];
-  const found = /* @__PURE__ */ new Set();
-  const backtick = /`([^`\s]+\.[a-zA-Z]{1,5})(?::\d+(?:-\d+)?)?`/g;
-  let m;
-  while ((m = backtick.exec(scoped)) !== 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(scoped)) !== 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() };
-}
-var verifyFixAlignment = async (ctx, profile) => {
-  if (profile.name !== "fix") return;
-  if (ctx.skipAgent) return;
-  if (!ctx.data.agentDone) return;
-  const feedbackActions = ctx.data.feedbackActions ?? "";
-  const summary = summarizeFeedbackActions(feedbackActions);
-  ctx.data.feedbackActionsSummary = summary;
-  const committed = Boolean(ctx.data.commitResult?.committed);
-  if (summary.totalItems === 0) {
-    return failOnce(ctx, "FIX_FAILED", "fix produced no FEEDBACK_ACTIONS items", 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", {
-      feedbackActionsSummary: summary,
-      note: "agent declined all feedback items; no commit made"
-    });
-  }
-};
-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) {
   const cfg = ctx.config.watch;
@@ -3913,9 +3799,9 @@ var postflightScripts = {
   ensurePr: ensurePr2,
   postIssueComment: postIssueComment2,
   postPlanComment,
+  postResearchComment,
   postReviewResult,
   persistArtifacts,
-  verifyFixAlignment,
   writeRunSummary,
   saveTaskState
 };

package/dist/executables/fix/profile.json

@@ -72,9 +72,6 @@
       {
         "script": "commitAndPush"
       },
-      {
-        "script": "verifyFixAlignment"
-      },
       {
         "script": "ensurePr"
       },

package/dist/executables/research/profile.json

@@ -0,0 +1,78 @@
+{
+  "name": "research",
+  "describe": "Research an issue: understand the ask, map relevant repo context, and surface clarifying questions + gaps. Read-only — no branches, no commits, no prescribed next steps.",
+  "inputs": [
+    {
+      "name": "issue",
+      "flag": "--issue",
+      "type": "int",
+      "required": true,
+      "describe": "GitHub issue number to research."
+    }
+  ],
+  "claudeCode": {
+    "model": "inherit",
+    "permissionMode": "default",
+    "maxTurns": null,
+    "systemPromptAppend": null,
+    "tools": [
+      "Read",
+      "Grep",
+      "Glob",
+      "Bash"
+    ],
+    "hooks": [],
+    "skills": [],
+    "commands": [],
+    "subagents": [],
+    "plugins": [],
+    "mcpServers": []
+  },
+  "cliTools": [],
+  "scripts": {
+    "preflight": [
+      {
+        "script": "loadIssueContext"
+      },
+      {
+        "script": "loadTaskState"
+      },
+      {
+        "script": "loadConventions"
+      },
+      {
+        "script": "composePrompt"
+      }
+    ],
+    "postflight": [
+      {
+        "script": "parseAgentResult"
+      },
+      {
+        "script": "persistArtifacts"
+      },
+      {
+        "script": "postResearchComment"
+      },
+      {
+        "script": "writeRunSummary"
+      },
+      {
+        "script": "saveTaskState"
+      }
+    ]
+  },
+  "output": {
+    "actionTypes": [
+      "RESEARCH_COMPLETED",
+      "RESEARCH_FAILED"
+    ],
+    "artifacts": [
+      {
+        "name": "research",
+        "format": "markdown",
+        "from": "prSummary"
+      }
+    ]
+  }
+}

package/dist/executables/research/prompt.md

@@ -0,0 +1,61 @@
+You are a senior engineer **researching** a GitHub issue. Your job is to fill in missing information so a downstream planner (human or agent) can make a decision. You will NOT write code. You will NOT run git or gh commands. You will NOT modify files. You will NOT prescribe a next step.
+
+Use Read / Grep / Glob / Bash (read-only) to study the codebase as much as needed. Then emit a final message with the research doc wrapped in the required markers (see "Required output").
+
+---
+
+# Repo
+- {{repoOwner}}/{{repoName}}, default branch: {{defaultBranch}}
+
+# Issue #{{issue.number}}: {{issue.title}}
+
+{{issue.body}}
+
+Recent comments (most recent first, truncated):
+{{issue.commentsFormatted}}
+
+{{conventionsBlock}}
+
+---
+
+# Required output
+
+Your FINAL message must be exactly this shape (no extra text before or after):
+
+```
+DONE
+COMMIT_MSG: research: <very short title>
+PR_SUMMARY:
+<A research doc in markdown with EXACTLY these sections, in order:
+
+## Understood request
+One paragraph restating what the issue is asking for, in your own words.
+
+## Repo context
+Files, modules, and existing patterns most relevant to the request. Use
+`path/to/file.ts` references. Note anything that constrains the solution
+space (existing abstractions, invariants from AGENTS.md / CLAUDE.md).
+
+## Clarifying questions
+Numbered list. Each question must include a one-line "Why:" explaining why
+the answer changes the implementation. Skip if there are genuinely none.
+
+## Gaps & assumptions
+What is unknown, and — for each gap — what assumption the implementer would
+have to make if it stays unanswered.
+
+## Proposed scope
+Two bullet lists: **In scope** and **Out of scope**. Keep tight: only what
+the issue asks for; call out adjacent work that should NOT be bundled.
+
+Keep the whole doc to ~80 lines or less. No filler. No marketing language.
+Do NOT include a "Next steps" / "Recommendation" / "How to proceed" section —
+research stops at findings.>
+```
+
+# Rules
+- Read-only. Do NOT modify any file.
+- Do NOT run git or gh commands.
+- Do NOT propose an implementation plan — that's the planner's job.
+- Do NOT tell the user what command to run next.
+- If the issue is empty or incomprehensible, output `FAILED: <why>` instead.

package/package.json

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