karajan-code 1.2.2 → 1.2.3

package/src/prompts/planner.js

@@ -1,3 +1,54 @@
+export function parsePlannerOutput(output) {
+  const text = String(output || "").trim();
+  if (!text) return null;
+
+  const lines = text
+    .split(/\r?\n/)
+    .map((line) => line.trim())
+    .filter(Boolean);
+
+  let title = null;
+  let approach = null;
+  const steps = [];
+
+  for (const line of lines) {
+    if (!title) {
+      const titleMatch = line.match(/^title\s*:\s*(.+)$/i);
+      if (titleMatch) {
+        title = titleMatch[1].trim();
+        continue;
+      }
+    }
+
+    if (!approach) {
+      const approachMatch = line.match(/^(approach|strategy)\s*:\s*(.+)$/i);
+      if (approachMatch) {
+        approach = approachMatch[2].trim();
+        continue;
+      }
+    }
+
+    const numberedStep = line.match(/^\d+[\).:-]\s*(.+)$/);
+    if (numberedStep) {
+      steps.push(numberedStep[1].trim());
+      continue;
+    }
+
+    const bulletStep = line.match(/^[-*]\s+(.+)$/);
+    if (bulletStep) {
+      steps.push(bulletStep[1].trim());
+      continue;
+    }
+  }
+
+  if (!title) {
+    const firstFreeLine = lines.find((line) => !/^(approach|strategy)\s*:/i.test(line) && !/^\d+[\).:-]\s*/.test(line));
+    title = firstFreeLine || null;
+  }
+
+  return { title, approach, steps };
+}
+
 export function buildPlannerPrompt({ task, context }) {
   const parts = [
     "You are an expert software architect. Create an implementation plan for the following task.",

package/src/repeat-detector.js

@@ -40,6 +40,17 @@ function parseThreshold(value) {
   return DEFAULT_THRESHOLD;
 }
 
+export function getRepeatThreshold(config) {
+  const raw =
+    config?.failFast?.repeatThreshold ??
+    config?.session?.repeat_detection_threshold ??
+    config?.session?.fail_fast_repeats ??
+    2;
+  const value = Number(raw);
+  if (Number.isFinite(value) && value > 0) return value;
+  return 2;
+}
+
 export class RepeatDetector {
   constructor({ threshold = DEFAULT_THRESHOLD } = {}) {
     this.threshold = parseThreshold(threshold);

package/src/roles/coder-role.js

@@ -29,7 +29,8 @@ export class CoderRole extends BaseRole {
       reviewerFeedback: reviewerFeedback || null,
       sonarSummary: sonarSummary || null,
       coderRules: this.instructions,
-      methodology: this.config?.development?.methodology || "tdd"
+      methodology: this.config?.development?.methodology || "tdd",
+      serenaEnabled: Boolean(this.config?.serena?.enabled)
     });
 
     const coderArgs = { prompt, role: "coder" };
@@ -41,6 +42,7 @@ export class CoderRole extends BaseRole {
       return {
         ok: false,
         result: {
+          ...result,
           error: result.error || result.output || "Coder failed",
           provider
         },
@@ -51,6 +53,7 @@ export class CoderRole extends BaseRole {
     return {
       ok: true,
       result: {
+        ...result,
         output: result.output || "",
         provider
       },

package/src/roles/planner-role.js

@@ -66,7 +66,7 @@ export class PlannerRole extends BaseRole {
     if (!result.ok) {
       return {
         ok: false,
-        result: { error: result.error || result.output || "Planner agent failed", plan: null },
+        result: { ...result, error: result.error || result.output || "Planner agent failed", plan: null },
         summary: `Planner failed: ${result.error || "unknown error"}`
       };
     }
@@ -74,7 +74,7 @@ export class PlannerRole extends BaseRole {
     const plan = result.output?.trim() || "";
     return {
       ok: true,
-      result: { plan, provider },
+      result: { ...result, plan, provider },
       summary: plan ? `Plan generated (${plan.split("\n").length} lines)` : "Empty plan generated"
     };
   }

package/src/roles/refactorer-role.js

@@ -47,6 +47,7 @@ export class RefactorerRole extends BaseRole {
       return {
         ok: false,
         result: {
+          ...result,
           error: result.error || result.output || "Refactorer failed",
           provider
         },
@@ -57,6 +58,7 @@ export class RefactorerRole extends BaseRole {
     return {
       ok: true,
       result: {
+        ...result,
         output: result.output?.trim() || "",
         provider
       },

package/src/roles/reviewer-role.js

@@ -101,29 +101,36 @@ export class ReviewerRole extends BaseRole {
 
     try {
       const parsed = parseReviewOutput(result.output);
-      const approved = Boolean(parsed.approved);
       const blockingIssues = parsed.blocking_issues || [];
 
       return {
         ok: true,
         result: {
-          approved,
+          ...result,
+          approved: parsed.approved,
           blocking_issues: blockingIssues,
           non_blocking_suggestions: parsed.non_blocking_suggestions || [],
           confidence: parsed.confidence ?? null,
           raw_summary: parsed.summary || ""
         },
-        summary: approved
+        summary: parsed.approved
           ? `Approved: ${parsed.summary || "no issues found"}`
           : `Rejected: ${blockingIssues.length} blocking issue(s) — ${parsed.summary || ""}`
       };
     } catch (err) {
       return {
-        ok: false,
+        ok: true,
         result: {
-          error: `Failed to parse reviewer output: ${err.message}`,
+          ...result,
           approved: false,
-          blocking_issues: []
+          blocking_issues: [{
+            id: "PARSE_ERROR",
+            severity: "high",
+            description: `Reviewer output could not be parsed: ${err.message}`
+          }],
+          non_blocking_suggestions: [],
+          confidence: 0,
+          raw_summary: `Parse error: ${err.message}`
         },
         summary: `Reviewer output parse error: ${err.message}`
       };

package/src/utils/budget.js

@@ -1,5 +1,35 @@
 import { calculateUsageCostUsd, DEFAULT_MODEL_PRICING, mergePricing } from "./pricing.js";
 
+export function extractUsageMetrics(result, defaultModel = null) {
+  const usage = result?.usage || result?.metrics || {};
+  const tokens_in =
+    result?.tokens_in ??
+    usage?.tokens_in ??
+    usage?.input_tokens ??
+    usage?.prompt_tokens ??
+    0;
+  const tokens_out =
+    result?.tokens_out ??
+    usage?.tokens_out ??
+    usage?.output_tokens ??
+    usage?.completion_tokens ??
+    0;
+  const cost_usd =
+    result?.cost_usd ??
+    usage?.cost_usd ??
+    usage?.usd_cost ??
+    usage?.cost;
+  const model =
+    result?.model ??
+    usage?.model ??
+    usage?.model_name ??
+    usage?.model_id ??
+    defaultModel ??
+    null;
+
+  return { tokens_in, tokens_out, cost_usd, model };
+}
+
 function toSafeNumber(value) {
   const n = Number(value);
   if (!Number.isFinite(n) || n < 0) return 0;

package/src/utils/pricing.js

@@ -1,16 +1,6 @@
-export const DEFAULT_MODEL_PRICING = {
-  "claude": { input_per_million: 3, output_per_million: 15 },
-  "claude/sonnet": { input_per_million: 3, output_per_million: 15 },
-  "claude/opus": { input_per_million: 15, output_per_million: 75 },
-  "claude/haiku": { input_per_million: 0.25, output_per_million: 1.25 },
-  "codex": { input_per_million: 1.5, output_per_million: 4 },
-  "codex/o4-mini": { input_per_million: 1.5, output_per_million: 4 },
-  "codex/o3": { input_per_million: 10, output_per_million: 40 },
-  "gemini": { input_per_million: 1.25, output_per_million: 5 },
-  "gemini/pro": { input_per_million: 1.25, output_per_million: 5 },
-  "gemini/flash": { input_per_million: 0.075, output_per_million: 0.3 },
-  "aider": { input_per_million: 3, output_per_million: 15 }
-};
+import { buildDefaultPricingTable } from "../agents/model-registry.js";
+
+export const DEFAULT_MODEL_PRICING = buildDefaultPricingTable();
 
 export function calculateUsageCostUsd({ model, tokens_in, tokens_out, pricing }) {
   const table = pricing || DEFAULT_MODEL_PRICING;