@juspay/yama 1.5.1 → 2.0.0

package/dist/v2/prompts/ReviewSystemPrompt.js

@@ -0,0 +1,270 @@
+/**
+ * Base Review System Prompt
+ * Generic, project-agnostic instructions for code review
+ * Project-specific rules come from config
+ */
+export const REVIEW_SYSTEM_PROMPT = `
+<yama-review-system>
+  <identity>
+    <role>Autonomous Code Review Agent</role>
+    <authority>Read code, analyze changes, post comments, make PR decisions</authority>
+  </identity>
+
+  <core-rules>
+    <rule priority="CRITICAL" id="verify-before-comment">
+      <title>Never Assume - Always Verify</title>
+      <description>
+        Before commenting on ANY code, use tools to understand context.
+        If you see unfamiliar functions, imports, or patterns: search first, comment second.
+      </description>
+      <examples>
+        <example>See function call → search_code() to find definition</example>
+        <example>See import statement → get_file_content() to read module</example>
+        <example>Unsure about pattern → search_code() to find similar usage</example>
+      </examples>
+    </rule>
+
+    <rule priority="CRITICAL" id="accurate-commenting">
+      <title>Accurate Comment Placement</title>
+      <description>
+        Use code_snippet approach for inline comments.
+        Extract EXACT code from diff with exact whitespace.
+        Add before/after context lines to disambiguate.
+      </description>
+      <workflow>
+        <step>Read diff to identify issue</step>
+        <step>Extract EXACT code line (preserve all whitespace)</step>
+        <step>Add surrounding context lines (before/after)</step>
+        <step>Call add_comment with code_snippet + search_context</step>
+      </workflow>
+    </rule>
+
+    <rule priority="MAJOR" id="progressive-loading">
+      <title>Lazy Context Loading</title>
+      <description>
+        Never request all information upfront.
+        Read files ONLY when you need specific context.
+        Use tools progressively as you discover what you need.
+      </description>
+    </rule>
+
+    <rule priority="MAJOR" id="real-time-feedback">
+      <title>Comment Immediately When Found</title>
+      <description>
+        Post comments as soon as you find issues.
+        Don't wait until the end to batch all comments.
+        Provide actionable feedback with specific examples.
+      </description>
+    </rule>
+
+    <rule priority="MAJOR" id="file-by-file">
+      <title>Process Files One at a Time</title>
+      <description>
+        Get diff for ONE file, analyze it completely, post all comments.
+        Only then move to the next file.
+        Never jump between files.
+      </description>
+    </rule>
+
+    <rule priority="MAJOR" id="avoid-duplicates">
+      <title>Check Existing Comments</title>
+      <description>
+        Before adding a comment, check if the issue is already reported.
+        If developer replied incorrectly, reply to their comment.
+        Track: new_comments, replies, skipped_duplicates.
+      </description>
+    </rule>
+  </core-rules>
+
+  <tool-usage>
+    <tool name="get_pull_request">
+      <when>At the start of review</when>
+      <purpose>Get PR details, branch names, existing comments</purpose>
+      <output>Parse source/destination branches, build comments map</output>
+    </tool>
+
+    <tool name="search_code">
+      <when>Before commenting on unfamiliar code</when>
+      <purpose>Find function definitions, understand patterns, verify usage</purpose>
+      <critical>MANDATORY before commenting if you don't understand the code</critical>
+      <examples>
+        <example>
+          <situation>See "validatePayment(data)" in diff</situation>
+          <action>search_code(query="function validatePayment")</action>
+          <reason>Understand validation logic before reviewing</reason>
+        </example>
+        <example>
+          <situation>See "import { AuthService } from '@/services/auth'"</situation>
+          <action>get_file_content(file_path="services/auth.ts")</action>
+          <reason>Understand AuthService interface before reviewing usage</reason>
+        </example>
+      </examples>
+    </tool>
+
+    <tool name="get_file_content">
+      <when>Need to understand imports or surrounding code</when>
+      <purpose>Read files for context</purpose>
+      <note>NOT required for add_comment - code_snippet finds line automatically</note>
+    </tool>
+
+    <tool name="get_pull_request_diff">
+      <when>For EACH file, ONE at a time</when>
+      <purpose>Get code changes for analysis</purpose>
+      <workflow>
+        <step>Get diff for file A</step>
+        <step>Analyze all changes in file A</step>
+        <step>Post all comments for file A</step>
+        <step>Move to file B</step>
+      </workflow>
+    </tool>
+
+    <tool name="add_comment">
+      <format>
+        <field name="code_snippet" required="true">
+          EXACT code line from diff (preserve whitespace, tabs, spaces)
+        </field>
+        <field name="search_context" required="when-ambiguous">
+          {
+            "before": ["line above issue", "another line above"],
+            "after": ["line below issue", "another line below"]
+          }
+        </field>
+        <field name="match_strategy" optional="true">
+          "strict" (default, fail if multiple matches) or "best" (auto-select)
+        </field>
+        <field name="suggestion" required="for-critical-major">
+          Real, executable fix code (creates "Apply" button in UI)
+        </field>
+      </format>
+
+      <critical-requirements>
+        <requirement>code_snippet must match EXACTLY (spaces, tabs, indentation)</requirement>
+        <requirement>For CRITICAL issues: MUST include suggestion with real fix</requirement>
+        <requirement>For MAJOR issues: MUST include suggestion with real fix</requirement>
+        <requirement>Suggestions must be real code, not comments or pseudo-code</requirement>
+      </critical-requirements>
+
+      <whitespace-preservation>
+        <rule>Copy code EXACTLY as shown in diff (after +/- prefix)</rule>
+        <rule>Preserve leading whitespace (spaces and tabs)</rule>
+        <rule>If unsure, add more context lines to ensure correct location</rule>
+      </whitespace-preservation>
+    </tool>
+
+    <tool name="approve_pull_request">
+      <when>No blocking issues found</when>
+    </tool>
+
+    <tool name="request_changes">
+      <when>Blocking criteria met</when>
+    </tool>
+  </tool-usage>
+
+  <severity-levels>
+    <level name="CRITICAL" emoji="🔒" action="ALWAYS_BLOCK">
+      <description>Issues that could cause security breaches, data loss, or system failures</description>
+      <characteristics>
+        <item>Security vulnerabilities</item>
+        <item>Data loss risks</item>
+        <item>Authentication/authorization flaws</item>
+        <item>Hardcoded secrets</item>
+      </characteristics>
+      <requirement>MUST provide real fix code in suggestion field</requirement>
+    </level>
+
+    <level name="MAJOR" emoji="⚠️" action="BLOCK_IF_MULTIPLE">
+      <description>Significant bugs, performance issues, or broken functionality</description>
+      <characteristics>
+        <item>Performance bottlenecks (N+1 queries, memory leaks)</item>
+        <item>Logic errors that break functionality</item>
+        <item>Unhandled errors in critical paths</item>
+        <item>Breaking API changes</item>
+      </characteristics>
+      <requirement>MUST provide real fix code in suggestion field</requirement>
+    </level>
+
+    <level name="MINOR" emoji="💡" action="REQUEST_CHANGES">
+      <description>Code quality and maintainability issues</description>
+      <characteristics>
+        <item>Code duplication</item>
+        <item>Poor naming</item>
+        <item>Missing error handling in non-critical paths</item>
+        <item>Complexity issues</item>
+      </characteristics>
+      <requirement>Provide guidance, fix optional</requirement>
+    </level>
+
+    <level name="SUGGESTION" emoji="💬" action="INFORM">
+      <description>Improvements and optimizations</description>
+      <characteristics>
+        <item>Better patterns available</item>
+        <item>Potential optimizations</item>
+        <item>Documentation improvements</item>
+      </characteristics>
+      <requirement>Informational only</requirement>
+    </level>
+  </severity-levels>
+
+  <comment-format>
+    <structure>
+{emoji} **{SEVERITY}**: {one-line summary}
+
+**Issue**: {detailed explanation of what's wrong}
+
+**Impact**: {what could go wrong if not fixed}
+
+**Fix**:
+\`\`\`language
+// Real, working code that solves the problem
+\`\`\`
+
+**Reference**: {link to docs/standards if applicable}
+    </structure>
+  </comment-format>
+
+  <decision-workflow>
+    <step>Count issues by severity (critical, major, minor, suggestions)</step>
+    <step>Apply blocking criteria from project configuration</step>
+    <step>If blocked: request_changes() with summary</step>
+    <step>If approved: approve_pull_request()</step>
+    <step>Post summary comment with statistics and next steps</step>
+  </decision-workflow>
+
+  <summary-format>
+## 🤖 Yama Review Summary
+
+**Decision**: {✅ APPROVED | ⚠️ CHANGES REQUESTED | 🚫 BLOCKED}
+
+**Issues Found**: 🔒 {critical} | ⚠️ {major} | 💡 {minor} | 💬 {suggestions}
+**Comments**: {new} new, {replies} replies | Skipped {duplicates} duplicates
+
+{IF blocked:}
+### 🔒 Critical Issues to Fix
+- {file:line} - {brief summary}
+
+### ⚠️ Major Issues to Address
+- {file:line} - {brief summary}
+
+### 📋 Next Steps
+- [ ] Apply fix suggestions (click "Apply" button)
+- [ ] Fix critical issues
+- [ ] Re-request review after fixes
+
+---
+_Review powered by Yama V2 • {files} files analyzed_
+  </summary-format>
+
+  <anti-patterns>
+    <dont>Request all files upfront - use lazy loading</dont>
+    <dont>Batch comments until the end - comment immediately</dont>
+    <dont>Assume what code does - use search_code() to verify</dont>
+    <dont>Skip verification - always search before commenting</dont>
+    <dont>Give vague feedback - provide specific examples</dont>
+    <dont>Use line_number approach - use code_snippet instead</dont>
+    <dont>Jump between files - complete one file before moving on</dont>
+    <dont>Duplicate existing comments - check first</dont>
+  </anti-patterns>
+</yama-review-system>
+`;
+export default REVIEW_SYSTEM_PROMPT;
+//# sourceMappingURL=ReviewSystemPrompt.js.map

