muonroi-cli 1.4.0 → 1.4.1

package/src/mcp/auto-setup.ts

@@ -1,99 +0,0 @@
-import type { McpServerConfig } from "../utils/settings.js";
-import { loadMcpServers, saveMcpServers } from "../utils/settings.js";
-
-const DEFAULT_CONFIGS: McpServerConfig[] = [
-  {
-    id: "filesystem",
-    label: "Filesystem",
-    enabled: true,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "@modelcontextprotocol/server-filesystem", "."],
-  },
-  {
-    id: "playwright",
-    label: "Playwright",
-    enabled: true,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "@playwright/mcp"],
-  },
-  {
-    id: "memory",
-    label: "Memory",
-    enabled: false,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "@modelcontextprotocol/server-memory"],
-  },
-  {
-    id: "figma",
-    label: "Figma",
-    enabled: false,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "figma-developer-mcp", "--stdio"],
-    env: { FIGMA_API_KEY: "" },
-  },
-  {
-    id: "context7",
-    label: "Context7 (Library Docs)",
-    enabled: true,
-    transport: "http",
-    url: "https://mcp.context7.com/mcp",
-  },
-  {
-    id: "fetch",
-    label: "Fetch (URL → markdown)",
-    enabled: true,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "mcp-fetch-server"],
-  },
-  {
-    id: "tavily",
-    label: "Tavily Web Search",
-    enabled: false,
-    transport: "stdio",
-    command: "npx",
-    args: ["-y", "tavily-mcp"],
-    env: { TAVILY_API_KEY: "" },
-  },
-];
-
-const PACKAGE_MIGRATIONS: Record<string, string> = {
-  "@anthropic-ai/figma-mcp": "figma-developer-mcp",
-  "@anthropic-ai/mcp-playwright": "@playwright/mcp",
-};
-
-function migrateServers(servers: McpServerConfig[]): boolean {
-  let changed = false;
-  for (const server of servers) {
-    const pkgArg = server.args?.find((a) => PACKAGE_MIGRATIONS[a]);
-    if (pkgArg) {
-      server.args = server.args!.map((a) => PACKAGE_MIGRATIONS[a] ?? a);
-      changed = true;
-    }
-    if (server.id === "figma" && server.args?.includes("figma-developer-mcp") && !server.args.includes("--stdio")) {
-      server.args = [...server.args, "--stdio"];
-      changed = true;
-    }
-  }
-  return changed;
-}
-
-export function ensureDefaultMcpServers(): McpServerConfig[] {
-  try {
-    const existing = loadMcpServers();
-    let dirty = migrateServers(existing);
-    const existingIds = new Set(existing.map((s) => s.id));
-    const toAdd = DEFAULT_CONFIGS.filter((s) => !existingIds.has(s.id));
-    if (toAdd.length > 0) dirty = true;
-    if (!dirty) return existing;
-    const merged = toAdd.length > 0 ? [...toAdd, ...existing] : existing;
-    saveMcpServers(merged);
-    return merged;
-  } catch {
-    return [];
-  }
-}

package/src/mcp/cap-tool-result.ts

