ai-sdlc 0.2.0-alpha.6 → 0.2.0-alpha.60

package/dist/agents/implementation.d.ts

@@ -104,6 +104,22 @@ export interface RetryAttemptOptions {
     reworkContext?: string;
     onProgress?: AgentProgressCallback;
 }
+/**
+ * Outcome type for retry attempts
+ */
+export type AttemptOutcome = 'failed_tests' | 'failed_build' | 'no_change';
+/**
+ * Result from a single retry attempt
+ */
+export interface AttemptHistoryEntry {
+    attempt: number;
+    testFailures: number;
+    buildFailures: number;
+    testSnippet: string;
+    buildSnippet: string;
+    changesSummary: string;
+    outcome: AttemptOutcome;
+}
 /**
  * Attempt implementation with retry logic
  *
@@ -134,6 +150,18 @@ export declare function captureCurrentDiffHash(workingDir: string): string;
  * @returns True if changes occurred (hashes are different)
  */
 export declare function hasChangesOccurred(previousHash: string, currentHash: string): boolean;
+/**
+ * Extract list of changed files from git diff
+ * @param workingDir The working directory
+ * @returns Comma-separated list of changed files, or descriptive message
+ */
+export declare function extractChangedFiles(workingDir: string): string;
+/**
+ * Build formatted retry history section for agent prompts
+ * @param history Array of attempt history entries
+ * @returns Formatted string for inclusion in retry prompts
+ */
+export declare function buildRetryHistorySection(history: AttemptHistoryEntry[]): string;
 /**
  * Sanitize test output to remove ANSI escape sequences and potential injection patterns
  * @param output Test output string
@@ -147,12 +175,19 @@ export declare function sanitizeTestOutput(output: string): string;
  * @returns Truncated and sanitized output with notice if truncated
  */
 export declare function truncateTestOutput(output: string, maxLength?: number): string;
+/**
+ * Detect if errors are related to missing dependencies
+ * Returns module names that are missing, if any
+ */
+export declare function detectMissingDependencies(output: string): string[];
 /**
  * Build retry prompt for implementation agent
  * @param testOutput Test failure output
+ * @param buildOutput Build output
  * @param attemptNumber Current attempt number (1-indexed)
  * @param maxRetries Maximum number of retries
+ * @param attemptHistory Optional history of previous attempts
  * @returns Prompt string for retry attempt
  */
-export declare function buildRetryPrompt(testOutput: string, buildOutput: string, attemptNumber: number, maxRetries: number): string;
+export declare function buildRetryPrompt(testOutput: string, buildOutput: string, attemptNumber: number, maxRetries: number, attemptHistory?: AttemptHistoryEntry[]): string;
 //# sourceMappingURL=implementation.d.ts.map

