codereview-aia 0.1.3 → 0.2.0

package/dist/clients/base/abstractClient.d.ts

@@ -0,0 +1,99 @@
+/**
+ * @fileoverview Abstract base client for interacting with various AI APIs.
+ *
+ * This module defines an abstract base class that encapsulates common functionality
+ * across different AI client implementations (OpenAI, Anthropic, Google, etc.).
+ * It provides a unified interface for model detection, initialization, review generation,
+ * and other shared operations.
+ */
+import type { CostInfo, FileInfo, ReviewOptions, ReviewResult, ReviewType } from '../../types/review';
+import type { ProjectDocs } from '../../utils/projectDocs';
+/**
+ * Abstract base class for AI model clients
+ */
+export declare abstract class AbstractClient {
+    protected modelName: string;
+    protected isInitialized: boolean;
+    /**
+     * Check if the client is initialized
+     * @returns True if initialized, false otherwise
+     */
+    getIsInitialized(): boolean;
+    /**
+     * Check if the provided model name is supported by this client
+     * @param modelName The full model name (potentially with provider prefix)
+     * @returns Object indicating if this is the correct client for the model, and parsed model info
+     */
+    abstract isModelSupported(modelName: string): {
+        isCorrect: boolean;
+        adapter: string;
+        modelName: string;
+    };
+    /**
+     * Initialize the client with the appropriate configuration
+     * @returns Promise resolving to a boolean indicating success
+     */
+    abstract initialize(): Promise<boolean>;
+    /**
+     * Generate a review for a single file
+     * @param fileContent Content of the file to review
+     * @param filePath Path to the file
+     * @param reviewType Type of review to perform
+     * @param projectDocs Optional project documentation
+     * @param options Review options
+     * @returns Promise resolving to the review result
+     */
+    abstract generateReview(fileContent: string, filePath: string, reviewType: ReviewType, projectDocs?: ProjectDocs | null, options?: ReviewOptions): Promise<ReviewResult>;
+    /**
+     * Generate a consolidated review for multiple files
+     * @param files Array of file information objects
+     * @param projectName Name of the project
+     * @param reviewType Type of review to perform
+     * @param projectDocs Optional project documentation
+     * @param options Review options
+     * @returns Promise resolving to the review result
+     */
+    abstract generateConsolidatedReview(files: FileInfo[], projectName: string, reviewType: ReviewType, projectDocs?: ProjectDocs | null, options?: ReviewOptions): Promise<ReviewResult>;
+    /**
+     * Generate an architectural review for a project
+     * @param files Array of file information objects
+     * @param projectName Name of the project
+     * @param projectDocs Optional project documentation
+     * @param options Review options
+     * @returns Promise resolving to the review result
+     */
+    abstract generateArchitecturalReview(files: FileInfo[], projectName: string, projectDocs?: ProjectDocs | null, options?: ReviewOptions): Promise<ReviewResult>;
+    /**
+     * Process the response and extract structured data if possible
+     * @param content The response content to process
+     * @returns The processed structured data or null
+     */
+    protected processResponseForStructuredData(content: string): unknown | null;
+    /**
+     * Create a standard review result object
+     * @param content The review content
+     * @param filePath The file path or identifier
+     * @param reviewType The type of review
+     * @param cost Optional cost information
+     * @returns The standardized review result object
+     */
+    protected createReviewResult(content: string, filePath: string, reviewType: ReviewType, cost?: CostInfo): ReviewResult;
+    /**
+     * Get the full model name including provider prefix
+     * @returns The full model name
+     */
+    protected getFullModelName(): string;
+    /**
+     * Get the provider name for this client
+     * @returns The provider name (e.g., "openai", "anthropic", "gemini")
+     */
+    protected abstract getProviderName(): string;
+    /**
+     * Handle common error cases in API interactions
+     * @param error The error that occurred
+     * @param operation The operation that was being performed
+     * @param filePath The file path or identifier related to the error
+     * @throws The processed error
+     */
+    protected handleApiError(error: unknown, operation: string, filePath: string): never;
+}

package/dist/clients/base/abstractClient.js

