edsger 0.34.0 → 0.35.0

package/dist/api/features/approval-checker.js

@@ -89,14 +89,12 @@ export async function checkApprovalBeforePhaseExecution(featureId, verbose = fal
                 message: `Approval request created for status: ${currentStatus}`,
             };
         }
-        else {
-            logError('Failed to create approval request');
-            return {
-                canProceed: false,
-                requiresApproval: true,
-                message: `Failed to create approval request for status: ${currentStatus}`,
-            };
-        }
+        logError('Failed to create approval request');
+        return {
+            canProceed: false,
+            requiresApproval: true,
+            message: `Failed to create approval request for status: ${currentStatus}`,
+        };
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);

package/dist/commands/agent-workflow/chat-worker.js

@@ -115,7 +115,8 @@ function stopPolling() {
 // Event Handlers
 // ============================================================
 async function handleInit(msg) {
-    config = msg.config;
+    ;
+    ({ config } = msg);
     verbose = msg.verbose ?? false;
     log('info', `Chat worker initialized (id: ${WORKER_ID})`);
     // Load existing channels before starting the poll loop

package/dist/commands/agent-workflow/processor.js

@@ -27,7 +27,9 @@ import { calculateStats, createCompletedState, createFailedState, createInitialS
  */
 export const MAX_CONCURRENCY = 10;
 /** Path to the compiled worker entry points */
+// eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
 const __filename = fileURLToPath(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
 const __dirname = dirname(__filename);
 const WORKER_SCRIPT = join(__dirname, 'feature-worker.js');
 const CHAT_WORKER_SCRIPT = join(__dirname, 'chat-worker.js');
@@ -362,7 +364,7 @@ export class AgentWorkflowProcessor {
                 }
             });
             // Set up worker timeout
-            const workerTimeout = this.config.workflow.workerTimeout;
+            const { workerTimeout } = this.config.workflow;
             activeWorker.timeoutTimer = setTimeout(() => {
                 logError(`Worker for ${feature.name} timed out after ${Math.round(workerTimeout / 1000)}s, killing...`);
                 try {

package/dist/commands/code-review/index.js

@@ -22,13 +22,11 @@ export const runCodeReview = async (options) => {
         if (exitCode === 0) {
             logSuccess('Review completed successfully');
         }
-        else {
-            if (hasBlockingIssues) {
-                logError('Review failed with blocking issues');
-            }
-            else if (hasWarnings) {
-                logError('Review failed with warnings');
-            }
+        else if (hasBlockingIssues) {
+            logError('Review failed with blocking issues');
+        }
+        else if (hasWarnings) {
+            logError('Review failed with warnings');
         }
         process.exit(exitCode);
     }

package/dist/commands/workflow/phase-orchestrator.js

@@ -221,10 +221,8 @@ const runFromUserStoriesAnalysis = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -274,10 +272,8 @@ const runFromTestCasesAnalysis = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -378,10 +374,8 @@ const runFromPRSplitting = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -430,10 +424,8 @@ const runFromPRExecution = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -496,10 +488,8 @@ const runFromFeatureAnalysis = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -545,10 +535,8 @@ const runFromTechnicalDesign = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -588,10 +576,8 @@ const runFromCodeImplementation = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);
@@ -625,10 +611,8 @@ const runFromFunctionalTesting = async (options, config) => {
         if (prCreated) {
             await continueWithCodeReviewAndRefine(options, config, results, verbose);
         }
-        else {
-            if (verbose) {
-                logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
-            }
+        else if (verbose) {
+            logInfo('⚠️ Skipping code review and refine workflow: pull request creation failed');
         }
     }
     return finalizePipelineExecution(options, results, verbose);

package/dist/constants.js

@@ -16,7 +16,9 @@ export function getVersion() {
         return _cachedVersion;
     }
     try {
+        // eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
         const __filename = fileURLToPath(import.meta.url);
+        // eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
         const __dirname = dirname(__filename);
         const pkg = JSON.parse(readFileSync(join(__dirname, '../package.json'), 'utf-8'));
         _cachedVersion = pkg.version;

package/dist/index.js

@@ -19,10 +19,12 @@ import { runTaskWorker } from './commands/task-worker/index.js';
 import { runWorkflow } from './commands/workflow/index.js';
 import { logError, logInfo } from './utils/logger.js';
 // Get package.json version dynamically
+// eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
 const __filename = fileURLToPath(import.meta.url);
+// eslint-disable-next-line @typescript-eslint/naming-convention -- ESM __filename/__dirname polyfill
 const __dirname = dirname(__filename);
 const packageJson = JSON.parse(readFileSync(join(__dirname, '../package.json'), 'utf-8'));
-const version = packageJson.version;
+const { version } = packageJson;
 // Load environment variables (each only sets vars not already defined):
 // 1. Real environment variables (highest priority - already in process.env)
 // 2. .env file in cwd (medium priority - loaded first)

package/dist/phases/app-store-generation/__tests__/agent.test.js

@@ -18,9 +18,7 @@ function parseAppStoreResult(responseText) {
         if (jsonResult && jsonResult.app_store) {
             return { appStore: jsonResult.app_store };
         }
-        else {
-            return { error: 'Invalid JSON structure: missing app_store key' };
-        }
+        return { error: 'Invalid JSON structure: missing app_store key' };
     }
     catch (error) {
         return {

package/dist/phases/app-store-generation/__tests__/screenshot-composer.test.js

@@ -91,8 +91,7 @@ function createCompositionHtml(framedScreenshotDataUrl, spec, targetWidth, targe
 </body>
 </html>`;
     }
-    else {
-        return `<!DOCTYPE html>
+    return `<!DOCTYPE html>
 <html>
 <head>
 <style>
@@ -152,7 +151,6 @@ function createCompositionHtml(framedScreenshotDataUrl, spec, targetWidth, targe
   </div>
 </body>
 </html>`;
-    }
 }
 // ============================================================
 // escapeHtml tests
@@ -381,7 +379,7 @@ void describe('STORE_SCREENSHOT_SIZES - Orientation', () => {
         }
     });
     void it('should have Google phone in portrait (height > width)', () => {
-        const phone = STORE_SCREENSHOT_SIZES.google.phone;
+        const { phone } = STORE_SCREENSHOT_SIZES.google;
         assert.ok(phone.height > phone.width, `Google phone should be portrait: ${phone.width}x${phone.height}`);
     });
     void it('should have Google tablet_7 in portrait (height > width)', () => {

package/dist/phases/app-store-generation/agent.js

@@ -20,9 +20,7 @@ function parseAppStoreResult(responseText) {
         if (jsonResult && jsonResult.app_store) {
             return { appStore: jsonResult.app_store };
         }
-        else {
-            return { error: 'Invalid JSON structure: missing app_store key' };
-        }
+        return { error: 'Invalid JSON structure: missing app_store key' };
     }
     catch (error) {
         return {

package/dist/phases/app-store-generation/index.js

@@ -5,7 +5,7 @@ import { cloneFeatureRepo, ensureWorkspaceDir, } from '../../workspace/workspace
 import { executeAppStoreQuery } from './agent.js';
 import { prepareAppStoreContext } from './context.js';
 import { createAppStoreSystemPrompt } from './prompts.js';
-import { generateStoreScreenshots } from './screenshot-composer.js';
+import { generateStoreScreenshots, } from './screenshot-composer.js';
 import { uploadAppStoreScreenshots } from './uploader.js';
 export const generateAppStoreAssets = async (options, config
 // eslint-disable-next-line complexity -- orchestration function with sequential asset generation steps
@@ -17,7 +17,7 @@ export const generateAppStoreAssets = async (options, config
     }
     try {
         // Resolve locale: use explicit param, or fetch from Apple App Store Connect
-        let locale = options.locale;
+        let { locale } = options;
         if (!locale) {
             logInfo('No locale specified, fetching primary locale from Apple App Store Connect...');
             const appleLocale = await getAppStorePrimaryLocale(productId, verbose);
@@ -53,7 +53,8 @@ export const generateAppStoreAssets = async (options, config
         if (!aiResult) {
             throw new Error('No results received from AI');
         }
-        const listings = (aiResult.listings || {});
+        const listings = (aiResult.listings ||
+            {});
         const screenshotSpecs = (aiResult.screenshots || []);
         logInfo(`AI generated: ${Object.keys(listings).length} locale(s), ${screenshotSpecs.length} screenshot spec(s)`);
         // Ensure store configs exist

package/dist/phases/app-store-generation/screenshot-composer.js

@@ -52,7 +52,7 @@ async function composeStoreScreenshot(opts) {
 }
 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;
     }
@@ -144,9 +144,8 @@ function createCompositionHtml(framedScreenshotDataUrl, spec, targetWidth, targe
 </body>
 </html>`;
     }
-    else {
-        // Landscape: text on left, device on right
-        return `<!DOCTYPE html>
+    // Landscape: text on left, device on right
+    return `<!DOCTYPE html>
 <html>
 <head>
 <style>
@@ -206,7 +205,6 @@ function createCompositionHtml(framedScreenshotDataUrl, spec, targetWidth, targe
   </div>
 </body>
 </html>`;
-    }
 }
 function escapeHtml(str) {
     return str

package/dist/phases/autonomous/index.js

@@ -48,7 +48,9 @@ function userMessage(content) {
 }
 async function* prompt(userPrompt) {
     yield userMessage(userPrompt);
-    await new Promise((res) => setTimeout(res, 10000));
+    await new Promise((res) => {
+        setTimeout(res, 10000);
+    });
 }
 /**
  * Run a single autonomous iteration using Claude Code SDK
@@ -143,8 +145,7 @@ async function handleSuccessfulIteration(opts) {
     let prNumber;
     if (!prUrl) {
         const prInfo = await createAutonomousPR(githubConfig, devBranchName, currentBranch, verbose);
-        prUrl = prInfo.prUrl;
-        prNumber = prInfo.prNumber;
+        ({ prUrl, prNumber } = prInfo);
     }
     // Log iteration completion to audit logs
     await logFeaturePhaseEvent({

package/dist/phases/branch-planning/context.js

@@ -11,7 +11,7 @@ export async function fetchBranchPlanningContext(featureId, verbose) {
             logInfo(`Fetching branch planning context for feature: ${featureId}`);
         }
         // Fetch all required data in parallel for better performance
-        const [feature, user_stories, test_cases, existing_branches] = await Promise.all([
+        const [feature, userStories, testCases, existingBranches] = await Promise.all([
             getFeature(featureId, verbose),
             getUserStories(featureId, verbose),
             getTestCases(featureId, verbose),
@@ -22,17 +22,17 @@ export async function fetchBranchPlanningContext(featureId, verbose) {
             logInfo(`✅ Branch planning context fetched successfully:`);
             logInfo(`   Feature: ${feature.name}`);
             logInfo(`   Product: ${product.name}`);
-            logInfo(`   User Stories: ${user_stories.length}`);
-            logInfo(`   Test Cases: ${test_cases.length}`);
+            logInfo(`   User Stories: ${userStories.length}`);
+            logInfo(`   Test Cases: ${testCases.length}`);
             logInfo(`   Technical Design: ${feature.technical_design ? 'Yes' : 'No'}`);
-            logInfo(`   Existing Branches: ${existing_branches.length}`);
+            logInfo(`   Existing Branches: ${existingBranches.length}`);
         }
         return {
             feature,
             product,
-            user_stories,
-            test_cases,
-            existing_branches,
+            user_stories: userStories,
+            test_cases: testCases,
+            existing_branches: existingBranches,
         };
     }
     catch (error) {

package/dist/phases/branch-planning/index.js

@@ -29,7 +29,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);
+    });
 }
 /**
  * Fetch feedbacks for branch planning phase
@@ -81,8 +83,8 @@ async function buildUserPrompt(opts) {
         }
         const feedbacksInfo = await formatFeedbacksForContext(feedbacksContext);
         const currentPlanInfo = existingBranchesInfo || 'No existing branch plan documented.';
-        const prompt = createImprovementPrompt(feedbacksInfo, currentPlanInfo);
-        return `${prompt}\n\n## Feature Context\n\n${contextInfo}`;
+        const systemPrompt = createImprovementPrompt(feedbacksInfo, currentPlanInfo);
+        return `${systemPrompt}\n\n## Feature Context\n\n${contextInfo}`;
     }
     return createBranchPlanningPromptWithContext(featureId, contextInfo, existingBranchesInfo || '');
 }

package/dist/phases/bug-fixing/analyzer.js

@@ -27,7 +27,9 @@ function userMessage(content) {
 }
 async function* prompt(bugFixPrompt) {
     yield userMessage(bugFixPrompt);
-    await new Promise((res) => setTimeout(res, 10000));
+    await new Promise((res) => {
+        setTimeout(res, 10000);
+    });
 }
 export const fixTestFailures = async (options, config
 // eslint-disable-next-line complexity -- orchestration function with agent setup, message processing, and result extraction
@@ -131,15 +133,13 @@ export const fixTestFailures = async (options, config
                 attemptNumber,
             };
         }
-        else {
-            return {
-                featureId,
-                fixSummary: null,
-                status: 'error',
-                message: 'Bug fixing failed or incomplete',
-                attemptNumber,
-            };
-        }
+        return {
+            featureId,
+            fixSummary: null,
+            status: 'error',
+            message: 'Bug fixing failed or incomplete',
+            attemptNumber,
+        };
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
@@ -272,16 +272,16 @@ function parseBugFixResponse(response) {
     const summaryMatch = response.match(/## Fix Summary\n([\s\S]*?)(?=\n##|\n\n|$)/);
     const summary = summaryMatch ? summaryMatch[1].trim() : 'Bug fixes applied';
     const filesMatch = response.match(/## Files Fixed\n([\s\S]*?)(?=\n##|\n\n|$)/);
-    let files_modified = [];
+    let filesModified = [];
     if (filesMatch) {
-        files_modified = filesMatch[1]
+        filesModified = filesMatch[1]
             .split('\n')
             .filter((line) => line.trim().startsWith('-'))
             .map((line) => line.replace(/^-\s*/, '').trim());
     }
     return {
         summary,
-        files_modified,
+        files_modified: filesModified,
         tests_passed: true,
         fixes_applied: [],
     };

package/dist/phases/bug-fixing/context-fetcher.js

@@ -10,13 +10,13 @@ export async function fetchBugFixingContext(featureId, verbose) {
             logInfo(`Fetching complete bug fixing 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),
         ]);
         // Get latest test report to understand what has been tested
-        let latest_test_report = null;
+        let latestTestReport = null;
         try {
             if (verbose) {
                 logInfo('Fetching latest test report...');
@@ -25,7 +25,7 @@ export async function fetchBugFixingContext(featureId, verbose) {
                 feature_id: featureId,
             }));
             if (testReportResult.test_report) {
-                latest_test_report = testReportResult.test_report;
+                latestTestReport = testReportResult.test_report;
             }
         }
         catch (error) {
@@ -36,16 +36,16 @@ export async function fetchBugFixingContext(featureId, verbose) {
         if (verbose) {
             logInfo(`✅ Bug fixing context fetched successfully:`);
             logInfo(`   Feature: ${feature.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(`   Technical Design: ${feature.technical_design ? 'Available' : 'Not available'}`);
-            logInfo(`   Latest Test Report: ${latest_test_report ? 'Available' : 'Not available'}`);
+            logInfo(`   Latest Test Report: ${latestTestReport ? 'Available' : 'Not available'}`);
         }
         return {
             feature,
-            user_stories,
-            test_cases,
-            latest_test_report,
+            user_stories: userStories,
+            test_cases: testCases,
+            latest_test_report: latestTestReport,
         };
     }
     catch (error) {

package/dist/phases/bug-fixing/mcp-server.js

@@ -58,23 +58,20 @@ export const createBugFixingMcpServer = () => {
                 files_modified: z
                     .array(z.string())
                     .describe('List of files that were modified'),
-            }, async (args) => {
-                // This tool verifies the bug fix
-                return {
-                    content: [
-                        {
-                            type: 'text',
-                            text: JSON.stringify({
-                                verified: true,
-                                feature_id: args.feature_id,
-                                summary: args.fix_summary,
-                                files_modified: args.files_modified,
-                                message: 'Bug fix verification complete',
-                            }, null, 2),
-                        },
-                    ],
-                };
-            }),
+            }, (args) => Promise.resolve({
+                content: [
+                    {
+                        type: 'text',
+                        text: JSON.stringify({
+                            verified: true,
+                            feature_id: args.feature_id,
+                            summary: args.fix_summary,
+                            files_modified: args.files_modified,
+                            message: 'Bug fix verification complete',
+                        }, null, 2),
+                    },
+                ],
+            })),
         ],
     });
 };

package/dist/phases/code-implementation/context.js

@@ -10,7 +10,7 @@ export async function fetchCodeImplementationContext(featureId, verbose) {
             logInfo(`Fetching complete code implementation 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 fetchCodeImplementationContext(featureId, verbose) {
             logInfo(`✅ Code implementation context fetched successfully:`);
             logInfo(`   Feature: ${feature.name}`);
             logInfo(`   Product: ${product.name}`);
-            logInfo(`   User Stories: ${user_stories.length}`);
-            logInfo(`   Test Cases: ${test_cases.length}`);
+            logInfo(`   User Stories: ${userStories.length}`);
+            logInfo(`   Test Cases: ${testCases.length}`);
             logInfo(`   Technical Design: ${feature.technical_design ? 'Available' : 'Not available'}`);
         }
         return {
             feature,
             product,
-            user_stories,
-            test_cases,
+            user_stories: userStories,
+            test_cases: testCases,
             technical_design: feature.technical_design,
         };
     }

package/dist/phases/code-implementation/index.js

@@ -33,7 +33,9 @@ function userMessage(content) {
 }
 async function* prompt(implementationPrompt) {
     yield userMessage(implementationPrompt);
-    await new Promise((res) => setTimeout(res, 10000));
+    await new Promise((res) => {
+        setTimeout(res, 10000);
+    });
 }
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 function processAssistantContent(content, verbose) {
@@ -265,8 +267,7 @@ export const implementFeatureCode = async (options, config, checklistContext
             });
             const baseBranchInfo = getBaseBranchInfo(currentBranch, allBranches, baseBranch);
             actualBaseBranch = baseBranchInfo.baseBranch;
-            rebaseTargetBranch = baseBranchInfo.rebaseTargetBranch;
-            originalBaseBranch = baseBranchInfo.originalBaseBranch;
+            ({ rebaseTargetBranch, originalBaseBranch } = baseBranchInfo);
             baseBranchCompleted = baseBranchInfo.baseBranchCompleted || false;
             if (verbose) {
                 logInfo(`🔗 Branch chaining detected:`);
@@ -525,7 +526,7 @@ export const implementFeatureCode = async (options, config, checklistContext
                 maxIterations,
                 verbose,
             });
-            verificationResult = verificationCycle.verificationResult;
+            ({ verificationResult } = verificationCycle);
             // If verification passed, exit
             if (verificationCycle.passed) {
                 if (verbose) {

package/dist/phases/code-implementation-verification/agent.js

@@ -10,7 +10,9 @@ function userMessage(content) {
 }
 async function* prompt(verificationPrompt) {
     yield userMessage(verificationPrompt);
-    await new Promise((res) => setTimeout(res, 10000));
+    await new Promise((res) => {
+        setTimeout(res, 10000);
+    });
 }
 const FAILED_VERIFICATION = {
     overall_status: 'failed',

package/dist/phases/code-refine/context.js

@@ -118,7 +118,7 @@ export async function fetchPRDataGraphQL(octokit, owner, repo, prNumber, verbose
         repo,
         prNumber,
     });
-    const pullRequest = result.repository.pullRequest;
+    const { pullRequest } = result.repository;
     // Transform GraphQL reviews to our interface
     const reviews = (pullRequest.reviews.nodes || []).map(
     // eslint-disable-next-line @typescript-eslint/no-explicit-any

package/dist/phases/code-refine/index.js

@@ -9,8 +9,8 @@ import { getBaseBranchInfo, getBranches, updateBranch, } from '../../services/br
 import { getUncommittedFiles, hasUncommittedChanges, prepareCustomBranchGitEnvironmentAsync, preparePhaseGitEnvironmentAsync, syncFeatBranchWithMain, } from '../../utils/git-branch-manager.js';
 import { gitPushCurrentBranch } from '../../utils/git-push.js';
 import { logError, logInfo } from '../../utils/logger.js';
-import { fetchCodeRefineContext } from './context.js';
 import { verifyAndResolveComments, } from '../code-refine-verification/index.js';
+import { fetchCodeRefineContext } from './context.js';
 import { executeRefineIteration } from './refine-iteration.js';
 /**
  * Log base branch info for verbose mode
@@ -295,7 +295,7 @@ Please ensure Claude Code commits all changes before completing the refine phase
                 const verificationData = verificationResult.data;
                 const suggestions = verificationData.suggestions || [];
                 const unresolvedCommentDetails = verificationData.unresolvedCommentDetails || [];
-                const unresolvedReviewDetails = verificationData.unresolvedReviewDetails;
+                const { unresolvedReviewDetails } = verificationData;
                 verificationFailureContext = {
                     attempt: currentIteration + 1,
                     suggestions,
@@ -313,11 +313,9 @@ Please ensure Claude Code commits all changes before completing the refine phase
                     logInfo(`\n🔄 Will retry... (${MAX_REFINE_ITERATIONS - currentIteration} attempts remaining)`);
                 }
             }
-            else {
+            else if (verbose) {
                 // Max iterations reached
-                if (verbose) {
-                    logInfo(`\n⚠️  Maximum iterations (${MAX_REFINE_ITERATIONS}) reached. Verification still failing.`);
-                }
+                logInfo(`\n⚠️  Maximum iterations (${MAX_REFINE_ITERATIONS}) reached. Verification still failing.`);
             }
         }
         // Return last verification result (which failed)