@contextrail/code-review-agent 0.1.2-alpha.1 → 0.1.2-alpha.2

package/dist/orchestrator/agentic-orchestrator.d.ts

@@ -2,6 +2,7 @@ import { z } from 'zod';
 import type { ReviewInputs } from '../review-inputs/index.js';
 import type { McpClient } from '../mcp/client.js';
 import type { Logger } from '../logging/logger.js';
+import type { LlmService } from '../llm/service.js';
 declare const orchestratorOutputSchema: z.ZodObject<{
     understanding: z.ZodString;
     reviewers: z.ZodArray<z.ZodString, "many">;
@@ -22,6 +23,7 @@ export type AgenticOrchestratorConfig = {
 };
 export type AgenticOrchestratorDeps = {
     mcpClient: McpClient;
+    llmService: LlmService;
     config: AgenticOrchestratorConfig;
     logger?: Logger;
 };

package/dist/orchestrator/agentic-orchestrator.js

@@ -1,11 +1,10 @@
 import { z } from 'zod';
 import { DEFAULT_MAX_STEPS } from '../config/defaults.js';
-import { generateUserMessagePrompt } from './prompts.js';
+import { generateUserMessagePrompt } from '../prompts/index.js';
 import { writeOrchestratorOutputs } from './writer.js';
 import { filterValidReviewers, formatReviewerSelectionError } from './validation.js';
 import { extractFilePatterns } from '../reviewers/executor.js';
 import { fileMatchesPatterns } from '../review-inputs/file-patterns.js';
-import { createLlmService } from '../llm/factory.js';
 const extractSystemPrompt = (text) => {
     const start = text.indexOf('<system_prompt>');
     const end = text.indexOf('</system_prompt>');
@@ -20,7 +19,7 @@ const orchestratorOutputSchema = z.object({
     reviewers: z.array(z.string()).min(1).describe('Selected reviewer names from available list'),
 });
 export const runOrchestrator = async (inputs, outputDir, deps) => {
-    const { mcpClient, config, logger } = deps;
+    const { mcpClient, llmService, config, logger } = deps;
     // Fetch review-orchestration prompt from MCP
     const promptResult = await mcpClient.getPrompt({ name: 'review-orchestration' });
     const promptMessages = [];
@@ -34,10 +33,13 @@ export const runOrchestrator = async (inputs, outputDir, deps) => {
         const role = message.role === 'user' ? 'user' : message.role === 'assistant' ? 'assistant' : 'system';
         promptMessages.push({ role, content: raw });
     }
-    // Extract reviewer catalog from prompt
-    const promptText = promptMessages.map((m) => m.content).join('\n');
-    const reviewerMatches = Array.from(promptText.matchAll(/reviewer-[a-z-]+/g)).map((match) => match[0]);
-    const availableReviewers = Array.from(new Set(reviewerMatches));
+    // Extract reviewer catalog from prompt metadata
+    const promptMeta = promptResult.metadata;
+    if (!Array.isArray(promptMeta?.availableReviewers) || promptMeta.availableReviewers.length === 0) {
+        throw new Error('[ORCH] Orchestration prompt is missing metadata.availableReviewers or it is empty. ' +
+            'Please update review-orchestration.yaml to include a metadata.availableReviewers array.');
+    }
+    const availableReviewers = promptMeta.availableReviewers;
     // Pre-fetch reviewer file patterns before orchestrator runs
     logger?.debug(`Pre-fetching file patterns for ${availableReviewers.length} reviewers...`);
     const reviewerFilePatterns = new Map();
@@ -58,12 +60,6 @@ export const runOrchestrator = async (inputs, outputDir, deps) => {
     logger?.debug(`Pre-fetched file patterns for ${reviewerFilePatterns.size} reviewers`);
     // Build user message with change summary and reviewer file patterns
     const userMessage = generateUserMessagePrompt(inputs, availableReviewers, config.prDescription, reviewerFilePatterns, config.reviewDomains);
-    // Create LLM service with MCP tools
-    const { service: llmService } = createLlmService({
-        openRouterApiKey: config.openRouterApiKey,
-        mcpClient,
-        logger,
-    });
     // Prepare messages
     const messages = [
         ...promptMessages,

package/dist/orchestrator/writer.js

@@ -1,6 +1,6 @@
 import { mkdir, writeFile } from 'node:fs/promises';
 import path from 'node:path';
-import { generateActivityContentPrompt } from './prompts.js';
+import { generateActivityContentPrompt } from '../prompts/index.js';
 export const writeOrchestratorOutputs = async (outputDir, understanding, reviewers, toolCalls, contextIds) => {
     const orchestratorDir = path.join(outputDir, 'orchestrator');
     await mkdir(orchestratorDir, { recursive: true });

package/dist/output/aggregator.d.ts

@@ -1,7 +1,8 @@
 import type { ReviewerResult, ReviewDecision, AggregationTokenUsage, SynthesisResult } from './schema.js';
+import type { LlmService } from '../llm/service.js';
 import type { Logger } from '../logging/logger.js';
 export type AggregationConfig = {
-    openRouterApiKey: string;
+    llmService: LlmService;
     model: string;
     maxSteps?: number;
     logger?: Logger;

package/dist/output/aggregator.js

@@ -1,6 +1,5 @@
 import { reviewDecisionSchema, synthesisResultSchema, cleanFinding } from './schema.js';
-import { buildReviewDecisionPrompt, buildSynthesisPrompt } from './prompts.js';
-import { createLlmService } from '../llm/factory.js';
+import { buildReviewDecisionPrompt, buildSynthesisPrompt } from '../prompts/index.js';
 const DEFAULT_AGGREGATION_STEPS = 5;
 const MIN_AGGREGATION_STEPS = 1;
 const MAX_AGGREGATION_STEPS = 20;
@@ -20,11 +19,7 @@ export const validateAggregationMaxSteps = (value) => {
  * @returns Synthesis result with deduplicated findings, contradictions, and compound risks
  */
 export const synthesizeFindings = async (reviewerResults, config) => {
-    const { service: llmService } = createLlmService({
-        openRouterApiKey: config.openRouterApiKey,
-        logger: config.logger,
-        enableMetrics: true,
-    });
+    const llmService = config.llmService;
     const maxSteps = validateAggregationMaxSteps(config.maxSteps);
     const totalInputFindings = reviewerResults.reduce((sum, rr) => sum + rr.findings.length, 0);
     config.logger?.debug(`Synthesis evidence input: reviewers=${reviewerResults.length}, totalFindings=${totalInputFindings}, byReviewer=${JSON.stringify(reviewerResults.map((rr) => ({
@@ -75,11 +70,7 @@ export const synthesizeFindings = async (reviewerResults, config) => {
     return { synthesis: cleanedSynthesis, usage };
 };
 export const generateReviewDecision = async (understanding, synthesisResult, config) => {
-    const { service: llmService } = createLlmService({
-        openRouterApiKey: config.openRouterApiKey,
-        logger: config.logger,
-        enableMetrics: true,
-    });
+    const llmService = config.llmService;
     const maxSteps = validateAggregationMaxSteps(config.maxSteps);
     const blockingFindings = synthesisResult.findings.filter((finding) => finding.severity === 'critical' || finding.severity === 'major');
     config.logger?.debug(`Decision evidence input: findings=${synthesisResult.findings.length}, blockingFindings=${blockingFindings.length}, contradictions=${synthesisResult.contradictions.length}, compoundRisks=${synthesisResult.compoundRisks.length}`);