codereview-aia 0.1.3 → 0.1.4

package/src/utils/review/fixImplementation.ts

@@ -1,61 +0,0 @@
-/**
- * @fileoverview Utilities for implementing code review fixes
- *
- * This module provides functions for applying fixes from code review suggestions
- * to the corresponding files. Currently it only provides a placeholder for future
- * implementation as automatic fixes are not fully supported.
- */
-
-import readline from 'node:readline';
-import type { FixSuggestion } from './types';
-
-/**
- * Create a readline interface for user input
- */
-export function createReadlineInterface(): readline.Interface {
-  return readline.createInterface({
-    input: process.stdin,
-    output: process.stdout,
-  });
-}
-
-/**
- * Prompt the user for confirmation
- * @param message Message to display to the user
- * @returns Promise resolving to boolean indicating user's response
- */
-export async function promptForConfirmation(message: string): Promise<boolean> {
-  const rl = createReadlineInterface();
-
-  return new Promise((resolve) => {
-    rl.question(`${message} (y/n): `, (answer) => {
-      rl.close();
-      resolve(answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes');
-    });
-  });
-}
-
-/**
- * This function is a placeholder for future functionality.
- * Currently, the AI code review tool does not automatically apply fixes.
- * It only provides suggestions that developers must review and implement manually.
- *
- * @param suggestion The fix suggestion to apply
- * @returns Always returns false as automatic fixes are not supported
- */
-export async function applyFixToFile(suggestion: FixSuggestion): Promise<boolean> {
-  console.log(`\n⚠️ Automatic fixes are not supported.`);
-  console.log(
-    `The AI code review tool only provides suggestions that you must implement manually.`,
-  );
-  console.log(`Review the suggested fix and apply it yourself if appropriate.`);
-
-  if (suggestion.suggestedCode) {
-    console.log(`\nSuggested code:`);
-    console.log('```');
-    console.log(suggestion.suggestedCode);
-    console.log('```');
-  }
-
-  return false;
-}

package/src/utils/review/index.ts

@@ -1,36 +0,0 @@
-/**
- * @fileoverview Index file for review action modules
- *
- * This module re-exports all the functionality from the review action modules
- * to provide a clean interface for the main reviewActionHandler.
- */
-
-// Export consolidation utilities
-export { consolidateReview } from './consolidateReview';
-// Export fix display utilities
-export {
-  displayDetailedFixSuggestion,
-  displayFixSuggestions,
-} from './fixDisplay';
-
-// Export fix implementation utilities
-export {
-  applyFixToFile,
-  createReadlineInterface,
-  promptForConfirmation,
-} from './fixImplementation';
-// Export interactive processing utilities
-export {
-  displayReviewResults,
-  processReviewResults,
-} from './interactiveProcessing';
-// Export progress tracking utilities for multi-pass reviews
-export { MultiPassProgressTracker } from './progressTracker';
-// Export review extraction utilities
-export {
-  extractFixSuggestions,
-  extractSection,
-  parseSuggestions,
-} from './reviewExtraction';
-// Export types
-export * from './types';

package/src/utils/review/interactiveProcessing.ts

@@ -1,294 +0,0 @@
-/**
- * @fileoverview Utilities for interactive processing of code review fixes
- *
- * This module provides functions for interactively processing code review
- * fixes, either by displaying them for manual implementation or by
- * implementing them automatically with user confirmation.
- */
-
-import logger from '../logger';
-import { displayStructuredReview, parseReviewJson } from '../reviewParser';
-import { displayDetailedFixSuggestion, displayFixSuggestions } from './fixDisplay';
-import { applyFixToFile, promptForConfirmation } from './fixImplementation';
-import { extractFixSuggestions } from './reviewExtraction';
-import { FixPriority, type FixSuggestion, type FixSummary, type SuggestionSummary } from './types';
-
-/**
- * Process review results in non-interactive mode, just displaying recommendations
- * @param reviewContent Content of the review
- * @param projectPath Base path of the project
- * @param priorityFilter Optional filter to show only specific priority issues (h, m, l, or a for all)
- * @returns Summary of suggestions found
- */
-export async function displayReviewResults(
-  reviewContent: string,
-  projectPath: string,
-  priorityFilter?: 'h' | 'm' | 'l' | 'a',
-): Promise<SuggestionSummary> {
-  // First try to parse the review content as structured JSON
-  const parsedReview = parseReviewJson(reviewContent);
-
-  if (parsedReview) {
-    // If we have a structured review, display it using the structured format
-    displayStructuredReview(parsedReview);
-
-    // Convert the structured review to FixSuggestion format for compatibility
-    const highPrioritySuggestions: FixSuggestion[] = [];
-    const mediumPrioritySuggestions: FixSuggestion[] = [];
-    const lowPrioritySuggestions: FixSuggestion[] = [];
-
-    // Process each file and issue
-    parsedReview.review.files.forEach((file) => {
-      file.issues.forEach((issue) => {
-        const suggestion: FixSuggestion = {
-          priority:
-            issue.priority === 'HIGH'
-              ? FixPriority.HIGH
-              : issue.priority === 'MEDIUM'
-                ? FixPriority.MEDIUM
-                : FixPriority.LOW,
-          file: file.filePath,
-          description: issue.description,
-          currentCode: issue.currentCode,
-          suggestedCode: issue.suggestedCode,
-          lineNumbers: issue.location
-            ? {
-                start: issue.location.startLine,
-                end: issue.location.endLine,
-              }
-            : undefined,
-        };
-
-        // Add to the appropriate array based on priority
-        if (suggestion.priority === FixPriority.HIGH) {
-          highPrioritySuggestions.push(suggestion);
-        } else if (suggestion.priority === FixPriority.MEDIUM) {
-          mediumPrioritySuggestions.push(suggestion);
-        } else {
-          lowPrioritySuggestions.push(suggestion);
-        }
-      });
-    });
-
-    const totalSuggestions =
-      highPrioritySuggestions.length +
-      mediumPrioritySuggestions.length +
-      lowPrioritySuggestions.length;
-
-    return {
-      highPrioritySuggestions,
-      mediumPrioritySuggestions,
-      lowPrioritySuggestions,
-      totalSuggestions,
-    };
-  }
-  // Fall back to the original extraction method if parsing fails
-  logger.info('Using legacy format for review results (no structured schema detected)');
-
-  // Extract all suggestions
-  const highPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.HIGH,
-  );
-  const mediumPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.MEDIUM,
-  );
-  const lowPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.LOW,
-  );
-
-  const totalSuggestions =
-    highPrioritySuggestions.length +
-    mediumPrioritySuggestions.length +
-    lowPrioritySuggestions.length;
-
-  // Display summary of all suggestions
-  logger.info('\n=== CODE REVIEW RECOMMENDATIONS ===');
-  logger.info(`Total issues found: ${totalSuggestions}`);
-  logger.info(`🟥 High priority: ${highPrioritySuggestions.length}`);
-  logger.info(`🟧 Medium priority: ${mediumPrioritySuggestions.length}`);
-  logger.info(`🟩 Low priority: ${lowPrioritySuggestions.length}`);
-
-  // Display instructions for interactive mode
-  logger.info('\nShowing ALL issues by default. To filter by priority, use these options:');
-  logger.info('  (h) High priority issues only');
-  logger.info('  (m) Medium priority issues only');
-  logger.info('  (l) Low priority issues only');
-  logger.info('  (a) All issues (default)');
-  logger.info('\nExample: cr-aia src --interactive h');
-
-  // Display suggestions based on priority filter
-  // If no filter is provided, show all issues by default
-  if (!priorityFilter || priorityFilter.toLowerCase() === 'a') {
-    // Show all issues
-    displayFixSuggestions(highPrioritySuggestions, FixPriority.HIGH);
-    displayFixSuggestions(mediumPrioritySuggestions, FixPriority.MEDIUM);
-    displayFixSuggestions(lowPrioritySuggestions, FixPriority.LOW);
-  } else {
-    // Show issues based on the specified filter
-    switch (priorityFilter.toLowerCase()) {
-      case 'h':
-        displayFixSuggestions(highPrioritySuggestions, FixPriority.HIGH);
-        break;
-      case 'm':
-        displayFixSuggestions(mediumPrioritySuggestions, FixPriority.MEDIUM);
-        break;
-      case 'l':
-        displayFixSuggestions(lowPrioritySuggestions, FixPriority.LOW);
-        break;
-      default:
-        logger.warn('Invalid priority filter. Use h, m, l, or a.');
-        // Show all issues if the filter is invalid
-        displayFixSuggestions(highPrioritySuggestions, FixPriority.HIGH);
-        displayFixSuggestions(mediumPrioritySuggestions, FixPriority.MEDIUM);
-        displayFixSuggestions(lowPrioritySuggestions, FixPriority.LOW);
-    }
-  }
-
-  return {
-    highPrioritySuggestions,
-    mediumPrioritySuggestions,
-    lowPrioritySuggestions,
-    totalSuggestions,
-  };
-}
-
-/**
- * Process review results and implement fixes
- * @param reviewContent Content of the review
- * @param projectPath Base path of the project
- * @param autoImplementHighPriority Whether to automatically implement high priority fixes
- * @param promptForMediumLow Whether to prompt for confirmation on medium and low priority fixes
- * @returns Summary of actions taken
- */
-export async function processReviewResults(
-  reviewContent: string,
-  projectPath: string,
-  autoImplementHighPriority = true,
-  promptForMediumLow = true,
-): Promise<FixSummary> {
-  // Initialize counters
-  let highPriorityFixed = 0;
-  let mediumPriorityFixed = 0;
-  let lowPriorityFixed = 0;
-  let totalSuggestions = 0;
-
-  // Process high priority suggestions
-  console.log('\nExtracting high priority issues...');
-  const highPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.HIGH,
-  );
-  totalSuggestions += highPrioritySuggestions.length;
-
-  if (highPrioritySuggestions.length > 0) {
-    console.log(`Found ${highPrioritySuggestions.length} high priority issues.`);
-
-    if (autoImplementHighPriority) {
-      console.log('Automatically implementing high priority fixes...');
-
-      for (const suggestion of highPrioritySuggestions) {
-        console.log(`\nImplementing fix for: ${suggestion.description}`);
-        console.log(`File: ${suggestion.file}`);
-
-        const success = await applyFixToFile(suggestion);
-        if (success) {
-          console.log('✅ Fix applied successfully.');
-          highPriorityFixed++;
-        } else {
-          console.log('❌ Could not apply fix automatically.');
-        }
-      }
-    } else {
-      console.log('Skipping high priority fixes as auto-implementation is disabled.');
-    }
-  } else {
-    console.log('No high priority issues found.');
-  }
-
-  // Process medium priority suggestions
-  console.log('\nExtracting medium priority issues...');
-  const mediumPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.MEDIUM,
-  );
-  totalSuggestions += mediumPrioritySuggestions.length;
-
-  if (mediumPrioritySuggestions.length > 0) {
-    console.log(`Found ${mediumPrioritySuggestions.length} medium priority issues.`);
-
-    if (promptForMediumLow) {
-      for (const suggestion of mediumPrioritySuggestions) {
-        displayDetailedFixSuggestion(suggestion, 0, FixPriority.MEDIUM);
-
-        const shouldImplement = await promptForConfirmation('Implement this fix?');
-        if (shouldImplement) {
-          const success = await applyFixToFile(suggestion);
-          if (success) {
-            console.log('✅ Fix applied successfully.');
-            mediumPriorityFixed++;
-          } else {
-            console.log('❌ Could not apply fix automatically.');
-          }
-        } else {
-          console.log('Skipping this fix.');
-        }
-      }
-    } else {
-      console.log('Skipping medium priority fixes as prompting is disabled.');
-    }
-  } else {
-    console.log('No medium priority issues found.');
-  }
-
-  // Process low priority suggestions
-  console.log('\nExtracting low priority issues...');
-  const lowPrioritySuggestions = await extractFixSuggestions(
-    reviewContent,
-    projectPath,
-    FixPriority.LOW,
-  );
-  totalSuggestions += lowPrioritySuggestions.length;
-
-  if (lowPrioritySuggestions.length > 0) {
-    console.log(`Found ${lowPrioritySuggestions.length} low priority issues.`);
-
-    if (promptForMediumLow) {
-      for (const suggestion of lowPrioritySuggestions) {
-        displayDetailedFixSuggestion(suggestion, 0, FixPriority.LOW);
-
-        const shouldImplement = await promptForConfirmation('Implement this fix?');
-        if (shouldImplement) {
-          const success = await applyFixToFile(suggestion);
-          if (success) {
-            console.log('✅ Fix applied successfully.');
-            lowPriorityFixed++;
-          } else {
-            console.log('❌ Could not apply fix automatically.');
-          }
-        } else {
-          console.log('Skipping this fix.');
-        }
-      }
-    } else {
-      console.log('Skipping low priority fixes as prompting is disabled.');
-    }
-  } else {
-    console.log('No low priority issues found.');
-  }
-
-  // Return summary
-  return {
-    highPriorityFixed,
-    mediumPriorityFixed,
-    lowPriorityFixed,
-    totalSuggestions,
-  };
-}

