scai 0.1.160 → 0.1.162

package/dist/agents/MainAgent.js

@@ -2,7 +2,6 @@ import { builtInModules } from "../pipeline/registry/moduleRegistry.js";
 import { logInputOutput } from "../utils/promptLogHelper.js";
 import { infoPlanGenStep } from "./infoPlanGenStep.js";
 import { understandIntentStep } from "./understandIntentStep.js";
-import { contextReviewStep } from "./contextReviewStep.js";
 import { transformPlanGenStep } from "./transformPlanGenStep.js";
 import { getDbForRepo } from "../db/client.js";
 import { writeFileStep } from "./writeFileStep.js";
@@ -13,12 +12,13 @@ import { readinessGateStep } from "./readinessGateStep.js";
 import { scopeClassificationStep } from "./scopeClassificationStep.js";
 import { evidenceVerifierStep } from "./evidenceVerifierStep.js";
 import { validateChangesStep } from './validateChangesStep.js';
-import { reasonNextIterationStep } from './reasonNextIterationStep.js';
+import { reasonNextTaskStep } from './reasonNextTaskStep.js';
 import { collaboratorStep } from './collaboratorStep.js';
 import { integrateFeedbackStep } from './integrateFeedbackStep.js';
 import { selectRelevantSourcesStep } from "./selectRelevantSourcesStep.js";
 import { iterationFileSelector } from "./iterationFileSelector.js";
 import { finalAnswerModule } from "../pipeline/modules/finalAnswerModule.js";
+import { reasonNextStep } from "./reasonNextStep.js";
 /* ───────────────────────── registry ───────────────────────── */
 /**
  * Registry mapping action names to their corresponding Module implementations.
@@ -65,42 +65,40 @@ export class MainAgent {
     /* ───────────── main run ───────────── */
     async run() {
         this.runCount = 0;
-        /* ===== BOOT ===== */
+        await this.runBoot();
+        await this.runPrecheck();
+        await this.runScope();
+        await this.runGrounding();
+        await this.runWorkLoop();
+        await this.runFinalize();
+    }
+    /* ───────────── boot ───────────── */
+    async runBoot() {
+        var _a;
         await understandIntentStep.run({ context: this.context });
         await resolveExecutionModeStep.run(this.context);
+        // Boot the task and get the real DB taskId
         this.taskId = bootTaskForRepo(this.context, getDbForRepo(), this.logLine.bind(this));
-        /* ===== PRECHECK ===== */
+        // Ensure context.task exists and has all required fields
+        (_a = this.context).task || (_a.task = {
+            id: this.taskId,
+            projectId: 0, // optionally set to a real projectId if available
+            status: "active",
+            initialQuery: this.context.initContext?.userQuery ?? "",
+            createdAt: new Date().toISOString(),
+            updatedAt: new Date().toISOString(),
+            taskSteps: [],
+        });
+        // If task existed but id wasn’t set, set it now
+        this.context.task.id = this.taskId;
+    }
+    /* ───────────── precheck ───────────── */
+    async runPrecheck() {
         await fileCheckStep(this.context);
-        /* ===== SCOPE ===== */
+    }
+    /* ───────────── scope ───────────── */
+    async runScope() {
         await scopeClassificationStep.run(this.context);
-        /* ===== GROUNDING / INFO ACQUISITION ===== */
-        await this.runGrounding();
-        /* ===== WORK LOOP ===== */
-        const MAX_ITERATIONS = 5;
-        let loopCount = 0;
-        // Helper: determine next iteration action
-        const getNextIterationAction = () => {
-            const nextAction = this.context.analysis?.iterationReasoning?.nextAction;
-            if (!nextAction)
-                return "complete"; // default safe exit
-            return nextAction;
-        };
-        while (loopCount < MAX_ITERATIONS) {
-            this.runCount++;
-            loopCount++;
-            await this.runWorkIteration();
-            const nextAction = getNextIterationAction();
-            this.logLine("LOOP", "nextAction", undefined, nextAction);
-            if (nextAction === "complete")
-                break;
-            if (nextAction === "request-feedback")
-                return;
-        }
-        /* ===== FINALIZE ===== */
-        // Directly generate the final answer for the user
-        await finalAnswerModule.run({ query: this.query, context: this.context });
-        // Persist task as usual
-        persistTaskData(this.context, this.taskId, getDbForRepo(), this.logLine.bind(this));
     }
     /* ───────────── grounding / info acquisition loop ───────────── */
     async runGrounding() {
@@ -146,25 +144,106 @@ export class MainAgent {
             }
         }
     }