package/dist/agents/implementation.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"implementation.d.ts","sourceRoot":"","sources":["../../src/agents/implementation.ts"],"names":[],"mappings":"AAYA,OAAO,EAAE,aAAa,EAAE,qBAAqB,EAAE,MAAM,mBAAmB,CAAC;AACzE,OAAO,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,EAAa,MAAM,mBAAmB,CAAC;AAChF,OAAO,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAM7C,YAAY,EAAE,qBAAqB,EAAE,CAAC;AAEtC;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAED;;GAEG;AACH,MAAM,WAAW,wBAAwB;IACvC,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,WAAW,CAAC,EAAE,OAAO,WAAW,CAAC;IACjC,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAED,eAAO,MAAM,iBAAiB,8zBAuB+B,CAAC;AAsB9D;;GAEG;AACH,wBAAsB,aAAa,CACjC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,MAAM,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC,CAgD9C;AAED;;GAEG;AACH,wBAAsB,WAAW,CAC/B,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,MAAM,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC,CAgD9C;AAWD;;;;;;;;GAQG;AACH,wBAAsB,oBAAoB,CACxC,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,MAAM,EACnB,UAAU,GAAE,OAAO,WAAyB,GAC3C,OAAO,CAAC;IAAE,SAAS,EAAE,OAAO,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAiDlD;AAmDD;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAqCzB;AAED;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAoCzB;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAwCzB;AAED;;GAEG;AACH,wBAAgB,cAAc,CAC5B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,cAAc,EACzB,WAAW,EAAE,cAAc,EAC3B,cAAc,EAAE,cAAc,GAC7B,YAAY,CAYd;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO,CAuBrD;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,KAAK,EACZ,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,wBAA6B,GACrC,OAAO,CAAC,WAAW,CAAC,CA6ItB;AAED;;GAEG;AACH,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,KAAK,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAaD;;;;;;;;;GASG;AACH,wBAAsB,gCAAgC,CACpD,OAAO,EAAE,mBAAmB,EAC5B,WAAW,EAAE,MAAM,EAAE,GACpB,OAAO,CAAC,WAAW,CAAC,CA2MtB;AAED;;;;GAIG;AACH,wBAAsB,sBAAsB,CAC1C,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,YAAiB,GACzB,OAAO,CAAC,WAAW,CAAC,CAyLtB;AAgCD;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAuBjE;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,YAAY,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAErF;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAkBzD;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,EAAE,SAAS,GAAE,MAAa,GAAG,MAAM,CAYnF;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,GACjB,MAAM,CAsCR"}
+{"version":3,"file":"implementation.d.ts","sourceRoot":"","sources":["../../src/agents/implementation.ts"],"names":[],"mappings":"AAcA,OAAO,EAAE,aAAa,EAAE,qBAAqB,EAAE,MAAM,mBAAmB,CAAC;AAEzE,OAAO,EAAE,KAAK,EAAE,WAAW,EAAE,YAAY,EAAa,MAAM,mBAAmB,CAAC;AAChF,OAAO,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAO7C,YAAY,EAAE,qBAAqB,EAAE,CAAC;AAEtC;;GAEG;AACH,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,MAAM,CAAC;IACf,OAAO,EAAE,OAAO,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAED;;GAEG;AACH,MAAM,WAAW,wBAAwB;IACvC,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,aAAa,CAAC,EAAE,OAAO,aAAa,CAAC;IACrC,WAAW,CAAC,EAAE,OAAO,WAAW,CAAC;IACjC,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAED,eAAO,MAAM,iBAAiB,8zBAuB+B,CAAC;AAsB9D;;GAEG;AACH,wBAAsB,aAAa,CACjC,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,MAAM,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC,CAkD9C;AAED;;GAEG;AACH,wBAAsB,WAAW,CAC/B,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,GAClB,OAAO,CAAC;IAAE,MAAM,EAAE,OAAO,CAAC;IAAC,MAAM,EAAE,MAAM,CAAA;CAAE,CAAC,CAkD9C;AAWD;;;;;;;;GAQG;AACH,wBAAsB,oBAAoB,CACxC,UAAU,EAAE,MAAM,EAClB,OAAO,EAAE,MAAM,EACf,WAAW,EAAE,MAAM,EACnB,UAAU,GAAE,OAAO,WAAyB,GAC3C,OAAO,CAAC;IAAE,SAAS,EAAE,OAAO,CAAC;IAAC,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAiDlD;AAmDD;;;;;GAKG;AACH,wBAAsB,eAAe,CACnC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAqCzB;AAED;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAoCzB;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,KAAK,EACZ,WAAW,EAAE,MAAM,EACnB,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,eAAe,GACvB,OAAO,CAAC,cAAc,CAAC,CAwCzB;AAED;;GAEG;AACH,wBAAgB,cAAc,CAC5B,WAAW,EAAE,MAAM,EACnB,SAAS,EAAE,cAAc,EACzB,WAAW,EAAE,cAAc,EAC3B,cAAc,EAAE,cAAc,GAC7B,YAAY,CAYd;AAED;;GAEG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE,KAAK,GAAG,OAAO,CAuBrD;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,KAAK,EAAE,KAAK,EACZ,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,wBAA6B,GACrC,OAAO,CAAC,WAAW,CAAC,CA6ItB;AAED;;GAEG;AACH,MAAM,WAAW,mBAAmB;IAClC,KAAK,EAAE,KAAK,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,UAAU,CAAC,EAAE,qBAAqB,CAAC;CACpC;AAED;;GAEG;AACH,MAAM,MAAM,cAAc,GAAG,cAAc,GAAG,cAAc,GAAG,WAAW,CAAC;AAE3E;;GAEG;AACH,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,EAAE,MAAM,CAAC;IACrB,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,MAAM,CAAC;IACpB,YAAY,EAAE,MAAM,CAAC;IACrB,cAAc,EAAE,MAAM,CAAC;IACvB,OAAO,EAAE,cAAc,CAAC;CACzB;AAED;;;;;;;;;GASG;AACH,wBAAsB,gCAAgC,CACpD,OAAO,EAAE,mBAAmB,EAC5B,WAAW,EAAE,MAAM,EAAE,GACpB,OAAO,CAAC,WAAW,CAAC,CAiOtB;AAED;;;;GAIG;AACH,wBAAsB,sBAAsB,CAC1C,SAAS,EAAE,MAAM,EACjB,QAAQ,EAAE,MAAM,EAChB,OAAO,GAAE,YAAiB,GACzB,OAAO,CAAC,WAAW,CAAC,CAyPtB;AAgCD;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAuBjE;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,YAAY,EAAE,MAAM,EAAE,WAAW,EAAE,MAAM,GAAG,OAAO,CAErF;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,UAAU,EAAE,MAAM,GAAG,MAAM,CAuB9D;AAED;;;;GAIG;AACH,wBAAgB,wBAAwB,CAAC,OAAO,EAAE,mBAAmB,EAAE,GAAG,MAAM,CA0C/E;AAED;;;;GAIG;AACH,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAkBzD;AAED;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,MAAM,EAAE,SAAS,GAAE,MAAa,GAAG,MAAM,CAYnF;AAED;;;GAGG;AACH,wBAAgB,yBAAyB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,EAAE,CAqClE;AAED;;;;;;;;GAQG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EACrB,UAAU,EAAE,MAAM,EAClB,cAAc,CAAC,EAAE,mBAAmB,EAAE,GACrC,MAAM,CAkGR"}