@@ -0,0 +1,98 @@
+"use strict";
+/**
+ * @fileoverview Abstract base client for interacting with various AI APIs.
+ *
+ * This module defines an abstract base class that encapsulates common functionality
+ * across different AI client implementations (OpenAI, Anthropic, Google, etc.).
+ * It provides a unified interface for model detection, initialization, review generation,
+ * and other shared operations.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.AbstractClient = void 0;
+const logger_1 = __importDefault(require("../../utils/logger"));
+/**
+ * Abstract base class for AI model clients
+ */
+class AbstractClient {
+    modelName = '';
+    isInitialized = false;
+    /**
+     * Check if the client is initialized
+     * @returns True if initialized, false otherwise
+     */
+    getIsInitialized() {
+        return this.isInitialized;
+    }
+    /**
+     * Process the response and extract structured data if possible
+     * @param content The response content to process
+     * @returns The processed structured data or null
+     */
+    processResponseForStructuredData(content) {
+        try {
+            // First, check if the response is wrapped in a code block
+            const jsonMatch = content.match(/```(?:json)?\s*([\s\S]*?)\s*```/);
+            const jsonContent = jsonMatch ? jsonMatch[1] : content;
+            // Check if the content is valid JSON
+            const structuredData = JSON.parse(jsonContent);
+            // Validate that it has the expected structure
+            if (!structuredData.summary || !Array.isArray(structuredData.issues)) {
+                logger_1.default.warn('Response is valid JSON but does not have the expected structure');
+            }
+            return structuredData;
+        }
+        catch (parseError) {
+            logger_1.default.warn(`Response is not valid JSON: ${parseError instanceof Error ? parseError.message : String(parseError)}`);
+            // Return null for non-JSON content
+            return null;
+        }
+    }
+    /**
+     * Create a standard review result object
+     * @param content The review content
+     * @param filePath The file path or identifier
+     * @param reviewType The type of review
+     * @param cost Optional cost information
+     * @returns The standardized review result object
+     */
+    createReviewResult(content, filePath, reviewType, cost) {
+        const structuredData = this.processResponseForStructuredData(content);
+        return {
+            content,
+            cost,
+            modelUsed: this.getFullModelName(),
+            filePath,
+            reviewType,
+            timestamp: new Date().toISOString(),
+            structuredData: structuredData,
+        };
+    }
+    /**
+     * Get the full model name including provider prefix
+     * @returns The full model name
+     */
+    getFullModelName() {
+        const modelParts = this.modelName.split(':');
+        if (modelParts.length === 2) {
+            return this.modelName; // Already has provider prefix
+        }
+        return `${this.getProviderName()}:${this.modelName}`;
+    }
+    /**
+     * Handle common error cases in API interactions
+     * @param error The error that occurred
+     * @param operation The operation that was being performed
+     * @param filePath The file path or identifier related to the error
+     * @throws The processed error
+     */
+    handleApiError(error, operation, filePath) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logger_1.default.error(`Error ${operation} for ${filePath}: ${errorMessage}`);
+        throw error;
+    }
+}
+exports.AbstractClient = AbstractClient;
+//# sourceMappingURL=abstractClient.js.map

package/dist/clients/base/abstractClient.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"abstractClient.js","sourceRoot":"","sources":["../../../src/clients/base/abstractClient.ts"],"names":[],"mappings":";AAAA;;;;;;;GAOG;;;;;;AAUH,gEAAwC;AAGxC;;GAEG;AACH,MAAsB,cAAc;IACxB,SAAS,GAAG,EAAE,CAAC;IACf,aAAa,GAAG,KAAK,CAAC;IAEhC;;;OAGG;IACI,gBAAgB;QACrB,OAAO,IAAI,CAAC,aAAa,CAAC;IAC5B,CAAC;IAoED;;;;OAIG;IACO,gCAAgC,CAAC,OAAe;QACxD,IAAI,CAAC;YACH,0DAA0D;YAC1D,MAAM,SAAS,GAAG,OAAO,CAAC,KAAK,CAAC,iCAAiC,CAAC,CAAC;YACnE,MAAM,WAAW,GAAG,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC;YAEvD,qCAAqC;YACrC,MAAM,cAAc,GAAG,IAAI,CAAC,KAAK,CAAC,WAAW,CAAC,CAAC;YAE/C,8CAA8C;YAC9C,IAAI,CAAC,cAAc,CAAC,OAAO,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,cAAc,CAAC,MAAM,CAAC,EAAE,CAAC;gBACrE,gBAAM,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAAC;YACjF,CAAC;YAED,OAAO,cAAc,CAAC;QACxB,CAAC;QAAC,OAAO,UAAU,EAAE,CAAC;YACpB,gBAAM,CAAC,IAAI,CACT,+BACE,UAAU,YAAY,KAAK,CAAC,CAAC,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,UAAU,CACtE,EAAE,CACH,CAAC;YACF,mCAAmC;YACnC,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED;;;;;;;OAOG;IACO,kBAAkB,CAC1B,OAAe,EACf,QAAgB,EAChB,UAAsB,EACtB,IAAe;QAEf,MAAM,cAAc,GAAG,IAAI,CAAC,gCAAgC,CAAC,OAAO,CAAC,CAAC;QAEtE,OAAO;YACL,OAAO;YACP,IAAI;YACJ,SAAS,EAAE,IAAI,CAAC,gBAAgB,EAAE;YAClC,QAAQ;YACR,UAAU;YACV,SAAS,EAAE,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE;YACnC,cAAc,EAAE,cAA8C;SAC/D,CAAC;IACJ,CAAC;IAED;;;OAGG;IACO,gBAAgB;QACxB,MAAM,UAAU,GAAG,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC;QAC7C,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC5B,OAAO,IAAI,CAAC,SAAS,CAAC,CAAC,8BAA8B;QACvD,CAAC;QACD,OAAO,GAAG,IAAI,CAAC,eAAe,EAAE,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;IACvD,CAAC;IAQD;;;;;;OAMG;IACO,cAAc,CAAC,KAAc,EAAE,SAAiB,EAAE,QAAgB;QAC1E,MAAM,YAAY,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QAC5E,gBAAM,CAAC,KAAK,CAAC,SAAS,SAAS,QAAQ,QAAQ,KAAK,YAAY,EAAE,CAAC,CAAC;QACpE,MAAM,KAAK,CAAC;IACd,CAAC;CACF;AAtKD,wCAsKC"}

