@prmichaelsen/remember-mcp 0.2.3 → 0.2.5

package/README.md

@@ -4,17 +4,49 @@ Multi-tenant memory system MCP server with vector search, relationships, and tru
 
 ## Features
 
-- 24 MCP tools for memory management
+- 10 MCP tools for memory and relationship management
 - Multi-tenant with per-user isolation
-- Vector search with Weaviate
-- Relationship graph between memories
-- Template system with auto-suggestion
-- Trust-based cross-user access control
-- User preferences via conversation
+- Vector search with Weaviate (semantic + keyword hybrid search)
+- Knowledge graph with relationship tracking
+- RAG queries with natural language
+- 45 content types (notes, events, people, recipes, etc.)
+- Trust-based access control (planned for M7)
 
 ## Quick Start
 
-### Standalone (stdio transport)
+### Option 1: Use with Claude Desktop (Recommended)
+
+Add to your Claude Desktop MCP configuration:
+
+**macOS**: `~/Library/Application Support/Claude/claude_desktop_config.json`
+**Linux**: `~/.config/Claude/claude_desktop_config.json`
+**Windows**: `%APPDATA%\Claude\claude_desktop_config.json`
+
+```json
+{
+  "mcpServers": {
+    "remember": {
+      "command": "npx",
+      "args": ["-y", "@prmichaelsen/remember-mcp"],
+      "env": {
+        "WEAVIATE_REST_URL": "https://your-instance.weaviate.cloud",
+        "WEAVIATE_API_KEY": "your-weaviate-api-key",
+        "OPENAI_EMBEDDINGS_API_KEY": "sk-...",
+        "FIREBASE_ADMIN_SERVICE_ACCOUNT_KEY": "{\"type\":\"service_account\",\"project_id\":\"your-project\",\"private_key\":\"-----BEGIN PRIVATE KEY-----\\nYOUR_KEY\\n-----END PRIVATE KEY-----\\n\",\"client_email\":\"firebase-adminsdk@your-project.iam.gserviceaccount.com\"}",
+        "FIREBASE_PROJECT_ID": "your-project-id"
+      }
+    }
+  }
+}
+```
+
+**Important**:
+- Use `\\n` (double backslash) for newlines in private_key
+- Escape all quotes with `\"`
+- Get Weaviate Cloud at https://console.weaviate.cloud
+- Get Firebase service account from Firebase Console → Project Settings → Service Accounts
+
+### Option 2: Standalone (stdio transport)
 
 ```bash
 # Install dependencies
@@ -32,10 +64,10 @@ npm run build
 npm start
 ```
 