@@ -1,68 +0,0 @@
-import { MAX_TOOL_OUTPUT_CHARS, truncateOutput } from "../tools/registry.js";
-
-/**
- * Cap the SIZE of an MCP tool result so it can't bypass the per-call output
- * budget that every built-in tool already respects (`MAX_TOOL_OUTPUT_CHARS`).
- *
- * Why this exists: built-in tools route their output through `formatResult` →
- * `truncateOutput`, so a single read/grep/bash result is capped at 32 KB. MCP
- * tools, however, are registered by spreading the AI-SDK MCP client's tool
- * object verbatim — its `execute()` returns the raw server payload UNCAPPED. A
- * single MCP call (docs fetch, large query, devtools snapshot, multi-file read)
- * could therefore inject arbitrarily large text into the model context, which
- * is exactly the cost leak that hurts cheap models the most.
- *
- * The AI-SDK MCP `execute()` (when the tool has no outputSchema) returns
- * `{ type: "content", value: [{ type: "text", text }, { type: "image", ... }] }`.
- * We truncate the text parts under a shared cumulative budget and leave
- * non-text parts (images, etc.) untouched — corrupting base64 media would be
- * worse than the bloat. A plain string result is truncated directly. Anything
- * else (structured `outputSchema` results) is returned unchanged: those are
- * schema-bounded and the AI SDK parses them, so truncation would break them.
- */
-
-interface McpContentResult {
-  type: "content";
-  value: unknown[];
-}
-
-function isContentResult(value: unknown): value is McpContentResult {
-  return (
-    !!value &&
-    typeof value === "object" &&
-    (value as { type?: unknown }).type === "content" &&
-    Array.isArray((value as { value?: unknown }).value)
-  );
-}
-
-function isTextPart(part: unknown): part is { type: "text"; text: string } {
-  return (
-    !!part &&
-    typeof part === "object" &&
-    (part as { type?: unknown }).type === "text" &&
-    typeof (part as { text?: unknown }).text === "string"
-  );
-}
-
-export function capMcpToolResult(result: unknown, maxChars: number = MAX_TOOL_OUTPUT_CHARS): unknown {
-  if (typeof result === "string") {
-    return truncateOutput(result, maxChars);
-  }
-
-  if (isContentResult(result)) {
-    let remaining = maxChars;
-    const value = result.value.map((part) => {
-      if (!isTextPart(part)) return part;
-      const { text } = part;
-      if (remaining <= 0) {
-        return { ...part, text: `... [${text.length} chars omitted; full MCP output in transcript] ...` };
-      }
-      const capped = truncateOutput(text, remaining);
-      remaining -= Math.min(text.length, remaining);
-      return { ...part, text: capped };
-    });
-    return { ...result, value };
-  }
-
-  return result;
-}

package/src/mcp/catalog.ts

