@kb-labs/agent-core 2.89.0 → 2.93.0

package/dist/index.js

@@ -6102,7 +6102,10 @@ You MUST fix these issues and return a concrete implementation/refactoring plan.
     if (!markdown) {
       issues.push("Plan tool output is missing markdown draft.");
     }
-    const textFields = steps.map((step) => `${step?.action || ""} ${step?.expectedOutcome || ""} ${JSON.stringify(step?.args || {})}`);
+    const textFields = steps.map((step) => {
+      const s = step;
+      return `${s?.action || ""} ${s?.expectedOutcome || ""} ${JSON.stringify(s?.args || {})}`;
+    });
     const hasPlaceholders = textFields.some((text) => /<[^>]+>|\bTBD\b|\bTODO\b/.test(text));
     if (hasPlaceholders) {
       const reason = "Plan contains placeholders instead of concrete file/module references.";
@@ -6143,17 +6146,22 @@ You MUST fix these issues and return a concrete implementation/refactoring plan.
     const changeCount = steps.filter((step) => changeTools.has(String(step?.tool || ""))).length;
     const changeRatio = steps.length > 0 ? changeCount / steps.length : 0;
     const analysisPhases = phases.filter((phase) => /(discovery|inventory|audit|analysis|research)/i.test(String(phase?.name || ""))).length;
-    const solutionStepCount = steps.filter(
-      (step) => /(extract|split|rename|decouple|inject|interface|abstraction|remove|replace|refactor|test|coverage|consolidate|modular)/i.test(`${step?.action || ""} ${step?.expectedOutcome || ""}`)
-    ).length;
+    const solutionStepCount = steps.filter((step) => {
+      const s = step;
+      return /(extract|split|rename|decouple|inject|interface|abstraction|remove|replace|refactor|test|coverage|consolidate|modular)/i.test(`${s?.action || ""} ${s?.expectedOutcome || ""}`);
+    }).length;
     const solutionStepRatio = steps.length > 0 ? solutionStepCount / steps.length : 0;
-    const missingToolCount = steps.filter((step) => typeof step?.tool !== "string" || String(step.tool).trim().length === 0).length;
+    const missingToolCount = steps.filter((step) => {
+      const s = step;
+      return typeof s?.tool !== "string" || String(s.tool).trim().length === 0;
+    }).length;
     if (missingToolCount > 0) {
       issues.push("Some plan steps are missing explicit tools.");
     }
-    const missingActionOrOutcomeCount = steps.filter(
-      (step) => typeof step?.action !== "string" || step.action.trim().length === 0 || typeof step?.expectedOutcome !== "string" || step.expectedOutcome.trim().length === 0
-    ).length;
+    const missingActionOrOutcomeCount = steps.filter((step) => {
+      const s = step;
+      return typeof s?.action !== "string" || s.action.trim().length === 0 || typeof s?.expectedOutcome !== "string" || s.expectedOutcome.trim().length === 0;
+    }).length;
     if (missingActionOrOutcomeCount > 0) {
       const reason = "Some plan steps are missing action/expectedOutcome.";
       issues.push(reason);
@@ -6686,21 +6694,27 @@ You MUST fix these issues and return a concrete implementation/refactoring plan.
    * Normalize phases from tool call
    */
   normalizePhasesFromTool(phases) {
-    return phases.map((p, idx) => ({
-      id: p.id || `phase-${idx + 1}`,
-      name: this.normalizePhaseName(p.name, p.description, p.steps, idx + 1),
-      description: this.normalizePhaseDescription(p.description, p.steps),
-      dependencies: Array.isArray(p.dependencies) ? p.dependencies : [],
-      status: "pending",
-      steps: (p.steps || []).map((s, stepIdx) => ({
-        id: s.id || `step-${idx + 1}-${stepIdx + 1}`,
-        action: this.normalizeStepAction(s.action),
-        tool: s.tool,
-        args: s.args || {},
-        expectedOutcome: this.normalizeExpectedOutcome(s.expectedOutcome, s.action),
-        status: "pending"
-      }))
-    }));
+    return phases.map((phase, idx) => {
+      const p = phase;
+      return {
+        id: typeof p.id === "string" ? p.id : `phase-${idx + 1}`,
+        name: this.normalizePhaseName(p.name, p.description, p.steps, idx + 1),
+        description: this.normalizePhaseDescription(p.description, p.steps),
+        dependencies: Array.isArray(p.dependencies) ? p.dependencies : [],
+        status: "pending",
+        steps: (Array.isArray(p.steps) ? p.steps : []).map((step, stepIdx) => {
+          const s = step;
+          return {
+            id: typeof s.id === "string" ? s.id : `step-${idx + 1}-${stepIdx + 1}`,
+            action: this.normalizeStepAction(s.action),
+            tool: typeof s.tool === "string" ? s.tool : void 0,
+            args: s.args && typeof s.args === "object" ? s.args : {},
+            expectedOutcome: this.normalizeExpectedOutcome(s.expectedOutcome, s.action),
+            status: "pending"
+          };
+        })
+      };
+    });
   }
   parsePhasesFromMarkdown(markdown) {
     const source = typeof markdown === "string" ? markdown.replace(/\r\n/g, "\n") : "";