package/dist/agents/implementation.js

@@ -1,10 +1,13 @@
 import { spawn, spawnSync } from 'child_process';
 import path from 'path';
-import { parseStory, writeStory, updateStoryStatus, updateStoryField, resetImplementationRetryCount, incrementImplementationRetryCount, getEffectiveMaxImplementationRetries, } from '../core/story.js';
+import { ProcessManager } from '../core/process-manager.js';
+import { parseStory, writeStory, updateStoryStatus, updateStoryField, resetImplementationRetryCount, incrementImplementationRetryCount, getEffectiveMaxImplementationRetries, incrementTotalRecoveryAttempts, } from '../core/story.js';
 import { runAgentQuery } from '../core/client.js';
+import { getLogger } from '../core/logger.js';
 import { loadConfig, DEFAULT_TDD_CONFIG } from '../core/config.js';
 import { verifyImplementation } from './verification.js';
 import { createHash } from 'crypto';
+import { parseTypeScriptErrors, classifyAndSortErrors } from '../services/error-classifier.js';
 export const TDD_SYSTEM_PROMPT = `You are practicing strict Test-Driven Development.
 
 Your workflow MUST follow this exact cycle:
@@ -59,6 +62,7 @@ export async function runSingleTest(testFile, workingDir, testTimeout) {
             cwd: workingDir,
             stdio: ['pipe', 'pipe', 'pipe'],
         });
+        ProcessManager.getInstance().registerChild(child);
         const timeoutId = setTimeout(() => {
             killed = true;
             child.kill('SIGTERM');
@@ -106,6 +110,7 @@ export async function runAllTests(workingDir, testTimeout) {
             cwd: workingDir,
             stdio: ['pipe', 'pipe', 'pipe'],
         });
+        ProcessManager.getInstance().registerChild(child);
         const timeoutId = setTimeout(() => {
             killed = true;
             child.kill('SIGTERM');
@@ -508,7 +513,7 @@ export async function runTDDImplementation(story, sdlcRoot, options = {}) {
         story.frontmatter.tdd_test_history = history.slice(-100);
         story.frontmatter.tdd_current_test = cycle;
         // Persist the TDD cycle history to disk
-        writeStory(story);
+        await writeStory(story);
         changesMade.push(`Completed TDD cycle ${cycleNumber}`);
         // Check if all AC are now covered
         // Re-read story to get latest content (agent may have updated checkboxes)
@@ -568,7 +573,7 @@ severity issues - these must be resolved. Review the specific feedback and make
         }
         // Add retry context if this is a retry attempt
         if (attemptNumber > 1 && lastVerification) {
-            prompt += '\n\n' + buildRetryPrompt(lastVerification.testsOutput, lastVerification.buildOutput, attemptNumber, maxRetries);
+            prompt += '\n\n' + buildRetryPrompt(lastVerification.testsOutput, lastVerification.buildOutput, attemptNumber, maxRetries, attemptHistory);
         }
         else {
             prompt += `