package/dist/clients/base/httpClient.d.ts

@@ -0,0 +1,24 @@
+/**
+ * @fileoverview HTTP client utilities for API interactions.
+ *
+ * This module provides common HTTP request handling functionality with retry logic,
+ * error handling, and timeout management. It's designed to be used by the various
+ * AI API client implementations.
+ */
+/**
+ * Fetch with automatic retry for transient errors
+ * @param url The URL to fetch from
+ * @param options Request options
+ * @param retries Number of retries to attempt
+ * @returns Promise resolving to the Response object
+ * @throws Error if all retries fail
+ */
+export declare function fetchWithRetry(url: string, options: RequestInit, retries?: number): Promise<Response>;
+/**
+ * Generic retry mechanism for any asynchronous function
+ * @param fn The function to retry
+ * @param retries Maximum number of retries
+ * @returns Promise resolving to the function's result
+ * @throws Error if all retries fail
+ */
+export declare function withRetry<T>(fn: () => Promise<T>, retries?: number): Promise<T>;

package/dist/clients/base/httpClient.js

@@ -0,0 +1,147 @@
+"use strict";
+/**
+ * @fileoverview HTTP client utilities for API interactions.
+ *
+ * This module provides common HTTP request handling functionality with retry logic,
+ * error handling, and timeout management. It's designed to be used by the various
+ * AI API client implementations.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.fetchWithRetry = fetchWithRetry;
+exports.withRetry = withRetry;
+const apiErrorHandler_1 = require("../../utils/apiErrorHandler");
+const logger_1 = __importDefault(require("../../utils/logger"));
+/**
+ * Fetch with automatic retry for transient errors
+ * @param url The URL to fetch from
+ * @param options Request options
+ * @param retries Number of retries to attempt
+ * @returns Promise resolving to the Response object
+ * @throws Error if all retries fail
+ */
+async function fetchWithRetry(url, options, retries = 3) {
+    logger_1.default.debug(`[FETCH DEBUG] fetchWithRetry called with url: ${url}`);
+    for (let i = 0; i < retries; i++) {
+        try {
+            logger_1.default.debug(`Making API request to ${url} (attempt ${i + 1}/${retries})`);
+            const res = await fetch(url, options);
+            if (res.ok) {
+                // Log successful response headers for debugging (OpenRouter specific)
+                const requestId = res.headers.get('x-request-id') || res.headers.get('request-id');
+                if (requestId) {
+                    logger_1.default.debug(`Successful API response - Request ID: ${requestId}`);
+                }
+                return res;
+            }
+            logger_1.default.error(`[FETCH DEBUG] Request failed with status: ${res.status}`);
+            // Handle rate limiting and server errors with exponential backoff
+            if (res.status === 429 || res.status >= 500) {
+                const retryAfter = res.headers.get('Retry-After');
+                const delayMs = retryAfter ? parseInt(retryAfter, 10) * 1000 : 1000 * 2 ** i;
+                logger_1.default.warn(`Request failed with status ${res.status}. Retrying in ${delayMs / 1000}s...`);
+                await new Promise((r) => setTimeout(r, delayMs));
+            }
+            else {
+                // For other errors, try to get more detailed information
+                try {
+                    const errorBody = await res.json();
+                    const requestId = res.headers.get('x-request-id') || res.headers.get('request-id');
+                    logger_1.default.error(`API error response: ${JSON.stringify(errorBody, null, 2)}`);
+                    logger_1.default.error(`Request URL: ${url}`);
+                    logger_1.default.error(`Request headers: ${JSON.stringify(options.headers, null, 2)}`);
+                    // Log request body if it exists (but mask API keys)
+                    let modelInfo = '';
+                    if (options.body) {
+                        try {
+                            const body = JSON.parse(options.body);
+                            if (body.messages) {
+                                // Only log message structure, not content
+                                logger_1.default.error(`Request had ${body.messages.length} messages`);
+                            }
+                            modelInfo = body.model || '';
+                            logger_1.default.error(`Request model: ${body.model}`);
+                            logger_1.default.error(`Request max_tokens: ${body.max_tokens}`);
+                            logger_1.default.error(`Request max_completion_tokens: ${body.max_completion_tokens}`);
+                        }
+                        catch {
+                            // If body isn't JSON, just note that
+                            logger_1.default.error('Request body was not JSON');
+                        }
+                    }
+                    // Create detailed error message
+                    const errorDetails = [
+                        `API request failed`,
+                        `Endpoint: ${url}`,
+                        `Status: ${res.status}`,
+                        `Error: ${errorBody.error?.message || JSON.stringify(errorBody)}`,
+                    ];
+                    if (requestId) {
+                        errorDetails.push(`Request ID: ${requestId}`);
+                    }
+                    if (modelInfo) {
+                        errorDetails.push(`Model: ${modelInfo}`);
+                    }
+                    throw new apiErrorHandler_1.ApiError(errorDetails.join('\n  '));
+                }
+                catch (parseError) {
+                    // If we can't parse the error response, just use the status
+                    if (parseError instanceof apiErrorHandler_1.ApiError) {
+                        throw parseError;
+                    }
+                    logger_1.default.error(`Failed to parse error response, status: ${res.status}`);
+                    throw new apiErrorHandler_1.ApiError(`API request failed\n  Endpoint: ${url}\n  Status: ${res.status}\n  Error: Unable to parse error response`);
+                }
+            }
+        }
+        catch (error) {
+            // For network errors or other exceptions, retry if we have retries left
+            if (i < retries - 1 && !(error instanceof apiErrorHandler_1.ApiError)) {
+                const delayMs = 1000 * 2 ** i;
+                logger_1.default.warn(`Request failed with error: ${error instanceof Error ? error.message : String(error)}. Retrying in ${delayMs / 1000}s...`);
+                await new Promise((r) => setTimeout(r, delayMs));
+            }
+            else {
+                throw error;
+            }
+        }
+    }
+    throw new apiErrorHandler_1.ApiError(`Failed after ${retries} retries`);
+}
+/**
+ * Generic retry mechanism for any asynchronous function
+ * @param fn The function to retry
+ * @param retries Maximum number of retries
+ * @returns Promise resolving to the function's result
+ * @throws Error if all retries fail
+ */
+async function withRetry(fn, retries = 3) {
+    for (let i = 0; i < retries; i++) {
+        try {
+            return await fn();
+        }
+        catch (error) {
+            // Type guard for errors with status property
+            const err = error;
+            // Handle rate limiting and server errors
+            if ((err.status === 429 || (err.status && err.status >= 500)) && i < retries - 1) {
+                const delayMs = 1000 * 2 ** i;
+                logger_1.default.warn(`Operation failed with status ${err.status}. Retrying in ${delayMs / 1000}s...`);
+                await new Promise((r) => setTimeout(r, delayMs));
+            }
+            else if (i < retries - 1) {
+                // For other errors, retry with backoff
+                const delayMs = 1000 * 2 ** i;
+                logger_1.default.warn(`Operation failed: ${error instanceof Error ? error.message : String(error)}. Retrying in ${delayMs / 1000}s...`);
+                await new Promise((r) => setTimeout(r, delayMs));
+            }
+            else {
+                throw error;
+            }
+        }
+    }
+    throw new apiErrorHandler_1.ApiError(`Failed after ${retries} retries`);
+}
+//# sourceMappingURL=httpClient.js.map

