@kody-ade/kody-engine-lite 0.1.10 → 0.1.12

package/dist/bin/cli.js

@@ -1078,31 +1078,35 @@ function buildPrBody(ctx) {
       const raw = fs4.readFileSync(taskJsonPath, "utf-8");
       const cleaned = raw.replace(/^```json\s*\n?/m, "").replace(/\n?```\s*$/m, "");
       const task = JSON.parse(cleaned);
-      sections.push(`## Summary`);
-      sections.push(`**Type:** ${task.task_type ?? "unknown"} | **Risk:** ${task.risk_level ?? "unknown"}`);
-      if (task.description) sections.push(`
+      if (task.description) {
+        sections.push(`## What
+
 ${task.description}`);
-      if (task.scope?.length) sections.push(`
-**Scope:** ${task.scope.join(", ")}`);
-    } catch {
-    }
-  }
-  const planPath = path4.join(ctx.taskDir, "plan.md");
-  if (fs4.existsSync(planPath)) {
-    const plan = fs4.readFileSync(planPath, "utf-8").trim();
-    if (plan) {
-      const truncated = plan.length > 500 ? plan.slice(0, 500) + "\n..." : plan;
-      sections.push(`
-## Plan
-<details><summary>Implementation plan</summary>
+      }
+      if (task.scope?.length) {
+        sections.push(`
+## Scope
 
-${truncated}
-</details>`);
+${task.scope.map((s) => `- \`${s}\``).join("\n")}`);
+      }
+      sections.push(`
+**Type:** ${task.task_type ?? "unknown"} | **Risk:** ${task.risk_level ?? "unknown"}`);
+    } catch {
     }
   }
   const reviewPath = path4.join(ctx.taskDir, "review.md");
   if (fs4.existsSync(reviewPath)) {
     const review = fs4.readFileSync(reviewPath, "utf-8");
+    const summaryMatch = review.match(/## Summary\s*\n([\s\S]*?)(?=\n## |\n*$)/);
+    if (summaryMatch) {
+      const summary = summaryMatch[1].trim();
+      if (summary) {
+        sections.push(`
+## Changes
+
+${summary}`);
+      }
+    }
     const verdictMatch = review.match(/## Verdict:\s*(PASS|FAIL)/i);
     if (verdictMatch) {
       sections.push(`
@@ -1114,6 +1118,18 @@ ${truncated}
     const verify = fs4.readFileSync(verifyPath, "utf-8");
     if (/PASS/i.test(verify)) sections.push(`**Verify:** \u2705 typecheck + tests + lint passed`);
   }
+  const planPath = path4.join(ctx.taskDir, "plan.md");
+  if (fs4.existsSync(planPath)) {
+    const plan = fs4.readFileSync(planPath, "utf-8").trim();
+    if (plan) {
+      const truncated = plan.length > 800 ? plan.slice(0, 800) + "\n..." : plan;
+      sections.push(`
+<details><summary>\u{1F4CB} Implementation plan</summary>
+
+${truncated}
+</details>`);
+    }
+  }
   if (ctx.input.issueNumber) {
     sections.push(`
 Closes #${ctx.input.issueNumber}`);
@@ -1154,12 +1170,33 @@ function executeShipStage(ctx, _def) {
       } catch {
       }
     }
-    const taskMdPath = path4.join(ctx.taskDir, "task.md");
     let title = "Update";
-    if (fs4.existsSync(taskMdPath)) {
-      const content = fs4.readFileSync(taskMdPath, "utf-8");
-      const lines = content.split("\n").filter((l) => l.trim() && !l.startsWith("#"));
-      title = (lines[0] ?? "Update").slice(0, 72);
+    const TYPE_PREFIX = {
+      feature: "feat",
+      bugfix: "fix",
+      refactor: "refactor",
+      docs: "docs",
+      chore: "chore"
+    };
+    const taskJsonPath = path4.join(ctx.taskDir, "task.json");
+    if (fs4.existsSync(taskJsonPath)) {
+      try {
+        const raw = fs4.readFileSync(taskJsonPath, "utf-8");
+        const cleaned = raw.replace(/^```json\s*\n?/m, "").replace(/\n?```\s*$/m, "");
+        const task = JSON.parse(cleaned);
+        const prefix = TYPE_PREFIX[task.task_type] ?? "chore";
+        const taskTitle = task.title ?? "Update";
+        title = `${prefix}: ${taskTitle}`.slice(0, 72);
+      } catch {
+      }
+    }
+    if (title === "Update") {
+      const taskMdPath = path4.join(ctx.taskDir, "task.md");
+      if (fs4.existsSync(taskMdPath)) {
+        const content = fs4.readFileSync(taskMdPath, "utf-8");
+        const firstLine = content.split("\n").find((l) => l.trim() && !l.startsWith("#") && !l.startsWith("*"));
+        if (firstLine) title = `chore: ${firstLine.trim()}`.slice(0, 72);
+      }
     }
     const body = buildPrBody(ctx);
     const pr = createPR(head, base, title, body);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kody-ade/kody-engine-lite",
-  "version": "0.1.10",
+  "version": "0.1.12",
   "description": "Autonomous SDLC pipeline: Kody orchestration + Claude Code + LiteLLM",
   "license": "MIT",
   "type": "module",

package/templates/kody.yml

@@ -113,19 +113,20 @@ jobs:
               ;;
           esac
 
-          # Auto-generate task-id if not provided
           ISSUE_NUM="${{ github.event.issue.number }}"
-          if [ -z "$TASK_ID" ]; then
-            TASK_ID="${ISSUE_NUM}-$(date +%y%m%d-%H%M%S)"
-          fi
 
           # For approve mode: extract answer body and convert to rerun
+          # Must run BEFORE task-id generation so we don't create a new task
           if [ "$MODE" = "approve" ]; then
-            # Everything after @kody approve [task-id] is the feedback
             APPROVE_BODY=$(echo "$BODY" | sed -n '/\(@kody\|\/kody\)\s*approve/,$p' | tail -n +2)
             FEEDBACK="$APPROVE_BODY"
             MODE="rerun"
-            # FROM_STAGE will be determined by entry.ts from paused state
+            # Leave TASK_ID empty — entry.ts findLatestTaskForIssue will find the paused task
+          fi
+
+          # Auto-generate task-id if not provided (only for non-rerun modes)
+          if [ -z "$TASK_ID" ] && [ "$MODE" != "rerun" ]; then
+            TASK_ID="${ISSUE_NUM}-$(date +%y%m%d-%H%M%S)"
           fi
 
           echo "task_id=$TASK_ID" >> $GITHUB_OUTPUT
@@ -200,7 +201,8 @@ jobs:
           CMD="run"
           [ "$MODE" = "rerun" ] && CMD="rerun"
           [ "$MODE" = "fix" ] && CMD="fix"
-          ARGS="--task-id $TASK_ID --issue-number $ISSUE_NUMBER"
+          ARGS="--issue-number $ISSUE_NUMBER"
+          [ -n "$TASK_ID" ] && ARGS="$ARGS --task-id $TASK_ID"
           [ -n "$FROM_STAGE" ] && ARGS="$ARGS --from $FROM_STAGE"
           [ -n "$FEEDBACK" ] && ARGS="$ARGS --feedback \"$FEEDBACK\""
           [ "$DRY_RUN" = "true" ] && ARGS="$ARGS --dry-run"