@@ -606,18 +611,43 @@ ${implementationResult}
         // Append to story content
         const updatedStory = parseStory(storyPath);
         updatedStory.content += '\n\n' + implementationNotes;
-        writeStory(updatedStory);
+        await writeStory(updatedStory);
         changesMade.push(attemptNumber > 1 ? `Added retry ${attemptNumber - 1} notes` : 'Added implementation notes');
+        // PRE-FLIGHT CHECK: Capture diff hash BEFORE verification to detect no-change scenarios early
+        const currentDiffHash = captureCurrentDiffHash(workingDir);
+        const changesSummary = extractChangedFiles(workingDir);
+        // Check for no-change scenario BEFORE running verification (saves ~30 seconds)
+        if (attemptNumber > 1 && lastDiffHash && lastDiffHash === currentDiffHash) {
+            changesMade.push('No changes detected since last attempt - skipping verification');
+            // Record this no-change attempt in history
+            attemptHistory.push({
+                attempt: attemptNumber,
+                testFailures: 0,
+                buildFailures: 0,
+                testSnippet: '',
+                buildSnippet: '',
+                changesSummary: 'No changes detected',
+                outcome: 'no_change',
+            });
+            return {
+                success: false,
+                story: parseStory(storyPath),
+                changesMade,
+                error: 'No progress detected - agent made no file changes',
+            };
+        }
+        // Update lastDiffHash for next iteration
+        lastDiffHash = currentDiffHash;
         changesMade.push('Running verification before marking complete...');
         const verification = await verifyImplementation(updatedStory, workingDir);
