@ariso-ai/ivan 1.0.10 → 1.0.12

package/dist/services/address-task-executor.d.ts

@@ -11,7 +11,7 @@ export declare class AddressTaskExecutor {
     constructor();
     private getClaudeExecutor;
     private getOpenAIService;
-    executeAddressTasks(tasks: Task[]): Promise<void>;
+    executeAddressTasks(tasks: Task[], quiet?: boolean): Promise<void>;
     private getUnaddressedComments;
     private generateReviewInstructions;
     private tryCommitWithFixes;

package/dist/services/address-task-executor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"address-task-executor.d.ts","sourceRoot":"","sources":["../../src/services/address-task-executor.ts"],"names":[],"mappings":"AAQA,OAAO,EAAE,IAAI,EAAE,MAAM,gBAAgB,CAAC;AAItC,qBAAa,mBAAmB;IAC9B,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,UAAU,CAA4B;IAC9C,OAAO,CAAC,cAAc,CAAgC;IACtD,OAAO,CAAC,aAAa,CAA8B;IACnD,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,SAAS,CAA2B;IAC5C,OAAO,CAAC,UAAU,CAAS;IAC3B,OAAO,CAAC,gBAAgB,CAAqB;;IAQ7C,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,gBAAgB;IAOlB,mBAAmB,CAAC,KAAK,EAAE,IAAI,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC;YA6oBzC,sBAAsB;YAwGtB,0BAA0B;YAyD1B,kBAAkB;CAoEjC"}
+{"version":3,"file":"address-task-executor.d.ts","sourceRoot":"","sources":["../../src/services/address-task-executor.ts"],"names":[],"mappings":"AAQA,OAAO,EAAE,IAAI,EAAE,MAAM,gBAAgB,CAAC;AAItC,qBAAa,mBAAmB;IAC9B,OAAO,CAAC,UAAU,CAAa;IAC/B,OAAO,CAAC,UAAU,CAA4B;IAC9C,OAAO,CAAC,cAAc,CAAgC;IACtD,OAAO,CAAC,aAAa,CAA8B;IACnD,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,SAAS,CAA2B;IAC5C,OAAO,CAAC,UAAU,CAAS;IAC3B,OAAO,CAAC,gBAAgB,CAAqB;;IAQ7C,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,gBAAgB;IAOlB,mBAAmB,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,KAAK,GAAE,OAAe,GAAG,OAAO,CAAC,IAAI,CAAC;YA8pBjE,sBAAsB;YAwGtB,0BAA0B;YAyD1B,kBAAkB;CAuEjC"}

package/dist/services/address-task-executor.js

@@ -32,13 +32,16 @@ export class AddressTaskExecutor {
         }
         return this.openaiService;
     }
