codereview-aia 0.1.3 → 0.2.0

package/src/strategies/ReviewStrategy.ts

@@ -1,64 +0,0 @@
-/**
- * @fileoverview Review strategy interface and base class.
- *
- * This module defines the interface and base class for review strategies,
- * which encapsulate the logic for different types of code reviews.
- */
-
-import type { ApiClientConfig } from '../core/ApiClientSelector';
-import type { FileInfo, ReviewOptions, ReviewResult, ReviewType } from '../types/review';
-import type { ProjectDocs } from '../utils/projectDocs';
-
-/**
- * Interface for review strategies
- */
-export interface IReviewStrategy {
-  /**
-   * Execute the review strategy
-   * @param files Files to review
-   * @param projectName Project name
-   * @param projectDocs Project documentation
-   * @param options Review options
-   * @param apiClientConfig API client configuration
-   * @returns Promise resolving to the review result
-   */
-  execute(
-    files: FileInfo[],
-    projectName: string,
-    projectDocs: ProjectDocs | null,
-    options: ReviewOptions,
-    apiClientConfig: ApiClientConfig,
-  ): Promise<ReviewResult>;
-}
-
-/**
- * Base class for review strategies
- */
-export abstract class BaseReviewStrategy implements IReviewStrategy {
-  protected reviewType: ReviewType;
-
-  /**
-   * Create a new review strategy
-   * @param reviewType Type of review to perform
-   */
-  constructor(reviewType: ReviewType) {
-    this.reviewType = reviewType;
-  }
-
-  /**
-   * Execute the review strategy
-   * @param files Files to review
-   * @param projectName Project name
-   * @param projectDocs Project documentation
-   * @param options Review options
-   * @param apiClientConfig API client configuration
-   * @returns Promise resolving to the review result
-   */
-  abstract execute(
-    files: FileInfo[],
-    projectName: string,
-    projectDocs: ProjectDocs | null,
-    options: ReviewOptions,
-    apiClientConfig: ApiClientConfig,
-  ): Promise<ReviewResult>;
-}

package/src/strategies/StrategyFactory.ts

@@ -1,79 +0,0 @@
-/**
- * @fileoverview Strategy factory for creating review strategies.
- *
- * This module provides a factory for creating the appropriate review strategy
- * based on the review options.
- */
-
-import { PluginManager } from '../plugins/PluginManager';
-import type { ReviewOptions, ReviewType } from '../types/review';
-import logger from '../utils/logger';
-import { ArchitecturalReviewStrategy } from './ArchitecturalReviewStrategy';
-import { CodingTestReviewStrategy } from './CodingTestReviewStrategy';
-import { ConsolidatedReviewStrategy } from './ConsolidatedReviewStrategy';
-import { ExtractPatternsReviewStrategy } from './ExtractPatternsReviewStrategy';
-import { MultiPassReviewStrategy } from './MultiPassReviewStrategy';
-import type { IReviewStrategy } from './ReviewStrategy';
-
-/**
- * Factory for creating review strategies
- */
-export class StrategyFactory {
-  /**
-   * Create a review strategy based on options
-   * @param options Review options
-   * @returns The appropriate review strategy
-   */
-  static createStrategy(options: ReviewOptions): IReviewStrategy {
-    // Check if a custom strategy is specified
-    if (options.strategy) {
-      const pluginManager = PluginManager.getInstance();
-      const customStrategy = pluginManager.getPlugin(options.strategy);
-
-      if (customStrategy) {
-        logger.info(`Using custom strategy: ${options.strategy}`);
-        return customStrategy;
-      }
-      logger.warn(
-        `Custom strategy "${options.strategy}" not found. Falling back to default strategy.`,
-      );
-    }
-
-    // Use default strategies if no custom strategy is specified or if the custom strategy is not found
-    const reviewType = options.type as ReviewType;
-
-    // Debug logging for strategy selection
-    logger.debug(`StrategyFactory.createStrategy: reviewType = "${reviewType}"`);
-
-    // Check if multi-pass mode is explicitly requested
-    if (options.multiPass) {
-      logger.info('Using Multi-Pass Review Strategy');
-      return new MultiPassReviewStrategy(reviewType);
-    }
-
-    if (reviewType === 'architectural') {
-      logger.debug('Creating ArchitecturalReviewStrategy');
-      return new ArchitecturalReviewStrategy();
-    }
-    if (reviewType === 'extract-patterns') {
-      logger.debug('Creating ExtractPatternsReviewStrategy');
-      return new ExtractPatternsReviewStrategy();
-    }
-    if (reviewType === 'coding-test') {
-      logger.debug('Creating CodingTestReviewStrategy');
-      return new CodingTestReviewStrategy();
-    }
-    if (
-      reviewType === 'unused-code' ||
-      reviewType === 'focused-unused-code' ||
-      reviewType === 'code-tracing-unused-code' ||
-      reviewType === 'quick-fixes'
-    ) {
-      logger.warn(
-        `Review type "${reviewType}" is not supported in this runtime. Falling back to consolidated review.`,
-      );
-    }
-    logger.debug(`Creating ConsolidatedReviewStrategy for reviewType: "${reviewType}"`);
-    return new ConsolidatedReviewStrategy(reviewType);
-  }
-}

