@monoes/monograph 1.5.6 → 1.5.8

package/src/groups/service-boundary.ts

@@ -1,119 +0,0 @@
-import { readdirSync, statSync, existsSync } from 'fs';
-import { join, basename, relative } from 'path';
-
-export interface ServiceBoundary {
-  /** Absolute path to the service root directory */
-  servicePath: string;
-  /** Derived service name (directory name) */
-  serviceName: string;
-  /** List of marker filenames found in this service directory */
-  markers: string[];
-  /** Confidence score 0–1 based on number/weight of markers found */
-  confidence: number;
-}
-
-/** Weighted service markers. Higher weight = stronger signal. */
-const SERVICE_MARKERS: Record<string, number> = {
-  'package.json': 1.0,
-  'go.mod': 1.0,
-  'Cargo.toml': 1.0,
-  'pyproject.toml': 0.9,
-  'setup.py': 0.8,
-  'pom.xml': 1.0,
-  'build.gradle': 0.9,
-  'Dockerfile': 0.8,
-  'docker-compose.yml': 0.7,
-  'docker-compose.yaml': 0.7,
-  '.gitmodules': 0.5,
-};
-
-/**
- * Walk a root directory and identify service boundaries by looking for
- * well-known manifest/configuration files in each immediate subdirectory.
- *
- * Only examines one level deep (immediate children of rootDir) to avoid
- * false positives from nested node_modules or dist directories.
- *
- * @param rootDir - Root directory to scan (e.g., monorepo root)
- * @returns Array of detected service boundaries
- */
-export function detectServiceBoundaries(rootDir: string): ServiceBoundary[] {
-  if (!existsSync(rootDir)) return [];
-
-  const boundaries: ServiceBoundary[] = [];
-  let entries: string[];
-
-  try {
-    entries = readdirSync(rootDir);
-  } catch {
-    return [];
-  }
-
-  for (const entry of entries) {
-    const entryPath = join(rootDir, entry);
-    let stat: ReturnType<typeof statSync>;
-    try {
-      stat = statSync(entryPath);
-    } catch {
-      continue;
-    }
-    if (!stat.isDirectory()) continue;
-    // Skip hidden dirs and common non-service dirs
-    if (entry.startsWith('.') || entry === 'node_modules' || entry === 'dist' || entry === 'build') {
-      continue;
-    }
-
-    const foundMarkers: string[] = [];
-    let totalWeight = 0;
-
-    for (const [marker, weight] of Object.entries(SERVICE_MARKERS)) {
-      if (existsSync(join(entryPath, marker))) {
-        foundMarkers.push(marker);
-        totalWeight += weight;
-      }
-    }
-
-    if (foundMarkers.length === 0) continue;
-
-    // Confidence: cap at 1.0, based on total marker weight
-    const confidence = Math.min(1.0, totalWeight);
-
-    boundaries.push({
-      servicePath: entryPath,
-      serviceName: entry,
-      markers: foundMarkers,
-      confidence,
-    });
-  }
-
-  return boundaries.sort((a, b) => b.confidence - a.confidence);
-}
-
-/**
- * Determine which service boundary a given file belongs to.
- *
- * @param filePath - Absolute path to the file
- * @param boundaries - List of detected service boundaries
- * @returns The matching ServiceBoundary, or undefined if none match
- */
-export function assignService(
-  filePath: string,
-  boundaries: ServiceBoundary[],
-): ServiceBoundary | undefined {
-  // Find the boundary with the longest matching prefix (most specific)
-  let best: ServiceBoundary | undefined;
-  let bestLen = 0;
-
-  for (const boundary of boundaries) {
-    const rel = relative(boundary.servicePath, filePath);
-    // relative() returns a path that starts with '..' if filePath is outside boundary
-    if (!rel.startsWith('..') && rel !== '' && rel !== '.') {
-      if (boundary.servicePath.length > bestLen) {
-        bestLen = boundary.servicePath.length;
-        best = boundary;
-      }
-    }
-  }
-
-  return best;
-}

package/src/groups/topic-extractor.ts

