@nifrajs/cli 2.13.0 → 3.1.0

package/dist/mcp.js

@@ -40,16 +40,15 @@ import { basename, isAbsolute, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import { reflectRoutes } from "@nifrajs/core/reflection";
 import { Glob } from "bun";
+import { commandCatalog, commandMcpName, findCommandSpec, } from "./command-catalog.js";
 import { collectContractProof } from "./contract-proof.js";
 import { loadDocsCorpus } from "./docs-search.js";
 import { loadExamplesCorpus } from "./examples.js";
-import { describeProject } from "./introspect.js";
 import { detectMonorepo, loadMonorepoApps } from "./load.js";
 import { docsTools } from "./mcp-docs-tools.js";
 import { createMcpProtocolState, handleRpc, rpcError, } from "./mcp-protocol.js";
 import { applyClientRoots, detectToolingDrift, driftNote, pathsFromRootsResult, resolveRootState, rootInstructions, rootVerdict, } from "./mcp-root.js";
 import { loadTypesCorpus } from "./types-search.js";
-import { collectProjectWorkGraph } from "./work-graph.js";
 /** Path to a sibling child entry (`mcp-run` / `mcp-render` / `mcp-ws`), resolved next to this module (`.ts` in
  * dev, `.js` once built). Each runs in a FRESH subprocess per call so the project's current code loads. */
 function childPath(name) {
@@ -69,6 +68,8 @@ function childPath(name) {
  * within one attention span rather than after minutes of polling.
  */
 const CHILD_TIMEOUT_MS = 30_000;
+/** Maximum stdout or stderr retained from a project subprocess. The process is killed at the limit. */
+export const CHILD_OUTPUT_MAX_BYTES = 1_048_576;
 /** Local dev-tool reads are intentionally bounded: MCP runs in an agent process and must not hang on or
  * buffer an unrelated loopback service just because a caller supplied its port. */
 export const LOCAL_TOOL_FETCH_TIMEOUT_MS = 2_000;
@@ -126,6 +127,39 @@ const timeoutMessage = (label, ms) => `${label} timed out after ${ms / 1000}s an
     `The app was loaded but the process did not finish. Most often this is a module-level side effect ` +
     `that keeps the event loop alive (a database pool, a Redis client, an interval) opened during import ` +
     `rather than lazily. Check for top-level connections in the app entry or anything it imports.`;
+/** Read child output incrementally and cancel as soon as the byte budget is crossed. */
+export async function readBoundedStream(stream, maxBytes = CHILD_OUTPUT_MAX_BYTES, onLimit) {
+    const reader = stream.getReader();
+    const decoder = new TextDecoder();
+    const chunks = [];
+    let totalBytes = 0;
+    try {
+        for (;;) {
+            const { done, value } = await reader.read();
+            if (done) {
+                chunks.push(decoder.decode());
+                return { text: chunks.join(""), truncated: false };
+            }
+            if (value === undefined)
+                continue;
+            const remaining = maxBytes - totalBytes;
+            if (remaining <= 0 || value.byteLength > remaining) {
+                if (remaining > 0) {
+                    chunks.push(decoder.decode(value.subarray(0, remaining), { stream: true }));
+                    chunks.push(decoder.decode());
+                }
+                await reader.cancel();
+                onLimit?.();
+                return { text: chunks.join(""), truncated: true };
+            }
+            totalBytes += value.byteLength;
+            chunks.push(decoder.decode(value, { stream: true }));
+        }
+    }
+    finally {
+        reader.releaseLock();
+    }
+}
 async function spawnChild(child, cwd, input, label, signal) {
     if (signal?.aborted)
         return `${label} cancelled before it started.`;
@@ -134,31 +168,48 @@ async function spawnChild(child, cwd, input, label, signal) {
         stdout: "pipe",
         stderr: "pipe",
     });
-    const abort = () => proc.kill();
+    let killed = false;
+    const kill = () => {
+        if (killed)
+            return;
+        killed = true;
+        proc.kill();
+    };
+    const abort = () => kill();
     signal?.addEventListener("abort", abort, { once: true });
     let timedOut = false;
     const timer = setTimeout(() => {
         timedOut = true;
-        proc.kill();
+        kill();
     }, CHILD_TIMEOUT_MS);
+    let outputExceeded = false;
     try {
         proc.stdin.write(JSON.stringify(input));
         await proc.stdin.end();
         const [out, err] = await Promise.all([
-            new Response(proc.stdout).text(),
-            new Response(proc.stderr).text(),
+            readBoundedStream(proc.stdout, CHILD_OUTPUT_MAX_BYTES, () => {
+                outputExceeded = true;
+                kill();
+            }),
+            readBoundedStream(proc.stderr, CHILD_OUTPUT_MAX_BYTES, () => {
+                outputExceeded = true;
+                kill();
+            }),
         ]);
         await proc.exited;
         if (signal?.aborted) {
             const reason = typeof signal.reason === "string" ? `: ${signal.reason}` : "";
             return `${label} cancelled${reason}.`;
         }
+        if (outputExceeded) {
+            return `${label} output exceeded ${CHILD_OUTPUT_MAX_BYTES} bytes and was terminated.`;
+        }
         // A killed child may still have flushed a complete result before the timer fired; prefer real
         // output over the timeout message so a slow-but-successful render is not reported as a failure.
-        const text = out.trim();
+        const text = out.text.trim();
         if (timedOut && text === "")
             return timeoutMessage(label, CHILD_TIMEOUT_MS);
-        return text || `${label} failed:\n${err.trim() || "(no output)"}`;
+        return text || `${label} failed:\n${err.text.trim() || "(no output)"}`;
     }
     finally {
         clearTimeout(timer);
@@ -280,7 +331,17 @@ export class WarmWorker {
             const { done, value } = await reader.read();
             if (done)
                 break;
+            if (value !== undefined && value.byteLength > CHILD_OUTPUT_MAX_BYTES) {
+                this.stderrBuffer = boundedAppend(this.stderrBuffer, `warm ${this.label} worker stdout exceeded ${CHILD_OUTPUT_MAX_BYTES} bytes\n`);
+                this.stop();
+                return;
+            }
             this.stdoutBuffer += decoder.decode(value, { stream: true });
+            if (this.stdoutBuffer.length > CHILD_OUTPUT_MAX_BYTES) {
+                this.stderrBuffer = boundedAppend(this.stderrBuffer, `warm ${this.label} worker stdout exceeded ${CHILD_OUTPUT_MAX_BYTES} bytes\n`);
+                this.stop();
+                return;
+            }
             let nl = this.stdoutBuffer.indexOf("\n");
             while (nl !== -1) {
                 const line = this.stdoutBuffer.slice(0, nl).trim();
@@ -593,11 +654,66 @@ export function resolveProjectDir(root, dir) {
 function dirError(dir) {
     return JSON.stringify({ ok: false, error: `dir must be a subdirectory of the project root - "${dir}" escapes it.` }, null, 2);
 }
-/** Build the project-scoped tools for `cwd`. */
+/** Adapt one executable catalog spec to its MCP descriptor and project-scoped handler. */
+export function toMcpTool(source, options) {
+    const spec = "run" in source ? source : findCommandSpec(source.name);
+    if (spec === undefined)
+        throw new Error(`missing command spec for ${source.name}`);
+    const entry = "run" in source ? undefined : source;
+    const { cwd } = options;
+    const loadAppCached = options.loadAppCached ?? createCachedAppLoader(cwd);
+    return {
+        name: commandMcpName(entry?.name ?? spec.name),
+        description: entry?.summary ?? spec.summary,
+        inputSchema: entry?.inputSchema ?? spec.input.jsonSchema,
+        handler: async (args, context) => {
+            const raw = { ...args };
+            const dir = raw.dir;
+            if (dir !== undefined && typeof dir !== "string")
+                return dirError(undefined);
+            const target = resolveProjectDir(cwd, dir);
+            if (target === null)
+                return dirError(dir);
+            if (typeof raw.config === "string") {
+                const config = resolve(target, raw.config);
+                if (config !== target && !config.startsWith(`${target}${sep}`))
+                    return JSON.stringify({ ok: false, error: "config must stay inside the selected project directory" }, null, 2);
+            }
+            delete raw.dir;
+            let input;
+            try {
+                input = spec.input.parse(raw);
+            }
+            catch (error) {
+                return JSON.stringify({ ok: false, error: error instanceof Error ? error.message : String(error) }, null, 2);
+            }
+            const commandContext = {
+                cwd: target,
+                signal: context.signal,
+                progress: () => context.reportProgress?.(0.5, 1),
+                ...(target === cwd ? { loadApp: loadAppCached } : {}),
+            };
+            const output = await spec.run(input, commandContext);
+            const value = spec.json?.(output, input) ?? output;
+            return typeof value === "string" ? value : JSON.stringify(value, null, 2);
+        },
+    };
+}
+export function catalogProjectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
+    return commandCatalog
+        .filter((entry) => entry.transports.includes("mcp"))
+        .map((entry) => {
+        const spec = findCommandSpec(entry.name);
+        if (spec === undefined)
+            throw new Error(`missing command spec for ${entry.name}`);
+        return toMcpTool(spec, { cwd, loadAppCached });
+    });
+}
 export function projectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
     const warmRun = createWarmHandler("mcp-run", cwd, "run");
     const warmRender = createWarmHandler("mcp-render", cwd, "render");
-    return [
+    const catalogTools = catalogProjectTools(cwd, () => loadAppCached());
+    const legacyTools = [
         {
             name: "nifra_contract_proof",
             description: "Compare the current route contract with its snapshot and join each changed route to its route-assurance and capability evidence. Returns hasBreaking as the gate. The typed-contract check is lazy and runs only when check is true.",
@@ -646,75 +762,6 @@ export function projectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
                 }
             },
         },
-        {
-            name: "nifra_prove",
-            description: "Build the static verification work graph for this project. Pass changed files to get only impacted routes and the cheapest proof plan. The result includes a serializable evidence bundle and a machine-checkable stop condition. Requires a fresh build and never probes a running application.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    files: {
-                        type: "array",
-                        items: { type: "string" },
-                        description: "Project-relative changed files. Omit for a graph overview.",
-                    },
-                    minLevel: {
-                        type: "integer",
-                        minimum: 0,
-                        maximum: 4,
-                        description: "Required proof level for the stop condition. Defaults to 1.",
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const files = args.files;
-                const minLevel = args.minLevel;
-                if ((files !== undefined &&
-                    (!Array.isArray(files) || files.some((file) => typeof file !== "string"))) ||
-                    (minLevel !== undefined &&
-                        (typeof minLevel !== "number" ||
-                            !Number.isInteger(minLevel) ||
-                            minLevel < 0 ||
-                            minLevel > 4))) {
-                    return JSON.stringify({
-                        ok: false,
-                        error: "files must be strings and minLevel must be an integer from 0 to 4",
-                    }, null, 2);
-                }
-                try {
-                    return JSON.stringify(await collectProjectWorkGraph(cwd, {
-                        ...(files === undefined ? {} : { changedFiles: files }),
-                        ...(minLevel === undefined ? {} : { minLevel }),
-                    }), null, 2);
-                }
-                catch (error) {
-                    return JSON.stringify({ ok: false, error: error instanceof Error ? error.message : "work graph failed" }, null, 2);
-                }
-            },
-        },
-        {
-            name: "nifra_context",
-            description: "Get this nifra project's surface. Call it once UNFILTERED for a tight INDEX: the route list (API routes as `METHOD path`, page routes as `pattern → file`) + framework conventions + a pointer - cheap even on a big app, no per-route schema dump. Then pass `path` (a route prefix like /api/orders) and/or `kind` (api|pages) to fetch the FULL contract for that slice (body/query/response TS shapes + the exact typed-client call form).",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    path: {
-                        type: "string",
-                        description: "Only routes whose path/pattern starts with this prefix.",
-                    },
-                    kind: {
-                        type: "string",
-                        enum: ["api", "pages"],
-                        description: "Limit to API routes or page routes.",
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const filter = args;
-                return describeProject(await loadAppCached(), filter);
-            },
-        },
         {
             name: "nifra_explain",
             description: "Turn a nifra error into a STRUCTURED diagnostic instead of eyeballing a stack trace: a stable `code`, the top frame in YOUR source, a codeframe around the offending line, and - when nifra recognises the failure - the plain-language `cause` + `fix` + docs anchor. Pass `error` (and `stack` if you have it, e.g. from nifra_run/nifra_test output or a failing build) to explain a specific failure; or pass `port` to fetch the running dev server's most recent SSR failure from `/__nifra/last-error`. Returns the same JSON the dev overlay renders.",
@@ -844,25 +891,6 @@ export function projectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
                 }
             },
         },
-        {
-            name: "nifra_routes",
-            description: "List this project's API routes as STRUCTURED JSON - each `{ method, path, call, body?, query?, response? }`, where `call` is the exact typed-client call form and the shapes are compact TS-typed contracts. For programmatic use (list_routes / get_route_schema) instead of parsing the nifra_context Markdown. No args = every route; pass `path` (a path or prefix like /api/orders) to narrow to those routes.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    path: {
-                        type: "string",
-                        description: "Only routes whose path starts with this prefix (omit for all routes).",
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const { routesToJson } = await import("./introspect.js");
-                const { path } = args;
-                return JSON.stringify(routesToJson(await loadAppCached(), path), null, 2);
-            },
-        },
         {
             name: "nifra_openapi",
             description: 'Return this project\'s backend OpenAPI 3.1 document generated from backend.ts route schemas via @nifrajs/schema. Use `format:"json"` for machine edits (default) or `format:"yaml"` for humans. Pass `path` (a route prefix like /api/orders, mirroring nifra_routes) to narrow a large backend to operations under that prefix instead of the whole document. Frontend-only apps return a valid empty paths object.',
@@ -1043,180 +1071,6 @@ export function projectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
                 return `${status}\n\n${renderScaffold(path, framework)}`;
             },
         },
-        {
-            name: "nifra_check",
-            description: 'Run the project\'s drift gate and return a structured result { ok, typecheck, diagnostics[], pipeline? }: typecheck (the frontend↔backend contract), plus lints for hand-rolled fetch() to your own API, untyped client("…") calls missing <typeof app>, and server-only imports in routes/. `pipeline` answers "which bundler does this app run on" (bun|vite, for dev AND build alike) without starting a server, and its `pipeline` diagnostics catch the hazards of having two: a plugin in the slot the other bundler reads (accepted, then never called), a dev toolchain imported by the file `nifra build` bundles into the production server, and `conditions` that cannot reach Bun\'s dev client bundle. Read it before adding a plugin or a compiler. Pass lintsOnly:true for a near-instant lint pass while iterating; run the full gate (default) to confirm the work is done - fix every diagnostic before finishing.',
-            inputSchema: {
-                type: "object",
-                properties: {
-                    lintsOnly: {
-                        type: "boolean",
-                        description: "Skip tsc; run only the near-instant source lints (inner-loop mode).",
-                    },
-                    dir: {
-                        type: "string",
-                        description: 'Run the check in this subdirectory (relative to the project root), e.g. "app" or "packages/api". Use it when the MCP server\'s root is a monorepo but you want to check just one app. Default: the project root.',
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args, context) => {
-                const { collectCheckResult } = await import("./check.js");
-                const opts = args;
-                const target = resolveProjectDir(cwd, opts.dir);
-                if (target === null)
-                    return dirError(opts.dir);
-                const result = await collectCheckResult(target, {
-                    lintsOnly: opts.lintsOnly ?? false,
-                    signal: context.signal,
-                    // Bound the result so a large project can't emit an MCP message big enough to break the stdio
-                    // transport (`-32000: Connection closed`). If `truncated` comes back, fix the shown diagnostics
-                    // and re-run. The scan already skips gitignored trees (walkSource), so this is the safety net.
-                    maxDiagnostics: 100,
-                });
-                return JSON.stringify(result.structuredDiagnostics === undefined
-                    ? result
-                    : { ...result, diagnostics: result.structuredDiagnostics }, null, 2);
-            },
-        },
-        {
-            name: "nifra_fix",
-            description: "Automatically fix diagnostic lints (such as rewriting hand-rolled fetch() calls to the typed nifra client, adding generic types to client factory calls, and resolving dependency drift in package.json). Runs diagnostics, applies all mechanical edit suggestions, applies doctor dependency fixes, and returns the remaining unresolved diagnostics.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    code: {
-                        type: "string",
-                        description: "Apply the registered recipe for one stable diagnostic code when available.",
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args, context) => {
-                const { collectCheckResult } = await import("./check.js");
-                const { applyDiagnosticRecipe } = await import("./fix-recipes.js");
-                const { applyDoctorAutoFix } = await import("./doctor.js");
-                const { resolveInsideProject } = await import("./project-path.js");
-                const { writeFile, readFile } = await import("node:fs/promises");
-                let doctorResult = null;
-                try {
-                    doctorResult = await applyDoctorAutoFix(cwd);
-                }
-                catch {
-                    // ignore doctor errors
-                }
-                const checkResult = await collectCheckResult(cwd, {
-                    lintsOnly: true,
-                    signal: context.signal,
-                    maxDiagnostics: 100,
-                });
-                const requestedCode = args.code;
-                const recipeFixed = [];
-                if (typeof requestedCode === "string") {
-                    for (const diagnostic of checkResult.structuredDiagnostics ?? []) {
-                        if (diagnostic.code !== requestedCode)
-                            continue;
-                        recipeFixed.push(...(await applyDiagnosticRecipe(cwd, diagnostic)));
-                    }
-                }
-                const fixed = [];
-                for (const diag of checkResult.diagnostics) {
-                    if (diag.file && diag.line && diag.suggestion?.kind === "edit" && diag.suggestion.diff) {
-                        try {
-                            const diffLines = diag.suggestion.diff.split("\n");
-                            const beforeLine = diffLines.find((l) => l.startsWith("-"))?.slice(1);
-                            const afterLine = diffLines.find((l) => l.startsWith("+"))?.slice(1);
-                            if (beforeLine !== undefined && afterLine !== undefined) {
-                                const filePath = await resolveInsideProject(cwd, diag.file);
-                                if (filePath === undefined)
-                                    continue;
-                                const content = await readFile(filePath, "utf-8");
-                                const lines = content.split("\n");
-                                const idx = diag.line - 1;
-                                if (lines[idx] === beforeLine) {
-                                    lines[idx] = afterLine;
-                                    await writeFile(filePath, lines.join("\n"), "utf-8");
-                                    fixed.push({
-                                        file: diag.file,
-                                        line: diag.line,
-                                        title: diag.suggestion.title,
-                                    });
-                                }
-                            }
-                        }
-                        catch {
-                            // ignore edit errors
-                        }
-                    }
-                }
-                const finalResult = await collectCheckResult(cwd, {
-                    lintsOnly: false,
-                    signal: context.signal,
-                    maxDiagnostics: 100,
-                });
-                return JSON.stringify({
-                    ok: finalResult.ok,
-                    fixed,
-                    recipeFixed,
-                    doctorFixed: doctorResult?.fixed ?? [],
-                    remainingDiagnostics: finalResult.diagnostics,
-                }, null, 2);
-            },
-        },
-        {
-            name: "nifra_assure",
-            description: "Evaluate nifra.assurance.ts and return the complete route-assurance report: every reflected route's first matching policy rule, enforcement evidence, missing/forbidden evidence, and the fail-closed ok bit. Use after adding or changing routes/security middleware; fix every finding before finishing.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    config: {
-                        type: "string",
-                        description: "Config path relative to the selected project directory. Default: nifra.assurance.ts.",
-                    },
-                    bundle: {
-                        type: "boolean",
-                        description: "Return the single gate bundle with verdict and explicit skipped gates.",
-                    },
-                    strict: {
-                        type: "boolean",
-                        description: "Treat warning diagnostics as failures in the bundle.",
-                    },
-                    hydration: {
-                        type: "boolean",
-                        description: "Run the hydration assurance gate.",
-                    },
-                    interact: {
-                        type: "boolean",
-                        description: "Run declared hydration probes after the client mounts.",
-                    },
-                    dir: {
-                        type: "string",
-                        description: 'Evaluate this project subdirectory (relative to the MCP root), e.g. "apps/api". Default: the project root.',
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const opts = args;
-                const target = resolveProjectDir(cwd, opts.dir);
-                if (target === null)
-                    return dirError(opts.dir);
-                const config = opts.config === undefined ? undefined : resolve(target, opts.config);
-                if (config !== undefined && config !== target && !config.startsWith(target + sep)) {
-                    return JSON.stringify({ ok: false, error: "config must stay inside the selected project directory" }, null, 2);
-                }
-                const { collectAssuranceReport, collectAssureBundle } = await import("./assure.js");
-                if (opts.bundle === true || opts.strict === true || opts.hydration === true) {
-                    return JSON.stringify(await collectAssureBundle(target, {
-                        ...(config === undefined ? {} : { config }),
-                        ...(opts.strict === undefined ? {} : { strict: opts.strict }),
-                        ...(opts.hydration === undefined ? {} : { hydration: opts.hydration }),
-                        ...(opts.interact === undefined ? {} : { interact: opts.interact }),
-                    }), null, 2);
-                }
-                return JSON.stringify(await collectAssuranceReport(target, config), null, 2);
-            },
-        },
         {
             name: "nifra_hydrate",
             description: "Run the hydration assurance gate and return stable NF-H diagnostics.",
@@ -1237,90 +1091,8 @@ export function projectTools(cwd, loadAppCached = createCachedAppLoader(cwd)) {
                 return JSON.stringify(await runHydrationAssurance(target, { interact: opts.interact === true }), null, 2);
             },
         },
-        {
-            name: "nifra_replay",
-            description: "Validate and dispatch a token-only replay metadata file.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    file: { type: "string", description: "Replay file relative to the project root." },
-                },
-                required: ["file"],
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const file = args.file;
-                if (typeof file !== "string" || file.trim() === "")
-                    return JSON.stringify({ ok: false, error: "file is required" });
-                const { runReplay } = await import("./replay.js");
-                try {
-                    return JSON.stringify(await runReplay(cwd, file), null, 2);
-                }
-                catch (error) {
-                    return JSON.stringify({ ok: false, error: error instanceof Error ? error.message : String(error) }, null, 2);
-                }
-            },
-        },
-        {
-            name: "nifra_levels",
-            description: "Compute this project's verification ladder and return { achieved, levels[] }, where each level carries { level, name, ok, reasons[] }. The ladder is cumulative: L0 typed contract (nifra check), L1 route assurance, L2 reviewed capability lockfile, L3 route trust manifest in sync, L4 contract invariants, so a level only holds when every level below it holds, and `achieved` is -1 when even L0 fails. This is the single gate that reports what a change actually proves: run it after finishing work, and treat any `reasons` on a level the project already claimed as a regression to fix.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    config: {
-                        type: "string",
-                        description: "Assurance config path relative to the selected project directory. Default: nifra.assurance.ts.",
-                    },
-                    seed: {
-                        type: "number",
-                        description: "Deterministic seed for the L4 invariant run. Default 1.",
-                    },
-                    dir: {
-                        type: "string",
-                        description: 'Evaluate this project subdirectory (relative to the MCP root), e.g. "apps/api". Default: the project root.',
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const opts = args;
-                const target = resolveProjectDir(cwd, opts.dir);
-                if (target === null)
-                    return dirError(opts.dir);
-                if (opts.seed !== undefined && !Number.isSafeInteger(opts.seed)) {
-                    return JSON.stringify({ ok: false, error: "seed must be a safe integer" }, null, 2);
-                }
-                const config = opts.config === undefined ? undefined : resolve(target, opts.config);
-                if (config !== undefined && config !== target && !config.startsWith(target + sep)) {
-                    return JSON.stringify({ ok: false, error: "config must stay inside the selected project directory" }, null, 2);
-                }
-                const { collectVerificationLevels } = await import("./levels-tool.js");
-                return JSON.stringify(await collectVerificationLevels(target, {
-                    ...(opts.config === undefined ? {} : { config: opts.config }),
-                    ...(opts.seed === undefined ? {} : { seed: opts.seed }),
-                }), null, 2);
-            },
-        },
-        {
-            name: "nifra_doctor",
-            description: "Check this project for packages imported in source but NOT declared in package.json - the Bun-workspace trap where an import resolves at runtime (hoisting/workspace) so tests pass and `bun install` says no changes, yet tsc fails and a fresh/standalone install can't resolve it. Also reports `pipeline`: which bundler (bun|vite) this app's dev AND build phases run on and why, plus config hazards that only exist because there are two - a plugin sitting in the slot the other bundler reads (accepted, then never called), a dev toolchain imported by the file `nifra build` bundles into the production server (builds clean, dies at startup), and `conditions` that cannot reach the client bundle Bun's dev server serves. Returns { ok, ran, findings[], pipeline?, fixed?, skippedFixes? }. Pass autoFix:true to update package.json only when the dependency version can be inferred locally from an ancestor package.json or installed package metadata; otherwise the tool returns the exact bun add command to run.",
-            inputSchema: {
-                type: "object",
-                properties: {
-                    autoFix: {
-                        type: "boolean",
-                        description: "When true, write safe package.json fixes using only locally inferred versions. Does not run install or use the network.",
-                    },
-                },
-                additionalProperties: false,
-            },
-            handler: async (args) => {
-                const opts = args;
-                const { applyDoctorAutoFix, collectDoctorResult } = await import("./doctor.js");
-                return JSON.stringify(opts.autoFix === true ? await applyDoctorAutoFix(cwd) : await collectDoctorResult(cwd), null, 2);
-            },
-        },
     ];
+    return [...catalogTools, ...legacyTools];
 }
 /** Extract tools registered via .tool() routes on the Nifra backend. Exported for the test that proves a
  * `server().tool()` route surfaces in `tools/list` and executes through `tools/call`. */