package/src/tokenizers/baseTokenizer.ts

@@ -1,61 +0,0 @@
-/**
- * Shared tokenizer contract for the cr-aia.
- * Tokenizers just need to report how many tokens a given model would consume.
- */
-export interface Tokenizer {
-  countTokens(text: string): number;
-  getModelName(): string;
-  supportsModel(modelName: string): boolean;
-}
-
-export class TokenizerRegistry {
-  private static tokenizers: Tokenizer[] = [];
-
-  static register(tokenizer: Tokenizer): void {
-    TokenizerRegistry.tokenizers.push(tokenizer);
-  }
-
-  static find(modelName: string): Tokenizer | undefined {
-    return TokenizerRegistry.tokenizers.find((tokenizer) => tokenizer.supportsModel(modelName));
-  }
-
-  static list(): Tokenizer[] {
-    return [...TokenizerRegistry.tokenizers];
-  }
-}
-
-export class FallbackTokenizer implements Tokenizer {
-  countTokens(text: string): number {
-    // Rule of thumb for GPT-style models: ~4 characters per token.
-    return Math.ceil(text.length / 4);
-  }
-
-  getModelName(): string {
-    return 'fallback';
-  }
-
-  supportsModel(_modelName: string): boolean {
-    return true;
-  }
-}
-
-const fallbackTokenizer = new FallbackTokenizer();
-
-export function getTokenizer(modelName: string): Tokenizer {
-  return TokenizerRegistry.find(modelName) ?? fallbackTokenizer;
-}
-
-/**
- * Count tokens using whichever tokenizer matches the supplied model.
- */
-export function countTokens(text: string, modelName: string): number {
-  const tokenizer = getTokenizer(modelName);
-  const count = tokenizer.countTokens(text);
-
-  if (modelName === 'test-small-context') {
-    // Tests rely on this inflated value to stress chunking.
-    return text.length;
-  }
-
-  return count;
-}

package/src/tokenizers/gptTokenizer.ts

@@ -1,27 +0,0 @@
-import { encode } from 'gpt-tokenizer';
-import { type Tokenizer, TokenizerRegistry } from './baseTokenizer';
-
-export class GPTTokenizer implements Tokenizer {
-  private modelPatterns: RegExp[] = [/gpt/i];
-
-  countTokens(text: string): number {
-    try {
-      const tokens = encode(text);
-      return tokens.length;
-    } catch (error) {
-      console.warn(`Error counting tokens with GPT tokenizer: ${error}`);
-      return Math.ceil(text.length / 4);
-    }
-  }
-
-  getModelName(): string {
-    return 'gpt';
-  }
-
-  supportsModel(modelName: string): boolean {
-    const lowerModelName = modelName.toLowerCase();
-    return this.modelPatterns.some((pattern) => pattern.test(lowerModelName));
-  }
-}
-
-TokenizerRegistry.register(new GPTTokenizer());

package/src/types/apiResponses.ts

@@ -1,40 +0,0 @@
-/**
- * Type definitions for cr-aia responses.
- *
- * We only keep the pieces of the responses that are actually consumed in this fork.
- * Additional provider-specific interfaces were removed because they were never used.
- */
-
-/**
- * Minimal JSON payload we expect when a provider returns structured output.
- */
-export interface AIJsonResponse {
-  review?: {
-    version?: string;
-    timestamp?: string;
-    files?: Array<{
-      filePath?: string;
-      issues?: Array<{
-        id?: string;
-        priority?: string;
-        description?: string;
-        location?: {
-          startLine?: number;
-          endLine?: number;
-        };
-        currentCode?: string;
-        suggestedCode?: string;
-        explanation?: string;
-      }>;
-    }>;
-    summary?: {
-      grade?: string;
-      highPriorityIssues?: number;
-      mediumPriorityIssues?: number;
-      lowPriorityIssues?: number;
-      totalIssues?: number;
-    };
-    positiveAspects?: string[];
-    recommendations?: string[];
-  };
-}