@@ -1,90 +0,0 @@
-export type TopicRole = 'producer' | 'consumer';
-export type BrokerType = 'kafka' | 'rabbitmq' | 'sqs' | 'pubsub' | 'unknown';
-
-export interface TopicContract {
-  topicName: string;
-  role: TopicRole;
-  broker: BrokerType;
-  filePath: string;
-}
-
-// Kafka patterns
-const KAFKA_PRODUCE_RE = /(?:producer\.send|sendMessage)\s*\(\s*\{[^}]*?topic\s*:\s*['"`]([^'"`]+)['"`]/g;
-const KAFKA_CONSUME_RE = /consumer\.subscribe\s*\(\s*\{[^}]*?topic\s*:\s*['"`]([^'"`]+)['"`]/g;
-
-// RabbitMQ patterns — publish/assertExchange = producer, bindQueue/consume = consumer
-const RABBIT_PRODUCE_RE = /channel\.(?:assertExchange|publish)\s*\(\s*['"`]([^'"`]+)['"`]/g;
-const RABBIT_CONSUME_RE = /channel\.(?:bindQueue|consume)\s*\(\s*['"`]([^'"`]+)['"`]/g;
-
-// AWS SQS
-const SQS_SEND_RE = /(?:sendMessage|SendMessage)\s*\(\s*\{[^}]*?QueueUrl\s*:\s*['"`][^'"`]*\/([^'"`/]+)['"`]/g;
-
-// Generic topic string patterns (fallback)
-const GENERIC_TOPIC_RE = /(?:topic|TOPIC|queue|QUEUE)\s*[:=]\s*['"`]([a-z][a-z0-9._-]{2,})['"`]/g;
-
-function detectBroker(source: string): BrokerType {
-  if (/kafka/i.test(source) || /kafkajs/i.test(source)) return 'kafka';
-  if (/amqplib|rabbitmq|channel\./i.test(source)) return 'rabbitmq';
-  if (/SQSClient|aws-sdk.*SQS/i.test(source)) return 'sqs';
-  if (/@google-cloud\/pubsub/i.test(source)) return 'pubsub';
-  return 'unknown';
-}
-
-export function extractTopicContracts(source: string, filePath: string): TopicContract[] {
-  const results: TopicContract[] = [];
-  const broker = detectBroker(source);
-
-  if (broker === 'kafka' || broker === 'unknown') {
-    KAFKA_PRODUCE_RE.lastIndex = 0;
-    let m: RegExpExecArray | null;
-    while ((m = KAFKA_PRODUCE_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'producer', broker: 'kafka', filePath });
-    }
-    KAFKA_CONSUME_RE.lastIndex = 0;
-    while ((m = KAFKA_CONSUME_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'consumer', broker: 'kafka', filePath });
-    }
-  }
-
-  if (broker === 'rabbitmq' || broker === 'unknown') {
-    RABBIT_PRODUCE_RE.lastIndex = 0;
-    let m: RegExpExecArray | null;
-    while ((m = RABBIT_PRODUCE_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'producer', broker: 'rabbitmq', filePath });
-    }
-    RABBIT_CONSUME_RE.lastIndex = 0;
-    while ((m = RABBIT_CONSUME_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'consumer', broker: 'rabbitmq', filePath });
-    }
-  }
-
-  if (broker === 'sqs') {
-    SQS_SEND_RE.lastIndex = 0;
-    let m: RegExpExecArray | null;
-    while ((m = SQS_SEND_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'producer', broker: 'sqs', filePath });
-    }
-    // SQS receive — ReceiveMessage with QueueUrl
-    const SQS_RECV_RE = /(?:receiveMessage|ReceiveMessage)\s*\(\s*\{[^}]*?QueueUrl\s*:\s*['"`][^'"`]*\/([^'"`/]+)['"`]/g;
-    while ((m = SQS_RECV_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'consumer', broker: 'sqs', filePath });
-    }
-  }
-
-  if (results.length === 0 && broker === 'unknown') {
-    GENERIC_TOPIC_RE.lastIndex = 0;
-    let m: RegExpExecArray | null;
-    while ((m = GENERIC_TOPIC_RE.exec(source)) !== null) {
-      results.push({ topicName: m[1]!, role: 'producer', broker: 'unknown', filePath });
-    }
-  }
-
-  // Deduplicate by topicName+role
-  const seen = new Set<string>();
-  return results.filter(r => {
-    const key = `${r.role}:${r.topicName}`;
-    if (seen.has(key)) return false;
-    seen.add(key);
-    return true;
-  });
-}

package/src/health/analysis-counts-snapshot.ts

@@ -1,57 +0,0 @@
-export interface FileScoreOutput {
-  complexityModerate: number;
-  complexityHigh: number;
-  complexityCritical: number;
-  crapModerate: number;
-  crapHigh: number;
-  crapCritical: number;
-  functionCount: number;
-  fileLoc: number;
-}
-
-export interface AnalysisCountsSnapshot {
-  counts: Map<string, FileScoreOutput>;
-}
-
-export function buildAnalysisCountsSnapshot(
-  fileScores: Array<{ filePath: string } & FileScoreOutput>
-): AnalysisCountsSnapshot {
-  const counts = new Map<string, FileScoreOutput>();
-  for (const { filePath, ...rest } of fileScores) {
-    counts.set(filePath, rest);
-  }
-  return { counts };
-}
-
-export function countsFor(
-  snapshot: AnalysisCountsSnapshot,
-  roots: string[]
-): AnalysisCountsSnapshot {
-  const filtered = new Map<string, FileScoreOutput>();
-  for (const [key, value] of snapshot.counts) {
-    if (roots.some((root) => key.startsWith(root))) {
-      filtered.set(key, value);
-    }
-  }
-  return { counts: filtered };
-}
-
-export function serializeSnapshot(
-  snapshot: AnalysisCountsSnapshot
-): Record<string, FileScoreOutput> {
-  const result: Record<string, FileScoreOutput> = {};
-  for (const [key, value] of snapshot.counts) {
-    result[key] = value;
-  }
-  return result;
-}
-
-export function deserializeSnapshot(
-  data: Record<string, FileScoreOutput>
-): AnalysisCountsSnapshot {
-  const counts = new Map<string, FileScoreOutput>();
-  for (const [key, value] of Object.entries(data)) {
-    counts.set(key, value);
-  }
-  return { counts };
-}

package/src/health/analysis-counts.ts

@@ -1,70 +0,0 @@
-// Lightweight aggregate counts computed from analysis results,
-// fed into VitalSignsInput for percentage-based metric derivation.
-
-export interface AnalysisCounts {
-  totalExports: number;
-  deadFiles: number;
-  deadExports: number;
-  unusedDeps: number;
-  circularDeps: number;
-  totalDeps: number;
-  duplicateExports: number;
-  boundaryViolations: number;
-}
-
-export const ZERO_ANALYSIS_COUNTS: AnalysisCounts = {
-  totalExports: 0,
-  deadFiles: 0,
-  deadExports: 0,
-  unusedDeps: 0,
-  circularDeps: 0,
-  totalDeps: 0,
-  duplicateExports: 0,
-  boundaryViolations: 0,
-};
-
-export interface AnalysisResultsInput {
-  unusedExports?: unknown[];
-  deadFiles?: unknown[];
-  unusedDependencies?: unknown[];
-  circularDependencies?: unknown[];
-  allDependencies?: unknown[];
-  duplicateExports?: unknown[];
-  boundaryViolations?: unknown[];
-}
-
-/** Compute aggregate counts from analysis results (all fields optional). */
-export function computeAnalysisCounts(results: AnalysisResultsInput): AnalysisCounts {
-  return {
-    totalExports: (results.unusedExports?.length ?? 0),
-    deadFiles: results.deadFiles?.length ?? 0,
-    deadExports: results.unusedExports?.length ?? 0,
-    unusedDeps: results.unusedDependencies?.length ?? 0,
-    circularDeps: results.circularDependencies?.length ?? 0,
-    totalDeps: results.allDependencies?.length ?? 0,
-    duplicateExports: results.duplicateExports?.length ?? 0,
-    boundaryViolations: results.boundaryViolations?.length ?? 0,
-  };
-}
-
-/** Compute percentages from counts for use in vital signs. */
-export function deadCodePct(counts: AnalysisCounts): number {
-  if (counts.totalExports === 0) return 0;
-  return (counts.deadExports / counts.totalExports) * 100;
-}
-
-export function unusedDepsPct(counts: AnalysisCounts): number {
-  if (counts.totalDeps === 0) return 0;
-  return (counts.unusedDeps / counts.totalDeps) * 100;
-}
-
-export function formatAnalysisCounts(counts: AnalysisCounts): string {
-  return [
-    `Dead files:    ${counts.deadFiles}`,
-    `Dead exports:  ${counts.deadExports} / ${counts.totalExports} (${deadCodePct(counts).toFixed(1)}%)`,
-    `Unused deps:   ${counts.unusedDeps} / ${counts.totalDeps} (${unusedDepsPct(counts).toFixed(1)}%)`,
-    `Circular deps: ${counts.circularDeps}`,
-    `Boundary viol: ${counts.boundaryViolations}`,
-    `Dupe exports:  ${counts.duplicateExports}`,
-  ].join('\n');
-}

package/src/health/complexity-findings.ts

@@ -1,115 +0,0 @@
-// Rich per-function complexity finding types with CRAP score support,
-// coverage tiering, and severity classification.
-
-export type FindingSeverity = 'moderate' | 'high' | 'critical';
-
-export type ExceededThreshold =
-  | 'cyclomatic' | 'cognitive' | 'both'
-  | 'crap' | 'cyclomaticCrap' | 'cognitiveCrap' | 'all';
-
-export type CoverageModel = 'staticBinary' | 'staticEstimated' | 'istanbul';
-
-export type CoverageTier = 'none' | 'partial' | 'high';
-
-export function coverageTierFromPct(pct: number): CoverageTier {
-  if (pct >= 80) return 'high';
-  if (pct > 0) return 'partial';
-  return 'none';
-}
-
-export interface HealthFinding {
-  functionName: string;
-  filePath: string;
-  startLine: number;
-  endLine: number;
-  cyclomatic: number;
-  cognitive: number;
-  crapScore: number;
-  coveragePct: number;
-  coverageTier: CoverageTier;
-  severity: FindingSeverity;
-  exceeded: ExceededThreshold;
-  maintainabilityIndex: number;
-}
-
-export interface HealthSummary {
-  totalFunctions: number;
-  moderateCount: number;
-  highCount: number;
-  criticalCount: number;
-  averageCrapScore: number;
-  istanbulCoverageAvailable: boolean;
-  coverageModel: CoverageModel;
-  thresholdCyclomaticHigh: number;
-  thresholdCyclomaticCritical: number;
-  thresholdCognitiveHigh: number;
-  thresholdCognitiveCritical: number;
-  thresholdCrapHigh: number;
-  thresholdCrapCritical: number;
-}
-
-export interface FileHealthScore {
-  filePath: string;
-  maintainabilityIndex: number;
-  crapScore: number;
-  fanIn: number;
-  fanOut: number;
-  severity: FindingSeverity | 'ok';
-  findings: HealthFinding[];
-}
-
-export const DEFAULT_CYCLOMATIC_HIGH = 10;
-export const DEFAULT_CYCLOMATIC_CRITICAL = 20;
-export const DEFAULT_COGNITIVE_HIGH = 15;
-export const DEFAULT_COGNITIVE_CRITICAL = 30;
-export const DEFAULT_CRAP_HIGH = 30;
-export const DEFAULT_CRAP_CRITICAL = 100;
-export const COGNITIVE_EXTRACTION_THRESHOLD = 5;
-
-export function classifyFindingSeverity(
-  cyclomatic: number,
-  cognitive: number,
-  crap: number,
-  opts: {
-    cyclomaticHigh?: number;
-    cyclomaticCritical?: number;
-    cognitiveHigh?: number;
-    cognitiveCritical?: number;
-    crapHigh?: number;
-    crapCritical?: number;
-  } = {},
-): FindingSeverity {
-  const ch = opts.cyclomaticHigh ?? DEFAULT_CYCLOMATIC_HIGH;
-  const cc = opts.cyclomaticCritical ?? DEFAULT_CYCLOMATIC_CRITICAL;
-  const kh = opts.cognitiveHigh ?? DEFAULT_COGNITIVE_HIGH;
-  const kc = opts.cognitiveCritical ?? DEFAULT_COGNITIVE_CRITICAL;
-  const rh = opts.crapHigh ?? DEFAULT_CRAP_HIGH;
-  const rc = opts.crapCritical ?? DEFAULT_CRAP_CRITICAL;
-  if (cyclomatic >= cc || cognitive >= kc || crap >= rc) return 'critical';
-  if (cyclomatic >= ch || cognitive >= kh || crap >= rh) return 'high';
-  return 'moderate';
-}
-
-export function summarizeFindings(findings: HealthFinding[]): HealthSummary {
-  const counts = { moderate: 0, high: 0, critical: 0 };
-  let totalCrap = 0;
-  for (const f of findings) {
-    counts[f.severity]++;
-    totalCrap += f.crapScore;
-  }
-  return {
-    totalFunctions: findings.length,
-    moderateCount: counts.moderate,
-    highCount: counts.high,
-    criticalCount: counts.critical,
-    averageCrapScore: findings.length > 0 ? totalCrap / findings.length : 0,
-    istanbulCoverageAvailable: findings.some(f => f.coverageTier !== 'none'),
-    coverageModel: 'staticBinary',
-    thresholdCyclomaticHigh: DEFAULT_CYCLOMATIC_HIGH,
-    thresholdCyclomaticCritical: DEFAULT_CYCLOMATIC_CRITICAL,
-    thresholdCognitiveHigh: DEFAULT_COGNITIVE_HIGH,
-    thresholdCognitiveCritical: DEFAULT_COGNITIVE_CRITICAL,
-    thresholdCrapHigh: DEFAULT_CRAP_HIGH,
-    thresholdCrapCritical: DEFAULT_CRAP_CRITICAL,
-  };
-}

package/src/health/distribution-thresholds.ts

@@ -1,54 +0,0 @@
-// Adaptive percentile thresholds derived from the actual distribution
-// of file-level graph topology scores.
-
-export interface DistributionThresholds {
-  fanInP95: number;
-  fanInP75: number;
-  fanInP25: number;
-  fanOutP95: number;
-  fanOutP90: number;
-}
-
-export const THRESHOLD_FLOORS: DistributionThresholds = {
-  fanInP95: 8,
-  fanInP75: 4,
-  fanInP25: 1,
-  fanOutP95: 20,
-  fanOutP90: 12,
-};
-
-function sortedPercentile(sorted: number[], p: number): number {
-  if (sorted.length === 0) return 0;
-  const idx = Math.ceil(p * sorted.length) - 1;
-  return sorted[Math.max(0, Math.min(idx, sorted.length - 1))];
-}
-
-export interface FileTopologyScore {
-  fanIn: number;
-  fanOut: number;
-}
-
-/** Compute distribution thresholds from an array of per-file topology scores. */
-export function computeDistributionThresholds(
-  scores: FileTopologyScore[],
-): DistributionThresholds {
-  if (scores.length === 0) return { ...THRESHOLD_FLOORS };
-
-  const fanIns  = scores.map(s => s.fanIn).sort((a, b) => a - b);
-  const fanOuts = scores.map(s => s.fanOut).sort((a, b) => a - b);
-
-  return {
-    fanInP95:  Math.max(THRESHOLD_FLOORS.fanInP95,  sortedPercentile(fanIns,  0.95)),
-    fanInP75:  Math.max(THRESHOLD_FLOORS.fanInP75,  sortedPercentile(fanIns,  0.75)),
-    fanInP25:  Math.max(THRESHOLD_FLOORS.fanInP25,  sortedPercentile(fanIns,  0.25)),
-    fanOutP95: Math.max(THRESHOLD_FLOORS.fanOutP95, sortedPercentile(fanOuts, 0.95)),
-    fanOutP90: Math.max(THRESHOLD_FLOORS.fanOutP90, sortedPercentile(fanOuts, 0.90)),
-  };
-}
-
-export function formatDistributionThresholds(t: DistributionThresholds): string {
-  return [
-    `Fan-in  p25=${t.fanInP25}  p75=${t.fanInP75}  p95=${t.fanInP95}`,
-    `Fan-out p90=${t.fanOutP90}  p95=${t.fanOutP95}`,
-  ].join('\n');
-}

package/src/health/grouping.ts

@@ -1,138 +0,0 @@
-import type { VitalSigns, HealthScore } from './vital-signs-snapshot.js';
-import type { SubsetFilter } from '../analysis/workspace-filter.js';
-
-export type HealthGrouping = 'package' | 'owner' | 'directory' | 'section';
-
-export interface HealthGroup {
-  key: string;
-  vitalSigns: VitalSigns;
-  healthScore: HealthScore;
-  fileCount: number;
-}
-
-function gradeFromValue(value: number): string {
-  if (value >= 90) return 'A';
-  if (value >= 75) return 'B';
-  if (value >= 60) return 'C';
-  if (value >= 45) return 'D';
-  return 'F';
-}
-
-function averageVitalSigns(signs: VitalSigns[]): VitalSigns {
-  const count = signs.length;
-  if (count === 0) {
-    return {
-      deadCodePct: 0,
-      duplicationPct: 0,
-      complexityHighPct: 0,
-      complexityCriticalPct: 0,
-      crapHighPct: 0,
-      crapCriticalPct: 0,
-      hotspotDensity: 0,
-      busFactor: 0,
-      unusedDepsPct: 0,
-      maintainabilityIndex: 0,
-    };
-  }
-  const sum = signs.reduce(
-    (acc, v) => ({
-      deadCodePct: acc.deadCodePct + v.deadCodePct,
-      duplicationPct: acc.duplicationPct + v.duplicationPct,
-      complexityHighPct: acc.complexityHighPct + v.complexityHighPct,
-      complexityCriticalPct: acc.complexityCriticalPct + v.complexityCriticalPct,
-      crapHighPct: acc.crapHighPct + v.crapHighPct,
-      crapCriticalPct: acc.crapCriticalPct + v.crapCriticalPct,
-      hotspotDensity: acc.hotspotDensity + v.hotspotDensity,
-      busFactor: acc.busFactor + v.busFactor,
-      unusedDepsPct: acc.unusedDepsPct + v.unusedDepsPct,
-      maintainabilityIndex: acc.maintainabilityIndex + v.maintainabilityIndex,
-    }),
-    {
-      deadCodePct: 0,
-      duplicationPct: 0,
-      complexityHighPct: 0,
-      complexityCriticalPct: 0,
-      crapHighPct: 0,
-      crapCriticalPct: 0,
-      hotspotDensity: 0,
-      busFactor: 0,
-      unusedDepsPct: 0,
-      maintainabilityIndex: 0,
-    }
-  );
-  return {
-    deadCodePct: sum.deadCodePct / count,
-    duplicationPct: sum.duplicationPct / count,
-    complexityHighPct: sum.complexityHighPct / count,
-    complexityCriticalPct: sum.complexityCriticalPct / count,
-    crapHighPct: sum.crapHighPct / count,
-    crapCriticalPct: sum.crapCriticalPct / count,
-    hotspotDensity: sum.hotspotDensity / count,
-    busFactor: sum.busFactor / count,
-    unusedDepsPct: sum.unusedDepsPct / count,
-    maintainabilityIndex: sum.maintainabilityIndex / count,
-  };
-}
-
-function resolveKey(
-  filePath: string,
-  grouping: HealthGrouping,
-  codeownersMap?: Map<string, string>
-): string {
-  switch (grouping) {
-    case 'directory': {
-      const lastSlash = filePath.lastIndexOf('/');
-      return lastSlash >= 0 ? filePath.slice(0, lastSlash) : '.';
-    }
-    case 'owner':
-      return codeownersMap?.get(filePath) ?? 'UNOWNED';
-    case 'package': {
-      // Use first path segment after leading slash or relative root
-      const parts = filePath.replace(/^\//, '').split('/');
-      return parts[0] ?? filePath;
-    }
-    case 'section':
-      return codeownersMap?.get(filePath) ?? '(no section)';
-  }
-}
-
-export function groupHealthResults(
-  fileVitals: Array<{ filePath: string; vitalSigns: VitalSigns; healthScore: HealthScore }>,
-  grouping: HealthGrouping,
-  codeownersMap?: Map<string, string>
-): HealthGroup[] {
-  const buckets = new Map<
-    string,
-    { vitalSigns: VitalSigns[]; scores: number[] }
-  >();
-
-  for (const entry of fileVitals) {
-    const key = resolveKey(entry.filePath, grouping, codeownersMap);
-    let bucket = buckets.get(key);
-    if (!bucket) {
-      bucket = { vitalSigns: [], scores: [] };
-      buckets.set(key, bucket);
-    }
-    bucket.vitalSigns.push(entry.vitalSigns);
-    bucket.scores.push(entry.healthScore.value);
-  }
-
-  const groups: HealthGroup[] = [];
-  for (const [key, bucket] of buckets) {
-    const avgValue =
-      bucket.scores.length > 0
-        ? bucket.scores.reduce((a, b) => a + b, 0) / bucket.scores.length
-        : 0;
-    groups.push({
-      key,
-      vitalSigns: averageVitalSigns(bucket.vitalSigns),
-      healthScore: { value: avgValue, grade: gradeFromValue(avgValue) },
-      fileCount: bucket.vitalSigns.length,
-    });
-  }
-
-  // Sort ascending by healthScore.value (worst first)
-  groups.sort((a, b) => a.healthScore.value - b.healthScore.value);
-
-  return groups;
-}