package/dist/clients/base/httpClient.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"httpClient.js","sourceRoot":"","sources":["../../../src/clients/base/httpClient.ts"],"names":[],"mappings":";AAAA;;;;;;GAMG;;;;;AAaH,wCAsGC;AASD,8BA6BC;AAvJD,iEAAuD;AACvD,gEAAwC;AAExC;;;;;;;GAOG;AACI,KAAK,UAAU,cAAc,CAClC,GAAW,EACX,OAAoB,EACpB,OAAO,GAAG,CAAC;IAEX,gBAAM,CAAC,KAAK,CAAC,iDAAiD,GAAG,EAAE,CAAC,CAAC;IACrE,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,OAAO,EAAE,CAAC,EAAE,EAAE,CAAC;QACjC,IAAI,CAAC;YACH,gBAAM,CAAC,KAAK,CAAC,yBAAyB,GAAG,aAAa,CAAC,GAAG,CAAC,IAAI,OAAO,GAAG,CAAC,CAAC;YAC3E,MAAM,GAAG,GAAG,MAAM,KAAK,CAAC,GAAG,EAAE,OAAO,CAAC,CAAC;YAEtC,IAAI,GAAG,CAAC,EAAE,EAAE,CAAC;gBACX,sEAAsE;gBACtE,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,IAAI,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;gBACnF,IAAI,SAAS,EAAE,CAAC;oBACd,gBAAM,CAAC,KAAK,CAAC,yCAAyC,SAAS,EAAE,CAAC,CAAC;gBACrE,CAAC;gBACD,OAAO,GAAG,CAAC;YACb,CAAC;YAED,gBAAM,CAAC,KAAK,CAAC,6CAA6C,GAAG,CAAC,MAAM,EAAE,CAAC,CAAC;YAExE,kEAAkE;YAClE,IAAI,GAAG,CAAC,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,MAAM,IAAI,GAAG,EAAE,CAAC;gBAC5C,MAAM,UAAU,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;gBAClD,MAAM,OAAO,GAAG,UAAU,CAAC,CAAC,CAAC,QAAQ,CAAC,UAAU,EAAE,EAAE,CAAC,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC,IAAI,CAAC,CAAC;gBAE7E,gBAAM,CAAC,IAAI,CAAC,8BAA8B,GAAG,CAAC,MAAM,iBAAiB,OAAO,GAAG,IAAI,MAAM,CAAC,CAAC;gBAC3F,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC;YACnD,CAAC;iBAAM,CAAC;gBACN,yDAAyD;gBACzD,IAAI,CAAC;oBACH,MAAM,SAAS,GAAG,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;oBACnC,MAAM,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,cAAc,CAAC,IAAI,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;oBAEnF,gBAAM,CAAC,KAAK,CAAC,uBAAuB,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;oBAC1E,gBAAM,CAAC,KAAK,CAAC,gBAAgB,GAAG,EAAE,CAAC,CAAC;oBACpC,gBAAM,CAAC,KAAK,CAAC,oBAAoB,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,EAAE,CAAC,CAAC;oBAE7E,oDAAoD;oBACpD,IAAI,SAAS,GAAG,EAAE,CAAC;oBACnB,IAAI,OAAO,CAAC,IAAI,EAAE,CAAC;wBACjB,IAAI,CAAC;4BACH,MAAM,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,IAAc,CAAC,CAAC;4BAChD,IAAI,IAAI,CAAC,QAAQ,EAAE,CAAC;gCAClB,0CAA0C;gCAC1C,gBAAM,CAAC,KAAK,CAAC,eAAe,IAAI,CAAC,QAAQ,CAAC,MAAM,WAAW,CAAC,CAAC;4BAC/D,CAAC;4BACD,SAAS,GAAG,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC;4BAC7B,gBAAM,CAAC,KAAK,CAAC,kBAAkB,IAAI,CAAC,KAAK,EAAE,CAAC,CAAC;4BAC7C,gBAAM,CAAC,KAAK,CAAC,uBAAuB,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC;4BACvD,gBAAM,CAAC,KAAK,CAAC,kCAAkC,IAAI,CAAC,qBAAqB,EAAE,CAAC,CAAC;wBAC/E,CAAC;wBAAC,MAAM,CAAC;4BACP,qCAAqC;4BACrC,gBAAM,CAAC,KAAK,CAAC,2BAA2B,CAAC,CAAC;wBAC5C,CAAC;oBACH,CAAC;oBAED,gCAAgC;oBAChC,MAAM,YAAY,GAAa;wBAC7B,oBAAoB;wBACpB,aAAa,GAAG,EAAE;wBAClB,WAAW,GAAG,CAAC,MAAM,EAAE;wBACvB,UAAU,SAAS,CAAC,KAAK,EAAE,OAAO,IAAI,IAAI,CAAC,SAAS,CAAC,SAAS,CAAC,EAAE;qBAClE,CAAC;oBAEF,IAAI,SAAS,EAAE,CAAC;wBACd,YAAY,CAAC,IAAI,CAAC,eAAe,SAAS,EAAE,CAAC,CAAC;oBAChD,CAAC;oBAED,IAAI,SAAS,EAAE,CAAC;wBACd,YAAY,CAAC,IAAI,CAAC,UAAU,SAAS,EAAE,CAAC,CAAC;oBAC3C,CAAC;oBAED,MAAM,IAAI,0BAAQ,CAAC,YAAY,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC;gBAChD,CAAC;gBAAC,OAAO,UAAU,EAAE,CAAC;oBACpB,4DAA4D;oBAC5D,IAAI,UAAU,YAAY,0BAAQ,EAAE,CAAC;wBACnC,MAAM,UAAU,CAAC;oBACnB,CAAC;oBAED,gBAAM,CAAC,KAAK,CAAC,2CAA2C,GAAG,CAAC,MAAM,EAAE,CAAC,CAAC;oBACtE,MAAM,IAAI,0BAAQ,CAChB,mCAAmC,GAAG,eAAe,GAAG,CAAC,MAAM,2CAA2C,CAC3G,CAAC;gBACJ,CAAC;YACH,CAAC;QACH,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,wEAAwE;YACxE,IAAI,CAAC,GAAG,OAAO,GAAG,CAAC,IAAI,CAAC,CAAC,KAAK,YAAY,0BAAQ,CAAC,EAAE,CAAC;gBACpD,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC9B,gBAAM,CAAC,IAAI,CACT,8BAA8B,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,iBAAiB,OAAO,GAAG,IAAI,MAAM,CAC1H,CAAC;gBACF,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC;YACnD,CAAC;iBAAM,CAAC;gBACN,MAAM,KAAK,CAAC;YACd,CAAC;QACH,CAAC;IACH,CAAC;IAED,MAAM,IAAI,0BAAQ,CAAC,gBAAgB,OAAO,UAAU,CAAC,CAAC;AACxD,CAAC;AAED;;;;;;GAMG;AACI,KAAK,UAAU,SAAS,CAAI,EAAoB,EAAE,OAAO,GAAG,CAAC;IAClE,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,OAAO,EAAE,CAAC,EAAE,EAAE,CAAC;QACjC,IAAI,CAAC;YACH,OAAO,MAAM,EAAE,EAAE,CAAC;QACpB,CAAC;QAAC,OAAO,KAAc,EAAE,CAAC;YACxB,6CAA6C;YAC7C,MAAM,GAAG,GAAG,KAA4B,CAAC;YAEzC,yCAAyC;YACzC,IAAI,CAAC,GAAG,CAAC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,GAAG,OAAO,GAAG,CAAC,EAAE,CAAC;gBACjF,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC9B,gBAAM,CAAC,IAAI,CACT,gCAAgC,GAAG,CAAC,MAAM,iBAAiB,OAAO,GAAG,IAAI,MAAM,CAChF,CAAC;gBACF,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC;YACnD,CAAC;iBAAM,IAAI,CAAC,GAAG,OAAO,GAAG,CAAC,EAAE,CAAC;gBAC3B,uCAAuC;gBACvC,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,CAAC;gBAC9B,gBAAM,CAAC,IAAI,CACT,qBAAqB,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,iBAAiB,OAAO,GAAG,IAAI,MAAM,CACjH,CAAC;gBACF,MAAM,IAAI,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,EAAE,OAAO,CAAC,CAAC,CAAC;YACnD,CAAC;iBAAM,CAAC;gBACN,MAAM,KAAK,CAAC;YACd,CAAC;QACH,CAAC;IACH,CAAC;IAED,MAAM,IAAI,0BAAQ,CAAC,gBAAgB,OAAO,UAAU,CAAC,CAAC;AACxD,CAAC"}