-        updateStoryField(updatedStory, 'last_test_run', {
+        await updateStoryField(updatedStory, 'last_test_run', {
             passed: verification.passed,
             failures: verification.failures,
             timestamp: verification.timestamp,
         });
         if (verification.passed) {
             // Success! Reset retry count and return success
-            resetImplementationRetryCount(updatedStory);
+            await resetImplementationRetryCount(updatedStory);
             changesMade.push('Verification passed - implementation successful');
             // Send success progress callback
             if (onProgress) {
@@ -631,10 +661,10 @@ ${implementationResult}
         }
         // Verification failed - check for retry conditions
         lastVerification = verification;
-        // Capture current diff hash for no-change detection
-        const currentDiffHash = captureCurrentDiffHash(workingDir);
         // Track retry attempt
-        incrementImplementationRetryCount(updatedStory);
+        await incrementImplementationRetryCount(updatedStory);
+        // Increment global recovery counter
+        await incrementTotalRecoveryAttempts(updatedStory);
         // Extract first 100 chars of test and build output for history
         const testSnippet = verification.testsOutput.substring(0, 100).replace(/\n/g, ' ');
         const buildSnippet = verification.buildOutput.substring(0, 100).replace(/\n/g, ' ');
@@ -644,6 +674,8 @@ ${implementationResult}
                 verification.buildOutput.includes('Error') ||
                 verification.buildOutput.includes('failed'));
         const buildFailures = hasBuildErrors ? 1 : 0;
+        // Determine outcome based on what failed
+        const outcome = hasBuildErrors ? 'failed_build' : 'failed_tests';
         // Record this attempt in history with both test and build failures
         attemptHistory.push({
             attempt: attemptNumber,
@@ -651,6 +683,8 @@ ${implementationResult}
             buildFailures,
             testSnippet,
             buildSnippet,
+            changesSummary,
+            outcome,
         });
         // Add structured retry entry to changes array
         if (attemptNumber > 1) {
@@ -659,17 +693,6 @@ ${implementationResult}
         else {
             changesMade.push(`Attempt ${attemptNumber}: ${verification.failures} test(s) failing`);
         }
-        // Check for no-change scenario (agent made no progress)
-        // Only check after first failure (attemptNumber > 1)
-        // Check this BEFORE max retries to fail fast on identical changes
-        if (attemptNumber > 1 && lastDiffHash && lastDiffHash === currentDiffHash) {
-            return {
-                success: false,
-                story: parseStory(storyPath),
-                changesMade,
-                error: `Implementation blocked: No progress detected on retry attempt ${attemptNumber - 1}. Agent made identical changes. Stopping retries early.\n\nLast test output:\n${truncateTestOutput(verification.testsOutput, 1000)}`,
-            };
-        }
         // Check if we've reached max retries
         if (attemptHistory.length > maxRetries) {
             const attemptSummary = attemptHistory
@@ -700,7 +723,6 @@ ${implementationResult}
                 error: `Implementation blocked after ${attemptNumber} attempts:\n${attemptSummary}\n\nLast test output:\n${truncateTestOutput(verification.testsOutput, 5000)}`,
             };
         }
-        lastDiffHash = currentDiffHash;
         // Continue to next retry attempt - send progress update
         if (onProgress) {
             onProgress({ type: 'assistant_message', content: `Retry ${attemptNumber} failed: ${verification.failures} test(s) failing, attempting retry ${attemptNumber + 1}...` });
@@ -720,10 +742,16 @@ ${implementationResult}
  * Executes the implementation plan, creating code changes and tests.
  */
 export async function runImplementationAgent(storyPath, sdlcRoot, options = {}) {
+    const logger = getLogger();
+    const startTime = Date.now();
     let story = parseStory(storyPath);
     let currentStoryPath = storyPath;
     const changesMade = [];
     const workingDir = path.dirname(sdlcRoot);
+    logger.info('implementation', 'Starting implementation phase', {
+        storyId: story.frontmatter.id,
+        retryCount: story.frontmatter.implementation_retry_count || 0,
+    });
     try {
         // Security: Validate working directory before git operations
         validateWorkingDir(workingDir);
@@ -767,7 +795,7 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
                     }
                 }
                 // Update story with branch info
-                updateStoryField(story, 'branch', branchName);
+                await updateStoryField(story, 'branch', branchName);
             }
         }
         catch {
@@ -776,13 +804,44 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
         }
         // Update status to in-progress if not already there
         if (story.frontmatter.status !== 'in-progress') {
-            story = updateStoryStatus(story, 'in-progress');
+            story = await updateStoryStatus(story, 'in-progress');
             currentStoryPath = story.path;
             changesMade.push('Updated status to in-progress');
         }
         // Check if TDD is enabled for this story
         const config = loadConfig(workingDir);
         const tddEnabled = story.frontmatter.tdd_enabled ?? config.tdd?.enabled ?? false;
+        // Check if orchestrator is enabled
+        if (config.useOrchestrator && !tddEnabled) {
+            changesMade.push('Using sequential task orchestrator for implementation');
+            const { runImplementationOrchestrator } = await import('./orchestrator.js');
+            const orchestratorResult = await runImplementationOrchestrator(currentStoryPath, sdlcRoot, {
+                maxRetriesPerTask: config.implementation.maxRetries,
+                commitAfterEachTask: true,
+                stopOnFirstFailure: true,
+            });
+            if (!orchestratorResult.success) {
+                // Orchestration failed
+                const errorDetails = orchestratorResult.failedTasks
+                    .map((ft) => `  - ${ft.taskId}: ${ft.error} (${ft.attempts} attempts)`)
+                    .join('\n');
+                return {
+                    success: false,
+                    story: parseStory(currentStoryPath),
+                    changesMade,
+                    error: `Implementation orchestration failed:\n${errorDetails}\n\nCompleted: ${orchestratorResult.tasksCompleted}, Failed: ${orchestratorResult.tasksFailed}, Remaining: ${orchestratorResult.tasksRemaining}`,
+                };
+            }
+            // Orchestration succeeded - mark implementation complete
+            await updateStoryField(story, 'implementation_complete', true);
+            changesMade.push('Marked implementation_complete: true');
+            changesMade.push(`Orchestration complete: ${orchestratorResult.tasksCompleted} tasks completed in ${orchestratorResult.totalAgentInvocations} agent invocations`);
+            return {
+                success: true,
+                story: parseStory(currentStoryPath),
+                changesMade,
+            };
+        }
         if (tddEnabled) {
             changesMade.push('TDD mode enabled - using Red-Green-Refactor implementation');
             // Run TDD implementation loop
@@ -795,7 +854,7 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
                 // TDD completed all cycles - now verify with retry support
                 changesMade.push('Running final verification...');
                 const verification = await verifyImplementation(tddResult.story, workingDir);
-                updateStoryField(tddResult.story, 'last_test_run', {
+                await updateStoryField(tddResult.story, 'last_test_run', {
                     passed: verification.passed,
                     failures: verification.failures,
                     timestamp: verification.timestamp,
@@ -803,7 +862,7 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
                 if (!verification.passed) {
                     // TDD final verification failed - this is unexpected since TDD should ensure all tests pass
                     // Reset retry count since this is the first failure at this stage
-                    resetImplementationRetryCount(tddResult.story);
+                    await resetImplementationRetryCount(tddResult.story);
                     return {
                         success: false,
                         story: parseStory(currentStoryPath),
@@ -812,8 +871,8 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
                     };
                 }
                 // Success - reset retry count
-                resetImplementationRetryCount(tddResult.story);
-                updateStoryField(tddResult.story, 'implementation_complete', true);
+                await resetImplementationRetryCount(tddResult.story);
+                await updateStoryField(tddResult.story, 'implementation_complete', true);
                 changesMade.push('Marked implementation_complete: true');
                 return {
                     success: true,
@@ -862,8 +921,13 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
             const errorMsg = error instanceof Error ? error.message : String(error);
             changesMade.push(`Commit warning: ${errorMsg} (continuing implementation)`);
         }
-        updateStoryField(updatedStory, 'implementation_complete', true);
+        await updateStoryField(updatedStory, 'implementation_complete', true);
         changesMade.push('Marked implementation_complete: true');
+        logger.info('implementation', 'Implementation phase complete', {
+            storyId: story.frontmatter.id,
+            durationMs: Date.now() - startTime,
+            changesCount: changesMade.length,
+        });
         return {
             success: true,
             story: parseStory(currentStoryPath),
@@ -871,11 +935,17 @@ export async function runImplementationAgent(storyPath, sdlcRoot, options = {})
         };
     }
     catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logger.error('implementation', 'Implementation phase failed', {
+            storyId: story.frontmatter.id,
+            durationMs: Date.now() - startTime,
+            error: errorMessage,
+        });
         return {
             success: false,
             story,
             changesMade,
-            error: error instanceof Error ? error.message : String(error),
+            error: errorMessage,
         };
     }
 }
@@ -942,6 +1012,72 @@ export function captureCurrentDiffHash(workingDir) {
 export function hasChangesOccurred(previousHash, currentHash) {
     return previousHash !== currentHash;
 }
+/**
+ * Extract list of changed files from git diff
+ * @param workingDir The working directory
+ * @returns Comma-separated list of changed files, or descriptive message
+ */
+export function extractChangedFiles(workingDir) {
+    try {
+        validateWorkingDir(workingDir);
+        const result = spawnSync('git', ['diff', 'HEAD', '--name-only'], {
+            cwd: workingDir,
+            shell: false,
+            encoding: 'utf-8',
+            stdio: ['ignore', 'pipe', 'pipe'],
+        });
+        if (result.status === 0 && result.stdout) {
+            const files = result.stdout.trim().split('\n').filter(Boolean);
+            if (files.length === 0) {
+                return 'No changes detected';
+            }
+            return files.join(', ');
+        }
+        return 'No changes detected';
+    }
+    catch {
+        return 'Unable to determine changes';
+    }
+}
+/**
+ * Build formatted retry history section for agent prompts
+ * @param history Array of attempt history entries
+ * @returns Formatted string for inclusion in retry prompts
+ */
+export function buildRetryHistorySection(history) {
+    if (!history || history.length === 0) {
+        return '';
+    }
+    const recentHistory = history.slice(-3);
+    let section = `PREVIOUS ATTEMPT HISTORY (Last ${recentHistory.length} attempts):
+
+`;
+    for (const entry of recentHistory) {
+        const outcomeLabel = entry.outcome === 'failed_tests'
+            ? 'Tests failed'
+            : entry.outcome === 'failed_build'
+                ? 'Build failed'
+                : 'No changes made';
+        section += `Attempt ${entry.attempt}: ${entry.changesSummary} -> ${outcomeLabel}\n`;
+        const errors = [];
+        if (entry.testSnippet && entry.testSnippet.trim()) {
+            errors.push(entry.testSnippet.trim());
+        }
+        if (entry.buildSnippet && entry.buildSnippet.trim()) {
+            errors.push(entry.buildSnippet.trim());
+        }
+        const errorsToShow = errors.slice(0, 2);
+        if (errorsToShow.length > 0) {
+            for (const err of errorsToShow) {
+                section += `  - ${err.substring(0, 100)}\n`;
+            }
+        }
+    }
+    section += `
+**IMPORTANT: Do NOT repeat the same fixes. Try a different approach.**
+`;
+    return section;
+}
 /**
  * Sanitize test output to remove ANSI escape sequences and potential injection patterns
  * @param output Test output string
@@ -982,21 +1118,114 @@ export function truncateTestOutput(output, maxLength = 5000) {
     const truncated = sanitized.substring(0, maxLength);
     return truncated + `\n\n[Output truncated. Showing first ${maxLength} characters of ${sanitized.length} total.]`;
 }
+/**
+ * Detect if errors are related to missing dependencies
+ * Returns module names that are missing, if any
+ */
+export function detectMissingDependencies(output) {
+    if (!output)
+        return [];
+    const missingModules = [];
+    // Pattern: Cannot find module 'package-name'
+    const cannotFindPattern = /Cannot find module ['"]([^'"]+)['"]/g;
+    let match;
+    while ((match = cannotFindPattern.exec(output)) !== null) {
+        const moduleName = match[1];
+        // Only include external packages, not relative imports
+        if (!moduleName.startsWith('.') && !moduleName.startsWith('/')) {
+            // Extract base package name (handle scoped packages like @types/foo)
+            const baseName = moduleName.startsWith('@')
+                ? moduleName.split('/').slice(0, 2).join('/')
+                : moduleName.split('/')[0];
+            if (!missingModules.includes(baseName)) {
+                missingModules.push(baseName);
+            }
+        }
+    }
+    // Pattern: Module not found: Error: Can't resolve 'package-name'
+    const cantResolvePattern = /(?:Module not found|Can't resolve)[:\s]+['"]([^'"]+)['"]/g;
+    while ((match = cantResolvePattern.exec(output)) !== null) {
+        const moduleName = match[1];
+        if (!moduleName.startsWith('.') && !moduleName.startsWith('/')) {
+            const baseName = moduleName.startsWith('@')
+                ? moduleName.split('/').slice(0, 2).join('/')
+                : moduleName.split('/')[0];
+            if (!missingModules.includes(baseName)) {
+                missingModules.push(baseName);
+            }
+        }
+    }
+    return missingModules;
+}
 /**
  * Build retry prompt for implementation agent
  * @param testOutput Test failure output
+ * @param buildOutput Build output
  * @param attemptNumber Current attempt number (1-indexed)
  * @param maxRetries Maximum number of retries
+ * @param attemptHistory Optional history of previous attempts
  * @returns Prompt string for retry attempt
  */
-export function buildRetryPrompt(testOutput, buildOutput, attemptNumber, maxRetries) {
+export function buildRetryPrompt(testOutput, buildOutput, attemptNumber, maxRetries, attemptHistory) {
     const truncatedTestOutput = truncateTestOutput(testOutput);
     const truncatedBuildOutput = truncateTestOutput(buildOutput);
+    // Detect if this is a dependency issue
+    const combinedOutput = (buildOutput || '') + '\n' + (testOutput || '');
+    const missingDeps = detectMissingDependencies(combinedOutput);
+    // Parse and classify TypeScript errors from build output
+    const tsErrors = parseTypeScriptErrors(buildOutput || '');
+    const classified = classifyAndSortErrors(tsErrors);
     let prompt = `CRITICAL: Tests are failing. You attempted implementation but verification failed.
 
 This is retry attempt ${attemptNumber} of ${maxRetries}. Previous attempts failed with similar errors.
 
 `;
+    // Add special guidance for missing dependencies
+    if (missingDeps.length > 0) {
+        prompt += `**DEPENDENCY ISSUE DETECTED**
+
+The errors indicate missing npm packages: ${missingDeps.join(', ')}
+
+This is NOT a code bug - the packages need to be installed. Before making any code changes:
+1. Run \`npm install ${missingDeps.join(' ')}\` to add the missing packages
+2. If these are type definitions, also run \`npm install -D @types/${missingDeps.filter(d => !d.startsWith('@')).join(' @types/')}\`
+3. Re-run the build/tests after installing
+
+`;
+    }
+    // Add TypeScript error classification if errors were found
+    if (classified.source.length > 0 || classified.cascading.length > 0) {
+        prompt += `TYPESCRIPT ERROR CLASSIFICATION
+
+`;
+        if (classified.source.length > 0) {
+            prompt += `⚠️ SOURCE ERRORS (Fix these first - root causes):
+
+`;
+            classified.source.forEach((err) => {
+                const location = err.line ? `${err.filePath}:${err.line}` : err.filePath;
+                prompt += `- ${err.code} in ${location}: ${err.message}\n`;
+            });
+            prompt += '\n';
+        }
+        if (classified.cascading.length > 0) {
+            prompt += `💡 CASCADING ERRORS (may automatically resolve):
+
+`;
+            classified.cascading.forEach((err) => {
+                const location = err.line ? `${err.filePath}:${err.line}` : err.filePath;
+                prompt += `- ${err.code} in ${location}: ${err.message}\n`;
+            });
+            prompt += '\n';
+        }
+        prompt += `**Strategy:** Fix source errors first, as they may automatically resolve multiple cascading errors.
+
+`;
+    }
+    if (attemptHistory && attemptHistory.length > 0) {
+        prompt += buildRetryHistorySection(attemptHistory);
+        prompt += '\n';
+    }
     if (buildOutput && buildOutput.trim().length > 0) {
         prompt += `Build Output:
 \`\`\`