@shepai/cli 1.19.0 → 1.20.1

package/dist/src/infrastructure/services/agents/feature-agent/feature-agent-worker.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-agent-worker.d.ts","sourceRoot":"","sources":["../../../../../../src/infrastructure/services/agents/feature-agent/feature-agent-worker.ts"],"names":[],"mappings":";AACA;;;;;;;;GAQG;AAEH,OAAO,kBAAkB,CAAC;AAc1B,MAAM,WAAW,UAAU;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,mBAAmB,CAAC,EAAE,OAAO,CAAC;CAC/B;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU,CAmC1D;AAgCD;;;GAGG;AACH,wBAAsB,SAAS,CAAC,IAAI,EAAE,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,CAmI/D"}
+{"version":3,"file":"feature-agent-worker.d.ts","sourceRoot":"","sources":["../../../../../../src/infrastructure/services/agents/feature-agent/feature-agent-worker.ts"],"names":[],"mappings":";AACA;;;;;;;;GAQG;AAEH,OAAO,kBAAkB,CAAC;AAgB1B,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAElE,MAAM,WAAW,UAAU;IACzB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,mBAAmB,CAAC,EAAE,OAAO,CAAC;CAC/B;AAED;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,UAAU,CA0C1D;AAgCD;;;GAGG;AACH,wBAAsB,SAAS,CAAC,IAAI,EAAE,UAAU,GAAG,OAAO,CAAC,IAAI,CAAC,CAsI/D"}

package/dist/src/infrastructure/services/agents/feature-agent/feature-agent-worker.js

@@ -19,6 +19,7 @@ import { AgentRunStatus } from '../../../../domain/generated/output.js';
 import { initializeSettings } from '../../../../infrastructure/services/settings.service.js';
 import { InitializeSettingsUseCase } from '../../../../application/use-cases/settings/initialize-settings.use-case.js';
 import { setHeartbeatContext } from './heartbeat.js';
+import { setPhaseTimingContext } from './phase-timing-context.js';
 /**
  * Parse CLI arguments into a WorkerArgs object.
  * Throws if any required argument is missing.
@@ -33,8 +34,17 @@ export function parseWorkerArgs(args) {
     };
     const worktreeIdx = args.indexOf('--worktree-path');
     const worktreePath = worktreeIdx !== -1 && worktreeIdx + 1 < args.length ? args[worktreeIdx + 1] : undefined;
-    const approvalIdx = args.indexOf('--approval-mode');
-    const approvalMode = approvalIdx !== -1 && approvalIdx + 1 < args.length ? args[approvalIdx + 1] : undefined;
+    const approvalIdx = args.indexOf('--approval-gates');
+    let approvalGates;
+    if (approvalIdx !== -1 && approvalIdx + 1 < args.length) {
+        try {
+            approvalGates = JSON.parse(args[approvalIdx + 1]);
+        }
+        catch {
+            // Fall back to no gates if JSON parse fails
+            approvalGates = undefined;
+        }
+    }
     const resume = args.includes('--resume');
     const resumeFromInterrupt = args.includes('--resume-from-interrupt');
     const threadIdx = args.indexOf('--thread-id');
@@ -45,7 +55,7 @@ export function parseWorkerArgs(args) {
         repo: getArg('repo'),
         specDir: getArg('spec-dir'),
         worktreePath,
-        approvalMode,
+        approvalGates,
         resume,
         threadId,
         resumeFromInterrupt,
@@ -87,7 +97,7 @@ export async function runWorker(args) {
     log(`  repo=${args.repo}`);
     log(`  specDir=${args.specDir}`);
     log(`  worktreePath=${args.worktreePath ?? '(none)'}`);
-    log(`  approvalMode=${args.approvalMode ?? '(none)'}`);
+    log(`  approvalGates=${args.approvalGates ? JSON.stringify(args.approvalGates) : '(none)'}`);
     log(`  resume=${args.resume}`);
     log('Initializing container...');
     await initializeContainer();
@@ -121,6 +131,9 @@ export async function runWorker(args) {
     const stopHeartbeat = startHeartbeat(args.runId, runRepository);
     // Set heartbeat context so node-helpers can update current node
     setHeartbeatContext(args.runId, runRepository);
+    // Set phase timing context so executeNode() records per-phase durations
+    const timingRepository = container.resolve('IPhaseTimingRepository');
+    setPhaseTimingContext(args.runId, timingRepository);
     try {
         const graphConfig = { configurable: { thread_id: checkpointId } };
         let result;
@@ -139,7 +152,7 @@ export async function runWorker(args) {
                 worktreePath: args.worktreePath ?? args.repo,
                 specDir: args.specDir,
                 error: undefined, // Clear previous error state
-                ...(args.approvalMode ? { approvalMode: args.approvalMode } : {}),
+                ...(args.approvalGates ? { approvalGates: args.approvalGates } : {}),
             }, graphConfig);
         }
         else {
@@ -149,7 +162,7 @@ export async function runWorker(args) {
                 repositoryPath: args.repo,
                 worktreePath: args.worktreePath ?? args.repo,
                 specDir: args.specDir,
-                ...(args.approvalMode ? { approvalMode: args.approvalMode } : {}),
+                ...(args.approvalGates ? { approvalGates: args.approvalGates } : {}),
             }, graphConfig);
         }
         log(`Graph invocation completed. Error: ${result.error ?? 'none'}`);
@@ -159,7 +172,6 @@ export async function runWorker(args) {
         if (interruptPayload && interruptPayload.length > 0) {
             const now = new Date();
             await runRepository.updateStatus(args.runId, AgentRunStatus.waitingApproval, {
-                approvalStatus: 'waiting',
                 updatedAt: now,
             });
             log('Run paused — waiting for human approval');

package/dist/src/infrastructure/services/agents/feature-agent/nodes/implement.node.js

@@ -132,7 +132,7 @@ export function createImplementNode(executor) {
             log.info(`All phases complete — ${completedTasks}/${totalTasks} tasks (${elapsed}s)`);
             updateFeatureProgress(state.specDir, totalTasks, totalTasks, 'implementation-complete', null, log);
             messages.push(`[implement] Complete: ${totalTasks} tasks across ${totalPhases} phases (${elapsed}s)`);
-            if (shouldInterrupt('implement', state.approvalMode)) {
+            if (shouldInterrupt('implement', state.approvalGates)) {
                 log.info('Interrupting for human approval');
                 interrupt({
                     node: 'implement',

package/dist/src/infrastructure/services/agents/feature-agent/nodes/node-helpers.d.ts

@@ -5,6 +5,7 @@
  * and error handling so every node follows the same patterns.
  */
 import type { IAgentExecutor, AgentExecutionOptions, AgentExecutionResult } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
