@mrclrchtr/supi-lsp 3.2.0 → 4.0.0

package/src/diagnostics/diagnostic-display.ts

@@ -1,69 +0,0 @@
-import type { Diagnostic } from "../config/types.ts";
-import type { OutstandingDiagnosticSummaryEntry } from "../manager/manager-types.ts";
-
-export function formatDiagnosticsDisplayContent(
-  diagnostics: OutstandingDiagnosticSummaryEntry[],
-  detailed?: Array<{ file: string; diagnostics: Diagnostic[] }>,
-): string {
-  const totals = collectDisplayTotals(diagnostics);
-  const summary = buildDisplaySummary(totals);
-
-  if (!detailed || detailed.length === 0) return summary;
-
-  const detailLines = buildDisplayDetailLines(detailed);
-  return detailLines.length > 0 ? `${summary}\n${detailLines.join("\n")}` : summary;
-}
-
-function collectDisplayTotals(
-  diagnostics: Array<{ errors: number; warnings: number; information: number; hints: number }>,
-) {
-  return diagnostics.reduce(
-    (acc, d) => ({
-      errors: acc.errors + d.errors,
-      warnings: acc.warnings + d.warnings,
-      information: acc.information + d.information,
-      hints: acc.hints + d.hints,
-    }),
-    { errors: 0, warnings: 0, information: 0, hints: 0 },
-  );
-}
-
-function buildDisplaySummary(totals: {
-  errors: number;
-  warnings: number;
-  information: number;
-  hints: number;
-}): string {
-  const parts: string[] = [];
-  if (totals.errors > 0) parts.push(`${totals.errors} error${totals.errors === 1 ? "" : "s"}`);
-  if (totals.warnings > 0)
-    parts.push(`${totals.warnings} warning${totals.warnings === 1 ? "" : "s"}`);
-  if (totals.information > 0)
-    parts.push(`${totals.information} info${totals.information === 1 ? "" : "s"}`);
-  if (totals.hints > 0) parts.push(`${totals.hints} hint${totals.hints === 1 ? "" : "s"}`);
-
-  return parts.length > 0
-    ? `LSP diagnostics injected (${parts.join(", ")})`
-    : "LSP diagnostics injected";
-}
-
-function buildDisplayDetailLines(
-  detailed: Array<{
-    file: string;
-    diagnostics: Diagnostic[];
-  }>,
-): string[] {
-  const lines: string[] = [];
-  for (const entry of detailed) {
-    for (const d of entry.diagnostics.slice(0, 3)) {
-      const line = d.range.start.line + 1;
-      const source = d.source ? ` ${d.source}` : "";
-      const msg = typeof d.message === "string" ? d.message : d.message.value;
-      lines.push(`   ${entry.file} L${line}${source}: ${msg}`);
-    }
-    if (entry.diagnostics.length > 3) {
-      lines.push(`   +${entry.diagnostics.length - 3} more`);
-    }
-  }
-  return lines;
-}

package/src/manager/capability-index.ts

@@ -1,24 +0,0 @@
-// Capability index — indexes and queries server capabilities.
-
-import { getSupportedLspServerActions } from "../config/server-actions.ts";
-import type { ServerCapabilities } from "../config/types.ts";
-import type { LspManager } from "./manager.ts";
-
-/**
- * Indexes and queries what each server supports.
- */
-export interface CapabilityIndex {
-  /** Get the list of supported LSP actions for a server's capabilities. */
-  getSupportedActions(capabilities: ServerCapabilities | null | undefined): string[];
-}
-
-/**
- * Create a CapabilityIndex for a workspace.
- */
-export function createCapabilityIndex(_manager: LspManager): CapabilityIndex {
-  return {
-    getSupportedActions(capabilities) {
-      return getSupportedLspServerActions(capabilities);
-    },
-  };
-}

package/src/manager/client-pool.ts

@@ -1,33 +0,0 @@
-// Client pool — manages LSP client lifecycle (start, stop, reconnect).
-
-import type { LspManager } from "./manager.ts";
-
-/**
- * Manages the lifecycle of LSP client instances for a workspace.
- * Currently delegates to LspManager; the extraction will grow
- * as responsibilities are migrated from manager.ts.
- */
-export interface ClientPool {
-  /**
-   * Ensure a file has an active client and return it.
-   * Returns null if no server can serve the file.
-   */
-  ensureFileOpen(filePath: string): ReturnType<LspManager["ensureFileOpen"]>;
-
-  /** Shut down all active clients. */
-  shutdownAll(): Promise<void>;
-}
-
-/**
- * Create a ClientPool backed by LspManager.
- */
-export function createClientPool(manager: LspManager): ClientPool {
-  return {
-    async ensureFileOpen(filePath: string) {
-      return manager.ensureFileOpen(filePath);
-    },
-    async shutdownAll() {
-      return manager.shutdownAll();
-    },
-  };
-}

package/src/manager/diagnostic-store.ts

