@juspay/yama 1.1.0 → 1.2.0

package/dist/features/CodeReviewer.js

@@ -1,15 +1,16 @@
-"use strict";
 /**
  * Enhanced Code Reviewer - Optimized to work with Unified Context
  * Preserves all original functionality from pr-police.js but optimized
  */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.CodeReviewer = void 0;
-exports.createCodeReviewer = createCodeReviewer;
 // NeuroLink will be dynamically imported
-const types_1 = require("../types");
-const Logger_1 = require("../utils/Logger");
-class CodeReviewer {
+import { ProviderError, } from "../types/index.js";
+import { logger } from "../utils/Logger.js";
+import { getProviderTokenLimit } from "../utils/ProviderLimits.js";
+export class CodeReviewer {
+    neurolink;
+    bitbucketProvider;
+    aiConfig;
+    reviewConfig;
     constructor(bitbucketProvider, aiConfig, reviewConfig) {
         this.bitbucketProvider = bitbucketProvider;
         this.aiConfig = aiConfig;
@@ -21,8 +22,8 @@ class CodeReviewer {
     async reviewCodeWithContext(context, options) {
         const startTime = Date.now();
         try {
-            Logger_1.logger.phase('🧪 Conducting AI-powered code analysis...');
-            Logger_1.logger.info(`Analyzing ${context.diffStrategy.fileCount} files using ${context.diffStrategy.strategy} strategy`);
+            logger.phase("🧪 Conducting AI-powered code analysis...");
+            logger.info(`Analyzing ${context.diffStrategy.fileCount} files using ${context.diffStrategy.strategy} strategy`);
             const analysisPrompt = this.buildAnalysisPrompt(context, options);
             const violations = await this.analyzeWithAI(analysisPrompt, context);
             const validatedViolations = this.validateViolations(violations, context);
@@ -31,12 +32,12 @@ class CodeReviewer {
             }
             const duration = Math.round((Date.now() - startTime) / 1000);
             const result = this.generateReviewResult(validatedViolations, duration, context);
-            Logger_1.logger.success(`Code review completed in ${duration}s: ${validatedViolations.length} violations found`);
+            logger.success(`Code review completed in ${duration}s: ${validatedViolations.length} violations found`);
             return result;
         }
         catch (error) {
-            Logger_1.logger.error(`Code review failed: ${error.message}`);
-            throw new types_1.ProviderError(`Code review failed: ${error.message}`);
+            logger.error(`Code review failed: ${error.message}`);
+            throw new ProviderError(`Code review failed: ${error.message}`);
         }
     }
     /**
@@ -46,7 +47,9 @@ class CodeReviewer {
         const validatedViolations = [];
         const diffContent = this.extractDiffContent(context);
         for (const violation of violations) {
-            if (violation.type === 'inline' && violation.code_snippet && violation.file) {
+            if (violation.type === "inline" &&
+                violation.code_snippet &&
+                violation.file) {
                 // Check if the code snippet exists in the diff
                 if (diffContent.includes(violation.code_snippet)) {
                     validatedViolations.push(violation);
@@ -58,8 +61,8 @@ class CodeReviewer {
                         validatedViolations.push(fixedViolation);
                     }
                     else {
-                        Logger_1.logger.debug(`⚠️ Skipping violation - snippet not found in diff: ${violation.file}`);
-                        Logger_1.logger.debug(`   Original snippet: "${violation.code_snippet}"`);
+                        logger.debug(`⚠️ Skipping violation - snippet not found in diff: ${violation.file}`);
+                        logger.debug(`   Original snippet: "${violation.code_snippet}"`);
                     }
                 }
             }
@@ -68,50 +71,56 @@ class CodeReviewer {
                 validatedViolations.push(violation);
             }
         }
-        Logger_1.logger.debug(`Validated ${validatedViolations.length} out of ${violations.length} violations`);
+        logger.debug(`Validated ${validatedViolations.length} out of ${violations.length} violations`);
         return validatedViolations;
     }
     /**
      * Try to fix code snippet by finding it in the actual diff
      */
     tryFixCodeSnippet(violation, context) {
-        if (!violation.file || !violation.code_snippet)
+        if (!violation.file || !violation.code_snippet) {
             return null;
+        }
         try {
             // Get the diff for this specific file
             let fileDiff;
-            if (context.diffStrategy.strategy === 'whole' && context.prDiff) {
+            if (context.diffStrategy.strategy === "whole" && context.prDiff) {
                 // Extract file diff from whole diff - handle different path formats
-                const diffLines = context.prDiff.diff.split('\n');
+                const diffLines = context.prDiff.diff.split("\n");
                 let fileStartIndex = -1;
                 // Generate all possible path variations
                 const pathVariations = this.generatePathVariations(violation.file);
                 // Try to find the file in the diff with various path formats
                 for (let i = 0; i < diffLines.length; i++) {
                     const line = diffLines[i];
-                    if (line.startsWith('diff --git') || line.startsWith('Index:')) {
+                    if (line.startsWith("diff --git") || line.startsWith("Index:")) {
                         for (const pathVariation of pathVariations) {
                             if (line.includes(pathVariation)) {
                                 fileStartIndex = i;
                                 break;
                             }
                         }
-                        if (fileStartIndex >= 0)
+                        if (fileStartIndex >= 0) {
                             break;
+                        }
                     }
                 }
                 if (fileStartIndex >= 0) {
-                    const nextFileIndex = diffLines.findIndex((line, idx) => idx > fileStartIndex && (line.startsWith('diff --git') || line.startsWith('Index:')));
-                    fileDiff = diffLines.slice(fileStartIndex, nextFileIndex > 0 ? nextFileIndex : diffLines.length).join('\n');
+                    const nextFileIndex = diffLines.findIndex((line, idx) => idx > fileStartIndex &&
+                        (line.startsWith("diff --git") || line.startsWith("Index:")));
+                    fileDiff = diffLines
+                        .slice(fileStartIndex, nextFileIndex > 0 ? nextFileIndex : diffLines.length)
+                        .join("\n");
                 }
             }
-            else if (context.diffStrategy.strategy === 'file-by-file' && context.fileDiffs) {
+            else if (context.diffStrategy.strategy === "file-by-file" &&
+                context.fileDiffs) {
                 // Try all path variations
                 const pathVariations = this.generatePathVariations(violation.file);
                 for (const path of pathVariations) {
                     fileDiff = context.fileDiffs.get(path);
                     if (fileDiff) {
-                        Logger_1.logger.debug(`Found diff for ${violation.file} using variation: ${path}`);
+                        logger.debug(`Found diff for ${violation.file} using variation: ${path}`);
                         break;
                     }
                 }
@@ -120,14 +129,14 @@ class CodeReviewer {
                     for (const [key, value] of context.fileDiffs.entries()) {
                         if (key.endsWith(violation.file) || violation.file.endsWith(key)) {
                             fileDiff = value;
-                            Logger_1.logger.debug(`Found diff for ${violation.file} using partial match: ${key}`);
+                            logger.debug(`Found diff for ${violation.file} using partial match: ${key}`);
                             break;
                         }
                     }
                 }
             }
             if (!fileDiff) {
-                Logger_1.logger.debug(`❌ Could not find diff for file: ${violation.file}`);
+                logger.debug(`❌ Could not find diff for file: ${violation.file}`);
                 return null;
             }
             // First, try to find the exact line with line number extraction
@@ -136,27 +145,28 @@ class CodeReviewer {
                 const fixedViolation = { ...violation };
                 fixedViolation.line_type = lineInfo.lineType;
                 // Extract search context from the diff
-                const diffLines = fileDiff.split('\n');
-                const snippetIndex = diffLines.findIndex(line => line === violation.code_snippet);
+                const diffLines = fileDiff.split("\n");
+                const snippetIndex = diffLines.findIndex((line) => line === violation.code_snippet);
                 if (snippetIndex > 0 && snippetIndex < diffLines.length - 1) {
                     fixedViolation.search_context = {
                         before: [diffLines[snippetIndex - 1]],
-                        after: [diffLines[snippetIndex + 1]]
+                        after: [diffLines[snippetIndex + 1]],
                     };
                 }
-                Logger_1.logger.debug(`✅ Found exact match with line number for ${violation.file}`);
+                logger.debug(`✅ Found exact match with line number for ${violation.file}`);
                 return fixedViolation;
             }
             // Fallback: Clean the snippet and try fuzzy matching
             const cleanSnippet = violation.code_snippet
                 .trim()
-                .replace(/^[+\-\s]/, ''); // Remove diff prefix for searching
+                .replace(/^[+\-\s]/, ""); // Remove diff prefix for searching
             // Look for the clean snippet in the diff
-            const diffLines = fileDiff.split('\n');
+            const diffLines = fileDiff.split("\n");
             for (let i = 0; i < diffLines.length; i++) {
                 const line = diffLines[i];
-                const cleanLine = line.replace(/^[+\-\s]/, '').trim();
-                if (cleanLine.includes(cleanSnippet) || cleanSnippet.includes(cleanLine)) {
+                const cleanLine = line.replace(/^[+\-\s]/, "").trim();
+                if (cleanLine.includes(cleanSnippet) ||
+                    cleanSnippet.includes(cleanLine)) {
                     // Found a match! Update the violation with the correct snippet
                     const fixedViolation = { ...violation };
                     fixedViolation.code_snippet = line; // Use the full line with diff prefix
@@ -164,18 +174,18 @@ class CodeReviewer {
                     if (i > 0 && i < diffLines.length - 1) {
                         fixedViolation.search_context = {
                             before: [diffLines[i - 1]],
-                            after: [diffLines[i + 1]]
+                            after: [diffLines[i + 1]],
                         };
                     }
-                    Logger_1.logger.debug(`✅ Fixed code snippet for ${violation.file} using fuzzy match`);
+                    logger.debug(`✅ Fixed code snippet for ${violation.file} using fuzzy match`);
                     return fixedViolation;
                 }
             }
-            Logger_1.logger.debug(`❌ Could not find snippet in diff for ${violation.file}`);
-            Logger_1.logger.debug(`   Looking for: "${violation.code_snippet}"`);
+            logger.debug(`❌ Could not find snippet in diff for ${violation.file}`);
+            logger.debug(`   Looking for: "${violation.code_snippet}"`);
         }
         catch (error) {
-            Logger_1.logger.debug(`Error fixing code snippet: ${error.message}`);
+            logger.debug(`Error fixing code snippet: ${error.message}`);
         }
         return null;
     }
@@ -183,7 +193,7 @@ class CodeReviewer {
      * Get system prompt for security-focused code review
      */
     getSecurityReviewSystemPrompt() {
-        return this.reviewConfig.systemPrompt ||
+        return (this.reviewConfig.systemPrompt ||
             `You are an Expert Security Code Reviewer for enterprise applications. Your role is to:
 
 🔒 SECURITY FIRST: Prioritize security vulnerabilities and data protection
@@ -194,14 +204,17 @@ class CodeReviewer {
 You provide actionable, educational feedback with specific examples and solutions.
 Focus on critical issues that could impact production systems.
 
-CRITICAL INSTRUCTION: When identifying issues, you MUST copy the EXACT line from the diff, including the diff prefix (+, -, or space). Do not modify or clean the line in any way.`;
+CRITICAL INSTRUCTION: When identifying issues, you MUST copy the EXACT line from the diff, including the diff prefix (+, -, or space). Do not modify or clean the line in any way.`);
     }
     /**
      * Get analysis requirements from config or defaults
      */
     getAnalysisRequirements() {
-        if (this.reviewConfig.focusAreas && this.reviewConfig.focusAreas.length > 0) {
-            return this.reviewConfig.focusAreas.map(area => `### ${area}`).join('\n\n');
+        if (this.reviewConfig.focusAreas &&
+            this.reviewConfig.focusAreas.length > 0) {
+            return this.reviewConfig.focusAreas
+                .map((area) => `### ${area}`)
+                .join("\n\n");
         }
         // Default analysis requirements
         return `### 🔒 Security Analysis (CRITICAL PRIORITY)
@@ -228,7 +241,7 @@ CRITICAL INSTRUCTION: When identifying issues, you MUST copy the EXACT line from
      */
     buildCoreAnalysisPrompt(context) {
         const diffContent = this.extractDiffContent(context);
-        return `Conduct a comprehensive security and quality analysis of this ${context.diffStrategy.strategy === 'whole' ? 'pull request' : 'code changeset'}.
+        return `Conduct a comprehensive security and quality analysis of this ${context.diffStrategy.strategy === "whole" ? "pull request" : "code changeset"}.
 
 ## COMPLETE PR CONTEXT:
 **Title**: ${context.pr.title}
@@ -248,7 +261,7 @@ CRITICAL INSTRUCTION: When identifying issues, you MUST copy the EXACT line from
 ${context.projectContext.memoryBank.projectContext || context.projectContext.memoryBank.summary}
 
 ## PROJECT RULES & STANDARDS:
-${context.projectContext.clinerules || 'No specific rules defined'}
+${context.projectContext.clinerules || "No specific rules defined"}
 
 ## COMPLETE CODE CHANGES (NO TRUNCATION):
 ${diffContent}
@@ -310,17 +323,18 @@ Return ONLY valid JSON:
      * Extract diff content based on strategy
      */
     extractDiffContent(context) {
-        if (context.diffStrategy.strategy === 'whole' && context.prDiff) {
+        if (context.diffStrategy.strategy === "whole" && context.prDiff) {
             return context.prDiff.diff || JSON.stringify(context.prDiff, null, 2);
         }
-        else if (context.diffStrategy.strategy === 'file-by-file' && context.fileDiffs) {
+        else if (context.diffStrategy.strategy === "file-by-file" &&
+            context.fileDiffs) {
             const fileDiffArray = Array.from(context.fileDiffs.entries()).map(([file, diff]) => ({
                 file,
-                diff
+                diff,
             }));
             return JSON.stringify(fileDiffArray, null, 2);
         }
-        return 'No diff content available';
+        return "No diff content available";
     }
     /**
      * Detect project type for better context
@@ -329,42 +343,53 @@ Return ONLY valid JSON:
         const fileExtensions = new Set();
         // Extract file extensions from changes
         if (context.pr.fileChanges) {
-            context.pr.fileChanges.forEach(file => {
-                const ext = file.split('.').pop()?.toLowerCase();
-                if (ext)
+            context.pr.fileChanges.forEach((file) => {
+                const ext = file.split(".").pop()?.toLowerCase();
+                if (ext) {
                     fileExtensions.add(ext);
+                }
             });
         }
-        if (fileExtensions.has('rs') || fileExtensions.has('res'))
-            return 'rescript';
-        if (fileExtensions.has('ts') || fileExtensions.has('tsx'))
-            return 'typescript';
-        if (fileExtensions.has('js') || fileExtensions.has('jsx'))
-            return 'javascript';
-        if (fileExtensions.has('py'))
-            return 'python';
-        if (fileExtensions.has('go'))
-            return 'golang';
-        if (fileExtensions.has('java'))
-            return 'java';
-        if (fileExtensions.has('cpp') || fileExtensions.has('c'))
-            return 'cpp';
-        return 'mixed';
+        if (fileExtensions.has("rs") || fileExtensions.has("res")) {
+            return "rescript";
+        }
+        if (fileExtensions.has("ts") || fileExtensions.has("tsx")) {
+            return "typescript";
+        }
+        if (fileExtensions.has("js") || fileExtensions.has("jsx")) {
+            return "javascript";
+        }
+        if (fileExtensions.has("py")) {
+            return "python";
+        }
+        if (fileExtensions.has("go")) {
+            return "golang";
+        }
+        if (fileExtensions.has("java")) {
+            return "java";
+        }
+        if (fileExtensions.has("cpp") || fileExtensions.has("c")) {
+            return "cpp";
+        }
+        return "mixed";
     }
     /**
      * Assess complexity level for better AI context
      */
     assessComplexity(context) {
         const fileCount = context.diffStrategy.fileCount;
-        const hasLargeFiles = context.diffStrategy.estimatedSize.includes('Large');
+        const hasLargeFiles = context.diffStrategy.estimatedSize.includes("Large");
         const hasComments = (context.pr.comments?.length || 0) > 0;
-        if (fileCount > 50)
-            return 'very-high';
-        if (fileCount > 20 || hasLargeFiles)
-            return 'high';
-        if (fileCount > 10 || hasComments)
-            return 'medium';
-        return 'low';
+        if (fileCount > 50) {
+            return "very-high";
+        }
+        if (fileCount > 20 || hasLargeFiles) {
+            return "high";
+        }
+        if (fileCount > 10 || hasComments) {
+            return "medium";
+        }
+        return "low";
     }
     /**
      * Legacy method - kept for compatibility but simplified
@@ -373,21 +398,37 @@ Return ONLY valid JSON:
         // Legacy method - now delegates to new structure
         return this.buildCoreAnalysisPrompt(context);
     }
+    /**
+     * Get safe token limit based on AI provider using shared utility
+     */
+    getSafeTokenLimit() {
+        const provider = this.aiConfig.provider || "auto";
+        const configuredTokens = this.aiConfig.maxTokens;
+        // Use conservative limits for CodeReviewer (safer for large diffs)
+        const providerLimit = getProviderTokenLimit(provider, true);
+        // Use the smaller of configured tokens or provider limit
+        if (configuredTokens && configuredTokens > 0) {
+            const safeLimit = Math.min(configuredTokens, providerLimit);
+            logger.debug(`Token limit: configured=${configuredTokens}, provider=${providerLimit}, using=${safeLimit}`);
+            return safeLimit;
+        }
+        logger.debug(`Token limit: using provider default=${providerLimit} for ${provider}`);
+        return providerLimit;
+    }
     /**
      * Analyze code with AI using the enhanced prompt
      */
     async analyzeWithAI(prompt, context) {
         try {
-            Logger_1.logger.debug('Starting AI analysis...');
+            logger.debug("Starting AI analysis...");
             // Initialize NeuroLink with eval-based dynamic import
             if (!this.neurolink) {
-                const dynamicImport = eval('(specifier) => import(specifier)');
-                const { NeuroLink } = await dynamicImport('@juspay/neurolink');
+                const { NeuroLink } = await import("@juspay/neurolink");
                 this.neurolink = new NeuroLink();
             }
             // Extract context from unified context for better AI understanding
             const aiContext = {
-                operation: 'code-review',
+                operation: "code-review",
                 repository: `${context.identifier.workspace}/${context.identifier.repository}`,
                 branch: context.identifier.branch,
                 prId: context.identifier.pullRequestId,
@@ -395,52 +436,59 @@ Return ONLY valid JSON:
                 prAuthor: context.pr.author,
                 fileCount: context.diffStrategy.fileCount,
                 diffStrategy: context.diffStrategy.strategy,
-                analysisType: context.diffStrategy.strategy === 'whole' ? 'comprehensive' : 'file-by-file',
+                analysisType: context.diffStrategy.strategy === "whole"
+                    ? "comprehensive"
+                    : "file-by-file",
                 projectType: this.detectProjectType(context),
                 hasExistingComments: (context.pr.comments?.length || 0) > 0,
-                complexity: this.assessComplexity(context)
+                complexity: this.assessComplexity(context),
             };
             // Simplified, focused prompt without context pollution
             const corePrompt = this.buildCoreAnalysisPrompt(context);
+            // Get safe token limit based on provider
+            const safeMaxTokens = this.getSafeTokenLimit();
+            logger.debug(`Using AI provider: ${this.aiConfig.provider || "auto"}`);
+            logger.debug(`Configured maxTokens: ${this.aiConfig.maxTokens}`);
+            logger.debug(`Safe maxTokens limit: ${safeMaxTokens}`);
             const result = await this.neurolink.generate({
                 input: { text: corePrompt },
                 systemPrompt: this.getSecurityReviewSystemPrompt(),
-                provider: this.aiConfig.provider || 'auto', // Auto-select best provider
-                model: this.aiConfig.model || 'best', // Use most capable model
+                provider: this.aiConfig.provider || "auto", // Auto-select best provider
+                model: this.aiConfig.model || "best", // Use most capable model
                 temperature: this.aiConfig.temperature || 0.3, // Lower for more focused analysis
-                maxTokens: Math.max(this.aiConfig.maxTokens || 0, 2000000), // Quality first - always use higher limit
-                timeout: '15m', // Allow plenty of time for thorough analysis
+                maxTokens: safeMaxTokens, // Use provider-aware safe token limit
+                timeout: "15m", // Allow plenty of time for thorough analysis
                 context: aiContext,
                 enableAnalytics: this.aiConfig.enableAnalytics || true,
-                enableEvaluation: false // Disabled to prevent evaluation warnings
+                enableEvaluation: false, // Disabled to prevent evaluation warnings
             });
             // Log analytics if available
             if (result.analytics) {
-                Logger_1.logger.debug(`AI Analytics - Provider: ${result.provider}, Response Time: ${result.responseTime}ms, Quality Score: ${result.evaluation?.overallScore}`);
+                logger.debug(`AI Analytics - Provider: ${result.provider}, Response Time: ${result.responseTime}ms, Quality Score: ${result.evaluation?.overallScore}`);
             }
-            Logger_1.logger.debug('AI analysis completed, parsing response...');
+            logger.debug("AI analysis completed, parsing response...");
             // Modern NeuroLink returns { content: string }
             const analysisData = this.parseAIResponse(result);
             // Display AI response for debugging
-            if (Logger_1.logger.getConfig().verbose) {
-                Logger_1.logger.debug('AI Analysis Response:');
-                Logger_1.logger.debug('═'.repeat(80));
-                Logger_1.logger.debug(JSON.stringify(analysisData, null, 2));
-                Logger_1.logger.debug('═'.repeat(80));
+            if (logger.getConfig().verbose) {
+                logger.debug("AI Analysis Response:");
+                logger.debug("═".repeat(80));
+                logger.debug(JSON.stringify(analysisData, null, 2));
+                logger.debug("═".repeat(80));
             }
             if (!analysisData.violations || !Array.isArray(analysisData.violations)) {
-                Logger_1.logger.debug('No violations array found in AI response');
+                logger.debug("No violations array found in AI response");
                 return [];
             }
-            Logger_1.logger.debug(`AI analysis found ${analysisData.violations.length} violations`);
+            logger.debug(`AI analysis found ${analysisData.violations.length} violations`);
             return analysisData.violations;
         }
         catch (error) {
-            if (error.message?.includes('timeout')) {
-                Logger_1.logger.error('⏰ AI analysis timed out after 15 minutes');
-                throw new Error('Analysis timeout - try reducing diff size or adjusting timeout');
+            if (error.message?.includes("timeout")) {
+                logger.error("⏰ AI analysis timed out after 15 minutes");
+                throw new Error("Analysis timeout - try reducing diff size or adjusting timeout");
             }
-            Logger_1.logger.error(`AI analysis failed: ${error.message}`);
+            logger.error(`AI analysis failed: ${error.message}`);
             throw error;
         }
     }
@@ -448,12 +496,12 @@ Return ONLY valid JSON:
      * Post comments to PR using unified context - matching pr-police.js exactly
      */
     async postComments(context, violations, _options) {
-        Logger_1.logger.phase('📝 Posting review comments...');
+        logger.phase("📝 Posting review comments...");
         let commentsPosted = 0;
         let commentsFailed = 0;
         const failedComments = [];
         // Post inline comments
-        const inlineViolations = violations.filter(v => v.type === 'inline' && v.file && v.code_snippet);
+        const inlineViolations = violations.filter((v) => v.type === "inline" && v.file && v.code_snippet);
         for (const violation of inlineViolations) {
             try {
                 // Clean file path - remove protocol prefixes ONLY (keep a/ and b/ prefixes)
@@ -467,19 +515,19 @@ Return ONLY valid JSON:
                 // Clean code snippet and fix search context - EXACTLY like pr-police.js
                 const processedViolation = this.cleanCodeSnippet(violation);
                 if (!processedViolation) {
-                    Logger_1.logger.debug(`⚠️ Skipping invalid violation for ${cleanFilePath}`);
+                    logger.debug(`⚠️ Skipping invalid violation for ${cleanFilePath}`);
                     continue;
                 }
                 const formattedComment = this.formatInlineComment(processedViolation);
                 // Debug logging
-                Logger_1.logger.debug(`🔍 Posting inline comment:`);
-                Logger_1.logger.debug(`   File: ${cleanFilePath}`);
-                Logger_1.logger.debug(`   Issue: ${processedViolation.issue}`);
-                Logger_1.logger.debug(`   Original snippet: ${violation.code_snippet}`);
-                Logger_1.logger.debug(`   Processed snippet: ${processedViolation.code_snippet}`);
+                logger.debug(`🔍 Posting inline comment:`);
+                logger.debug(`   File: ${cleanFilePath}`);
+                logger.debug(`   Issue: ${processedViolation.issue}`);
+                logger.debug(`   Original snippet: ${violation.code_snippet}`);
+                logger.debug(`   Processed snippet: ${processedViolation.code_snippet}`);
                 if (processedViolation.search_context) {
-                    Logger_1.logger.debug(`   Search context before: ${JSON.stringify(processedViolation.search_context.before)}`);
-                    Logger_1.logger.debug(`   Search context after: ${JSON.stringify(processedViolation.search_context.after)}`);
+                    logger.debug(`   Search context before: ${JSON.stringify(processedViolation.search_context.before)}`);
+                    logger.debug(`   Search context after: ${JSON.stringify(processedViolation.search_context.after)}`);
                 }
                 // Use new code snippet approach - EXACTLY like pr-police.js
                 await this.bitbucketProvider.addComment(context.identifier, formattedComment, {
@@ -489,21 +537,21 @@ Return ONLY valid JSON:
                     codeSnippet: processedViolation.code_snippet,
                     searchContext: processedViolation.search_context,
                     matchStrategy: "best", // Use best match strategy instead of strict for flexibility
-                    suggestion: processedViolation.suggestion // Pass the suggestion for inline code suggestions
+                    suggestion: processedViolation.suggestion, // Pass the suggestion for inline code suggestions
                 });
                 commentsPosted++;
-                Logger_1.logger.debug(`✅ Posted inline comment: ${cleanFilePath} (${processedViolation.issue})`);
+                logger.debug(`✅ Posted inline comment: ${cleanFilePath} (${processedViolation.issue})`);
             }
             catch (error) {
                 commentsFailed++;
                 const errorMsg = error.message;
-                Logger_1.logger.debug(`❌ Failed to post inline comment: ${errorMsg}`);
-                Logger_1.logger.debug(`   File: ${violation.file}, Issue: ${violation.issue}`);
-                Logger_1.logger.debug(`   Code snippet: ${violation.code_snippet}`);
+                logger.debug(`❌ Failed to post inline comment: ${errorMsg}`);
+                logger.debug(`   File: ${violation.file}, Issue: ${violation.issue}`);
+                logger.debug(`   Code snippet: ${violation.code_snippet}`);
                 failedComments.push({
                     file: violation.file,
                     issue: violation.issue,
-                    error: errorMsg
+                    error: errorMsg,
                 });
             }
         }
@@ -513,15 +561,15 @@ Return ONLY valid JSON:
                 const summaryComment = this.generateSummaryComment(violations, context, failedComments);
                 await this.bitbucketProvider.addComment(context.identifier, summaryComment);
                 commentsPosted++;
-                Logger_1.logger.debug('✅ Posted summary comment');
+                logger.debug("✅ Posted summary comment");
             }
             catch (error) {
-                Logger_1.logger.debug(`❌ Failed to post summary comment: ${error.message}`);
+                logger.debug(`❌ Failed to post summary comment: ${error.message}`);
             }
         }
-        Logger_1.logger.success(`✅ Posted ${commentsPosted} comments successfully`);
+        logger.success(`✅ Posted ${commentsPosted} comments successfully`);
         if (commentsFailed > 0) {
-            Logger_1.logger.warn(`⚠️ Failed to post ${commentsFailed} inline comments`);
+            logger.warn(`⚠️ Failed to post ${commentsFailed} inline comments`);
         }
     }
     /**
@@ -529,14 +577,23 @@ Return ONLY valid JSON:
      */
     formatInlineComment(violation) {
         const severityConfig = {
-            CRITICAL: { emoji: '🚨', badge: '**🚨 CRITICAL SECURITY ISSUE**', color: 'red' },
-            MAJOR: { emoji: '⚠️', badge: '**⚠️ MAJOR ISSUE**', color: 'orange' },
-            MINOR: { emoji: '📝', badge: '**📝 MINOR IMPROVEMENT**', color: 'blue' },
-            SUGGESTION: { emoji: '💡', badge: '**💡 SUGGESTION**', color: 'green' }
+            CRITICAL: {
+                emoji: "🚨",
+                badge: "**🚨 CRITICAL SECURITY ISSUE**",
+                color: "red",
+            },
+            MAJOR: { emoji: "⚠️", badge: "**⚠️ MAJOR ISSUE**", color: "orange" },
+            MINOR: { emoji: "📝", badge: "**📝 MINOR IMPROVEMENT**", color: "blue" },
+            SUGGESTION: { emoji: "💡", badge: "**💡 SUGGESTION**", color: "green" },
         };
         const categoryIcons = {
-            security: '🔒', performance: '⚡', maintainability: '🏗️',
-            functionality: '⚙️', error_handling: '🛡️', testing: '🧪', general: '📋'
+            security: "🔒",
+            performance: "⚡",
+            maintainability: "🏗️",
+            functionality: "⚙️",
+            error_handling: "🛡️",
+            testing: "🧪",
+            general: "📋",
         };
         const config = severityConfig[violation.severity] || severityConfig.MINOR;
         const categoryIcon = categoryIcons[violation.category] || categoryIcons.general;
@@ -544,20 +601,30 @@ Return ONLY valid JSON:
 
 **${categoryIcon} ${violation.issue}**
 
-**Category**: ${violation.category.replace(/_/g, ' ').replace(/\b\w/g, l => l.toUpperCase())}
+**Category**: ${violation.category.replace(/_/g, " ").replace(/\b\w/g, (l) => l.toUpperCase())}
 
 **Issue**: ${violation.message}`;
         if (violation.impact) {
             comment += `\n\n**Impact**: ${violation.impact}`;
         }
         if (violation.suggestion) {
-            const fileExt = violation.file?.split('.').pop() || 'text';
+            const fileExt = violation.file?.split(".").pop() || "text";
             const langMap = {
-                js: 'javascript', jsx: 'javascript', ts: 'typescript', tsx: 'typescript',
-                res: 'rescript', resi: 'rescript', py: 'python', java: 'java',
-                go: 'go', rb: 'ruby', php: 'php', sql: 'sql', json: 'json'
+                js: "javascript",
+                jsx: "javascript",
+                ts: "typescript",
+                tsx: "typescript",
+                res: "rescript",
+                resi: "rescript",
+                py: "python",
+                java: "java",
+                go: "go",
+                rb: "ruby",
+                php: "php",
+                sql: "sql",
+                json: "json",
             };
-            const language = langMap[fileExt] || 'text';
+            const language = langMap[fileExt] || "text";
             // Use the escape method for code blocks
             const escapedCodeBlock = this.escapeMarkdownCodeBlock(violation.suggestion, language);
             comment += `\n\n**💡 Suggested Fix**:\n${escapedCodeBlock}`;
@@ -570,13 +637,20 @@ Return ONLY valid JSON:
      */
     generateSummaryComment(violations, context, failedComments = []) {
         const stats = this.calculateStats(violations);
-        const statusEmoji = stats.criticalCount > 0 ? '🚨' :
-            stats.majorCount > 0 ? '⚠️ ' :
-                stats.minorCount > 0 ? '📝' : '✅';
-        const statusText = stats.criticalCount > 0 ? 'CRITICAL ISSUES FOUND' :
-            stats.majorCount > 0 ? 'ISSUES DETECTED' :
-                stats.minorCount > 0 ? 'IMPROVEMENTS SUGGESTED' :
-                    'CODE QUALITY APPROVED';
+        const statusEmoji = stats.criticalCount > 0
+            ? "🚨"
+            : stats.majorCount > 0
+                ? "⚠️ "
+                : stats.minorCount > 0
+                    ? "📝"
+                    : "✅";
+        const statusText = stats.criticalCount > 0
+            ? "CRITICAL ISSUES FOUND"
+            : stats.majorCount > 0
+                ? "ISSUES DETECTED"
+                : stats.minorCount > 0
+                    ? "IMPROVEMENTS SUGGESTED"
+                    : "CODE QUALITY APPROVED";
         let comment = `
 ╭─────────────────────────────────────────────────────────────╮
 │                    ⚔️ **YAMA REVIEW REPORT** ⚔️               │
@@ -587,10 +661,10 @@ Return ONLY valid JSON:
 ### 📊 **Security & Quality Analysis**
 | **Severity** | **Count** | **Status** |
 |--------------|-----------|------------|
-| 🚨 Critical | ${stats.criticalCount} | ${stats.criticalCount > 0 ? '⛔ Must Fix' : '✅ Clear'} |
-| ⚠️ Major | ${stats.majorCount} | ${stats.majorCount > 0 ? '⚠️ Should Fix' : '✅ Clear'} |
-| 📝 Minor | ${stats.minorCount} | ${stats.minorCount > 0 ? '📝 Consider Fixing' : '✅ Clear'} |
-| 💡 Suggestions | ${stats.suggestionCount} | ${stats.suggestionCount > 0 ? '💡 Optional' : '✅ Clear'} |
+| 🚨 Critical | ${stats.criticalCount} | ${stats.criticalCount > 0 ? "⛔ Must Fix" : "✅ Clear"} |
+| ⚠️ Major | ${stats.majorCount} | ${stats.majorCount > 0 ? "⚠️ Should Fix" : "✅ Clear"} |
+| 📝 Minor | ${stats.minorCount} | ${stats.minorCount > 0 ? "📝 Consider Fixing" : "✅ Clear"} |
+| 💡 Suggestions | ${stats.suggestionCount} | ${stats.suggestionCount > 0 ? "💡 Optional" : "✅ Clear"} |
 
 ### 🔍 **Analysis Summary**
 - **📁 Files Analyzed**: ${context.diffStrategy.fileCount}
@@ -603,12 +677,19 @@ Return ONLY valid JSON:
             comment += `\n\n### 📍 **Issues by Category**\n`;
             for (const [category, categoryViolations] of Object.entries(violationsByCategory)) {
                 const categoryIcons = {
-                    security: '🔒', performance: '⚡', maintainability: '🏗️',
-                    functionality: '⚙️', error_handling: '🛡️', testing: '🧪', general: '📋'
+                    security: "🔒",
+                    performance: "⚡",
+                    maintainability: "🏗️",
+                    functionality: "⚙️",
+                    error_handling: "🛡️",
+                    testing: "🧪",
+                    general: "📋",
                 };
-                const icon = categoryIcons[category] || '📋';
-                const name = category.replace(/_/g, ' ').replace(/\b\w/g, l => l.toUpperCase());
-                comment += `**${icon} ${name}**: ${categoryViolations.length} issue${categoryViolations.length !== 1 ? 's' : ''}\n`;
+                const icon = categoryIcons[category] || "📋";
+                const name = category
+                    .replace(/_/g, " ")
+                    .replace(/\b\w/g, (l) => l.toUpperCase());
+                comment += `**${icon} ${name}**: ${categoryViolations.length} issue${categoryViolations.length !== 1 ? "s" : ""}\n`;
             }
         }
         // Add failed comments section if any
@@ -617,17 +698,17 @@ Return ONLY valid JSON:
             comment += `Some inline comments could not be posted due to code matching issues. `;
             comment += `Please review the following issues manually:\n\n`;
             for (const failed of failedComments) {
-                comment += `- **${failed.issue}** in \`${failed.file || 'unknown file'}\`\n`;
+                comment += `- **${failed.issue}** in \`${failed.file || "unknown file"}\`\n`;
             }
         }
         // Add recommendation
         const recommendation = stats.criticalCount > 0
-            ? '🚨 **URGENT**: Critical security issues must be resolved before merge'
+            ? "🚨 **URGENT**: Critical security issues must be resolved before merge"
             : stats.majorCount > 0
-                ? '⚠️ **RECOMMENDED**: Address major issues before merge'
+                ? "⚠️ **RECOMMENDED**: Address major issues before merge"
                 : stats.minorCount > 0
-                    ? '📝 **OPTIONAL**: Consider addressing minor improvements'
-                    : '✅ **APPROVED**: Code meets security and quality standards';
+                    ? "📝 **OPTIONAL**: Consider addressing minor improvements"
+                    : "✅ **APPROVED**: Code meets security and quality standards";
         comment += `\n\n### 💡 **Recommendation**
 ${recommendation}
 
@@ -642,11 +723,10 @@ ${recommendation}
     cleanFilePath(filePath) {
         // Clean the file path but preserve the structure - EXACTLY like pr-police.js
         // Only clean src:// and dst:// prefixes, keep a/ and b/ prefixes
-        const cleaned = filePath
-            .replace(/^(src|dst):\/\//, '');
+        const cleaned = filePath.replace(/^(src|dst):\/\//, "");
         // Log the cleaning for debugging
         if (cleaned !== filePath) {
-            Logger_1.logger.debug(`Cleaned file path: ${filePath} -> ${cleaned}`);
+            logger.debug(`Cleaned file path: ${filePath} -> ${cleaned}`);
         }
         return cleaned;
     }
@@ -654,12 +734,13 @@ ${recommendation}
      * Extract exact file path from diff
      */
     extractFilePathFromDiff(diff, fileName) {
-        const lines = diff.split('\n');
+        const lines = diff.split("\n");
         for (const line of lines) {
-            if (line.startsWith('diff --git')) {
+            if (line.startsWith("diff --git")) {
                 // Extract both paths: a/path/to/file b/path/to/file
                 const match = line.match(/diff --git a\/(.*?) b\/(.*?)$/);
-                if (match && (match[1].includes(fileName) || match[2].includes(fileName))) {
+                if (match &&
+                    (match[1].includes(fileName) || match[2].includes(fileName))) {
                     return match[2]; // Return the 'b/' path (destination)
                 }
             }
@@ -670,12 +751,12 @@ ${recommendation}
      * Extract line number from diff for a specific code snippet
      */
     extractLineNumberFromDiff(fileDiff, codeSnippet) {
-        const lines = fileDiff.split('\n');
+        const lines = fileDiff.split("\n");
         let currentNewLine = 0;
         let currentOldLine = 0;
         let inHunk = false;
         // Debug logging
-        Logger_1.logger.debug(`Looking for snippet: "${codeSnippet}"`);
+        logger.debug(`Looking for snippet: "${codeSnippet}"`);
         for (let i = 0; i < lines.length; i++) {
             const line = lines[i];
             // Parse hunk headers (e.g., @@ -10,6 +10,8 @@)
@@ -685,48 +766,51 @@ ${recommendation}
                 currentOldLine = parseInt(hunkMatch[1]);
                 currentNewLine = parseInt(hunkMatch[2]);
                 inHunk = true;
-                Logger_1.logger.debug(`Found hunk header: old=${currentOldLine}, new=${currentNewLine}`);
+                logger.debug(`Found hunk header: old=${currentOldLine}, new=${currentNewLine}`);
                 continue;
             }
             // Skip lines that aren't part of the diff content
-            if (!inHunk || (!line.startsWith('+') && !line.startsWith('-') && !line.startsWith(' '))) {
+            if (!inHunk ||
+                (!line.startsWith("+") &&
+                    !line.startsWith("-") &&
+                    !line.startsWith(" "))) {
                 continue;
             }
             // Check if this line matches our snippet
             if (line === codeSnippet) {
                 let resultLine;
                 let lineType;
-                if (line.startsWith('+')) {
+                if (line.startsWith("+")) {
                     resultLine = currentNewLine;
-                    lineType = 'ADDED';
+                    lineType = "ADDED";
                 }
-                else if (line.startsWith('-')) {
+                else if (line.startsWith("-")) {
                     resultLine = currentOldLine;
-                    lineType = 'REMOVED';
+                    lineType = "REMOVED";
                 }
                 else {
                     resultLine = currentNewLine;
-                    lineType = 'CONTEXT';
+                    lineType = "CONTEXT";
                 }
-                Logger_1.logger.debug(`Found match at line ${resultLine} (${lineType})`);
+                logger.debug(`Found match at line ${resultLine} (${lineType})`);
                 return { lineNumber: resultLine, lineType };
             }
             // Update line counters AFTER checking for match
             // For added lines: only increment new line counter
             // For removed lines: only increment old line counter
             // For context lines: increment both counters
-            if (line.startsWith('+')) {
+            if (line.startsWith("+")) {
                 currentNewLine++;
             }
-            else if (line.startsWith('-')) {
+            else if (line.startsWith("-")) {
                 currentOldLine++;
             }
-            else if (line.startsWith(' ')) {
+            else if (line.startsWith(" ")) {
                 currentNewLine++;
                 currentOldLine++;
             }
         }
-        Logger_1.logger.debug(`Snippet not found in diff`);
+        logger.debug(`Snippet not found in diff`);
         return null;
     }
     /**
@@ -734,7 +818,7 @@ ${recommendation}
      */
     escapeMarkdownCodeBlock(code, language) {
         // If code contains triple backticks, use quadruple backticks
-        if (code.includes('```')) {
+        if (code.includes("```")) {
             return `\`\`\`\`${language}\n${code}\n\`\`\`\``;
         }
         return `\`\`\`${language}\n${code}\n\`\`\``;
@@ -745,49 +829,51 @@ ${recommendation}
             const fixed = JSON.parse(JSON.stringify(violation));
             // Fix search_context arrays if they contain embedded newlines
             if (fixed.search_context) {
-                if (fixed.search_context.before && Array.isArray(fixed.search_context.before)) {
+                if (fixed.search_context.before &&
+                    Array.isArray(fixed.search_context.before)) {
                     fixed.search_context.before = this.splitArrayLines(fixed.search_context.before);
                 }
-                if (fixed.search_context.after && Array.isArray(fixed.search_context.after)) {
+                if (fixed.search_context.after &&
+                    Array.isArray(fixed.search_context.after)) {
                     fixed.search_context.after = this.splitArrayLines(fixed.search_context.after);
                 }
             }
             // Ensure line_type is set based on code snippet prefix BEFORE cleaning
             if (!fixed.line_type && fixed.code_snippet) {
-                if (fixed.code_snippet.startsWith('+')) {
-                    fixed.line_type = 'ADDED';
+                if (fixed.code_snippet.startsWith("+")) {
+                    fixed.line_type = "ADDED";
                 }
-                else if (fixed.code_snippet.startsWith('-')) {
-                    fixed.line_type = 'REMOVED';
+                else if (fixed.code_snippet.startsWith("-")) {
+                    fixed.line_type = "REMOVED";
                 }
                 else {
-                    fixed.line_type = 'CONTEXT';
+                    fixed.line_type = "CONTEXT";
                 }
             }
             // Clean the code_snippet field to remove diff symbols - EXACTLY like pr-police.js
             if (fixed.code_snippet) {
-                fixed.code_snippet = fixed.code_snippet.replace(/^[+\-\s]/, '').trim();
+                fixed.code_snippet = fixed.code_snippet.replace(/^[+\-\s]/, "").trim();
             }
             // Clean the suggestion field to remove any diff symbols
             if (fixed.suggestion) {
                 fixed.suggestion = fixed.suggestion
-                    .split('\n')
-                    .map((line) => line.replace(/^[+\-\s]/, '')) // Remove diff symbols at start of each line
-                    .join('\n')
+                    .split("\n")
+                    .map((line) => line.replace(/^[+\-\s]/, "")) // Remove diff symbols at start of each line
+                    .join("\n")
                     .trim();
             }
             return fixed;
         }
         catch (error) {
-            Logger_1.logger.debug(`❌ Error cleaning code snippet: ${error.message}`);
+            logger.debug(`❌ Error cleaning code snippet: ${error.message}`);
             return null;
         }
     }
     splitArrayLines(arr) {
         const result = [];
         for (const item of arr) {
-            if (typeof item === 'string' && item.includes('\n')) {
-                result.push(...item.split('\n').filter(line => line.length > 0));
+            if (typeof item === "string" && item.includes("\n")) {
+                result.push(...item.split("\n").filter((line) => line.length > 0));
             }
             else {
                 result.push(item);
@@ -797,22 +883,24 @@ ${recommendation}
     }
     groupViolationsByCategory(violations) {
         const grouped = {};
-        violations.forEach(v => {
-            const category = v.category || 'general';
-            if (!grouped[category])
+        violations.forEach((v) => {
+            const category = v.category || "general";
+            if (!grouped[category]) {
                 grouped[category] = [];
+            }
             grouped[category].push(v);
         });
         return grouped;
     }
     calculateStats(violations) {
         return {
-            criticalCount: violations.filter(v => v.severity === 'CRITICAL').length,
-            majorCount: violations.filter(v => v.severity === 'MAJOR').length,
-            minorCount: violations.filter(v => v.severity === 'MINOR').length,
-            suggestionCount: violations.filter(v => v.severity === 'SUGGESTION').length,
+            criticalCount: violations.filter((v) => v.severity === "CRITICAL").length,
+            majorCount: violations.filter((v) => v.severity === "MAJOR").length,
+            minorCount: violations.filter((v) => v.severity === "MINOR").length,
+            suggestionCount: violations.filter((v) => v.severity === "SUGGESTION")
+                .length,
             totalIssues: violations.length,
-            filesReviewed: new Set(violations.filter(v => v.file).map(v => v.file)).size || 1
+            filesReviewed: new Set(violations.filter((v) => v.file).map((v) => v.file)).size || 1,
         };
     }
     generateReviewResult(violations, _duration, _context) {
@@ -826,9 +914,9 @@ ${recommendation}
                 criticalCount: stats.criticalCount,
                 majorCount: stats.majorCount,
                 minorCount: stats.minorCount,
-                suggestionCount: stats.suggestionCount
+                suggestionCount: stats.suggestionCount,
             },
-            positiveObservations: [] // Could be extracted from AI response
+            positiveObservations: [], // Could be extracted from AI response
         };
     }
     /**
@@ -836,7 +924,7 @@ ${recommendation}
      */
     parseAIResponse(result) {
         try {
-            const responseText = result.content || result.text || result.response || '';
+            const responseText = result.content || result.text || result.response || "";
             if (!responseText) {
                 return { violations: [] };
             }
@@ -848,7 +936,7 @@ ${recommendation}
             return { violations: [] };
         }
         catch (error) {
-            Logger_1.logger.debug(`Failed to parse AI response: ${error.message}`);
+            logger.debug(`Failed to parse AI response: ${error.message}`);
             return { violations: [] };
         }
     }
@@ -856,20 +944,21 @@ ${recommendation}
      * Extract line information for comment from context
      */
     extractLineInfoForComment(violation, context) {
-        if (!violation.file || !violation.code_snippet)
+        if (!violation.file || !violation.code_snippet) {
             return null;
+        }
         try {
             // Get the diff for this specific file
             let fileDiff;
-            if (context.diffStrategy.strategy === 'whole' && context.prDiff) {
+            if (context.diffStrategy.strategy === "whole" && context.prDiff) {
                 // Extract file diff from whole diff
-                const diffLines = context.prDiff.diff.split('\n');
+                const diffLines = context.prDiff.diff.split("\n");
                 let fileStartIndex = -1;
                 // Create all possible path variations for matching
                 const filePathVariations = this.generatePathVariations(violation.file);
                 for (let i = 0; i < diffLines.length; i++) {
                     const line = diffLines[i];
-                    if (line.startsWith('diff --git')) {
+                    if (line.startsWith("diff --git")) {
                         // Check if any variation matches
                         for (const pathVariation of filePathVariations) {
                             if (line.includes(pathVariation)) {
@@ -877,22 +966,26 @@ ${recommendation}
                                 break;
                             }
                         }
-                        if (fileStartIndex >= 0)
+                        if (fileStartIndex >= 0) {
                             break;
+                        }
                     }
                 }
                 if (fileStartIndex >= 0) {
-                    const nextFileIndex = diffLines.findIndex((line, idx) => idx > fileStartIndex && line.startsWith('diff --git'));
-                    fileDiff = diffLines.slice(fileStartIndex, nextFileIndex > 0 ? nextFileIndex : diffLines.length).join('\n');
+                    const nextFileIndex = diffLines.findIndex((line, idx) => idx > fileStartIndex && line.startsWith("diff --git"));
+                    fileDiff = diffLines
+                        .slice(fileStartIndex, nextFileIndex > 0 ? nextFileIndex : diffLines.length)
+                        .join("\n");
                 }
             }
-            else if (context.diffStrategy.strategy === 'file-by-file' && context.fileDiffs) {
+            else if (context.diffStrategy.strategy === "file-by-file" &&
+                context.fileDiffs) {
                 // Try all possible path variations
                 const pathVariations = this.generatePathVariations(violation.file);
                 for (const path of pathVariations) {
                     fileDiff = context.fileDiffs.get(path);
                     if (fileDiff) {
-                        Logger_1.logger.debug(`Found diff for ${violation.file} using variation: ${path}`);
+                        logger.debug(`Found diff for ${violation.file} using variation: ${path}`);
                         break;
                     }
                 }
@@ -901,7 +994,7 @@ ${recommendation}
                     for (const [key, value] of context.fileDiffs.entries()) {
                         if (key.endsWith(violation.file) || violation.file.endsWith(key)) {
                             fileDiff = value;
-                            Logger_1.logger.debug(`Found diff for ${violation.file} using partial match: ${key}`);
+                            logger.debug(`Found diff for ${violation.file} using partial match: ${key}`);
                             break;
                         }
                     }
@@ -910,16 +1003,16 @@ ${recommendation}
             if (fileDiff) {
                 const lineInfo = this.extractLineNumberFromDiff(fileDiff, violation.code_snippet);
                 if (lineInfo) {
-                    Logger_1.logger.debug(`Extracted line info for ${violation.file}: line ${lineInfo.lineNumber}, type ${lineInfo.lineType}`);
+                    logger.debug(`Extracted line info for ${violation.file}: line ${lineInfo.lineNumber}, type ${lineInfo.lineType}`);
                 }
                 return lineInfo;
             }
             else {
-                Logger_1.logger.debug(`No diff found for file: ${violation.file}`);
+                logger.debug(`No diff found for file: ${violation.file}`);
             }
         }
         catch (error) {
-            Logger_1.logger.debug(`Error extracting line info: ${error.message}`);
+            logger.debug(`Error extracting line info: ${error.message}`);
         }
         return null;
     }
@@ -933,29 +1026,29 @@ ${recommendation}
         // Add with a/ and b/ prefixes
         variations.add(`a/${filePath}`);
         variations.add(`b/${filePath}`);
-        // Handle nested paths 
-        if (filePath.includes('/')) {
-            const parts = filePath.split('/');
+        // Handle nested paths
+        if (filePath.includes("/")) {
+            const parts = filePath.split("/");
             // Try removing first directory
             if (parts.length > 1) {
-                variations.add(parts.slice(1).join('/'));
+                variations.add(parts.slice(1).join("/"));
             }
             // Try removing first two directories
             if (parts.length > 2) {
-                variations.add(parts.slice(2).join('/'));
+                variations.add(parts.slice(2).join("/"));
             }
             // Try with just the filename
             variations.add(parts[parts.length - 1]);
         }
         // Remove app/ prefix variations
-        if (filePath.startsWith('app/')) {
+        if (filePath.startsWith("app/")) {
             const withoutApp = filePath.substring(4);
             variations.add(withoutApp);
             variations.add(`a/${withoutApp}`);
             variations.add(`b/${withoutApp}`);
         }
         // Add app/ prefix variations
-        if (!filePath.startsWith('app/')) {
+        if (!filePath.startsWith("app/")) {
             variations.add(`app/${filePath}`);
             variations.add(`a/app/${filePath}`);
             variations.add(`b/app/${filePath}`);
@@ -963,8 +1056,7 @@ ${recommendation}
         return Array.from(variations);
     }
 }
-exports.CodeReviewer = CodeReviewer;
-function createCodeReviewer(bitbucketProvider, aiConfig, reviewConfig) {
+export function createCodeReviewer(bitbucketProvider, aiConfig, reviewConfig) {
     return new CodeReviewer(bitbucketProvider, aiConfig, reviewConfig);
 }
 //# sourceMappingURL=CodeReviewer.js.map