codereview-aia 0.1.2 → 0.1.3

package/src/runtime/ui/RuntimeApp.tsx

@@ -5,7 +5,7 @@ import { ModeSelection } from './screens/ModeSelection';
 import { ProgressScreen } from './screens/ProgressScreen';
 import { ResultsScreen } from './screens/ResultsScreen';
 import { getInk, getInkSpinner } from './inkModules';
-import type { ReviewMode, ReviewResult, ReviewStage } from '../reviewPipeline';
+import type { ReviewMode, ReviewResult, ReviewStage, StageUpdate } from '../reviewPipeline';
 import { executeReview } from '../reviewPipeline';
 import { loadSession, getSessionToken } from '../auth/session';
 import type { SessionUser } from '../auth/types';
@@ -24,6 +24,11 @@ interface RuntimeAppProps {
 interface ProgressState {
   stage: ReviewStage;
   filesFound?: number;
+  currentFile?: string;
+  currentIndex?: number;
+  totalFiles?: number;
+  uncommittedFiles?: number;
+  contextFiles?: number;
 }
 
 const defaultProgress: ProgressState = { stage: 'collecting' };
@@ -32,6 +37,19 @@ const GENERIC_ERROR_MESSAGE =
 const PREPROD_ERROR_MESSAGE =
   'Could not start the website check. Double-check the URL or try again later.';
 
+function transformStageUpdate(stage: ReviewStage, info?: StageUpdate): ProgressState {
+  return {
+    stage,
+    filesFound: info?.filesFound,
+    currentFile: info?.currentFile,
+    currentIndex:
+      typeof info?.fileIndex === 'number' && info.fileIndex > 0 ? info.fileIndex : undefined,
+    totalFiles: info?.totalFiles,
+    uncommittedFiles: info?.uncommittedFiles,
+    contextFiles: info?.contextFiles,
+  };
+}
+
 function AppBody({ debug = false }: RuntimeAppProps) {
   const { Box, Text, useApp, useInput } = getInk();
   const { exit } = useApp();
@@ -98,7 +116,7 @@ function AppBody({ debug = false }: RuntimeAppProps) {
     setScreen('progress');
     setResults(null);
     setError(null);
-    setProgress(mode === 'preprod' ? { stage: 'preparing' } : defaultProgress);
+    setProgress(mode === 'preprod' ? { stage: 'preparing' } : { ...defaultProgress });
     setLastDebugLogPath(null);
 
     let debugSession: DebugLogSession | null = null;
@@ -138,14 +156,15 @@ function AppBody({ debug = false }: RuntimeAppProps) {
 
     try {
       logger.info('Starting code review run', { mode, workspace: process.cwd(), preprodTarget: trimmedUrl });
-      setProgress({ stage: 'collecting' });
+      setProgress({ ...defaultProgress });
       const reviewResult = await executeReview({
         model: process.env.AI_CODE_REVIEW_MODEL || RUNTIME_CONFIG.DEFAULT_MODEL,
         outDir: './reports',
         debug: combinedDebugFlag,
         mode,
         preprodTargetUrl: trimmedUrl,
-        onStage: (stage, info) => setProgress({ stage, filesFound: info?.filesFound }),
+        onStage: (stage, info) =>
+          setProgress(transformStageUpdate(stage, info)),
       });
 
       logger.info('Code review run complete', {
@@ -182,7 +201,17 @@ function AppBody({ debug = false }: RuntimeAppProps) {
           onToggleDebug={handleToggleDebug}
         />
       )}
-      {screen === 'progress' && <ProgressScreen stage={progress.stage} filesFound={progress.filesFound} />}
+      {screen === 'progress' && (
+        <ProgressScreen
+          stage={progress.stage}
+          filesFound={progress.filesFound}
+          currentFile={progress.currentFile}
+          currentIndex={progress.currentIndex}
+          totalFiles={progress.totalFiles}
+          uncommittedFiles={progress.uncommittedFiles}
+          contextFiles={progress.contextFiles}
+        />
+      )}
       {screen === 'results' && results && (
         <ResultsScreen result={results} debugLogPath={lastDebugLogPath} />
       )}

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

@@ -6,6 +6,11 @@ import { getInk, getInkSpinner } from '../inkModules';
 interface ProgressScreenProps {
   stage: ReviewStage;
   filesFound?: number;
+  currentFile?: string;
+  currentIndex?: number;
+  totalFiles?: number;
+  uncommittedFiles?: number;
+  contextFiles?: number;
 }
 
 const STAGE_TEXT: Record<ReviewStage, { title: string; detail: string }> = {
@@ -27,12 +32,26 @@ const STAGE_TEXT: Record<ReviewStage, { title: string; detail: string }> = {
   },
 };
 
-export function ProgressScreen({ stage, filesFound }: ProgressScreenProps) {
+export function ProgressScreen({
+  stage,
+  filesFound,
+  currentFile,
+  currentIndex,
+  totalFiles,
+  uncommittedFiles,
+  contextFiles,
+}: ProgressScreenProps) {
   const { Box, Text } = getInk();
   const Spinner = getInkSpinner();
   const { frameWidth } = useLayout();
 
   const meta = useMemo(() => STAGE_TEXT[stage], [stage]);
+  const base = typeof uncommittedFiles === 'number' ? uncommittedFiles : undefined;
+  const context = typeof contextFiles === 'number' ? contextFiles : undefined;
+  const totalBreakdown =
+    base !== undefined || context !== undefined
+      ? `Queued: ${base ?? '?'} uncommitted${context ? ` + ${context} context` : ''}`
+      : '';
   const fileStatus =
     stage === 'collecting' && typeof filesFound === 'number' && filesFound >= 0
       ? `${filesFound} files queued`
@@ -40,6 +59,19 @@ export function ProgressScreen({ stage, filesFound }: ProgressScreenProps) {
         ? 'Preparing file list…'
         : '';
 
+  const showReviewingProgress =
+    stage === 'reviewing' &&
+    typeof currentIndex === 'number' &&
+    typeof totalFiles === 'number' &&
+    totalFiles > 0 &&
+    Boolean(currentFile);
+
+  const truncatedPath = useMemo(() => {
+    if (!currentFile) return '';
+    const availableWidth = Math.max(10, frameWidth - 10);
+    return truncateMiddle(currentFile, availableWidth);
+  }, [currentFile, frameWidth]);
+
   return (
     <Box
       width={frameWidth}
@@ -57,6 +89,22 @@ export function ProgressScreen({ stage, filesFound }: ProgressScreenProps) {
       </Box>
       <Text dimColor>{meta.detail}</Text>
       {fileStatus && <Text dimColor>{fileStatus}</Text>}
+      {totalBreakdown && <Text dimColor>{totalBreakdown}</Text>}
+      {showReviewingProgress && (
+        <Text dimColor>{`Reviewing (${currentIndex}/${totalFiles}): ${truncatedPath}`}</Text>
+      )}
     </Box>
   );
 }
+
+function truncateMiddle(value: string, maxLength: number): string {
+  if (value.length <= maxLength) {
+    return value;
+  }
+
+  const ellipsis = '…';
+  const keep = Math.max(1, Math.floor((maxLength - ellipsis.length) / 2));
+  const start = value.slice(0, keep);
+  const end = value.slice(value.length - keep);
+  return `${start}${ellipsis}${end}`;
+}

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

@@ -22,6 +22,14 @@ export function ResultsScreen({ result, debugLogPath }: ResultsScreenProps) {
     ];
   }, [result.totals]);
 
+  const scopeLine = useMemo(() => {
+    const breakdown = result.fileBreakdown;
+    if (!breakdown) return null;
+    return `Scope: ${breakdown.uncommitted} uncommitted + ${breakdown.context} context files`;
+  }, [result.fileBreakdown]);
+
+  const summaryText = result.reportSummary?.trim() || 'Summary unavailable. Open the saved report for details.';
+
   return (
     <Box
       width={frameWidth}
@@ -36,15 +44,26 @@ export function ResultsScreen({ result, debugLogPath }: ResultsScreenProps) {
         Review complete for {result.repo}
       </Text>
       <Text dimColor>Duration: {result.duration}s — Files reviewed: {result.filesReviewed}</Text>
+      {scopeLine && <Text dimColor>{scopeLine}</Text>}
+
+      <Box flexDirection="column" borderStyle="round" borderColor="cyan" paddingX={1} paddingY={0}>
+        <Text color="cyan" bold>
+          LLM summary
+        </Text>
+        <Text>{summaryText}</Text>
+      </Box>
 
       <Box flexDirection="column" marginTop={1}>
-        {severityRows.map((row) => (
-          <Box key={row.label}>
-            <Text>
-              {row.label}: {row.value}
-            </Text>
-          </Box>
-        ))}
+        <Text bold>Severity totals</Text>
+        <Box flexDirection="row" flexWrap="wrap">
+          {severityRows.map((row) => (
+            <Box key={row.label} width={24}>
+              <Text>
+                {row.label}: {row.value}
+              </Text>
+            </Box>
+          ))}
+        </Box>
       </Box>
 
       {result.findings.length === 0 ? (
@@ -53,8 +72,9 @@ export function ResultsScreen({ result, debugLogPath }: ResultsScreenProps) {
         <Box flexDirection="column" marginTop={1}>
           <Text bold>Sample findings</Text>
           {result.findings.slice(0, 5).map((finding, index) => (
-            <Text key={`${finding.filePath}-${index}`} dimColor>
-              · {finding.filePath ?? finding.file ?? 'unknown'} {finding.priority ? `(${finding.priority})` : ''}
+            <Text key={`${finding.filePath ?? finding.file ?? 'unknown'}-${index}`} dimColor>
+              · {finding.filePath ?? finding.file ?? 'unknown'}{' '}
+              {finding.priority ? `(${finding.priority})` : ''}
             </Text>
           ))}
           {result.findings.length > 5 && (

package/src/types/review.ts

@@ -97,6 +97,10 @@ export interface ReviewOptions {
   uiLanguage?: string;
   /** Target file or directory (used internally) */
   target?: string;
+  /** Explicit list of files to review (bypasses discovery) */
+  targetFileList?: string[];
+  /** Runtime context information for prompt enrichment */
+  runContext?: RunContext;
   /** Schema instructions for structured output */
   schemaInstructions?: string;
   /** Examples for prompts */
@@ -277,6 +281,20 @@ export interface ReviewResult {
   files?: FileInfo[];
 }
 
+export interface RunContext {
+  repoName: string;
+  reviewMode?: string;
+  totalFiles?: number;
+  groupIndex?: number;
+  groupCount?: number;
+  groupLabel?: string;
+  groupFiles?: string[];
+  docFiles?: string[];
+  uncommittedFiles?: number;
+  contextFiles?: number;
+  contextDocs?: Array<{ path: string; snippet: string }>;
+}
+
 /**
  * Cost information for a single pass in a multi-pass review
  */

package/dist/analysis/FindingsExtractor.d.ts

@@ -1,105 +0,0 @@
-/**
- * @fileoverview Findings Extractor for Code Reviews
- *
- * This service extracts, categorizes, and analyzes findings from code review
- * results. It provides utilities for grading, recommendation generation,
- * and finding prioritization.
- */
-/**
- * Categorized findings from code review
- */
-export interface CategorizedFindings {
-    high: Set<string>;
-    medium: Set<string>;
-    low: Set<string>;
-}
-/**
- * Grade information with justification
- */
-export interface GradeInfo {
-    grade: string;
-    justification: string;
-}
-/**
- * Service for extracting and analyzing findings from code reviews
- */
-export declare class FindingsExtractor {
-    /**
-     * Extract findings from review passes
-     * @param passes Array of review results with content
-     * @returns Categorized findings
-     */
-    extractFindingsFromPasses(passes: Array<{
-        content: string;
-    }>): CategorizedFindings;
-    /**
-     * Extract individual issue texts from content
-     * @param content Review content
-     * @returns Array of issue descriptions
-     */
-    extractIssueTexts(content: string): string[];
-    /**
-     * Check if an issue is high priority
-     * @param issue Issue text (lowercase)
-     * @returns True if high priority
-     */
-    private isHighPriority;
-    /**
-     * Check if an issue is medium priority
-     * @param issue Issue text (lowercase)
-     * @returns True if medium priority
-     */
-    private isMediumPriority;
-    /**
-     * Calculate grade based on findings for a specific category
-     * @param findings Categorized findings
-     * @param category Category name (for logging)
-     * @returns Grade information
-     */
-    calculateGrade(findings: CategorizedFindings, category: string): GradeInfo;
-    /**
-     * Calculate overall grade based on findings
-     * @param findings Categorized findings
-     * @returns Overall grade string
-     */
-    calculateOverallGrade(findings: CategorizedFindings): string;
-    /**
-     * Get justification for grade
-     * @param findings Categorized findings
-     * @param category Category name
-     * @returns Grade justification string
-     */
-    getGradeJustification(findings: CategorizedFindings, category: string): string;
-    /**
-     * Generate recommendations based on findings
-     * @param findings Categorized findings
-     * @param hasErrors Whether there are compilation errors
-     * @returns Array of recommendation strings
-     */
-    generateRecommendations(findings: CategorizedFindings, hasErrors: boolean): string[];
-    /**
-     * Check if findings contain security-related issues
-     * @param findings Categorized findings
-     * @returns True if security issues are present
-     */
-    private hasSecurityIssues;
-    /**
-     * Check if findings contain performance-related issues
-     * @param findings Categorized findings
-     * @returns True if performance issues are present
-     */
-    private hasPerformanceIssues;
-    /**
-     * Check if findings contain testing-related issues
-     * @param findings Categorized findings
-     * @returns True if testing issues are present
-     */
-    private hasTestingIssues;
-    /**
-     * Format findings for display in reports
-     * @param findings Categorized findings
-     * @param maxPerCategory Maximum items to show per category
-     * @returns Formatted findings string
-     */
-    formatFindings(findings: CategorizedFindings, maxPerCategory?: number): string;
-}