scai 0.1.161 → 0.1.163

package/dist/agents/MainAgent.js

@@ -155,9 +155,6 @@ export class MainAgent {
     async runWorkLoop() {
         const MAX_TASK_STEPS = 5;
         let stepCount = 0;
-        // ---------------------------
-        // 1️⃣ Task-level reasoning: decide remaining files / next action
-        // ---------------------------
         while (stepCount < MAX_TASK_STEPS) {
             await reasonNextTaskStep.run(this.context);
             const nextAction = this.context.analysis?.iterationReasoning?.nextAction;
@@ -165,32 +162,43 @@ export class MainAgent {
                 this.logLine("TASK", "All selected files processed — task complete");
                 return;
             }
-            // ---------------------------
-            // 2️⃣ Select next file to process
-            // ---------------------------
             const taskStep = await iterationFileSelector.run(this.context);
             if (!taskStep) {
                 this.logLine("TASK", "No eligible taskStep found — task complete");
                 return;
             }
-            // Only assign if taskStep is not null and context.task is defined
-            if (taskStep && this.context.task) {
+            // 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);
             // ---------------------------
-            // 3️⃣ Step-level iterations: reason only about this file
+            //  Start the step
+            // ---------------------------
+            taskStep.startTime = Date.now();
+            persistTaskStepStart(taskStep, getDbForRepo());
             // ---------------------------
-            await this.runStepIterations(taskStep);
-            // Mark step complete if iterationReasoning says complete for this file
-            const stepAction = this.context.analysis?.iterationReasoning?.nextAction;
+            // 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());
             }
         }
-        // Safety exit
         this.logLine("TASK", "Max task step limit reached — stopping work loop");
     }
     /* ───────────── step iterations ───────────── */
@@ -208,8 +216,7 @@ export class MainAgent {
             }
             return nextAction;
         };
-        // Here we loop through iterations for a single task step (file), 
-        // allowing the agent to reason about completeness, redo work, or request feedback.
+        // Loop through iterations for a single task step (file)
         while (loopCount < MAX_ITERATIONS) {
             this.runCount++;
             loopCount++;
@@ -221,11 +228,12 @@ export class MainAgent {
             const nextAction = getNextIterationAction();
             this.logLine("STEP-LOOP", "nextAction", undefined, nextAction);
             if (nextAction === "complete")
-                return;
+                return "complete";
             if (nextAction === "request-feedback")
-                return;
+                return "request-feedback";
             // else: continue
         }
+        return "continue";
     }
     /* ───────────── work iteration ───────────── */
     async runWorkIteration(taskStep) {
@@ -312,7 +320,6 @@ export class MainAgent {
                 content: input.data ?? input.content,
                 context: this.context
             });
-            this.logLine("EXECUTE", step.action, stop());
             // Return ModuleIO (can remain minimal since output is untyped)
             return { query: step.description ?? input.query, data: {} };
         }
@@ -344,7 +351,6 @@ export class MainAgent {
                 allowed = constraints?.allowFileWrites ?? false;
                 break;
         }
-        this.logLine("EXEC", "canExecutePhase", undefined, `phase=${phase}, docsOnly=${docsOnly}, allowed=${allowed}`);
         return allowed;
     }
     /* ───────────── scope gates ───────────── */
@@ -368,7 +374,6 @@ export class MainAgent {
                 allowed = true;
                 break;
         }
-        this.logLine("EXEC", "canExecuteScope", undefined, `phase=${phase}, scope=${scope}, allowed=${allowed}`);
         return allowed;
     }
     /* ----------------------------------- */
@@ -544,3 +549,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/selectRelevantSourcesStep.js

