@yawlabs/mcph 0.47.2 → 0.47.3

package/README.md

@@ -302,10 +302,6 @@ When both exist, the project guide is appended after the user guide with a `---`
 
 When a server fails to start with stderr like `GITHUB_TOKEN is required` and your client advertises the MCP [elicitation](https://modelcontextprotocol.io/specification/server/elicitation) capability, mcph prompts you for the missing value inline and retries the load. Values stay in-memory for the current mcph session only — persist them in the mcp.hosting dashboard if you want them across restarts.
 
-### Test from the dashboard
-
-The `/dashboard/connect` page in mcp.hosting has a **Test** button per server that loads it through your running mcph and shows pass/fail inline — no LLM round-trip needed. Useful when you've just added a server and want to confirm the token works without prompting your AI.
-
 ### Errors come with deep-links
 
 When a load fails (missing token, runtime not on PATH, server crashes on init), mcph emits a message ending with `→ Edit at https://mcp.hosting/dashboard/connect#server-<id>`. Most LLMs render that as a clickable link, and the dashboard scrolls to and highlights the matching card so you find the right server in one click.

package/dist/index.js

@@ -292,11 +292,11 @@ async function readConfigAt(path5, scope, warnings) {
       `${path5}: schema version ${version} is newer than this mcph (${CURRENT_SCHEMA_VERSION}); upgrade with \`npm i -g @yawlabs/mcph@latest\`. Loading best-effort.`
     );
   }
-  const token6 = typeof obj.token === "string" && obj.token.length > 0 ? obj.token : void 0;
+  const token5 = typeof obj.token === "string" && obj.token.length > 0 ? obj.token : void 0;
   const apiBase = typeof obj.apiBase === "string" && obj.apiBase.length > 0 ? obj.apiBase : void 0;
   const servers = Array.isArray(obj.servers) ? obj.servers.filter((v) => typeof v === "string") : void 0;
   const blocked = Array.isArray(obj.blocked) ? obj.blocked.filter((v) => typeof v === "string") : void 0;
