secondbrainos-mcp-server 1.4.7 → 1.4.9

package/build/index.js

@@ -1,7 +1,7 @@
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { ListResourcesRequestSchema, ReadResourceRequestSchema, ListToolsRequestSchema, CallToolRequestSchema, ListPromptsRequestSchema, GetPromptRequestSchema, ErrorCode, McpError } from "@modelcontextprotocol/sdk/types.js";
-import { OpenApi, HttpLlm, HttpError } from "@samchon/openapi";
+import { OpenApi, HttpLlm } from "@samchon/openapi";
 import axios from "axios";
 import dotenv from "dotenv";
 import yaml from 'js-yaml';
@@ -142,97 +142,52 @@ class SecondBrainOSServer {
             try {
                 // Intercept file_path arguments: upload local files to GCS
                 const processedArgs = await this.interceptFilePathArgs((request.params.arguments || {}));
-                // Use HttpLlm.execute for better error handling and type safety
-                const result = await HttpLlm.execute({
+                // Use HttpLlm.propagate to get full response (status + body)
+                // instead of HttpLlm.execute which throws HttpError and loses
+                // the response body (Error constructor coerces objects to "[object Object]")
+                const response = await HttpLlm.propagate({
                     connection: {
                         host: this.baseUrl,
                         headers: {
                             'Authorization': `Bearer ${this.userId}:${this.userSecret}`
                         }
                     },
-                    application: this.application, // Type assertion to avoid generic issues
+                    application: this.application,
                     function: func,
                     input: processedArgs
                 });
-                return {
-                    content: [{
-                            type: 'text',
-                            text: JSON.stringify(result, null, 2)
-                        }]
-                };
-            }
-            catch (error) {
-                // HttpError from @samchon/openapi — its constructor passes the
-                // parsed response body (an object) to super(), so Error coerces it
-                // to "[object Object]" and the original detail is lost.
-                // We can only rely on error.status for meaningful messages.
-                if (error instanceof HttpError) {
-                    let mcpError;
-                    switch (error.status) {
-                        case 401:
-                            mcpError = 'Authentication failed: Invalid or expired credentials';
-                            break;
-                        case 400:
-                            mcpError = `Bad request (${error.method} ${error.path})`;
-                            break;
-                        case 403:
-                            mcpError = `Insufficient credits or permissions (${error.method} ${error.path})`;
-                            break;
-                        case 404:
-                            mcpError = `Service or resource not found (${error.method} ${error.path})`;
-                            break;
-                        default:
-                            mcpError = `Error ${error.status} (${error.method} ${error.path})`;
-                    }
-                    return {
-                        content: [{
-                                type: 'text',
-                                text: mcpError
-                            }],
-                        isError: true
-                    };
-                }
-                else if (axios.isAxiosError(error)) {
-                    const status = error.response?.status;
-                    const rawError = error.response?.data?.error || error.response?.data?.message || error.message;
-                    const errorMessage = typeof rawError === 'object' ? JSON.stringify(rawError) : rawError;
-                    let mcpError;
-                    switch (status) {
-                        case 401:
-                            mcpError = 'Authentication failed: Invalid or expired credentials';
-                            break;
-                        case 400:
-                            mcpError = `Bad request: ${errorMessage}`;
-                            break;
-                        case 403:
-                            mcpError = 'Insufficient credits or permissions';
-                            break;
-                        case 404:
-                            mcpError = 'Service or resource not found';
-                            break;
-                        default:
-                            mcpError = `Error calling function: ${errorMessage}`;
-                    }
-                    return {
-                        content: [{
-                                type: 'text',
-                                text: mcpError
-                            }],
-                        isError: true
-                    };
-                }
-                else {
-                    const msg = error instanceof Error
-                        ? error.message
-                        : (typeof error === 'object' ? JSON.stringify(error) : String(error));
+                // Non-success status — return the full error body
+                if (response.status !== 200 && response.status !== 201) {
+                    const detail = typeof response.body === 'object'
+                        ? JSON.stringify(response.body)
+                        : String(response.body ?? `HTTP ${response.status}`);
                     return {
                         content: [{
                                 type: 'text',
-                                text: `Error calling function: ${msg}`
+                                text: detail
                             }],
                         isError: true
                     };
                 }
+                return {
+                    content: [{
+                            type: 'text',
+                            text: JSON.stringify(response.body, null, 2)
+                        }]
+                };
+            }
+            catch (error) {
+                // Connection failures or unexpected errors
+                const msg = error instanceof Error
+                    ? error.message
+                    : (typeof error === 'object' ? JSON.stringify(error) : String(error));
+                return {
+                    content: [{
+                            type: 'text',
+                            text: `Error calling function: ${msg}`
+                        }],
+                    isError: true
+                };
             }
         });
         // List available prompts (user's workflows + agents)
@@ -720,11 +675,13 @@ class SecondBrainOSServer {
         // This method is kept for future error handling implementations
     }
     async run() {
-        // Write skill files before connecting so Claude Code sees them at session start
-        await this.writeSkillFiles().catch(err => console.error('Failed to write skill files:', err));
         const transport = new StdioServerTransport();
         await this.server.connect(transport);
         console.error("Second Brain OS MCP server running on stdio");
+        // Write skill files AFTER connecting so the MCP handshake completes
+        // within Claude Code's 30s timeout. Skill files are written in the
+        // background — they'll be available for subsequent conversations.
+        this.writeSkillFiles().catch(err => console.error('Failed to write skill files:', err));
     }
 }
 SecondBrainOSServer.ALLOWED_EXTENSIONS = ['.txt', '.md'];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "secondbrainos-mcp-server",
-  "version": "1.4.7",
+  "version": "1.4.9",
   "description": "Second Brain OS MCP Server for Claude Desktop",
   "type": "module",
   "main": "build/index.js",