@agentvalet/mcp-server 0.3.2 → 0.3.4

package/dist/index.js

@@ -325,6 +325,40 @@ async function fetchWithAuth(url, init) {
 function errorContent(message) {
     return { content: [{ type: "text", text: message }], isError: true };
 }
+// Return helper that gives an MCP-aware host the structured content directly
+// (no parse step), while still emitting the text-content envelope every MCP
+// client understands. Avoids the "list of length 1 with empty fields → let me
+// parse the wrapper" round-trip Claude does on tool results that are raw
+// JSON strings — every read-heavy use_platform call pays that cost otherwise.
+//
+// If `body` isn't valid JSON, we just return the text envelope unchanged —
+// callers that emit prose (summaries, error strings) get the old behaviour.
+function jsonContent(body) {
+    const parsed = tryParseJson(body);
+    if (parsed === undefined) {
+        return { content: [{ type: "text", text: body }] };
+    }
+    return {
+        content: [{ type: "text", text: body }],
+        structuredContent: parsed,
+    };
+}
+function tryParseJson(text) {
+    const trimmed = text.trim();
+    // Only attempt parse if it looks structured. Avoids parsing a stray "true"
+    // / number / etc. into structuredContent and confusing the host.
+    if (!trimmed.startsWith("{") && !trimmed.startsWith("["))
+        return undefined;
+    try {
+        const v = JSON.parse(trimmed);
+        if (v && typeof v === "object")
+            return v;
+        return undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
 // ---------------------------------------------------------------------------
 // Tool handlers
 // ---------------------------------------------------------------------------
@@ -338,12 +372,34 @@ async function handleListPlatforms() {
         response = await fetchWithAuth(`${PROXY_URL}/v1/agent/permissions`, { method: "GET", headers: {} });
     }
     catch (err) {
-        return errorContent(`Network error: ${err instanceof Error ? err.message : err}`);
+        return errorContent(diagnoseNetworkError(err, PROXY_URL));
     }
     const body = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${body}`);
-    return { content: [{ type: "text", text: body }] };
+    return jsonContent(body);
+}
+// Translates a fetch() failure into something an end-user can actually act on.
+// The default "Network error: fetch failed" message tells a non-developer
+// nothing. Look at the underlying cause keyword and map to a concrete fix.
+function diagnoseNetworkError(err, proxyUrl) {
+    const raw = err instanceof Error ? err.message : String(err);
+    const lower = raw.toLowerCase();
+    // Node's undici surfaces DNS failures as "getaddrinfo ENOTFOUND <host>".
+    if (lower.includes("enotfound") || lower.includes("getaddrinfo")) {
+        return `Network error: cannot resolve ${proxyUrl}. Check your DNS / corporate proxy / VPN, or confirm the PROXY_URL setting is correct. Raw: ${raw}`;
+    }
+    // Connection refused / unreachable / TLS handshake failure.
+    if (lower.includes("econnrefused") || lower.includes("econnreset")) {
+        return `Network error: connection to ${proxyUrl} was refused or reset. The proxy may be down — check https://status.agentvalet.ai — or a firewall is blocking the request. Raw: ${raw}`;
+    }
+    if (lower.includes("etimedout") || lower.includes("timeout") || lower.includes("aborterror")) {
+        return `Network error: request to ${proxyUrl} timed out. Likely causes: VPN routing, corporate proxy buffering, or slow network. Try again or confirm api.agentvalet.ai is reachable from this machine. Raw: ${raw}`;
+    }
+    if (lower.includes("self signed") || lower.includes("cert") || lower.includes("ssl") || lower.includes("tls")) {
+        return `Network error: TLS / certificate problem talking to ${proxyUrl}. A corporate MITM proxy may be intercepting traffic. Raw: ${raw}`;
+    }
+    return `Network error reaching ${proxyUrl}: ${raw}. Check VPN, corporate proxy, and firewall rules for api.agentvalet.ai. If the proxy itself is down, see https://status.agentvalet.ai.`;
 }
 // Long-poll budget — under Claude Desktop's hardcoded 60s tool timeout
 // (with 10s of safety). After this we return a graceful "queued" message and
@@ -387,7 +443,7 @@ async function handleUsePlatform(params, progressToken) {
     }
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${body}`);
-    return { content: [{ type: "text", text: body }] };
+    return jsonContent(body);
 }
 async function waitForApproval(approvalId, originalCall, progressToken) {
     const startedAt = Date.now();
@@ -443,18 +499,13 @@ async function waitForApproval(approvalId, originalCall, progressToken) {
     // Timed out — fall through to async-recap path. The action is still
     // queued and will run when the owner approves; the user is notified via
     // push/email at that point. Layer 4's list_my_pending_actions surfaces it.
-    return {
-        content: [{
-                type: "text",
-                text: JSON.stringify({
-                    status: "pending_approval",
-                    approval_id: approvalId,
-                    message: `Owner hasn't approved within ${APPROVAL_POLL_BUDGET_MS / 1000}s. ` +
-                        `The action is queued — your owner will be notified, and you'll see it ` +
-                        `complete next time we chat (or you can ask me to check pending actions).`,
-                }),
-            }],
-    };
+    return jsonContent(JSON.stringify({
+        status: "pending_approval",
+        approval_id: approvalId,
+        message: `Owner hasn't approved within ${APPROVAL_POLL_BUDGET_MS / 1000}s. ` +
+            `The action is queued — your owner will be notified, and you'll see it ` +
+            `complete next time we chat (or you can ask me to check pending actions).`,
+    }));
 }
 function safeJsonParse(text) {
     try {
@@ -495,7 +546,7 @@ async function handleAgentRegister(args) {
     const body = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${body}`);
-    return { content: [{ type: "text", text: body }] };
+    return jsonContent(body);
 }
 async function handleAgentStatus(token) {
     let response;
@@ -511,7 +562,7 @@ async function handleAgentStatus(token) {
     const body = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${body}`);
-    return { content: [{ type: "text", text: body }] };
+    return jsonContent(body);
 }
 async function handleAuthzenEvaluate(platformId, scope) {
     const authzenBody = {
@@ -532,7 +583,7 @@ async function handleAuthzenEvaluate(platformId, scope) {
     const body = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${body}`);
-    return { content: [{ type: "text", text: body }] };
+    return jsonContent(body);
 }
 async function handleListMyPendingActions() {
     if (AGENT_PRIVATE_KEY_RAW === null) {
@@ -549,7 +600,7 @@ async function handleListMyPendingActions() {
     const text = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${text}`);
-    return { content: [{ type: "text", text }] };
+    return jsonContent(text);
 }
 async function handleReportSelfDiagnostic(args) {
     if (AGENT_PRIVATE_KEY_RAW === null) {
@@ -578,7 +629,7 @@ async function handleReportSelfDiagnostic(args) {
     const text = await response.text();
     if (!response.ok)
         return errorContent(`Proxy error ${response.status}: ${text}`);
-    return { content: [{ type: "text", text }] };
+    return jsonContent(text);
 }
 // ---------------------------------------------------------------------------
 // Connect transport

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentvalet/mcp-server",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "AgentValet MCP server — lets AI agents call approved platforms via the AgentValet proxy",
   "type": "module",
   "main": "dist/index.js",