edsger 0.34.0 → 0.35.0

package/dist/phases/task/agent.js

@@ -59,14 +59,12 @@ export async function executeTask(taskPrompt, systemPrompt, config, verbose) {
                         summary: message.result || lastAssistantResponse || 'Task completed',
                     };
                 }
-                else {
-                    logError(`Task incomplete: ${message.subtype}`);
-                    return {
-                        success: false,
-                        summary: lastAssistantResponse ||
-                            `Task ended with status: ${message.subtype}`,
-                    };
-                }
+                logError(`Task incomplete: ${message.subtype}`);
+                return {
+                    success: false,
+                    summary: lastAssistantResponse ||
+                        `Task ended with status: ${message.subtype}`,
+                };
             }
         }
         return {

package/dist/phases/technical-design/context.js

@@ -10,7 +10,7 @@ export async function fetchTechnicalDesignContext(featureId, verbose) {
             logInfo(`Fetching complete technical design context for feature: ${featureId}`);
         }
         // Fetch all required data in parallel for better performance
-        const [feature, user_stories, test_cases] = await Promise.all([
+        const [feature, userStories, testCases] = await Promise.all([
             getFeature(featureId, verbose),
             getUserStories(featureId, verbose),
             getTestCases(featureId, verbose),
@@ -20,15 +20,15 @@ export async function fetchTechnicalDesignContext(featureId, verbose) {
             logInfo(`✅ Technical design context fetched successfully:`);
             logInfo(`   Feature: ${feature.name}`);
             logInfo(`   Product: ${product.name}`);
-            logInfo(`   User Stories: ${user_stories.length}`);
-            logInfo(`   Test Cases: ${test_cases.length} (${test_cases.filter((tc) => tc.is_critical).length} critical)`);
+            logInfo(`   User Stories: ${userStories.length}`);
+            logInfo(`   Test Cases: ${testCases.length} (${testCases.filter((tc) => tc.is_critical).length} critical)`);
             logInfo(`   Existing Technical Design: ${feature.technical_design ? 'Yes' : 'No'}`);
         }
         return {
             feature,
             product,
-            user_stories,
-            test_cases,
+            user_stories: userStories,
+            test_cases: testCases,
         };
     }
     catch (error) {

package/dist/phases/technical-design/index.js

@@ -20,7 +20,9 @@ function userMessage(content) {
 }
 async function* prompt(analysisPrompt) {
     yield userMessage(analysisPrompt);
-    await new Promise((res) => setTimeout(res, 10000));
+    await new Promise((res) => {
+        setTimeout(res, 10000);
+    });
 }
 export const generateTechnicalDesign = async (options, config, checklistContext
 // eslint-disable-next-line complexity -- orchestration function with context assembly, design generation, and verification
@@ -111,6 +113,7 @@ export const generateTechnicalDesign = async (options, config, checklistContext
                         const parsed = JSON.parse(jsonStr);
                         const design = parsed.technical_design_result?.technical_design;
                         if (design && typeof design === 'string') {
+                            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
                             structuredDesignResult.technical_design = design;
                             await updateTechnicalDesign(featureId, design, verbose);
                         }
@@ -138,7 +141,7 @@ export const generateTechnicalDesign = async (options, config, checklistContext
                 maxIterations,
                 verbose,
             });
-            verificationResult = verificationCycle.verificationResult;
+            ({ verificationResult } = verificationCycle);
             // If verification passed, exit
             if (verificationCycle.passed) {
                 if (verbose) {
@@ -267,10 +270,8 @@ async function prepareDesignContext(featureId, checklistContext, verbose) {
                 logInfo(`📄 Feedbacks section length: ${feedbacksInfo.length} characters`);
             }
         }
-        else {
-            if (verbose) {
-                logInfo('ℹ️  No unresolved feedbacks found for this phase');
-            }
+        else if (verbose) {
+            logInfo('ℹ️  No unresolved feedbacks found for this phase');
         }
     }
     catch (error) {
@@ -325,9 +326,9 @@ async function prepareDesignContext(featureId, checklistContext, verbose) {
  * Strip the "Your Previous Technical Design" section from the improvement prompt.
  * When resuming a session, the agent already has the design in its history.
  */
-function stripPreviousDesignSection(prompt) {
+function stripPreviousDesignSection(systemPrompt) {
     // Remove content between deterministic markers (added in prompts.ts)
-    return prompt.replace(/<!-- PREVIOUS_DESIGN_START -->[\s\S]*?<!-- PREVIOUS_DESIGN_END -->/, '');
+    return systemPrompt.replace(/<!-- PREVIOUS_DESIGN_START -->[\s\S]*?<!-- PREVIOUS_DESIGN_END -->/, '');
 }
 /**
  * Execute design query and parse result

package/dist/phases/test-cases-analysis/agent.js

@@ -15,9 +15,7 @@ export function parseAnalysisResult(responseText) {
         if (jsonResult && jsonResult.analysis) {
             return { analysis: jsonResult.analysis };
         }
-        else {
-            return { error: 'Invalid JSON structure' };
-        }
+        return { error: 'Invalid JSON structure' };
     }
     catch (error) {
         return {

package/dist/phases/test-cases-analysis/context.js

@@ -10,7 +10,7 @@ export async function fetchTestCasesAnalysisContext(featureId, verbose) {
         if (verbose) {
             logInfo(`Fetching test cases analysis context for feature: ${featureId}`);
         }
-        const [feature, existing_user_stories, existing_test_cases] = await Promise.all([
+        const [feature, existingUserStories, existingTestCases] = await Promise.all([
             getFeature(featureId, verbose),
             getUserStories(featureId, verbose),
             getTestCases(featureId, verbose),
@@ -20,14 +20,14 @@ export async function fetchTestCasesAnalysisContext(featureId, verbose) {
             logInfo(`✅ Test cases analysis context fetched successfully:`);
             logInfo(`   Feature: ${feature.name}`);
             logInfo(`   Product: ${product.name}`);
-            logInfo(`   User Stories: ${existing_user_stories.length}`);
-            logInfo(`   Existing Test Cases: ${existing_test_cases.length}`);
+            logInfo(`   User Stories: ${existingUserStories.length}`);
+            logInfo(`   Existing Test Cases: ${existingTestCases.length}`);
         }
         return {
             feature,
             product,
-            existing_user_stories,
-            existing_test_cases,
+            existing_user_stories: existingUserStories,
+            existing_test_cases: existingTestCases,
         };
     }
     catch (error) {

package/dist/phases/test-cases-analysis/index.js

@@ -89,8 +89,8 @@ export const analyseTestCases = async (options, config, checklistContext
                     const parsed = parseAnalysisResult(responseText);
                     if (parsed.analysis) {
                         await deleteTestCaseArtifacts(currentDraftTestCaseIds, verbose);
-                        const { testCaseIds } = await saveTestCasesAsDraft(featureId, parsed.analysis.created_test_cases || [], verbose);
-                        currentDraftTestCaseIds = testCaseIds;
+                        const { testCaseIds: newTestCaseIds } = await saveTestCasesAsDraft(featureId, parsed.analysis.created_test_cases || [], verbose);
+                        currentDraftTestCaseIds = newTestCaseIds;
                         structuredAnalysisResult = parsed.analysis;
                     }
                     else {
@@ -113,7 +113,7 @@ export const analyseTestCases = async (options, config, checklistContext
                 featureId,
                 verbose,
             });
-            verificationResult = verificationCycle.verificationResult;
+            ({ verificationResult } = verificationCycle);
             if (verificationCycle.passed) {
                 if (verbose) {
                     logInfo('✅ Verification passed! Updating all draft test cases to pending_approval status...');

package/dist/phases/test-cases-analysis/outcome.d.ts

@@ -4,7 +4,7 @@ export declare function getAllDraftTestCaseIds(featureId: string, verbose?: bool
 export declare function deleteTestCaseArtifacts(testCaseIds: string[], verbose?: boolean): Promise<void>;
 export declare function deleteSpecificTestCases(featureId: string, deletedTestCaseIds: string[], deletionReasons: Record<string, string>, verbose?: boolean): Promise<void>;
 export declare function updateTestCasesToReady(testCaseIds: string[], verbose?: boolean): Promise<void>;
-export declare function saveTestCasesAsDraft(featureId: string, created_test_cases: any[], verbose?: boolean): Promise<{
+export declare function saveTestCasesAsDraft(featureId: string, createdTestCases: any[], verbose?: boolean): Promise<{
     testCaseIds: string[];
 }>;
 export interface TestCasesAnalysisResult {

package/dist/phases/test-cases-analysis/outcome.js

@@ -77,16 +77,16 @@ export async function updateTestCasesToReady(testCaseIds, verbose) {
 }
 export async function saveTestCasesAsDraft(featureId, 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-created_test_cases, verbose) {
+createdTestCases, verbose) {
     const testCaseIds = [];
-    if (created_test_cases && created_test_cases.length > 0) {
+    if (createdTestCases && createdTestCases.length > 0) {
         if (verbose) {
-            logInfo(`Saving ${created_test_cases.length} test cases as draft...`);
+            logInfo(`Saving ${createdTestCases.length} test cases as draft...`);
         }
         try {
             const result = (await callMcpEndpoint('test_cases/create', {
                 feature_id: featureId,
-                test_cases: created_test_cases.map((testCase) => ({
+                test_cases: createdTestCases.map((testCase) => ({
                     name: testCase.name,
                     description: testCase.description,
                     is_critical: testCase.is_critical || false,

package/dist/phases/user-stories-analysis/agent.js

@@ -15,9 +15,7 @@ export function parseAnalysisResult(responseText) {
         if (jsonResult && jsonResult.analysis) {
             return { analysis: jsonResult.analysis };
         }
-        else {
-            return { error: 'Invalid JSON structure' };
-        }
+        return { error: 'Invalid JSON structure' };
     }
     catch (error) {
         return {

package/dist/phases/user-stories-analysis/context.js

@@ -10,7 +10,7 @@ export async function fetchUserStoriesAnalysisContext(featureId, verbose) {
         if (verbose) {
             logInfo(`Fetching user stories analysis context for feature: ${featureId}`);
         }
-        const [feature, existing_user_stories, existing_test_cases] = await Promise.all([
+        const [feature, existingUserStories, existingTestCases] = await Promise.all([
             getFeature(featureId, verbose),
             getUserStories(featureId, verbose),
             getTestCases(featureId, verbose),
@@ -20,14 +20,14 @@ export async function fetchUserStoriesAnalysisContext(featureId, verbose) {
             logInfo(`✅ User stories analysis context fetched successfully:`);
             logInfo(`   Feature: ${feature.name}`);
             logInfo(`   Product: ${product.name}`);
-            logInfo(`   Existing User Stories: ${existing_user_stories.length}`);
-            logInfo(`   Existing Test Cases: ${existing_test_cases.length}`);
+            logInfo(`   Existing User Stories: ${existingUserStories.length}`);
+            logInfo(`   Existing Test Cases: ${existingTestCases.length}`);
         }
         return {
             feature,
             product,
-            existing_user_stories,
-            existing_test_cases,
+            existing_user_stories: existingUserStories,
+            existing_test_cases: existingTestCases,
         };
     }
     catch (error) {

package/dist/phases/user-stories-analysis/index.js

@@ -89,8 +89,8 @@ export const analyseUserStories = async (options, config, checklistContext
                     const parsed = parseAnalysisResult(responseText);
                     if (parsed.analysis) {
                         await deleteUserStoryArtifacts(currentDraftUserStoryIds, verbose);
-                        const { userStoryIds } = await saveUserStoriesAsDraft(featureId, parsed.analysis.created_user_stories || [], verbose);
-                        currentDraftUserStoryIds = userStoryIds;
+                        const { userStoryIds: newUserStoryIds } = await saveUserStoriesAsDraft(featureId, parsed.analysis.created_user_stories || [], verbose);
+                        currentDraftUserStoryIds = newUserStoryIds;
                         structuredAnalysisResult = parsed.analysis;
                     }
                     else {
@@ -113,7 +113,7 @@ export const analyseUserStories = async (options, config, checklistContext
                 featureId,
                 verbose,
             });
-            verificationResult = verificationCycle.verificationResult;
+            ({ verificationResult } = verificationCycle);
             if (verificationCycle.passed) {
                 if (verbose) {
                     logInfo('✅ Verification passed! Updating all draft user stories to pending_approval status...');

package/dist/phases/user-stories-analysis/outcome.d.ts

@@ -4,7 +4,7 @@ export declare function getAllDraftUserStoryIds(featureId: string, verbose?: boo
 export declare function deleteUserStoryArtifacts(userStoryIds: string[], verbose?: boolean): Promise<void>;
 export declare function deleteSpecificUserStories(featureId: string, deletedUserStoryIds: string[], deletionReasons: Record<string, string>, verbose?: boolean): Promise<void>;
 export declare function updateUserStoriesToReady(userStoryIds: string[], verbose?: boolean): Promise<void>;
-export declare function saveUserStoriesAsDraft(featureId: string, created_user_stories: any[], verbose?: boolean): Promise<{
+export declare function saveUserStoriesAsDraft(featureId: string, createdUserStories: any[], verbose?: boolean): Promise<{
     userStoryIds: string[];
 }>;
 export interface UserStoriesAnalysisResult {

package/dist/phases/user-stories-analysis/outcome.js

@@ -77,16 +77,16 @@ export async function updateUserStoriesToReady(userStoryIds, verbose) {
 }
 export async function saveUserStoriesAsDraft(featureId, 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-created_user_stories, verbose) {
+createdUserStories, verbose) {
     const userStoryIds = [];
-    if (created_user_stories && created_user_stories.length > 0) {
+    if (createdUserStories && createdUserStories.length > 0) {
         if (verbose) {
-            logInfo(`Saving ${created_user_stories.length} user stories as draft...`);
+            logInfo(`Saving ${createdUserStories.length} user stories as draft...`);
         }
         try {
             const result = (await callMcpEndpoint('user_stories/create', {
                 feature_id: featureId,
-                user_stories: created_user_stories.map((story) => ({
+                user_stories: createdUserStories.map((story) => ({
                     title: story.title,
                     description: story.description,
                     status: 'draft',

package/dist/services/coaching/coaching-loop.js

@@ -71,7 +71,7 @@ export function isStagnant(ratings) {
 // eslint-disable-next-line complexity -- coaching loop with rating, decision, and improvement cycles
 export async function runCoachingLoop(options) {
     const { featureId, phase, onImproved, checklistItems, verbose } = options;
-    let executionSessionId = options.executionSessionId;
+    let { executionSessionId } = options;
     let coachingSessionId = null;
     const allRatings = [];
     let iteration = 0;

package/dist/services/coaching/self-rating.js

@@ -117,7 +117,7 @@ export function parseSelfRating(response, phase, verbose) {
         const rating = parsed.self_rating || parsed;
         // Validate score
         const score = Number(rating.score);
-        if (isNaN(score) || score < 0 || score > 100) {
+        if (Number.isNaN(score) || score < 0 || score > 100) {
             throw new Error(`Invalid score: ${rating.score}`);
         }
         // Build criteria_scores with defaults for missing criteria

package/dist/services/phase-ratings.js

@@ -60,7 +60,8 @@ export async function getPhaseRatings(featureId, phase, verbose) {
             feature_id: featureId,
             phase,
         });
-        const ratings = (result?.ratings || []);
+        const ratings = (result?.ratings ||
+            []);
         if (verbose) {
             logDebug(`Fetched ${ratings.length} ratings for ${phase} (feature: ${featureId})`, verbose);
         }

package/dist/services/video/__tests__/video-pipeline.test.js

@@ -233,7 +233,9 @@ void describe('Concurrency Limiter', () => {
             if (currentConcurrent > maxConcurrent) {
                 maxConcurrent = currentConcurrent;
             }
-            await new Promise((r) => setTimeout(r, 10));
+            await new Promise((r) => {
+                setTimeout(r, 10);
+            });
             currentConcurrent--;
             return i;
         });

package/dist/services/video/retry.js

@@ -31,7 +31,8 @@ export async function fetchWithRetry(url, init, options = {}) {
         }
         catch (error) {
             // Network errors (ECONNRESET, ETIMEDOUT, etc.) are retryable
-            if (error instanceof TypeError || (error instanceof Error && isNetworkError(error))) {
+            if (error instanceof TypeError ||
+                (error instanceof Error && isNetworkError(error))) {
                 lastError = error instanceof Error ? error : new Error(String(error));
                 if (attempt < maxRetries) {
                     const delay = calculateBackoff(attempt, baseDelayMs);
@@ -69,5 +70,7 @@ function isNetworkError(error) {
         message.includes('abort'));
 }
 function sleep(ms) {
-    return new Promise((resolve) => setTimeout(resolve, ms));
+    return new Promise((resolve) => {
+        setTimeout(resolve, ms);
+    });
 }

package/dist/services/video/screenshot-generator.js

@@ -14,7 +14,7 @@ import { generateDeviceFrameHtml, } from './device-frames.js';
  */
 async function loadChromium() {
     try {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any, no-new-func -- dynamic import to avoid bundler resolution of optional peer dependency
         const pw = await Function('return import("playwright")')();
         return pw.chromium;
     }

package/dist/services/video/video-assembler.d.ts

@@ -3,7 +3,7 @@
  * Uses ffmpeg for professional-quality video output.
  */
 import type { CapturedScene } from './screenshot-generator.js';
-import type { GeneratedAudio } from './tts-generator.js';
+import { type GeneratedAudio } from './tts-generator.js';
 export interface VideoAssemblyOptions {
     /** Output file path for the final video */
     outputPath: string;

package/dist/services/video/video-assembler.js

@@ -162,8 +162,8 @@ export async function assembleVideo(scenes, audioFiles, options, verbose) {
     if (!optWidth || !optHeight) {
         const detected = detectScreenshotResolution(scenes[0]?.screenshotPath);
         if (detected) {
-            width = detected.width;
-            height = detected.height;
+            ;
+            ({ width, height } = detected);
             if (verbose) {
                 logInfo(`Auto-detected video resolution: ${width}x${height}`);
             }
@@ -179,7 +179,7 @@ export async function assembleVideo(scenes, audioFiles, options, verbose) {
     const sceneDurations = [];
     for (const scene of sortedScenes) {
         const audio = audioMap.get(scene.order);
-        let duration = scene.duration;
+        let { duration } = scene;
         if (audio) {
             try {
                 duration = await getAudioDuration(audio.audioPath);

package/dist/utils/conflict-resolver.js

@@ -167,23 +167,21 @@ export async function resolveConflictsWithAgent(config = {}) {
                 failedFiles: [],
             };
         }
-        else {
-            // Rebase still in progress - something went wrong
-            const remainingConflicts = getConflictFiles();
-            if (verbose) {
-                logError(`❌ Rebase still in progress with ${remainingConflicts.length} conflicts`);
-                if (lastMessage) {
-                    logError(`   Last message: ${lastMessage.substring(0, 200)}...`);
-                }
+        // Rebase still in progress - something went wrong
+        const remainingConflicts = getConflictFiles();
+        if (verbose) {
+            logError(`❌ Rebase still in progress with ${remainingConflicts.length} conflicts`);
+            if (lastMessage) {
+                logError(`   Last message: ${lastMessage.substring(0, 200)}...`);
             }
-            abortRebase(verbose);
-            return {
-                success: false,
-                resolvedFiles: [],
-                failedFiles: remainingConflicts,
-                error: `Rebase not completed. Remaining conflicts: ${remainingConflicts.join(', ')}`,
-            };
         }
+        abortRebase(verbose);
+        return {
+            success: false,
+            resolvedFiles: [],
+            failedFiles: remainingConflicts,
+            error: `Rebase not completed. Remaining conflicts: ${remainingConflicts.join(', ')}`,
+        };
     }
     catch (error) {
         if (verbose) {

package/dist/utils/git-branch-manager-async.js

@@ -337,13 +337,11 @@ export async function switchToFeatureBranchAndRebaseAsync(options) {
                     resolvedFiles: result.resolvedFiles,
                 };
             }
-            else {
-                // Conflict resolution failed, abort rebase
-                abortRebase(verbose);
-                throw new Error(`Failed to automatically resolve conflicts during rebase ${featureBranch} with ${actualRebaseTarget}.\n` +
-                    `${result.error || 'Unknown error'}\n` +
-                    `Please resolve conflicts manually and try again.`);
-            }
+            // Conflict resolution failed, abort rebase
+            abortRebase(verbose);
+            throw new Error(`Failed to automatically resolve conflicts during rebase ${featureBranch} with ${actualRebaseTarget}.\n` +
+                `${result.error || 'Unknown error'}\n` +
+                `Please resolve conflicts manually and try again.`);
         }
         // No conflict resolution attempted or not a conflict error
         // Abort rebase and throw error

package/dist/utils/git-push.js

@@ -39,7 +39,7 @@ export function gitPush(options) {
     const credArgs = buildCredentialArgs(token);
     const execOpts = {
         cwd,
-        stdio: (verbose ? 'inherit' : 'pipe'),
+        stdio: verbose ? 'inherit' : 'pipe',
     };
     if (verbose) {
         logInfo(`📤 Pushing branch ${branchName} to remote...`);
@@ -87,7 +87,7 @@ export function gitForcePush(options) {
     const credArgs = buildCredentialArgs(token);
     const execOpts = {
         cwd,
-        stdio: (verbose ? 'inherit' : 'pipe'),
+        stdio: verbose ? 'inherit' : 'pipe',
     };
     if (verbose) {
         logInfo(`📤 Force pushing ${branchName} to remote...`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edsger",
-  "version": "0.34.0",
+  "version": "0.35.0",
   "type": "module",
   "bin": {
     "edsger": "dist/index.js"