byclaw-mcp 0.4.14 → 0.4.16

package/dist/index.js

@@ -53,16 +53,43 @@ function imageUrlOf(p) {
     const url = typeof p?.image_url === 'string' ? p.image_url.trim() : '';
     return url.startsWith('https://') || url.startsWith('http://') ? url : IMAGE_FALLBACK;
 }
+// Hard cap on every backend request. Without it, if the backend is wedged
+// (Anthropic 503 + SDK retry storm, ES lock, …) the MCP process would hang
+// forever and the client (Claude Desktop) would only break out at its own
+// 4 min cap with a "Server disconnected" disconnect — the very symptom
+// Finding 094 described. 60 s is comfortably above normal backend latency
+// (10-15 s for /api/mcp/shop) but below the client timeout, so the user
+// gets a real error message instead of a stall.
+const BACKEND_TIMEOUT_MS = 60000;
 async function apiCall(path, options = {}) {
     const headers = {
         'Authorization': `Bearer ${API_KEY}`,
         'Content-Type': 'application/json',
     };
-    const res = await fetch(`${API_BASE}${path}`, {
-        ...options,
-        headers: { ...headers, ...options.headers },
-    });
-    return res.json();
+    try {
+        const res = await fetch(`${API_BASE}${path}`, {
+            ...options,
+            signal: AbortSignal.timeout(BACKEND_TIMEOUT_MS),
+            headers: { ...headers, ...options.headers },
+        });
+        return res.json();
+    }
+    catch (err) {
+        // AbortError fires when the timeout above kicks in. Translate to an
+        // ok=false body the existing call sites already know how to print.
+        if (err?.name === 'AbortError' || err?.name === 'TimeoutError') {
+            return {
+                ok: false,
+                error: `Backend timeout after ${BACKEND_TIMEOUT_MS / 1000}s (${path})`,
+                retryable: true,
+            };
+        }
+        return {
+            ok: false,
+            error: `Backend request failed: ${err?.message || 'unknown error'} (${path})`,
+            retryable: false,
+        };
+    }
 }
 const server = new index_js_1.Server({ name: 'byclaw', version: SERVER_VERSION }, { capabilities: { tools: {}, prompts: {} } });
 // Prompts
@@ -115,6 +142,11 @@ server.setRequestHandler(types_js_1.ListToolsRequestSchema, async () => ({
                 required: ['query'],
             },
         },
+        {
+            name: 'whoami',
+            description: TOOL_PREFIX + 'Return the active byclaw agent: its codename (e.g. #1949-H), personality (smartpicker / dealmaker / luxpicker / vibechecker / quickshooter), tagline, tone, and review style. Use when the user asks "which agent am I using", "what persona is searching for me", or "wer entscheidet hier". Cheap, no Anthropic call. Every shop / search_products / test_web_search response also surfaces this in the `agent` and `advisor_profile` fields.',
+            inputSchema: { type: 'object', properties: {} },
+        },
         {
             name: 'leave_review',
             description: TOOL_PREFIX + 'Write a product review. Only call if the user explicitly asks to leave a review. Only de and en allowed.',
@@ -295,6 +327,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                         return { content: [{ type: 'text', text: JSON.stringify({
                                         _render_hint: RENDER_HINT,
                                         type: 'gift_found',
+                                        agent: d.agent || null,
                                         advisor: advisorLabel,
                                         advisor_profile: d.advisor_profile || null,
                                         message: d.message || null,
@@ -319,6 +352,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                     if (d.status === 'clarification_needed') {
                         return { content: [{ type: 'text', text: JSON.stringify({
                                         type: 'clarification_needed',
+                                        agent: d.agent || null,
                                         advisor: advisorLabel,
                                         advisor_profile: d.advisor_profile || null,
                                         message: d.message || '',
@@ -332,6 +366,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                     if (d.status === 'acknowledgement_legacy' || d.status === 'acknowledgement') {
                         return { content: [{ type: 'text', text: JSON.stringify({
                                         type: d.status,
+                                        agent: d.agent || null,
                                         advisor: advisorLabel,
                                         advisor_profile: d.advisor_profile || null,
                                         message: d.message || '',
@@ -349,6 +384,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                     if (d.status === 'no_match') {
                         content.push({ type: 'text', text: JSON.stringify({
                                 type: 'no_match',
+                                agent: d.agent || null,
                                 advisor: advisorLabel,
                                 advisor_profile: d.advisor_profile || null,
                                 message: d.message || '',
@@ -362,6 +398,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                         content.push({ type: 'text', text: JSON.stringify({
                                 _render_hint: RENDER_HINT,
                                 type: d.status || 'product_found',
+                                agent: d.agent || null,
                                 advisor: advisorLabel,
                                 advisor_profile: d.advisor_profile || null,
                                 picked: hasPick ? {
@@ -431,6 +468,13 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                 }
                 return { content: [{ type: 'text', text: JSON.stringify({ _render_hint: RENDER_HINT, ...result }, null, 2) }] };
             }
+            case 'whoami': {
+                // Cheap lookup — backend `/api/agents/profile` resolves the agent
+                // from the API key and returns name + persona + style fields.
+                // No Anthropic call, no DB writes; ~50 ms typical.
+                const result = await apiCall('/api/agents/profile');
+                return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
+            }
             case 'leave_review': {
                 const lang = args?.language;
                 if (lang !== 'de' && lang !== 'en') {
@@ -490,6 +534,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                 if (d.status === 'clarification_needed') {
                     return { content: [{ type: 'text', text: JSON.stringify({
                                     type: 'clarification_needed',
+                                    agent: d.agent || null,
                                     advisor: d.advisor_profile?.name || d.agent?.personality || null,
                                     advisor_profile: d.advisor_profile || null,
                                     message: d.message || null,
@@ -504,6 +549,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                     return { content: [{ type: 'text', text: JSON.stringify({
                                     _render_hint: RENDER_HINT,
                                     type: 'gift_found',
+                                    agent: d.agent || null,
                                     advisor: d.advisor_profile?.name || d.agent?.personality || null,
                                     advisor_profile: d.advisor_profile || null,
                                     message: d.message || null,
@@ -520,6 +566,7 @@ server.setRequestHandler(types_js_1.CallToolRequestSchema, async (request) => {
                 return { content: [{ type: 'text', text: JSON.stringify({
                                 _render_hint: RENDER_HINT,
                                 type: d.status || 'no_match',
+                                agent: d.agent || null,
                                 advisor: d.advisor_profile?.name || d.agent?.personality || null,
                                 advisor_profile: d.advisor_profile || null,
                                 message: d.message || null,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "byclaw-mcp",
-  "version": "0.4.14",
+  "version": "0.4.16",
   "description": "MCP Server for byclaw.io — Your AI shopping advisor. Connects Claude Desktop and other MCP clients to the byclaw.io product catalog.",
   "main": "dist/index.js",
   "bin": {