@fitchmultz/pi-mcp-adapter 6.0.0 → 7.1.0

package/mcp-status.ts

@@ -1,110 +0,0 @@
-import type { McpExtensionState } from "./state.ts";
-import {
-  MCP_STATUS_EVENT,
-  MCP_STATUS_SNAPSHOT_VERSION,
-  type McpServerStatusSnapshot,
-  type McpStatusSnapshot,
-} from "./types.ts";
-
-const FAILURE_BACKOFF_MS = 60 * 1000;
-
-export interface McpStatusEventBus {
-  emit(channel: string, data: unknown): void;
-}
-
-function getActiveFailureAgeSeconds(state: McpExtensionState, serverName: string): number | undefined {
-  const failedAt = state.failureTracker.get(serverName);
-  if (!failedAt) return undefined;
-  const ageMs = Date.now() - failedAt;
-  if (ageMs > FAILURE_BACKOFF_MS) return undefined;
-  return Math.round(ageMs / 1000);
-}
-
-/** Build a sanitized snapshot without connecting or querying any MCP server. */
-export function createMcpStatusSnapshot(state: McpExtensionState): McpStatusSnapshot {
-  const servers: McpServerStatusSnapshot[] = [];
-  let totalTools = 0;
-  let totalResources = 0;
-  let connectedCount = 0;
-  let disabledCount = 0;
-
-  for (const name of Object.keys(state.config.mcpServers)) {
-    const definition = state.config.mcpServers[name];
-    const disabled = definition?.disabled === true;
-    const connection = disabled ? undefined : state.manager.getConnection(name);
-    const metadata = disabled ? undefined : state.toolMetadata.get(name);
-    const toolCount = metadata?.filter(tool => !tool.resourceUri).length ?? (connection?.status === "connected" ? connection.tools.length : 0);
-    const resourceCount = disabled
-      ? undefined
-      : state.resourceCounts?.get(name) ?? (connection?.status === "connected" ? connection.resources.length : undefined);
-    const failedAgoSeconds = disabled ? undefined : getActiveFailureAgeSeconds(state, name);
-
-    let status: McpServerStatusSnapshot["status"] = "not-connected";
-    if (disabled) {
-      status = "disabled";
-      disabledCount++;
-    } else if (connection?.status === "connected") {
-      status = "connected";
-      connectedCount++;
-    } else if (connection?.status === "needs-auth") {
-      status = "needs-auth";
-    } else if (failedAgoSeconds !== undefined) {
-      status = "failed";
-    } else if (metadata !== undefined) {
-      status = "cached";
-    }
-
-    totalTools += disabled ? 0 : toolCount;
-    if (!disabled && resourceCount !== undefined) totalResources += resourceCount;
-    servers.push({
-      name,
-      status,
-      toolCount,
-      catalogKnown: metadata !== undefined || connection?.status === "connected",
-      ...(resourceCount !== undefined ? { resourceCount } : {}),
-      ...(status === "failed" && failedAgoSeconds !== undefined ? { failedAgoSeconds } : {}),
-      disabled,
-    });
-  }
-
-  return {
-    version: MCP_STATUS_SNAPSHOT_VERSION,
-    servers,
-    totalTools,
-    totalResources,
-    connectedCount,
-    disabledCount,
-  };
-}
-
-export function publishMcpStatusSnapshot(
-  state: McpExtensionState,
-  snapshot?: McpStatusSnapshot,
-): void {
-  const events = state.statusEvents;
-  if (!events) return;
-  try {
-    events.emit(MCP_STATUS_EVENT, snapshot ?? createMcpStatusSnapshot(state));
-  } catch {
-    // Event consumers must not be able to interrupt MCP operations.
-  }
-}
-
-export function publishMcpStatusShutdown(events: McpStatusEventBus | undefined): void {
-  if (!events) return;
-  try {
-    events.emit(MCP_STATUS_EVENT, {
-      version: MCP_STATUS_SNAPSHOT_VERSION,
-      servers: [],
-      totalTools: 0,
-      totalResources: 0,
-      connectedCount: 0,
-      disabledCount: 0,
-    } satisfies McpStatusSnapshot);
-  } catch {
-    // Event consumers must not be able to interrupt MCP shutdown.
-  }
-}
-
-export type { McpServerStatusSnapshot, McpStatusSnapshot } from "./types.ts";
-export { MCP_STATUS_EVENT, MCP_STATUS_SNAPSHOT_VERSION } from "./types.ts";