-    async executeAddressTasks(tasks) {
+    async executeAddressTasks(tasks, quiet = false) {
         try {
-            console.log(chalk.blue.bold('🚀 Starting address task workflow'));
-            console.log('');
+            if (!quiet) {
+                console.log(chalk.blue.bold('🚀 Starting address task workflow'));
+                console.log('');
+            }
             // Validate dependencies
             await this.getClaudeExecutor().validateClaudeCodeInstallation();
-            console.log(chalk.green('✅ Claude Code SDK configured'));
+            if (!quiet)
+                console.log(chalk.green('✅ Claude Code SDK configured'));
             // Get working directory from first task's job
             const db = this.jobManager['dbManager'].getKysely();
             const job = await db
@@ -53,9 +56,11 @@ export class AddressTaskExecutor {
             this.gitManager = createGitManager(this.workingDir);
             this.prService = createPRService(this.workingDir);
             this.gitManager.validateGitHubCliInstallation();
-            console.log(chalk.green('✅ GitHub CLI is installed'));
+            if (!quiet)
+                console.log(chalk.green('✅ GitHub CLI is installed'));
             this.gitManager.validateGitHubCliAuthentication();
-            console.log(chalk.green('✅ GitHub CLI is authenticated'));
+            if (!quiet)
+                console.log(chalk.green('✅ GitHub CLI is authenticated'));
             // Load repository instructions
             this.repoInstructions = await this.configManager.getRepoInstructions(this.workingDir);
             // Group tasks by branch
@@ -73,12 +78,15 @@ export class AddressTaskExecutor {
             // Execute tasks grouped by branch
             for (const [branch, branchTasks] of tasksByBranch) {
                 if (branch === 'unknown') {
-                    console.log(chalk.yellow('⚠️  Skipping tasks without branch information'));
+                    if (!quiet)
+                        console.log(chalk.yellow('⚠️  Skipping tasks without branch information'));
                     continue;
                 }
-                console.log('');
-                console.log(chalk.cyan.bold(`🔄 Creating worktree for branch: ${branch}`));
-                let spinner = ora('Creating worktree...').start();
+                if (!quiet) {
+                    console.log('');
+                    console.log(chalk.cyan.bold(`🔄 Creating worktree for branch: ${branch}`));
+                }
+                let spinner = quiet ? null : ora('Creating worktree...').start();
                 let worktreePath = null;
                 try {
                     // Create worktree for the branch
@@ -87,11 +95,14 @@ export class AddressTaskExecutor {
                     }
                     worktreePath = await this.gitManager.createWorktree(branch);
                     this.gitManager.switchToWorktree(worktreePath);
-                    spinner.succeed(`Worktree created: ${worktreePath}`);
+                    if (spinner)
+                        spinner.succeed(`Worktree created: ${worktreePath}`);
                 }
                 catch (error) {
-                    spinner.fail(`Failed to create worktree for branch: ${branch}`);
-                    console.error(error);
+                    if (spinner)
+                        spinner.fail(`Failed to create worktree for branch: ${branch}`);
+                    if (!quiet)
+                        console.error(error);
                     continue;
                 }
                 // Handle lint_and_test tasks separately
@@ -99,30 +110,38 @@ export class AddressTaskExecutor {
                 const addressTasks = branchTasks.filter(t => t.type === 'address');
                 // Process lint_and_test tasks first
                 for (const task of lintAndTestTasks) {
-                    console.log('');
-                    console.log(chalk.blue('🔧 Fixing test and lint failures'));
+                    if (!quiet) {
+                        console.log('');
+                        console.log(chalk.blue('🔧 Fixing test and lint failures'));
+                    }
                     await this.jobManager.updateTaskStatus(task.uuid, 'active');
                     // Extract PR number from task description
                     const prNumberMatch = task.description.match(/PR #(\d+)/);
                     const taskPrNumber = prNumberMatch ? parseInt(prNumberMatch[1]) : null;
-                    spinner = ora('Fetching GitHub Actions logs...').start();
+                    if (!quiet)
+                        spinner = ora('Fetching GitHub Actions logs...').start();
                     let actionLogs = '';
                     if (taskPrNumber && this.prService) {
                         try {
                             actionLogs = await this.prService.getFailingActionLogs(taskPrNumber);
-                            if (actionLogs) {
-                                spinner.succeed('GitHub Actions logs fetched');
-                            }
-                            else {
-                                spinner.info('No failing action logs found');
+                            if (spinner) {
+                                if (actionLogs) {
+                                    spinner.succeed('GitHub Actions logs fetched');
+                                }
+                                else {
+                                    spinner.info('No failing action logs found');
+                                }
                             }
                         }
                         catch (error) {
-                            spinner.warn('Could not fetch GitHub Actions logs');
-                            console.error(error);
+                            if (spinner)
+                                spinner.warn('Could not fetch GitHub Actions logs');
+                            if (!quiet)
+                                console.error(error);
                         }
                     }
-                    spinner = ora('Running Claude Code to fix test and lint failures...').start();
+                    if (!quiet)
+                        spinner = ora('Running Claude Code to fix test and lint failures...').start();
                     // Prepare the prompt for Claude
                     let prompt = 'Fix the failing tests and linting issues in this PR.\n\n';
                     prompt += 'The following GitHub Actions checks are failing:\n';
@@ -140,30 +159,35 @@ export class AddressTaskExecutor {
                     }
                     try {
                         const result = await this.getClaudeExecutor().executeTask(prompt, worktreePath || this.workingDir);
-                        spinner.succeed('Claude Code execution completed');
+                        if (spinner)
+                            spinner.succeed('Claude Code execution completed');
                         await this.jobManager.updateTaskExecutionLog(task.uuid, result.log);
                         if (!this.gitManager) {
                             throw new Error('GitManager not initialized');
                         }
                         const changedFiles = this.gitManager.getChangedFiles();
                         if (changedFiles.length === 0) {
-                            console.log(chalk.yellow('⚠️  No changes made'));
+                            if (!quiet)
+                                console.log(chalk.yellow('⚠️  No changes made'));
                             await this.jobManager.updateTaskStatus(task.uuid, 'completed');
                             continue;
                         }
                         // Create commit with co-author
-                        spinner = ora('Creating commit...').start();
+                        if (!quiet)
+                            spinner = ora('Creating commit...').start();
                         const commitMessage = 'Fix test and lint failures\n\nCo-authored-by: ivan-agent <ivan-agent@users.noreply.github.com>';
                         if (!this.gitManager) {
                             throw new Error('GitManager not initialized');
                         }
                         // Try to commit, handling pre-commit hook failures
-                        const commitResult = await this.tryCommitWithFixes(commitMessage, task, worktreePath || this.workingDir, spinner);
+                        const commitResult = await this.tryCommitWithFixes(commitMessage, task, worktreePath || this.workingDir, spinner, quiet);
                         if (commitResult.succeeded) {
-                            spinner.succeed('Changes committed');
+                            if (spinner)
+                                spinner.succeed('Changes committed');
                         }
                         else {
-                            spinner.fail('Failed to commit after multiple attempts');
+                            if (spinner)
+                                spinner.fail('Failed to commit after multiple attempts');
                             throw new Error('Pre-commit hook failures could not be fixed');
                         }
                         // Get the commit hash
@@ -174,21 +198,26 @@ export class AddressTaskExecutor {
                         // Save commit hash to task
                         await this.jobManager.updateTaskCommit(task.uuid, commitHash);
                         // Push the commit immediately
-                        spinner = ora('Pushing commit...').start();
+                        if (!quiet)
+                            spinner = ora('Pushing commit...').start();
                         try {
                             if (!this.gitManager) {
                                 throw new Error('GitManager not initialized');
                             }
                             await this.gitManager.pushBranch(branch);
-                            spinner.succeed('Commit pushed successfully');
+                            if (spinner)
+                                spinner.succeed('Commit pushed successfully');
                         }
                         catch (error) {
-                            spinner.fail('Failed to push commit');
-                            console.error(error);
+                            if (spinner)
+                                spinner.fail('Failed to push commit');
+                            if (!quiet)
+                                console.error(error);
                         }
                         // Add review comment for lint_and_test task
                         if (taskPrNumber) {
-                            spinner = ora('Adding review request comment...').start();
+                            if (!quiet)
+                                spinner = ora('Adding review request comment...').start();
                             try {
                                 const reviewAgent = this.configManager.getReviewAgent();
                                 const reviewComment = `${reviewAgent} please review the test and lint fixes that were applied to address the failing CI checks`;
@@ -196,18 +225,23 @@ export class AddressTaskExecutor {
                                     cwd: worktreePath || this.workingDir,
                                     stdio: 'pipe'
                                 });
-                                spinner.succeed('Review request comment added');
+                                if (spinner)
+                                    spinner.succeed('Review request comment added');
                             }
                             catch (error) {
-                                spinner.fail('Failed to add review comment');
-                                console.error(error);
+                                if (spinner)
+                                    spinner.fail('Failed to add review comment');
+                                if (!quiet)
+                                    console.error(error);
                             }
                         }
                         await this.jobManager.updateTaskStatus(task.uuid, 'completed');
                     }
                     catch (error) {
-                        spinner.fail('Failed to fix test and lint failures');
-                        console.error(error);
+                        if (spinner)
+                            spinner.fail('Failed to fix test and lint failures');
+                        if (!quiet)
+                            console.error(error);
                         const errorLog = error instanceof Error ? error.message : String(error);
                         await this.jobManager.updateTaskExecutionLog(task.uuid, `ERROR: ${errorLog}`);
                         await this.jobManager.updateTaskStatus(task.uuid, 'not_started');
@@ -225,18 +259,22 @@ export class AddressTaskExecutor {
                     continue;
                 }
                 // Get all unaddressed comments for this PR
-                spinner = ora('Fetching PR comments...').start();
+                if (!quiet)
+                    spinner = ora('Fetching PR comments...').start();
                 if (!prNumber) {
                     throw new Error('PR number not found');
                 }
                 const comments = await this.getUnaddressedComments(parseInt(prNumber));
-                spinner.succeed(`Found ${comments.length} unaddressed comments`);
+                if (spinner)
+                    spinner.succeed(`Found ${comments.length} unaddressed comments`);
                 if (comments.length === 0 && addressTasks.some(t => t.description.includes('comment'))) {
-                    console.log(chalk.yellow('⚠️  No unaddressed comments found'));
+                    if (!quiet)
+                        console.log(chalk.yellow('⚠️  No unaddressed comments found'));
                     continue;
                 }
                 // Process each comment
                 for (const comment of comments) {
+                    // In quiet mode, output the comment being addressed
                     console.log('');
                     console.log(chalk.blue(`📝 Addressing comment from @${comment.author}:`));
                     console.log(chalk.gray(`   "${comment.body.substring(0, 100)}${comment.body.length > 100 ? '...' : ''}"`));
@@ -244,7 +282,8 @@ export class AddressTaskExecutor {
                     const task = addressTasks.find(t => t.description.includes(comment.author) &&
                         t.description.includes(comment.body.substring(0, 50)));
                     if (!task) {
-                        console.log(chalk.yellow('⚠️  No task found for this comment'));
+                        if (!quiet)
+                            console.log(chalk.yellow('⚠️  No task found for this comment'));
                         continue;
                     }
                     await this.jobManager.updateTaskStatus(task.uuid, 'active');
@@ -261,7 +300,8 @@ export class AddressTaskExecutor {
                         const commentUrl = `https://github.com/${owner.login}/${repoName}/pull/${prNumber}#discussion_r${comment.id}`;
                         await this.jobManager.updateTaskCommentUrl(task.uuid, commentUrl);
                     }
-                    spinner = ora('Running Claude Code to address comment...').start();
+                    if (!quiet)
+                        spinner = ora('Running Claude Code to address comment...').start();
                     // Prepare the prompt for Claude
                     let prompt = 'Address the following PR review comment:\n\n';
                     prompt += `Comment from @${comment.author}:\n"${comment.body}"\n\n`;
@@ -278,7 +318,11 @@ export class AddressTaskExecutor {
                     }
                     try {
                         const result = await this.getClaudeExecutor().executeTask(prompt, worktreePath || this.workingDir);
-                        spinner.succeed('Claude Code execution completed');
+                        if (spinner) {
+                            spinner.succeed('Claude Code execution completed');
+                        }
+                        // Always output the final message from Claude (even in quiet mode)
+                        console.log(result.lastMessage);
                         await this.jobManager.updateTaskExecutionLog(task.uuid, result.log);
                         // Use the last message from Claude's response
                         const lastMessage = result.lastMessage;
@@ -287,9 +331,11 @@ export class AddressTaskExecutor {
                         }
                         const changedFiles = this.gitManager.getChangedFiles();
                         if (changedFiles.length === 0) {
-                            console.log(chalk.yellow('⚠️  No changes made - Claude determined no changes were needed'));
+                            if (!quiet)
+                                console.log(chalk.yellow('⚠️  No changes made - Claude determined no changes were needed'));
                             // Reply to the comment explaining why no changes were made
-                            spinner = ora('Replying to comment...').start();
+                            if (!quiet)
+                                spinner = ora('Replying to comment...').start();
                             try {
                                 // Truncate the message if it's too long (GitHub has a 65536 character limit)
                                 const maxLength = 60000;
@@ -364,21 +410,25 @@ export class AddressTaskExecutor {
                                         cwd: worktreePath || this.workingDir,
                                         stdio: 'pipe'
                                     });
-                                    spinner.succeed('Reply added to comment');
+                                    if (spinner)
+                                        spinner.succeed('Reply added to comment');
                                 }
                                 finally {
                                     unlinkSync(tempFile);
                                 }
                             }
                             catch (error) {
-                                spinner.fail('Failed to reply to comment');
-                                console.error(error);
+                                if (spinner)
+                                    spinner.fail('Failed to reply to comment');
+                                if (!quiet)
+                                    console.error(error);
                             }
                             await this.jobManager.updateTaskStatus(task.uuid, 'completed');
                             continue;
                         }
                         // Create commit with co-author
-                        spinner = ora('Creating commit...').start();
+                        if (!quiet)
+                            spinner = ora('Creating commit...').start();
                         const commitMessage = `Address review comment from @${comment.author}
 
 ${comment.body.substring(0, 200)}${comment.body.length > 200 ? '...' : ''}
@@ -388,12 +438,14 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                             throw new Error('GitManager not initialized');
                         }
                         // Try to commit, handling pre-commit hook failures
-                        const commitResult = await this.tryCommitWithFixes(commitMessage, task, worktreePath || this.workingDir, spinner);
+                        const commitResult = await this.tryCommitWithFixes(commitMessage, task, worktreePath || this.workingDir, spinner, quiet);
                         if (commitResult.succeeded) {
-                            spinner.succeed('Changes committed');
+                            if (spinner)
+                                spinner.succeed('Changes committed');
                         }
                         else {
-                            spinner.fail('Failed to commit after multiple attempts');
+                            if (spinner)
+                                spinner.fail('Failed to commit after multiple attempts');
                             throw new Error('Pre-commit hook failures could not be fixed');
                         }
                         // Get the commit hash
@@ -404,20 +456,25 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                         // Save commit hash to task
                         await this.jobManager.updateTaskCommit(task.uuid, commitHash);
                         // Push the commit immediately
-                        spinner = ora('Pushing commit...').start();
+                        if (!quiet)
+                            spinner = ora('Pushing commit...').start();
                         try {
                             if (!this.gitManager) {
                                 throw new Error('GitManager not initialized');
                             }
                             await this.gitManager.pushBranch(branch);
-                            spinner.succeed('Commit pushed successfully');
+                            if (spinner)
+                                spinner.succeed('Commit pushed successfully');
                         }
                         catch (error) {
-                            spinner.fail('Failed to push commit');
-                            console.error(error);
+                            if (spinner)
+                                spinner.fail('Failed to push commit');
+                            if (!quiet)
+                                console.error(error);
                         }
                         // Reply to the comment with the fix
-                        spinner = ora('Replying to comment...').start();
+                        if (!quiet)
+                            spinner = ora('Replying to comment...').start();
                         try {
                             // Truncate the message if it's too long (GitHub has a 65536 character limit)
                             const maxLength = 60000;
@@ -488,17 +545,22 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                                 cwd: worktreePath || this.workingDir,
                                 stdio: 'pipe'
                             });
-                            spinner.succeed('Reply added to comment');
+                            if (spinner)
+                                spinner.succeed('Reply added to comment');
                         }
                         catch (error) {
-                            spinner.fail('Failed to reply to comment');
-                            console.error(error);
+                            if (spinner)
+                                spinner.fail('Failed to reply to comment');
+                            if (!quiet)
+                                console.error(error);
                         }
                         await this.jobManager.updateTaskStatus(task.uuid, 'completed');
                     }
                     catch (error) {
-                        spinner.fail('Failed to address comment');
-                        console.error(error);
+                        if (spinner)
+                            spinner.fail('Failed to address comment');
+                        if (!quiet)
+                            console.error(error);
                         const errorLog = error instanceof Error ? error.message : String(error);
                         await this.jobManager.updateTaskExecutionLog(task.uuid, `ERROR: ${errorLog}`);
                         await this.jobManager.updateTaskStatus(task.uuid, 'not_started');
@@ -506,7 +568,8 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                 }
                 // Generate and add specific review comment (only if we have a PR number and made changes)
                 if (prNumber && (addressTasks.length > 0 || lintAndTestTasks.length > 0)) {
-                    spinner = ora('Generating review request...').start();
+                    if (!quiet)
+                        spinner = ora('Generating review request...').start();
                     try {
                         // Get the latest commit changes
                         const latestCommit = execSync('git rev-parse HEAD', {
@@ -523,20 +586,25 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                         }).trim().split('\n').filter(Boolean);
                         // Generate specific review instructions using OpenAI
                         const reviewInstructions = await this.generateReviewInstructions(commitDiff, changedFiles, parseInt(prNumber));
-                        spinner.succeed('Review request generated');
+                        if (spinner)
+                            spinner.succeed('Review request generated');
                         // Add the review comment
-                        spinner = ora('Adding review request comment...').start();
+                        if (!quiet)
+                            spinner = ora('Adding review request comment...').start();
                         const reviewAgent = this.configManager.getReviewAgent();
                         const reviewComment = `${reviewAgent} ${reviewInstructions}`;
                         execSync(`gh pr comment ${prNumber} --body "${reviewComment.replace(/"/g, '\\"')}"`, {
                             cwd: worktreePath || this.workingDir,
                             stdio: 'pipe'
                         });
-                        spinner.succeed('Review request comment added');
+                        if (spinner)
+                            spinner.succeed('Review request comment added');
                     }
                     catch (error) {
-                        spinner.fail('Failed to add review comment');
-                        console.error(error);
+                        if (spinner)
+                            spinner.fail('Failed to add review comment');
+                        if (!quiet)
+                            console.error(error);
                     }
                 }
                 // Clean up worktree after processing branch
@@ -546,15 +614,19 @@ Co-authored-by: ivan-agent <ivan-agent@users.noreply.github.com}`;
                         await this.gitManager.removeWorktree(branch);
                     }
                     catch (error) {
-                        console.log(chalk.yellow(`⚠️ Could not clean up worktree: ${error}`));
+                        if (!quiet)
+                            console.log(chalk.yellow(`⚠️ Could not clean up worktree: ${error}`));
                     }
                 }
             }
-            console.log('');
-            console.log(chalk.green.bold('🎉 All address tasks completed!'));
+            if (!quiet) {
+                console.log('');
+                console.log(chalk.green.bold('🎉 All address tasks completed!'));
+            }
         }
         catch (error) {
-            console.error(chalk.red.bold('❌ Address workflow failed:'), error);
+            if (!quiet)
+                console.error(chalk.red.bold('❌ Address workflow failed:'), error);
             throw error;
         }
     }
@@ -681,7 +753,7 @@ Return ONLY the review request text, without any prefix like "Please review" sin
             return 'please review the latest changes and verify all review comments have been properly addressed';
         }
     }
-    async tryCommitWithFixes(commitMessage, task, workingDir, spinner) {
+    async tryCommitWithFixes(commitMessage, task, workingDir, spinner, quiet = false) {
         let commitAttempts = 0;
         const maxCommitAttempts = 3;
         let commitSucceeded = false;
@@ -693,7 +765,7 @@ Return ONLY the review request text, without any prefix like "Please review" sin
                 await this.gitManager.commitChanges(commitMessage);
                 commitSucceeded = true;
                 // Stop the spinner if we're retrying
-                if (commitAttempts > 0 && spinner.isSpinning) {
+                if (commitAttempts > 0 && spinner && spinner.isSpinning) {
                     spinner.succeed('Commit successful after retry');
                 }
             }
@@ -702,26 +774,33 @@ Return ONLY the review request text, without any prefix like "Please review" sin
                 const errorMessage = commitError instanceof Error ? commitError.message : String(commitError);
                 // Check if this is a pre-commit hook failure
                 if (errorMessage.includes('pre-commit') && commitAttempts < maxCommitAttempts) {
-                    spinner.fail(`Pre-commit hook failed (attempt ${commitAttempts}/${maxCommitAttempts})`);
-                    console.log(chalk.yellow('🔧 Running Claude to fix pre-commit errors...'));
+                    if (spinner) {
+                        spinner.fail(`Pre-commit hook failed (attempt ${commitAttempts}/${maxCommitAttempts})`);
+                        console.log(chalk.yellow('🔧 Running Claude to fix pre-commit errors...'));
+                    }
                     // Extract the error details from the commit error
                     const errorDetails = errorMessage;
                     // Prepare prompt for Claude to fix the errors
                     const fixPrompt = `Fix the following pre-commit hook errors:\n\n${errorDetails}\n\nPlease fix all TypeScript errors, linting issues, and any other problems preventing the commit.`;
-                    spinner = ora('Running Claude to fix pre-commit errors...').start();
+                    if (!quiet)
+                        spinner = ora('Running Claude to fix pre-commit errors...').start();
                     try {
                         // Run Claude to fix the errors
                         const fixResult = await this.getClaudeExecutor().executeTask(fixPrompt, workingDir);
-                        spinner.succeed('Claude attempted to fix the errors');
+                        if (spinner)
+                            spinner.succeed('Claude attempted to fix the errors');
                         // Update the execution log with the fix attempt
                         const previousLog = await this.jobManager.getTaskExecutionLog(task.uuid);
                         await this.jobManager.updateTaskExecutionLog(task.uuid, `${previousLog}\n\n--- Pre-commit Fix Attempt ${commitAttempts} ---\n${fixResult.log}`);
                         // Try to commit again on the next iteration
-                        spinner = ora('Retrying commit...').start();
+                        if (!quiet)
+                            spinner = ora('Retrying commit...').start();
                     }
                     catch (fixError) {
-                        spinner.fail('Failed to run Claude to fix errors');
-                        console.error(chalk.red('Claude fix attempt failed:'), fixError);
+                        if (spinner)
+                            spinner.fail('Failed to run Claude to fix errors');
+                        if (!quiet)
+                            console.error(chalk.red('Claude fix attempt failed:'), fixError);
                         throw commitError; // Re-throw the original error
                     }
                 }