package/src/utils/review/progressTracker.ts

@@ -1,296 +0,0 @@
-/**
- * @fileoverview Progress tracker for multi-pass reviews.
- *
- * This module provides utilities for tracking and displaying progress
- * during multi-pass reviews.
- */
-
-import readline from 'node:readline';
-import logger from '../logger';
-
-/**
- * Progress state for a multi-pass review
- */
-export interface MultiPassProgress {
-  /** Total number of passes */
-  totalPasses: number;
-  /** Current pass number */
-  currentPass: number;
-  /** Files being processed in the current pass */
-  currentFiles: string[];
-  /** Total number of files */
-  totalFiles: number;
-  /** Number of files processed so far */
-  processedFiles: number;
-  /** Start time of the review */
-  startTime: Date;
-  /** Estimated completion time */
-  estimatedCompletionTime?: Date;
-  /** Whether the review is complete */
-  isComplete: boolean;
-  /** Current phase of the review */
-  currentPhase:
-    | 'preparing'
-    | 'analyzing'
-    | 'reviewing'
-    | 'processing'
-    | 'consolidating'
-    | 'complete';
-}
-
-/**
- * Progress tracker for multi-pass reviews
- */
-export class MultiPassProgressTracker {
-  private progress: MultiPassProgress;
-  private updateInterval: NodeJS.Timeout | null = null;
-  private useAnsiEscapes: boolean;
-  private completedFiles: string[] = [];
-
-  /**
-   * Create a new progress tracker
-   * @param totalPasses Total number of passes
-   * @param totalFiles Total number of files
-   * @param options Options for the progress tracker
-   */
-  constructor(
-    totalPasses = 1,
-    totalFiles = 0,
-    options: {
-      useAnsiEscapes?: boolean;
-      quiet?: boolean;
-    } = {},
-  ) {
-    this.progress = {
-      totalPasses,
-      currentPass: 0,
-      currentFiles: [],
-      totalFiles,
-      processedFiles: 0,
-      startTime: new Date(),
-      isComplete: false,
-      currentPhase: 'preparing',
-    };
-
-    this.useAnsiEscapes = options.useAnsiEscapes !== false;
-
-    // Start the progress update interval if not in quiet mode
-    if (!options.quiet) {
-      this.startProgressUpdates();
-    }
-  }
-
-  /**
-   * Initialize the progress tracker with a total file count
-   * @param totalFiles Total number of files
-   */
-  public initialize(totalFiles: number): void {
-    this.progress.totalFiles = totalFiles;
-    logger.info(`Initialized progress tracker with ${totalFiles} total files`);
-  }
-
-  /**
-   * Start the progress update interval
-   */
-  private startProgressUpdates(): void {
-    // Update the progress display every 1 second
-    this.updateInterval = setInterval(() => {
-      this.updateProgressDisplay();
-    }, 1000);
-  }
-
-  /**
-   * Stop the progress update interval
-   */
-  public stopProgressUpdates(): void {
-    if (this.updateInterval) {
-      clearInterval(this.updateInterval);
-      this.updateInterval = null;
-    }
-  }
-
-  /**
-   * Update the progress display
-   */
-  private updateProgressDisplay(): void {
-    if (this.progress.isComplete) {
-      return;
-    }
-
-    const { currentPass, totalPasses, currentPhase, startTime } = this.progress;
-
-    // Calculate elapsed time
-    const elapsed = Math.floor((Date.now() - startTime.getTime()) / 1000);
-    const elapsedMinutes = Math.floor(elapsed / 60);
-    const elapsedSeconds = elapsed % 60;
-
-    // Clear the previous line if terminal supports ANSI escapes
-    if (this.useAnsiEscapes) {
-      readline.clearLine(process.stdout, 0);
-      readline.cursorTo(process.stdout, 0);
-    }
-
-    // Create the progress message
-    let progressMessage = '';
-
-    if (currentPhase === 'preparing') {
-      progressMessage = `Preparing multi-pass review...`;
-    } else if (currentPhase === 'analyzing') {
-      progressMessage = `Analyzing files for multi-pass review...`;
-    } else if (currentPhase === 'reviewing') {
-      const passProgress = (currentPass / totalPasses) * 100;
-      progressMessage = `Pass ${currentPass}/${totalPasses} (${passProgress.toFixed(1)}%)`;
-
-      if (this.progress.currentFiles.length > 0) {
-        const fileNames = this.progress.currentFiles.map((f) => f.split('/').pop()).join(', ');
-        progressMessage += ` | Processing: ${fileNames.length > 50 ? `${fileNames.substring(0, 50)}...` : fileNames}`;
-      }
-    } else if (currentPhase === 'processing') {
-      progressMessage = `Processing results...`;
-    } else if (currentPhase === 'consolidating') {
-      progressMessage = `Consolidating multi-pass review and generating final graded report...`;
-    }
-
-    // Add timing information
-    progressMessage += ` | Elapsed: ${elapsedMinutes}m ${elapsedSeconds}s`;
-
-    // Print the progress message without a newline if terminal supports ANSI escapes
-    if (this.useAnsiEscapes) {
-      process.stdout.write(progressMessage);
-    } else {
-      // Otherwise, just log the progress
-      logger.info(progressMessage);
-    }
-  }
-
-  /**
-   * Start a new pass
-   * @param passNumber Pass number
-   * @param files Files being processed in this pass
-   */
-  public startPass(passNumber: number, files: string[]): void {
-    this.progress.currentPass = passNumber;
-    this.progress.currentFiles = files;
-    this.progress.currentPhase = 'reviewing';
-
-    // Log the start of a new pass (with a newline to avoid overwriting the progress bar)
-    if (this.useAnsiEscapes) {
-      process.stdout.write('\n');
-    }
-
-    logger.info(
-      `Starting pass ${passNumber}/${this.progress.totalPasses} with ${files.length} files`,
-    );
-  }
-
-  /**
-   * Complete a pass
-   * @param passNumber Pass number
-   */
-  public completePass(passNumber: number): void {
-    if (passNumber !== this.progress.currentPass) {
-      logger.warn(`Completed pass ${passNumber} but current pass is ${this.progress.currentPass}`);
-    }
-
-    this.progress.processedFiles += this.progress.currentFiles.length;
-    this.progress.currentFiles = [];
-
-    // Log the completion of a pass (with a newline to avoid overwriting the progress bar)
-    if (this.useAnsiEscapes) {
-      process.stdout.write('\n');
-    }
-
-    logger.info(`Completed pass ${passNumber}/${this.progress.totalPasses}`);
-
-    // If this was the last pass, mark the review as complete
-    if (passNumber === this.progress.totalPasses) {
-      this.complete();
-    }
-  }
-
-  /**
-   * Set the current phase
-   * @param phase Current phase
-   */
-  public setPhase(phase: MultiPassProgress['currentPhase']): void {
-    this.progress.currentPhase = phase;
-
-    if (phase === 'complete') {
-      this.complete();
-    }
-  }
-
-  /**
-   * Complete the review
-   */
-  public complete(): void {
-    this.progress.isComplete = true;
-    this.progress.currentPhase = 'complete';
-
-    // Calculate total time
-    const elapsed = Math.floor((Date.now() - this.progress.startTime.getTime()) / 1000);
-    const elapsedMinutes = Math.floor(elapsed / 60);
-    const elapsedSeconds = elapsed % 60;
-
-    // Clear the progress bar
-    if (this.useAnsiEscapes) {
-      readline.clearLine(process.stdout, 0);
-      readline.cursorTo(process.stdout, 0);
-    }
-
-    // Log the completion message
-    logger.info(`Multi-pass review completed in ${elapsedMinutes}m ${elapsedSeconds}s`);
-
-    // Stop the progress updates
-    this.stopProgressUpdates();
-  }
-
-  /**
-   * Mark a file as completed
-   * @param filePath Path to the file that was completed
-   */
-  public completeFile(filePath: string): void {
-    // Add to completed files list
-    if (!this.completedFiles.includes(filePath)) {
-      this.completedFiles.push(filePath);
-    }
-
-    // Remove from current files if it's there
-    const fileIndex = this.progress.currentFiles.indexOf(filePath);
-    if (fileIndex !== -1) {
-      this.progress.currentFiles.splice(fileIndex, 1);
-    }
-
-    // Update processed files count
-    const progressPercent = (this.completedFiles.length / this.progress.totalFiles) * 100;
-    logger.debug(`File completed: ${filePath} (${progressPercent.toFixed(1)}% complete)`);
-  }
-
-  /**
-   * Get the current progress
-   * @returns Current progress state
-   */
-  public getProgress(): MultiPassProgress {
-    return { ...this.progress };
-  }
-
-  /**
-   * Get the full state including completed files
-   * @returns Full state object
-   */
-  public getState(): {
-    progressData: MultiPassProgress;
-    completedFiles: string[];
-    progress: number;
-    completed: boolean;
-    currentPass: number;
-  } {
-    return {
-      progressData: this.getProgress(),
-      completedFiles: [...this.completedFiles],
-      progress: this.completedFiles.length / this.progress.totalFiles,
-      completed: this.progress.isComplete,
-      currentPass: this.progress.currentPass,
-    };
-  }
-}