+    /* ───────────── finalize ───────────── */
+    async runFinalize() {
+        // generate final answer
+        await finalAnswerModule.run({ query: this.query, context: this.context });
+        // persist task and steps
+        persistTaskData(this.context, this.taskId, getDbForRepo(), this.logLine.bind(this));
+    }
+    /* ───────────── work loop ───────────── */
+    async runWorkLoop() {
+        const MAX_TASK_STEPS = 5;
+        let stepCount = 0;
+        while (stepCount < MAX_TASK_STEPS) {
+            await reasonNextTaskStep.run(this.context);
+            const nextAction = this.context.analysis?.iterationReasoning?.nextAction;
+            if (nextAction === "complete") {
+                this.logLine("TASK", "All selected files processed — task complete");
+                return;
+            }
+            const taskStep = await iterationFileSelector.run(this.context);
+            if (!taskStep) {
+                this.logLine("TASK", "No eligible taskStep found — task complete");
+                return;
+            }
+            // Set current step in context
+            if (this.context.task) {
+                this.context.task.currentStep = taskStep;
+            }
+            stepCount++;
+            taskStep.taskId = this.taskId;
+            taskStep.stepIndex = stepCount;
+            taskStep.status = "pending";
+            // ⬇️ Persist newly created step
+            persistTaskStepInsert(taskStep, getDbForRepo());
+            logInputOutput("Step to process", "input", taskStep);
+            this.logLine("TASK", `Processing taskStep ${stepCount}/${MAX_TASK_STEPS}`, undefined, taskStep.filePath);
+            // ---------------------------
+            //  Start the step
+            // ---------------------------
+            taskStep.startTime = Date.now();
+            persistTaskStepStart(taskStep, getDbForRepo());
+            // ---------------------------
+            // Step-level iterations: reason only about this file
+            // ---------------------------
+            const stepAction = await this.runStepIterations(taskStep);
+            if (stepAction === "complete") {
+                taskStep.status = "completed";
+                taskStep.endTime = Date.now();
+                persistTaskStepCompletion(taskStep, getDbForRepo());
+            }
+            else {
+                // optionally mark as pending or redo-step
+                taskStep.status = "pending";
+                persistTaskStepCompletion(taskStep, getDbForRepo());
+            }
+        }
+        this.logLine("TASK", "Max task step limit reached — stopping work loop");
+    }
+    /* ───────────── step iterations ───────────── */
+    async runStepIterations(taskStep) {
+        const MAX_ITERATIONS = 5;
+        let loopCount = 0;
+        const getNextIterationAction = () => {
+            const nextAction = this.context.analysis?.iterationReasoning?.nextAction;
+            if (nextAction !== "continue" &&
+                nextAction !== "redo-step" &&
+                nextAction !== "expand-scope" &&
+                nextAction !== "request-feedback" &&
+                nextAction !== "complete") {
+                return "complete"; // fallback
+            }
+            return nextAction;
+        };
+        // Loop through iterations for a single task step (file)
+        while (loopCount < MAX_ITERATIONS) {
+            this.runCount++;
+            loopCount++;
+            // 🔴 HARD RESET — no carryover allowed
+            if (this.context.analysis?.iterationReasoning) {
+                this.context.analysis.iterationReasoning.nextAction = undefined;
+            }
+            await this.runWorkIteration(taskStep);
+            const nextAction = getNextIterationAction();
+            this.logLine("STEP-LOOP", "nextAction", undefined, nextAction);
+            if (nextAction === "complete")
+                return "complete";
+            if (nextAction === "request-feedback")
+                return "request-feedback";
+            // else: continue
+        }
+        return "continue";
+    }
     /* ───────────── work iteration ───────────── */
