@juspay/yama 1.6.0 → 2.0.0

package/dist/features/CodeReviewer.js

@@ -1,1707 +0,0 @@
-/**
- * Enhanced Code Reviewer - Optimized to work with Unified Context
- * Preserves all original functionality from pr-police.js but optimized
- */
-// NeuroLink will be dynamically imported
-import { ProviderError, } from "../types/index.js";
-import { createMultiInstanceProcessor, } from "./MultiInstanceProcessor.js";
-import { logger } from "../utils/Logger.js";
-import { getProviderTokenLimit } from "../utils/ProviderLimits.js";
-import { Semaphore, TokenBudgetManager, calculateOptimalConcurrency, } from "../utils/ParallelProcessing.js";
-import { createExactDuplicateRemover } from "../utils/ExactDuplicateRemover.js";
-export class CodeReviewer {
-    neurolink;
-    bitbucketProvider;
-    aiConfig;
-    reviewConfig;
-    constructor(bitbucketProvider, aiConfig, reviewConfig) {
-        this.bitbucketProvider = bitbucketProvider;
-        this.aiConfig = aiConfig;
-        this.reviewConfig = reviewConfig;
-    }
-    /**
-     * Review code using pre-gathered unified context (OPTIMIZED with Multi-Instance and Batch Processing)
-     */
-    async reviewCodeWithContext(context, options, multiInstanceConfig) {
-        const startTime = Date.now();
-        try {
-            logger.phase("🧪 Conducting AI-powered code analysis...");
-            logger.info(`Analyzing ${context.diffStrategy.fileCount} files using ${context.diffStrategy.strategy} strategy`);
-            let violations;
-            let processingStrategy;
-            // Check if multi-instance processing is enabled and configured
-            if (multiInstanceConfig?.enabled &&
-                multiInstanceConfig.instances?.length > 1) {
-                logger.info("🚀 Using multi-instance processing for enhanced analysis");
-                const multiInstanceResult = await this.reviewWithMultipleInstances(context, options, multiInstanceConfig);
-                violations = multiInstanceResult.finalViolations;
-                processingStrategy = "multi-instance";
-            }
-            else {
-                // Determine if we should use batch processing
-                const batchConfig = this.getBatchProcessingConfig();
-                const shouldUseBatchProcessing = this.shouldUseBatchProcessing(context, batchConfig);
-                if (shouldUseBatchProcessing) {
-                    logger.info("🔄 Using batch processing for large PR analysis");
-                    const batchResult = await this.reviewWithBatchProcessing(context, options, batchConfig);
-                    violations = batchResult.violations;
-                    processingStrategy = "batch-processing";
-                }
-                else {
-                    logger.info("⚡ Using single-request analysis for small PR");
-                    const analysisPrompt = this.buildAnalysisPrompt(context, options);
-                    violations = await this.analyzeWithAI(analysisPrompt, context);
-                    processingStrategy = "single-request";
-                }
-            }
-            if (!options.dryRun && violations.length > 0) {
-                violations = await this.postComments(context, violations, options);
-            }
-            const duration = Math.round((Date.now() - startTime) / 1000);
-            const result = this.generateReviewResult(violations, duration, context, processingStrategy);
-            logger.success(`Code review completed in ${duration}s: ${violations.length} violations found (${processingStrategy})`);
-            return result;
-        }
-        catch (error) {
-            logger.error(`Code review failed: ${error.message}`);
-            throw new ProviderError(`Code review failed: ${error.message}`);
-        }
-    }
-    /**
-     * Review code using multiple instances for enhanced analysis
-     */
-    async reviewWithMultipleInstances(context, options, multiInstanceConfig) {
-        try {
-            // Create multi-instance processor
-            const multiInstanceProcessor = createMultiInstanceProcessor(this.bitbucketProvider, this.reviewConfig);
-            // Execute multi-instance processing
-            const result = await multiInstanceProcessor.processWithMultipleInstances(context, options, multiInstanceConfig);
-            return result;
-        }
-        catch (error) {
-            logger.error(`Multi-instance processing failed: ${error.message}`);
-            throw error;
-        }
-    }
-    /**
-     * Get system prompt for security-focused code review
-     */
-    getSecurityReviewSystemPrompt() {
-        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
-⚡ PERFORMANCE AWARE: Identify performance bottlenecks and optimization opportunities  
-🏗️ QUALITY FOCUSED: Ensure maintainable, readable, and robust code
-🛡️ ERROR RESILIENT: Verify comprehensive error handling and edge cases
-
-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.`);
-    }
-    /**
-     * 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");
-        }
-        // Default analysis requirements
-        return `### 🔒 Security Analysis (CRITICAL PRIORITY)
-- SQL/XSS/Command injection vulnerabilities
-- Authentication/authorization flaws
-- Input validation and sanitization
-- Hardcoded secrets or credentials
-- Data exposure and privacy concerns
-
-### ⚡ Performance Review
-- Algorithm efficiency and complexity
-- Database query optimization
-- Memory management and resource leaks
-- Caching opportunities
-
-### 🏗️ Code Quality
-- SOLID principles compliance
-- Error handling robustness
-- Code organization and readability
-- Test coverage considerations`;
-    }
-    /**
-     * Build focused analysis prompt separated from context
-     */
-    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"}.
-
-## COMPLETE PR CONTEXT:
-**Title**: ${context.pr.title}
-**Author**: ${context.pr.author}  
-**Description**: ${context.pr.description}
-**Files Changed**: ${context.pr.fileChanges?.length || 0}
-**Existing Comments**: ${JSON.stringify(context.pr.comments || [], null, 2)}
-**Branch**: ${context.identifier.branch}
-**Repository**: ${context.identifier.workspace}/${context.identifier.repository}
-
-## DIFF STRATEGY (${context.diffStrategy.strategy.toUpperCase()}):
-**Reason**: ${context.diffStrategy.reason}
-**File Count**: ${context.diffStrategy.fileCount}
-**Estimated Size**: ${context.diffStrategy.estimatedSize}
-
-## COMPLETE PROJECT CONTEXT:
-${context.projectContext.memoryBank.projectContext || context.projectContext.memoryBank.summary}
-
-## PROJECT RULES & STANDARDS:
-${context.projectContext.clinerules || "No specific rules defined"}
-
-## COMPLETE CODE CHANGES (NO TRUNCATION):
-${diffContent}
-
-## CRITICAL INSTRUCTIONS FOR CODE SNIPPETS:
-
-When you identify an issue in the code, you MUST:
-1. Copy the EXACT line from the diff above, including the diff prefix (+, -, or space at the beginning)
-2. Do NOT modify, clean, or reformat the line
-3. Include the complete line as it appears in the diff
-4. If the issue spans multiple lines, choose the most relevant single line
-
-Example of CORRECT snippet format:
-- For added lines: "+    const password = 'hardcoded123';"
-- For removed lines: "-    return userData;"  
-- For context lines: "     function processPayment() {"
-
-Example of INCORRECT snippet format (DO NOT DO THIS):
-- "const password = 'hardcoded123';" (missing the + prefix)
-- "return userData" (missing the - prefix and semicolon)
-
-## ANALYSIS REQUIREMENTS:
-
-${this.getAnalysisRequirements()}
-
-### 📋 OUTPUT FORMAT
-Return ONLY valid JSON:
-{
-  "violations": [
-    {
-      "type": "inline",
-      "file": "exact/file/path.ext",
-      "code_snippet": "EXACT line from diff INCLUDING the +/- prefix",
-      "search_context": {
-        "before": ["line before from diff with prefix"],
-        "after": ["line after from diff with prefix"]
-      },
-      "severity": "CRITICAL|MAJOR|MINOR|SUGGESTION",
-      "category": "security|performance|maintainability|functionality",
-      "issue": "Brief issue title",
-      "message": "Detailed explanation",
-      "impact": "Potential impact description",
-      "suggestion": "Clean, executable code fix (no diff symbols)"
-    }
-  ],
-  "summary": "Analysis summary",
-  "positiveObservations": ["Good practices found"],
-  "statistics": {
-    "filesReviewed": ${context.diffStrategy.fileCount},
-    "totalIssues": 0,
-    "criticalCount": 0,
-    "majorCount": 0,
-    "minorCount": 0,
-    "suggestionCount": 0
-  }
-}`;
-    }
-    /**
-     * Extract diff content based on strategy
-     */
-    extractDiffContent(context) {
-        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) {
-            const fileDiffArray = Array.from(context.fileDiffs.entries()).map(([file, diff]) => ({
-                file,
-                diff,
-            }));
-            return JSON.stringify(fileDiffArray, null, 2);
-        }
-        return "No diff content available";
-    }
-    /**
-     * Detect project type for better context
-     */
-    detectProjectType(context) {
-        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) {
-                    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";
-    }
-    /**
-     * Assess complexity level for better AI context
-     */
-    assessComplexity(context) {
-        const fileCount = context.diffStrategy.fileCount;
-        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";
-    }
-    /**
-     * Legacy method - kept for compatibility but simplified
-     */
-    buildAnalysisPrompt(context, _options) {
-        // 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.debug("Starting AI analysis...");
-            // Initialize NeuroLink with eval-based dynamic import
-            if (!this.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",
-                repository: `${context.identifier.workspace}/${context.identifier.repository}`,
-                branch: context.identifier.branch,
-                prId: context.identifier.pullRequestId,
-                prTitle: context.pr.title,
-                prAuthor: context.pr.author,
-                fileCount: context.diffStrategy.fileCount,
-                diffStrategy: context.diffStrategy.strategy,
-                analysisType: context.diffStrategy.strategy === "whole"
-                    ? "comprehensive"
-                    : "file-by-file",
-                projectType: this.detectProjectType(context),
-                hasExistingComments: (context.pr.comments?.length || 0) > 0,
-                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
-                temperature: this.aiConfig.temperature || 0.3, // Lower for more focused 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
-            });
-            // Log analytics if available
-            if (result.analytics) {
-                logger.debug(`AI Analytics - Provider: ${result.provider}, Response Time: ${result.responseTime}ms, Quality Score: ${result.evaluation?.overallScore}`);
-            }
-            logger.debug("AI analysis completed, parsing response...");
-            // Modern NeuroLink returns { content: string }
-            const analysisData = this.parseAIResponse(result);
-            // Display AI response for debugging
-            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.debug("No violations array found in AI response");
-                return [];
-            }
-            logger.debug(`AI analysis found ${analysisData.violations.length} violations`);
-            return analysisData.violations;
-        }
-        catch (error) {
-            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.error(`AI analysis failed: ${error.message}`);
-            throw error;
-        }
-    }
-    /**
-     * Post comments to PR using unified context - matching pr-police.js exactly
-     */
-    async postComments(context, violations, _options) {
-        logger.phase("📝 Posting review comments...");
-        // NEW: Apply semantic comment deduplication before posting
-        const duplicateRemover = createExactDuplicateRemover();
-        const deduplicationResult = await duplicateRemover.removeAgainstExistingComments(violations, context.pr.comments || [], this.aiConfig, 85);
-        logger.info(`🔍 Semantic deduplication: ${violations.length} → ${deduplicationResult.uniqueViolations.length} violations ` +
-            `(${deduplicationResult.duplicatesRemoved} duplicates removed)`);
-        // Log deduplication details if any duplicates were found
-        if (deduplicationResult.duplicatesRemoved > 0) {
-            logger.info(duplicateRemover.getCommentDeduplicationStats(deduplicationResult));
-            // Log details of semantic matches
-            deduplicationResult.semanticMatches.forEach((match, index) => {
-                logger.debug(`🎯 Semantic match ${index + 1}: "${match.violation}" matches ${match.comment} ` +
-                    `(${match.similarityScore}% similarity)${match.reasoning ? ` - ${match.reasoning}` : ""}`);
-            });
-        }
-        // Use deduplicated violations for posting
-        const uniqueViolations = deduplicationResult.uniqueViolations;
-        let commentsPosted = 0;
-        let commentsFailed = 0;
-        const failedComments = [];
-        // Post inline comments
-        const inlineViolations = uniqueViolations.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)
-                let cleanFilePath = violation.file;
-                if (cleanFilePath.startsWith("src://")) {
-                    cleanFilePath = cleanFilePath.replace("src://", "");
-                }
-                if (cleanFilePath.startsWith("dst://")) {
-                    cleanFilePath = cleanFilePath.replace("dst://", "");
-                }
-                // Clean code snippet and fix search context - EXACTLY like pr-police.js
-                const processedViolation = this.cleanCodeSnippet(violation);
-                if (!processedViolation) {
-                    logger.debug(`⚠️ Skipping invalid violation for ${cleanFilePath}`);
-                    continue;
-                }
-                const formattedComment = this.formatInlineComment(processedViolation);
-                // Debug logging
-                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.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, {
-                    filePath: cleanFilePath,
-                    lineNumber: undefined, // No line number needed - use pure snippet matching
-                    lineType: processedViolation.line_type || "ADDED", // Default to ADDED if not specified
-                    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
-                });
-                commentsPosted++;
-                logger.debug(`✅ Posted inline comment: ${cleanFilePath} (${processedViolation.issue})`);
-            }
-            catch (error) {
-                commentsFailed++;
-                const errorMsg = error.message;
-                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,
-                });
-            }
-        }
-        // Post summary comment (include failed comments info if any) - only if enabled in config
-        const shouldPostSummary = this.reviewConfig.postSummaryComment !== false; // Default to true if not specified
-        if (uniqueViolations.length > 0 && shouldPostSummary) {
-            try {
-                const summaryComment = this.generateSummaryComment(uniqueViolations, context, failedComments);
-                await this.bitbucketProvider.addComment(context.identifier, summaryComment);
-                commentsPosted++;
-                logger.debug("✅ Posted summary comment");
-            }
-            catch (error) {
-                logger.debug(`❌ Failed to post summary comment: ${error.message}`);
-            }
-        }
-        else if (uniqueViolations.length > 0 && !shouldPostSummary) {
-            logger.debug("📝 Summary comment posting disabled in configuration");
-        }
-        logger.success(`✅ Posted ${commentsPosted} comments successfully`);
-        if (commentsFailed > 0) {
-            logger.warn(`⚠️ Failed to post ${commentsFailed} inline comments`);
-        }
-        return uniqueViolations;
-    }
-    /**
-     * Format inline comment for specific violation
-     */
-    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" },
-        };
-        const categoryIcons = {
-            security: "🔒",
-            performance: "⚡",
-            maintainability: "🏗️",
-            functionality: "⚙️",
-            error_handling: "🛡️",
-            testing: "🧪",
-            general: "📋",
-        };
-        const config = severityConfig[violation.severity] || severityConfig.MINOR;
-        const categoryIcon = categoryIcons[violation.category] || categoryIcons.general;
-        let comment = `${config.badge}
-
-**${categoryIcon} ${violation.issue}**
-
-**Category**: ${violation.category.replace(/_/g, " ").replace(/\b\w/g, (l) => l.toUpperCase())}
-
-**Issue**: ${violation.message}`;
-        if (violation.impact) {
-            comment += `\n\n**Impact**: ${violation.impact}`;
-        }
-        // Add suggested fix section if suggestion is provided
-        if (violation.suggestion) {
-            comment += `\n\n**Suggested Fix**:\n`;
-            // Detect the language for syntax highlighting
-            const language = this.detectLanguageFromFile(violation.file || "");
-            // Use proper markdown escaping for code blocks
-            const escapedCodeBlock = this.escapeMarkdownCodeBlock(violation.suggestion, language);
-            comment += escapedCodeBlock;
-        }
-        comment += `\n\n---\n*🛡️ Automated review by **Yama** • Powered by AI*`;
-        return comment;
-    }
-    /**
-     * Generate comprehensive summary comment with failed comments info
-     */
-    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";
-        let comment = `
-╭─────────────────────────────────────────────────────────────╮
-│                    ⚔️ **YAMA REVIEW REPORT** ⚔️               │
-╰─────────────────────────────────────────────────────────────╯
-
-## ${statusEmoji} **${statusText}**
-
-### 📊 **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"} |
-
-### 🔍 **Analysis Summary**
-- **📁 Files Analyzed**: ${context.diffStrategy.fileCount}
-- **📊 Strategy Used**: ${context.diffStrategy.strategy} (${context.diffStrategy.reason})
-- **🎯 Total Issues**: ${stats.totalIssues}
-- **🏷️ PR**: #${context.pr.id} - "${context.pr.title}"`;
-        // Add category breakdown if there are violations
-        const violationsByCategory = this.groupViolationsByCategory(violations);
-        if (Object.keys(violationsByCategory).length > 0) {
-            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: "📋",
-                };
-                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
-        if (failedComments.length > 0) {
-            comment += `\n\n### ⚠️ **Note on Inline Comments**\n`;
-            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`;
-            }
-        }
-        // Add recommendation
-        const recommendation = stats.criticalCount > 0
-            ? "🚨 **URGENT**: Critical security issues must be resolved before merge"
-            : stats.majorCount > 0
-                ? "⚠️ **RECOMMENDED**: Address major issues before merge"
-                : stats.minorCount > 0
-                    ? "📝 **OPTIONAL**: Consider addressing minor improvements"
-                    : "✅ **APPROVED**: Code meets security and quality standards";
-        comment += `\n\n### 💡 **Recommendation**
-${recommendation}
-
----
-**🛡️ Automated Security & Quality Review**  
-*Powered by Yama AI • Keeping your code secure and maintainable* 🚀`;
-        return comment;
-    }
-    /**
-     * Helper methods for processing violations
-     */
-    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):\/\//, "");
-        // Log the cleaning for debugging
-        if (cleaned !== filePath) {
-            logger.debug(`Cleaned file path: ${filePath} -> ${cleaned}`);
-        }
-        return cleaned;
-    }
-    /**
-     * Extract exact file path from diff
-     */
-    extractFilePathFromDiff(diff, fileName) {
-        const lines = diff.split("\n");
-        for (const line of lines) {
-            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))) {
-                    return match[2]; // Return the 'b/' path (destination)
-                }
-            }
-        }
-        return null;
-    }
-    /**
-     * Extract line number from diff for a specific code snippet
-     */
-    extractLineNumberFromDiff(fileDiff, codeSnippet) {
-        const lines = fileDiff.split("\n");
-        let currentNewLine = 0;
-        let currentOldLine = 0;
-        let inHunk = false;
-        // Debug logging
-        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 @@)
-            const hunkMatch = line.match(/@@ -(\d+),?\d* \+(\d+),?\d* @@/);
-            if (hunkMatch) {
-                // Hunk headers show the starting line numbers (1-based)
-                currentOldLine = parseInt(hunkMatch[1]);
-                currentNewLine = parseInt(hunkMatch[2]);
-                inHunk = true;
-                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(" "))) {
-                continue;
-            }
-            // Check if this line matches our snippet
-            if (line === codeSnippet) {
-                let resultLine;
-                let lineType;
-                if (line.startsWith("+")) {
-                    resultLine = currentNewLine;
-                    lineType = "ADDED";
-                }
-                else if (line.startsWith("-")) {
-                    resultLine = currentOldLine;
-                    lineType = "REMOVED";
-                }
-                else {
-                    resultLine = currentNewLine;
-                    lineType = "CONTEXT";
-                }
-                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("+")) {
-                currentNewLine++;
-            }
-            else if (line.startsWith("-")) {
-                currentOldLine++;
-            }
-            else if (line.startsWith(" ")) {
-                currentNewLine++;
-                currentOldLine++;
-            }
-        }
-        logger.debug(`Snippet not found in diff`);
-        return null;
-    }
-    /**
-     * Escape markdown code blocks properly
-     */
-    escapeMarkdownCodeBlock(code, language) {
-        // If code contains triple backticks, use quadruple backticks
-        if (code.includes("```")) {
-            return `\`\`\`\`${language}\n${code}\n\`\`\`\``;
-        }
-        return `\`\`\`${language}\n${code}\n\`\`\``;
-    }
-    cleanCodeSnippet(violation) {
-        try {
-            // Clone the violation to avoid modifying the original - EXACTLY like pr-police.js
-            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)) {
-                    fixed.search_context.before = this.splitArrayLines(fixed.search_context.before);
-                }
-                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";
-                }
-                else if (fixed.code_snippet.startsWith("-")) {
-                    fixed.line_type = "REMOVED";
-                }
-                else {
-                    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();
-            }
-            // 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")
-                    .trim();
-            }
-            return fixed;
-        }
-        catch (error) {
-            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));
-            }
-            else {
-                result.push(item);
-            }
-        }
-        return result;
-    }
-    groupViolationsByCategory(violations) {
-        const grouped = {};
-        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,
-            totalIssues: violations.length,
-            filesReviewed: new Set(violations.filter((v) => v.file).map((v) => v.file)).size || 1,
-        };
-    }
-    generateReviewResult(violations, _duration, _context, processingStrategy) {
-        const stats = this.calculateStats(violations);
-        return {
-            violations,
-            summary: `Review found ${stats.criticalCount} critical, ${stats.majorCount} major, ${stats.minorCount} minor issues, and ${stats.suggestionCount} suggestions`,
-            statistics: {
-                filesReviewed: stats.filesReviewed,
-                totalIssues: stats.totalIssues,
-                criticalCount: stats.criticalCount,
-                majorCount: stats.majorCount,
-                minorCount: stats.minorCount,
-                suggestionCount: stats.suggestionCount,
-                processingStrategy,
-            },
-            positiveObservations: [], // Could be extracted from AI response
-        };
-    }
-    // ============================================================================
-    // BATCH PROCESSING METHODS
-    // ============================================================================
-    /**
-     * Get batch processing configuration with defaults
-     */
-    getBatchProcessingConfig() {
-        const defaultConfig = {
-            enabled: true,
-            maxFilesPerBatch: 3,
-            prioritizeSecurityFiles: true,
-            parallelBatches: false, // Keep for backward compatibility
-            batchDelayMs: 1000,
-            singleRequestThreshold: 5, // Use single request for ≤5 files
-            // NEW: Parallel processing defaults
-            parallel: {
-                enabled: true, // Enable parallel processing by default
-                maxConcurrentBatches: 3,
-                rateLimitStrategy: "fixed",
-                tokenBudgetDistribution: "equal",
-                failureHandling: "continue",
-            },
-        };
-        const mergedConfig = {
-            ...defaultConfig,
-            ...this.reviewConfig.batchProcessing,
-        };
-        // Merge parallel config separately to handle nested object properly
-        if (mergedConfig.parallel && this.reviewConfig.batchProcessing?.parallel) {
-            mergedConfig.parallel = {
-                ...defaultConfig.parallel,
-                ...this.reviewConfig.batchProcessing.parallel,
-            };
-        }
-        else if (!mergedConfig.parallel) {
-            mergedConfig.parallel = defaultConfig.parallel;
-        }
-        return mergedConfig;
-    }
-    /**
-     * Determine if batch processing should be used
-     */
-    shouldUseBatchProcessing(context, batchConfig) {
-        if (!batchConfig.enabled) {
-            logger.debug("Batch processing disabled in config");
-            return false;
-        }
-        const fileCount = context.diffStrategy.fileCount;
-        if (fileCount <= batchConfig.singleRequestThreshold) {
-            logger.debug(`File count (${fileCount}) ≤ threshold (${batchConfig.singleRequestThreshold}), using single request`);
-            return false;
-        }
-        // Force batch processing for file-by-file strategy with many files
-        if (context.diffStrategy.strategy === "file-by-file" && fileCount > 10) {
-            logger.debug(`File-by-file strategy with ${fileCount} files, forcing batch processing`);
-            return true;
-        }
-        logger.debug(`File count (${fileCount}) > threshold (${batchConfig.singleRequestThreshold}), using batch processing`);
-        return true;
-    }
-    /**
-     * Main batch processing method with parallel processing support
-     */
-    async reviewWithBatchProcessing(context, options, batchConfig) {
-        const startTime = Date.now();
-        try {
-            // Step 1: Prioritize and organize files
-            const prioritizedFiles = await this.prioritizeFiles(context, batchConfig);
-            logger.info(`📋 Prioritized ${prioritizedFiles.length} files: ${prioritizedFiles.filter((f) => f.priority === "high").length} high, ${prioritizedFiles.filter((f) => f.priority === "medium").length} medium, ${prioritizedFiles.filter((f) => f.priority === "low").length} low priority`);
-            // Step 2: Create batches
-            const batches = this.createBatches(prioritizedFiles, batchConfig);
-            logger.info(`📦 Created ${batches.length} batches (max ${batchConfig.maxFilesPerBatch} files per batch)`);
-            // Step 3: Determine processing strategy
-            const useParallel = batchConfig.parallel?.enabled && batches.length > 1;
-            if (useParallel) {
-                logger.info(`🚀 Using parallel processing: ${batches.length} batches, max ${batchConfig.parallel?.maxConcurrentBatches} concurrent`);
-                return await this.processInParallel(batches, context, options, batchConfig);
-            }
-            else {
-                logger.info(`🔄 Using serial processing: ${batches.length} batches`);
-                return await this.processSerially(batches, context, options, batchConfig);
-            }
-        }
-        catch (error) {
-            logger.error(`Batch processing failed: ${error.message}`);
-            throw error;
-        }
-    }
-    /**
-     * Process batches in parallel with concurrency control
-     */
-    async processInParallel(batches, context, options, batchConfig) {
-        const startTime = Date.now();
-        const parallelConfig = batchConfig.parallel;
-        // Calculate optimal concurrency
-        const avgTokensPerBatch = batches.reduce((sum, b) => sum + b.estimatedTokens, 0) / batches.length;
-        const optimalConcurrency = calculateOptimalConcurrency(batches.length, parallelConfig.maxConcurrentBatches, avgTokensPerBatch, this.getSafeTokenLimit());
-        // Initialize concurrency control
-        const semaphore = new Semaphore(optimalConcurrency);
-        const tokenBudget = new TokenBudgetManager(this.getSafeTokenLimit() * 0.8); // 80% for safety
-        // NEW: Pre-allocate tokens based on distribution strategy
-        const distributionStrategy = parallelConfig.tokenBudgetDistribution || "equal";
-        logger.info(`🎯 Using ${distributionStrategy} token distribution strategy for ${batches.length} batches`);
-        const tokenAllocations = this.preAllocateTokens(batches, tokenBudget, distributionStrategy);
-        if (!tokenAllocations) {
-            const totalRequired = batches.reduce((sum, b) => sum + b.estimatedTokens, 0);
-            const totalBudget = tokenBudget.getTotalBudget();
-            throw new Error(`Insufficient token budget: required ${totalRequired}, available ${totalBudget}. ` +
-                `Consider reducing batch count (current: ${batches.length}) or increasing token limit.`);
-        }
-        // Apply pre-allocated tokens to the budget manager
-        if (!tokenBudget.preAllocateAllBatches(tokenAllocations)) {
-            throw new Error("Failed to pre-allocate tokens for all batches");
-        }
-        logger.info(`🎯 Parallel processing: ${optimalConcurrency} concurrent batches, ${tokenBudget.getTotalBudget()} token budget (${distributionStrategy} distribution)`);
-        // Log allocation details
-        tokenAllocations.forEach((tokens, batchIndex) => {
-            logger.debug(`Batch ${batchIndex + 1}: ${tokens} tokens allocated`);
-        });
-        const batchResults = new Array(batches.length);
-        const allViolations = [];
-        const processingPromises = [];
-        // Process batches with controlled concurrency
-        for (let i = 0; i < batches.length; i++) {
-            const batch = batches[i];
-            const processingPromise = this.processBatchWithConcurrency(batch, context, options, semaphore, tokenBudget, i, batches.length)
-                .then((result) => {
-                batchResults[i] = result; // Maintain order
-                if (result.violations) {
-                    allViolations.push(...result.violations);
-                }
-            })
-                .catch((error) => {
-                logger.error(`❌ Batch ${i + 1} failed: ${error.message}`);
-                batchResults[i] = {
-                    batchIndex: i,
-                    files: batch.files,
-                    violations: [],
-                    processingTime: 0,
-                    error: error.message,
-                };
-                // Handle failure strategy
-                if (parallelConfig.failureHandling === "stop-all") {
-                    throw error;
-                }
-            });
-            processingPromises.push(processingPromise);
-            // Add small delay between batch starts to avoid overwhelming
-            if (i < batches.length - 1) {
-                await new Promise((resolve) => setTimeout(resolve, 200));
-            }
-        }
-        // Wait for all batches to complete
-        await Promise.allSettled(processingPromises);
-        // Filter out undefined results and sort by batch index
-        const validResults = batchResults
-            .filter((r) => r !== undefined)
-            .sort((a, b) => a.batchIndex - b.batchIndex);
-        const totalTime = Date.now() - startTime;
-        const avgBatchSize = batches.reduce((sum, b) => sum + b.files.length, 0) / batches.length;
-        const budgetStatus = tokenBudget.getBudgetStatus();
-        logger.success(`🎯 Parallel processing completed: ${allViolations.length} total violations from ${batches.length} batches in ${Math.round(totalTime / 1000)}s (avg ${avgBatchSize.toFixed(1)} files/batch, ${budgetStatus.utilizationPercent}% token usage)`);
-        return { violations: allViolations, batchResults: validResults };
-    }
-    /**
-     * Process batches serially (original implementation)
-     */
-    async processSerially(batches, context, options, batchConfig) {
-        const startTime = Date.now();
-        const batchResults = [];
-        const allViolations = [];
-        for (let i = 0; i < batches.length; i++) {
-            const batch = batches[i];
-            logger.info(`🔄 Processing batch ${i + 1}/${batches.length} (${batch.files.length} files, ${batch.priority} priority, serial)`);
-            try {
-                const batchResult = await this.processBatch(batch, context, options);
-                batchResults.push(batchResult);
-                allViolations.push(...batchResult.violations);
-                logger.info(`✅ Batch ${i + 1} completed: ${batchResult.violations.length} violations found in ${Math.round(batchResult.processingTime / 1000)}s`);
-                // Add delay between batches if configured
-                if (i < batches.length - 1 && batchConfig.batchDelayMs > 0) {
-                    logger.debug(`⏳ Waiting ${batchConfig.batchDelayMs}ms before next batch`);
-                    await new Promise((resolve) => setTimeout(resolve, batchConfig.batchDelayMs));
-                }
-            }
-            catch (error) {
-                logger.error(`❌ Batch ${i + 1} failed: ${error.message}`);
-                // Record failed batch
-                batchResults.push({
-                    batchIndex: i,
-                    files: batch.files,
-                    violations: [],
-                    processingTime: Date.now() - startTime,
-                    error: error.message,
-                });
-            }
-        }
-        const totalTime = Date.now() - startTime;
-        const avgBatchSize = batches.reduce((sum, b) => sum + b.files.length, 0) / batches.length;
-        logger.success(`🎯 Serial processing completed: ${allViolations.length} total violations from ${batches.length} batches in ${Math.round(totalTime / 1000)}s (avg ${avgBatchSize.toFixed(1)} files/batch)`);
-        return { violations: allViolations, batchResults };
-    }
-    /**
-     * Pre-allocate tokens based on distribution strategy with proper integer arithmetic
-     */
-    preAllocateTokens(batches, tokenBudget, strategy) {
-        // Ensure we're working with integer budget to avoid floating-point issues
-        const totalBudget = Math.floor(tokenBudget.getTotalBudget());
-        const allocations = new Map();
-        if (strategy === "equal") {
-            // Equal distribution: divide budget equally among all batches with proper remainder handling
-            const baseTokens = Math.floor(totalBudget / batches.length);
-            const remainder = totalBudget % batches.length;
-            if (baseTokens < 1000) {
-                // Minimum viable tokens per batch
-                logger.error(`Equal distribution would give ${baseTokens} tokens per batch, which is insufficient`);
-                return null;
-            }
-            let totalAllocated = 0;
-            for (let i = 0; i < batches.length; i++) {
-                // Distribute remainder to first few batches
-                const tokens = baseTokens + (i < remainder ? 1 : 0);
-                allocations.set(i, tokens);
-                totalAllocated += tokens;
-            }
-            // Double-check that we haven't exceeded budget due to any calculation errors
-            if (totalAllocated > totalBudget) {
-                logger.error(`Equal distribution calculation error: ${totalAllocated} > ${totalBudget}`);
-                // Adjust the last batch to fit within budget
-                const lastBatchIndex = batches.length - 1;
-                const lastBatchTokens = allocations.get(lastBatchIndex);
-                const adjustment = totalAllocated - totalBudget;
-                const newLastBatchTokens = lastBatchTokens - adjustment;
-                if (newLastBatchTokens < 1000) {
-                    logger.error(`Adjustment would result in last batch having ${newLastBatchTokens} tokens, which is below the minimum threshold (1000). Aborting allocation.`);
-                    return null;
-                }
-                allocations.set(lastBatchIndex, newLastBatchTokens);
-                totalAllocated = totalBudget;
-                logger.warn(`Adjusted last batch by -${adjustment} tokens to fit budget`);
-            }
-            logger.info(`Equal distribution: ${baseTokens} tokens per batch for ${batches.length} batches`);
-            logger.debug(`Pre-allocated ${totalAllocated} tokens across ${batches.length} batches (${totalBudget - totalAllocated} remaining)`);
-        }
-        else if (strategy === "weighted") {
-            // Weighted distribution: try weighted first, automatically fallback to equal if needed
-            logger.debug(`Attempting weighted distribution...`);
-            const weightedResult = this.tryWeightedAllocation(batches, totalBudget);
-            if (weightedResult) {
-                // Weighted allocation succeeded
-                weightedResult.forEach((tokens, batchIndex) => {
-                    allocations.set(batchIndex, tokens);
-                });
-                logger.info(`✅ Weighted distribution: optimal allocation successful`);
-                logger.debug(`Pre-allocated ${Array.from(weightedResult.values()).reduce((sum, tokens) => sum + tokens, 0)} tokens across ${batches.length} batches`);
-            }
-            else {
-                // Weighted allocation failed, automatically fallback to equal distribution
-                logger.warn(`⚠️ Weighted distribution: insufficient budget for optimal allocation, falling back to equal distribution`);
-                const equalResult = this.tryEqualAllocation(batches, totalBudget);
-                if (!equalResult) {
-                    logger.error(`Weighted distribution: both optimal and equal allocation failed`);
-                    return null;
-                }
-                equalResult.forEach((tokens, batchIndex) => {
-                    allocations.set(batchIndex, tokens);
-                });
-                logger.info(`✅ Weighted distribution: equal allocation fallback successful`);
-                logger.debug(`Pre-allocated ${Array.from(equalResult.values()).reduce((sum, tokens) => sum + tokens, 0)} tokens across ${batches.length} batches`);
-            }
-        }
-        // Final validation with strict integer checking
-        const totalAllocated = Array.from(allocations.values()).reduce((sum, tokens) => sum + tokens, 0);
-        if (totalAllocated > totalBudget) {
-            logger.error(`CRITICAL: Total allocation (${totalAllocated}) exceeds budget (${totalBudget}) - this should never happen`);
-            logger.error(`Budget type: ${typeof totalBudget}, Allocation type: ${typeof totalAllocated}`);
-            logger.error(`Individual allocations: ${Array.from(allocations.entries())
-                .map(([i, tokens]) => `batch${i}:${tokens}`)
-                .join(", ")}`);
-            throw new Error(`Total allocation (${totalAllocated}) exceeds budget (${totalBudget}) - this should never happen`);
-        }
-        return allocations;
-    }
-    /**
-     * Try weighted allocation for batches
-     */
-    tryWeightedAllocation(batches, totalBudget) {
-        const totalEstimated = batches.reduce((sum, batch) => sum + batch.estimatedTokens, 0);
-        if (totalEstimated > totalBudget) {
-            logger.debug(`Total estimated tokens (${totalEstimated}) exceed budget (${totalBudget})`);
-            return null;
-        }
-        let totalAllocated = 0;
-        const minTokensPerBatch = 1000;
-        const allocations = new Map();
-        for (let i = 0; i < batches.length; i++) {
-            const batch = batches[i];
-            const weight = batch.estimatedTokens / totalEstimated;
-            const allocation = Math.floor(weight * totalBudget);
-            const finalAllocation = Math.max(allocation, minTokensPerBatch);
-            allocations.set(i, finalAllocation);
-            totalAllocated += finalAllocation;
-        }
-        // Check if we exceeded budget due to minimum allocations
-        if (totalAllocated > totalBudget) {
-            logger.debug(`Weighted allocation with minimums (${totalAllocated}) exceeds budget (${totalBudget})`);
-            return null;
-        }
-        return allocations;
-    }
-    /**
-     * Try equal allocation for batches
-     */
-    tryEqualAllocation(batches, totalBudget) {
-        const baseTokens = Math.floor(totalBudget / batches.length);
-        const remainder = totalBudget % batches.length;
-        if (baseTokens < 1000) {
-            // Minimum viable tokens per batch
-            logger.debug(`Equal distribution would give ${baseTokens} tokens per batch, which is insufficient`);
-            return null;
-        }
-        const allocations = new Map();
-        let totalAllocated = 0;
-        for (let i = 0; i < batches.length; i++) {
-            // Distribute remainder to first few batches
-            const tokens = baseTokens + (i < remainder ? 1 : 0);
-            allocations.set(i, tokens);
-            totalAllocated += tokens;
-        }
-        // Double-check that we haven't exceeded budget due to any calculation errors
-        if (totalAllocated > totalBudget) {
-            logger.debug(`Equal distribution calculation error: ${totalAllocated} > ${totalBudget}`);
-            // Adjust the last batch to fit within budget
-            const lastBatchIndex = batches.length - 1;
-            const lastBatchTokens = allocations.get(lastBatchIndex);
-            const adjustment = totalAllocated - totalBudget;
-            const newLastBatchTokens = lastBatchTokens - adjustment;
-            if (newLastBatchTokens < 1000) {
-                logger.error(`Adjustment would result in last batch having ${newLastBatchTokens} tokens, which is below the minimum threshold (1000). Aborting allocation.`);
-                return null;
-            }
-            allocations.set(lastBatchIndex, newLastBatchTokens);
-        }
-        return allocations;
-    }
-    /**
-     * Process a single batch with concurrency control
-     */
-    async processBatchWithConcurrency(batch, context, options, semaphore, tokenBudget, batchIndex, totalBatches) {
-        // Acquire semaphore permit
-        await semaphore.acquire();
-        try {
-            // NEW: In pre-allocation mode, tokens are already allocated, just verify and mark as processing
-            if (tokenBudget.isPreAllocationMode()) {
-                const batchState = tokenBudget.getBatchState(batchIndex);
-                if (batchState !== "pending") {
-                    throw new Error(`Batch ${batchIndex + 1} is not in pending state (current: ${batchState})`);
-                }
-                // Mark as processing (this is handled in allocateForBatch for pre-allocation mode)
-                if (!tokenBudget.allocateForBatch(batchIndex, batch.estimatedTokens)) {
-                    throw new Error(`Failed to mark batch ${batchIndex + 1} as processing`);
-                }
-            }
-            else {
-                // Legacy mode: allocate tokens dynamically
-                if (!tokenBudget.allocateForBatch(batchIndex, batch.estimatedTokens)) {
-                    throw new Error(`Insufficient token budget for batch ${batchIndex + 1}`);
-                }
-            }
-            logger.info(`🔄 Processing batch ${batchIndex + 1}/${totalBatches} (${batch.files.length} files, parallel)`);
-            // Process the batch (existing logic)
-            const result = await this.processBatch(batch, context, options);
-            logger.info(`✅ Batch ${batchIndex + 1} completed: ${result.violations.length} violations in ${Math.round(result.processingTime / 1000)}s`);
-            return result;
-        }
-        catch (error) {
-            // Mark batch as failed in token budget
-            tokenBudget.markBatchFailed(batchIndex, error.message);
-            throw error;
-        }
-        finally {
-            // Always release resources
-            tokenBudget.releaseBatch(batchIndex);
-            semaphore.release();
-        }
-    }
-    /**
-     * Prioritize files based on security importance and file type
-     */
-    async prioritizeFiles(context, batchConfig) {
-        const files = context.pr.fileChanges || [];
-        const prioritizedFiles = [];
-        for (const filePath of files) {
-            const priority = this.calculateFilePriority(filePath, batchConfig);
-            const estimatedTokens = await this.estimateFileTokens(filePath, context);
-            prioritizedFiles.push({
-                path: filePath,
-                priority,
-                estimatedTokens,
-                diff: context.fileDiffs?.get(filePath),
-            });
-        }
-        // Sort by priority (high -> medium -> low) then by estimated tokens (smaller first)
-        prioritizedFiles.sort((a, b) => {
-            const priorityOrder = { high: 0, medium: 1, low: 2 };
-            const priorityDiff = priorityOrder[a.priority] - priorityOrder[b.priority];
-            if (priorityDiff !== 0) {
-                return priorityDiff;
-            }
-            return a.estimatedTokens - b.estimatedTokens;
-        });
-        return prioritizedFiles;
-    }
-    /**
-     * Calculate file priority based on path and content
-     */
-    calculateFilePriority(filePath, batchConfig) {
-        if (!batchConfig.prioritizeSecurityFiles) {
-            return "medium"; // All files same priority if not prioritizing
-        }
-        const path = filePath.toLowerCase();
-        // High priority: Security-sensitive files
-        const highPriorityPatterns = [
-            /auth/i,
-            /login/i,
-            /password/i,
-            /token/i,
-            /jwt/i,
-            /oauth/i,
-            /crypto/i,
-            /encrypt/i,
-            /decrypt/i,
-            /hash/i,
-            /security/i,
-            /payment/i,
-            /billing/i,
-            /transaction/i,
-            /money/i,
-            /wallet/i,
-            /admin/i,
-            /privilege/i,
-            /permission/i,
-            /role/i,
-            /access/i,
-            /config/i,
-            /env/i,
-            /secret/i,
-            /key/i,
-            /credential/i,
-            /api/i,
-            /endpoint/i,
-            /route/i,
-            /controller/i,
-            /middleware/i,
-        ];
-        if (highPriorityPatterns.some((pattern) => pattern.test(path))) {
-            return "high";
-        }
-        // Low priority: Documentation, tests, config files
-        const lowPriorityPatterns = [
-            /\.md$/i,
-            /\.txt$/i,
-            /readme/i,
-            /changelog/i,
-            /license/i,
-            /test/i,
-            /spec/i,
-            /\.test\./i,
-            /\.spec\./i,
-            /__tests__/i,
-            /\.json$/i,
-            /\.yaml$/i,
-            /\.yml$/i,
-            /\.toml$/i,
-            /\.ini$/i,
-            /\.lock$/i,
-            /package-lock/i,
-            /yarn\.lock/i,
-            /pnpm-lock/i,
-            /\.gitignore/i,
-            /\.eslint/i,
-            /\.prettier/i,
-            /tsconfig/i,
-            /\.svg$/i,
-            /\.png$/i,
-            /\.jpg$/i,
-            /\.jpeg$/i,
-            /\.gif$/i,
-        ];
-        if (lowPriorityPatterns.some((pattern) => pattern.test(path))) {
-            return "low";
-        }
-        // Medium priority: Everything else
-        return "medium";
-    }
-    /**
-     * Estimate token count for a file
-     */
-    async estimateFileTokens(filePath, context) {
-        try {
-            let content = "";
-            if (context.fileDiffs?.has(filePath)) {
-                content = context.fileDiffs.get(filePath) || "";
-            }
-            else if (context.prDiff) {
-                // Extract file content from whole diff
-                const diffLines = context.prDiff.diff.split("\n");
-                let inFile = false;
-                for (const line of diffLines) {
-                    if (line.startsWith("diff --git") && line.includes(filePath)) {
-                        inFile = true;
-                        continue;
-                    }
-                    if (inFile && line.startsWith("diff --git")) {
-                        break;
-                    }
-                    if (inFile) {
-                        content += line + "\n";
-                    }
-                }
-            }
-            // Rough estimation: ~4 characters per token
-            const estimatedTokens = Math.ceil(content.length / 4);
-            // Add base overhead for context and prompts
-            const baseOverhead = 1000;
-            return estimatedTokens + baseOverhead;
-        }
-        catch (error) {
-            logger.debug(`Error estimating tokens for ${filePath}: ${error.message}`);
-            return 2000; // Default estimate
-        }
-    }
-    /**
-     * Create batches from prioritized files
-     */
-    createBatches(prioritizedFiles, batchConfig) {
-        const batches = [];
-        const maxTokensPerBatch = this.getSafeTokenLimit() * 0.7; // Use 70% of limit for safety
-        let currentBatch = {
-            files: [],
-            priority: "medium",
-            estimatedTokens: 0,
-            batchIndex: 0,
-        };
-        for (const file of prioritizedFiles) {
-            const wouldExceedTokens = currentBatch.estimatedTokens + file.estimatedTokens > maxTokensPerBatch;
-            const wouldExceedFileCount = currentBatch.files.length >= batchConfig.maxFilesPerBatch;
-            if ((wouldExceedTokens || wouldExceedFileCount) &&
-                currentBatch.files.length > 0) {
-                // Finalize current batch
-                batches.push(currentBatch);
-                // Start new batch
-                currentBatch = {
-                    files: [],
-                    priority: file.priority,
-                    estimatedTokens: 0,
-                    batchIndex: batches.length,
-                };
-            }
-            // Add file to current batch
-            currentBatch.files.push(file.path);
-            currentBatch.estimatedTokens += file.estimatedTokens;
-            // Update batch priority to highest priority file in batch
-            if (file.priority === "high" ||
-                (file.priority === "medium" && currentBatch.priority === "low")) {
-                currentBatch.priority = file.priority;
-            }
-        }
-        // Add final batch if it has files
-        if (currentBatch.files.length > 0) {
-            batches.push(currentBatch);
-        }
-        return batches;
-    }
-    /**
-     * Process a single batch of files
-     */
-    async processBatch(batch, context, options) {
-        const startTime = Date.now();
-        try {
-            // Create batch-specific context
-            const batchContext = this.createBatchContext(batch, context);
-            // Build batch-specific prompt
-            const batchPrompt = this.buildBatchAnalysisPrompt(batchContext, batch, options);
-            // Analyze with AI
-            const violations = await this.analyzeWithAI(batchPrompt, batchContext);
-            const processingTime = Date.now() - startTime;
-            return {
-                batchIndex: batch.batchIndex,
-                files: batch.files,
-                violations,
-                processingTime,
-            };
-        }
-        catch (error) {
-            const processingTime = Date.now() - startTime;
-            return {
-                batchIndex: batch.batchIndex,
-                files: batch.files,
-                violations: [],
-                processingTime,
-                error: error.message,
-            };
-        }
-    }
-    /**
-     * Create context for a specific batch
-     */
-    createBatchContext(batch, originalContext) {
-        // Create a filtered context containing only the files in this batch
-        const batchFileDiffs = new Map();
-        if (originalContext.fileDiffs) {
-            for (const filePath of batch.files) {
-                const diff = originalContext.fileDiffs.get(filePath);
-                if (diff) {
-                    batchFileDiffs.set(filePath, diff);
-                }
-            }
-        }
-        return {
-            ...originalContext,
-            fileDiffs: batchFileDiffs,
-            diffStrategy: {
-                ...originalContext.diffStrategy,
-                fileCount: batch.files.length,
-                strategy: "file-by-file", // Always use file-by-file for batches
-                reason: `Batch processing ${batch.files.length} files`,
-            },
-            pr: {
-                ...originalContext.pr,
-                fileChanges: batch.files,
-            },
-        };
-    }
-    /**
-     * Build analysis prompt for a specific batch
-     */
-    buildBatchAnalysisPrompt(batchContext, batch, options) {
-        const diffContent = this.extractDiffContent(batchContext);
-        return `Conduct a focused security and quality analysis of this batch of ${batch.files.length} files (${batch.priority} priority).
-
-## BATCH CONTEXT:
-**Batch**: ${batch.batchIndex + 1}
-**Files**: ${batch.files.length}
-**Priority**: ${batch.priority}
-**Files in batch**: ${batch.files.join(", ")}
-
-## PR CONTEXT:
-**Title**: ${batchContext.pr.title}
-**Author**: ${batchContext.pr.author}
-**Repository**: ${batchContext.identifier.workspace}/${batchContext.identifier.repository}
-
-## PROJECT CONTEXT:
-${batchContext.projectContext.memoryBank.projectContext || batchContext.projectContext.memoryBank.summary}
-
-## PROJECT RULES & STANDARDS:
-${batchContext.projectContext.clinerules || "No specific rules defined"}
-
-## BATCH CODE CHANGES:
-${diffContent}
-
-## CRITICAL INSTRUCTIONS FOR CODE SNIPPETS:
-
-When you identify an issue in the code, you MUST:
-1. Copy the EXACT line from the diff above, including the diff prefix (+, -, or space at the beginning)
-2. Do NOT modify, clean, or reformat the line
-3. Include the complete line as it appears in the diff
-4. If the issue spans multiple lines, choose the most relevant single line
-
-## ANALYSIS REQUIREMENTS:
-
-${this.getAnalysisRequirements()}
-
-### 📋 OUTPUT FORMAT
-Return ONLY valid JSON:
-{
-  "violations": [
-    {
-      "type": "inline",
-      "file": "exact/file/path.ext",
-      "code_snippet": "EXACT line from diff INCLUDING the +/- prefix",
-      "search_context": {
-        "before": ["line before from diff with prefix"],
-        "after": ["line after from diff with prefix"]
-      },
-      "severity": "CRITICAL|MAJOR|MINOR|SUGGESTION",
-      "category": "security|performance|maintainability|functionality",
-      "issue": "Brief issue title",
-      "message": "Detailed explanation",
-      "impact": "Potential impact description",
-      "suggestion": "Clean, executable code fix (no diff symbols)"
-    }
-  ],
-  "summary": "Batch analysis summary",
-  "positiveObservations": ["Good practices found"],
-  "statistics": {
-    "filesReviewed": ${batch.files.length},
-    "totalIssues": 0,
-    "criticalCount": 0,
-    "majorCount": 0,
-    "minorCount": 0,
-    "suggestionCount": 0
-  }
-}`;
-    }
-    /**
-     * Utility methods
-     */
-    parseAIResponse(result) {
-        try {
-            const responseText = result.content || result.text || result.response || "";
-            if (!responseText) {
-                return { violations: [] };
-            }
-            // Extract JSON from response
-            const jsonMatch = responseText.match(/\{[\s\S]*\}/);
-            if (jsonMatch) {
-                return JSON.parse(jsonMatch[0]);
-            }
-            return { violations: [] };
-        }
-        catch (error) {
-            logger.debug(`Failed to parse AI response: ${error.message}`);
-            return { violations: [] };
-        }
-    }
-    /**
-     * Extract line information for comment from context
-     */
-    extractLineInfoForComment(violation, context) {
-        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) {
-                // Extract file diff from whole diff
-                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")) {
-                        // Check if any variation matches
-                        for (const pathVariation of filePathVariations) {
-                            if (line.includes(pathVariation)) {
-                                fileStartIndex = i;
-                                break;
-                            }
-                        }
-                        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");
-                }
-            }
-            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.debug(`Found diff for ${violation.file} using variation: ${path}`);
-                        break;
-                    }
-                }
-                // If still not found, try to find by partial match
-                if (!fileDiff) {
-                    for (const [key, value] of context.fileDiffs.entries()) {
-                        if (key.endsWith(violation.file) || violation.file.endsWith(key)) {
-                            fileDiff = value;
-                            logger.debug(`Found diff for ${violation.file} using partial match: ${key}`);
-                            break;
-                        }
-                    }
-                }
-            }
-            if (fileDiff) {
-                const lineInfo = this.extractLineNumberFromDiff(fileDiff, violation.code_snippet);
-                if (lineInfo) {
-                    logger.debug(`Extracted line info for ${violation.file}: line ${lineInfo.lineNumber}, type ${lineInfo.lineType}`);
-                }
-                return lineInfo;
-            }
-            else {
-                logger.debug(`No diff found for file: ${violation.file}`);
-            }
-        }
-        catch (error) {
-            logger.debug(`Error extracting line info: ${error.message}`);
-        }
-        return null;
-    }
-    /**
-     * Detect programming language from file extension
-     */
-    detectLanguageFromFile(filePath) {
-        const ext = filePath.split(".").pop()?.toLowerCase();
-        const languageMap = {
-            js: "javascript",
-            jsx: "javascript",
-            ts: "typescript",
-            tsx: "typescript",
-            py: "python",
-            java: "java",
-            cpp: "cpp",
-            c: "c",
-            cs: "csharp",
-            php: "php",
-            rb: "ruby",
-            go: "go",
-            rs: "rust",
-            res: "rescript",
-            kt: "kotlin",
-            swift: "swift",
-            scala: "scala",
-            sh: "bash",
-            sql: "sql",
-            json: "json",
-            yaml: "yaml",
-            yml: "yaml",
-            xml: "xml",
-            html: "html",
-            css: "css",
-            scss: "scss",
-            sass: "sass",
-            md: "markdown",
-        };
-        return languageMap[ext || ""] || "text";
-    }
-    /**
-     * Generate all possible path variations for a file
-     */
-    generatePathVariations(filePath) {
-        const variations = new Set();
-        // Add original path
-        variations.add(filePath);
-        // Add with a/ and b/ prefixes
-        variations.add(`a/${filePath}`);
-        variations.add(`b/${filePath}`);
-        // Handle nested paths
-        if (filePath.includes("/")) {
-            const parts = filePath.split("/");
-            // Try removing first directory
-            if (parts.length > 1) {
-                variations.add(parts.slice(1).join("/"));
-            }
-            // Try removing first two directories
-            if (parts.length > 2) {
-                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/")) {
-            const withoutApp = filePath.substring(4);
-            variations.add(withoutApp);
-            variations.add(`a/${withoutApp}`);
-            variations.add(`b/${withoutApp}`);
-        }
-        // Add app/ prefix variations
-        if (!filePath.startsWith("app/")) {
-            variations.add(`app/${filePath}`);
-            variations.add(`a/app/${filePath}`);
-            variations.add(`b/app/${filePath}`);
-        }
-        return Array.from(variations);
-    }
-}
-export function createCodeReviewer(bitbucketProvider, aiConfig, reviewConfig) {
-    return new CodeReviewer(bitbucketProvider, aiConfig, reviewConfig);
-}
-//# sourceMappingURL=CodeReviewer.js.map