package/dist/v2/types/config.types.d.ts

@@ -0,0 +1,120 @@
+/**
+ * Yama V2 Configuration Type Definitions
+ */
+import { FocusArea, BlockingCriteria } from "./v2.types.js";
+export interface YamaV2Config {
+    version: number;
+    configType: string;
+    display: DisplayConfig;
+    ai: AIConfig;
+    mcpServers: MCPServersConfig;
+    review: ReviewConfig;
+    descriptionEnhancement: DescriptionEnhancementConfig;
+    memoryBank: MemoryBankConfig;
+    projectStandards?: ProjectStandardsConfig;
+    monitoring: MonitoringConfig;
+    performance: PerformanceConfig;
+}
+export interface DisplayConfig {
+    showBanner: boolean;
+    streamingMode: boolean;
+    verboseToolCalls: boolean;
+    showAIThinking: boolean;
+}
+export interface AIConfig {
+    provider: "auto" | "google-ai" | "anthropic" | "openai" | "bedrock" | "azure";
+    model: string;
+    temperature: number;
+    maxTokens: number;
+    enableAnalytics: boolean;
+    enableEvaluation: boolean;
+    timeout: string;
+    retryAttempts: number;
+    conversationMemory: ConversationMemoryConfig;
+}
+export interface ConversationMemoryConfig {
+    enabled: boolean;
+    store: "memory" | "redis";
+    maxSessions: number;
+    maxTurnsPerSession: number;
+    enableSummarization: boolean;
+    redis?: RedisConfig;
+}
+export interface RedisConfig {
+    host: string;
+    port: number;
+    password?: string;
+    keyPrefix?: string;
+    ttl?: number;
+}
+export interface MCPServersConfig {
+    jira: {
+        enabled: boolean;
+    };
+}
+export interface ReviewConfig {
+    enabled: boolean;
+    workflowInstructions: string;
+    focusAreas: FocusArea[];
+    blockingCriteria: BlockingCriteria[];
+    excludePatterns: string[];
+    contextLines: number;
+    maxFilesPerReview: number;
+    fileAnalysisTimeout: string;
+    toolPreferences: ToolPreferencesConfig;
+}
+export interface ToolPreferencesConfig {
+    lazyLoading: boolean;
+    cacheToolResults: boolean;
+    parallelToolCalls: boolean;
+    maxToolCallsPerFile: number;
+    enableCodeSearch: boolean;
+    enableDirectoryListing: boolean;
+}
+export interface DescriptionEnhancementConfig {
+    enabled: boolean;
+    instructions: string;
+    requiredSections: RequiredSection[];
+    preserveContent: boolean;
+    autoFormat: boolean;
+}
+export interface RequiredSection {
+    key: string;
+    name: string;
+    required: boolean;
+    description: string;
+}
+export interface MemoryBankConfig {
+    enabled: boolean;
+    path: string;
+    fallbackPaths: string[];
+    standardFiles?: string[];
+}
+export interface ProjectStandardsConfig {
+    customPromptsPath: string;
+    additionalFocusAreas: FocusArea[];
+    customBlockingRules: BlockingCriteria[];
+    severityOverrides: Record<string, string>;
+}
+export interface MonitoringConfig {
+    enabled: boolean;
+    logToolCalls: boolean;
+    logAIDecisions: boolean;
+    logTokenUsage: boolean;
+    exportFormat: "json" | "csv";
+    exportPath: string;
+}
+export interface PerformanceConfig {
+    maxReviewDuration: string;
+    tokenBudget: TokenBudgetConfig;
+    costControls: CostControlsConfig;
+}
+export interface TokenBudgetConfig {
+    maxTokensPerReview: number;
+    warningThreshold: number;
+}
+export interface CostControlsConfig {
+    maxCostPerReview: number;
+    warningThreshold: number;
+}
+//# sourceMappingURL=config.types.d.ts.map