package/src/types/cli.ts

@@ -1,24 +0,0 @@
-import type { ReviewOptions } from './review';
-
-/**
- * Minimal CLI options interface consumed by legacy config utilities.
- * The actual argument parser is gone, but these modules still expect
- * a typed shape for overrides coming from the UI runtime.
- */
-export interface CliOptions extends ReviewOptions {
-  /** Target file or directory */
-  target: string;
-  /** Output directory for generated files */
-  outputDir?: string;
-  /** Writer model for consolidation */
-  writerModel?: string;
-  /** API keys map (legacy singular naming) */
-  apiKey?: Record<string, string>;
-  /** API keys map (preferred naming) */
-  apiKeys?: Record<string, string>;
-  /** Log level override */
-  logLevel?: string;
-  /** Path to configuration file */
-  config?: string;
-}
-

package/src/types/configuration.ts

@@ -1,201 +0,0 @@
-/**
- * Centralized configuration type definitions for cr-aia.
- *
- * The previous upstream project exposed dozens of knobs we do not use, so this fork
- * focuses on the configuration surfaces that actually exist in our runtime.
- */
-
-import { z } from 'zod';
-
-/**
- * Environment source for a configuration value
- */
-export type EnvSource =
-  // Primary environment variables
-  | 'AI_CODE_REVIEW_GOOGLE_API_KEY'
-  | 'AI_CODE_REVIEW_OPENROUTER_API_KEY'
-  | 'AI_CODE_REVIEW_ANTHROPIC_API_KEY'
-  | 'AI_CODE_REVIEW_OPENAI_API_KEY'
-  | 'AI_CODE_REVIEW_MODEL'
-  | 'AI_CODE_REVIEW_WRITER_MODEL'
-  | 'AI_CODE_REVIEW_LOG_LEVEL'
-  | 'AI_CODE_REVIEW_OUTPUT_DIR'
-  | 'AI_CODE_REVIEW_CONTEXT'
-  | 'AI_CODE_REVIEW_DIR'
-  | 'AI_CODE_REVIEW_DEBUG'
-  // Legacy environment variables
-  | 'CODE_REVIEW_GOOGLE_API_KEY'
-  | 'CODE_REVIEW_OPENROUTER_API_KEY'
-  | 'CODE_REVIEW_ANTHROPIC_API_KEY'
-  | 'CODE_REVIEW_OPENAI_API_KEY'
-  // Generic environment variables
-  | 'GOOGLE_GENERATIVE_AI_KEY'
-  | 'GOOGLE_AI_STUDIO_KEY'
-  | 'OPENROUTER_API_KEY'
-  | 'ANTHROPIC_API_KEY'
-  | 'OPENAI_API_KEY'
-  // CLI option
-  | 'cli_option'
-  // Default value
-  | 'default_value'
-  // None (not set)
-  | 'none';
-
-/**
- * Configuration value record with its source
- */
-export interface ConfigValue<T> {
-  value: T;
-  source: EnvSource;
-}
-
-/**
- * API keys configuration
- */
-export interface ApiKeysConfig {
-  openRouter?: ConfigValue<string>;
-}
-
-/**
- * API provider types
- */
-export type ApiProvider = 'openrouter';
-
-/**
- * API endpoint configuration
- */
-export interface ApiEndpointsConfig {
-  openRouter: ConfigValue<string>;
-}
-
-/**
- * API version configuration
- */
-export interface ApiVersionsConfig {
-  openRouter: ConfigValue<string>;
-}
-
-/**
- * Log level type
- */
-export type LogLevel = 'debug' | 'info' | 'warn' | 'error' | 'none';
-
-/**
- * Paths configuration
- */
-export interface PathsConfig {
-  outputDir: ConfigValue<string>;
-  promptsDir: ConfigValue<string>;
-  templatesDir: ConfigValue<string>;
-  contextPaths?: ConfigValue<string[]>;
-}
-
-/**
- * Rate limiting configuration
- */
-export interface RateLimitConfig {
-  tokensPerSecond: ConfigValue<number>;
-  maxConcurrentRequests: ConfigValue<number>;
-  retryDelayMs: ConfigValue<number>;
-  maxRetries: ConfigValue<number>;
-}
-
-/**
- * Token configuration
- */
-export interface TokenConfig {
-  maxTokensPerRequest: ConfigValue<number>;
-  contextWindowSize: Record<ApiProvider, ConfigValue<number>>;
-  costPerInputToken: Record<ApiProvider, ConfigValue<number>>;
-  costPerOutputToken: Record<ApiProvider, ConfigValue<number>>;
-}
-
-/**
- * Complete application configuration interface
- */
-export interface ApplicationConfig {
-  // API connections
-  apiKeys: ApiKeysConfig;
-  apiEndpoints: ApiEndpointsConfig;
-  apiVersions: ApiVersionsConfig;
-
-  // Model configuration
-  selectedModel: ConfigValue<string>;
-  writerModel?: ConfigValue<string>;
-  modelProvider: ConfigValue<ApiProvider>;
-
-  // Logging and debugging
-  debug: ConfigValue<boolean>;
-  logLevel: ConfigValue<LogLevel>;
-
-  // File system
-  paths: PathsConfig;
-
-  // Rate limiting and performance
-  rateLimit: RateLimitConfig;
-
-  // Token management
-  tokens: TokenConfig;
-}
-
-/**
- * Zod schema for application configuration validation
- */
-export const applicationConfigSchema = z.object({
-  apiKeys: z.object({
-    openRouter: z.object({ value: z.string().optional(), source: z.string() }).optional(),
-  }),
-
-  apiEndpoints: z.object({
-    openRouter: z.object({ value: z.string(), source: z.string() }),
-  }),
-
-  apiVersions: z.object({
-    openRouter: z.object({ value: z.string(), source: z.string() }),
-  }),
-
-  selectedModel: z.object({ value: z.string(), source: z.string() }),
-  writerModel: z.object({ value: z.string(), source: z.string() }).optional(),
-  modelProvider: z.object({
-    value: z.enum(['openrouter']),
-    source: z.string(),
-  }),
-
-  debug: z.object({ value: z.boolean(), source: z.string() }),
-  logLevel: z.object({
-    value: z.enum(['debug', 'info', 'warn', 'error', 'none']),
-    source: z.string(),
-  }),
-
-  paths: z.object({
-    outputDir: z.object({ value: z.string(), source: z.string() }),
-    promptsDir: z.object({ value: z.string(), source: z.string() }),
-    templatesDir: z.object({ value: z.string(), source: z.string() }),
-    contextPaths: z
-      .object({
-        value: z.array(z.string()),
-        source: z.string(),
-      })
-      .optional(),
-  }),
-
-  rateLimit: z.object({
-    tokensPerSecond: z.object({ value: z.number(), source: z.string() }),
-    maxConcurrentRequests: z.object({ value: z.number(), source: z.string() }),
-    retryDelayMs: z.object({ value: z.number(), source: z.string() }),
-    maxRetries: z.object({ value: z.number(), source: z.string() }),
-  }),
-
-  tokens: z.object({
-    maxTokensPerRequest: z.object({ value: z.number(), source: z.string() }),
-    contextWindowSize: z.object({
-      openrouter: z.object({ value: z.number(), source: z.string() }),
-    }),
-    costPerInputToken: z.object({
-      openrouter: z.object({ value: z.number(), source: z.string() }),
-    }),
-    costPerOutputToken: z.object({
-      openrouter: z.object({ value: z.number(), source: z.string() }),
-    }),
-  }),
-});

package/src/types/handlebars.d.ts

@@ -1,5 +0,0 @@
-/**
- * Minimal declaration so we can compile isolated Handlebars helpers
- * without pulling the entire upstream type package into runtime builds.
- */
-declare type HandlebarsTemplateDelegate<T = any> = (context: T, options?: any) => string;

package/src/types/patch.d.ts

@@ -1,25 +0,0 @@
-/**
- * Local module patches so cr-aia can use optional file paths without rewriting the formatter.
- */
-
-// Allow optional filePath in formatSimpleMarkdown and formatStructuredReviewAsMarkdown
-declare module '../formatters/outputFormatter' {
-  function formatSimpleMarkdown(
-    content: string,
-    filePath: string | undefined,
-    reviewType: string,
-    timestamp: string,
-    costInfo: string,
-    modelInfo: string,
-    metadataSection?: string,
-  ): string;
-
-  function formatStructuredReviewAsMarkdown(
-    structuredReview: import('./structuredReview').StructuredReview,
-    filePath: string | undefined,
-    reviewType: string,
-    timestamp: string,
-    costInfo: string,
-    modelInfo: string,
-  ): string;
-}