@@ -1,155 +0,0 @@
-import type { McpRemoteTransport } from "../utils/settings.js";
-
-export interface McpCatalogEntry {
-  id: string;
-  name: string;
-  description: string;
-  directoryUrl: string;
-  sourceUrl?: string;
-  starterTransport?: McpRemoteTransport | "stdio";
-}
-
-export const POPULAR_MCP_CATALOG: McpCatalogEntry[] = [
-  {
-    id: "filesystem",
-    name: "Filesystem",
-    description: "Read/write access to the local filesystem with safety controls.",
-    directoryUrl: "https://github.com/modelcontextprotocol/servers/tree/main/src/filesystem",
-    sourceUrl: "https://github.com/modelcontextprotocol/servers",
-    starterTransport: "stdio",
-  },
-  {
-    id: "playwright",
-    name: "Playwright",
-    description: "Browser automation and web testing via Playwright.",
-    directoryUrl: "https://github.com/microsoft/playwright-mcp",
-    sourceUrl: "https://github.com/microsoft/playwright-mcp",
-    starterTransport: "stdio",
-  },
-  {
-    id: "memory",
-    name: "Memory",
-    description: "Persistent knowledge graph memory for context recall.",
-    directoryUrl: "https://github.com/modelcontextprotocol/servers/tree/main/src/memory",
-    sourceUrl: "https://github.com/modelcontextprotocol/servers",
-    starterTransport: "stdio",
-  },
-  {
-    id: "github",
-    name: "GitHub",
-    description: "Integration with GitHub issues and related workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-github",
-    sourceUrl: "https://github.com/modelcontextprotocol/servers/tree/main/src/github",
-    starterTransport: "stdio",
-  },
-  {
-    id: "supabase",
-    name: "Supabase",
-    description: "PostgREST-backed operations for Postgres databases.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-supabase",
-  },
-  {
-    id: "vercel",
-    name: "Vercel",
-    description: "Vercel infrastructure and deployment workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-vercel",
-  },
-  {
-    id: "cloudflare",
-    name: "Cloudflare",
-    description: "Workers, KV, R2, and D1 operations on Cloudflare.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-cloudflare",
-  },
-  {
-    id: "stripe",
-    name: "Stripe",
-    description: "Stripe API access for billing and payment workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-stripe",
-  },
-  {
-    id: "notion",
-    name: "Notion",
-    description: "Databases, pages, blocks, and search inside Notion workspaces.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-notion",
-  },
-  {
-    id: "slack",
-    name: "Slack",
-    description: "Slack workspace messaging and channel workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-slack",
-  },
-  {
-    id: "sentry",
-    name: "Sentry",
-    description: "Issue analysis and debugging workflows for Sentry projects.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-sentry",
-  },
-  {
-    id: "figma",
-    name: "Figma",
-    description: "Design data from Figma for implementation workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-figma",
-  },
-  {
-    id: "context7",
-    name: "Context7 (Library Docs)",
-    description: "Version-pinned, chunked library documentation for Claude/Gemini/Codex-style coding agents.",
-    directoryUrl: "https://context7.com",
-    sourceUrl: "https://github.com/upstash/context7",
-    starterTransport: "http",
-  },
-  {
-    id: "fetch",
-    name: "Fetch (URL → markdown)",
-    description: "Fetches a URL and converts HTML to markdown for the agent to read.",
-    directoryUrl: "https://www.npmjs.com/package/mcp-fetch-server",
-    sourceUrl: "https://www.npmjs.com/package/mcp-fetch-server",
-    starterTransport: "stdio",
-  },
-  {
-    id: "tavily",
-    name: "Tavily Web Search",
-    description: "LLM-tuned web search with answer summaries and rerank. Free tier 1k/month. Requires TAVILY_API_KEY.",
-    directoryUrl: "https://tavily.com",
-    sourceUrl: "https://github.com/tavily-ai/tavily-mcp",
-    starterTransport: "stdio",
-  },
-  {
-    id: "firebase",
-    name: "Firebase",
-    description: "Auth, Firestore, and Storage operations.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-firebase",
-  },
-  {
-    id: "docker",
-    name: "Docker",
-    description: "Manage containers, images, volumes, and networks.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-docker",
-    starterTransport: "stdio",
-  },
-  {
-    id: "prisma",
-    name: "Prisma",
-    description: "Prisma database workflows and Postgres management.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-prisma",
-  },
-  {
-    id: "mongodb",
-    name: "MongoDB",
-    description: "Read-only exploration, queries, and aggregation for MongoDB.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-mongodb",
-  },
-  {
-    id: "puppeteer",
-    name: "Puppeteer",
-    description: "Browser automation via Puppeteer MCP.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-puppeteer",
-    starterTransport: "stdio",
-  },
-  {
-    id: "browserbase",
-    name: "Browserbase",
-    description: "Cloud browser automation workflows.",
-    directoryUrl: "https://cursor.directory/plugins/mcp-browserbase",
-  },
-];

package/src/mcp/ee-tools.ts

