codereview-aia 0.1.0 → 0.1.2

package/src/runtime/ui/screens/ProgressScreen.tsx

@@ -9,6 +9,10 @@ interface ProgressScreenProps {
 }
 
 const STAGE_TEXT: Record<ReviewStage, { title: string; detail: string }> = {
+  preparing: {
+    title: 'Preparing run',
+    detail: 'Triggering website checks and configuring the reviewer',
+  },
   collecting: {
     title: 'Collecting files',
     detail: 'Inspecting your working tree',
@@ -30,7 +34,11 @@ export function ProgressScreen({ stage, filesFound }: ProgressScreenProps) {
 
   const meta = useMemo(() => STAGE_TEXT[stage], [stage]);
   const fileStatus =
-    typeof filesFound === 'number' && filesFound >= 0 ? `${filesFound} files queued` : 'Preparing file list…';
+    stage === 'collecting' && typeof filesFound === 'number' && filesFound >= 0
+      ? `${filesFound} files queued`
+      : stage === 'collecting'
+        ? 'Preparing file list…'
+        : '';
 
   return (
     <Box
@@ -48,8 +56,7 @@ export function ProgressScreen({ stage, filesFound }: ProgressScreenProps) {
         </Text>
       </Box>
       <Text dimColor>{meta.detail}</Text>
-      <Text dimColor>{fileStatus}</Text>
+      {fileStatus && <Text dimColor>{fileStatus}</Text>}
     </Box>
   );
 }
-

package/src/runtime/ui/screens/ResultsScreen.tsx

@@ -5,9 +5,10 @@ import { getInk } from '../inkModules';
 
 interface ResultsScreenProps {
   result: ReviewResult;
+  debugLogPath?: string | null;
 }
 
-export function ResultsScreen({ result }: ResultsScreenProps) {
+export function ResultsScreen({ result, debugLogPath }: ResultsScreenProps) {
   const { Box, Text } = getInk();
   const { frameWidth } = useLayout();
 
@@ -68,6 +69,12 @@ export function ResultsScreen({ result }: ResultsScreenProps) {
         </Box>
       )}
 
+      {debugLogPath && (
+        <Box marginTop={1}>
+          <Text dimColor>Debug logs saved to: {debugLogPath}</Text>
+        </Box>
+      )}
+
       <Box marginTop={1}>
         <Text dimColor>Press Enter to return to mode selection.</Text>
       </Box>

package/src/utils/logger.ts

@@ -25,6 +25,9 @@ const LOG_LEVEL_MAP: Record<string, LogLevel> = {
   none: LogLevel.NONE,
 };
 
+import { createWriteStream, mkdirSync, type WriteStream } from 'node:fs';
+import { dirname } from 'node:path';
+
 // ANSI color codes for terminal output
 const COLORS = {
   reset: '\x1b[0m',
@@ -39,6 +42,8 @@ const COLORS = {
 
 // Track if we're initializing to avoid circular dependencies
 const isInitializing = false;
+let logFileStream: WriteStream | null = null;
+let suppressConsoleOutput = false;
 
 // Get the current log level from environment variables
 function getCurrentLogLevel(): LogLevel {
@@ -131,19 +136,54 @@ export function getLogLevel(): LogLevel {
   return currentLogLevel;
 }
 
-/**
- * Format a log message with timestamp and level
- * @param level The log level
- * @param message The message to log
- * @returns The formatted message
- */
-function formatLogMessage(level: string, message: string): string {
-  const timestamp = new Date().toISOString();
+function formatConsoleMessage(level: string, message: string, timestamp: string): string {
   const levelUpper = level.toUpperCase().padEnd(5);
-
   return `${COLORS.time}[${timestamp}]${COLORS.reset} ${COLORS[level as keyof typeof COLORS]}${levelUpper}${COLORS.reset} ${message}`;
 }
 
+function formatFileMessage(level: string, message: string, timestamp: string, args: any[]): string {
+  const levelUpper = level.toUpperCase().padEnd(5);
+  const extras = args.length ? ` ${args.map((arg) => serializeLogArg(arg)).join(' ')}` : '';
+  return `[${timestamp}] ${levelUpper} ${message}${extras}`;
+}
+
+function serializeLogArg(arg: unknown): string {
+  if (typeof arg === 'string') {
+    return arg;
+  }
+  if (arg instanceof Error) {
+    return arg.stack || arg.message;
+  }
+  try {
+    return JSON.stringify(arg);
+  } catch {
+    return String(arg);
+  }
+}
+
+export function enableFileLogging(filePath: string, options?: { suppressConsole?: boolean }): void {
+  try {
+    mkdirSync(dirname(filePath), { recursive: true });
+    if (logFileStream) {
+      logFileStream.end();
+    }
+    logFileStream = createWriteStream(filePath, { flags: 'a' });
+    suppressConsoleOutput = options?.suppressConsole ?? true;
+  } catch (error) {
+    console.error('Failed to enable file logging:', error);
+    logFileStream = null;
+    suppressConsoleOutput = false;
+  }
+}
+
+export function disableFileLogging(): void {
+  if (logFileStream) {
+    logFileStream.end();
+    logFileStream = null;
+  }
+  suppressConsoleOutput = false;
+}
+
 /**
  * Log a message if the current log level allows it
  * @param level The log level
@@ -151,9 +191,16 @@ function formatLogMessage(level: string, message: string): string {
  * @param args Additional arguments to log
  */
 function log(level: LogLevel, levelName: string, message: string, ...args: any[]): void {
-  // Only log if the current log level is less than or equal to the specified level
-  if (level >= currentLogLevel) {
-    const formattedMessage = formatLogMessage(levelName, message);
+  const timestamp = new Date().toISOString();
+
+  if (logFileStream) {
+    logFileStream.write(`${formatFileMessage(levelName, message, timestamp, args)}\n`);
+  }
+
+  const shouldLogToConsole = !suppressConsoleOutput && level >= currentLogLevel;
+
+  if (shouldLogToConsole) {
+    const formattedMessage = formatConsoleMessage(levelName, message, timestamp);
 
     switch (level) {
       case LogLevel.DEBUG:
@@ -168,9 +215,10 @@ function log(level: LogLevel, levelName: string, message: string, ...args: any[]
       case LogLevel.ERROR:
         console.error(formattedMessage, ...args);
         break;
+      default:
+        break;
     }
-  } else if (level === LogLevel.DEBUG && process.argv.includes('--trace-logger')) {
-    // Only show debug suppression messages when explicitly requested
+  } else if (!suppressConsoleOutput && level === LogLevel.DEBUG && process.argv.includes('--trace-logger')) {
     console.error(
       `Suppressing DEBUG log because currentLogLevel=${currentLogLevel}, message was: ${message}`,
     );
@@ -237,4 +285,6 @@ export default {
   getLogLevel,
   createLogger,
   LogLevel,
+  enableFileLogging,
+  disableFileLogging,
 };