@eldrforge/kodrdriv 1.2.1 → 1.2.3

package/dist/commands/review.js

@@ -2,20 +2,122 @@
 import { Formatter } from '@riotprompt/riotprompt';
 import { ValidationError, FileOperationError, CommandError } from '../error/CommandErrors.js';
 import { getLogger } from '../logging.js';
-import { getModelForCommand, createCompletion } from '../util/openai.js';
+import { getModelForCommand, createCompletion, getOpenAIMaxOutputTokensForCommand, getOpenAIReasoningForCommand } from '../util/openai.js';
 import { createPrompt } from '../prompt/review.js';
-import { create } from '../content/log.js';
+import { create as create$1 } from '../content/log.js';
 import { getReviewExcludedPatterns, getRecentDiffsForReview } from '../content/diff.js';
 import { get } from '../content/releaseNotes.js';
-import { get as get$1, handleIssueCreation } from '../content/issues.js';
+import { handleIssueCreation, get as get$1 } from '../content/issues.js';
 import { DEFAULT_EXCLUDED_PATTERNS, DEFAULT_OUTPUT_DIRECTORY } from '../constants.js';
-import { getTimestampedReviewNotesFilename, getOutputPath, getTimestampedResponseFilename, getTimestampedRequestFilename, getTimestampedReviewFilename } from '../util/general.js';
-import { create as create$1 } from '../util/storage.js';
+import { getTimestampedReviewNotesFilename, getOutputPath, getTimestampedReviewFilename, getTimestampedResponseFilename, getTimestampedRequestFilename } from '../util/general.js';
+import { create } from '../util/storage.js';
+import { getUserChoice } from '../util/interactive.js';
 import path__default from 'path';
 import os__default from 'os';
 import { spawn } from 'child_process';
 import fs__default from 'fs/promises';
 
+// Utility function to read a review note from a file
+const readReviewNoteFromFile = async (filePath)=>{
+    const logger = getLogger();
+    try {
+        logger.debug(`Reading review note from file: ${filePath}`);
+        const content = await fs__default.readFile(filePath, 'utf8');
+        if (!content.trim()) {
+            throw new ValidationError(`Review file is empty: ${filePath}`);
+        }
+        logger.debug(`Successfully read review note from file: ${filePath} (${content.length} characters)`);
+        return content.trim();
+    } catch (error) {
+        if (error.code === 'ENOENT') {
+            throw new FileOperationError(`Review file not found: ${filePath}`, filePath, error);
+        }
+        if (error instanceof ValidationError) {
+            throw error;
+        }
+        throw new FileOperationError(`Failed to read review file: ${error.message}`, filePath, error);
+    }
+};
+// Utility function to get all review files in a directory
+const getReviewFilesInDirectory = async (directoryPath)=>{
+    const logger = getLogger();
+    try {
+        logger.debug(`Scanning directory for review files: ${directoryPath}`);
+        const entries = await fs__default.readdir(directoryPath, {
+            withFileTypes: true
+        });
+        // Filter for regular files (not directories) and get full paths
+        const files = entries.filter((entry)=>entry.isFile()).map((entry)=>path__default.join(directoryPath, entry.name)).sort(); // Sort alphabetically
+        logger.debug(`Found ${files.length} files in directory: ${directoryPath}`);
+        return files;
+    } catch (error) {
+        if (error.code === 'ENOENT') {
+            throw new FileOperationError(`Directory not found: ${directoryPath}`, directoryPath, error);
+        }
+        throw new FileOperationError(`Failed to read directory: ${directoryPath}`, directoryPath, error);
+    }
+};
+// New function for file selection phase
+const selectFilesForProcessing = async (reviewFiles, senditMode)=>{
+    const logger = getLogger();
+    if (senditMode) {
+        logger.info(`Auto-selecting all ${reviewFiles.length} files for processing (--sendit mode)`);
+        return reviewFiles;
+    }
+    // Check if we're in an interactive environment
+    if (!isTTYSafe()) {
+        logger.warn(`Non-interactive environment detected, selecting all files for processing`);
+        return reviewFiles;
+    }
+    logger.info(`\n📁 File Selection Phase`);
+    logger.info(`Found ${reviewFiles.length} files to review. Select which ones to process:`);
+    logger.info(`[c] Confirm this file for processing`);
+    logger.info(`[s] Skip this file`);
+    logger.info(`[a] Abort the entire review process`);
+    logger.info(``);
+    const selectedFiles = [];
+    let shouldAbort = false;
+    for(let i = 0; i < reviewFiles.length; i++){
+        const filePath = reviewFiles[i];
+        logger.info(`File ${i + 1}/${reviewFiles.length}: ${filePath}`);
+        const choice = await getUserChoice(`Select action for this file:`, [
+            {
+                key: 'c',
+                label: 'Confirm and process'
+            },
+            {
+                key: 's',
+                label: 'Skip this file'
+            },
+            {
+                key: 'a',
+                label: 'Abort entire review'
+            }
+        ]);
+        if (choice === 'a') {
+            logger.info(`🛑 Aborting review process as requested`);
+            shouldAbort = true;
+            break;
+        } else if (choice === 'c') {
+            selectedFiles.push(filePath);
+            logger.info(`✅ File selected for processing: ${filePath}`);
+        } else if (choice === 's') {
+            logger.info(`⏭️  File skipped: ${filePath}`);
+        }
+    }
+    if (shouldAbort) {
+        throw new Error('Review process aborted by user');
+    }
+    if (selectedFiles.length === 0) {
+        throw new Error('No files were selected for processing');
+    }
+    logger.info(`\n📋 File selection complete. ${selectedFiles.length} files selected for processing:`);
+    selectedFiles.forEach((file, index)=>{
+        logger.info(`  ${index + 1}. ${file}`);
+    });
+    logger.info(``);
+    return selectedFiles;
+};
 // Safe temp file handling with proper permissions and validation
 const createSecureTempFile = async ()=>{
     const logger = getLogger();
@@ -192,99 +294,10 @@ const safeWriteFile = async (filePath, content, encoding = 'utf-8')=>{
         throw new Error(`Failed to write file ${filePath}: ${error.message}`);
     }
 };