@@ -1,204 +0,0 @@
-/**
- * src/mcp/ee-tools.ts
- *
- * EE (Experience Engine) MCP tools: ee_query (semantic recall) + ee_health +
- * ee_feedback (rate a recalled entry). Read-mostly; ee_feedback POSTs a verdict.
- *
- * Anti-mù: ee_query supports explicit "recent task checkpoint" / "Progress DONE" queries
- * so the agent (or sub-agent) can deliberately confirm finished subtasks after compactions.
- * collections: prefer "experience-behavioral" for compaction checkpoints (see layer3).
- *
- * Feedback gate: ee_query stamps every returned `[id col]` into a session-scoped
- * pending ledger (recall-ledger.ts). Subsequent ee_query calls surface (soft) or
- * refuse on (hard) accumulated unrated debt, so useful recalls actually get a
- * verdict via ee_feedback — the signal the brain needs to keep the good entries
- * and prune the rest. Mode via EXPERIENCE_RECALL_FEEDBACK_GATE = off|soft|hard
- * (default soft); hard-mode threshold via EXPERIENCE_RECALL_FEEDBACK_THRESHOLD.
- *
- * Dependencies are injected (deps) so unit tests never touch the network/singleton.
- */
-
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { z } from "zod";
-import { type PendingRecall, type RecallLedger, sessionRecallLedger } from "../ee/recall-ledger.js";
-import {
-  type FeedbackResult,
-  type FeedbackVerdict,
-  formatRecallForAgent,
-  healthEE,
-  type NoiseReason,
-  recallEE,
-} from "../ee/search.js";
-import type { EERecallResponse } from "../ee/types.js";
-
-export interface EEToolDeps {
-  recall?: (query: string, opts: { project?: string }) => Promise<EERecallResponse | null>;
-  health?: () => Promise<{ ok: boolean; status: number }>;
-  feedback?: (
-    pointId: string,
-    collection: string,
-    verdict: FeedbackVerdict,
-    reason?: NoiseReason,
-  ) => Promise<FeedbackResult>;
-  /** Session pending-recall ledger. Defaults to the process singleton. */
-  ledger?: RecallLedger;
-}
-
-function ok(data: unknown) {
-  return { content: [{ type: "text" as const, text: JSON.stringify(data) }] };
-}
-function okText(text: string) {
-  return { content: [{ type: "text" as const, text }] };
-}
-function fail(error: string, message: string) {
-  return {
-    content: [{ type: "text" as const, text: JSON.stringify({ error, message }) }],
-    isError: true,
-  };
-}
-
-type GateMode = "off" | "soft" | "hard";
-
-/** Resolve the feedback-gate policy from env at call time (so tests can vary it). */
-function resolveGate(): { mode: GateMode; threshold: number } {
-  const raw = String(process.env.EXPERIENCE_RECALL_FEEDBACK_GATE ?? "soft")
-    .trim()
-    .toLowerCase();
-  const mode: GateMode = raw === "off" || raw === "hard" ? raw : "soft";
-  const parsed = Number.parseInt(process.env.EXPERIENCE_RECALL_FEEDBACK_THRESHOLD ?? "", 10);
-  const threshold = Number.isFinite(parsed) && parsed > 0 ? parsed : 3;
-  return { mode, threshold };
-}
-
-/** One readable block naming the still-unrated recalls + how to clear them. */
-function formatPendingBlock(pending: PendingRecall[], hard: boolean, max = 8): string {
-  const shown = pending.slice(0, max);
-  const lines = shown.map((p) => `  - [${p.id} ${p.collection ?? "?"}]  (from recall: "${p.query}")`);
-  const more = pending.length > max ? `\n  …and ${pending.length - max} more` : "";
-  const head = hard
-    ? `⚠️ FEEDBACK GATE — ${pending.length} earlier recall(s) are still unrated. Rate them with ` +
-      `ee_feedback(id, collection, verdict) before pulling new recalls:`
-    : `⚠️ ${pending.length} earlier recall(s) still unrated — call ` +
-      `ee_feedback(id, collection, verdict=followed|ignored|noise) so the brain keeps what helped and prunes the rest:`;
-  return `${head}\n${lines.join("\n")}${more}`;
-}
-
-export function registerEETools(server: McpServer, deps: EEToolDeps = {}): void {
-  // Default to the shared EE recall/health/feedback helpers (src/ee/search.ts) so
-  // the MCP tools and the in-CLI builtin tools resolve auth/baseUrl the same way.
-  // Tests inject `deps` to avoid the network and a fresh `ledger` to stay isolated.
-  const recall = deps.recall ?? ((q, o) => recallEE(q, o));
-  const health = deps.health ?? (() => healthEE());
-  const ledger = deps.ledger ?? sessionRecallLedger;
-
-  server.registerTool(
-    "ee_query",
-    {
-      description:
-        "Active recall over the Experience Engine brain — prior decisions, gotchas, learned warnings/recipes, and " +
-        "task checkpoints for this codebase — via the recallMode pipeline (same path as exp-recall.js). " +
-        "CALL THIS PROACTIVELY, before acting: when starting work in an unfamiliar area, when unsure how something " +
-        "is done in this stack, before a risky or hard-to-reverse step, or to recall finished work after a " +
-        "compaction (e.g. query='recent compaction checkpoint Progress DONE for <subtask>'). A deliberate query " +
-        "here is cheaper than re-deriving or repeating a past mistake. Returns a formatted index whose entries " +
-        "carry `[id col]` handles — after you act on a recall, rate each entry you used or judged with the " +
-        "ee_feedback tool (followed/ignored/noise) so the brain keeps what helped and prunes the rest; unrated " +
-        "recalls are surfaced back to you on the next ee_query. Optional project scopes the recall. Returns a " +
-        "compact ranked index (cosine-ranked, strongest first), capped at maxChars (default 6000, range " +
-        "500-20000) and truncated from the tail; raise maxChars to see more. Returns ee_unavailable if EE is " +
-        "down (then proceed without it).",
-      inputSchema: {
-        query: z.string().min(1).max(1000),
-        project: z.string().max(200).optional(),
-        maxChars: z.number().int().min(500).max(20_000).optional(),
-      },
-    },
-    async ({ query, project, maxChars }) => {
-      const gate = resolveGate();
-      const pendingBefore = gate.mode === "off" ? [] : ledger.pending();
-      // Hard gate: refuse a NEW recall while unrated debt is at/over threshold —
-      // do not even spend the brain call. Below threshold, fall through to soft.
-      if (gate.mode === "hard" && pendingBefore.length >= gate.threshold) {
-        return fail("feedback_required", formatPendingBlock(pendingBefore, true));
-      }
-      try {
-        const resp = await recall(query, { project });
-        if (resp === null) {
-          return fail("ee_unavailable", "EE recall returned no response (server down, timeout, or circuit open)");
-        }
-        // Stamp the new entries as pending debt (unless the gate is off).
-        if (gate.mode !== "off") ledger.record(resp.entries, query);
-        // Return the compact `[id col]` index (capped) rather than JSON.stringify
-        // of the whole response — the wide-net recall text (~30k) otherwise blows
-        // the MCP per-result token cap and spills to a file. See formatRecallForAgent.
-        const index = formatRecallForAgent(resp, { query, maxChars });
-        if (gate.mode !== "off" && pendingBefore.length > 0) {
-          return okText(`${formatPendingBlock(pendingBefore, false)}\n\n${index}`);
-        }
-        return okText(index);
-      } catch (e) {
-        return fail("ee_unavailable", e instanceof Error ? e.message : String(e));
-      }
-    },
-  );
-
-  server.registerTool(
-    "ee_feedback",
-    {
-      description:
-        "Rate an Experience Engine recall entry so the brain keeps what helped and prunes the rest. Call after " +
-        "acting on an ee_query result — once per `[id col]` you used or judged. verdict: 'followed' (you changed " +
-        "your approach because of it), 'ignored' (topical but did not apply this time), 'noise' (wrong by category — " +
-        "REQUIRES reason: wrong_repo | wrong_language | wrong_task | stale_rule). id may be a short prefix; the " +
-        "server resolves it. Clears the entry from this session's pending-feedback gate.",
-      inputSchema: {
-        id: z.string().min(1).max(200),
-        collection: z.string().min(1).max(200),
-        verdict: z.enum(["followed", "ignored", "noise"]),
-        reason: z.enum(["wrong_repo", "wrong_language", "wrong_task", "stale_rule"]).optional(),
-      },
-    },
-    async ({ id, collection, verdict, reason }) => {
-      if (verdict === "noise" && !reason) {
-        return fail(
-          "reason_required",
-          "verdict 'noise' requires reason: wrong_repo | wrong_language | wrong_task | stale_rule",
-        );
-      }
-      const feedback =
-        deps.feedback ?? ((pid, col, v, r) => import("../ee/search.js").then((m) => m.feedbackEE(pid, col, v, r)));
-      try {
-        const result = await feedback(id, collection, verdict, reason);
-        if (!result.ok) {
-          return fail("feedback_failed", result.error ?? "feedback POST failed");
-        }
-        // Clear by the server-resolved full id AND the (possibly short) id the
-        // agent passed, so a prefix-based call still settles the ledger debt.
-        const clearedId = result.resolvedId ?? id;
-        ledger.clear(clearedId);
-        ledger.clear(id);
-        return ok({
-          ok: true,
-          id: clearedId,
-          verdict: result.verdict,
-          ...(result.reason ? { reason: result.reason } : {}),
-          pendingRemaining: ledger.pendingCount(),
-        });
-      } catch (e) {
-        return fail("feedback_failed", e instanceof Error ? e.message : String(e));
-      }
-    },
-  );
-
-  server.registerTool(
-    "ee_health",
-    { description: "Check Experience Engine server reachability.", inputSchema: {} },
-    async () => {
-      try {
-        return ok(await health());
-      } catch (e) {
-        return fail("ee_unavailable", e instanceof Error ? e.message : String(e));
-      }
-    },
-  );
-}

