diff --git a/package-lock.json b/package-lock.json index d682159928..7bbb204d4b 100644 --- a/package-lock.json +++ b/package-lock.json @@ -5795,7 +5795,7 @@ }, "src/redis": { "name": "@modelcontextprotocol/server-redis", - "version": "0.1.0", + "version": "0.1.1", "license": "MIT", "dependencies": { "@modelcontextprotocol/sdk": "^1.7.0", @@ -5804,7 +5804,7 @@ "redis": "^4.7.0" }, "bin": { - "redis": "build/index.js" + "mcp-server-redis": "dist/index.js" }, "devDependencies": { "shx": "^0.3.4", diff --git a/src/redis/README.md b/src/redis/README.md index 992471dc5f..48de29f86b 100644 --- a/src/redis/README.md +++ b/src/redis/README.md @@ -14,12 +14,18 @@ A Model Context Protocol server that provides access to Redis databases. This se ### Connection Errors **ECONNREFUSED** - - **Cause**: Redis server is not running or unreachable + - **Cause**: Redis/Memurai server is not running or unreachable - **Solution**: - - Verify Redis is running: `redis-cli ping` should return "PONG" - - Check Redis service status: `systemctl status redis` (Linux) or `brew services list` (macOS) + - Verify server is running: + - Redis: `redis-cli ping` should return "PONG" + - Memurai (Windows): `memurai-cli ping` should return "PONG" + - Check service status: + - Linux: `systemctl status redis` + - macOS: `brew services list` + - Windows: Check Memurai in Services (services.msc) - Ensure correct port (default 6379) is not blocked by firewall - Verify Redis URL format: `redis://hostname:port` + - If `redis://localhost:6379` fails with ECONNREFUSED, try using the explicit IP: `redis://127.0.0.1:6379` ### Server Behavior @@ -57,7 +63,7 @@ To use this server with the Claude Desktop app, add the following configuration ### Docker * when running docker on macos, use host.docker.internal if the server is running on the host network (eg localhost) -* Redis URL can be specified as an argument, defaults to "redis://localhost:6379" +* Redis URL can be specified as an argument, defaults to "redis://localhost:6379" (use "redis://127.0.0.1:6379" if localhost fails) ```json { diff --git a/src/redis/package.json b/src/redis/package.json index 3315526514..e2fa50b150 100644 --- a/src/redis/package.json +++ b/src/redis/package.json @@ -1,6 +1,6 @@ { "name": "@modelcontextprotocol/server-redis", - "version": "0.1.0", + "version": "0.1.1", "description": "MCP server for using Redis", "license": "MIT", "author": "Anthropic, PBC (https://anthropic.com)", @@ -8,13 +8,13 @@ "bugs": "https://github.com/modelcontextprotocol/servers/issues", "type": "module", "bin": { - "redis": "./build/index.js" + "mcp-server-redis": "dist/index.js" }, "files": [ - "build" + "dist" ], "scripts": { - "build": "tsc && shx chmod +x build/*.js", + "build": "tsc && shx chmod +x dist/*.js", "prepare": "npm run build", "watch": "tsc --watch" }, diff --git a/src/redis/src/index.ts b/src/redis/src/index.ts index 27ce91fae8..f3a21fb787 100644 --- a/src/redis/src/index.ts +++ b/src/redis/src/index.ts @@ -1,3 +1,5 @@ +#!/usr/bin/env node + import { Server } from "@modelcontextprotocol/sdk/server/index.js"; import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"; import { @@ -19,11 +21,14 @@ const redisClient = createClient({ socket: { reconnectStrategy: (retries) => { if (retries >= MAX_RETRIES) { - console.error(`Maximum retries (${MAX_RETRIES}) reached. Giving up.`); + console.error(`[Redis Error] Maximum retries (${MAX_RETRIES}) reached. Giving up.`); + console.error(`[Redis Error] Connection: ${REDIS_URL}`); return new Error('Max retries reached'); } const delay = Math.min(Math.pow(2, retries) * MIN_RETRY_DELAY, MAX_RETRY_DELAY); - console.error(`Reconnection attempt ${retries + 1}/${MAX_RETRIES} in ${delay}ms`); + console.error(`[Redis Retry] Attempt ${retries + 1}/${MAX_RETRIES} failed`); + console.error(`[Redis Retry] Next attempt in ${delay}ms`); + console.error(`[Redis Retry] Connection: ${REDIS_URL}`); return delay; } } @@ -233,26 +238,27 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => { } }); -// Start the server -async function main() { - try { - // Set up Redis event handlers - redisClient.on('error', (err: Error) => { - console.error('Redis Client Error:', err); - }); +// Set up Redis event handlers +redisClient.on('error', (err: Error) => { + console.error(`[Redis Error] ${err.name}: ${err.message}`); + console.error(`[Redis Error] Connection: ${REDIS_URL}`); + console.error(`[Redis Error] Stack: ${err.stack}`); +}); - redisClient.on('connect', () => { - console.error(`Connected to Redis at ${REDIS_URL}`); - }); +redisClient.on('connect', () => { + console.error(`[Redis Connected] Successfully connected to ${REDIS_URL}`); +}); - redisClient.on('reconnecting', () => { - console.error('Attempting to reconnect to Redis...'); - }); +redisClient.on('reconnecting', () => { + console.error('[Redis Reconnecting] Connection lost, attempting to reconnect...'); +}); - redisClient.on('end', () => { - console.error('Redis connection closed'); - }); +redisClient.on('end', () => { + console.error('[Redis Disconnected] Connection closed'); +}); +async function runServer() { + try { // Connect to Redis await redisClient.connect(); @@ -261,26 +267,25 @@ async function main() { await server.connect(transport); console.error("Redis MCP Server running on stdio"); } catch (error) { - console.error("Error during startup:", error); - await cleanup(); + const err = error as Error; + console.error("[Redis Fatal] Server initialization failed"); + console.error(`[Redis Fatal] Error: ${err.name}: ${err.message}`); + console.error(`[Redis Fatal] Connection: ${REDIS_URL}`); + console.error(`[Redis Fatal] Stack: ${err.stack}`); + await redisClient.quit().catch(() => {}); + process.exit(1); } } -// Cleanup function -async function cleanup() { - try { - await redisClient.quit(); - } catch (error) { - console.error("Error during cleanup:", error); - } - process.exit(1); -} - // Handle process termination -process.on('SIGINT', cleanup); -process.on('SIGTERM', cleanup); +process.on('SIGINT', async () => { + await redisClient.quit().catch(() => {}); + process.exit(0); +}); -main().catch((error) => { - console.error("Fatal error in main():", error); - cleanup(); +process.on('SIGTERM', async () => { + await redisClient.quit().catch(() => {}); + process.exit(0); }); + +runServer(); diff --git a/src/redis/tsconfig.json b/src/redis/tsconfig.json index efcd96e7e1..e4d02eb531 100644 --- a/src/redis/tsconfig.json +++ b/src/redis/tsconfig.json @@ -3,7 +3,7 @@ "target": "ES2022", "module": "Node16", "moduleResolution": "Node16", - "outDir": "./build", + "outDir": "./dist", "rootDir": "./src", "strict": true, "esModuleInterop": true, @@ -13,4 +13,3 @@ "include": ["src/**/*"], "exclude": ["node_modules"] } - \ No newline at end of file