+import type { ApprovalGates } from '../../../../../domain/generated/output.js';
 import type { FeatureAgentState } from '../state.js';
 /**
  * Create a scoped logger that prefixes messages with the node name.
@@ -31,9 +32,17 @@ export declare function buildExecutorOptions(state: FeatureAgentState): AgentExe
 export declare function safeYamlLoad(content: string): unknown;
 /**
  * Determine whether the current node should trigger an interrupt
- * for human approval given the configured approval mode.
+ * for human approval given the configured approval gates.
+ *
+ * Gates control which phases auto-approve:
+ * - allowPrd: when true, skip interrupt after requirements phase
+ * - allowPlan: when true, skip interrupt after plan phase
+ *
+ * Nodes not covered by a gate (analyze, research) never interrupt.
+ * The implement node always interrupts when gates are present
+ * (unless both gates are true, meaning fully autonomous).
  */
-export declare function shouldInterrupt(nodeName: string, approvalMode: string | undefined): boolean;
+export declare function shouldInterrupt(nodeName: string, gates: ApprovalGates | undefined): boolean;
 export type ErrorCategory = 'retryable-api' | 'retryable-network' | 'non-retryable' | 'unknown';
 /**
  * Classify an error message into a retry category.

package/dist/src/infrastructure/services/agents/feature-agent/nodes/node-helpers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"node-helpers.d.ts","sourceRoot":"","sources":["../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/node-helpers.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACrB,MAAM,+DAA+D,CAAC;AACvE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAGrD;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM;kBAE/B,MAAM,GAAG,IAAI;mBAIZ,MAAM,GAAG,IAAI;EAK/B;AAED,MAAM,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;AAE7D;;GAEG;AACH,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAMtE;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,iBAAiB,GAAG,qBAAqB,CAMpF;AAeD;;;;GAIG;AACH,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAQrD;AAUD;;;GAGG;AACH,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM,GAAG,SAAS,GAAG,OAAO,CAI3F;AAMD,MAAM,MAAM,aAAa,GAAG,eAAe,GAAG,mBAAmB,GAAG,eAAe,GAAG,SAAS,CAAC;AAMhG;;;;;;;GAOG;AACH,wBAAgB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,aAAa,CAKjE;AAED,MAAM,WAAW,YAAY;IAC3B,wDAAwD;IACxD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,kFAAkF;IAClF,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,0CAA0C;IAC1C,MAAM,CAAC,EAAE,UAAU,CAAC;CACrB;AAED;;;;;;GAMG;AACH,wBAAsB,YAAY,CAChC,QAAQ,EAAE,cAAc,EACxB,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,qBAAqB,EAC9B,SAAS,CAAC,EAAE,YAAY,GACvB,OAAO,CAAC,oBAAoB,CAAC,CA6B/B;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAW5D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,GAAG,IAAI,CAwB1F;AAED;;;;;;;GAOG;AACH,wBAAgB,WAAW,CACzB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,cAAc,EACxB,WAAW,EAAE,CAAC,KAAK,EAAE,iBAAiB,EAAE,GAAG,EAAE,UAAU,KAAK,MAAM,GACjE,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAgDnE"}
+{"version":3,"file":"node-helpers.d.ts","sourceRoot":"","sources":["../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/node-helpers.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAMH,OAAO,KAAK,EACV,cAAc,EACd,qBAAqB,EACrB,oBAAoB,EACrB,MAAM,+DAA+D,CAAC;AACvE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAClE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAIrD;;;GAGG;AACH,wBAAgB,gBAAgB,CAAC,QAAQ,EAAE,MAAM;kBAE/B,MAAM,GAAG,IAAI;mBAIZ,MAAM,GAAG,IAAI;EAK/B;AAED,MAAM,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;AAE7D;;GAEG;AACH,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAMtE;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,iBAAiB,GAAG,qBAAqB,CAMpF;AAeD;;;;GAIG;AACH,wBAAgB,YAAY,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAQrD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,eAAe,CAAC,QAAQ,EAAE,MAAM,EAAE,KAAK,EAAE,aAAa,GAAG,SAAS,GAAG,OAAO,CAO3F;AAMD,MAAM,MAAM,aAAa,GAAG,eAAe,GAAG,mBAAmB,GAAG,eAAe,GAAG,SAAS,CAAC;AAMhG;;;;;;;GAOG;AACH,wBAAgB,aAAa,CAAC,YAAY,EAAE,MAAM,GAAG,aAAa,CAKjE;AAED,MAAM,WAAW,YAAY;IAC3B,wDAAwD;IACxD,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,kFAAkF;IAClF,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,0CAA0C;IAC1C,MAAM,CAAC,EAAE,UAAU,CAAC;CACrB;AAED;;;;;;GAMG;AACH,wBAAsB,YAAY,CAChC,QAAQ,EAAE,cAAc,EACxB,MAAM,EAAE,MAAM,EACd,OAAO,EAAE,qBAAqB,EAC9B,SAAS,CAAC,EAAE,YAAY,GACvB,OAAO,CAAC,oBAAoB,CAAC,CA6B/B;AAED;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAW5D;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,GAAG,IAAI,CAwB1F;AAED;;;;;;;GAOG;AACH,wBAAgB,WAAW,CACzB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,cAAc,EACxB,WAAW,EAAE,CAAC,KAAK,EAAE,iBAAiB,EAAE,GAAG,EAAE,UAAU,KAAK,MAAM,GACjE,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA4EnE"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/node-helpers.js

@@ -9,6 +9,7 @@ import { readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { interrupt, isGraphBubbleUp } from '@langchain/langgraph';
 import { reportNodeStart } from '../heartbeat.js';
+import { recordPhaseStart, recordPhaseEnd } from '../phase-timing-context.js';
 /**
  * Create a scoped logger that prefixes messages with the node name.
  * Output goes to stdout which the worker redirects to the log file.
@@ -70,22 +71,30 @@ export function safeYamlLoad(content) {
         return yaml.load(sanitized);
     }
 }
-/** Nodes that are auto-approved (no interrupt) for each approval mode. */
-const AUTO_APPROVED_NODES = {
-    interactive: [],
-    'allow-prd': ['analyze', 'requirements'],
-    'allow-plan': ['analyze', 'requirements', 'research', 'plan'],
-    'allow-all': ['analyze', 'requirements', 'research', 'plan', 'implement'],
-};
 /**
  * Determine whether the current node should trigger an interrupt
- * for human approval given the configured approval mode.
+ * for human approval given the configured approval gates.
+ *
+ * Gates control which phases auto-approve:
+ * - allowPrd: when true, skip interrupt after requirements phase
+ * - allowPlan: when true, skip interrupt after plan phase
+ *
+ * Nodes not covered by a gate (analyze, research) never interrupt.
+ * The implement node always interrupts when gates are present
+ * (unless both gates are true, meaning fully autonomous).
  */
