@mrclrchtr/supi-code-intelligence 3.2.0 → 4.1.0

package/src/ui/tui/common.ts

@@ -26,11 +26,10 @@ export const EVIDENCE_KEY_LABELS: Record<string, string> = {
   "exports.symbols": "exports",
   "find.astMatches": "AST matches",
   "find.semanticSymbols": "symbols",
-  "find.textMatches": "matches",
-  "health.dirtyFiles": "dirty files",
   "implements.locations": "implementations",
   "imports.modules": "imports",
-  "inspect.codeActions": "code actions",
+  "inspect.definitions": "definitions",
+  "inspect.diagnostics": "nearby diagnostics",
   "references.locations": "references",
   "refactor.edits": "edits",
   "resolve.candidates": "candidates",
@@ -42,34 +41,89 @@ export function evidenceLabel(key: string): string {
   return EVIDENCE_KEY_LABELS[key] ?? key;
 }
 
-// ── Evidence entires ─────────────────────────────────────────────
+// ── Serialized evidence ──────────────────────────────────────────
 
+/** Evidence metadata projected by result assembly into tool details. */
 export interface EvidenceEntry {
-  key?: string;
-  shownCount?: number;
-  totalCount?: number | null;
-  omittedCount?: number | null;
-  partialReason?: string | null;
+  key: string;
+  shownCount: number;
+  totalCount: number | null;
+  omittedCount: number | null;
+  partialReason: string | null;
+  invalidLocationCount?: number;
 }
 
-// ── Evidence section ─────────────────────────────────────────────
+/**
+ * Read only complete serialized evidence metadata from tool details.
+ *
+ * Renderers intentionally discard malformed or absent entries rather than
+ * reconstructing totals from candidate counts or omission hints.
+ */
+export function readEvidenceEntries(value: unknown): EvidenceEntry[] {
+  if (!Array.isArray(value)) return [];
+  return value.flatMap((entry) => {
+    const parsed = readEvidenceEntry(entry);
+    return parsed ? [parsed] : [];
+  });
+}
+
+function readEvidenceEntry(value: unknown): EvidenceEntry | null {
+  if (typeof value !== "object" || value === null || Array.isArray(value)) return null;
+  const record = value as Record<string, unknown>;
+  const { key, shownCount, totalCount, omittedCount, partialReason, invalidLocationCount } = record;
+  if (
+    typeof key !== "string" ||
+    typeof shownCount !== "number" ||
+    !isNullableNumber(totalCount) ||
+    !isNullableNumber(omittedCount) ||
+    !isNullableString(partialReason) ||
+    !isOptionalCount(invalidLocationCount)
+  ) {
+    return null;
+  }
+  return {
+    key,
+    shownCount,
+    totalCount,
+    omittedCount,
+    partialReason,
+    ...(invalidLocationCount === undefined ? {} : { invalidLocationCount }),
+  };
+}
+
+function isNullableNumber(value: unknown): value is number | null {
+  return value === null || typeof value === "number";
+}
+
+function isNullableString(value: unknown): value is string | null {
+  return value === null || typeof value === "string";
+}
+
+function isOptionalCount(value: unknown): value is number | undefined {
+  return (
+    value === undefined || (typeof value === "number" && Number.isInteger(value) && value >= 0)
+  );
+}
+
+/** Format one assembled evidence list without recomputing its bounds. */
+export function formatEvidenceEntry(entry: EvidenceEntry): string {
+  const badge = formatEvidenceBadge({
+    shownCount: entry.shownCount,
+    totalCount: entry.totalCount,
+    omittedCount: entry.omittedCount,
+    partialReason: entry.partialReason,
+    label: evidenceLabel(entry.key),
+  });
+  if (!entry.invalidLocationCount) return badge;
+  const noun = entry.invalidLocationCount === 1 ? "location" : "locations";
+  const reason = entry.partialReason ? ` — ${entry.partialReason}` : "";
+  return `${badge}; ${entry.invalidLocationCount} invalid provider ${noun} omitted${reason}`;
+}
 
 /** Append formatted evidence badge lines to a container. */
-export function renderEvidenceLines(
-  container: Container,
-  entries: EvidenceEntry[] | undefined,
-  theme: Theme,
-): void {
-  if (!entries || entries.length === 0) return;
-  for (const ev of entries) {
-    const badge = formatEvidenceBadge({
-      shownCount: Number(ev.shownCount ?? 0),
-      totalCount: ev.totalCount != null ? Number(ev.totalCount) : null,
-      omittedCount: ev.omittedCount != null ? Number(ev.omittedCount) : null,
-      partialReason: typeof ev.partialReason === "string" ? ev.partialReason : null,
-      label: evidenceLabel(String(ev.key ?? "")),
-    });
-    container.addChild(new Text(theme.fg("muted", badge), 0, 0));
+export function renderEvidenceLines(container: Container, entries: unknown, theme: Theme): void {
+  for (const entry of readEvidenceEntries(entries)) {
+    container.addChild(new Text(theme.fg("muted", formatEvidenceEntry(entry)), 0, 0));
   }
 }
 
@@ -103,14 +157,14 @@ export function renderError(label: string, theme: Theme): Text {
   return new Text(theme.fg("error", label), 0, 0);
 }
 
-// ── Result optios ────────────────────────────────────────────────
+// ── Result options ───────────────────────────────────────────────
 
 export interface ResultOptios {
   expanded: boolean;
   isPartial: boolean;
 }
 
-// ── Shared simple-result renderer ────────────────────────────────
+// ── Shared simple-result renderer ─────────────────────────────────
 
 /**
  * Shared result renderer for the "simple" code-intelligence tools
@@ -143,49 +197,36 @@ function buildExpandedView(
   const header = buildSimpleHeader(data, theme);
   if (header) container.addChild(header);
 
-  const lists = data?.evidenceLists as EvidenceEntry[] | undefined;
-  if (lists && lists.length > 0) {
-    container.addChild(new Spacer(1));
-    renderEvidenceLines(container, lists, theme);
-  }
-
   renderStructuredDetailBody(container, data, theme);
   renderMarkdownDetail(container, result, theme);
 
   return container;
 }
 
-/** Options for {@link buildSimpleCompact} and {@link buildSimpleHeader}. */
-export interface SimpleResultOptions {
-  /** Omitted count for the evidence badge. Defaults to 0. */
-  omittedCount?: number;
+function evidenceBadges(data: Record<string, unknown> | undefined): string[] {
+  return readEvidenceEntries(data?.evidenceLists).map(formatEvidenceEntry);
 }
 
-export function buildSimpleCompact(
-  data: Record<string, unknown> | undefined,
-  theme: Theme,
-  opts?: SimpleResultOptions,
-): Text {
+export function buildSimpleCompact(data: Record<string, unknown> | undefined, theme: Theme): Text {
   if (!data) return new Text(theme.fg("dim", "No results"), 0, 0);
 
-  const candidateCount = (data.candidateCount as number) ?? (data.targetCount as number) ?? 0;
-  const omittedCount = opts?.omittedCount ?? 0;
-  const confidence = (data.confidence as string) ?? "";
-
-  const badge = formatEvidenceBadge({
-    shownCount: candidateCount,
-    totalCount: candidateCount + omittedCount,
-    omittedCount,
-    partialReason: null,
-    label: "results",
-  });
-
+  const badges = evidenceBadges(data);
+  const confidence = typeof data.confidence === "string" ? data.confidence : "";
   const dot = theme.fg("dim", "·");
-  const segments = [theme.fg("success", theme.bold(badge))];
+  const segments = badges.map((badge) => theme.fg("success", theme.bold(badge)));
+  const unknownNestingCount = data.groupUnknownNestingCount;
+  if (
+    typeof unknownNestingCount === "number" &&
+    Number.isInteger(unknownNestingCount) &&
+    unknownNestingCount > 0
+  ) {
+    segments.push(theme.fg("warning", `${unknownNestingCount} hierarchy unknown`));
+  }
   if (confidence) {
     segments.push(`${theme.fg("dim", "confidence")} ${theme.fg("muted", confidence)}`);
   }
 
+  if (segments.length === 0) return new Text(theme.fg("dim", "No assembled evidence"), 0, 0);
   return new Text(segments.join(` ${dot} `), 0, 0);
 }
 
@@ -210,9 +251,15 @@ function structuredDetailLines(data: Record<string, unknown>): string[] {
     lines.push(`target: ${focusTarget}`);
   }
 
-  const unavailable = data.unavailableSections;
-  if (Array.isArray(unavailable) && unavailable.length > 0) {
-    lines.push(`unavailable: ${unavailable.join(", ")}`);
+  lines.push(...inspectSectionStatusLines(data.sections));
+
+  const unknownNestingCount = data.groupUnknownNestingCount;
+  if (
+    typeof unknownNestingCount === "number" &&
+    Number.isInteger(unknownNestingCount) &&
+    unknownNestingCount > 0
+  ) {
+    lines.push(`hierarchy unknown: ${unknownNestingCount}`);
   }
 
   const checkNext = data.checkNext;
@@ -233,32 +280,32 @@ function structuredDetailLines(data: Record<string, unknown>): string[] {
   return lines;
 }
 
+function inspectSectionStatusLines(value: unknown): string[] {
+  if (!Array.isArray(value)) return [];
+  return value.flatMap((section) => {
+    if (typeof section !== "object" || section === null || Array.isArray(section)) return [];
+    const record = section as Record<string, unknown>;
+    if (record.status !== "partial" && record.status !== "unavailable") return [];
+    const title = typeof record.title === "string" ? record.title : "section";
+    const reason = typeof record.reason === "string" ? ` — ${record.reason}` : "";
+    return [`${title.toLowerCase()}: ${record.status}${reason}`];
+  });
+}
+
 export function buildSimpleHeader(
   data: Record<string, unknown> | undefined,
   theme: Theme,
-  opts?: SimpleResultOptions,
 ): Text | null {
   if (!data) return null;
 
-  const candidateCount = (data.candidateCount as number) ?? (data.targetCount as number) ?? 0;
-  const omittedCount = opts?.omittedCount ?? 0;
-  const confidence = (data.confidence as string) ?? "";
-
-  if (candidateCount === 0 && !confidence) return null;
-
-  const badge = formatEvidenceBadge({
-    shownCount: candidateCount,
-    totalCount: candidateCount + omittedCount,
-    omittedCount,
-    partialReason: null,
-    label: "results",
-  });
-
+  const badges = evidenceBadges(data);
+  const confidence = typeof data.confidence === "string" ? data.confidence : "";
   const dot = theme.fg("dim", "·");
-  const segments = [theme.fg("accent", theme.bold(badge))];
+  const segments = badges.map((badge) => theme.fg("accent", theme.bold(badge)));
   if (confidence) {
     segments.push(`${theme.fg("dim", "confidence")} ${theme.fg("muted", confidence)}`);
   }
 
+  if (segments.length === 0) return null;
   return new Text(segments.join(` ${dot} `), 0, 0);
 }

package/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/abort-utils.ts

@@ -1,31 +0,0 @@
-/**
- * Abort controller/signal utility helpers.
- *
- * @module
- */
-
-/**
- * Combine a caller-provided abort controller with a generation timeout signal.
- *
- * Returns the combined signal and a cleanup function that removes both
- * event listeners. Callers must invoke cleanup in a `finally` block
- * to avoid listener leaks.
- */
-export function combineAbortSignals(
-  abort: AbortController,
-  timeoutMs: number,
-): { signal: AbortSignal; cleanup: () => void } {
-  const timeoutSignal = AbortSignal.timeout(timeoutMs);
-  const combinedController = new AbortController();
-  const onAbort = () => combinedController.abort();
-  abort.signal.addEventListener("abort", onAbort, { once: true });
-  timeoutSignal.addEventListener("abort", onAbort, { once: true });
-
-  return {
-    signal: combinedController.signal,
-    cleanup: () => {
-      abort.signal.removeEventListener("abort", onAbort);
-      timeoutSignal.removeEventListener("abort", onAbort);
-    },
-  };
-}

package/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/substrate-types.ts

@@ -1,11 +0,0 @@
-/** 0-based position used by LSP and code-intelligence internally. */
-export interface CodePosition {
-  line: number;
-  character: number;
-}
-
-/** Normalized location — flat replacement for LSP's nested Location/range shape. */
-export interface CodeLocation {
-  uri: string;
-  range: { start: CodePosition; end: CodePosition };
-}

package/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/types.ts

@@ -1,2 +0,0 @@
-// supi-core types domain — shared substrate types (zero dependencies, pure types).
-export type { CodeLocation, CodePosition } from "./substrate-types.ts";

package/node_modules/@mrclrchtr/supi-core/src/abort-utils.ts

@@ -1,31 +0,0 @@
-/**
- * Abort controller/signal utility helpers.
- *
- * @module
- */
-
-/**
- * Combine a caller-provided abort controller with a generation timeout signal.
- *
- * Returns the combined signal and a cleanup function that removes both
- * event listeners. Callers must invoke cleanup in a `finally` block
- * to avoid listener leaks.
- */
-export function combineAbortSignals(
-  abort: AbortController,
-  timeoutMs: number,
-): { signal: AbortSignal; cleanup: () => void } {
-  const timeoutSignal = AbortSignal.timeout(timeoutMs);
-  const combinedController = new AbortController();
-  const onAbort = () => combinedController.abort();
-  abort.signal.addEventListener("abort", onAbort, { once: true });
-  timeoutSignal.addEventListener("abort", onAbort, { once: true });
-
-  return {
-    signal: combinedController.signal,
-    cleanup: () => {
-      abort.signal.removeEventListener("abort", onAbort);
-      timeoutSignal.removeEventListener("abort", onAbort);
-    },
-  };
-}

package/node_modules/@mrclrchtr/supi-core/src/substrate-types.ts

@@ -1,11 +0,0 @@
-/** 0-based position used by LSP and code-intelligence internally. */
-export interface CodePosition {
-  line: number;
-  character: number;
-}
-
-/** Normalized location — flat replacement for LSP's nested Location/range shape. */
-export interface CodeLocation {
-  uri: string;
-  range: { start: CodePosition; end: CodePosition };
-}

package/node_modules/@mrclrchtr/supi-core/src/types.ts

@@ -1,2 +0,0 @@
-// supi-core types domain — shared substrate types (zero dependencies, pure types).
-export type { CodeLocation, CodePosition } from "./substrate-types.ts";

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/abort-utils.ts

@@ -1,31 +0,0 @@
-/**
- * Abort controller/signal utility helpers.
- *
- * @module
- */
-
-/**
- * Combine a caller-provided abort controller with a generation timeout signal.
- *
- * Returns the combined signal and a cleanup function that removes both
- * event listeners. Callers must invoke cleanup in a `finally` block
- * to avoid listener leaks.
- */
-export function combineAbortSignals(
-  abort: AbortController,
-  timeoutMs: number,
-): { signal: AbortSignal; cleanup: () => void } {
-  const timeoutSignal = AbortSignal.timeout(timeoutMs);
-  const combinedController = new AbortController();
-  const onAbort = () => combinedController.abort();
-  abort.signal.addEventListener("abort", onAbort, { once: true });
-  timeoutSignal.addEventListener("abort", onAbort, { once: true });
-
-  return {
-    signal: combinedController.signal,
-    cleanup: () => {
-      abort.signal.removeEventListener("abort", onAbort);
-      timeoutSignal.removeEventListener("abort", onAbort);
-    },
-  };
-}

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/substrate-types.ts

@@ -1,11 +0,0 @@
-/** 0-based position used by LSP and code-intelligence internally. */
-export interface CodePosition {
-  line: number;
-  character: number;
-}
-
-/** Normalized location — flat replacement for LSP's nested Location/range shape. */
-export interface CodeLocation {
-  uri: string;
-  range: { start: CodePosition; end: CodePosition };
-}

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-code-runtime/node_modules/@mrclrchtr/supi-core/src/types.ts

@@ -1,2 +0,0 @@
-// supi-core types domain — shared substrate types (zero dependencies, pure types).
-export type { CodeLocation, CodePosition } from "./substrate-types.ts";

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/src/abort-utils.ts

@@ -1,31 +0,0 @@
-/**
- * Abort controller/signal utility helpers.
- *
- * @module
- */
-
-/**
- * Combine a caller-provided abort controller with a generation timeout signal.
- *
- * Returns the combined signal and a cleanup function that removes both
- * event listeners. Callers must invoke cleanup in a `finally` block
- * to avoid listener leaks.
- */
-export function combineAbortSignals(
-  abort: AbortController,
-  timeoutMs: number,
-): { signal: AbortSignal; cleanup: () => void } {
-  const timeoutSignal = AbortSignal.timeout(timeoutMs);
-  const combinedController = new AbortController();
-  const onAbort = () => combinedController.abort();
-  abort.signal.addEventListener("abort", onAbort, { once: true });
-  timeoutSignal.addEventListener("abort", onAbort, { once: true });
-
-  return {
-    signal: combinedController.signal,
-    cleanup: () => {
-      abort.signal.removeEventListener("abort", onAbort);
-      timeoutSignal.removeEventListener("abort", onAbort);
-    },
-  };
-}

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/src/substrate-types.ts

@@ -1,11 +0,0 @@
-/** 0-based position used by LSP and code-intelligence internally. */
-export interface CodePosition {
-  line: number;
-  character: number;
-}
-
-/** Normalized location — flat replacement for LSP's nested Location/range shape. */
-export interface CodeLocation {
-  uri: string;
-  range: { start: CodePosition; end: CodePosition };
-}

package/node_modules/@mrclrchtr/supi-lsp/node_modules/@mrclrchtr/supi-core/src/types.ts

@@ -1,2 +0,0 @@
-// supi-core types domain — shared substrate types (zero dependencies, pure types).
-export type { CodeLocation, CodePosition } from "./substrate-types.ts";

package/node_modules/@mrclrchtr/supi-lsp/src/diagnostics/diagnostic-augmentation.ts

@@ -1,82 +0,0 @@
-import type { Diagnostic, Hover, MarkedString, MarkupContent } from "../config/types.ts";
-import type { LspManager } from "../manager/manager.ts";
-import { resolveSessionPath } from "../utils.ts";
-
-const AUGMENT_TIMEOUT_MS = 500;
-
-/**
- * Augment diagnostics with LSP hover and code_actions at the first severity-1 error.
- * Silently returns null if LSP is unavailable, times out, or there are no errors.
- */
-export async function augmentDiagnostics(
-  filePath: string,
-  diags: Diagnostic[],
-  manager: LspManager,
-  cwd: string,
-): Promise<string | null> {
-  const firstError = diags.find((d) => d.severity === 1);
-  if (!firstError) return null;
-
-  const resolvedPath = resolveSessionPath(cwd, filePath);
-  const client = await manager.getClientForFile(resolvedPath);
-  if (!client) return null;
-
-  const pos = firstError.range.start;
-
-  const [hoverResult, codeActionsResult] = await Promise.all([
-    withTimeout(client.hover(resolvedPath, pos), AUGMENT_TIMEOUT_MS),
-    withTimeout(
-      client.codeActions(resolvedPath, { start: pos, end: pos }, { diagnostics: [firstError] }),
-      AUGMENT_TIMEOUT_MS,
-    ),
-  ]);
-
-  const parts: string[] = [];
-
-  if (hoverResult) {
-    const hoverText = formatHoverForDiagnostics(hoverResult);
-    if (hoverText) parts.push(`💡 Hover info:\n${hoverText}`);
-  }
-
-  if (codeActionsResult && codeActionsResult.length > 0) {
-    const titles = codeActionsResult.map((a) => a.title).join(", ");
-    parts.push(`💡 Available fix: ${titles}`);
-  }
-
-  return parts.length > 0 ? parts.join("\n") : null;
-}
-
-/**
- * Extract raw hover text for inline diagnostic augmentation.
- * Intentionally strips markdown code-block framing (unlike formatHover)
- * to keep augmentation concise and readable inside diagnostic output.
- */
-function formatHoverForDiagnostics(hover: Hover): string {
-  const contents = hover.contents;
-  let text = "";
-
-  if (typeof contents === "string") {
-    text = contents;
-  } else if ("value" in contents) {
-    const mc = contents as MarkupContent | { language: string; value: string };
-    if ("kind" in mc) text = mc.value;
-    else text = mc.value;
-  } else if (Array.isArray(contents)) {
-    text = (contents as MarkedString[])
-      .map((c) => (typeof c === "string" ? c : c.value))
-      .join("\n");
-  }
-
-  const lines = text.split("\n").slice(0, 3);
-  return lines.join("\n");
-}
-
-async function withTimeout<T>(promise: Promise<T | null>, ms: number): Promise<T | null> {
-  let timer: ReturnType<typeof setTimeout> | null = null;
-  const timeoutPromise = new Promise<null>((resolve) => {
-    timer = setTimeout(() => resolve(null), ms);
-  });
-  const result = await Promise.race([promise, timeoutPromise]);
-  if (timer) clearTimeout(timer);
-  return result;
-}

package/node_modules/@mrclrchtr/supi-lsp/src/diagnostics/diagnostic-context.ts

@@ -1,116 +0,0 @@
-// Diagnostic context formatting for the LSP extension.
-// Extracted from guidance.ts to keep prompt surfaces separate from formatting logic.
-
-import type { Diagnostic } from "../config/types.ts";
-import type { OutstandingDiagnosticSummaryEntry } from "../manager/manager-types.ts";
-import { splitSuppressionDiagnostics } from "./suppression-diagnostics.ts";
-
-export const MAX_DETAILED_DIAGNOSTICS = 5;
-const MAX_DETAIL_LINES_PER_FILE = 3;
-
-interface DetailedDiagnostics {
-  file: string;
-  diagnostics: Diagnostic[];
-}
-
-export function formatDiagnosticsContext(
-  diagnostics: OutstandingDiagnosticSummaryEntry[],
-  maxFiles: number = 3,
-  detailed?: DetailedDiagnostics[],
-  staleWarning?: string | null,
-): string | null {
-  if (diagnostics.length === 0) return null;
-
-  const totalDiags = diagnostics.reduce((sum, d) => sum + d.total, 0);
-  const detailMap = buildDetailMap(totalDiags, detailed);
-
-  const lines: string[] = [];
-  if (staleWarning) lines.push(staleWarning);
-  const visible = diagnostics.slice(0, maxFiles);
-
-  for (const entry of visible) {
-    lines.push(`- ${entry.file}: ${formatCounts(entry)}`);
-    appendDetailLines(lines, detailMap?.get(entry.file));
-  }
-
-  const remaining = diagnostics.length - visible.length;
-  if (remaining > 0) {
-    lines.push(`- +${remaining} more file${remaining === 1 ? "" : "s"}`);
-  }
-
-  appendSuppressionCleanup(
-    lines,
-    visible.map((entry) => entry.file),
-    detailMap,
-  );
-
-  return [
-    '<extension-context source="supi-lsp">',
-    "Outstanding diagnostics — fix these before proceeding:",
-    ...lines,
-    "</extension-context>",
-  ].join("\n");
-}
-
-function buildDetailMap(
-  totalDiags: number,
-  detailed?: DetailedDiagnostics[],
-): Map<string, Diagnostic[]> | null {
-  if (totalDiags > MAX_DETAILED_DIAGNOSTICS || !detailed || detailed.length === 0) return null;
-  return new Map(detailed.map((d) => [d.file, d.diagnostics]));
-}
-
-function appendDetailLines(lines: string[], details?: Diagnostic[]): void {
-  if (!details) return;
-  for (const d of details.slice(0, MAX_DETAIL_LINES_PER_FILE)) {
-    const line = d.range.start.line + 1;
-    const char = d.range.start.character + 1;
-    const source = d.source ? ` ${d.source}` : "";
-    const msg = typeof d.message === "string" ? d.message : d.message.value;
-    lines.push(`  L${line} C${char}${source}: ${msg}`);
-  }
-  if (details.length > MAX_DETAIL_LINES_PER_FILE) {
-    const extra = details.length - MAX_DETAIL_LINES_PER_FILE;
-    lines.push(`  +${extra} more`);
-  }
-}
-
-function appendSuppressionCleanup(
-  lines: string[],
-  visibleFiles: string[],
-  detailMap: Map<string, Diagnostic[]> | null,
-): void {
-  if (!detailMap) return;
-
-  const suppressionLines: string[] = [];
-  for (const file of visibleFiles) {
-    const diagnostics = detailMap.get(file);
-    if (!diagnostics) continue;
-
-    const { suppressions } = splitSuppressionDiagnostics(diagnostics, 1);
-    if (suppressions.length === 0) continue;
-
-    suppressionLines.push(`- ${file}`);
-    appendDetailLines(suppressionLines, suppressions);
-  }
-
-  if (suppressionLines.length === 0) return;
-  lines.push("", "Stale suppression comments — clean these up:", ...suppressionLines);
-}
-
-export function diagnosticsContextFingerprint(content: string | null): string | null {
-  return content;
-}
-
-function formatCounts(entry: OutstandingDiagnosticSummaryEntry): string {
-  const counts: string[] = [];
-  if (entry.errors > 0) counts.push(pluralize(entry.errors, "error"));
-  if (entry.warnings > 0) counts.push(pluralize(entry.warnings, "warning"));
-  if (entry.information > 0) counts.push(pluralize(entry.information, "info"));
-  if (entry.hints > 0) counts.push(pluralize(entry.hints, "hint"));
-  return counts.join(", ");
-}
-
-function pluralize(count: number, word: string): string {
-  return `${count} ${word}${count === 1 ? "" : "s"}`;
-}