@juspay/yama 1.3.0 → 1.4.0

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+# [1.4.0](https://github.com/juspay/yama/compare/v1.3.0...v1.4.0) (2025-09-18)
+
+### Features
+
+- **Multi-Instance:** Added support for Multi-Instance Processing and Deduplication ([2724758](https://github.com/juspay/yama/commit/27247587f44740b26218f23694ebdcde4c323266))
+
 # [1.3.0](https://github.com/juspay/yama/compare/v1.2.0...v1.3.0) (2025-09-01)
 
 ### Features

package/README.md

@@ -199,6 +199,34 @@ features:
       - "🧪 Testing & Error Handling"
       - "📖 Documentation & Maintainability"
 
+    # Multi-Instance Processing Configuration
+    multiInstance:
+      enabled: true
+      instanceCount: 2
+      instances:
+        - name: "primary"
+          provider: "vertex"
+          model: "gemini-2.5-pro"
+          temperature: 0.3
+        - name: "secondary"
+          provider: "vertex"
+          model: "gemini-2.5-pro"
+          temperature: 0.1
+      deduplication:
+        enabled: true
+        similarityThreshold: 40 # 0-100 percentage scale
+        maxCommentsToPost: 30
+        prioritizeBy: "severity"
+
+    # Semantic Deduplication Configuration
+    semanticDeduplication:
+      enabled: true
+      similarityThreshold: 70 # 0-100 percentage scale
+      batchSize: 15
+      timeout: "5m"
+      fallbackOnError: true
+      logMatches: true
+
   descriptionEnhancement:
     enabled: true
     preserveContent: true # NEVER remove existing content
@@ -477,7 +505,7 @@ await guardian.processPR({
 });
 ```
 
-----
+---
 
 ## 📋 **Configurable Description Sections**
 

package/dist/core/Guardian.js

@@ -281,7 +281,6 @@ export class Guardian {
             logger.info("Code review is disabled in configuration");
             return { skipped: true, reason: "disabled in config" };
         }
-        logger.phase("🔍 Executing code review...");
         const reviewOptions = {
             workspace: context.identifier.workspace,
             repository: context.identifier.repository,
@@ -291,8 +290,12 @@ export class Guardian {
             excludePatterns: this.config.features.codeReview.excludePatterns,
             contextLines: this.config.features.codeReview.contextLines,
         };
-        // Use the already gathered context instead of gathering again
-        return await this.codeReviewer.reviewCodeWithContext(context, reviewOptions);
+        logger.phase("🔍 Executing code review...");
+        // Check if multi-instance processing is configured
+        const multiInstanceConfig = this.config.features?.codeReview
+            ?.multiInstance;
+        // Use code review with multi-instance support
+        return await this.codeReviewer.reviewCodeWithContext(context, reviewOptions, multiInstanceConfig);
     }
     /**
      * Execute description enhancement using shared context

package/dist/core/providers/BitbucketProvider.d.ts

@@ -56,7 +56,7 @@ export declare class BitbucketProvider {
         message: string;
     }>;
     /**
-     * Add comment to PR with smart positioning
+     * Add comment to PR with smart positioning and validation
      */
     addComment(identifier: PRIdentifier, commentText: string, options?: {
         filePath?: string;

package/dist/core/providers/BitbucketProvider.js

@@ -286,7 +286,7 @@ export class BitbucketProvider {
         }
     }
     /**
-     * Add comment to PR with smart positioning
+     * Add comment to PR with smart positioning and validation
      */
     async addComment(identifier, commentText, options = {}) {
         await this.initialize();
@@ -351,6 +351,36 @@ export class BitbucketProvider {
         }
         catch (error) {
             logger.error(`Failed to add comment: ${error.message}`);
+            // If inline comment fails, try posting as general comment
+            if (options.filePath && options.codeSnippet) {
+                logger.debug(`Attempting fallback to general comment...`);
+                try {
+                    const fallbackArgs = {
+                        workspace,
+                        repository,
+                        pull_request_id: pullRequestId,
+                        comment_text: `**File: ${options.filePath}**\n\n${commentText}`,
+                    };
+                    const fallbackResult = await this.pullRequestHandlers.handleAddComment(fallbackArgs);
+                    let fallbackData;
+                    if (fallbackResult.content &&
+                        fallbackResult.content[0] &&
+                        fallbackResult.content[0].text) {
+                        fallbackData = JSON.parse(fallbackResult.content[0].text);
+                    }
+                    else {
+                        fallbackData = fallbackResult;
+                    }
+                    logger.debug(`Fallback comment posted successfully`);
+                    return {
+                        success: true,
+                        commentId: fallbackData.id || fallbackData.comment_id,
+                    };
+                }
+                catch (fallbackError) {
+                    logger.error(`Fallback comment also failed: ${fallbackError.message}`);
+                }
+            }
             throw new ProviderError(`Comment failed: ${error.message}`);
         }
     }

package/dist/features/CodeReviewer.d.ts

@@ -12,17 +12,13 @@ export declare class CodeReviewer {
     private reviewConfig;
     constructor(bitbucketProvider: BitbucketProvider, aiConfig: AIProviderConfig, reviewConfig: CodeReviewConfig);
     /**
-     * Review code using pre-gathered unified context (OPTIMIZED with Batch Processing)
+     * Review code using pre-gathered unified context (OPTIMIZED with Multi-Instance and Batch Processing)
      */
-    reviewCodeWithContext(context: UnifiedContext, options: ReviewOptions): Promise<ReviewResult>;
+    reviewCodeWithContext(context: UnifiedContext, options: ReviewOptions, multiInstanceConfig?: any): Promise<ReviewResult>;
     /**
-     * Validate violations to ensure code snippets exist in diff
+     * Review code using multiple instances for enhanced analysis
      */
-    private validateViolations;
-    /**
-     * Try to fix code snippet by finding it in the actual diff
-     */
-    private tryFixCodeSnippet;
+    reviewWithMultipleInstances(context: UnifiedContext, options: ReviewOptions, multiInstanceConfig: any): Promise<any>;
     /**
      * Get system prompt for security-focused code review
      */
@@ -101,9 +97,21 @@ export declare class CodeReviewer {
      */
     private shouldUseBatchProcessing;
     /**
-     * Main batch processing method
+     * Main batch processing method with parallel processing support
      */
     private reviewWithBatchProcessing;
+    /**
+     * Process batches in parallel with concurrency control
+     */
+    private processInParallel;
+    /**
+     * Process batches serially (original implementation)
+     */
+    private processSerially;
+    /**
+     * Process a single batch with concurrency control
+     */
+    private processBatchWithConcurrency;
     /**
      * Prioritize files based on security importance and file type
      */