@doist/doistbot-cli 1.0.3 → 1.0.5

package/sandbox/dist/tasks/review/review.js

@@ -1,27 +1,22 @@
 import { performance } from 'node:perf_hooks';
-import { parseCouncilModelsEnabled as parseConfiguredCouncilModelsEnabled } from 'doistbot-repo-config';
+import { parseReviewModelsEnabled as parseConfiguredReviewModelsEnabled } from 'doistbot-repo-config';
 import { recordTaskMetrics } from '../../core/datadog-metrics.js';
 import { logger } from '../../core/logging.js';
+import { createOpenRouterCredential } from '../../providers/credentials.js';
 import { cloneRepository, createGitHubClients, parseBaseContext, requiredEnv, WORKSPACE_DIR, } from '../../core/shared.js';
-import { resolvePrAuthorGreetingName, shouldExcludePrAuthorFromReviewSummary, } from './author-profile.js';
 import { getReviewCheckRunTitle, isAnotherReviewInProgress, postReviewFailureCheckRun, postReviewInProgressCheckRun, postReviewSuccessCheckRun, postStaleReviewPointerIfHeadMoved, } from './check-run.js';
 import { fetchReviewConversationContext } from './conversation-context.js';
 import {} from './diff.js';
 import { formatReviewForTerminal } from './format.js';
 import { formatGithubReviewComments } from './github-comment-format.js';
-import { resolvePromptTemplate } from './prompt.js';
-import { parseReviewEngine } from './review-engine.js';
 import { appendReviewFeedbackLink } from './review-feedback.js';
 import { runSharedReview } from './runner.js';
 const REVIEW_EVENT = 'COMMENT';