-  if (token6) {
+  if (token5) {
     if (scope === "project") {
       warnings.push(
         `${path5}: 'token' should not appear in a project-shared file. Move it to ${CONFIG_DIRNAME}/${LOCAL_CONFIG_FILENAME} (gitignored) or ~/${CONFIG_DIRNAME}/${CONFIG_FILENAME}.`
@@ -304,7 +304,7 @@ async function readConfigAt(path5, scope, warnings) {
     }
     await checkPermissions(path5, warnings);
   }
-  return { path: path5, scope, version, token: token6, apiBase, servers, blocked };
+  return { path: path5, scope, version, token: token5, apiBase, servers, blocked };
 }
 async function checkPermissions(path5, warnings) {
   if (process.platform === "win32") return;
@@ -361,16 +361,16 @@ async function loadMcphConfig(opts = {}) {
   if (project) loadedFiles.push(project);
   const global = await readConfigAt(globalPath, "global", warnings);
   if (global) loadedFiles.push(global);
-  let token6 = null;
+  let token5 = null;
   let tokenSource = "missing";
   if (typeof env.MCPH_TOKEN === "string" && env.MCPH_TOKEN.length > 0) {
-    token6 = env.MCPH_TOKEN;
+    token5 = env.MCPH_TOKEN;
     tokenSource = "env";
   } else if (local?.token) {
-    token6 = local.token;
+    token5 = local.token;
     tokenSource = "local";
   } else if (global?.token) {
-    token6 = global.token;
+    token5 = global.token;
     tokenSource = "global";
   }
   let apiBase = DEFAULT_API_BASE;
@@ -389,7 +389,7 @@ async function loadMcphConfig(opts = {}) {
     apiBaseSource = "global";
   }
   return {
-    token: token6,
+    token: token5,
     tokenSource,
     apiBase,
     apiBaseSource,
@@ -400,10 +400,10 @@ async function loadMcphConfig(opts = {}) {
     warnings
   };
 }
-function tokenFingerprint(token6) {
-  if (!token6) return "(none)";
-  if (token6.length <= 8) return `***${token6.slice(-2)}`;
-  return `${token6.slice(0, 8)}\u2026${token6.slice(-4)}`;
+function tokenFingerprint(token5) {
+  if (!token5) return "(none)";
+  if (token5.length <= 8) return `***${token5.slice(-2)}`;
+  return `${token5.slice(0, 8)}\u2026${token5.slice(-4)}`;
 }
 function toProfile(config) {
   if (config.servers === void 0 && config.blocked === void 0) return null;
@@ -442,10 +442,10 @@ function profileAllows(profile, namespace) {
 
 // src/config.ts
 import { request } from "undici";
-async function fetchConfig(apiUrl6, token6, currentVersion) {
-  const url = `${apiUrl6.replace(/\/$/, "")}/api/connect/config`;
+async function fetchConfig(apiUrl5, token5, currentVersion) {
+  const url = `${apiUrl5.replace(/\/$/, "")}/api/connect/config`;
   const headers = {
-    Authorization: `Bearer ${token6}`,
+    Authorization: `Bearer ${token5}`,
     Accept: "application/json"
   };
   if (currentVersion) {
@@ -466,7 +466,7 @@ async function fetchConfig(apiUrl6, token6, currentVersion) {
     await res.body.text().catch(() => {
     });
     throw new ConfigError(
-      `Token rejected (HTTP 401) \u2014 the token ${tokenFingerprint(token6)} is invalid or revoked.
+      `Token rejected (HTTP 401) \u2014 the token ${tokenFingerprint(token5)} is invalid or revoked.
   Generate a new token at https://mcp.hosting/dashboard/settings/tokens,
   then re-run \`mcph install <client> --token mcp_pat_...\` or set MCPH_TOKEN.`,
       true
@@ -476,7 +476,7 @@ async function fetchConfig(apiUrl6, token6, currentVersion) {
     await res.body.text().catch(() => {
     });
     throw new ConfigError(
-      `Access denied (HTTP 403) \u2014 the token ${tokenFingerprint(token6)} was accepted but lacks permission to read this account's servers.
+      `Access denied (HTTP 403) \u2014 the token ${tokenFingerprint(token5)} was accepted but lacks permission to read this account's servers.
   The account may be suspended or the token scope reduced \u2014 check
   https://mcp.hosting/dashboard/settings/tokens, or reach support@mcp.hosting.`,
       true
@@ -900,12 +900,12 @@ async function runComplianceCommand(argv) {
     );
     return 1;
   }
-  const apiUrl6 = process.env.MCPH_URL ?? "https://mcp.hosting";
+  const apiUrl5 = process.env.MCPH_URL ?? "https://mcp.hosting";
   const report = await runTest(args);
   if (!report) return 1;
   printSummary(report);
   if (publish) {
-    const result = await publishReport(apiUrl6, report);
+    const result = await publishReport(apiUrl5, report);
     if (!result) return 1;
     process.stdout.write(`
 Published: ${result.reportUrl}
@@ -965,9 +965,9 @@ Target: ${url}
 `
   );
 }
-async function publishReport(apiUrl6, report) {
+async function publishReport(apiUrl5, report) {
   try {
-    const res = await request2(`${apiUrl6.replace(/\/$/, "")}/api/compliance/ext`, {
+    const res = await request2(`${apiUrl5.replace(/\/$/, "")}/api/compliance/ext`, {
       method: "POST",
       headers: { "Content-Type": "application/json" },
       body: JSON.stringify(report)
@@ -1650,7 +1650,7 @@ function selectFlakyNamespaces(entries, limit) {
 }
 
 // src/doctor-cmd.ts
-var VERSION = true ? "0.47.2" : "dev";
+var VERSION = true ? "0.47.3" : "dev";
 async function runDoctor(opts = {}) {
   if (opts.json) return runDoctorJson(opts);
   const lines = [];
@@ -2348,12 +2348,12 @@ ${USAGE}`);
   }
   log2(`Target: ${target.label} (${scope})`);
   log2(`File:   ${resolved.absolute}`);
-  let token6 = opts.token ?? null;
-  if (!token6) {
+  let token5 = opts.token ?? null;
+  if (!token5) {
     const cfg = await loadMcphConfig({ home: opts.home, cwd: process.cwd(), env: {} });
-    token6 = cfg.token;
+    token5 = cfg.token;
   }
-  if (!token6) {
+  if (!token5) {
     err(
       "\nmcph install: no token available.\n  Pass one with --token mcp_pat_\u2026, or run `mcph install` with --token once to seed ~/.mcph/config.json,\n  or create the token at https://mcp.hosting \u2192 Settings \u2192 API Tokens."
     );
@@ -2423,7 +2423,7 @@ ${USAGE}`);
   const writeMcphConfig = !opts.skipMcphConfig;
   const home = opts.home ?? homedir5();
   const mcphConfigPath = join5(home, CONFIG_DIRNAME, CONFIG_FILENAME);
-  const mcphConfigComposed = await composeMcphConfig(mcphConfigPath, token6);
+  const mcphConfigComposed = await composeMcphConfig(mcphConfigPath, token5);
   if (mcphConfigComposed.backupPath) {
     log2(
       `mcph install: existing ${mcphConfigPath} was malformed; original bytes backed up to ${mcphConfigComposed.backupPath} before overwriting.`
@@ -2579,7 +2579,7 @@ function mergeClientConfig(existing, containerPath, entry) {
   parent[leafKey] = container;
   return out;
 }
-async function composeMcphConfig(path5, token6) {
+async function composeMcphConfig(path5, token5) {
   let existing = {};
   let backupPath;
   if (existsSync2(path5)) {
@@ -2606,7 +2606,7 @@ async function composeMcphConfig(path5, token6) {
     }
   }
   const next = { version: CURRENT_SCHEMA_VERSION, ...existing };
-  next.token = token6;
+  next.token = token5;
   if (typeof next.version !== "number") next.version = CURRENT_SCHEMA_VERSION;
   return { json: `${JSON.stringify(next, null, 2)}
 `, backupPath };
@@ -2903,7 +2903,7 @@ import {
   ListToolsRequestSchema,
   ReadResourceRequestSchema
 } from "@modelcontextprotocol/sdk/types.js";
-import { request as request9 } from "undici";
+import { request as request8 } from "undici";
 
 // src/compliance.ts
 var GRADE_ORDER = {
@@ -3944,9 +3944,9 @@ var PackDetector = class {
 
 // src/progress.ts
 function createProgressReporter(extra) {
-  const token6 = extra?._meta?.progressToken;
+  const token5 = extra?._meta?.progressToken;
   const send = extra?.sendNotification;
-  if (token6 === void 0 || token6 === null || !send) {
+  if (token5 === void 0 || token5 === null || !send) {
     return () => {
     };
   }
@@ -3954,7 +3954,7 @@ function createProgressReporter(extra) {
   return (message, progress, total) => {
     step += 1;
     const params = {
-      progressToken: token6,
+      progressToken: token5,
       progress: progress ?? step,
       message
     };
@@ -4727,9 +4727,6 @@ function evaluateServerCap(namespace, loaded, cap) {
   };
 }
 
-// src/test-runner.ts
-import { request as request8 } from "undici";
-
 // src/upstream.ts
 import { Client } from "@modelcontextprotocol/sdk/client/index.js";
 import { SSEClientTransport } from "@modelcontextprotocol/sdk/client/sse.js";
@@ -4974,7 +4971,7 @@ function categorizeSpawnError(err) {
 }
 async function connectToUpstream(config, onDisconnect, onListChanged) {
   const client = new Client(
-    { name: "mcph", version: true ? "0.47.2" : "dev" },
+    { name: "mcph", version: true ? "0.47.3" : "dev" },
     { capabilities: {} }
   );
   let transport;
@@ -5199,163 +5196,6 @@ async function fetchToolsFromUpstream(client, namespace) {
   }));
 }
 
-// src/test-runner.ts
-var POLL_INTERVAL_MS = 3e4;
-var REQUEST_TIMEOUT_MS = 1e4;
-var CONSECUTIVE_404_LIMIT = 10;
-var apiUrl5 = "";
-var token5 = "";
-var pollTimer = null;
-var running = false;
-var consecutive404 = 0;
-var configRef = () => null;
-function initTestRunner(url, tok, getConfig) {
-  apiUrl5 = url;
-  token5 = tok;
-  configRef = getConfig;
-}
-function startTestRunner() {
-  if (running) return;
-  running = true;
-  consecutive404 = 0;
-  schedule();
-}
-function stopTestRunner() {
-  running = false;
-  consecutive404 = 0;
-  if (pollTimer) {
-    clearTimeout(pollTimer);
-    pollTimer = null;
-  }
-}
-function schedule() {
-  pollTimer = setTimeout(async () => {
-    try {
-      await pollOnce();
-    } catch (err) {
-      log("warn", "Test runner poll failed", { error: err?.message });
-    }
-    if (running) schedule();
-  }, POLL_INTERVAL_MS);
-  pollTimer.unref?.();
-}
-async function pollOnce() {
-  if (!apiUrl5 || !token5) return;
-  const list = await fetchPending();
-  if (list.length === 0) return;
-  for (const pending2 of list) {
-    if (!running) return;
-    await runOne(pending2).catch((err) => {
-      log("warn", "Test execution failed", { requestId: pending2.requestId, error: err?.message });
-    });
-  }
-}
-async function fetchPending() {
-  try {
-    const res = await request8(`${apiUrl5.replace(/\/$/, "")}/api/connect/test-requests`, {
-      method: "GET",
-      headers: { Authorization: `Bearer ${token5}` },
-      headersTimeout: REQUEST_TIMEOUT_MS,
-      bodyTimeout: REQUEST_TIMEOUT_MS
-    });
-    if (res.statusCode === 404) {
-      await res.body.text().catch(() => {
-      });
-      consecutive404++;
-      if (consecutive404 === 1) {
-        log("warn", "Test runner endpoint returned 404; will retry", { url: `${apiUrl5}/api/connect/test-requests` });
-      }
-      if (consecutive404 >= CONSECUTIVE_404_LIMIT) {
-        log("warn", "Test runner endpoint persistently 404; stopping poller", {
-          consecutive: consecutive404
-        });
-        stopTestRunner();
-      }
-      return [];
-    }
-    if (res.statusCode !== 200) {
-      await res.body.text().catch(() => {
-      });
-      consecutive404 = 0;
-      return [];
-    }
-    consecutive404 = 0;
-    const body = await res.body.json();
-    return Array.isArray(body?.requests) ? body.requests : [];
-  } catch {
-    return [];
-  }
-}
-async function runOne(pending2) {
-  const config = configRef();
-  const serverConfig = config?.servers.find((s) => s.id === pending2.serverId);
-  if (!serverConfig) {
-    await postResult(pending2.requestId, {
-      status: "failed",
-      message: `Server "${pending2.serverId}" is not in this mcph's current config \u2014 restart mcph or refresh the dashboard.`,
-      errorCategory: "not_in_config"
-    });
-    return;
-  }
-  if (!serverConfig.isActive) {
-    await postResult(pending2.requestId, {
-      status: "failed",
-      message: `Server "${serverConfig.namespace}" is disabled in the dashboard \u2014 re-enable it before testing.`,
-      errorCategory: "disabled"
-    });
-    return;
-  }
-  let connection = null;
-  try {
-    connection = await connectToUpstream(serverConfig);
-    await postResult(pending2.requestId, {
-      status: "passed",
-      toolCount: connection.tools.length,
-      message: `Connected \u2014 ${connection.tools.length} tool${connection.tools.length === 1 ? "" : "s"} available.`
-    });
-  } catch (err) {
-    if (err instanceof ActivationError) {
-      await postResult(pending2.requestId, {
-        status: "failed",
-        message: err.message,
-        errorCategory: err.category
-      });
-    } else {
-      await postResult(pending2.requestId, {
-        status: "failed",
-        message: err instanceof Error ? err.message : String(err),
-        errorCategory: "unknown"
-      });
-    }
-  } finally {
-    if (connection) {
-      await disconnectFromUpstream(connection).catch(() => {
-      });
-    }
-  }
-}
-async function postResult(requestId, result) {
-  try {
-    const res = await request8(
-      `${apiUrl5.replace(/\/$/, "")}/api/connect/test-requests/${encodeURIComponent(requestId)}/result`,
-      {
-        method: "POST",
-        headers: {
-          Authorization: `Bearer ${token5}`,
-          "Content-Type": "application/json"
-        },
-        body: JSON.stringify(result),
-        headersTimeout: REQUEST_TIMEOUT_MS,
-        bodyTimeout: REQUEST_TIMEOUT_MS
-      }
-    );
-    await res.body.text().catch(() => {
-    });
-  } catch (err) {
-    log("warn", "Posting test result failed", { requestId, error: err?.message });
-  }
-}
-
 // src/server.ts
 var DEFAULT_POLL_INTERVAL_MS = 6e4;
 function resolvePollIntervalMs() {
@@ -5435,11 +5275,11 @@ function computeToolOverlaps(connections) {
   return overlaps;
 }
 var ConnectServer = class _ConnectServer {
-  constructor(apiUrl6, token6) {
-    this.apiUrl = apiUrl6;
-    this.token = token6;
+  constructor(apiUrl5, token5) {
+    this.apiUrl = apiUrl5;
+    this.token = token5;
     this.server = new Server(
-      { name: "mcph", version: true ? "0.47.2" : "dev" },
+      { name: "mcph", version: true ? "0.47.3" : "dev" },
       {
         capabilities: {
           tools: { listChanged: true },
@@ -5580,23 +5420,23 @@ var ConnectServer = class _ConnectServer {
     this.server.setRequestHandler(ListToolsRequestSchema, async () => ({
       tools: buildToolList(this.connections, this.getDeferredServers(), this.toolFilters)
     }));
-    this.server.setRequestHandler(CallToolRequestSchema, async (request10, extra) => {
-      const { name, arguments: args } = request10.params;
+    this.server.setRequestHandler(CallToolRequestSchema, async (request9, extra) => {
+      const { name, arguments: args } = request9.params;
       return this.handleToolCall(name, args ?? {}, extra);
     });
     this.server.setRequestHandler(ListResourcesRequestSchema, async () => ({
       resources: buildResourceList(this.connections, this.getBuiltinResources())
     }));
-    this.server.setRequestHandler(ReadResourceRequestSchema, async (request10) => {
-      return routeResourceRead(request10.params.uri, this.resourceRoutes, this.connections, this.getBuiltinResourceMap());
+    this.server.setRequestHandler(ReadResourceRequestSchema, async (request9) => {
+      return routeResourceRead(request9.params.uri, this.resourceRoutes, this.connections, this.getBuiltinResourceMap());
     });
     this.server.setRequestHandler(ListPromptsRequestSchema, async () => ({
       prompts: buildPromptList(this.connections)
     }));
-    this.server.setRequestHandler(GetPromptRequestSchema, async (request10) => {
+    this.server.setRequestHandler(GetPromptRequestSchema, async (request9) => {
       return routePromptGet(
-        request10.params.name,
-        request10.params.arguments,
+        request9.params.name,
+        request9.params.arguments,
         this.promptRoutes,
         this.connections
       );
@@ -5701,12 +5541,10 @@ var ConnectServer = class _ConnectServer {
     initToolReport(this.apiUrl, this.token);
     initRerank(this.apiUrl, this.token);
     initRuntimeDetect(this.apiUrl, this.token);
-    initTestRunner(this.apiUrl, this.token, () => this.config);
     reportRuntimes().catch((err) => log("warn", "reportRuntimes failed", { error: err?.message }));
     if (this.config?.servers.some((s) => s.command === "uv" || s.command === "uvx")) {
       ensureUv().catch((err) => log("warn", "uv prewarm failed", { error: err?.message }));
     }
-    startTestRunner();
     const transport = new StdioServerTransport();
     await this.server.connect(transport);
     this.startPolling();
@@ -6943,7 +6781,7 @@ ${activeCount} loaded in this session, ${totalTools} tools in context${tokenSumm
         nsToKeys.set(s.namespace, existing);
       }
       const collisions = [...nsToKeys.entries()].filter(([, keys]) => keys.length > 1);
-      const res = await request9(`${this.apiUrl.replace(/\/$/, "")}/api/connect/import`, {
+      const res = await request8(`${this.apiUrl.replace(/\/$/, "")}/api/connect/import`, {
         method: "POST",
         headers: {
           Authorization: `Bearer ${this.token}`,
@@ -7004,7 +6842,7 @@ Use mcp_connect_discover to see imported servers.`
     }
     const payload = built.payload;
     try {
-      const res = await request9(`${this.apiUrl.replace(/\/$/, "")}/api/connect/servers`, {
+      const res = await request8(`${this.apiUrl.replace(/\/$/, "")}/api/connect/servers`, {
         method: "POST",
         headers: {
           Authorization: `Bearer ${this.token}`,
@@ -7468,7 +7306,6 @@ To load the top pack in one step, call \`mcp_connect_activate\` with namespaces=
     if (this.persistenceReady) {
       await this.flushStateSave();
     }
-    stopTestRunner();
     await shutdownAnalytics();
     const disconnects = Array.from(this.connections.values()).map((conn) => disconnectFromUpstream(conn));
     await Promise.allSettled(disconnects);
@@ -7803,7 +7640,7 @@ Or re-run with --run to upgrade in place.`);
   return { exitCode: 3, lines };
 }
 function readCurrentVersion() {
-  return true ? "0.47.2" : "dev";
+  return true ? "0.47.3" : "dev";
 }
 
 // src/index.ts
@@ -7971,7 +7808,7 @@ if (subcommand === "compliance") {
 `);
   process.exit(0);
 } else if (subcommand === "--version" || subcommand === "-V") {
-  process.stdout.write(`mcph ${true ? "0.47.2" : "dev"}
+  process.stdout.write(`mcph ${true ? "0.47.3" : "dev"}
 `);
   process.exit(0);
 } else if (subcommand && !subcommand.startsWith("-")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yawlabs/mcph",
-  "version": "0.47.2",
+  "version": "0.47.3",
   "description": "mcp.hosting — one install, all your MCP servers, managed from the cloud",
   "license": "UNLICENSED",
   "author": "Yaw Labs <support@mcp.hosting> (https://mcp.hosting)",