@eldrforge/kodrdriv 1.2.26 → 1.2.28

package/dist/commands/review.js

@@ -61,25 +61,23 @@ const getReviewFilesInDirectory = async (directoryPath)=>{
 const selectFilesForProcessing = async (reviewFiles, senditMode)=>{
     const logger = getLogger();
     if (senditMode) {
-        logger.info(`Auto-selecting all ${reviewFiles.length} files for processing (--sendit mode)`);
+        logger.info(`REVIEW_AUTO_SELECT: Auto-selecting all files for processing | Mode: sendit | File Count: ${reviewFiles.length} | Confirmation: automatic`);
         return reviewFiles;
     }
     // Check if we're in an interactive environment
     if (!isTTYSafe()) {
-        logger.warn(`Non-interactive environment detected, selecting all files for processing`);
+        logger.warn(`REVIEW_NON_INTERACTIVE_SELECT: Non-interactive environment detected | Action: Selecting all files | Mode: non-interactive`);
         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(`\nREVIEW_SELECTION_PHASE: Starting file selection phase | File Count: ${reviewFiles.length} | Purpose: Choose files to process`);
+    logger.info(`REVIEW_SELECTION_FILES: Found files to review | Count: ${reviewFiles.length} | Action: Select files for processing`);
+    logger.info(`REVIEW_SELECTION_OPTIONS: File selection options available | [c]=Confirm | [s]=Skip | [a]=Abort`);
     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}`);
+        logger.info(`REVIEW_SELECTION_FILE: File for review | Progress: ${i + 1}/${reviewFiles.length} | File: ${filePath}`);
         const choice = await getUserChoice(`Select action for this file:`, [
             {
                 key: 'c',
@@ -95,14 +93,14 @@ const selectFilesForProcessing = async (reviewFiles, senditMode)=>{
             }
         ]);
         if (choice === 'a') {
-            logger.info(`🛑 Aborting review process as requested`);
+            logger.info(`REVIEW_ABORTED: User aborted review process | Action: Aborting | Reason: User request`);
             shouldAbort = true;
             break;
         } else if (choice === 'c') {
             selectedFiles.push(filePath);
-            logger.info(`✅ File selected for processing: ${filePath}`);
+            logger.info(`REVIEW_FILE_SELECTED: File selected for processing | File: ${filePath} | Action: Will be processed`);
         } else if (choice === 's') {
-            logger.info(`⏭️  File skipped: ${filePath}`);
+            logger.info(`REVIEW_FILE_SKIPPED: File skipped during selection | File: ${filePath} | Action: Will not be processed`);
         }
     }
     if (shouldAbort) {
@@ -128,7 +126,7 @@ const createSecureTempFile = async ()=>{
         const W_OK = 2; // fs.constants.W_OK value
         await fs.access(tmpDir, W_OK);
     } catch (error) {
-        logger.error(`Temp directory not writable: ${tmpDir}`);
+        logger.error(`TEMP_DIR_NOT_WRITABLE: Temporary directory is not writable | Directory: ${tmpDir} | Impact: Cannot create temp files`);
         throw new FileOperationError(`Temp directory not writable: ${error.message}`, tmpDir, error);
     }
     const tmpFilePath = path__default.join(tmpDir, `kodrdriv_review_${Date.now()}_${Math.random().toString(36).substring(7)}.md`);
@@ -139,7 +137,7 @@ const createSecureTempFile = async ()=>{
         logger.debug(`Created secure temp file: ${tmpFilePath}`);
         return tmpFilePath;
     } catch (error) {
-        logger.error(`Failed to create temp file: ${error.message}`);
+        logger.error(`TEMP_FILE_CREATE_FAILED: Unable to create temporary file | Error: ${error.message} | Impact: Cannot proceed with review`);
         throw new FileOperationError(`Failed to create temp file: ${error.message}`, 'temporary file', error);
     }
 };
@@ -152,7 +150,7 @@ const cleanupTempFile = async (filePath)=>{
     } catch (error) {
         // Only ignore ENOENT (file not found) errors, log others
         if (error.code !== 'ENOENT') {
-            logger.warn(`Failed to cleanup temp file ${filePath}: ${error.message}`);
+            logger.warn(`TEMP_FILE_CLEANUP_FAILED: Unable to cleanup temporary file | File: ${filePath} | Error: ${error.message} | Impact: File may remain`);
         // Don't throw here to avoid masking the main operation
         }
     }
@@ -387,7 +385,7 @@ const processSingleReview = async (reviewNote, runConfig, outputDirectory)=>{
         }
     }
     // Analyze review note for issues using OpenAI
-    logger.info('🤖 Analyzing review note for project issues...');
+    logger.info('REVIEW_ANALYSIS_STARTING: Analyzing review note for project issues | Source: review note | Purpose: Identify actionable issues');
     logger.debug('Context summary:');
     logger.debug('  - Review note: %d chars', reviewNote.length);
     logger.debug('  - Log context: %d chars', logContext.length);
@@ -436,10 +434,10 @@ const processSingleReview = async (reviewNote, runConfig, outputDirectory)=>{
         // Validate the API response before using it
         analysisResult = validateReviewResult(rawResult);
     } catch (error) {
-        logger.error(`Failed to analyze review note: ${error.message}`);
+        logger.error(`REVIEW_ANALYSIS_FAILED: Unable to analyze review note | Error: ${error.message} | Impact: Cannot identify issues`);
         throw new Error(`Review analysis failed: ${error.message}`);
     }
-    logger.info('✅ Analysis completed');
+    logger.info('REVIEW_ANALYSIS_COMPLETE: Review note analysis completed successfully | Status: completed | Next: Issue creation if enabled');
     logger.debug('Analysis result summary: %s', analysisResult.summary);
     logger.debug('Total issues found: %d', analysisResult.totalIssues);
     logger.debug('Issues array length: %d', ((_analysisResult_issues = analysisResult.issues) === null || _analysisResult_issues === void 0 ? void 0 : _analysisResult_issues.length) || 0);