-function parseCouncilEnabled() {
-    // Council is enabled by default unless explicitly disabled
-    return process.env.COUNCIL_ENABLED?.toLowerCase() !== 'false';
-}
-function parseCouncilModelsEnabled() {
+const REQUIRED_PRODUCTION_REVIEW_MODELS = ['gemini', 'openai'];
+function parseReviewModelsEnabled() {
     // Parsed/defaulted in webhook-handler before being passed as task env.
-    const raw = process.env.COUNCIL_MODELS_ENABLED;
-    return parseConfiguredCouncilModelsEnabled(raw, { defaultModels: [] });
+    const raw = process.env.REVIEW_MODELS_ENABLED;
+    return parseConfiguredReviewModelsEnabled(raw, { defaultModels: [] });
 }
 function optionalEnv(name) {
     const value = process.env[name]?.trim();
@@ -29,33 +24,25 @@ function optionalEnv(name) {
 }
 function parseContext() {
     const base = parseBaseContext();
-    const requestedReviewEngine = parseReviewEngine(process.env.REVIEW_ENGINE);
     return {
         ...base,
         branch: requiredEnv('BRANCH'),
         baseBranch: requiredEnv('BASE_BRANCH'),
         headSha: optionalEnv('HEAD_SHA'),
-        geminiApiKey: optionalEnv('GEMINI_API_KEY'),
+        openRouterApiKey: optionalEnv('OPENROUTER_API_KEY'),
         openaiApiKey: optionalEnv('OPENAI_KEY'),
-        anthropicApiKey: optionalEnv('ANTHROPIC_KEY'),
-        requestedReviewEngine,
-        reviewEngine: requestedReviewEngine,
-        councilEnabled: parseCouncilEnabled(),
-        councilModelsEnabled: parseCouncilModelsEnabled(),
+        reviewModelsEnabled: parseReviewModelsEnabled(),
         dryRun: process.env.DRY_RUN?.toLowerCase() === 'true',
     };
 }
 function getConversationSummaryApiKeys(ctx) {
     const apiKeys = {};
-    if (ctx.councilModelsEnabled.includes('gemini') && ctx.geminiApiKey) {
-        apiKeys.gemini = ctx.geminiApiKey;
+    if (ctx.reviewModelsEnabled.includes('gemini') && ctx.openRouterApiKey) {
+        apiKeys.gemini = createOpenRouterCredential(ctx.openRouterApiKey);
     }
-    if (ctx.councilModelsEnabled.includes('openai') && ctx.openaiApiKey) {
+    if (ctx.reviewModelsEnabled.includes('openai') && ctx.openaiApiKey) {
         apiKeys.openai = ctx.openaiApiKey;
     }
-    if (ctx.councilModelsEnabled.includes('anthropic') && ctx.anthropicApiKey) {
-        apiKeys.anthropic = ctx.anthropicApiKey;
-    }
     return apiKeys;
 }
 async function main() {
@@ -69,17 +56,8 @@ async function main() {
     logger.setBaseContext(logContext);
     logger.info('Review task started', {
         dryRun: ctx.dryRun,
-        requestedReviewEngine: ctx.requestedReviewEngine,
-        reviewEngine: ctx.reviewEngine,
-        councilEnabled: ctx.councilEnabled,
-        councilModelsEnabled: ctx.councilModelsEnabled,
+        reviewModelsEnabled: ctx.reviewModelsEnabled,
     });
-    if (ctx.requestedReviewEngine !== ctx.reviewEngine) {
-        logger.warn('Requested review engine is not implemented yet, falling back', {
-            requestedReviewEngine: ctx.requestedReviewEngine,
-            effectiveReviewEngine: ctx.reviewEngine,
-        });
-    }
     const { app, readOctokit, writeOctokit } = await createGitHubClients(ctx);
     // GitHub's Check Runs API requires App-installation auth; calling it with
     // writeOctokit (the doistbot PAT) returns 403. Centralize the choice here
@@ -193,7 +171,6 @@ async function main() {
         ]);
         const pr = prResponse.data;
         const prAuthor = pr.user?.login ?? 'unknown';
-        const excludePrAuthorFromSummary = shouldExcludePrAuthorFromReviewSummary(prAuthor);
         function loadPrConversation() {
             return fetchReviewConversationContext({
                 octokit: readOctokit,
@@ -205,14 +182,14 @@ async function main() {
                 workspaceDir: WORKSPACE_DIR,
             });
         }
-        const promptTemplate = ctx.reviewEngine === 'council'
-            ? await resolvePromptTemplate({
-                octokit: readOctokit,
-                owner: ctx.owner,
-                repo: ctx.repo,
-            })
-            : undefined;
-        reviewMetricModels = [...ctx.councilModelsEnabled];
+        reviewMetricModels = [...ctx.reviewModelsEnabled];
+        const reviewApiKeys = {};
+        if (ctx.openRouterApiKey) {
+            reviewApiKeys.gemini = createOpenRouterCredential(ctx.openRouterApiKey);
+        }
+        if (ctx.openaiApiKey) {
+            reviewApiKeys.openai = ctx.openaiApiKey;
+        }
         const reviewResult = await runSharedReview({
             owner: ctx.owner,
             repo: ctx.repo,
@@ -223,21 +200,10 @@ async function main() {
             prBody: pr.body ?? '',
             loadPrConversation,
             prAuthor,
-            resolvePrAuthorGreetingName: () => resolvePrAuthorGreetingName({
-                octokit: readOctokit,
-                login: prAuthor,
-            }),
-            excludePrAuthorFromSummary,
             files: files,
-            reviewEngine: ctx.reviewEngine,
-            promptTemplate,
-            apiKeys: {
-                gemini: ctx.geminiApiKey,
-                openai: ctx.openaiApiKey,
-                anthropic: ctx.anthropicApiKey,
-            },
-            councilEnabled: ctx.councilEnabled,
-            councilModelsEnabled: ctx.councilModelsEnabled,
+            apiKeys: reviewApiKeys,
+            reviewModelsEnabled: ctx.reviewModelsEnabled,
+            requiredReviewModels: REQUIRED_PRODUCTION_REVIEW_MODELS,
             workspaceDir: WORKSPACE_DIR,
         });
         reviewMetricModels = reviewResult.metricModels;

package/sandbox/dist/tasks/review/runner.js

@@ -1,10 +1,11 @@
 import fs from 'fs';
 import { logger } from '../../core/logging.js';
-import { getReviewEngineRunner } from './engines/index.js';
+import { hasReviewProviderCredential } from '../../providers/credentials.js';
+import { runMultiFocusReviewEngine } from './engines/multi-focus.js';
 import {} from './engines/types.js';
 import { buildAnnotatedDiff, buildDiffSection } from './diff.js';
 import { buildMultiFocusSharedPrompt } from './multi-focus-prompt.js';
-import { buildPrompt, loadStandardsSection } from './prompt.js';
+import { loadStandardsSection } from './prompt.js';
 const REVIEW_GUIDANCE_FILES = ['AGENTS.md', 'CLAUDE.md', 'GEMINI.md'];
 const MASKED_PR_AUTHOR_LABEL = 'PR author';
 function getGuidanceFilePresence(workspaceDir) {
@@ -42,25 +43,15 @@ function buildEnginePrompt({ input, diffSection, truncated, standardsSection, pr
         truncated,
         standardsSection,
     };
-    if (input.reviewEngine === 'council') {
-        const promptTemplate = input.promptTemplate;
-        if (!promptTemplate) {
-            throw new Error('Council review requires a prompt template');
-        }
-        return {
-            prompt: buildPrompt({
-                promptTemplate,
-                ...sharedPromptArgs,
-            }),
-            promptTemplateLength: promptTemplate.length,
-        };
-    }
     return {
         prompt: buildMultiFocusSharedPrompt(sharedPromptArgs),
     };
 }
-function getEnabledReviewModels({ apiKeys, councilModelsEnabled, }) {
-    return councilModelsEnabled.filter((model) => apiKeys[model]);
+function getEnabledReviewModels({ apiKeys, reviewModelsEnabled, }) {
+    return reviewModelsEnabled.filter((model) => hasReviewProviderCredential(apiKeys[model]));
+}
+function getMissingRequiredReviewModels(availableModels, requiredReviewModels = []) {
+    return requiredReviewModels.filter((model) => !availableModels.includes(model));
 }
 export async function runSharedReview(input) {
     const { annotatedFiles, lineLookup } = buildAnnotatedDiff(input.files);
@@ -85,35 +76,40 @@ export async function runSharedReview(input) {
             providerFailures: [],
         };
     }
-    if (getEnabledReviewModels(input).length === 0) {
+    const configuredAvailableModels = getEnabledReviewModels(input);
+    if (configuredAvailableModels.length === 0) {
         logger.error('No enabled review models with API keys are available, skipping review', {
-            review_engine: input.reviewEngine,
-            council_models_enabled: input.councilModelsEnabled,
+            review_models_enabled: input.reviewModelsEnabled,
         });
         return {
             status: 'skipped',
             reason: 'no_models',
             availableModels: [],
-            metricModels: [...input.councilModelsEnabled],
-            mode: input.reviewEngine === 'multi_focus' ? 'multi_focus' : 'single',
+            metricModels: [...input.reviewModelsEnabled],
+            mode: 'multi_focus',
             providerFailures: [],
         };
     }
-    const unresolvedPrAuthorGreetingNamePromise = input.excludePrAuthorFromSummary === true
-        ? undefined
-        : input.prAuthorGreetingName != null
-            ? Promise.resolve(input.prAuthorGreetingName)
-            : input.resolvePrAuthorGreetingName?.();
-    const prAuthorGreetingNamePromise = unresolvedPrAuthorGreetingNamePromise?.catch((error) => {
-        logger.warn('Failed to resolve PR author greeting name for review summary', {
-            error: error instanceof Error ? error.message : String(error),
+    const missingRequiredModels = getMissingRequiredReviewModels(configuredAvailableModels, input.requiredReviewModels);
+    if (missingRequiredModels.length > 0) {
+        logger.error('Required review models with API keys are unavailable, skipping review', {
+            review_models_enabled: input.reviewModelsEnabled,
+            available_models: configuredAvailableModels,
+            missing_required_models: missingRequiredModels,
         });
-        return undefined;
-    });
+        return {
+            status: 'skipped',
+            reason: 'no_models',
+            availableModels: configuredAvailableModels,
+            metricModels: [...input.reviewModelsEnabled],
+            mode: 'multi_focus',
+            providerFailures: [],
+        };
+    }
     const prConversationPromise = input.loadPrConversation?.() ?? Promise.resolve(undefined);
     const prConversation = maskPrAuthorInConversation(await prConversationPromise, input.prAuthor);
     const standardsSection = loadStandardsSection();
-    const { prompt, promptTemplateLength } = buildEnginePrompt({
+    const { prompt } = buildEnginePrompt({
         input,
         diffSection,
         truncated,
@@ -121,9 +117,8 @@ export async function runSharedReview(input) {
         prConversation,
     });
     logger.info('Review prompt prepared', {
-        engine: input.reviewEngine,
+        engine: 'multi_focus',
         prompt_length: prompt.length,
-        prompt_template_length: promptTemplateLength,
         pr_body_length: input.prBody.trim().slice(0, 4000).length,
         pr_conversation_length: prConversation?.promptText.trim().length ?? 0,
         diff_length: diffSection.length,
@@ -134,39 +129,28 @@ export async function runSharedReview(input) {
         guidance_files: getGuidanceFilePresence(input.workspaceDir),
     });
     input.onProgress?.({ type: 'prompt_prepared', fileCount: annotatedFiles.length });
-    const runReviewEngine = getReviewEngineRunner(input.reviewEngine);
     const engineContext = {
         owner: input.owner,
         repo: input.repo,
         prNumber: input.prNumber,
         workspaceDir: input.workspaceDir,
-        geminiApiKey: input.apiKeys.gemini,
+        geminiCredential: input.apiKeys.gemini,
         openaiApiKey: input.apiKeys.openai,
-        anthropicApiKey: input.apiKeys.anthropic,
-        reviewEngine: input.reviewEngine,
-        councilEnabled: input.councilEnabled,
-        councilModelsEnabled: input.councilModelsEnabled,
+        reviewModelsEnabled: input.reviewModelsEnabled,
     };
-    const { review, availableModels, metricModels, mode, providerFailures } = await runReviewEngine({
+    const { review, availableModels, metricModels, mode, providerFailures } = await runMultiFocusReviewEngine({
         ctx: engineContext,
         prompt,
         standardsSection,
         lineLookup,
         prTitle: input.prTitle,
         prBody: input.prBody,
-        prAuthor: input.prAuthor,
-        prAuthorGreetingName: input.prAuthorGreetingName,
-        resolvePrAuthorGreetingName: prAuthorGreetingNamePromise
-            ? () => prAuthorGreetingNamePromise
-            : undefined,
-        excludePrAuthorFromSummary: input.excludePrAuthorFromSummary,
         lowPriorityFindingPlacement: input.lowPriorityFindingPlacement,
         onProgress: input.onProgress,
     });
     if (availableModels.length === 0) {
         logger.error('No enabled review models with API keys are available, skipping review', {
-            review_engine: input.reviewEngine,
-            council_models_enabled: input.councilModelsEnabled,
+            review_models_enabled: input.reviewModelsEnabled,
         });
         return {
             status: 'skipped',
@@ -200,7 +184,7 @@ export async function runSharedReview(input) {
         };
     }
     logger.info('Review completed', {
-        engine: input.reviewEngine,
+        engine: 'multi_focus',
         mode,
         comments_count: review.comments.length,
     });

package/sandbox/dist/tasks/review/summary-model.js

@@ -1,6 +1,7 @@
 import { logger } from '../../core/logging.js';
 import { invokePiPrompt } from '../../core/pi.js';
 import { WORKSPACE_DIR } from '../../core/shared.js';
+import { resolveReviewProviderCredential } from '../../providers/credentials.js';
 const REVIEW_SUMMARY_TIMEOUT_MS = 5 * 60 * 1000;
 const DIFF_SIDE = {
     LEFT: 'LEFT',
@@ -9,11 +10,19 @@ const DIFF_SIDE = {
 function getWorkspaceDir() {
     return WORKSPACE_DIR;
 }
-export async function requestReviewSummary({ apiKey, prompt, workspaceDir, }) {
+export async function requestReviewSummary({ credential, prompt, workspaceDir, }) {
     try {
+        const resolvedCredential = resolveReviewProviderCredential(credential);
+        if (!resolvedCredential) {
+            return null;
+        }
         const result = await invokePiPrompt({
             provider: 'gemini',
-            apiKey,
+            apiKey: resolvedCredential.apiKey,
+            ...(resolvedCredential.piProviderId
+                ? { piProviderId: resolvedCredential.piProviderId }
+                : {}),
+            ...(resolvedCredential.modelName ? { modelName: resolvedCredential.modelName } : {}),
             prompt,
             cwd: workspaceDir ?? getWorkspaceDir(),
             timeoutMs: REVIEW_SUMMARY_TIMEOUT_MS,

package/sandbox/dist/tasks/review/thinking-level.js

@@ -4,7 +4,5 @@ export function getReviewThinkingLevel(model) {
             return 'high';
         case 'gemini':
             return 'high';
-        case 'anthropic':
-            return 'xhigh';
     }
 }

package/sandbox/node_modules/doistbot-repo-config/dist/index.d.ts

@@ -1,9 +1,11 @@
 import type { Logger } from 'doistbot-logging';
 import type { Octokit } from 'octokit';
 export declare const CONFIG_FILE_PATH = ".github/doistbot.yml";
-export declare const DEFAULT_COUNCIL_MODELS: readonly ["gemini", "openai"];
-type ParseCouncilModelsEnabledOptions = {
+export declare const DEFAULT_REVIEW_MODELS: readonly ["gemini", "openai"];
+export declare const REQUIRED_REVIEW_MODELS: readonly ["gemini", "openai"];
+type ParseReviewModelsEnabledOptions = {
     defaultModels?: readonly string[];
+    requiredModels?: readonly string[];
 };
 export type RepoConfig = {
     review?: {
@@ -11,11 +13,6 @@ export type RepoConfig = {
         skip_drafts?: boolean;
         skip_title_pattern?: string;
         skip_branch_pattern?: string;
-        prompt_file?: string;
-        engine?: 'council' | 'multi_focus';
-        council?: {
-            enabled: boolean;
-        };
         /**
          * @deprecated Ignored as of #274. Doistbot now posts a GitHub Check
          * Run for every `/review` (always-on; no opt-in) when `HEAD_SHA` is
@@ -53,17 +50,6 @@ export declare function getLocalRepoConfig({ workspaceDir, logger, }: {
     workspaceDir: string;
     logger: Logger;
 }): RepoConfig | null;
-/**
- * Fetches a custom prompt file from the repository.
- * Returns null if the file doesn't exist.
- */
-export declare function getCustomPrompt({ octokit, owner, repo, path, ref, logger, }: FetchFileParams): Promise<string | null>;
-export declare function getLocalCustomPrompt({ workspaceDir, path: promptPath, logger, }: {
-    workspaceDir: string;
-    path: string;
-    logger: Logger;
-}): string | null;
-export declare function parseCouncilModelsEnabled(raw: string | undefined, { defaultModels }?: ParseCouncilModelsEnabledOptions): string[];
-export declare function getDefaultCouncilModels(): string[];
-export type { ReviewEngine } from './review-engine.js';
-export { DEFAULT_REVIEW_ENGINE, parseReviewEngine, resolveReviewEngine } from './review-engine.js';
+export declare function parseReviewModelsEnabled(raw: string | undefined, { defaultModels, requiredModels, }?: ParseReviewModelsEnabledOptions): string[];
+export declare function getDefaultReviewModels(): string[];
+export {};

package/sandbox/node_modules/doistbot-repo-config/dist/index.js

@@ -2,19 +2,11 @@ import fs from 'fs';
 import path from 'path';
 import { load as yamlLoad } from 'js-yaml';
 export const CONFIG_FILE_PATH = '.github/doistbot.yml';
-export const DEFAULT_COUNCIL_MODELS = ['gemini', 'openai'];
+export const DEFAULT_REVIEW_MODELS = ['gemini', 'openai'];
+export const REQUIRED_REVIEW_MODELS = DEFAULT_REVIEW_MODELS;
 function parseRepoConfigContent(content) {
     return yamlLoad(content);
 }
-function resolveWorkspacePath(workspaceDir, repoPath) {
-    const root = path.resolve(workspaceDir);
-    const resolved = path.resolve(root, repoPath);
-    const relative = path.relative(root, resolved);
-    if (relative.startsWith('..') || path.isAbsolute(relative)) {
-        return null;
-    }
-    return resolved;
-}
 async function fetchFileContent({ octokit, owner, repo, path, ref, logger, }) {
     const params = {
         owner,
@@ -99,67 +91,11 @@ export function getLocalRepoConfig({ workspaceDir, logger, }) {
         return null;
     }
 }
-/**
- * Fetches a custom prompt file from the repository.
- * Returns null if the file doesn't exist.
- */
-export async function getCustomPrompt({ octokit, owner, repo, path, ref, logger, }) {
-    let content;
-    try {
-        content = await fetchFileContent({ octokit, owner, repo, path, ref, logger });
-    }
-    catch (error) {
-        logger.warn('Error fetching custom prompt', {
-            component: 'repo-config',
-            repository: `${owner}/${repo}`,
-            path,
-            error: error instanceof Error ? error.message : String(error),
-        });
-        return null;
-    }
-    if (content !== null) {
-        logger.info('Loaded custom prompt', {
-            component: 'repo-config',
-            repository: `${owner}/${repo}`,
-            path,
-        });
-    }
-    return content;
-}
-export function getLocalCustomPrompt({ workspaceDir, path: promptPath, logger, }) {
-    const resolvedPath = resolveWorkspacePath(workspaceDir, promptPath);
-    if (resolvedPath === null) {
-        logger.warn('Custom prompt path escapes workspace', {
-            component: 'repo-config',
-            path: promptPath,
-        });
-        return null;
-    }
-    if (!fs.existsSync(resolvedPath)) {
-        return null;
-    }
-    try {
-        const content = fs.readFileSync(resolvedPath, 'utf8');
-        logger.info('Loaded local custom prompt', {
-            component: 'repo-config',
-            path: promptPath,
-        });
-        return content;
-    }
-    catch (error) {
-        logger.warn('Failed to load local custom prompt', {
-            component: 'repo-config',
-            path: promptPath,
-            error: error instanceof Error ? error.message : String(error),
-        });
-        return null;
-    }
-}
-export function parseCouncilModelsEnabled(raw, { defaultModels = DEFAULT_COUNCIL_MODELS } = {}) {
+export function parseReviewModelsEnabled(raw, { defaultModels = DEFAULT_REVIEW_MODELS, requiredModels = REQUIRED_REVIEW_MODELS, } = {}) {
     if (!raw?.trim()) {
         return [...defaultModels];
     }
-    const allowedModels = new Set(DEFAULT_COUNCIL_MODELS);
+    const allowedModels = new Set(DEFAULT_REVIEW_MODELS);
     const models = [];
     for (const token of raw.split(',')) {
         const model = token.trim().toLowerCase();
@@ -170,9 +106,9 @@ export function parseCouncilModelsEnabled(raw, { defaultModels = DEFAULT_COUNCIL
             models.push(model);
         }
     }
-    return models.length > 0 ? models : [...defaultModels];
+    const hasRequiredModels = requiredModels.every((model) => models.includes(model));
+    return models.length > 0 && hasRequiredModels ? models : [...defaultModels];
 }
-export function getDefaultCouncilModels() {
-    return [...DEFAULT_COUNCIL_MODELS];
+export function getDefaultReviewModels() {
+    return [...DEFAULT_REVIEW_MODELS];
 }
-export { DEFAULT_REVIEW_ENGINE, parseReviewEngine, resolveReviewEngine } from './review-engine.js';

package/sandbox/_node_modules/doistbot-repo-config/dist/review-engine.d.ts

@@ -1,11 +0,0 @@
-export declare const DEFAULT_REVIEW_ENGINE: "multi_focus";
-export type ReviewEngine = 'council' | 'multi_focus';
-export declare function parseReviewEngine(raw: unknown): ReviewEngine | null;
-export declare function resolveReviewEngine({ configuredEngine, envEngine, logger, repository, }: {
-    configuredEngine?: unknown;
-    envEngine?: unknown;
-    logger?: {
-        warn: (message: string, context?: Record<string, unknown>) => void;
-    };
-    repository?: string;
-}): ReviewEngine;

package/sandbox/_node_modules/doistbot-repo-config/dist/review-engine.js

@@ -1,41 +0,0 @@
-export const DEFAULT_REVIEW_ENGINE = 'multi_focus';
-function isReviewEngine(value) {
-    return value === 'council' || value === 'multi_focus';
-}
-export function parseReviewEngine(raw) {
-    if (typeof raw !== 'string') {
-        return null;
-    }
-    const normalized = raw.trim().toLowerCase();
-    if (!normalized) {
-        return null;
-    }
-    return isReviewEngine(normalized) ? normalized : null;
-}
-export function resolveReviewEngine({ configuredEngine, envEngine, logger, repository, }) {
-    const configured = parseReviewEngine(configuredEngine);
-    if (configured) {
-        return configured;
-    }
-    if (configuredEngine != null) {
-        logger?.warn('Invalid configured review engine, checking environment override next', {
-            component: 'review-engine',
-            repository,
-            configured_engine: configuredEngine,
-            next_source: 'env',
-        });
-    }
-    const fromEnv = parseReviewEngine(envEngine);
-    if (fromEnv) {
-        return fromEnv;
-    }
-    if (envEngine != null) {
-        logger?.warn('Invalid env review engine, falling back', {
-            component: 'review-engine',
-            repository,
-            env_engine: envEngine,
-            fallback: DEFAULT_REVIEW_ENGINE,
-        });
-    }
-    return DEFAULT_REVIEW_ENGINE;
-}

package/sandbox/dist/providers/anthropic.js

@@ -1,30 +0,0 @@
-/**
- * Anthropic provider for the multi-model code review system.
- * Uses the Pi harness instead of the Claude CLI.
- */
-import { DEFAULT_PI_MODELS } from '../core/pi.js';
-import { resolveProviderTimeoutMs } from './config.js';
-import { extractJsonFromResponse } from './helpers.js';
-import { runPiReviewProvider } from './pi-review.js';
-export class AnthropicProvider {
-    name = 'anthropic';
-    async review(prompt, config, logOptions) {
-        return await runPiReviewProvider({
-            provider: this.name,
-            modelName: config.modelName ?? DEFAULT_PI_MODELS.anthropic,
-            prompt,
-            config: {
-                ...config,
-                timeout: resolveProviderTimeoutMs(config.timeout),
-            },
-            prompts: config.jsonSchema
-                ? [
-                    prompt,
-                    `${prompt}\n\nReturn only JSON that matches the provided schema. Do not include analysis text.`,
-                ]
-                : [prompt],
-            parseOutput: (output) => JSON.parse(extractJsonFromResponse(output)),
-            logOptions,
-        });
-    }
-}

package/sandbox/dist/tasks/review/author-format.js

@@ -1,14 +0,0 @@
-export const REVIEW_AUTHOR_EMAIL_PATTERN = /^[^\s@]+@[^\s@]+\.[^\s@]+$/;
-export function normalizePrAuthor(prAuthor) {
-    return (prAuthor ?? '').trim().replace(/^@+/, '');
-}
-export function formatPrAuthorForGreeting(prAuthor) {
-    const normalizedAuthor = normalizePrAuthor(prAuthor);
-    if (!normalizedAuthor || normalizedAuthor.toLowerCase() === 'unknown') {
-        return null;
-    }
-    if (REVIEW_AUTHOR_EMAIL_PATTERN.test(normalizedAuthor)) {
-        return null;
-    }
-    return normalizedAuthor;
-}