-const executeInternal = async (runConfig)=>{
-    var _runConfig_review, _runConfig_review1, _runConfig_review2, _runConfig_review3, _runConfig_review4, _runConfig_review5, _runConfig_review6, _runConfig_review7, _runConfig_review8, _runConfig_review9, _runConfig_review10, _runConfig_review11, _runConfig_review12, _runConfig_review13, _runConfig_review14, _runConfig_review_context, _runConfig_review15, _runConfig_review16, _analysisResult_issues, _runConfig_review17;
+// Helper function to process a single review note
+const processSingleReview = async (reviewNote, runConfig, outputDirectory)=>{
+    var _runConfig_review, _runConfig_review1, _runConfig_review2, _runConfig_review3, _runConfig_review_context, _runConfig_review4, _runConfig_review5, _analysisResult_issues;
     const logger = getLogger();
-    const isDryRun = runConfig.dryRun || false;
-    // Show configuration even in dry-run mode
-    logger.debug('Review context configuration:');
-    logger.debug('  Include commit history: %s', (_runConfig_review = runConfig.review) === null || _runConfig_review === void 0 ? void 0 : _runConfig_review.includeCommitHistory);
-    logger.debug('  Include recent diffs: %s', (_runConfig_review1 = runConfig.review) === null || _runConfig_review1 === void 0 ? void 0 : _runConfig_review1.includeRecentDiffs);
-    logger.debug('  Include release notes: %s', (_runConfig_review2 = runConfig.review) === null || _runConfig_review2 === void 0 ? void 0 : _runConfig_review2.includeReleaseNotes);
-    logger.debug('  Include GitHub issues: %s', (_runConfig_review3 = runConfig.review) === null || _runConfig_review3 === void 0 ? void 0 : _runConfig_review3.includeGithubIssues);
-    logger.debug('  Commit history limit: %d', (_runConfig_review4 = runConfig.review) === null || _runConfig_review4 === void 0 ? void 0 : _runConfig_review4.commitHistoryLimit);
-    logger.debug('  Diff history limit: %d', (_runConfig_review5 = runConfig.review) === null || _runConfig_review5 === void 0 ? void 0 : _runConfig_review5.diffHistoryLimit);
-    logger.debug('  Release notes limit: %d', (_runConfig_review6 = runConfig.review) === null || _runConfig_review6 === void 0 ? void 0 : _runConfig_review6.releaseNotesLimit);
-    logger.debug('  GitHub issues limit: %d', (_runConfig_review7 = runConfig.review) === null || _runConfig_review7 === void 0 ? void 0 : _runConfig_review7.githubIssuesLimit);
-    logger.debug('  Sendit mode (auto-create issues): %s', (_runConfig_review8 = runConfig.review) === null || _runConfig_review8 === void 0 ? void 0 : _runConfig_review8.sendit);
-    if (isDryRun) {
-        var _runConfig_review18, _runConfig_review19;
-        logger.info('DRY RUN: Would analyze provided note for review');
-        logger.info('DRY RUN: Would gather additional context based on configuration above');
-        logger.info('DRY RUN: Would analyze note and identify issues');
-        if ((_runConfig_review18 = runConfig.review) === null || _runConfig_review18 === void 0 ? void 0 : _runConfig_review18.sendit) {
-            logger.info('DRY RUN: Would automatically create GitHub issues (sendit mode enabled)');
-        } else {
-            logger.info('DRY RUN: Would prompt for confirmation before creating GitHub issues');
-        }
-        // Show what exclusion patterns would be used in dry-run mode
-        if ((_runConfig_review19 = runConfig.review) === null || _runConfig_review19 === void 0 ? void 0 : _runConfig_review19.includeRecentDiffs) {
-            var _runConfig_excludedPatterns;
-            const basePatterns = (_runConfig_excludedPatterns = runConfig.excludedPatterns) !== null && _runConfig_excludedPatterns !== void 0 ? _runConfig_excludedPatterns : DEFAULT_EXCLUDED_PATTERNS;
-            const reviewExcluded = getReviewExcludedPatterns(basePatterns);
-            logger.info('DRY RUN: Would use %d exclusion patterns for diff context', reviewExcluded.length);
-            logger.debug('DRY RUN: Sample exclusions: %s', reviewExcluded.slice(0, 15).join(', ') + (reviewExcluded.length > 15 ? '...' : ''));
-        }
-        return 'DRY RUN: Review command would analyze note, gather context, and create GitHub issues';
-    }
-    // Enhanced TTY check with proper error handling
-    const isInteractive = isTTYSafe();
-    if (!isInteractive && !((_runConfig_review9 = runConfig.review) === null || _runConfig_review9 === void 0 ? void 0 : _runConfig_review9.sendit)) {
-        logger.error('❌ STDIN is piped but --sendit flag is not enabled');
-        logger.error('   Interactive prompts cannot be used when input is piped');
-        logger.error('   Solutions:');
-        logger.error('   • Add --sendit flag to auto-create all issues');
-        logger.error('   • Use terminal input instead of piping');
-        logger.error('   • Example: echo "note" | kodrdriv review --sendit');
-        throw new ValidationError('Piped input requires --sendit flag for non-interactive operation');
-    }
-    // Get the review note from configuration
-    let reviewNote = (_runConfig_review10 = runConfig.review) === null || _runConfig_review10 === void 0 ? void 0 : _runConfig_review10.note;
-    // If no review note was provided via CLI arg or STDIN, open the user's editor to capture it.
-    if (!reviewNote || !reviewNote.trim()) {
-        const editor = process.env.EDITOR || process.env.VISUAL || 'vi';
-        let tmpFilePath = null;
-        try {
-            var _runConfig_review20;
-            // Create secure temporary file
-            tmpFilePath = await createSecureTempFile();
-            // Pre-populate the file with a helpful header so users know what to do.
-            const templateContent = [
-                '# Kodrdriv Review Note',
-                '',
-                '# Please enter your review note below. Lines starting with "#" will be ignored.',
-                '# Save and close the editor when you are done.',
-                '',
-                ''
-            ].join('\n');
-            await safeWriteFile(tmpFilePath, templateContent);
-            logger.info(`No review note provided – opening ${editor} to capture input...`);
-            // Open the editor with optional timeout protection
-            const editorTimeout = (_runConfig_review20 = runConfig.review) === null || _runConfig_review20 === void 0 ? void 0 : _runConfig_review20.editorTimeout; // No default timeout - let user take their time
-            await openEditorWithTimeout(editor, tmpFilePath, editorTimeout);
-            // Read the file back in, stripping comment lines and whitespace.
-            const fileContent = (await fs__default.readFile(tmpFilePath, 'utf8')).split('\n').filter((line)=>!line.trim().startsWith('#')).join('\n').trim();
-            if (!fileContent) {
-                throw new ValidationError('Review note is empty – aborting. Provide a note as an argument, via STDIN, or through the editor.');
-            }
-            reviewNote = fileContent;
-            // If the original runConfig.review object exists, update it so downstream code has the note.
-            if (runConfig.review) {
-                runConfig.review.note = reviewNote;
-            }
-        } catch (error) {
-            logger.error(`Failed to capture review note via editor: ${error.message}`);
-            throw error;
-        } finally{
-            // Always clean up the temp file
-            if (tmpFilePath) {
-                await cleanupTempFile(tmpFilePath);
-            }
-        }
-    }
-    logger.info('📝 Starting review analysis...');
-    logger.debug('Review note: %s', reviewNote);
-    logger.debug('Review note length: %d characters', reviewNote.length);
     // Gather additional context based on configuration with improved error handling
     let logContext = '';
     let diffContext = '';
@@ -292,10 +305,10 @@ const executeInternal = async (runConfig)=>{
     let issuesContext = '';
     const contextErrors = [];
     // Fetch commit history if enabled
-    if ((_runConfig_review11 = runConfig.review) === null || _runConfig_review11 === void 0 ? void 0 : _runConfig_review11.includeCommitHistory) {
+    if ((_runConfig_review = runConfig.review) === null || _runConfig_review === void 0 ? void 0 : _runConfig_review.includeCommitHistory) {
         try {
             logger.debug('Fetching recent commit history...');
-            const log = await create({
+            const log = await create$1({
                 limit: runConfig.review.commitHistoryLimit
             });
             const logContent = await log.get();
@@ -310,11 +323,11 @@ const executeInternal = async (runConfig)=>{
         }
     }
     // Fetch recent diffs if enabled
-    if ((_runConfig_review12 = runConfig.review) === null || _runConfig_review12 === void 0 ? void 0 : _runConfig_review12.includeRecentDiffs) {
+    if ((_runConfig_review1 = runConfig.review) === null || _runConfig_review1 === void 0 ? void 0 : _runConfig_review1.includeRecentDiffs) {
         try {
             logger.debug('Fetching recent commit diffs...');
-            var _runConfig_excludedPatterns1;
-            const basePatterns = (_runConfig_excludedPatterns1 = runConfig.excludedPatterns) !== null && _runConfig_excludedPatterns1 !== void 0 ? _runConfig_excludedPatterns1 : DEFAULT_EXCLUDED_PATTERNS;
+            var _runConfig_excludedPatterns;
+            const basePatterns = (_runConfig_excludedPatterns = runConfig.excludedPatterns) !== null && _runConfig_excludedPatterns !== void 0 ? _runConfig_excludedPatterns : DEFAULT_EXCLUDED_PATTERNS;
             const recentDiffs = await getRecentDiffsForReview({
                 limit: runConfig.review.diffHistoryLimit,
                 baseExcludedPatterns: basePatterns
@@ -330,7 +343,7 @@ const executeInternal = async (runConfig)=>{
         }
     }
     // Fetch release notes if enabled
-    if ((_runConfig_review13 = runConfig.review) === null || _runConfig_review13 === void 0 ? void 0 : _runConfig_review13.includeReleaseNotes) {
+    if ((_runConfig_review2 = runConfig.review) === null || _runConfig_review2 === void 0 ? void 0 : _runConfig_review2.includeReleaseNotes) {
         try {
             logger.debug('Fetching recent release notes from GitHub...');
             const releaseNotesContent = await get({
@@ -347,7 +360,7 @@ const executeInternal = async (runConfig)=>{
         }
     }
     // Fetch GitHub issues if enabled
-    if ((_runConfig_review14 = runConfig.review) === null || _runConfig_review14 === void 0 ? void 0 : _runConfig_review14.includeGithubIssues) {
+    if ((_runConfig_review3 = runConfig.review) === null || _runConfig_review3 === void 0 ? void 0 : _runConfig_review3.includeGithubIssues) {
         try {
             logger.debug('Fetching open GitHub issues...');
             issuesContext = await get$1({
@@ -364,11 +377,11 @@ const executeInternal = async (runConfig)=>{
     }
     // Report context gathering results
     if (contextErrors.length > 0) {
-        var _runConfig_review21;
+        var _runConfig_review6;
         logger.warn(`Context gathering completed with ${contextErrors.length} error(s):`);
         contextErrors.forEach((error)=>logger.warn(`  - ${error}`));
         // For critical operations, consider failing if too many context sources fail
-        const maxContextErrors = ((_runConfig_review21 = runConfig.review) === null || _runConfig_review21 === void 0 ? void 0 : _runConfig_review21.maxContextErrors) || contextErrors.length; // Default: allow all errors
+        const maxContextErrors = ((_runConfig_review6 = runConfig.review) === null || _runConfig_review6 === void 0 ? void 0 : _runConfig_review6.maxContextErrors) || contextErrors.length; // Default: allow all errors
         if (contextErrors.length > maxContextErrors) {
             throw new Error(`Too many context gathering errors (${contextErrors.length}), aborting review. Consider checking your configuration and network connectivity.`);
         }
@@ -381,7 +394,7 @@ const executeInternal = async (runConfig)=>{
     logger.debug('  - Diff context: %d chars', diffContext.length);
     logger.debug('  - Release notes context: %d chars', releaseNotesContext.length);
     logger.debug('  - Issues context: %d chars', issuesContext.length);
-    logger.debug('  - User context: %d chars', ((_runConfig_review15 = runConfig.review) === null || _runConfig_review15 === void 0 ? void 0 : (_runConfig_review_context = _runConfig_review15.context) === null || _runConfig_review_context === void 0 ? void 0 : _runConfig_review_context.length) || 0);
+    logger.debug('  - User context: %d chars', ((_runConfig_review4 = runConfig.review) === null || _runConfig_review4 === void 0 ? void 0 : (_runConfig_review_context = _runConfig_review4.context) === null || _runConfig_review_context === void 0 ? void 0 : _runConfig_review_context.length) || 0);
     const promptConfig = {
         overridePaths: runConfig.discoveredConfigDirs || [],
         overrides: runConfig.overrides || false
@@ -390,47 +403,13 @@ const executeInternal = async (runConfig)=>{
         notes: reviewNote
     };
     const promptContext = {
-        context: (_runConfig_review16 = runConfig.review) === null || _runConfig_review16 === void 0 ? void 0 : _runConfig_review16.context,
+        context: (_runConfig_review5 = runConfig.review) === null || _runConfig_review5 === void 0 ? void 0 : _runConfig_review5.context,
         logContext,
         diffContext,
         releaseNotesContext,
         issuesContext
     };
     const prompt = await createPrompt(promptConfig, promptContent, promptContext);
-    const outputDirectory = runConfig.outputDirectory || DEFAULT_OUTPUT_DIRECTORY;
-    const storage = create$1({
-        log: logger.info
-    });
-    await storage.ensureDirectory(outputDirectory);
-    // Save timestamped copy of review notes and context to output directory
-    try {
-        var _runConfig_review_context1, _runConfig_review22;
-        // Save the original review note
-        const reviewNotesFilename = getTimestampedReviewNotesFilename();
-        const reviewNotesPath = getOutputPath(outputDirectory, reviewNotesFilename);
-        let reviewNotesContent = `# Review Notes\n\n${reviewNote}\n\n`;
-        // Add all context sections if they exist
-        if (logContext.trim()) {
-            reviewNotesContent += `# Commit History Context\n\n${logContext}\n\n`;
-        }
-        if (diffContext.trim()) {
-            reviewNotesContent += `# Recent Diffs Context\n\n${diffContext}\n\n`;
-        }
-        if (releaseNotesContext.trim()) {
-            reviewNotesContent += `# Release Notes Context\n\n${releaseNotesContext}\n\n`;
-        }
-        if (issuesContext.trim()) {
-            reviewNotesContent += `# GitHub Issues Context\n\n${issuesContext}\n\n`;
-        }
-        if ((_runConfig_review22 = runConfig.review) === null || _runConfig_review22 === void 0 ? void 0 : (_runConfig_review_context1 = _runConfig_review22.context) === null || _runConfig_review_context1 === void 0 ? void 0 : _runConfig_review_context1.trim()) {
-            reviewNotesContent += `# User Context\n\n${runConfig.review.context}\n\n`;
-        }
-        await safeWriteFile(reviewNotesPath, reviewNotesContent);
-        logger.debug('Saved timestamped review notes and context: %s', reviewNotesPath);
-    } catch (error) {
-        logger.warn('Failed to save timestamped review notes: %s', error.message);
-    // Don't fail the entire operation for this
-    }
     const modelToUse = getModelForCommand(runConfig, 'review');
     const request = Formatter.create({
         logger
@@ -439,6 +418,8 @@ const executeInternal = async (runConfig)=>{
     try {
         const rawResult = await createCompletion(request.messages, {
             model: modelToUse,
+            openaiReasoning: getOpenAIReasoningForCommand(runConfig, 'review'),
+            openaiMaxOutputTokens: getOpenAIMaxOutputTokensForCommand(runConfig, 'review'),
             responseFormat: {
                 type: 'json_object'
             },
@@ -473,8 +454,238 @@ const executeInternal = async (runConfig)=>{
         logger.warn('Failed to save timestamped review analysis: %s', error.message);
     // Don't fail the entire operation for this
     }
+    return analysisResult;
+};
+const executeInternal = async (runConfig)=>{
+    var _runConfig_review, _runConfig_review1, _runConfig_review2, _runConfig_review3, _runConfig_review4, _runConfig_review5, _runConfig_review6, _runConfig_review7, _runConfig_review8, _runConfig_review9, _runConfig_review10, _runConfig_review11, _runConfig_review12, _runConfig_review13, _runConfig_review14, _runConfig_review15, _runConfig_review16, _runConfig_review17, _runConfig_review18;
+    const logger = getLogger();
+    const isDryRun = runConfig.dryRun || false;
+    // Show configuration even in dry-run mode
+    logger.debug('Review context configuration:');
+    logger.debug('  Include commit history: %s', (_runConfig_review = runConfig.review) === null || _runConfig_review === void 0 ? void 0 : _runConfig_review.includeCommitHistory);
+    logger.debug('  Include recent diffs: %s', (_runConfig_review1 = runConfig.review) === null || _runConfig_review1 === void 0 ? void 0 : _runConfig_review1.includeRecentDiffs);
+    logger.debug('  Include release notes: %s', (_runConfig_review2 = runConfig.review) === null || _runConfig_review2 === void 0 ? void 0 : _runConfig_review2.includeReleaseNotes);
+    logger.debug('  Include GitHub issues: %s', (_runConfig_review3 = runConfig.review) === null || _runConfig_review3 === void 0 ? void 0 : _runConfig_review3.includeGithubIssues);
+    logger.debug('  Commit history limit: %d', (_runConfig_review4 = runConfig.review) === null || _runConfig_review4 === void 0 ? void 0 : _runConfig_review4.commitHistoryLimit);
+    logger.debug('  Diff history limit: %d', (_runConfig_review5 = runConfig.review) === null || _runConfig_review5 === void 0 ? void 0 : _runConfig_review5.diffHistoryLimit);
+    logger.debug('  Release notes limit: %d', (_runConfig_review6 = runConfig.review) === null || _runConfig_review6 === void 0 ? void 0 : _runConfig_review6.releaseNotesLimit);
+    logger.debug('  GitHub issues limit: %d', (_runConfig_review7 = runConfig.review) === null || _runConfig_review7 === void 0 ? void 0 : _runConfig_review7.githubIssuesLimit);
+    logger.debug('  Sendit mode (auto-create issues): %s', (_runConfig_review8 = runConfig.review) === null || _runConfig_review8 === void 0 ? void 0 : _runConfig_review8.sendit);
+    logger.debug('  File: %s', ((_runConfig_review9 = runConfig.review) === null || _runConfig_review9 === void 0 ? void 0 : _runConfig_review9.file) || 'not specified');
+    logger.debug('  Directory: %s', ((_runConfig_review10 = runConfig.review) === null || _runConfig_review10 === void 0 ? void 0 : _runConfig_review10.directory) || 'not specified');
+    if (isDryRun) {
+        var _runConfig_review19, _runConfig_review20, _runConfig_review21, _runConfig_review22, _runConfig_review23;
+        if ((_runConfig_review19 = runConfig.review) === null || _runConfig_review19 === void 0 ? void 0 : _runConfig_review19.file) {
+            logger.info('DRY RUN: Would read review note from file: %s', runConfig.review.file);
+        } else if ((_runConfig_review20 = runConfig.review) === null || _runConfig_review20 === void 0 ? void 0 : _runConfig_review20.directory) {
+            logger.info('DRY RUN: Would process review files in directory: %s', runConfig.review.directory);
+            logger.info('DRY RUN: Would first select which files to process, then analyze selected files');
+        } else if ((_runConfig_review21 = runConfig.review) === null || _runConfig_review21 === void 0 ? void 0 : _runConfig_review21.note) {
+            logger.info('DRY RUN: Would analyze provided note for review');
+        } else {
+            logger.info('DRY RUN: Would open editor to capture review note');
+        }
+        logger.info('DRY RUN: Would gather additional context based on configuration above');
+        logger.info('DRY RUN: Would analyze note and identify issues');
+        if ((_runConfig_review22 = runConfig.review) === null || _runConfig_review22 === void 0 ? void 0 : _runConfig_review22.sendit) {
+            logger.info('DRY RUN: Would automatically create GitHub issues (sendit mode enabled)');
+        } else {
+            logger.info('DRY RUN: Would prompt for confirmation before creating GitHub issues');
+        }
+        // Show what exclusion patterns would be used in dry-run mode
+        if ((_runConfig_review23 = runConfig.review) === null || _runConfig_review23 === void 0 ? void 0 : _runConfig_review23.includeRecentDiffs) {
+            var _runConfig_excludedPatterns;
+            const basePatterns = (_runConfig_excludedPatterns = runConfig.excludedPatterns) !== null && _runConfig_excludedPatterns !== void 0 ? _runConfig_excludedPatterns : DEFAULT_EXCLUDED_PATTERNS;
+            const reviewExcluded = getReviewExcludedPatterns(basePatterns);
+            logger.info('DRY RUN: Would use %d exclusion patterns for diff context', reviewExcluded.length);
+            logger.debug('DRY RUN: Sample exclusions: %s', reviewExcluded.slice(0, 15).join(', ') + (reviewExcluded.length > 15 ? '...' : ''));
+        }
+        return 'DRY RUN: Review command would analyze note, gather context, and create GitHub issues';
+    }
+    // Enhanced TTY check with proper error handling
+    const isInteractive = isTTYSafe();
+    if (!isInteractive && !((_runConfig_review11 = runConfig.review) === null || _runConfig_review11 === void 0 ? void 0 : _runConfig_review11.sendit)) {
+        logger.error('❌ STDIN is piped but --sendit flag is not enabled');
+        logger.error('   Interactive prompts cannot be used when input is piped');
+        logger.error('   Solutions:');
+        logger.error('   • Add --sendit flag to auto-create all issues');
+        logger.error('   • Use terminal input instead of piping');
+        logger.error('   • Example: echo "note" | kodrdriv review --sendit');
+        throw new ValidationError('Piped input requires --sendit flag for non-interactive operation');
+    }
+    // Get the review note from configuration
+    let reviewNote = (_runConfig_review12 = runConfig.review) === null || _runConfig_review12 === void 0 ? void 0 : _runConfig_review12.note;
+    let reviewFiles = [];
+    // Check if we should process a single file
+    if ((_runConfig_review13 = runConfig.review) === null || _runConfig_review13 === void 0 ? void 0 : _runConfig_review13.file) {
+        logger.info(`📁 Reading review note from file: ${runConfig.review.file}`);
+        reviewNote = await readReviewNoteFromFile(runConfig.review.file);
+        reviewFiles = [
+            runConfig.review.file
+        ];
+    } else if ((_runConfig_review14 = runConfig.review) === null || _runConfig_review14 === void 0 ? void 0 : _runConfig_review14.directory) {
+        var _runConfig_review24;
+        logger.info(`📁 Processing review files in directory: ${runConfig.review.directory}`);
+        reviewFiles = await getReviewFilesInDirectory(runConfig.review.directory);
+        if (reviewFiles.length === 0) {
+            throw new ValidationError(`No review files found in directory: ${runConfig.review.directory}`);
+        }
+        logger.info(`📁 Found ${reviewFiles.length} files to process`);
+        // Set a dummy reviewNote for directory mode to satisfy validation
+        // The actual review notes will be read from each file during processing
+        reviewNote = `Processing ${reviewFiles.length} files from directory`;
+        // If not in sendit mode, explain the two-phase process
+        if (!((_runConfig_review24 = runConfig.review) === null || _runConfig_review24 === void 0 ? void 0 : _runConfig_review24.sendit)) {
+            logger.info(`📝 Interactive mode: You will first select which files to process, then they will be analyzed in order.`);
+            logger.info(`📝 Use --sendit to process all files automatically without confirmation.`);
+        }
+    } else if ((_runConfig_review15 = runConfig.review) === null || _runConfig_review15 === void 0 ? void 0 : _runConfig_review15.note) {
+        reviewNote = runConfig.review.note;
+        reviewFiles = [
+            'provided note'
+        ];
+    } else {
+        // Open editor to capture review note
+        const editor = process.env.EDITOR || process.env.VISUAL || 'vi';
+        let tmpFilePath = null;
+        try {
+            var _runConfig_review25;
+            // Create secure temporary file
+            tmpFilePath = await createSecureTempFile();
+            // Pre-populate the file with a helpful header so users know what to do.
+            const templateContent = [
+                '# Kodrdriv Review Note',
+                '',
+                '# Please enter your review note below. Lines starting with "#" will be ignored.',
+                '# Save and close the editor when you are done.',
+                '',
+                ''
+            ].join('\n');
+            await safeWriteFile(tmpFilePath, templateContent);
+            logger.info(`No review note provided – opening ${editor} to capture input...`);
+            // Open the editor with optional timeout protection
+            const editorTimeout = (_runConfig_review25 = runConfig.review) === null || _runConfig_review25 === void 0 ? void 0 : _runConfig_review25.editorTimeout; // No default timeout - let user take their time
+            await openEditorWithTimeout(editor, tmpFilePath, editorTimeout);
+            // Read the file back in, stripping comment lines and whitespace.
+            const fileContent = (await fs__default.readFile(tmpFilePath, 'utf8')).split('\n').filter((line)=>!line.trim().startsWith('#')).join('\n').trim();
+            if (!fileContent) {
+                throw new ValidationError('Review note is empty – aborting. Provide a note as an argument, via STDIN, or through the editor.');
+            }
+            reviewNote = fileContent;
+            // If the original runConfig.review object exists, update it so downstream code has the note.
+            if (runConfig.review) {
+                runConfig.review.note = reviewNote;
+            }
+        } catch (error) {
+            logger.error(`Failed to capture review note via editor: ${error.message}`);
+            throw error;
+        } finally{
+            // Always clean up the temp file
+            if (tmpFilePath) {
+                await cleanupTempFile(tmpFilePath);
+            }
+        }
+        reviewFiles = [
+            'editor input'
+        ];
+    }
+    if (!reviewNote || !reviewNote.trim()) {
+        throw new ValidationError('No review note provided or captured');
+    }
+    logger.info('📝 Starting review analysis...');
+    logger.debug('Review note: %s', reviewNote);
+    logger.debug('Review note length: %d characters', reviewNote.length);
+    const outputDirectory = runConfig.outputDirectory || DEFAULT_OUTPUT_DIRECTORY;
+    const storage = create({
+        log: logger.info
+    });
+    await storage.ensureDirectory(outputDirectory);
+    // Save timestamped copy of review notes to output directory
+    try {
+        const reviewNotesFilename = getTimestampedReviewNotesFilename();
+        const reviewNotesPath = getOutputPath(outputDirectory, reviewNotesFilename);
+        const reviewNotesContent = `# Review Notes\n\n${reviewNote}\n\n`;
+        await safeWriteFile(reviewNotesPath, reviewNotesContent);
+        logger.debug('Saved timestamped review notes: %s', reviewNotesPath);
+    } catch (error) {
+        logger.warn('Failed to save review notes: %s', error.message);
+    }
+    // Phase 1: File selection (only for directory mode)
+    let selectedFiles;
+    if ((_runConfig_review16 = runConfig.review) === null || _runConfig_review16 === void 0 ? void 0 : _runConfig_review16.directory) {
+        var _runConfig_review26;
+        selectedFiles = await selectFilesForProcessing(reviewFiles, ((_runConfig_review26 = runConfig.review) === null || _runConfig_review26 === void 0 ? void 0 : _runConfig_review26.sendit) || false);
+    } else {
+        // For single note mode, just use the note directly
+        selectedFiles = [
+            'single note'
+        ];
+    }
+    // Phase 2: Process selected files in order
+    logger.info(`\n📝 Starting analysis phase...`);
+    const results = [];
+    const processedFiles = [];
+    if ((_runConfig_review17 = runConfig.review) === null || _runConfig_review17 === void 0 ? void 0 : _runConfig_review17.directory) {
+        // Directory mode: process each selected file
+        for(let i = 0; i < selectedFiles.length; i++){
+            const filePath = selectedFiles[i];
+            try {
+                logger.info(`📝 Processing file ${i + 1}/${selectedFiles.length}: ${filePath}`);
+                const fileNote = await readReviewNoteFromFile(filePath);
+                const fileResult = await processSingleReview(fileNote, runConfig, outputDirectory);
+                results.push(fileResult);
+                processedFiles.push(filePath);
+            } catch (error) {
+                // Check if this is a critical error that should be propagated
+                if (error.message.includes('Too many context gathering errors')) {
+                    throw error; // Propagate critical context errors
+                }
+                logger.warn(`Failed to process file ${filePath}: ${error.message}`);
+            // Continue with other files for non-critical errors
+            }
+        }
+    } else {
+        // Single note mode: process the note directly
+        try {
+            logger.info(`📝 Processing single review note`);
+            const fileResult = await processSingleReview(reviewNote, runConfig, outputDirectory);
+            results.push(fileResult);
+            processedFiles.push('single note');
+        } catch (error) {
+            logger.warn(`Failed to process review note: ${error.message}`);
+            throw error; // Re-throw for single note mode since there's only one item
+        }
+    }
+    if (results.length === 0) {
+        throw new ValidationError('No files were processed successfully');
+    }
+    // Combine results if we processed multiple files
+    let analysisResult;
+    if (results.length === 1) {
+        analysisResult = results[0];
+    } else {
+        logger.info(`✅ Successfully processed ${results.length} review files`);
+        // Create a combined summary
+        const totalIssues = results.reduce((sum, result)=>sum + result.totalIssues, 0);
+        const allIssues = results.flatMap((result)=>result.issues || []);
+        analysisResult = {
+            summary: `Combined analysis of ${results.length} review files. Total issues found: ${totalIssues}`,
+            totalIssues,
+            issues: allIssues
+        };
+        // Save combined results
+        try {
+            const combinedFilename = getTimestampedReviewFilename();
+            const combinedPath = getOutputPath(outputDirectory, combinedFilename);
+            const combinedContent = `# Combined Review Analysis Result\n\n` + `## Summary\n${analysisResult.summary}\n\n` + `## Total Issues Found\n${totalIssues}\n\n` + `## Files Processed\n${processedFiles.join('\n')}\n\n` + `## Issues\n\n${JSON.stringify(allIssues, null, 2)}\n\n` + `---\n\n*Combined analysis completed at ${new Date().toISOString()}*`;
+            await safeWriteFile(combinedPath, combinedContent);
+            logger.debug('Saved combined review analysis: %s', combinedPath);
+        } catch (error) {
+            logger.warn('Failed to save combined review analysis: %s', error.message);
+        }
+    }
     // Handle GitHub issue creation using the issues module
-    const senditMode = ((_runConfig_review17 = runConfig.review) === null || _runConfig_review17 === void 0 ? void 0 : _runConfig_review17.sendit) || false;
+    const senditMode = ((_runConfig_review18 = runConfig.review) === null || _runConfig_review18 === void 0 ? void 0 : _runConfig_review18.sendit) || false;
     return await handleIssueCreation(analysisResult, senditMode);
 };
 const execute = async (runConfig)=>{
@@ -488,14 +699,14 @@ const execute = async (runConfig)=>{
         }
         if (error instanceof FileOperationError) {
             logger.error(`review failed: ${error.message}`);
-            if (error.cause) {
+            if (error.cause && typeof error.cause === 'object' && 'message' in error.cause) {
                 logger.debug(`Caused by: ${error.cause.message}`);
             }
             throw error;
         }
         if (error instanceof CommandError) {
             logger.error(`review failed: ${error.message}`);
-            if (error.cause) {
+            if (error.cause && typeof error.cause === 'object' && 'message' in error.cause) {
                 logger.debug(`Caused by: ${error.cause.message}`);
             }
             throw error;