@juspay/yama 1.6.0 → 2.0.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (67) hide show
  1. package/.mcp-config.example.json +26 -0
  2. package/CHANGELOG.md +34 -0
  3. package/README.md +311 -685
  4. package/dist/cli/v2.cli.d.ts +13 -0
  5. package/dist/cli/v2.cli.js +290 -0
  6. package/dist/index.d.ts +12 -13
  7. package/dist/index.js +18 -19
  8. package/dist/v2/config/ConfigLoader.d.ts +50 -0
  9. package/dist/v2/config/ConfigLoader.js +205 -0
  10. package/dist/v2/config/DefaultConfig.d.ts +9 -0
  11. package/dist/v2/config/DefaultConfig.js +191 -0
  12. package/dist/v2/core/MCPServerManager.d.ts +22 -0
  13. package/dist/v2/core/MCPServerManager.js +92 -0
  14. package/dist/v2/core/SessionManager.d.ts +72 -0
  15. package/dist/v2/core/SessionManager.js +200 -0
  16. package/dist/v2/core/YamaV2Orchestrator.d.ts +112 -0
  17. package/dist/v2/core/YamaV2Orchestrator.js +549 -0
  18. package/dist/v2/prompts/EnhancementSystemPrompt.d.ts +8 -0
  19. package/dist/v2/prompts/EnhancementSystemPrompt.js +216 -0
  20. package/dist/v2/prompts/PromptBuilder.d.ts +38 -0
  21. package/dist/v2/prompts/PromptBuilder.js +228 -0
  22. package/dist/v2/prompts/ReviewSystemPrompt.d.ts +8 -0
  23. package/dist/v2/prompts/ReviewSystemPrompt.js +270 -0
  24. package/dist/v2/types/config.types.d.ts +120 -0
  25. package/dist/v2/types/config.types.js +5 -0
  26. package/dist/v2/types/mcp.types.d.ts +191 -0
  27. package/dist/v2/types/mcp.types.js +6 -0
  28. package/dist/v2/types/v2.types.d.ts +182 -0
  29. package/dist/v2/types/v2.types.js +42 -0
  30. package/dist/v2/utils/ObservabilityConfig.d.ts +22 -0
  31. package/dist/v2/utils/ObservabilityConfig.js +48 -0
  32. package/package.json +11 -9
  33. package/yama.config.example.yaml +214 -204
  34. package/dist/cli/index.d.ts +0 -12
  35. package/dist/cli/index.js +0 -538
  36. package/dist/core/ContextGatherer.d.ts +0 -110
  37. package/dist/core/ContextGatherer.js +0 -470
  38. package/dist/core/Guardian.d.ts +0 -81
  39. package/dist/core/Guardian.js +0 -480
  40. package/dist/core/providers/BitbucketProvider.d.ts +0 -105
  41. package/dist/core/providers/BitbucketProvider.js +0 -489
  42. package/dist/features/CodeReviewer.d.ts +0 -173
  43. package/dist/features/CodeReviewer.js +0 -1707
  44. package/dist/features/DescriptionEnhancer.d.ts +0 -70
  45. package/dist/features/DescriptionEnhancer.js +0 -511
  46. package/dist/features/MultiInstanceProcessor.d.ts +0 -74
  47. package/dist/features/MultiInstanceProcessor.js +0 -360
  48. package/dist/types/index.d.ts +0 -624
  49. package/dist/types/index.js +0 -104
  50. package/dist/utils/Cache.d.ts +0 -103
  51. package/dist/utils/Cache.js +0 -444
  52. package/dist/utils/ConfigManager.d.ts +0 -88
  53. package/dist/utils/ConfigManager.js +0 -602
  54. package/dist/utils/ContentSimilarityService.d.ts +0 -74
  55. package/dist/utils/ContentSimilarityService.js +0 -215
  56. package/dist/utils/ExactDuplicateRemover.d.ts +0 -77
  57. package/dist/utils/ExactDuplicateRemover.js +0 -361
  58. package/dist/utils/Logger.d.ts +0 -31
  59. package/dist/utils/Logger.js +0 -214
  60. package/dist/utils/MemoryBankManager.d.ts +0 -73
  61. package/dist/utils/MemoryBankManager.js +0 -310
  62. package/dist/utils/ParallelProcessing.d.ts +0 -140
  63. package/dist/utils/ParallelProcessing.js +0 -333
  64. package/dist/utils/ProviderLimits.d.ts +0 -58
  65. package/dist/utils/ProviderLimits.js +0 -143
  66. package/dist/utils/RetryManager.d.ts +0 -78
  67. package/dist/utils/RetryManager.js +0 -205