package/{src/clients/base/index.ts → dist/clients/base/index.d.ts}

@@ -5,7 +5,6 @@
  * providing a unified interface for client implementation, HTTP requests,
  * response processing, and model detection.
  */
-
 export * from './abstractClient';
 export * from './httpClient';
 export * from './modelDetection';

package/dist/clients/base/index.js

@@ -0,0 +1,28 @@
+"use strict";
+/**
+ * @fileoverview Common base functionality for AI API clients.
+ *
+ * This module serves as the main entry point for the base client functionality,
+ * providing a unified interface for client implementation, HTTP requests,
+ * response processing, and model detection.
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __exportStar = (this && this.__exportStar) || function(m, exports) {
+    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+__exportStar(require("./abstractClient"), exports);
+__exportStar(require("./httpClient"), exports);
+__exportStar(require("./modelDetection"), exports);
+__exportStar(require("./responseProcessor"), exports);
+//# sourceMappingURL=index.js.map

package/dist/clients/base/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/clients/base/index.ts"],"names":[],"mappings":";AAAA;;;;;;GAMG;;;;;;;;;;;;;;;;AAEH,mDAAiC;AACjC,+CAA6B;AAC7B,mDAAiC;AACjC,sDAAoC"}

package/dist/clients/base/modelDetection.d.ts

@@ -0,0 +1,41 @@
+/**
+ * @fileoverview Model detection and initialization utilities for AI clients.
+ *
+ * This module provides common functionality for detecting model providers,
+ * parsing model names, validating API keys, and initializing client instances.
+ */
+/**
+ * Result of model detection
+ */
+export interface ModelDetectionResult {
+    /** Whether this is the correct client for the model */
+    isCorrect: boolean;
+    /** The provider/adapter name (e.g., "openai", "anthropic", "gemini") */
+    adapter: string;
+    /** The specific model name without provider prefix */
+    modelName: string;
+}
+/**
+ * Parse a model name string into provider and model components
+ * @param modelNameString The model name string (possibly with provider prefix)
+ * @param defaultProvider The default provider to use if none is specified
+ * @returns Object with adapter and modelName properties
+ */
+export declare function parseModelName(modelNameString: string, defaultProvider: string): {
+    adapter: string;
+    modelName: string;
+};
+/**
+ * Detect if a model name is for a specific provider
+ * @param providerName The provider to check for
+ * @param modelNameString The model name to check (with or without provider prefix)
+ * @returns ModelDetectionResult with detection information
+ */
+export declare function detectModelProvider(providerName: string, modelNameString?: string): ModelDetectionResult;
+/**
+ * Validate API key for a provider
+ * @param providerName The provider to validate the API key for
+ * @param apiKeyName Environment variable name for the API key (optional)
+ * @returns Whether the API key is valid
+ */
+export declare function validateApiKey(providerName: string, apiKeyName?: string): boolean;