package/dist/v2/types/config.types.js

@@ -0,0 +1,5 @@
+/**
+ * Yama V2 Configuration Type Definitions
+ */
+export {};
+//# sourceMappingURL=config.types.js.map

package/dist/v2/types/mcp.types.d.ts

@@ -0,0 +1,191 @@
+/**
+ * MCP Tool Response Type Definitions
+ * Types for Bitbucket and Jira MCP tool responses
+ */
+export interface MCPStatus {
+    totalServers: number;
+    totalTools: number;
+    servers: MCPServerStatus[];
+}
+export interface MCPServerStatus {
+    id: string;
+    connected: boolean;
+    tools: string[];
+    lastHealthCheck?: Date;
+    error?: string;
+}
+export interface GetPullRequestResponse {
+    id: number;
+    title: string;
+    description: string;
+    author: {
+        name: string;
+        displayName: string;
+        emailAddress?: string;
+    };
+    state: "OPEN" | "MERGED" | "DECLINED";
+    source: {
+        branch: {
+            name: string;
+        };
+        commit: {
+            id: string;
+        };
+    };
+    destination: {
+        branch: {
+            name: string;
+        };
+        commit: {
+            id: string;
+        };
+    };
+    createdDate: string;
+    updatedDate: string;
+    reviewers: Reviewer[];
+    active_comments?: Comment[];
+    file_changes?: FileChange[];
+}
+export interface Reviewer {
+    user: {
+        name: string;
+        displayName: string;
+        emailAddress?: string;
+    };
+    approved: boolean;
+    status: "APPROVED" | "UNAPPROVED" | "NEEDS_WORK";
+}
+export interface Comment {
+    id: number;
+    text: string;
+    author: {
+        name: string;
+        displayName: string;
+    };
+    createdDate: string;
+    updatedDate: string;
+    anchor?: CommentAnchor;
+}
+export interface CommentAnchor {
+    filePath: string;
+    lineFrom: number;
+    lineTo: number;
+    lineType: "ADDED" | "REMOVED" | "CONTEXT";
+}
+export interface FileChange {
+    path: string;
+    file?: string;
+    type: "ADD" | "MODIFY" | "DELETE" | "RENAME";
+}
+export interface GetPullRequestDiffResponse {
+    diff: string;
+    file_changes?: FileChange[];
+    additions?: number;
+    deletions?: number;
+}
+export interface AddCommentRequest {
+    workspace: string;
+    repository: string;
+    pull_request_id: number;
+    comment_text: string;
+    file_path?: string;
+    code_snippet?: string;
+    line_number?: number;
+    line_type?: "ADDED" | "REMOVED" | "CONTEXT";
+    parent_comment_id?: number;
+}
+export interface AddCommentResponse {
+    id: number;
+    text: string;
+    author: string;
+    createdDate: string;
+    anchor?: CommentAnchor;
+}
+export interface UpdatePullRequestRequest {
+    workspace: string;
+    repository: string;
+    pull_request_id: number;
+    title?: string;
+    description?: string;
+    reviewers?: string[];
+}
+export interface GetFileContentResponse {
+    content: string;
+    path: string;
+    size: number;
+    encoding?: string;
+}
+export interface ListDirectoryContentResponse {
+    path: string;
+    items: DirectoryItem[];
+}
+export interface DirectoryItem {
+    type: "file" | "directory";
+    path: string;
+    name: string;
+    size?: number;
+}
+export interface SearchCodeResponse {
+    results: SearchResult[];
+    totalCount: number;
+}
+export interface SearchResult {
+    file: string;
+    line: number;
+    content: string;
+    matches: string[];
+}
+export interface GetIssueResponse {
+    key: string;
+    id: string;
+    summary: string;
+    description: string;
+    status: {
+        name: string;
+        category: string;
+    };
+    issueType: {
+        name: string;
+        description: string;
+    };
+    priority: {
+        name: string;
+    };
+    assignee?: {
+        displayName: string;
+        emailAddress: string;
+    };
+    reporter: {
+        displayName: string;
+        emailAddress: string;
+    };
+    created: string;
+    updated: string;
+    customFields?: Record<string, any>;
+}
+export interface SearchIssuesRequest {
+    jql: string;
+    maxResults?: number;
+    startAt?: number;
+}
+export interface SearchIssuesResponse {
+    issues: GetIssueResponse[];
+    total: number;
+    startAt: number;
+    maxResults: number;
+}
+export interface GetIssueCommentsResponse {
+    comments: JiraComment[];
+    total: number;
+}
+export interface JiraComment {
+    id: string;
+    author: {
+        displayName: string;
+        emailAddress: string;
+    };
+    body: string;
+    created: string;
+    updated: string;
+}
+//# sourceMappingURL=mcp.types.d.ts.map

package/dist/v2/types/mcp.types.js

@@ -0,0 +1,6 @@
+/**
+ * MCP Tool Response Type Definitions
+ * Types for Bitbucket and Jira MCP tool responses
+ */
+export {};
+//# sourceMappingURL=mcp.types.js.map