-export function shouldInterrupt(nodeName, approvalMode) {
-    if (!approvalMode || approvalMode === 'allow-all')
+export function shouldInterrupt(nodeName, gates) {
+    if (!gates)
+        return false;
+    if (gates.allowPrd && gates.allowPlan)
         return false;
-    const autoApproved = AUTO_APPROVED_NODES[approvalMode] ?? [];
-    return !autoApproved.includes(nodeName);
+    if (nodeName === 'requirements')
+        return !gates.allowPrd;
+    if (nodeName === 'plan')
+        return !gates.allowPlan;
+    if (nodeName === 'implement')
+        return true;
+    return false;
 }
 const API_ERROR_RE = /API Error: (400|429|5\d{2})/;
 const NETWORK_ERROR_RE = /ECONNREFUSED|ETIMEDOUT|ENOTFOUND|timed out/i;
@@ -193,21 +202,41 @@ export function executeNode(nodeName, executor, buildPrompt) {
     return async (state) => {
         log.info('Starting...');
         reportNodeStart(nodeName);
+        // On resume from interrupt, LangGraph re-executes the node function from
+        // the top. Skip the expensive executor call if this phase already completed
+        // (markPhaseComplete is called before interrupt, so completed = done but
+        // awaiting approval; the approval has now been granted).
+        const completedPhases = getCompletedPhases(state.specDir);
+        if (completedPhases.includes(nodeName)) {
+            log.info('Phase already completed (resuming from approval), skipping execution');
+            return {
+                currentNode: nodeName,
+                messages: [`[${nodeName}] Approved — continuing`],
+            };
+        }
         const startTime = Date.now();
+        // Record phase start (no-op if timing context not set)
+        const timingId = await recordPhaseStart(nodeName);
         try {
             const prompt = buildPrompt(state, log);
             const options = buildExecutorOptions(state);
             log.info(`Executing agent at cwd=${options.cwd}`);
             log.info(`Prompt length: ${prompt.length} chars`);
             const result = await executor.execute(prompt, options);
-            const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
+            const durationMs = Date.now() - startTime;
+            const elapsed = (durationMs / 1000).toFixed(1);
             log.info(`Complete (${result.result.length} chars, ${elapsed}s)`);
+            // Record phase completion
+            await recordPhaseEnd(timingId, durationMs);
+            // Mark phase complete BEFORE interrupting so that on resume the
+            // node detects the work is already done and skips re-execution.
+            markPhaseComplete(state.specDir, nodeName, log);
             const nodeResult = {
                 currentNode: nodeName,
                 messages: [`[${nodeName}] Complete (${result.result.length} chars, ${elapsed}s)`],
             };
             // Human-in-the-loop: interrupt after node execution for review
-            if (shouldInterrupt(nodeName, state.approvalMode)) {
+            if (shouldInterrupt(nodeName, state.approvalGates)) {
                 log.info('Interrupting for human approval');
                 interrupt({
                     node: nodeName,
@@ -222,8 +251,10 @@ export function executeNode(nodeName, executor, buildPrompt) {
             if (isGraphBubbleUp(err))
                 throw err;
             const message = err instanceof Error ? err.message : String(err);
-            const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
+            const durationMs = Date.now() - startTime;
+            const elapsed = (durationMs / 1000).toFixed(1);
             log.error(`${message} (after ${elapsed}s)`);
+            // Leave timingId without completedAt (partial timing for failed nodes)
             // Throw so LangGraph does NOT checkpoint this node as "completed".
             // The worker catch block marks the run as failed, and on resume
             // LangGraph re-executes from the last successfully checkpointed node.

package/dist/src/infrastructure/services/agents/feature-agent/nodes/repair.node.d.ts

@@ -0,0 +1,23 @@
+/**
+ * Repair node factory for the feature-agent graph.
+ *
+ * Reads broken YAML content and validation errors from state,
+ * builds a constrained repair prompt, and sends it to the executor
+ * with limited tools (write-only) to fix the YAML files.
+ */
+import type { FeatureAgentState } from '../state.js';
+import type { IAgentExecutor } from '../../../../../application/ports/output/agents/agent-executor.interface.js';
+/**
+ * Build a repair prompt containing the broken YAML, validation errors,
+ * and file paths for the executor to write fixed files.
+ */
+export declare function buildRepairPrompt(filename: string | string[], content: string, errors: string[], specDir: string): string;
+/**
+ * Factory that creates a repair node for a specific YAML file (or files).
+ *
+ * Reads the broken YAML, builds a repair prompt with validation errors
+ * from state, and calls the executor with constrained options (maxTurns=5,
+ * write-only tools, no MCP).
+ */
+export declare function createRepairNode(filename: string | string[], executor: IAgentExecutor): (state: FeatureAgentState) => Promise<Partial<FeatureAgentState>>;
+//# sourceMappingURL=repair.node.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/repair.node.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"repair.node.d.ts","sourceRoot":"","sources":["../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/repair.node.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAGH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AACrD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAGpG;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,QAAQ,EAAE,MAAM,GAAG,MAAM,EAAE,EAC3B,OAAO,EAAE,MAAM,EACf,MAAM,EAAE,MAAM,EAAE,EAChB,OAAO,EAAE,MAAM,GACd,MAAM,CA0BR;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAC9B,QAAQ,EAAE,MAAM,GAAG,MAAM,EAAE,EAC3B,QAAQ,EAAE,cAAc,GACvB,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAuCnE"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/repair.node.js

@@ -0,0 +1,77 @@
+/**
+ * Repair node factory for the feature-agent graph.
+ *
+ * Reads broken YAML content and validation errors from state,
+ * builds a constrained repair prompt, and sends it to the executor
+ * with limited tools (write-only) to fix the YAML files.
+ */
+import { join } from 'node:path';
+import { readSpecFile, createNodeLogger } from './node-helpers.js';
+/**
+ * Build a repair prompt containing the broken YAML, validation errors,
+ * and file paths for the executor to write fixed files.
+ */
+export function buildRepairPrompt(filename, content, errors, specDir) {
+    const filenames = Array.isArray(filename) ? filename : [filename];
+    const filePaths = filenames.map((f) => join(specDir, f));
+    return [
+        '## YAML Repair Task',
+        '',
+        `Fix the following YAML validation errors in ${filenames.join(' and ')}.`,
+        '',
+        '### Validation Errors',
+        ...errors.map((e) => `- ${e}`),
+        '',
+        '### Current File Content',
+        '```yaml',
+        content,
+        '```',
+        '',
+        '### Output Files',
+        ...filePaths.map((p) => `- Write fixed YAML to: ${p}`),
+        '',
+        '### Rules',
+        '- Fix ONLY the reported errors',
+        '- Preserve all existing valid content',
+        '- Use proper YAML formatting',
+        '- Do not add fields that were not in the original',
+    ].join('\n');
+}
+/**
+ * Factory that creates a repair node for a specific YAML file (or files).
+ *
+ * Reads the broken YAML, builds a repair prompt with validation errors
+ * from state, and calls the executor with constrained options (maxTurns=5,
+ * write-only tools, no MCP).
+ */
+export function createRepairNode(filename, executor) {
+    const label = Array.isArray(filename) ? filename.join('+') : filename;
+    const log = createNodeLogger(`repair:${label}`);
+    return async (state) => {
+        log.info(`Repairing ${label} (attempt ${state.validationRetries})`);
+        const filenames = Array.isArray(filename) ? filename : [filename];
+        const contents = filenames.map((f) => readSpecFile(state.specDir, f));
+        const combinedContent = filenames.map((f, i) => `--- ${f} ---\n${contents[i]}`).join('\n\n');
+        const prompt = buildRepairPrompt(filename, combinedContent, state.lastValidationErrors, state.specDir);
+        const options = {
+            cwd: state.worktreePath || state.repositoryPath,
+            maxTurns: 5,
+            disableMcp: true,
+            allowedTools: ['write'],
+        };
+        try {
+            const result = await executor.execute(prompt, options);
+            log.info(`Repair complete (${result.result.length} chars)`);
+            return {
+                messages: [`[repair:${label}] Repair attempt ${state.validationRetries} complete`],
+            };
+        }
+        catch (err) {
+            const msg = err instanceof Error ? err.message : String(err);
+            log.error(`Repair failed: ${msg}`);
+            return {
+                messages: [`[repair:${label}] Repair failed: ${msg}`],
+            };
+        }
+    };
+}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/plan.schema.d.ts

@@ -0,0 +1,4 @@
+import { type ValidationResult } from './validation.js';
+export declare function validatePlan(data: unknown): ValidationResult;
+export declare function validateTasks(data: unknown, phaseIds: string[]): ValidationResult;
+//# sourceMappingURL=plan.schema.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/plan.schema.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"plan.schema.d.ts","sourceRoot":"","sources":["../../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/schemas/plan.schema.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAEzB,wBAAgB,YAAY,CAAC,IAAI,EAAE,OAAO,GAAG,gBAAgB,CA+B5D;AAED,wBAAgB,aAAa,CAAC,IAAI,EAAE,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,gBAAgB,CA+BjF"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/plan.schema.js

@@ -0,0 +1,54 @@
+import { requireString, requireNonEmptyArray, requireArrayOfShape, } from './validation.js';
+export function validatePlan(data) {
+    const errors = [];
+    if (!data || typeof data !== 'object') {
+        errors.push('YAML parsed to null or non-object');
+        return { valid: false, errors };
+    }
+    const d = data;
+    requireString(d, 'content', errors);
+    requireArrayOfShape(d, 'phases', errors, (item, idx, errs) => {
+        const phase = item;
+        if (!phase || typeof phase !== 'object') {
+            errs.push(`phases[${idx}] must be an object`);
+            return;
+        }
+        requireString(phase, 'id', errs, `phases[${idx}].`);
+        requireString(phase, 'name', errs, `phases[${idx}].`);
+        if (typeof phase.parallel !== 'boolean') {
+            errs.push(`phases[${idx}].parallel must be a boolean`);
+        }
+    });
+    const ftc = Array.isArray(d.filesToCreate) ? d.filesToCreate : [];
+    const ftm = Array.isArray(d.filesToModify) ? d.filesToModify : [];
+    if (ftc.length === 0 && ftm.length === 0) {
+        errors.push("At least one of 'filesToCreate' or 'filesToModify' must be non-empty");
+    }
+    return { valid: errors.length === 0, errors };
+}
+export function validateTasks(data, phaseIds) {
+    const errors = [];
+    if (!data || typeof data !== 'object') {
+        errors.push('YAML parsed to null or non-object');
+        return { valid: false, errors };
+    }
+    const d = data;
+    requireArrayOfShape(d, 'tasks', errors, (item, idx, errs) => {
+        const task = item;
+        if (!task || typeof task !== 'object') {
+            errs.push(`tasks[${idx}] must be an object`);
+            return;
+        }
+        requireString(task, 'id', errs, `tasks[${idx}].`);
+        requireString(task, 'title', errs, `tasks[${idx}].`);
+        requireString(task, 'state', errs, `tasks[${idx}].`);
+        requireNonEmptyArray(task, 'acceptanceCriteria', errs, `tasks[${idx}].`);
+        if (typeof task.phaseId === 'string' && !phaseIds.includes(task.phaseId)) {
+            errs.push(`tasks[${idx}].phaseId '${task.phaseId}' does not match any phase (valid: ${phaseIds.join(', ')})`);
+        }
+        else if (typeof task.phaseId !== 'string') {
+            requireString(task, 'phaseId', errs, `tasks[${idx}].`);
+        }
+    });
+    return { valid: errors.length === 0, errors };
+}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/research.schema.d.ts

@@ -0,0 +1,3 @@
+import { type ValidationResult } from './validation.js';
+export declare function validateResearch(data: unknown): ValidationResult;
+//# sourceMappingURL=research.schema.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/research.schema.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"research.schema.d.ts","sourceRoot":"","sources":["../../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/schemas/research.schema.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,gBAAgB,EACtB,MAAM,iBAAiB,CAAC;AAEzB,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,OAAO,GAAG,gBAAgB,CA0BhE"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/research.schema.js

@@ -0,0 +1,24 @@
+import { requireString, requireArrayOfShape, requireNonEmptyArray, } from './validation.js';
+export function validateResearch(data) {
+    const errors = [];
+    if (!data || typeof data !== 'object') {
+        errors.push('YAML parsed to null or non-object');
+        return { valid: false, errors };
+    }
+    const d = data;
+    requireString(d, 'name', errors);
+    requireString(d, 'summary', errors);
+    requireString(d, 'content', errors);
+    requireArrayOfShape(d, 'decisions', errors, (item, idx, errs) => {
+        const decision = item;
+        if (!decision || typeof decision !== 'object') {
+            errs.push(`decisions[${idx}] must be an object`);
+            return;
+        }
+        requireString(decision, 'title', errs, `decisions[${idx}].`);
+        requireString(decision, 'chosen', errs, `decisions[${idx}].`);
+        requireNonEmptyArray(decision, 'rejected', errs, `decisions[${idx}].`);
+        requireString(decision, 'rationale', errs, `decisions[${idx}].`);
+    });
+    return { valid: errors.length === 0, errors };
+}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/spec.schema.d.ts

@@ -0,0 +1,4 @@
+import { type ValidationResult } from './validation.js';
+export declare function validateSpecAnalyze(data: unknown): ValidationResult;
+export declare function validateSpecRequirements(data: unknown): ValidationResult;
+//# sourceMappingURL=spec.schema.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/spec.schema.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"spec.schema.d.ts","sourceRoot":"","sources":["../../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/schemas/spec.schema.ts"],"names":[],"mappings":"AAAA,OAAO,EAAuC,KAAK,gBAAgB,EAAE,MAAM,iBAAiB,CAAC;AA4B7F,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,OAAO,GAAG,gBAAgB,CAInE;AAED,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,OAAO,GAAG,gBAAgB,CAqBxE"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/spec.schema.js

@@ -0,0 +1,50 @@
+import { requireString, requireNonEmptyArray } from './validation.js';
+const VALID_SIZE_ESTIMATES = ['S', 'M', 'L', 'XL'];
+function validateBaseSpec(data, errors) {
+    if (!data || typeof data !== 'object') {
+        errors.push('YAML parsed to null or non-object');
+        return false;
+    }
+    const d = data;
+    requireString(d, 'name', errors);
+    requireString(d, 'oneLiner', errors);
+    requireString(d, 'summary', errors);
+    requireString(d, 'phase', errors);
+    requireString(d, 'content', errors);
+    requireNonEmptyArray(d, 'technologies', errors);
+    if (typeof d.sizeEstimate === 'string' && !VALID_SIZE_ESTIMATES.includes(d.sizeEstimate)) {
+        errors.push(`Field 'sizeEstimate' must be one of ${VALID_SIZE_ESTIMATES.join(', ')}, got '${d.sizeEstimate}'`);
+    }
+    else if (typeof d.sizeEstimate !== 'string') {
+        requireString(d, 'sizeEstimate', errors);
+    }
+    return true;
+}
+export function validateSpecAnalyze(data) {
+    const errors = [];
+    validateBaseSpec(data, errors);
+    return { valid: errors.length === 0, errors };
+}
+export function validateSpecRequirements(data) {
+    const errors = [];
+    if (!validateBaseSpec(data, errors))
+        return { valid: false, errors };
+    const d = data;
+    if (!('openQuestions' in d)) {
+        errors.push("Missing required field 'openQuestions'");
+    }
+    else if (Array.isArray(d.openQuestions)) {
+        for (let i = 0; i < d.openQuestions.length; i++) {
+            const q = d.openQuestions[i];
+            if (!q || typeof q !== 'object') {
+                errors.push(`openQuestions[${i}] must be an object`);
+                continue;
+            }
+            requireString(q, 'question', errors, `openQuestions[${i}].`);
+        }
+    }
+    else {
+        errors.push("Field 'openQuestions' must be an array");
+    }
+    return { valid: errors.length === 0, errors };
+}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/validation.d.ts

@@ -0,0 +1,8 @@
+export interface ValidationResult {
+    valid: boolean;
+    errors: string[];
+}
+export declare function requireString(data: Record<string, unknown>, field: string, errors: string[], prefix?: string): void;
+export declare function requireNonEmptyArray(data: Record<string, unknown>, field: string, errors: string[], prefix?: string): unknown[] | null;
+export declare function requireArrayOfShape(data: Record<string, unknown>, field: string, errors: string[], itemChecker: (item: unknown, index: number, errors: string[]) => void, prefix?: string): void;
+//# sourceMappingURL=validation.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/validation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.d.ts","sourceRoot":"","sources":["../../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/schemas/validation.ts"],"names":[],"mappings":"AAAA,MAAM,WAAW,gBAAgB;IAC/B,KAAK,EAAE,OAAO,CAAC;IACf,MAAM,EAAE,MAAM,EAAE,CAAC;CAClB;AAED,wBAAgB,aAAa,CAC3B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,KAAK,EAAE,MAAM,EACb,MAAM,EAAE,MAAM,EAAE,EAChB,MAAM,SAAK,GACV,IAAI,CASN;AAED,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,KAAK,EAAE,MAAM,EACb,MAAM,EAAE,MAAM,EAAE,EAChB,MAAM,SAAK,GACV,OAAO,EAAE,GAAG,IAAI,CAelB;AAED,wBAAgB,mBAAmB,CACjC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC7B,KAAK,EAAE,MAAM,EACb,MAAM,EAAE,MAAM,EAAE,EAChB,WAAW,EAAE,CAAC,IAAI,EAAE,OAAO,EAAE,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,KAAK,IAAI,EACrE,MAAM,SAAK,GACV,IAAI,CAMN"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/schemas/validation.js

@@ -0,0 +1,36 @@
+export function requireString(data, field, errors, prefix = '') {
+    const val = data[field];
+    if (val === undefined || val === null) {
+        errors.push(`Missing required string field '${prefix}${field}'`);
+    }
+    else if (typeof val !== 'string') {
+        errors.push(`Field '${prefix}${field}' must be a string, got ${typeof val}`);
+    }
+    else if (val.trim() === '') {
+        errors.push(`Field '${prefix}${field}' must not be empty`);
+    }
+}
+export function requireNonEmptyArray(data, field, errors, prefix = '') {
+    const val = data[field];
+    if (val === undefined || val === null) {
+        errors.push(`Missing required array field '${prefix}${field}'`);
+        return null;
+    }
+    if (!Array.isArray(val)) {
+        errors.push(`Field '${prefix}${field}' must be an array, got ${typeof val}`);
+        return null;
+    }
+    if (val.length === 0) {
+        errors.push(`Field '${prefix}${field}' must not be empty`);
+        return null;
+    }
+    return val;
+}
+export function requireArrayOfShape(data, field, errors, itemChecker, prefix = '') {
+    const arr = requireNonEmptyArray(data, field, errors, prefix);
+    if (!arr)
+        return;
+    for (let i = 0; i < arr.length; i++) {
+        itemChecker(arr[i], i, errors);
+    }
+}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/validate.node.d.ts

@@ -0,0 +1,12 @@
+import type { FeatureAgentState } from '../state.js';
+import type { ValidationResult } from './schemas/validation.js';
+type SchemaValidator = (data: unknown) => ValidationResult;
+/**
+ * Factory that creates a validate node for a specific YAML file.
+ *
+ * On success: resets validationRetries to 0, clears errors.
+ * On failure: increments validationRetries, sets errors for repair node.
+ */
+export declare function createValidateNode(filename: string, schema: SchemaValidator): (state: FeatureAgentState) => Promise<Partial<FeatureAgentState>>;
+export {};
+//# sourceMappingURL=validate.node.d.ts.map

package/dist/src/infrastructure/services/agents/feature-agent/nodes/validate.node.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validate.node.d.ts","sourceRoot":"","sources":["../../../../../../../src/infrastructure/services/agents/feature-agent/nodes/validate.node.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AACrD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAGhE,KAAK,eAAe,GAAG,CAAC,IAAI,EAAE,OAAO,KAAK,gBAAgB,CAAC;AAE3D;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAChC,QAAQ,EAAE,MAAM,EAChB,MAAM,EAAE,eAAe,GACtB,CAAC,KAAK,EAAE,iBAAiB,KAAK,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAsDnE"}

package/dist/src/infrastructure/services/agents/feature-agent/nodes/validate.node.js

@@ -0,0 +1,58 @@
+import { readSpecFile, safeYamlLoad, createNodeLogger } from './node-helpers.js';
+/**
+ * Factory that creates a validate node for a specific YAML file.
+ *
+ * On success: resets validationRetries to 0, clears errors.
+ * On failure: increments validationRetries, sets errors for repair node.
+ */
+export function createValidateNode(filename, schema) {
+    const log = createNodeLogger(`validate:${filename}`);
+    return async (state) => {
+        log.info(`Validating ${filename}`);
+        const content = readSpecFile(state.specDir, filename);
+        if (!content) {
+            const errors = [`File '${filename}' not found or empty in ${state.specDir}`];
+            log.error(errors[0]);
+            return {
+                lastValidationTarget: filename,
+                lastValidationErrors: errors,
+                validationRetries: state.validationRetries + 1,
+                messages: [`[validate:${filename}] FAIL: ${errors[0]}`],
+            };
+        }
+        // Try to parse
+        let parsed;
+        try {
+            parsed = safeYamlLoad(content);
+        }
+        catch (err) {
+            const msg = err instanceof Error ? err.message : String(err);
+            const errors = [`YAML parse error in ${filename}: ${msg}`];
+            log.error(errors[0]);
+            return {
+                lastValidationTarget: filename,
+                lastValidationErrors: errors,
+                validationRetries: state.validationRetries + 1,
+                messages: [`[validate:${filename}] FAIL: parse error`],
+            };
+        }
+        // Run schema validation
+        const result = schema(parsed);
+        if (result.valid) {
+            log.info('Validation passed');
+            return {
+                lastValidationTarget: filename,
+                lastValidationErrors: [],
+                validationRetries: 0,
+                messages: [`[validate:${filename}] PASS`],
+            };
+        }
+        log.error(`Validation failed: ${result.errors.join('; ')}`);
+        return {
+            lastValidationTarget: filename,
+            lastValidationErrors: result.errors,
+            validationRetries: state.validationRetries + 1,
+            messages: [`[validate:${filename}] FAIL: ${result.errors.length} error(s)`],
+        };
+    };
+}