package/src/mcp/forensics-tools.ts

@@ -1,65 +0,0 @@
-/**
- * src/mcp/forensics-tools.ts
- *
- * usage.forensics MCP tool: per-session token-cost forensics by id prefix.
- * Read-only (local SQLite). Dependencies injected for unit testability.
- */
-
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { z } from "zod";
-import type { CostForensicsSummary } from "../cli/cost-forensics.js";
-
-export interface ForensicsToolDeps {
-  resolve?: (prefix: string) => string[] | Promise<string[]>;
-  collect?: (sessionId: string) => CostForensicsSummary | Promise<CostForensicsSummary>;
-}
-
-function ok(data: unknown) {
-  return { content: [{ type: "text" as const, text: JSON.stringify(data) }] };
-}
-function fail(error: string, message: string, extra?: Record<string, unknown>) {
-  return {
-    content: [{ type: "text" as const, text: JSON.stringify({ error, message, ...extra }) }],
-    isError: true,
-  };
-}
-
-async function defaultResolve(prefix: string): Promise<string[]> {
-  const { resolveSessionIds } = await import("../cli/cost-forensics.js");
-  return resolveSessionIds(prefix);
-}
-async function defaultCollect(sessionId: string): Promise<CostForensicsSummary> {
-  const { collectCostForensics } = await import("../cli/cost-forensics.js");
-  return collectCostForensics(sessionId);
-}
-
-export function registerForensicsTools(server: McpServer, deps: ForensicsToolDeps = {}): void {
-  const resolve = deps.resolve ?? defaultResolve;
-  const collect = deps.collect ?? defaultCollect;
-
-  server.registerTool(
-    "usage_forensics",
-    {
-      description:
-        "Per-session token-cost forensics by session-id prefix: peak input, cache-hit ratio, per-event breakdown.",
-      inputSchema: { prefix: z.string().min(1).max(100) },
-    },
-    async ({ prefix }) => {
-      let ids: string[];
-      try {
-        ids = await resolve(prefix);
-      } catch (e) {
-        return fail("db_error", e instanceof Error ? e.message : String(e));
-      }
-      if (ids.length === 0) return fail("not_found", `no session matches prefix '${prefix}'`);
-      if (ids.length > 1)
-        return fail("ambiguous", `prefix '${prefix}' matched ${ids.length} sessions`, { matches: ids });
-      try {
-        const summary = await collect(ids[0]!);
-        return ok(summary);
-      } catch (e) {
-        return fail("db_error", e instanceof Error ? e.message : String(e));
-      }
-    },
-  );
-}