-### With mcp-auth (multi-tenant production)
+### Option 3: With mcp-auth (multi-tenant production)
 
 ```typescript
-import { wrapServer, JWTAuthProvider, APITokenResolver } from '@prmichaelsen/mcp-auth';
+import { wrapServer, JWTAuthProvider } from '@prmichaelsen/mcp-auth';
 import { createServer } from '@prmichaelsen/remember-mcp/factory';
 
 const wrapped = wrapServer({
@@ -43,10 +75,7 @@ const wrapped = wrapServer({
   authProvider: new JWTAuthProvider({
     jwtSecret: process.env.JWT_SECRET
   }),
-  tokenResolver: new APITokenResolver({
-    tenantManagerUrl: process.env.TENANT_MANAGER_URL,
-    serviceToken: process.env.SERVICE_TOKEN
-  }),
+  // tokenResolver not needed - remember-mcp is self-managed
   resourceType: 'remember',
   transport: { type: 'sse', port: 3000 }
 });

package/dist/server-factory.js

@@ -630,24 +630,40 @@ function shouldLog(level) {
   return LOG_LEVELS[level] >= currentLevel;
 }
 var logger = {
-  debug: (message, ...args) => {
+  debug: (message, data) => {
     if (shouldLog("debug")) {
-      console.debug(`[DEBUG] ${message}`, ...args);
+      if (data) {
+        console.debug(JSON.stringify({ level: "DEBUG", message, ...data }));
+      } else {
+        console.debug(`[DEBUG] ${message}`);
+      }
     }
   },
-  info: (message, ...args) => {
+  info: (message, data) => {
     if (shouldLog("info")) {
-      console.info(`[INFO] ${message}`, ...args);
+      if (data) {
+        console.info(JSON.stringify({ level: "INFO", message, ...data }));
+      } else {
+        console.info(`[INFO] ${message}`);
+      }
     }
   },
-  warn: (message, ...args) => {
+  warn: (message, data) => {
     if (shouldLog("warn")) {
-      console.warn(`[WARN] ${message}`, ...args);
+      if (data) {
+        console.warn(JSON.stringify({ level: "WARN", message, ...data }));
+      } else {
+        console.warn(`[WARN] ${message}`);
+      }
     }
   },
-  error: (message, ...args) => {
+  error: (message, data) => {
     if (shouldLog("error")) {
-      console.error(`[ERROR] ${message}`, ...args);
+      if (data) {
+        console.error(JSON.stringify({ level: "ERROR", message, ...data }));
+      } else {
+        console.error(`[ERROR] ${message}`);
+      }
     }
   }
 };
@@ -2700,8 +2716,14 @@ async function ensureDatabasesInitialized() {
       databasesInitialized = true;
       logger.info("Databases initialized successfully");
     } catch (error) {
-      logger.error("Database initialization failed:", error);
-      throw error;
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      const errorStack = error instanceof Error ? error.stack : void 0;
+      logger.error("Database initialization failed", {
+        error: errorMessage,
+        stack: errorStack,
+        type: error instanceof Error ? error.constructor.name : "Unknown"
+      });
+      throw new Error(`Database initialization failed: ${errorMessage}`);
     } finally {
       initializationPromise = null;
     }
@@ -2709,9 +2731,6 @@ async function ensureDatabasesInitialized() {
   return initializationPromise;
 }
 function createServer(accessToken, userId, options = {}) {
-  if (!accessToken) {
-    throw new Error("accessToken is required");
-  }
   if (!userId) {
     throw new Error("userId is required");
   }

package/dist/server.js

@@ -308,7 +308,7 @@ var require_main = __commonJS({
         return { parsed: parsedAll };
       }
     }
-    function config2(options) {
+    function config3(options) {
       if (_dotenvKey(options).length === 0) {
         return DotenvModule.configDotenv(options);
       }
@@ -375,7 +375,7 @@ var require_main = __commonJS({
       configDotenv,
       _configVault,
       _parseVault,
-      config: config2,
+      config: config3,
       decrypt,
       parse,
       populate
@@ -564,24 +564,40 @@ function shouldLog(level) {
   return LOG_LEVELS[level] >= currentLevel;
 }
 var logger = {
-  debug: (message, ...args) => {
+  debug: (message, data) => {
     if (shouldLog("debug")) {
-      console.debug(`[DEBUG] ${message}`, ...args);
+      if (data) {
+        console.debug(JSON.stringify({ level: "DEBUG", message, ...data }));
+      } else {
+        console.debug(`[DEBUG] ${message}`);
+      }
     }
   },
-  info: (message, ...args) => {
+  info: (message, data) => {
     if (shouldLog("info")) {
-      console.info(`[INFO] ${message}`, ...args);
+      if (data) {
+        console.info(JSON.stringify({ level: "INFO", message, ...data }));
+      } else {
+        console.info(`[INFO] ${message}`);
+      }
     }
   },
-  warn: (message, ...args) => {
+  warn: (message, data) => {
     if (shouldLog("warn")) {
-      console.warn(`[WARN] ${message}`, ...args);
+      if (data) {
+        console.warn(JSON.stringify({ level: "WARN", message, ...data }));
+      } else {
+        console.warn(`[WARN] ${message}`);
+      }
     }
   },
-  error: (message, ...args) => {
+  error: (message, data) => {
     if (shouldLog("error")) {
-      console.error(`[ERROR] ${message}`, ...args);
+      if (data) {
+        console.error(JSON.stringify({ level: "ERROR", message, ...data }));
+      } else {
+        console.error(`[ERROR] ${message}`);
+      }
     }
   }
 };
@@ -2643,14 +2659,6 @@ function registerHandlers(server) {
   server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
       tools: [
-        {
-          name: "health_check",
-          description: "Check server health and database connections",
-          inputSchema: {
-            type: "object",
-            properties: {}
-          }
-        },
         // Memory tools
         createMemoryTool,
         searchMemoryTool,
@@ -2672,9 +2680,6 @@ function registerHandlers(server) {
       let result;
       const userId = args.user_id || "default_user";
       switch (name) {
-        case "health_check":
-          result = await handleHealthCheck();
-          break;
         case "remember_create_memory":
           result = await handleCreateMemory(args, userId);
           break;
@@ -2731,42 +2736,6 @@ function registerHandlers(server) {
     }
   });
 }
-async function handleHealthCheck() {
-  const health = {
-    status: "healthy",
-    timestamp: (/* @__PURE__ */ new Date()).toISOString(),
-    server: {
-      name: "remember-mcp",
-      version: "0.1.0"
-    },
-    databases: {
-      weaviate: {
-        connected: false,
-        url: config.weaviate.url
-      },
-      firestore: {
-        connected: false,
-        projectId: config.firebase.projectId
-      }
-    }
-  };
-  try {
-    const weaviateClient = getWeaviateClient();
-    health.databases.weaviate.connected = await weaviateClient.isReady();
-  } catch (error) {
-    logger.error("Weaviate health check failed:", error);
-    health.databases.weaviate.connected = false;
-  }
-  try {
-    health.databases.firestore.connected = await testFirestoreConnection();
-  } catch (error) {
-    logger.error("Firestore health check failed:", error);
-    health.databases.firestore.connected = false;
-  }
-  const allHealthy = health.databases.weaviate.connected && health.databases.firestore.connected;
-  health.status = allHealthy ? "healthy" : "degraded";
-  return JSON.stringify(health, null, 2);
-}
 async function main() {
   try {
     logger.info("Starting remember-mcp server...");

package/dist/utils/logger.d.ts

@@ -1,7 +1,7 @@
 export declare const logger: {
-    debug: (message: string, ...args: any[]) => void;
-    info: (message: string, ...args: any[]) => void;
-    warn: (message: string, ...args: any[]) => void;
-    error: (message: string, ...args: any[]) => void;
+    debug: (message: string, data?: any) => void;
+    info: (message: string, data?: any) => void;
+    warn: (message: string, data?: any) => void;
+    error: (message: string, data?: any) => void;
 };
 //# sourceMappingURL=logger.d.ts.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "0.2.3",
+  "version": "0.2.5",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",
@@ -49,6 +49,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.4",
     "@prmichaelsen/firebase-admin-sdk-v8": "^2.2.0",
+    "@prmichaelsen/mcp-auth": "^7.0.4",
     "dotenv": "^16.4.5",
     "weaviate-client": "^3.2.0"
   },

package/src/server-factory.spec.ts

@@ -9,8 +9,10 @@ describe('Server Factory', () => {
       expect(server).toHaveProperty('setRequestHandler');
     });
 
-    it('should require accessToken', () => {
-      expect(() => createServer('', 'user123')).toThrow('accessToken is required');
+    it('should allow empty accessToken (not used by remember-mcp)', () => {
+      // accessToken is not used by remember-mcp (self-managed data)
+      // Should not throw even with empty string
+      expect(() => createServer('', 'user123')).not.toThrow();
     });
 
     it('should require userId', () => {

package/src/server-factory.ts

@@ -59,8 +59,17 @@ async function ensureDatabasesInitialized(): Promise<void> {
       databasesInitialized = true;
       logger.info('Databases initialized successfully');
     } catch (error) {
-      logger.error('Database initialization failed:', error);
-      throw error;
+      // Format error as single-line JSON for better cloud logging
+      const errorMessage = error instanceof Error ? error.message : String(error);
+      const errorStack = error instanceof Error ? error.stack : undefined;
+      
+      logger.error('Database initialization failed', {
+        error: errorMessage,
+        stack: errorStack,
+        type: error instanceof Error ? error.constructor.name : 'Unknown'
+      });
+      
+      throw new Error(`Database initialization failed: ${errorMessage}`);
     } finally {
       initializationPromise = null;
     }
@@ -105,9 +114,8 @@ export function createServer(
   userId: string,
   options: ServerOptions = {}
 ): Server {
-  if (!accessToken) {
-    throw new Error('accessToken is required');
-  }
+  // Note: accessToken is not used by remember-mcp (self-managed data)
+  // but required by mcp-auth contract. Can be any value including empty string.
   
   if (!userId) {
     throw new Error('userId is required');

package/src/server.ts

@@ -79,14 +79,6 @@ function registerHandlers(server: Server): void {
   server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
       tools: [
-        {
-          name: 'health_check',
-          description: 'Check server health and database connections',
-          inputSchema: {
-            type: 'object',
-            properties: {},
-          },
-        },
         // Memory tools
         createMemoryTool,
         searchMemoryTool,
@@ -115,10 +107,6 @@ function registerHandlers(server: Server): void {
       const userId = (args as any).user_id || 'default_user';
 
       switch (name) {
-        case 'health_check':
-          result = await handleHealthCheck();
-          break;
-
         case 'remember_create_memory':
           result = await handleCreateMemory(args as any, userId);
           break;
@@ -188,56 +176,6 @@ function registerHandlers(server: Server): void {
   });
 }
 
-/**
- * Health check handler
- */
-async function handleHealthCheck(): Promise<string> {
-  const health = {
-    status: 'healthy',
-    timestamp: new Date().toISOString(),
-    server: {
-      name: 'remember-mcp',
-      version: '0.1.0',
-    },
-    databases: {
-      weaviate: {
-        connected: false,
-        url: config.weaviate.url,
-      },
-      firestore: {
-        connected: false,
-        projectId: config.firebase.projectId,
-      },
-    },
-  };
-
-  try {
-    // Test Weaviate connection
-    const weaviateClient = getWeaviateClient();
-    health.databases.weaviate.connected = await weaviateClient.isReady();
-  } catch (error) {
-    logger.error('Weaviate health check failed:', error);
-    health.databases.weaviate.connected = false;
-  }
-
-  try {
-    // Test Firestore connection
-    health.databases.firestore.connected = await testFirestoreConnection();
-  } catch (error) {
-    logger.error('Firestore health check failed:', error);
-    health.databases.firestore.connected = false;
-  }
-
-  // Overall status
-  const allHealthy =
-    health.databases.weaviate.connected &&
-    health.databases.firestore.connected;
-
-  health.status = allHealthy ? 'healthy' : 'degraded';
-
-  return JSON.stringify(health, null, 2);
-}
-
 /**
  * Main server startup
  */

package/src/utils/logger.ts

@@ -16,27 +16,44 @@ function shouldLog(level: LogLevel): boolean {
 }
 
 export const logger = {
-  debug: (message: string, ...args: any[]) => {
+  debug: (message: string, data?: any) => {
     if (shouldLog('debug')) {
-      console.debug(`[DEBUG] ${message}`, ...args);
+      if (data) {
+        console.debug(JSON.stringify({ level: 'DEBUG', message, ...data }));
+      } else {
+        console.debug(`[DEBUG] ${message}`);
+      }
     }
   },
 
-  info: (message: string, ...args: any[]) => {
+  info: (message: string, data?: any) => {
     if (shouldLog('info')) {
-      console.info(`[INFO] ${message}`, ...args);
+      if (data) {
+        console.info(JSON.stringify({ level: 'INFO', message, ...data }));
+      } else {
+        console.info(`[INFO] ${message}`);
+      }
     }
   },
 
-  warn: (message: string, ...args: any[]) => {
+  warn: (message: string, data?: any) => {
     if (shouldLog('warn')) {
-      console.warn(`[WARN] ${message}`, ...args);
+      if (data) {
+        console.warn(JSON.stringify({ level: 'WARN', message, ...data }));
+      } else {
+        console.warn(`[WARN] ${message}`);
+      }
     }
   },
 
-  error: (message: string, ...args: any[]) => {
+  error: (message: string, data?: any) => {
     if (shouldLog('error')) {
-      console.error(`[ERROR] ${message}`, ...args);
+      if (data) {
+        // Structured logging for cloud environments
+        console.error(JSON.stringify({ level: 'ERROR', message, ...data }));
+      } else {
+        console.error(`[ERROR] ${message}`);
+      }
     }
   },
 };