@doist/doistbot-cli 1.0.4 → 1.0.5

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;
-}

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

@@ -1,67 +0,0 @@
-import { isKnownAutomationLogin } from '../../core/bot-logins.js';
-import { logger } from '../../core/logging.js';
-import { formatPrAuthorForGreeting, REVIEW_AUTHOR_EMAIL_PATTERN } from './author-format.js';
-const URL_PATTERN = /^https?:\/\//i;
-const UNSAFE_GREETING_CHARS = new Set([
-    '@',
-    '<',
-    '>',
-    '{',
-    '}',
-    '[',
-    ']',
-    '(',
-    ')',
-    '`',
-    '*',
-    '_',
-    '#',
-    '|',
-    '\\',
-]);
-const MAX_GREETING_NAME_CHARS = 40;
-function hasUnsafeGreetingChar(value) {
-    return Array.from(value).some((char) => UNSAFE_GREETING_CHARS.has(char));
-}
-export function shouldExcludePrAuthorFromReviewSummary(login) {
-    const normalizedLogin = login?.trim().toLowerCase();
-    return isKnownAutomationLogin(normalizedLogin);
-}
-export function extractFirstNameFromProfileName(profileName) {
-    const normalized = profileName?.replace(/\s+/g, ' ').trim() ?? '';
-    if (!normalized) {
-        return null;
-    }
-    const firstToken = normalized.split(' ')[0]?.replace(/[.,;:!]+$/g, '') ?? '';
-    if (!firstToken ||
-        firstToken.length > MAX_GREETING_NAME_CHARS ||
-        REVIEW_AUTHOR_EMAIL_PATTERN.test(firstToken) ||
-        URL_PATTERN.test(firstToken) ||
-        hasUnsafeGreetingChar(firstToken) ||
-        !/\p{L}/u.test(firstToken)) {
-        return null;
-    }
-    return firstToken;
-}
-export async function resolvePrAuthorGreetingName({ octokit, login, }) {
-    const normalizedLogin = formatPrAuthorForGreeting(login);
-    if (!normalizedLogin || shouldExcludePrAuthorFromReviewSummary(normalizedLogin)) {
-        return undefined;
-    }
-    try {
-        const response = await octokit.rest.users.getByUsername({ username: normalizedLogin });
-        const greetingName = extractFirstNameFromProfileName(response.data.name);
-        if (!greetingName) {
-            logger.info('PR author GitHub profile name unavailable for greeting', {
-                profile_name_available: Boolean(response.data.name?.trim()),
-            });
-        }
-        return greetingName ?? undefined;
-    }
-    catch (error) {
-        logger.warn('Failed to resolve PR author GitHub profile name', {
-            error: error instanceof Error ? error.message : String(error),
-        });
-        return undefined;
-    }
-}

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

@@ -1,104 +0,0 @@
-/**
- * Normalizer module for deduplicating findings across multiple models.
- * Groups similar findings together based on location and content similarity.
- */
-import { randomUUID } from 'crypto';
-import { logger } from '../../../core/logging.js';
-const FUZZY_LINE_TOLERANCE = 3; // Consider findings within +-3 lines as potentially related
-/**
- * Creates a location key for exact matching.
- */
-function getLocationKey(finding) {
-    return `${finding.path}:${finding.line}:${finding.side}`;
-}
-/**
- * Creates location keys for fuzzy matching (within tolerance).
- */
-function getFuzzyLocationKeys(finding) {
-    const keys = [];
-    for (let offset = -FUZZY_LINE_TOLERANCE; offset <= FUZZY_LINE_TOLERANCE; offset++) {
-        const fuzzyLine = finding.line + offset;
-        if (fuzzyLine > 0) {
-            keys.push(`${finding.path}:${fuzzyLine}:${finding.side}`);
-        }
-    }
-    return keys;
-}
-/**
- * Normalizes and deduplicates findings from multiple model results.
- * Findings are grouped by:
- * 1. Exact location match (path:line:side)
- * 2. Fuzzy location match (same path, within +-3 lines, same side)
- */
-export function normalizeFindings(allFindings) {
-    if (allFindings.length === 0) {
-        return [];
-    }
-    const groups = [];
-    const findingToGroup = new Map(); // finding.id -> group
-    const locationToGroup = new Map(); // location -> group
-    for (const finding of allFindings) {
-        const exactKey = getLocationKey(finding);
-        // Check for exact match first
-        const existingGroup = locationToGroup.get(exactKey);
-        if (existingGroup) {
-            existingGroup.relatedFindings.push(finding);
-            if (!existingGroup.sourceModels.includes(finding.sourceModel)) {
-                existingGroup.sourceModels.push(finding.sourceModel);
-            }
-            findingToGroup.set(finding.id, existingGroup);
-            continue;
-        }
-        // Check for fuzzy match
-        const fuzzyKeys = getFuzzyLocationKeys(finding);
-        let fuzzyMatchGroup;
-        for (const fuzzyKey of fuzzyKeys) {
-            const group = locationToGroup.get(fuzzyKey);
-            if (group) {
-                // Only fuzzy match if it's from a different model
-                if (group.primaryFinding.sourceModel !== finding.sourceModel) {
-                    fuzzyMatchGroup = group;
-                    break;
-                }
-            }
-        }
-        if (fuzzyMatchGroup) {
-            fuzzyMatchGroup.relatedFindings.push(finding);
-            if (!fuzzyMatchGroup.sourceModels.includes(finding.sourceModel)) {
-                fuzzyMatchGroup.sourceModels.push(finding.sourceModel);
-            }
-            findingToGroup.set(finding.id, fuzzyMatchGroup);
-            // Also register this finding's exact location to the group
-            locationToGroup.set(exactKey, fuzzyMatchGroup);
-            continue;
-        }
-        // No match found, create a new group
-        const newGroup = {
-            id: randomUUID(),
-            primaryFinding: finding,
-            relatedFindings: [],
-            sourceModels: [finding.sourceModel],
-        };
-        groups.push(newGroup);
-        findingToGroup.set(finding.id, newGroup);
-        locationToGroup.set(exactKey, newGroup);
-    }
-    logger.info('Normalized council findings', {
-        originalCount: allFindings.length,
-        deduplicatedCount: groups.length,
-        groups,
-    });
-    return groups;
-}
-/**
- * Gets all findings from a group (primary + related).
- */
-export function getAllFindingsFromGroup(group) {
-    return [group.primaryFinding, ...group.relatedFindings];
-}
-/**
- * Gets the number of unique models that found something in this group.
- */
-export function getModelCountForGroup(group) {
-    return group.sourceModels.length;
-}