package/dist/clients/base/modelDetection.js

@@ -0,0 +1,88 @@
+"use strict";
+/**
+ * @fileoverview Model detection and initialization utilities for AI clients.
+ *
+ * This module provides common functionality for detecting model providers,
+ * parsing model names, validating API keys, and initializing client instances.
+ */
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.parseModelName = parseModelName;
+exports.detectModelProvider = detectModelProvider;
+exports.validateApiKey = validateApiKey;
+const ConfigurationService_1 = require("../../core/ConfigurationService");
+const logger_1 = __importDefault(require("../../utils/logger"));
+/**
+ * Parse a model name string into provider and model components
+ * @param modelNameString The model name string (possibly with provider prefix)
+ * @param defaultProvider The default provider to use if none is specified
+ * @returns Object with adapter and modelName properties
+ */
+function parseModelName(modelNameString, defaultProvider) {
+    if (!modelNameString) {
+        return { adapter: defaultProvider, modelName: '' };
+    }
+    // Parse the model name
+    return modelNameString.includes(':')
+        ? {
+            adapter: modelNameString.split(':')[0],
+            modelName: modelNameString.split(':')[1],
+        }
+        : {
+            adapter: defaultProvider,
+            modelName: modelNameString,
+        };
+}
+/**
+ * Detect if a model name is for a specific provider
+ * @param providerName The provider to check for
+ * @param modelNameString The model name to check (with or without provider prefix)
+ * @returns ModelDetectionResult with detection information
+ */
+function detectModelProvider(providerName, modelNameString) {
+    // Get the model from configuration if not provided
+    const config = (0, ConfigurationService_1.getConfig)();
+    const selectedModel = modelNameString || config.model || '';
+    // Parse the model name
+    const { adapter, modelName } = parseModelName(selectedModel, providerName);
+    return {
+        isCorrect: adapter === providerName,
+        adapter,
+        modelName,
+    };
+}
+/**
+ * Validate API key for a provider
+ * @param providerName The provider to validate the API key for
+ * @param apiKeyName Environment variable name for the API key (optional)
+ * @returns Whether the API key is valid
+ */
+function validateApiKey(providerName, apiKeyName) {
+    // Only support OpenRouter
+    if (providerName.toLowerCase() !== 'openrouter') {
+        logger_1.default.error(`Provider ${providerName} is not supported. Only OpenRouter is supported.`);
+        return false;
+    }
+    // Try to get the API key from the config
+    const config = (0, ConfigurationService_1.getConfig)();
+    const apiKey = config.openRouterApiKey;
+    // Check if we have an API key
+    if (!apiKey) {
+        const envVarName = apiKeyName || 'AI_CODE_REVIEW_OPENROUTER_API_KEY';
+        logger_1.default.error(`No OpenRouter API key found in configuration.`);
+        logger_1.default.error('Please add the following to your .env.local file:');
+        logger_1.default.error(`- ${envVarName}=your_openrouter_api_key_here`);
+        return false;
+    }
+    // Log API key status
+    if (config.debug) {
+        logger_1.default.info(`Using real OpenRouter API responses.`);
+    }
+    else {
+        logger_1.default.info(`API key found for OpenRouter. Using real API responses.`);
+    }
+    return true;
+}
+//# sourceMappingURL=modelDetection.js.map