package/src/mcp/lsp-tools.ts

@@ -1,78 +0,0 @@
-/**
- * src/mcp/lsp-tools.ts
- *
- * LSP code-intelligence MCP tool: a single lsp.query that mirrors the native
- * lsp tool surface (9 operations multiplexed through one query). Read-only.
- * Wraps queryLsp() (which caches one language-server manager per cwd).
- *
- * deps are injected so unit tests never spawn a real language server.
- */
-
-import type { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
-import { z } from "zod";
-import { LSP_TOOL_OPERATIONS, type LspQueryInput, type LspToolResponse } from "../lsp/types.js";
-
-export interface LspToolDeps {
-  query?: (cwd: string, input: LspQueryInput) => Promise<LspToolResponse>;
-  enabled?: (cwd: string) => boolean | Promise<boolean>;
-}
-
-function ok(data: unknown) {
-  return { content: [{ type: "text" as const, text: JSON.stringify(data) }] };
-}
-function fail(error: string, message: string) {
-  return {
-    content: [{ type: "text" as const, text: JSON.stringify({ error, message }) }],
-    isError: true,
-  };
-}
-
-async function defaultQuery(cwd: string, input: LspQueryInput): Promise<LspToolResponse> {
-  const { queryLsp } = await import("../lsp/runtime.js");
-  return queryLsp(cwd, input);
-}
-async function defaultEnabled(cwd: string): Promise<boolean> {
-  const { isLspToolEnabled } = await import("../lsp/runtime.js");
-  return isLspToolEnabled(cwd);
-}
-
-export function registerLspTools(server: McpServer, deps: LspToolDeps = {}): void {
-  const query = deps.query ?? defaultQuery;
-  const enabled = deps.enabled ?? defaultEnabled;
-
-  server.registerTool(
-    "lsp_query",
-    {
-      description:
-        "Semantic code intelligence via language servers. operation is one of: goToDefinition, findReferences, hover, documentSymbol, workspaceSymbol, goToImplementation, prepareCallHierarchy, incomingCalls, outgoingCalls. " +
-        "filePath: absolute, or relative to the workspace root (cwd). line/character: 1-based (line 1 = first line, like an editor or file:line reference) — required for position-based ops; omit for documentSymbol; use query (not position) for workspaceSymbol. " +
-        'Returns {success, output}: output is a pretty-printed JSON array of LSP results (each {uri, range} with LSP-native 0-based range positions), or "No results found." when empty. ' +
-        "A slow server that has not finished loading its workspace returns no results for that call (no hang) — retry once it has warmed up. Note: csharp-ls answers position ops (goToDefinition/hover/references) almost immediately but only returns documentSymbol after the project/solution has been restored and loaded.",
-      inputSchema: {
-        operation: z.enum(LSP_TOOL_OPERATIONS),
-        filePath: z.string().min(1).max(1000),
-        line: z.number().int().min(0).optional(),
-        character: z.number().int().min(0).optional(),
-        query: z.string().max(1000).optional(),
-      },
-    },
-    async (args) => {
-      const cwd = process.cwd();
-      let isEnabled: boolean;
-      try {
-        isEnabled = await enabled(cwd);
-      } catch (e) {
-        return fail("lsp_error", e instanceof Error ? e.message : String(e));
-      }
-      if (!isEnabled) {
-        return fail("lsp_disabled", "LSP tool is disabled in settings (lsp.enabled / lsp.tool)");
-      }
-      try {
-        const resp = await query(cwd, args as LspQueryInput);
-        return ok(resp);
-      } catch (e) {
-        return fail("lsp_error", e instanceof Error ? e.message : String(e));
-      }
-    },
-  );
-}