-    async runWorkIteration() {
-        // ---------------- FILE SELECTION ----------------
-        const tSelect = this.startTimer();
-        await iterationFileSelector.run(this.context); // <-- sets context.analysis.currentTargetFile
-        this.logLine("LOOP", "iterationFileSelector", tSelect());
+    async runWorkIteration(taskStep) {
+        if (!this.context.analysis)
+            this.context.analysis = {};
         // ---------------- ANALYSIS ----------------
         if (this.canExecutePhase("analysis") && this.canExecuteScope("analysis")) {
             const tAnalysis = this.startTimer();
             await analysisPlanGenStep.run(this.context);
             this.logLine("PLAN", "analysisPlanGen", tAnalysis());
-            /* // 🔎 DEBUG BREAKPOINT — inspect context BEFORE analysis planning
-            debugContext(this.context, {
-                step: "pre-analysisPlanGen",
-                note: "After iterationFileSelector, before analysisPlanGen",
-                exit: true,      // set to false if you want execution to continue
-                depth: 6
-            });
- */
             const analysisPlan = this.context.analysis?.planSuggestion?.plan ?? { steps: [] };
             for (const step of analysisPlan.steps) {
                 const tStep = this.startTimer();
@@ -181,13 +260,11 @@ export class MainAgent {
             await transformPlanGenStep.run(this.context);
             this.logLine("PLAN", "transformPlanGen", tTransform());
             const transformPlan = this.context.analysis?.planSuggestion?.plan ?? { steps: [] };
-            let stepCounter = 1; // 🔹 initialize step counter
-            for (const step of transformPlan.steps) {
+            const firstStep = transformPlan.steps[0];
+            if (firstStep) {
                 const tStep = this.startTimer();
-                await this.executeStep(step, { query: this.query });
-                // 🔹 include step number in log
-                this.logLine("STEP", `#${stepCounter} - ${step.action || "unnamedStep"}`, tStep());
-                stepCounter++; // 🔹 increment counter
+                await this.executeStep(firstStep, { query: this.query });
+                this.logLine("STEP", `#1 (only) - ${firstStep.action || "unnamedStep"}`, tStep());
             }
             if (this.context.analysis) {
                 this.context.analysis.planSuggestion = undefined;
@@ -205,8 +282,8 @@ export class MainAgent {
         }
         // ---------------- REASONING ----------------
         const tReason = this.startTimer();
-        await reasonNextIterationStep.run(this.context);
-        this.logLine("REASONING", "reasonNextIterationStep", tReason());
+        await reasonNextStep.run(this.context, taskStep);
+        this.logLine("REASONING", "reasonNextStep", tReason());
         // ---------------- COLLABORATOR FEEDBACK ----------------
         const tCollab = this.startTimer();
         await collaboratorStep.run(this.context);
@@ -215,22 +292,6 @@ export class MainAgent {
         const tIntegrate = this.startTimer();
         await integrateFeedbackStep.run(this.context);
         this.logLine("FEEDBACK", "integrateFeedbackStep", tIntegrate());
-        // ---------------- REVIEW / RECOVERY ----------------
-        const tReview = this.startTimer();
-        const review = await contextReviewStep(this.context);
-        this.logLine("REVIEW", "contextReviewStep", tReview());
-        if (review.decision === "gatherData" && this.runCount < this.maxRuns) {
-            this.runCount++;
-            if (this.context.analysis) {
-                this.context.analysis.readiness = {
-                    decision: 'not-ready',
-                    confidence: 0,
-                    rationale: 'Additional data gathering required.'
-                };
-                this.context.analysis.planSuggestion = undefined;
-            }
-            this.logLine("REVIEW", "contextReviewStep", undefined, "Not ready, looping back for additional data gathering");
-        }
     }
     /* ───────────── step executor ───────────── */
     /**
@@ -276,29 +337,22 @@ export class MainAgent {
      * @returns True if the phase can be executed, false otherwise.
      */
     canExecutePhase(phase) {
-        const mode = this.context.executionControl?.mode ?? "transform";
         const constraints = this.context.executionControl?.constraints;
-        let allowed;
-        // If docsOnly is true, prevent any non-doc actions
         const docsOnly = constraints?.docsOnly ?? false;
+        let allowed = false;
         switch (phase) {
             case "analysis":
-                allowed = !docsOnly && (constraints?.allowAnalysis ?? (mode !== "explain"));
-                break;
             case "planning":
-                allowed = !docsOnly && (constraints?.allowPlanning ?? (mode === "analyze" || mode === "transform"));
+                // Read-only phases are always allowed
+                allowed = !docsOnly;
                 break;
             case "transform":
             case "write":
-                allowed = constraints?.allowFileWrites ?? (mode === "transform");
-                break;
-            case "explain":
-                allowed = mode === "explain"; // explain is purely mode-based
+                // Side-effect phases require explicit permission
+                allowed = constraints?.allowFileWrites ?? false;
                 break;
-            default:
-                allowed = false;
         }
-        this.logLine("EXEC", "canExecutePhase", undefined, `phase=${phase}, mode=${mode}, docsOnly=${docsOnly}, allowed=${allowed}`);
+        this.logLine("EXEC", "canExecutePhase", undefined, `phase=${phase}, docsOnly=${docsOnly}, allowed=${allowed}`);
         return allowed;
     }
     /* ───────────── scope gates ───────────── */
@@ -310,24 +364,17 @@ export class MainAgent {
      */
     canExecuteScope(phase) {
         const scope = this.context.analysis?.scopeType ?? "repo-wide";
-        let allowed;
+        let allowed = false;
         switch (scope) {
             case "none":
-                // Non-repo queries: only explain makes sense
-                allowed = phase === "explain";
+                // Non-repo questions: analysis/planning only
+                allowed = phase === "analysis" || phase === "planning";
                 break;
             case "single-file":
             case "multi-file":
-                // Local scopes: all phases are safe
-                allowed = true;
-                break;
             case "repo-wide":
-                // Repo-wide explain must not short-circuit via analysis
-                // but analysis/planning/transform are allowed
-                allowed = phase !== "explain";
+                allowed = true;
                 break;
-            default:
-                allowed = false;
         }
         this.logLine("EXEC", "canExecuteScope", undefined, `phase=${phase}, scope=${scope}, allowed=${allowed}`);
         return allowed;
@@ -505,3 +552,58 @@ export function bootTaskForRepo(context, db, logLine) {
     logLine("TASK", `created task id = ${taskId}`);
     return taskId;
 }
+export function persistTaskStepInsert(taskStep, db) {
+    if (taskStep.id)
+        return;
+    const nowIso = new Date().toISOString();
+    const result = db.prepare(`
+        INSERT INTO task_steps (
+            task_id,
+            file_path,
+            action,
+            status,
+            step_index,
+            start_time,
+            result_json,
+            notes,
+            created_at,
+            updated_at
+        )
+        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
+    `).run(taskStep.taskId, taskStep.filePath, taskStep.action ?? null, taskStep.status, taskStep.stepIndex ?? null, taskStep.startTime ?? null, taskStep.result != null ? JSON.stringify(taskStep.result) : null, taskStep.notes ?? null, nowIso, nowIso);
+    taskStep.id = result.lastInsertRowid;
+}
+export function persistTaskStepStart(taskStep, db) {
+    if (!taskStep.id)
+        return;
+    if (!taskStep.startTime) {
+        taskStep.startTime = Date.now();
+    }
+    db.prepare(`
+        UPDATE task_steps
+        SET
+            start_time = ?,
+            action = ?,
+            notes = ?,
+            updated_at = ?
+        WHERE id = ?
+    `).run(taskStep.startTime, taskStep.action ?? null, taskStep.notes ?? null, new Date().toISOString(), taskStep.id);
+}
+export function persistTaskStepCompletion(taskStep, db) {
+    if (!taskStep.id)
+        return;
+    if (!taskStep.endTime) {
+        taskStep.endTime = Date.now();
+    }
+    db.prepare(`
+        UPDATE task_steps
+        SET
+            status = ?,
+            end_time = ?,
+            action = ?,
+            result_json = ?,
+            notes = ?,
+            updated_at = ?
+        WHERE id = ?
+    `).run(taskStep.status, taskStep.endTime, taskStep.action ?? null, taskStep.result != null ? JSON.stringify(taskStep.result) : null, taskStep.notes ?? null, new Date().toISOString(), taskStep.id);
+}

package/dist/agents/analysisPlanGenStep.js

@@ -10,22 +10,25 @@ import { cleanupModule } from "../pipeline/modules/cleanupModule.js";
 export const analysisPlanGenStep = {
     name: "analysisPlanGen",
     description: "Generates a single analysis reasoning step.",
-    requires: ["analysis.intent", "analysis.focus", "analysis.routingDecision", "analysis.currentTargetFile"],
+    requires: ["analysis.intent", "analysis.focus", "analysis.routingDecision"],
     produces: ["analysis.planSuggestion"],
     async run(context) {
         context.analysis || (context.analysis = {});
         context.execution || (context.execution = {});
-        // 🔁 Always overwrite previous planSuggestion for analysis phase
+        // Ensure task exists (required for currentStep)
+        if (!context.task) {
+            throw new Error("analysisPlanGenStep: context.task must exist — ensure runBoot has been called");
+        }
+        // 🔁 Always overwrite previous planSuggestion
         delete context.analysis.planSuggestion;
-        const targetFile = context.analysis.currentTargetFile;
-        const routingDecision = context.analysis.routingDecision?.decision;
-        // Determine whether semantic analysis has already been performed
-        const hasSemanticAnalysis = !!targetFile && !!context.analysis.fileAnalysis?.[targetFile];
-        // Only skip if we truly have nothing to analyze
-        if (!targetFile) {
+        // Use the DB-backed currentStep
+        const currentStep = context.task.currentStep;
+        if (!currentStep?.filePath) {
             context.analysis.planSuggestion = { plan: { steps: [] } };
+            logInputOutput("analysisPlanGen", "output", { steps: [] });
             return;
         }
+        const targetFile = currentStep.filePath;
         // Restrict actions to ANALYSIS only
         const effectiveActions = PLAN_ACTIONS.filter(a => a.groups?.includes("analysis"));
         if (!effectiveActions.length) {
@@ -33,16 +36,8 @@ export const analysisPlanGenStep = {
             logInputOutput("analysisPlanGen", "output", { steps: [] });
             return;
         }
-        // Guard: nothing to analyze
-        if (!targetFile) {
-            context.analysis.planSuggestion = { plan: { steps: [] } };
-            logInputOutput("analysisPlanGen", "output", { steps: [] });
-            return;
-        }
         const actionsJson = JSON.stringify(effectiveActions, null, 2);
-        const intentText = context.analysis.intent?.normalizedQuery ??
-            context.initContext?.userQuery ??
-            "";
+        const intentText = context.analysis.intent?.normalizedQuery ?? context.initContext?.userQuery ?? "";
         const intentCategory = context.analysis.intent?.intentCategory ?? "";
         const relevantFiles = JSON.stringify([targetFile], null, 2);
         const rationaleText = context.analysis.focus?.rationale ?? "";
@@ -87,29 +82,23 @@ Return a valid JSON object in this format:
 }
         `.trim();
         try {
-            logInputOutput("analysis prompt", "input", prompt);
             const genInput = { query: intentText, content: prompt };
             const genOutput = await generate(genInput);
             const raw = typeof genOutput.data === "string"
                 ? genOutput.data
                 : JSON.stringify(genOutput.data ?? "{}");
-            const cleaned = await cleanupModule.run({
-                query: intentText,
-                content: raw
-            });
+            const cleaned = await cleanupModule.run({ query: intentText, content: raw });
             const jsonString = typeof cleaned.content === "string"
                 ? cleaned.content
                 : JSON.stringify(cleaned.content ?? "{}");
             const planObj = JSON.parse(jsonString);
-            const steps = planObj.step
-                ? [planObj.step]
-                : [];
+            const steps = planObj.step ? [planObj.step] : [];
             const finalSteps = steps.map((step) => {
                 const actionDef = PLAN_ACTIONS.find(a => a.action === step.action);
                 return {
                     ...step,
                     groups: actionDef?.groups ?? ["analysis"],
-                    metadata: step.metadata ?? {}
+                    metadata: step.metadata ?? {},
                 };
             });
             context.analysis.planSuggestion = { plan: { steps: finalSteps } };

package/dist/agents/evidenceVerifierStep.js

@@ -21,7 +21,6 @@ export const evidenceVerifierStep = {
         const candidatePaths = [
             ...(context.analysis.focus?.selectedFiles ?? []),
             ...(context.analysis.focus?.candidateFiles ?? []),
-            ...(context.workingFiles?.map(f => f.path) ?? []),
         ];
         const uniquePaths = Array.from(new Set(candidatePaths));
         if (!uniquePaths.length) {
@@ -30,24 +29,37 @@ export const evidenceVerifierStep = {
         }
         const filesWithEvidence = [];
         // ----------------- Parse query for targets -----------------
-        const sentenceRegex = /['"`](.+?)['"`]/g;
         const sentenceTargets = [];
+        // 1️⃣ Capture text inside quotes (existing behavior)
+        const quoteRegex = /['"`](.+?)['"`]/g;
         let match;
-        while ((match = sentenceRegex.exec(query)) !== null) {
+        while ((match = quoteRegex.exec(query)) !== null) {
             sentenceTargets.push(match[1]);
         }
+        // 2️⃣ Capture heuristic unquoted sentences if none found inside quotes
+        if (!sentenceTargets.length) {
+            const heuristicSentences = query
+                .split(/[\.\n]/)
+                .map(s => s.trim())
+                .filter(s => s.length > 10);
+            heuristicSentences.forEach(s => {
+                if (!sentenceTargets.includes(s)) {
+                    sentenceTargets.push(s);
+                }
+            });
+        }
         const filenameTargets = query
             .split(/\s+/)
-            .filter(w => w.match(/\.(ts|js|tsx|md)$/));
+            .map(word => word.replace(/['",]/g, ''))
+            .filter(w => w.match(/\.(ts|js|tsx|md)$/) || w.length > 5);
+        const baseNameTargets = filenameTargets.map(t => t.replace(/\.(ts|js|tsx|md)$/, ''));
         const symbolTargets = [];
-        const symbolRegex = /\b([A-Z]\w+|[a-z]\w+)\(\)/g;
+        const symbolRegex = /\b([a-zA-Z_]\w+)(?:\(\))?\b/g;
         let symMatch;
         while ((symMatch = symbolRegex.exec(query)) !== null) {
             symbolTargets.push(symMatch[1]);
         }
         const mode = context.executionControl?.mode ?? "transform";
-        const allowAnalysis = context.executionControl?.constraints?.allowAnalysis ?? mode !== "explain";
-        const allowWrites = context.executionControl?.constraints?.allowFileWrites ?? mode === "transform";
         // ----------------- Process each file -----------------
         for (const path of uniquePaths) {
             let code = null;
@@ -86,11 +98,12 @@ export const evidenceVerifierStep = {
                 });
             });
             // Filename-level evidence
-            const fileName = path.split("/").pop() ?? "";
-            if (filenameTargets.includes(fileName)) {
+            const fullFileName = path.split("/").pop() ?? "";
+            const baseFileName = fullFileName.replace(/\.(ts|js|tsx|md)$/, "");
+            if (filenameTargets.includes(fullFileName) || baseNameTargets.includes(baseFileName)) {
                 evidenceItems.push({
-                    claim: `Filename matches query target: "${fileName}"`,
-                    excerpt: "",
+                    claim: `Filename matches query target: "${fullFileName}"`,
+                    excerpt: `Path: ${path}`,
                     span: { startLine: 0, endLine: 0 },
                     confidence: 1,
                 });
@@ -104,10 +117,9 @@ export const evidenceVerifierStep = {
                     intent: "relevant",
                     relevanceExplanation: `${evidenceItems.length} evidence item(s) match the query${isFocusFile ? " (focus file already selected)" : ""}`,
                     role: "primary",
-                    // 🔑 KEY FIX: modification is evidence-gated
                     action: {
                         isRelevant: true,
-                        shouldModify: hasEvidence && allowAnalysis && allowWrites,
+                        shouldModify: hasEvidence
                     },
                     proposedChanges: hasEvidence
                         ? {

package/dist/agents/finalPlanGenStep.js

@@ -74,9 +74,12 @@ Return strictly valid JSON representing one step:
             // Map groups & metadata
             const actionDef = PLAN_ACTIONS.find(a => a.action === step.action);
             step.groups = actionDef?.groups ?? ['finalize'];
+            const confidence = context.analysis?.routingDecision?.confidence;
             step.metadata = {
                 ...step.metadata,
-                routingConfidence: context.analysis?.routingDecision?.confidence ?? 0
+                ...(typeof confidence === 'number' && confidence > 0
+                    ? { routingConfidence: confidence }
+                    : {})
             };
             // Save single step to context
             context.analysis.planSuggestion = { plan: { steps: [step] } };
@@ -85,12 +88,15 @@ Return strictly valid JSON representing one step:
         catch (err) {
             console.warn('⚠️ Failed to generate final step:', err);
             // Fallback: always include finalAnswer
+            const confidence = context.analysis?.routingDecision?.confidence;
             const step = {
                 id: 'finalize',
                 action: 'finalAnswer',
                 description: 'Generate final user-facing response summarizing results.',
                 metadata: {
-                    routingConfidence: context.analysis?.routingDecision?.confidence ?? 0
+                    ...(typeof confidence === 'number' && confidence > 0
+                        ? { routingConfidence: confidence }
+                        : {})
                 },
                 groups: ['finalize']
             };

package/dist/agents/infoPlanGenStep.js

@@ -71,9 +71,13 @@ If no further information is required, return:
         try {
             const genInput = { query: intentText, content: prompt };
             const genOutput = await generate(genInput);
-            const raw = typeof genOutput.data === 'string' ? genOutput.data : JSON.stringify(genOutput.data ?? '{}');
+            const raw = typeof genOutput.data === 'string'
+                ? genOutput.data
+                : JSON.stringify(genOutput.data ?? '{}');
             const cleaned = await cleanupModule.run({ query: intentText, content: raw });
-            const jsonString = typeof cleaned.content === 'string' ? cleaned.content : JSON.stringify(cleaned.content ?? '{}');
+            const jsonString = typeof cleaned.content === 'string'
+                ? cleaned.content
+                : JSON.stringify(cleaned.content ?? '{}');
             let step = JSON.parse(jsonString);
             // Validate structure
             if (!step || typeof step.action !== 'string') {
@@ -83,9 +87,12 @@ If no further information is required, return:
             if (step) {
                 const actionDef = PLAN_ACTIONS.find(a => a.action === step.action);
                 step.groups = actionDef?.groups ?? ['info'];
+                const confidence = analysis?.routingDecision?.confidence;
                 step.metadata = {
                     ...step.metadata,
-                    routingConfidence: analysis?.routingDecision?.confidence ?? 0
+                    ...(typeof confidence === 'number' && confidence > 0
+                        ? { routingConfidence: confidence }
+                        : {})
                 };
             }
             // Save single step to context as an array (empty if null)