package/dist/clients/base/modelDetection.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"modelDetection.js","sourceRoot":"","sources":["../../../src/clients/base/modelDetection.ts"],"names":[],"mappings":";AAAA;;;;;GAKG;;;;;AAuBH,wCAkBC;AAQD,kDAgBC;AAQD,wCA4BC;AAnGD,0EAA4D;AAC5D,gEAAwC;AAcxC;;;;;GAKG;AACH,SAAgB,cAAc,CAC5B,eAAuB,EACvB,eAAuB;IAEvB,IAAI,CAAC,eAAe,EAAE,CAAC;QACrB,OAAO,EAAE,OAAO,EAAE,eAAe,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC;IACrD,CAAC;IAED,uBAAuB;IACvB,OAAO,eAAe,CAAC,QAAQ,CAAC,GAAG,CAAC;QAClC,CAAC,CAAC;YACE,OAAO,EAAE,eAAe,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;YACtC,SAAS,EAAE,eAAe,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;SACzC;QACH,CAAC,CAAC;YACE,OAAO,EAAE,eAAe;YACxB,SAAS,EAAE,eAAe;SAC3B,CAAC;AACR,CAAC;AAED;;;;;GAKG;AACH,SAAgB,mBAAmB,CACjC,YAAoB,EACpB,eAAwB;IAExB,mDAAmD;IACnD,MAAM,MAAM,GAAG,IAAA,gCAAS,GAAE,CAAC;IAC3B,MAAM,aAAa,GAAG,eAAe,IAAI,MAAM,CAAC,KAAK,IAAI,EAAE,CAAC;IAE5D,uBAAuB;IACvB,MAAM,EAAE,OAAO,EAAE,SAAS,EAAE,GAAG,cAAc,CAAC,aAAa,EAAE,YAAY,CAAC,CAAC;IAE3E,OAAO;QACL,SAAS,EAAE,OAAO,KAAK,YAAY;QACnC,OAAO;QACP,SAAS;KACV,CAAC;AACJ,CAAC;AAED;;;;;GAKG;AACH,SAAgB,cAAc,CAAC,YAAoB,EAAE,UAAmB;IACtE,0BAA0B;IAC1B,IAAI,YAAY,CAAC,WAAW,EAAE,KAAK,YAAY,EAAE,CAAC;QAChD,gBAAM,CAAC,KAAK,CAAC,YAAY,YAAY,kDAAkD,CAAC,CAAC;QACzF,OAAO,KAAK,CAAC;IACf,CAAC;IAED,yCAAyC;IACzC,MAAM,MAAM,GAAG,IAAA,gCAAS,GAAE,CAAC;IAC3B,MAAM,MAAM,GAAG,MAAM,CAAC,gBAAgB,CAAC;IAEvC,8BAA8B;IAC9B,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,UAAU,GAAG,UAAU,IAAI,mCAAmC,CAAC;QACrE,gBAAM,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC;QAC9D,gBAAM,CAAC,KAAK,CAAC,mDAAmD,CAAC,CAAC;QAClE,gBAAM,CAAC,KAAK,CAAC,KAAK,UAAU,+BAA+B,CAAC,CAAC;QAC7D,OAAO,KAAK,CAAC;IACf,CAAC;IAED,qBAAqB;IACrB,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;QACjB,gBAAM,CAAC,IAAI,CAAC,sCAAsC,CAAC,CAAC;IACtD,CAAC;SAAM,CAAC;QACN,gBAAM,CAAC,IAAI,CAAC,yDAAyD,CAAC,CAAC;IACzE,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC"}