@@ -1,51 +0,0 @@
-// Diagnostic store — stores, aggregates, and queries diagnostics.
-
-import type { Diagnostic } from "../config/types.ts";
-import type { LspManager } from "./manager.ts";
-
-/**
- * Stores and queries diagnostics per file.
- * Currently delegates to LspManager methods.
- */
-export interface DiagnosticStore {
-  /** Get workspace diagnostic summary grouped by file. */
-  getDiagnosticSummary(): Array<{ file: string; errors: number; warnings: number }>;
-
-  /** Get outstanding diagnostics at or above severity. */
-  getOutstandingDiagnostics(
-    maxSeverity?: number,
-  ): Array<{ file: string; diagnostics: Diagnostic[] }>;
-
-  /** Get outstanding diagnostic summary grouped by file. */
-  getOutstandingDiagnosticSummary(maxSeverity?: number): Array<{
-    file: string;
-    total: number;
-    errors: number;
-    warnings: number;
-    information: number;
-    hints: number;
-  }>;
-
-  /** Sync a file and return its diagnostics. */
-  syncAndGetDiagnostics(filePath: string, maxSeverity?: number): Promise<Diagnostic[] | null>;
-}
-
-/**
- * Create a DiagnosticStore backed by LspManager.
- */
-export function createDiagnosticStore(manager: LspManager): DiagnosticStore {
-  return {
-    getDiagnosticSummary() {
-      return manager.getDiagnosticSummary();
-    },
-    getOutstandingDiagnostics(maxSeverity = 1) {
-      return manager.getOutstandingDiagnostics(maxSeverity);
-    },
-    getOutstandingDiagnosticSummary(maxSeverity = 1) {
-      return manager.getOutstandingDiagnosticSummary(maxSeverity);
-    },
-    async syncAndGetDiagnostics(filePath: string, maxSeverity = 4) {
-      return manager.syncFileAndGetDiagnostics(filePath, maxSeverity);
-    },
-  };
-}

package/src/manager/manager-stale-resync.ts

@@ -1,47 +0,0 @@
-import * as path from "node:path";
-import { isLikelyStaleDiagnostic } from "../diagnostics/stale-diagnostics.ts";
-import type { LspManager } from "./manager.ts";
-
-/**
- * Force re-open files with module-resolution errors (e.g., "Cannot find module")
- * to trigger fresh analysis by the language server.
- *
- * The TypeScript server caches diagnostics by file content hash. When a new
- * file is created that resolves an existing file's import, the stale diagnostic
- * persists because the importing file's content hasn't changed. Re-opening the
- * file (didClose + didOpen) forces the server to re-resolve all imports.
- *
- * Returns true if any files were re-synced.
- */
-export async function forceResyncStaleModuleFiles(
-  manager: LspManager,
-  cwd: string,
-): Promise<boolean> {
-  const outstanding = manager.getOutstandingDiagnostics(1);
-  const staleFiles: string[] = [];
-
-  for (const entry of outstanding) {
-    if (entry.diagnostics.some((d) => isLikelyStaleDiagnostic(d))) {
-      staleFiles.push(entry.file);
-    }
-  }
-
-  if (staleFiles.length === 0) return false;
-
-  for (const file of staleFiles) {
-    const filePath = path.resolve(cwd, file);
-    // Close the file to clear cached diagnostics and remove from openDocs
-    manager.closeFile(filePath);
-    // Re-open to force the server to re-resolve imports
-    await manager.ensureFileOpen(filePath);
-  }
-
-  // Re-sync and wait for fresh diagnostics after the re-opens
-  try {
-    await manager.refreshOpenDiagnostics({ quietMs: 300, maxWaitMs: 2000 });
-  } catch {
-    // Best-effort: don't fail the agent turn if refresh has issues
-  }
-
-  return true;
-}

package/src/manager/recovery-coordinator.ts

@@ -1,37 +0,0 @@
-// Recovery coordinator — orchestrates stale diagnostic recovery.
-
-import type { LspManager } from "./manager.ts";
-
-/**
- * Orchestrates stale diagnostic detection and recovery.
- */
-export interface RecoveryCoordinator {
-  /**
-   * Trigger a workspace-wide diagnostics refresh.
-   * Returns info about refreshed/restarted clients and stale assessment.
-   */
-  recover(options?: {
-    restartIfStillStale?: boolean;
-    maxWaitMs?: number;
-    quietMs?: number;
-  }): Promise<{
-    refreshedClients: number;
-    restartedClients: number;
-    staleAssessment: {
-      suspected: boolean;
-      matchedFiles: Array<{ file: string; diagnostics: unknown[] }>;
-      warning: string | null;
-    };
-  }>;
-}
-
-/**
- * Create a RecoveryCoordinator backed by LspManager.
- */
-export function createRecoveryCoordinator(manager: LspManager): RecoveryCoordinator {
-  return {
-    async recover(options) {
-      return manager.recoverWorkspaceDiagnostics(options);
-    },
-  };
-}

package/src/manager/workspace-router.ts

@@ -1,44 +0,0 @@
-// Workspace router — routes files to the correct LSP client based on project root.
-
-import type { ProjectServerInfo } from "../config/server-config.ts";
-import type { LspManager } from "./manager.ts";
-
-/**
- * Routes file paths to the correct LSP client and tracks
- * which project roots are known.
- */
-export interface WorkspaceRouter {
-  /** Check whether a file can be served by any active server. */
-  canServeFile(filePath: string): boolean;
-
-  /** Check whether a source file type is supported. */
-  isSupportedSourceFile(filePath: string): boolean;
-
-  /** Get known project server info. */
-  getProjectServers(): ProjectServerInfo[];
-
-  /** Register detected servers from the workspace scan. */
-  registerDetectedServers(servers: Array<{ language: string; root: string }>): void;
-}
-
-/**
- * Create a WorkspaceRouter backed by LspManager.
- */
-export function createWorkspaceRouter(manager: LspManager): WorkspaceRouter {
-  return {
-    canServeFile(filePath: string) {
-      return manager.canServeFile(filePath);
-    },
-    isSupportedSourceFile(filePath: string) {
-      return manager.isSupportedSourceFile(filePath);
-    },
-    getProjectServers(): ProjectServerInfo[] {
-      return manager.getKnownProjectServers([]);
-    },
-    registerDetectedServers(servers: Array<{ language: string; root: string }>) {
-      manager.registerDetectedServers(
-        servers as unknown as Parameters<typeof manager.registerDetectedServers>[0],
-      );
-    },
-  };
-}