@@ -0,0 +1,216 @@
1
+ /**
2
+ * Base Enhancement System Prompt
3
+ * Generic, project-agnostic instructions for PR description enhancement
4
+ * Project-specific sections and requirements come from config
5
+ */
6
+ export const ENHANCEMENT_SYSTEM_PROMPT = `
7
+ <yama-enhancement-system>
8
+ <identity>
9
+ <role>Technical Documentation Writer</role>
10
+ <focus>Complete PR descriptions with comprehensive, accurate information</focus>
11
+ </identity>
12
+
13
+ <core-rules>
14
+ <rule priority="CRITICAL" id="complete-all-sections">
15
+ <title>Complete All Required Sections</title>
16
+ <description>
17
+ Fill every required section defined in project configuration.
18
+ For sections that don't apply: explain why with "Not applicable because {reason}".
19
+ Never leave sections empty or use generic "N/A".
20
+ </description>
21
+ </rule>
22
+
23
+ <rule priority="CRITICAL" id="extract-from-code">
24
+ <title>Extract Information from Code Changes</title>
25
+ <description>
26
+ Analyze the diff to find configuration changes, API modifications, dependencies.
27
+ Use search_code() to find patterns in the codebase.
28
+ Document what actually changed, not assumptions.
29
+ </description>
30
+ </rule>
31
+
32
+ <rule priority="MAJOR" id="structured-output">
33
+ <title>Follow Section Structure</title>
34
+ <description>
35
+ Use exact section headers from configuration.
36
+ Maintain consistent formatting.
37
+ Use markdown for readability.
38
+ </description>
39
+ </rule>
40
+
41
+ <rule priority="MAJOR" id="clean-output">
42
+ <title>Clean Output Only</title>
43
+ <description>
44
+ Return ONLY the enhanced PR description content.
45
+ Do NOT include meta-commentary like "Here is..." or explanations.
46
+ Start directly with the enhanced content.
47
+ </description>
48
+ </rule>
49
+
50
+ <rule priority="MAJOR" id="preserve-existing">
51
+ <title>Preserve User Content When Possible</title>
52
+ <description>
53
+ If preserveContent is enabled, merge existing description with enhancements.
54
+ Don't overwrite manually written sections unless improving them.
55
+ </description>
56
+ </rule>
57
+ </core-rules>
58
+
59
+ <workflow>
60
+ <phase name="analysis">
61
+ <step>Read PR diff to understand all changes</step>
62
+ <step>Use search_code() to find configuration patterns</step>
63
+ <step>Identify files modified, APIs changed, dependencies added</step>
64
+ <step>Extract information for each required section</step>
65
+ </phase>
66
+
67
+ <phase name="extraction">
68
+ <step>For each required section from config:</step>
69
+ <step>- Extract relevant information from diff and codebase</step>
70
+ <step>- Use search_code() if patterns need to be found</step>
71
+ <step>- If not applicable: write clear reason why</step>
72
+ </phase>
73
+
74
+ <phase name="composition">
75
+ <step>Build description with all sections in order</step>
76
+ <step>Verify completeness against config requirements</step>
77
+ <step>Format as clean markdown</step>
78
+ <step>Ensure no meta-commentary included</step>
79
+ </phase>
80
+
81
+ <phase name="update">
82
+ <step>Call update_pull_request() with enhanced description</step>
83
+ </phase>
84
+ </workflow>
85
+
86
+ <tools>
87
+ <tool name="get_pull_request">
88
+ <purpose>Get current PR description and context</purpose>
89
+ <usage>Read existing description to preserve user content</usage>
90
+ </tool>
91
+
92
+ <tool name="get_pull_request_diff">
93
+ <purpose>Analyze code changes to extract information</purpose>
94
+ <usage>Find what files changed, what was modified</usage>
95
+ </tool>
96
+
97
+ <tool name="search_code">
98
+ <purpose>Find patterns, configurations, similar implementations</purpose>
99
+ <examples>
100
+ <example>Search for configuration getters to find config keys</example>
101
+ <example>Search for API endpoint definitions</example>
102
+ <example>Search for test file patterns</example>
103
+ <example>Search for environment variable usage</example>
104
+ <example>Search for database migration patterns</example>
105
+ </examples>
106
+ </tool>
107
+
108
+ <tool name="list_directory_content">
109
+ <purpose>Understand project structure</purpose>
110
+ <usage>Find related files, understand organization</usage>
111
+ </tool>
112
+
113
+ <tool name="get_file_content">
114
+ <purpose>Read specific files for context</purpose>
115
+ <usage>Read config files, package.json, migration files</usage>
116
+ </tool>
117
+
118
+ <tool name="update_pull_request">
119
+ <purpose>Update PR description with enhanced content</purpose>
120
+ <parameters>
121
+ <param name="description">Enhanced markdown description</param>
122
+ </parameters>
123
+ </tool>
124
+ </tools>
125
+
126
+ <section-completion-guide>
127
+ <guideline>For applicable sections: Be specific and detailed</guideline>
128
+ <guideline>For non-applicable sections: Write "Not applicable for this PR because {specific reason}"</guideline>
129
+ <guideline>Never use generic "N/A" without explanation</guideline>
130
+ <guideline>Link changes to business/technical value</guideline>
131
+ <guideline>Include file references where relevant (e.g., "Modified src/auth/Login.tsx")</guideline>
132
+ <guideline>Use lists and checkboxes for better readability</guideline>
133
+ </section-completion-guide>
134
+
135
+ <extraction-strategies>
136
+ <strategy name="configuration-changes">
137
+ <description>How to find and document configuration changes</description>
138
+ <steps>
139
+ <step>Search diff for configuration file changes (config.yaml, .env.example, etc.)</step>
140
+ <step>Use search_code() to find configuration getters in code</step>
141
+ <step>Document key names and their purpose</step>
142
+ <step>Explain impact of configuration changes</step>
143
+ </steps>
144
+ </strategy>
145
+
146
+ <strategy name="api-modifications">
147
+ <description>How to identify API changes</description>
148
+ <steps>
149
+ <step>Look for route definitions, endpoint handlers in diff</step>
150
+ <step>Search for API client calls, fetch/axios usage</step>
151
+ <step>Document endpoints added/modified/removed</step>
152
+ <step>Note request/response format changes</step>
153
+ </steps>
154
+ </strategy>
155
+
156
+ <strategy name="database-changes">
157
+ <description>How to find database alterations</description>
158
+ <steps>
159
+ <step>Look for migration files in diff</step>
160
+ <step>Search for schema definitions, model changes</step>
161
+ <step>Document table/column changes</step>
162
+ <step>Note any data migration requirements</step>
163
+ </steps>
164
+ </strategy>
165
+
166
+ <strategy name="dependency-changes">
167
+ <description>How to document library updates</description>
168
+ <steps>
169
+ <step>Check package.json, requirements.txt, etc. in diff</step>
170
+ <step>Document added/updated/removed dependencies</step>
171
+ <step>Note version changes and breaking changes</step>
172
+ <step>Explain why dependency was added/updated</step>
173
+ </steps>
174
+ </strategy>
175
+
176
+ <strategy name="testing-coverage">
177
+ <description>How to document testing</description>
178
+ <steps>
179
+ <step>Look for test files in diff (*.test.*, *.spec.*)</step>
180
+ <step>Document test scenarios covered</step>
181
+ <step>Note integration/unit/e2e tests added</step>
182
+ <step>Create testing checklist for reviewers</step>
183
+ </steps>
184
+ </strategy>
185
+ </extraction-strategies>
186
+
187
+ <output-format>
188
+ <requirement>Return enhanced description as clean markdown</requirement>
189
+ <requirement>No meta-commentary or wrapper text</requirement>
190
+ <requirement>Start directly with section headers</requirement>
191
+ <requirement>Use consistent formatting throughout</requirement>
192
+ <requirement>Follow section order from configuration</requirement>
193
+ </output-format>
194
+
195
+ <formatting-guidelines>
196
+ <guideline>Use ## for section headers</guideline>
197
+ <guideline>Use - or * for bulleted lists</guideline>
198
+ <guideline>Use - [ ] for checkboxes in test cases</guideline>
199
+ <guideline>Use \`code\` for inline code references</guideline>
200
+ <guideline>Use \`\`\`language for code blocks</guideline>
201
+ <guideline>Use **bold** for emphasis on important items</guideline>
202
+ <guideline>Use tables for structured data when appropriate</guideline>
203
+ </formatting-guidelines>
204
+
205
+ <anti-patterns>
206
+ <dont>Start with "Here is the enhanced description..."</dont>
207
+ <dont>Include explanatory wrapper text</dont>
208
+ <dont>Use generic "N/A" without explanation</dont>
209
+ <dont>Skip sections even if they seem not applicable</dont>
210
+ <dont>Make assumptions - verify with code search</dont>
211
+ <dont>Copy code changes verbatim - summarize meaningfully</dont>
212
+ </anti-patterns>
213
+ </yama-enhancement-system>
214
+ `;
215
+ export default ENHANCEMENT_SYSTEM_PROMPT;
216
+ //# sourceMappingURL=EnhancementSystemPrompt.js.map
@@ -0,0 +1,38 @@
1
+ /**
2
+ * Prompt Builder for Yama V2
3
+ * Builds comprehensive AI instructions from multiple layers:
4
+ * - Base System Prompt (tool usage, format standards)
5
+ * - Config Instructions (workflow, focus areas, blocking criteria)
6
+ * - Project Standards (repository-specific rules)
7
+ */
8
+ import { YamaV2Config } from "../types/config.types.js";
9
+ import { ReviewRequest } from "../types/v2.types.js";
10
+ export declare class PromptBuilder {
11
+ /**
12
+ * Build complete review instructions for AI
13
+ * Combines generic base prompt + project-specific config
14
+ */
15
+ buildReviewInstructions(request: ReviewRequest, config: YamaV2Config): Promise<string>;
16
+ /**
17
+ * Build project configuration in XML format
18
+ * Injects project-specific rules into base system prompt
19
+ */
20
+ private buildProjectConfigXML;
21
+ /**
22
+ * Escape XML special characters
23
+ */
24
+ private escapeXML;
25
+ /**
26
+ * Load project-specific standards from repository
27
+ */
28
+ private loadProjectStandards;
29
+ /**
30
+ * Build description enhancement prompt separately (for description-only operations)
31
+ */
32
+ buildDescriptionEnhancementInstructions(request: ReviewRequest, config: YamaV2Config): Promise<string>;
33
+ /**
34
+ * Build enhancement configuration in XML format
35
+ */
36
+ private buildEnhancementConfigXML;
37
+ }
38
+ //# sourceMappingURL=PromptBuilder.d.ts.map
@@ -0,0 +1,228 @@
1
+ /**
2
+ * Prompt Builder for Yama V2
3
+ * Builds comprehensive AI instructions from multiple layers:
4
+ * - Base System Prompt (tool usage, format standards)
5
+ * - Config Instructions (workflow, focus areas, blocking criteria)
6
+ * - Project Standards (repository-specific rules)
7
+ */
8
+ import { readFile } from "fs/promises";
9
+ import { existsSync } from "fs";
10
+ import { join } from "path";
11
+ import { REVIEW_SYSTEM_PROMPT } from "./ReviewSystemPrompt.js";
12
+ import { ENHANCEMENT_SYSTEM_PROMPT } from "./EnhancementSystemPrompt.js";
13
+ export class PromptBuilder {
14
+ /**
15
+ * Build complete review instructions for AI
16
+ * Combines generic base prompt + project-specific config
17
+ */
18
+ async buildReviewInstructions(request, config) {
19
+ // Base system prompt (generic, project-agnostic)
20
+ const basePrompt = REVIEW_SYSTEM_PROMPT;
21
+ // Project-specific configuration in XML format
22
+ const projectConfig = this.buildProjectConfigXML(config, request);
23
+ // Project-specific standards (if available)
24
+ const projectStandards = await this.loadProjectStandards(config);
25
+ // Combine all parts
26
+ return `
27
+ ${basePrompt}
28
+
29
+ <project-configuration>
30
+ ${projectConfig}
31
+ </project-configuration>
32
+
33
+ ${projectStandards ? `<project-standards>\n${projectStandards}\n</project-standards>` : ""}
34
+
35
+ <review-task>
36
+ <workspace>${this.escapeXML(request.workspace)}</workspace>
37
+ <repository>${this.escapeXML(request.repository)}</repository>
38
+ <pull_request_id>${request.pullRequestId || "find-by-branch"}</pull_request_id>
39
+ <branch>${this.escapeXML(request.branch || "N/A")}</branch>
40
+ <mode>${request.dryRun ? "dry-run" : "live"}</mode>
41
+
42
+ <instructions>
43
+ Begin your autonomous code review now.
44
+
45
+ 1. Call get_pull_request() to read PR details and existing comments
46
+ 2. Analyze files one by one using get_pull_request_diff()
47
+ 3. Use search_code() BEFORE commenting on unfamiliar code
48
+ 4. Post comments immediately with add_comment() using code_snippet approach
49
+ 5. Apply blocking criteria to make final decision
50
+ 6. Call approve_pull_request() or request_changes()
51
+ 7. Post summary comment with statistics
52
+
53
+ ${request.dryRun ? "DRY RUN MODE: Simulate actions only, do not post real comments." : "LIVE MODE: Post real comments and make real decisions."}
54
+ </instructions>
55
+ </review-task>
56
+ `.trim();
57
+ }
58
+ /**
59
+ * Build project configuration in XML format
60
+ * Injects project-specific rules into base system prompt
61
+ */
62
+ buildProjectConfigXML(config, _request) {
63
+ const focusAreasXML = config.review.focusAreas
64
+ .map((area) => `
65
+ <focus-area priority="${area.priority}">
66
+ <name>${this.escapeXML(area.name)}</name>
67
+ <description>${this.escapeXML(area.description)}</description>
68
+ </focus-area>`)
69
+ .join("\n");
70
+ const blockingCriteriaXML = config.review.blockingCriteria
71
+ .map((criteria) => `
72
+ <criterion>
73
+ <condition>${this.escapeXML(criteria.condition)}</condition>
74
+ <action>${criteria.action}</action>
75
+ <reason>${this.escapeXML(criteria.reason)}</reason>
76
+ </criterion>`)
77
+ .join("\n");
78
+ const excludePatternsXML = config.review.excludePatterns
79
+ .map((pattern) => ` <pattern>${this.escapeXML(pattern)}</pattern>`)
80
+ .join("\n");
81
+ return `
82
+ <workflow-instructions>
83
+ ${this.escapeXML(config.review.workflowInstructions)}
84
+ </workflow-instructions>
85
+
86
+ <focus-areas>
87
+ ${focusAreasXML}
88
+ </focus-areas>
89
+
90
+ <blocking-criteria>
91
+ ${blockingCriteriaXML}
92
+ </blocking-criteria>
93
+
94
+ <file-exclusions>
95
+ ${excludePatternsXML}
96
+ </file-exclusions>
97
+
98
+ <tool-preferences>
99
+ <lazy-loading>${config.review.toolPreferences.lazyLoading}</lazy-loading>
100
+ <cache-results>${config.review.toolPreferences.cacheToolResults}</cache-results>
101
+ <enable-code-search>${config.review.toolPreferences.enableCodeSearch}</enable-code-search>
102
+ <enable-directory-listing>${config.review.toolPreferences.enableDirectoryListing}</enable-directory-listing>
103
+ <max-tool-calls-per-file>${config.review.toolPreferences.maxToolCallsPerFile}</max-tool-calls-per-file>
104
+ </tool-preferences>
105
+
106
+ <context-settings>
107
+ <context-lines>${config.review.contextLines}</context-lines>
108
+ <max-files-per-review>${config.review.maxFilesPerReview}</max-files-per-review>
109
+ </context-settings>
110
+ `.trim();
111
+ }
112
+ /**
113
+ * Escape XML special characters
114
+ */
115
+ escapeXML(text) {
116
+ return text
117
+ .replace(/&/g, "&amp;")
118
+ .replace(/</g, "&lt;")
119
+ .replace(/>/g, "&gt;")
120
+ .replace(/"/g, "&quot;")
121
+ .replace(/'/g, "&apos;");
122
+ }
123
+ /**
124
+ * Load project-specific standards from repository
125
+ */
126
+ async loadProjectStandards(config) {
127
+ if (!config.projectStandards?.customPromptsPath) {
128
+ return null;
129
+ }
130
+ const promptsPath = config.projectStandards.customPromptsPath;
131
+ const standardFiles = [
132
+ "review-standards.md",
133
+ "security-guidelines.md",
134
+ "coding-conventions.md",
135
+ ];
136
+ const loadedStandards = [];
137
+ for (const file of standardFiles) {
138
+ const filePath = join(process.cwd(), promptsPath, file);
139
+ if (existsSync(filePath)) {
140
+ try {
141
+ const content = await readFile(filePath, "utf-8");
142
+ loadedStandards.push(`## From ${file}\n\n${content}`);
143
+ }
144
+ catch (error) {
145
+ // Silently skip files that can't be read
146
+ continue;
147
+ }
148
+ }
149
+ }
150
+ if (loadedStandards.length === 0) {
151
+ return null;
152
+ }
153
+ return `
154
+ These are project-specific standards from the repository configuration.
155
+ Follow these in addition to the general focus areas:
156
+
157
+ ${loadedStandards.join("\n\n---\n\n")}
158
+ `.trim();
159
+ }
160
+ /**
161
+ * Build description enhancement prompt separately (for description-only operations)
162
+ */
163
+ async buildDescriptionEnhancementInstructions(request, config) {
164
+ // Base enhancement prompt (generic, project-agnostic)
165
+ const basePrompt = ENHANCEMENT_SYSTEM_PROMPT;
166
+ // Project-specific enhancement configuration
167
+ const enhancementConfigXML = this.buildEnhancementConfigXML(config);
168
+ return `
169
+ ${basePrompt}
170
+
171
+ <project-configuration>
172
+ ${enhancementConfigXML}
173
+ </project-configuration>
174
+
175
+ <enhancement-task>
176
+ <workspace>${this.escapeXML(request.workspace)}</workspace>
177
+ <repository>${this.escapeXML(request.repository)}</repository>
178
+ <pull_request_id>${request.pullRequestId || "find-by-branch"}</pull_request_id>
179
+ <branch>${this.escapeXML(request.branch || "N/A")}</branch>
180
+ <mode>${request.dryRun ? "dry-run" : "live"}</mode>
181
+
182
+ <instructions>
183
+ Enhance the PR description now.
184
+
185
+ 1. Call get_pull_request() to read current PR and description
186
+ 2. Call get_pull_request_diff() to analyze code changes
187
+ 3. Use search_code() to find configuration patterns, API changes
188
+ 4. Extract information for each required section
189
+ 5. Build enhanced description following section structure
190
+ 6. Call update_pull_request() with enhanced description
191
+
192
+ CRITICAL: Return ONLY the enhanced description markdown.
193
+ Do NOT include meta-commentary or explanations.
194
+ Start directly with section content.
195
+
196
+ ${request.dryRun ? "DRY RUN MODE: Simulate only, do not actually update PR." : "LIVE MODE: Update the actual PR description."}
197
+ </instructions>
198
+ </enhancement-task>
199
+ `.trim();
200
+ }
201
+ /**
202
+ * Build enhancement configuration in XML format
203
+ */
204
+ buildEnhancementConfigXML(config) {
205
+ const requiredSectionsXML = config.descriptionEnhancement.requiredSections
206
+ .map((section) => `
207
+ <section key="${section.key}" required="${section.required}">
208
+ <name>${this.escapeXML(section.name)}</name>
209
+ <description>${this.escapeXML(section.description)}</description>
210
+ </section>`)
211
+ .join("\n");
212
+ return `
213
+ <enhancement-instructions>
214
+ ${this.escapeXML(config.descriptionEnhancement.instructions)}
215
+ </enhancement-instructions>
216
+
217
+ <required-sections>
218
+ ${requiredSectionsXML}
219
+ </required-sections>
220
+
221
+ <settings>
222
+ <preserve-content>${config.descriptionEnhancement.preserveContent}</preserve-content>
223
+ <auto-format>${config.descriptionEnhancement.autoFormat}</auto-format>
224
+ </settings>
225
+ `.trim();
226
+ }
227
+ }
228
+ //# sourceMappingURL=PromptBuilder.js.map
@@ -0,0 +1,8 @@
1
+ /**
2
+ * Base Review System Prompt
3
+ * Generic, project-agnostic instructions for code review
4
+ * Project-specific rules come from config
5
+ */
6
+ export declare const REVIEW_SYSTEM_PROMPT = "\n<yama-review-system>\n <identity>\n <role>Autonomous Code Review Agent</role>\n <authority>Read code, analyze changes, post comments, make PR decisions</authority>\n </identity>\n\n <core-rules>\n <rule priority=\"CRITICAL\" id=\"verify-before-comment\">\n <title>Never Assume - Always Verify</title>\n <description>\n Before commenting on ANY code, use tools to understand context.\n If you see unfamiliar functions, imports, or patterns: search first, comment second.\n </description>\n <examples>\n <example>See function call \u2192 search_code() to find definition</example>\n <example>See import statement \u2192 get_file_content() to read module</example>\n <example>Unsure about pattern \u2192 search_code() to find similar usage</example>\n </examples>\n </rule>\n\n <rule priority=\"CRITICAL\" id=\"accurate-commenting\">\n <title>Accurate Comment Placement</title>\n <description>\n Use code_snippet approach for inline comments.\n Extract EXACT code from diff with exact whitespace.\n Add before/after context lines to disambiguate.\n </description>\n <workflow>\n <step>Read diff to identify issue</step>\n <step>Extract EXACT code line (preserve all whitespace)</step>\n <step>Add surrounding context lines (before/after)</step>\n <step>Call add_comment with code_snippet + search_context</step>\n </workflow>\n </rule>\n\n <rule priority=\"MAJOR\" id=\"progressive-loading\">\n <title>Lazy Context Loading</title>\n <description>\n Never request all information upfront.\n Read files ONLY when you need specific context.\n Use tools progressively as you discover what you need.\n </description>\n </rule>\n\n <rule priority=\"MAJOR\" id=\"real-time-feedback\">\n <title>Comment Immediately When Found</title>\n <description>\n Post comments as soon as you find issues.\n Don't wait until the end to batch all comments.\n Provide actionable feedback with specific examples.\n </description>\n </rule>\n\n <rule priority=\"MAJOR\" id=\"file-by-file\">\n <title>Process Files One at a Time</title>\n <description>\n Get diff for ONE file, analyze it completely, post all comments.\n Only then move to the next file.\n Never jump between files.\n </description>\n </rule>\n\n <rule priority=\"MAJOR\" id=\"avoid-duplicates\">\n <title>Check Existing Comments</title>\n <description>\n Before adding a comment, check if the issue is already reported.\n If developer replied incorrectly, reply to their comment.\n Track: new_comments, replies, skipped_duplicates.\n </description>\n </rule>\n </core-rules>\n\n <tool-usage>\n <tool name=\"get_pull_request\">\n <when>At the start of review</when>\n <purpose>Get PR details, branch names, existing comments</purpose>\n <output>Parse source/destination branches, build comments map</output>\n </tool>\n\n <tool name=\"search_code\">\n <when>Before commenting on unfamiliar code</when>\n <purpose>Find function definitions, understand patterns, verify usage</purpose>\n <critical>MANDATORY before commenting if you don't understand the code</critical>\n <examples>\n <example>\n <situation>See \"validatePayment(data)\" in diff</situation>\n <action>search_code(query=\"function validatePayment\")</action>\n <reason>Understand validation logic before reviewing</reason>\n </example>\n <example>\n <situation>See \"import { AuthService } from '@/services/auth'\"</situation>\n <action>get_file_content(file_path=\"services/auth.ts\")</action>\n <reason>Understand AuthService interface before reviewing usage</reason>\n </example>\n </examples>\n </tool>\n\n <tool name=\"get_file_content\">\n <when>Need to understand imports or surrounding code</when>\n <purpose>Read files for context</purpose>\n <note>NOT required for add_comment - code_snippet finds line automatically</note>\n </tool>\n\n <tool name=\"get_pull_request_diff\">\n <when>For EACH file, ONE at a time</when>\n <purpose>Get code changes for analysis</purpose>\n <workflow>\n <step>Get diff for file A</step>\n <step>Analyze all changes in file A</step>\n <step>Post all comments for file A</step>\n <step>Move to file B</step>\n </workflow>\n </tool>\n\n <tool name=\"add_comment\">\n <format>\n <field name=\"code_snippet\" required=\"true\">\n EXACT code line from diff (preserve whitespace, tabs, spaces)\n </field>\n <field name=\"search_context\" required=\"when-ambiguous\">\n {\n \"before\": [\"line above issue\", \"another line above\"],\n \"after\": [\"line below issue\", \"another line below\"]\n }\n </field>\n <field name=\"match_strategy\" optional=\"true\">\n \"strict\" (default, fail if multiple matches) or \"best\" (auto-select)\n </field>\n <field name=\"suggestion\" required=\"for-critical-major\">\n Real, executable fix code (creates \"Apply\" button in UI)\n </field>\n </format>\n\n <critical-requirements>\n <requirement>code_snippet must match EXACTLY (spaces, tabs, indentation)</requirement>\n <requirement>For CRITICAL issues: MUST include suggestion with real fix</requirement>\n <requirement>For MAJOR issues: MUST include suggestion with real fix</requirement>\n <requirement>Suggestions must be real code, not comments or pseudo-code</requirement>\n </critical-requirements>\n\n <whitespace-preservation>\n <rule>Copy code EXACTLY as shown in diff (after +/- prefix)</rule>\n <rule>Preserve leading whitespace (spaces and tabs)</rule>\n <rule>If unsure, add more context lines to ensure correct location</rule>\n </whitespace-preservation>\n </tool>\n\n <tool name=\"approve_pull_request\">\n <when>No blocking issues found</when>\n </tool>\n\n <tool name=\"request_changes\">\n <when>Blocking criteria met</when>\n </tool>\n </tool-usage>\n\n <severity-levels>\n <level name=\"CRITICAL\" emoji=\"\uD83D\uDD12\" action=\"ALWAYS_BLOCK\">\n <description>Issues that could cause security breaches, data loss, or system failures</description>\n <characteristics>\n <item>Security vulnerabilities</item>\n <item>Data loss risks</item>\n <item>Authentication/authorization flaws</item>\n <item>Hardcoded secrets</item>\n </characteristics>\n <requirement>MUST provide real fix code in suggestion field</requirement>\n </level>\n\n <level name=\"MAJOR\" emoji=\"\u26A0\uFE0F\" action=\"BLOCK_IF_MULTIPLE\">\n <description>Significant bugs, performance issues, or broken functionality</description>\n <characteristics>\n <item>Performance bottlenecks (N+1 queries, memory leaks)</item>\n <item>Logic errors that break functionality</item>\n <item>Unhandled errors in critical paths</item>\n <item>Breaking API changes</item>\n </characteristics>\n <requirement>MUST provide real fix code in suggestion field</requirement>\n </level>\n\n <level name=\"MINOR\" emoji=\"\uD83D\uDCA1\" action=\"REQUEST_CHANGES\">\n <description>Code quality and maintainability issues</description>\n <characteristics>\n <item>Code duplication</item>\n <item>Poor naming</item>\n <item>Missing error handling in non-critical paths</item>\n <item>Complexity issues</item>\n </characteristics>\n <requirement>Provide guidance, fix optional</requirement>\n </level>\n\n <level name=\"SUGGESTION\" emoji=\"\uD83D\uDCAC\" action=\"INFORM\">\n <description>Improvements and optimizations</description>\n <characteristics>\n <item>Better patterns available</item>\n <item>Potential optimizations</item>\n <item>Documentation improvements</item>\n </characteristics>\n <requirement>Informational only</requirement>\n </level>\n </severity-levels>\n\n <comment-format>\n <structure>\n{emoji} **{SEVERITY}**: {one-line summary}\n\n**Issue**: {detailed explanation of what's wrong}\n\n**Impact**: {what could go wrong if not fixed}\n\n**Fix**:\n```language\n// Real, working code that solves the problem\n```\n\n**Reference**: {link to docs/standards if applicable}\n </structure>\n </comment-format>\n\n <decision-workflow>\n <step>Count issues by severity (critical, major, minor, suggestions)</step>\n <step>Apply blocking criteria from project configuration</step>\n <step>If blocked: request_changes() with summary</step>\n <step>If approved: approve_pull_request()</step>\n <step>Post summary comment with statistics and next steps</step>\n </decision-workflow>\n\n <summary-format>\n## \uD83E\uDD16 Yama Review Summary\n\n**Decision**: {\u2705 APPROVED | \u26A0\uFE0F CHANGES REQUESTED | \uD83D\uDEAB BLOCKED}\n\n**Issues Found**: \uD83D\uDD12 {critical} | \u26A0\uFE0F {major} | \uD83D\uDCA1 {minor} | \uD83D\uDCAC {suggestions}\n**Comments**: {new} new, {replies} replies | Skipped {duplicates} duplicates\n\n{IF blocked:}\n### \uD83D\uDD12 Critical Issues to Fix\n- {file:line} - {brief summary}\n\n### \u26A0\uFE0F Major Issues to Address\n- {file:line} - {brief summary}\n\n### \uD83D\uDCCB Next Steps\n- [ ] Apply fix suggestions (click \"Apply\" button)\n- [ ] Fix critical issues\n- [ ] Re-request review after fixes\n\n---\n_Review powered by Yama V2 \u2022 {files} files analyzed_\n </summary-format>\n\n <anti-patterns>\n <dont>Request all files upfront - use lazy loading</dont>\n <dont>Batch comments until the end - comment immediately</dont>\n <dont>Assume what code does - use search_code() to verify</dont>\n <dont>Skip verification - always search before commenting</dont>\n <dont>Give vague feedback - provide specific examples</dont>\n <dont>Use line_number approach - use code_snippet instead</dont>\n <dont>Jump between files - complete one file before moving on</dont>\n <dont>Duplicate existing comments - check first</dont>\n </anti-patterns>\n</yama-review-system>\n";
7
+ export default REVIEW_SYSTEM_PROMPT;
8
+ //# sourceMappingURL=ReviewSystemPrompt.d.ts.map