package/dist/clients/base/responseProcessor.d.ts

@@ -0,0 +1,45 @@
+/**
+ * @fileoverview Common response processing and error handling for AI API clients.
+ *
+ * This module provides shared functionality for processing API responses,
+ * extracting structured data, handling errors, and standardizing output formats
+ * across different AI providers.
+ */
+import type { ReviewResult, ReviewType } from '../../types/review';
+import type { StructuredReview } from '../../types/structuredReview';
+import { ApiError } from '../../utils/apiErrorHandler';
+/**
+ * Process API response content and extract structured data if possible
+ * @param content The API response content
+ * @returns Structured data object or null if not valid JSON
+ */
+export declare function extractStructuredData(content: string): StructuredReview | undefined;
+/**
+ * Create a standardized review result object
+ * @param content The review content
+ * @param prompt The original prompt (for cost calculation)
+ * @param modelName The full model name
+ * @param filePath The file path or identifier
+ * @param reviewType The review type
+ * @param options Optional review options to determine expected output format
+ * @returns Standardized review result
+ */
+export declare function createStandardReviewResult(content: string, prompt: string, modelName: string, filePath: string, reviewType: ReviewType, options?: {
+    interactive?: boolean;
+    output?: string;
+}): ReviewResult;
+/**
+ * Handle API errors with standardized logging and wrapping
+ * @param error The original error
+ * @param operation Description of the operation that failed
+ * @param modelName The model being used
+ * @param context Additional context for debugging
+ * @returns Wrapped ApiError
+ */
+export declare function handleApiError(error: unknown, operation: string, modelName: string, context?: {
+    endpoint?: string;
+    statusCode?: number;
+    requestId?: string;
+    filePath?: string;
+    additionalInfo?: Record<string, any>;
+}): ApiError;