@@ -24,8 +24,9 @@ export const selectRelevantSourcesStep = {
         const discarded = [];
         const rationaleParts = [];
         // ---------------------------
-        // 1️⃣ Classify files by evidence
+        // 1️⃣ Classify files by relevance, evidence, and mode
         // ---------------------------
+        const mode = context.executionControl?.mode ?? "transform";
         for (const [path, analysis] of Object.entries(fileAnalysis)) {
             const isRelevant = analysis.action?.isRelevant === true;
             if (!isRelevant) {
@@ -41,11 +42,17 @@ export const selectRelevantSourcesStep = {
             catch {
                 code = undefined;
             }
-            if (hasEvidence && canModify) {
+            // 🔑 Mode-aware selection rule
+            const isSelected = mode === "transform"
+                ? hasEvidence && canModify
+                : true; // analyze: relevance is sufficient
+            if (isSelected) {
                 selected.push({
                     path,
                     code,
-                    selectionReason: "Selected: concrete evidence found and modification permitted.",
+                    selectionReason: mode === "transform"
+                        ? "Selected: concrete evidence found and modification permitted."
+                        : "Selected: relevant focus file for semantic analysis.",
                 });
             }
             else {

package/dist/commands/TasksCmd.js

@@ -55,11 +55,9 @@ export function showTaskDetails(taskId) {
         const isArray = Array.isArray(value);
         const isObject = typeof value === "object" && !isArray;
         if (typeof value === "string" && value.length < 80) {
-            // Short string: inline
             console.log(chalk.yellow(`${label}:`), value);
         }
         else if (isArray) {
-            // Arrays: print first 5 items each on new line, then show count if truncated
             console.log(chalk.yellow(`${label}:`));
             const items = value;
             const displayCount = Math.min(5, items.length);
@@ -71,24 +69,20 @@ export function showTaskDetails(taskId) {
             }
         }
         else if (isObject) {
-            // Objects: pretty print JSON
             console.log(chalk.yellow(`${label}:`));
             console.log(indent + JSON.stringify(value, null, 2).replace(/\n/g, "\n" + indent));
         }
         else {
-            // Long string: print on next line with indentation
             console.log(chalk.yellow(`${label}:`));
             console.log(indent + String(value));
         }
     };
     // Lifecycle
-    // Lifecycle
     print("Initial query", task.initial_query);
-    // Final answer (canonical)
     if (task.summary) {
         console.log(chalk.green.bold("\nFinal answer:\n"));
         console.log(chalk.white(task.summary));
-        console.log(); // spacing
+        console.log();
     }
     // Intent
     print("Agreed intent", task.agreed_intent);
@@ -124,6 +118,28 @@ export function showTaskDetails(taskId) {
     // Bookkeeping
     print("Created", task.created_at);
     print("Updated", task.updated_at);
+    // ----------------- TASK STEPS -----------------
+    const steps = db.prepare(`
+        SELECT id, file_path, action, status, step_index, start_time, end_time, created_at, updated_at
+        FROM task_steps
+        WHERE task_id = ?
+        ORDER BY step_index ASC
+    `).all(taskId);
+    if (steps.length === 0) {
+        console.log(chalk.dim("\nNo task steps recorded."));
+    }
+    else {
+        console.log(chalk.bold("\nTask Steps:\n"));
+        steps.forEach(s => {
+            console.log(`${chalk.cyan(`[${s.id}]`)} ` +
+                `${chalk.yellow((s.status || "").padEnd(9))} ` +
+                `${s.file_path}` +
+                (s.action ? ` → ${s.action}` : "") +
+                ` (index=${s.step_index ?? "-"})` +
+                (s.start_time ? ` started=${s.start_time}` : "") +
+                (s.end_time ? ` ended=${s.end_time}` : ""));
+        });
+    }
 }
 /** Helper: truncate long text to a single line */
 function oneLineTruncate(input, max = 80) {

package/dist/db/schema.js

@@ -52,6 +52,7 @@ export function initSchema() {
     project_id INTEGER NOT NULL,
 
     -- lifecycle
+    -- active | completed | paused | needs-feedback
     status TEXT NOT NULL DEFAULT 'active',
 
     -- user-facing / context
@@ -71,6 +72,9 @@ export function initSchema() {
     -- routing decision
     routing_decision_json TEXT,
 
+    -- 🔴 task-level feedback / questions (append-only)
+    task_questions_json TEXT,
+
     -- bookkeeping
     created_at TEXT NOT NULL,
     updated_at TEXT NOT NULL,
@@ -85,6 +89,47 @@ export function initSchema() {
 
   CREATE INDEX IF NOT EXISTS idx_tasks_status
     ON tasks(status);
+`);
+    // --- Task steps ---
+    db.exec(`
+  CREATE TABLE IF NOT EXISTS task_steps (
+    id INTEGER PRIMARY KEY AUTOINCREMENT,
+
+    task_id INTEGER NOT NULL,
+
+    file_path TEXT NOT NULL,
+    action TEXT,
+
+    -- pending | completed | skipped | needs-feedback
+    status TEXT NOT NULL DEFAULT 'pending',
+
+    start_time INTEGER,
+    end_time INTEGER,
+
+    result_json TEXT,
+    notes TEXT,
+
+    step_index INTEGER,
+
+    -- 🔴 step-level feedback / questions (append-only)
+    step_questions_json TEXT,
+
+    created_at TEXT NOT NULL,
+    updated_at TEXT NOT NULL,
+
+    FOREIGN KEY (task_id)
+      REFERENCES tasks(id)
+      ON DELETE CASCADE
+  );
+
+  CREATE INDEX IF NOT EXISTS idx_task_steps_task
+    ON task_steps(task_id);
+
+  CREATE INDEX IF NOT EXISTS idx_task_steps_status
+    ON task_steps(status);
+
+  CREATE UNIQUE INDEX IF NOT EXISTS idx_task_steps_task_file
+    ON task_steps(task_id, file_path);
 `);
     // --- Core tables ---
     db.exec(`

package/dist/scripts/dbcheck.js

@@ -90,6 +90,46 @@ else {
         sampleTasks.forEach(t => console.log(`   [${t.id}] "${t.initial_query}" (status=${t.status}, created=${t.created_at})`));
     }
 }
+/* ───────────────────────── task_steps sanity ───────────────────────── */
+header("🦾 task_steps");
+const taskStepsExists = db
+    .prepare(`SELECT 1 FROM sqlite_master WHERE type='table' AND name='task_steps'`)
+    .get();
+if (!taskStepsExists) {
+    console.log("❌ task_steps table missing — you need to run initSchema with task_steps");
+}
+else {
+    const totalTaskSteps = tableCount("task_steps");
+    const pendingSteps = nonEmptyCount("task_steps", "status"); // all steps have status
+    console.log(`📊 total task_steps: ${totalTaskSteps}`);
+    console.log(`📝 task_steps with status: ${pendingSteps}`);
+    const sampleSteps = db.prepare(`
+    SELECT id, task_id, file_path, status, step_index, created_at
+    FROM task_steps
+    ORDER BY created_at DESC
+    LIMIT 3
+  `).all();
+    if (sampleSteps.length === 0) {
+        console.log("⚠️ no task_steps yet");
+    }
+    else {
+        console.log("✅ sample task_steps:");
+        sampleSteps.forEach(s => console.log(`   [${s.id}] task ${s.task_id} - file "${s.file_path}" (status=${s.status}, index=${s.step_index}, created=${s.created_at})`));
+    }
+    // Check for steps referencing missing tasks
+    const danglingSteps = db.prepare(`
+    SELECT ts.id, ts.task_id
+    FROM task_steps ts
+    LEFT JOIN tasks t ON ts.task_id = t.id
+    WHERE t.id IS NULL
+  `).all();
+    if (danglingSteps.length > 0) {
+        console.log(`❌ ${danglingSteps.length} task_steps reference missing tasks`);
+    }
+    else {
+        console.log("✅ all task_steps reference valid tasks");
+    }
+}
 /* ───────────────────────── files ───────────────────────── */
 header("🗂 files");
 const totalFiles = tableCount("files");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scai",
-  "version": "0.1.161",
+  "version": "0.1.163",
   "type": "module",
   "bin": {
     "scai": "./dist/index.js"