package/mcp-trace.ts

@@ -1,295 +0,0 @@
-import { appendFile, mkdir, writeFile } from "node:fs/promises";
-import { Buffer } from "node:buffer";
-import { dirname, isAbsolute, resolve } from "node:path";
-import type { Transport } from "@modelcontextprotocol/client";
-import type { JSONRPCMessage, MessageExtraInfo } from "@modelcontextprotocol/client";
-
-export const MCP_TRACE_SCHEMA_VERSION = 1;
-export const DEFAULT_MCP_TRACE_MAX_BYTES = 256 * 1024;
-export const DEFAULT_MCP_TRACE_MAX_EVENTS = 10_000;
-
-export type McpTraceDirection = "outbound" | "inbound";
-export type McpTraceTransport = "stdio" | "unix-socket" | "sse" | "streamable-http" | "unknown";
-export type McpTraceMessageKind = "request" | "response" | "notification";
-
-export interface McpTraceSettings {
-  /** Enable metadata-only protocol tracing for all servers unless overridden. */
-  enabled?: boolean;
-  /** JSONL destination. Relative paths are resolved from the session cwd. */
-  file?: string;
-  /** Maximum bytes retained in the per-session JSONL file. */
-  maxBytes?: number;
-  /** Maximum events retained in the per-session JSONL file. */
-  maxEvents?: number;
-}
-
-export interface McpTraceEvent {
-  version: typeof MCP_TRACE_SCHEMA_VERSION;
-  timestamp: string;
-  direction: McpTraceDirection;
-  server: string;
-  transport: McpTraceTransport;
-  kind: McpTraceMessageKind;
-  status: "sent" | "received" | "error";
-  method?: string;
-  id?: string | number | null;
-  relatedRequestId?: string | number;
-  errorCode?: number | string;
-  bytes?: number;
-  durationMs?: number;
-}
-
-export interface McpTraceWriterOptions {
-  filePath: string;
-  maxBytes?: number;
-  maxEvents?: number;
-  appendFile?: (path: string, data: string, options: { encoding: "utf8" }) => Promise<void>;
-  writeFile?: (path: string, data: string, options: { encoding: "utf8" }) => Promise<void>;
-  mkdir?: (path: string, options: { recursive: true }) => Promise<string | undefined>;
-}
-
-function boundedPositiveInteger(value: number | undefined, fallback: number): number {
-  if (!Number.isFinite(value) || value === undefined || value <= 0) return fallback;
-  return Math.floor(value);
-}
-
-/** Redact strings before they can become durable trace metadata. */
-export function redactTraceText(value: string, maxLength = 160): string {
-  if (/\b(?:token|secret|password|passwd|api[_-]?key|authorization|cookie)\b/i.test(value)) {
-    return "[REDACTED]";
-  }
-  let redacted = value
-    .replace(/\b[a-z][a-z\d+.-]*:\/\/[^\s"'<>]+/gi, "[REDACTED_URL]")
-    .replace(/\b(?:bearer|basic)\s+[A-Za-z0-9._~+/=-]+/gi, "[REDACTED_AUTH]")
-    .replace(/\b(?:token|secret|password|passwd|api[_-]?key|authorization|cookie)\s*[:=]\s*[^\s,;]+/gi, "$1=[REDACTED]");
-  if (redacted.length > maxLength) redacted = `${redacted.slice(0, maxLength - 1)}…`;
-  return redacted;
-}
-
-function messageKind(message: JSONRPCMessage): McpTraceMessageKind {
-  if ("method" in message) return "id" in message ? "request" : "notification";
-  return "response";
-}
-
-function traceId(value: unknown): string | number | null | undefined {
-  if (value === null) return null;
-  if (typeof value === "number" && Number.isFinite(value)) return value;
-  if (typeof value === "string") return "[REDACTED_ID]";
-  return undefined;
-}
-
-function messageBytes(message: JSONRPCMessage): number | undefined {
-  try {
-    return Buffer.byteLength(JSON.stringify(message), "utf8");
-  } catch {
-    return undefined;
-  }
-}
-
-export function createMcpTraceEvent(
-  direction: McpTraceDirection,
-  server: string,
-  transport: McpTraceTransport,
-  message: JSONRPCMessage,
-  status: "sent" | "received" | "error",
-  options?: { relatedRequestId?: unknown; durationMs?: number },
-): McpTraceEvent {
-  const kind = messageKind(message);
-  const bytes = messageBytes(message);
-  const event: McpTraceEvent = {
-    version: MCP_TRACE_SCHEMA_VERSION,
-    timestamp: new Date().toISOString(),
-    direction,
-    server: redactTraceText(server, 120),
-    transport,
-    kind,
-    status,
-    ...(bytes !== undefined ? { bytes } : {}),
-  };
-  if ("method" in message) event.method = redactTraceText(message.method, 120);
-  if ("id" in message) event.id = traceId(message.id) ?? null;
-  const relatedRequestId = traceId(options?.relatedRequestId);
-  if (relatedRequestId !== undefined && relatedRequestId !== null) event.relatedRequestId = relatedRequestId;
-  if ("error" in message && message.error && typeof message.error.code === "number") {
-    event.errorCode = message.error.code;
-  }
-  if (options?.durationMs !== undefined && Number.isFinite(options.durationMs)) {
-    event.durationMs = Math.max(0, Math.round(options.durationMs * 100) / 100);
-  }
-  return event;
-}
-
-export class McpTraceWriter {
-  private readonly maxBytes: number;
-  private readonly maxEvents: number;
-  private readonly append: NonNullable<McpTraceWriterOptions["appendFile"]>;
-  private readonly resetFile: NonNullable<McpTraceWriterOptions["writeFile"]>;
-  private readonly makeDirectory: NonNullable<McpTraceWriterOptions["mkdir"]>;
-  private bytesWritten = 0;
-  private eventsWritten = 0;
-  private queue = Promise.resolve();
-  private disabled = false;
-  private initializationFailed = false;
-  private persistenceError: unknown;
-  private readonly fileReady: Promise<void>;
-
-  constructor(private readonly options: McpTraceWriterOptions) {
-    this.maxBytes = boundedPositiveInteger(options.maxBytes, DEFAULT_MCP_TRACE_MAX_BYTES);
-    this.maxEvents = boundedPositiveInteger(options.maxEvents, DEFAULT_MCP_TRACE_MAX_EVENTS);
-    this.append = options.appendFile ?? (async (path, data, appendOptions) => {
-      await appendFile(path, data, appendOptions);
-    });
-    this.resetFile = options.writeFile ?? (async (path, data, writeOptions) => {
-      await writeFile(path, data, writeOptions);
-    });
-    this.makeDirectory = options.mkdir ?? (async path => {
-      await mkdir(path, { recursive: true });
-      return undefined;
-    });
-    this.fileReady = this.makeDirectory(dirname(this.options.filePath), { recursive: true })
-      .then(() => this.resetFile(this.options.filePath, "", { encoding: "utf8" }))
-      .catch(error => {
-        // Tracing must never change MCP request/response behavior.
-        this.persistenceError = error;
-        this.initializationFailed = true;
-        this.disabled = true;
-      });
-  }
-
-  get filePath(): string {
-    return this.options.filePath;
-  }
-
-  get isDisabled(): boolean {
-    return this.disabled;
-  }
-
-  get stats(): { bytes: number; events: number } {
-    return { bytes: this.bytesWritten, events: this.eventsWritten };
-  }
-
-  write(event: McpTraceEvent): void {
-    if (this.disabled || this.eventsWritten >= this.maxEvents) return;
-    let line: string;
-    try {
-      line = `${JSON.stringify(event)}\n`;
-    } catch {
-      this.disabled = true;
-      return;
-    }
-    const bytes = Buffer.byteLength(line, "utf8");
-    if (bytes > this.maxBytes - this.bytesWritten) {
-      this.disabled = true;
-      return;
-    }
-
-    this.bytesWritten += bytes;
-    this.eventsWritten += 1;
-    this.queue = this.queue.then(async () => {
-      await this.fileReady;
-      if (this.initializationFailed) return;
-      await this.append(this.options.filePath, line, { encoding: "utf8" });
-    }).catch(error => {
-      // Tracing must never change MCP request/response behavior.
-      this.persistenceError ??= error;
-      this.disabled = true;
-    });
-  }
-
-  async flush(strict = false): Promise<void> {
-    await this.fileReady;
-    await this.queue;
-    if (strict && this.persistenceError !== undefined) throw this.persistenceError;
-  }
-}
-
-export function createMcpTraceWriter(
-  sessionCwd: string | undefined,
-  settings: McpTraceSettings = {},
-  randomSuffix = Math.random().toString(36).slice(2, 10),
-): McpTraceWriter {
-  const timestamp = new Date().toISOString().replace(/[:.]/g, "-");
-  const configuredPath = settings.file;
-  const filePath = configuredPath
-    ? (isAbsolute(configuredPath) ? configuredPath : resolve(sessionCwd ?? process.cwd(), configuredPath))
-    : resolve(sessionCwd ?? process.cwd(), ".pi", "fitch-mcp-adapter", "traces", `mcp-${timestamp}-${randomSuffix}.jsonl`);
-  return new McpTraceWriter({
-    filePath,
-    ...(settings.maxBytes !== undefined ? { maxBytes: settings.maxBytes } : {}),
-    ...(settings.maxEvents !== undefined ? { maxEvents: settings.maxEvents } : {}),
-  });
-}
-
-export interface McpTraceObserver {
-  record(event: McpTraceEvent): void;
-}
-
-export function isMcpTraceEnabled(
-  definition: { trace?: boolean },
-  settings?: McpTraceSettings,
-): boolean {
-  return definition.trace ?? settings?.enabled === true;
-}
-
-/**
- * Compose the SDK's transport callbacks instead of replacing them. Protocol
- * assigns `onmessage` during connect, so the setter must keep the observer in
- * front of whichever callback the SDK installs.
- */
-export function wrapTransportWithMcpTrace(
-  transport: Transport,
-  server: string,
-  transportKind: McpTraceTransport,
-  observer: McpTraceObserver,
-): Transport {
-  let messageHandler = transport.onmessage;
-  const record = (event: McpTraceEvent): void => {
-    try {
-      observer.record(event);
-    } catch {
-      // An observer failure must never alter SDK transport behavior.
-    }
-  };
-  // Instrument the native instance: the SDK inspects stdio identity and HTTP
-  // hasPerRequestStream to choose negotiation and cancellation behavior.
-  const send = transport.send.bind(transport);
-  transport.send = async (message, options) => {
-    const started = performance.now();
-    const messages = Array.isArray(message) ? message : [message];
-    try {
-      await send(message, options);
-      for (const item of messages) {
-        record(createMcpTraceEvent("outbound", server, transportKind, item, "sent", {
-          durationMs: performance.now() - started,
-        }));
-      }
-    } catch (error) {
-      for (const item of messages) {
-        record(createMcpTraceEvent("outbound", server, transportKind, item, "error", {
-          durationMs: performance.now() - started,
-        }));
-      }
-      throw error;
-    }
-  };
-  Object.defineProperty(transport, "onmessage", {
-    configurable: true,
-    get: () => messageHandler,
-    set: (handler: Transport["onmessage"]) => {
-      messageHandler = handler ? (message: JSONRPCMessage, extra?: MessageExtraInfo) => {
-        record(createMcpTraceEvent("inbound", server, transportKind, message, "received"));
-        handler(message, extra);
-      } : undefined;
-    },
-  });
-  return transport;
-}
-
-export function traceTransportKind(definition: { command?: string; url?: string; socket?: string }, transport: Transport): McpTraceTransport {
-  if (definition.command) return "stdio";
-  if (definition.socket) return "unix-socket";
-  const constructorName = transport.constructor?.name.toLowerCase() ?? "";
-  if (constructorName.includes("sse")) return "sse";
-  if (constructorName.includes("streamable")) return "streamable-http";
-  return definition.url ? "streamable-http" : "unknown";
-}

package/metadata-cache.ts

@@ -1,271 +0,0 @@
-// metadata-cache.ts - Persistent MCP metadata cache
-import { existsSync, readFileSync, writeFileSync, renameSync, mkdirSync } from "node:fs";
-import { dirname } from "node:path";
-import { getAdapterPath } from "./agent-dir.ts";
-import { createHash } from "node:crypto";
-import type {
-  CachedPrompt,
-  CachedResource,
-  CachedTool,
-  McpConfig,
-  McpTool,
-  McpResource,
-  McpPrompt,
-  McpPromptArgument,
-  MetadataCache,
-  ServerCacheEntry,
-  ServerEntry,
-  ToolMetadata,
-  PromptMetadata,
-} from "./types.ts";
-import { formatPromptCommandName, isServerDisabled, resolveToolPrefix, type ToolPrefix } from "./types.ts";
-import { buildToolMetadata } from "./tool-metadata.ts";
-import {
-  interpolateEnvRecord,
-  resolveBearerToken,
-  resolveConfigPath,
-  resolveServerUrl,
-} from "./utils.ts";
-
-// Version 2 stores raw descriptors; version 1 readers cannot enforce raw UI visibility.
-const CACHE_VERSION = 2;
-const CACHE_MAX_AGE_MS = 7 * 24 * 60 * 60 * 1000;
-
-export type { CachedPrompt, CachedResource, CachedTool, MetadataCache, ServerCacheEntry } from "./types.ts";
-
-export function getMetadataCachePath(): string {
-  return getAdapterPath("mcp-cache.json");
-}
-
-export function loadMetadataCache(enabled = true): MetadataCache | null {
-  if (!enabled) return null;
-  const cachePath = getMetadataCachePath();
-  if (!existsSync(cachePath)) return null;
-  try {
-    const raw = JSON.parse(readFileSync(cachePath, "utf-8"));
-    if (!raw || typeof raw !== "object") return null;
-    if (raw.version !== 1 && raw.version !== CACHE_VERSION) return null;
-    if (!raw.servers || typeof raw.servers !== "object") return null;
-    return raw as MetadataCache;
-  } catch {
-    return null;
-  }
-}
-
-export function saveMetadataCache(cache: MetadataCache, enabled = true): void {
-  if (!enabled) return;
-  const cachePath = getMetadataCachePath();
-  const dir = dirname(cachePath);
-  mkdirSync(dir, { recursive: true });
-
-  let merged: MetadataCache = { version: CACHE_VERSION, servers: {} };
-  try {
-    if (existsSync(cachePath)) {
-      const existing = JSON.parse(readFileSync(cachePath, "utf-8")) as MetadataCache;
-      if (existing && (existing.version === 1 || existing.version === CACHE_VERSION) && existing.servers) {
-        merged.servers = { ...existing.servers };
-      }
-    }
-  } catch {
-    // Ignore parse errors and proceed with empty cache
-  }
-
-  merged.version = CACHE_VERSION;
-  merged.servers = { ...merged.servers, ...cache.servers };
-
-  const tmpPath = `${cachePath}.${process.pid}.tmp`;
-  writeFileSync(tmpPath, JSON.stringify(merged, null, 2), "utf-8");
-  renameSync(tmpPath, cachePath);
-}
-
-export function computeServerHash(definition: ServerEntry): string {
-  // Hash only fields that affect server identity and tool/resource output.
-  // Exclude lifecycle, idleTimeout, requestTimeoutMs, debug — those are runtime behavior settings
-  // that don't change which tools a server exposes.
-  const identity: Record<string, unknown> = {
-    command: definition.command,
-    args: definition.args,
-    socket: resolveConfigPath(definition.socket),
-    env: interpolateEnvRecord(definition.env),
-    cwd: resolveConfigPath(definition.cwd),
-    url: resolveServerUrl(definition),
-    headers: interpolateEnvRecord(definition.headers),
-    auth: definition.auth,
-    bearerToken: resolveBearerToken(definition),
-    bearerTokenEnv: definition.bearerTokenEnv,
-    exposeResources: definition.exposeResources,
-    includeTools: definition.includeTools,
-    excludeTools: definition.excludeTools,
-  };
-  const normalized = stableStringify(identity);
-  return createHash("sha256").update(normalized).digest("hex");
-}
-
-export function isServerCacheValid(
-  entry: ServerCacheEntry,
-  definition: ServerEntry,
-  maxAgeMs: number = CACHE_MAX_AGE_MS
-): boolean {
-  let configHash: string;
-  try {
-    configHash = computeServerHash(definition);
-  } catch {
-    return false;
-  }
-  if (!entry || entry.configHash !== configHash) return false;
-  if (!entry.cachedAt || typeof entry.cachedAt !== "number") return false;
-  if (maxAgeMs > 0 && Date.now() - entry.cachedAt > maxAgeMs) return false;
-  return true;
-}
-
-export function parseDirectToolSelectors(selectors: string[]): {
-  servers: Set<string>;
-  tools: Map<string, Set<string>>;
-} {
-  const servers = new Set<string>();
-  const tools = new Map<string, Set<string>>();
-
-  for (let selector of selectors) {
-    selector = selector.replace(/\/+$/, "");
-    if (selector.includes("/")) {
-      const [server, tool] = selector.split("/", 2);
-      if (server && tool) {
-        const serverTools = tools.get(server) ?? new Set<string>();
-        serverTools.add(tool);
-        tools.set(server, serverTools);
-      } else if (server) {
-        servers.add(server);
-      }
-    } else if (selector) {
-      servers.add(selector);
-    }
-  }
-
-  return { servers, tools };
-}
-
-export function getMissingConfiguredDirectToolServers(
-  config: McpConfig,
-  cache: MetadataCache | null,
-  envOverride?: string[],
-): string[] {
-  const missing: string[] = [];
-  const globalDirect = config.settings?.directTools;
-  const envSelection = envOverride ? parseDirectToolSelectors(envOverride) : null;
-
-  for (const [serverName, definition] of Object.entries(config.mcpServers)) {
-    if (isServerDisabled(definition)) continue;
-    const hasDirectTools = envSelection
-      ? envSelection.servers.has(serverName) || envSelection.tools.has(serverName)
-      : definition.directTools !== undefined
-        ? !!definition.directTools
-        : !!globalDirect;
-
-    if (!hasDirectTools) continue;
-
-    const serverCache = cache?.servers?.[serverName];
-    if (!serverCache || !isServerCacheValid(serverCache, definition)) {
-      missing.push(serverName);
-    }
-  }
-
-  return missing;
-}
-
-export function reconstructToolMetadata(
-  serverName: string,
-  entry: ServerCacheEntry,
-  prefix: ToolPrefix,
-  definition: Pick<ServerEntry, "exposeResources" | "includeTools" | "excludeTools" | "toolPrefix" | "disabled">
-): ToolMetadata[] {
-  return buildToolMetadata(
-    (entry.tools ?? []).filter(tool => tool?.name).map(restoreCachedTool),
-    entry.resources ?? [],
-    definition,
-    serverName,
-    prefix,
-  ).metadata;
-}
-
-function restoreCachedTool(tool: CachedTool): McpTool {
-  const { uiResourceUri, uiVisibility, uiStreamMode, ...descriptor } = tool;
-  if (descriptor._meta !== undefined || (uiResourceUri === undefined && uiVisibility === undefined && uiStreamMode === undefined)) {
-    return descriptor;
-  }
-  return {
-    ...descriptor,
-    _meta: {
-      ui: {
-        ...(uiResourceUri !== undefined ? { resourceUri: uiResourceUri } : {}),
-        ...(uiVisibility !== undefined ? { visibility: uiVisibility } : {}),
-        ...(uiStreamMode !== undefined ? { "pi-mcp-adapter.streamMode": uiStreamMode } : {}),
-      },
-    },
-  };
-}
-
-export function serializeTools(tools: McpTool[]): CachedTool[] {
-  return tools.filter(tool => tool?.name);
-}
-
-export function serializeResources(resources: McpResource[]): CachedResource[] {
-  return resources.filter(resource => resource?.name && resource?.uri);
-}
-
-export function serializePrompts(prompts: McpPrompt[]): CachedPrompt[] {
-  return (prompts ?? [])
-    .filter(prompt => prompt?.name)
-    .map(prompt => ({
-      name: prompt.name,
-      ...(prompt.title !== undefined ? { title: prompt.title } : {}),
-      ...(prompt.description !== undefined ? { description: prompt.description } : {}),
-      ...(Array.isArray(prompt.arguments)
-        ? {
-            arguments: prompt.arguments.filter(argument => argument?.name).map(argument => ({
-              name: argument.name,
-              ...(argument.description !== undefined ? { description: argument.description } : {}),
-              ...(argument.required !== undefined ? { required: argument.required } : {}),
-            })),
-          }
-        : {}),
-    }));
-}
-
-export function reconstructPromptMetadata(
-  serverName: string,
-  prompts: ReadonlyArray<McpPrompt | CachedPrompt>,
-  prefix: ToolPrefix,
-  definition?: Pick<ServerEntry, "toolPrefix">,
-): PromptMetadata[] {
-  const effectivePrefix = resolveToolPrefix(definition, prefix);
-  return (prompts ?? []).filter(prompt => prompt?.name).map(prompt => {
-    const args: McpPromptArgument[] = Array.isArray(prompt.arguments)
-      ? prompt.arguments.filter(argument => argument?.name).map(argument => ({
-          name: argument.name,
-          ...(argument.description !== undefined ? { description: argument.description } : {}),
-          ...(argument.required !== undefined ? { required: argument.required } : {}),
-        }))
-      : [];
-    return {
-      serverName,
-      originalName: prompt.name,
-      commandName: formatPromptCommandName(prompt.name, serverName, effectivePrefix),
-      ...(prompt.title !== undefined ? { title: prompt.title } : {}),
-      description: prompt.description ?? "",
-      arguments: args,
-    };
-  });
-}
-
-function stableStringify(value: unknown): string {
-  if (value === null || value === undefined || typeof value !== "object") {
-    const serialized = JSON.stringify(value);
-    return serialized === undefined ? "undefined" : serialized;
-  }
-  if (Array.isArray(value)) {
-    return `[${value.map(v => stableStringify(v)).join(",")}]`;
-  }
-  const obj = value as Record<string, unknown>;
-  const keys = Object.keys(obj).sort();
-  return `{${keys.map(k => `${JSON.stringify(k)}:${stableStringify(obj[k])}`).join(",")}}`;
-}