@cainli/mcp-server-mysql 2.0.8 → 2.0.10

package/README.md

@@ -49,6 +49,27 @@ A Model Context Protocol server that provides access to MySQL databases through
 - MySQL user with appropriate permissions for the operations you need
 - For write operations: MySQL user with INSERT, UPDATE, and/or DELETE privileges
 
+## Platform Support
+
+This MCP server is fully compatible with:
+
+- **macOS Big Sur (11.x) and later** - Full support including Unix Socket connections
+- **Windows 10 and later** - Full support via TCP/IP connections
+- **Linux** - Full support including Unix Socket connections
+
+### Platform-Specific Notes
+
+#### macOS Big Sur
+- Can use either TCP/IP or Unix Socket connections
+- Unix Socket path example: `/tmp/mysql.sock` or `/var/run/mysql.sock`
+- No additional configuration required
+
+#### Windows 10
+- Must use TCP/IP connections (Unix Sockets are not supported)
+- Ensure MySQL is configured to accept TCP/IP connections
+- Default connection: `MYSQL_HOST=127.0.0.1` and `MYSQL_PORT=3306`
+- If using WSL2, you can connect to MySQL via Windows host IP (e.g., `MYSQL_HOST=172.x.x.x`)
+
 ## Installation
 
 ### Using Smithery
@@ -514,13 +535,54 @@ For more control over the MCP server's behavior, you can use these advanced conf
 
 ### Basic Connection
 
-- `MYSQL_SOCKET_PATH`: Unix socket path for local connections (e.g., "/tmp/mysql.sock")
-- `MYSQL_HOST`: MySQL server host (default: "127.0.0.1") - ignored if MYSQL_SOCKET_PATH is set
-- `MYSQL_PORT`: MySQL server port (default: "3306") - ignored if MYSQL_SOCKET_PATH is set
+- `MYSQL_SOCKET_PATH`: Unix socket path for local connections (e.g., "/tmp/mysql.sock") - **macOS/Linux only**
+- `MYSQL_HOST`: MySQL server host (default: "127.0.0.1") - **required for Windows**
+- `MYSQL_PORT`: MySQL server port (default: "3306") - **required for Windows**
 - `MYSQL_USER`: MySQL username (default: "root")
 - `MYSQL_PASS`: MySQL password
 - `MYSQL_DB`: Target database name (leave empty for multi-DB mode)
 
+### Platform-Specific Configuration
+
+#### macOS Big Sur
+
+On macOS, you can use either Unix sockets or TCP/IP:
+
+**Option 1: Unix Socket (Recommended for local MySQL)**
+```bash
+MYSQL_SOCKET_PATH="/tmp/mysql.sock"
+MYSQL_USER="root"
+MYSQL_PASS="your_password"
+MYSQL_DB="your_database"
+```
+
+**Option 2: TCP/IP**
+```bash
+MYSQL_HOST="127.0.0.1"
+MYSQL_PORT="3306"
+MYSQL_USER="root"
+MYSQL_PASS="your_password"
+MYSQL_DB="your_database"
+```
+
+#### Windows 10
+
+On Windows, you must use TCP/IP connections:
+
+```bash
+MYSQL_HOST="127.0.0.1"
+MYSQL_PORT="3306"
+MYSQL_USER="root"
+MYSQL_PASS="your_password"
+MYSQL_DB="your_database"
+```
+
+**Windows-Specific Notes:**
+- If MySQL is running in WSL2, you may need to use the Windows host IP instead of `127.0.0.1`
+- To find WSL2 IP: Run `wsl hostname -I` in Windows Command Prompt or PowerShell
+- Ensure MySQL is configured to accept TCP/IP connections (check `bind-address` in `my.cnf` or `my.ini`)
+- Windows Firewall may block connections - ensure port 3306 is allowed
+
 #### Alternative: Connection String
 
 For scenarios requiring frequent credential rotation or temporary connections, you can use a MySQL connection string instead of individual environment variables:

package/dist/index.js

@@ -72,6 +72,10 @@ export const configSchema = z.object({
     debug: z.boolean().default(false).describe("Enable debug logging"),
 });
 export default function createMcpServer({ sessionId, config, }) {
+    const isDebugEnabled = config?.debug ?? false;
+    if (isDebugEnabled) {
+        log("info", "Debug mode enabled");
+    }
     const server = new Server({
         name: "MySQL MCP Server",
         version: process.env.npm_package_version || "1.0.0",
@@ -123,12 +127,17 @@ export default function createMcpServer({ sessionId, config, }) {
         information_schema.tables
       WHERE
         table_schema NOT IN ('information_schema', 'mysql', 'performance_schema', 'sys')
-        ${!isMultiDbMode && mcpConfig.mysql.database ? `AND table_schema = '${mcpConfig.mysql.database}'` : ''}
       ORDER BY
         table_schema, table_name
     `;
-            const queryResult = (await executeReadOnlyQuery(tablesQuery));
-            const tables = JSON.parse(queryResult.content[0].text);
+            let queryParams = [];
+            let finalQuery = tablesQuery;
+            if (!isMultiDbMode && mcpConfig.mysql.database) {
+                finalQuery = tablesQuery.replace('ORDER BY', `AND table_schema = ?\n      ORDER BY`);
+                queryParams.push(mcpConfig.mysql.database);
+            }
+            const rawResult = await executeQuery(finalQuery, queryParams);
+            const tables = rawResult[0];
             log("info", `Found ${tables.length} tables`);
             const resources = tables.map((table) => ({
                 uri: `mysql://tables/${table.name}`,
@@ -164,11 +173,18 @@ export default function createMcpServer({ sessionId, config, }) {
             let columnsQuery = "SELECT column_name, data_type FROM information_schema.columns WHERE table_name = ?";
             let queryParams = [tableName];
             const schemaName = dbName || (!isMultiDbMode ? mcpConfig.mysql.database : null);
+            if (!/^[a-zA-Z0-9_]+$/.test(tableName)) {
+                throw new Error(`Invalid table name: ${tableName}`);
+            }
+            if (schemaName && !/^[a-zA-Z0-9_]+$/.test(schemaName)) {
+                throw new Error(`Invalid schema name: ${schemaName}`);
+            }
             if (schemaName) {
                 columnsQuery += " AND table_schema = ?";
                 queryParams.push(schemaName);
             }
-            const results = (await executeQuery(columnsQuery, queryParams));
+            const rawResult = await executeQuery(columnsQuery, queryParams);
+            const results = rawResult[0];
             return {
                 contents: [
                     {
@@ -306,7 +322,7 @@ if (isMainModule()) {
     log("info", "Running in standalone mode");
     (async () => {
         try {
-            const mcpServer = createMcpServer({ config: { debug: false } });
+            const mcpServer = createMcpServer({});
             if (IS_REMOTE_MCP && REMOTE_SECRET_KEY?.length) {
                 const app = express();
                 app.use(express.json());

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cainli/mcp-server-mysql",
-  "version": "2.0.8",
+  "version": "2.0.10",
   "description": "MCP server for interacting with MySQL databases with write operations support",
   "license": "MIT",
   "author": "cainli (https://github.com/cainli)",
@@ -16,30 +16,6 @@
     "README.md",
     "assets"
   ],
-  "scripts": {
-    "start": "node dist/index.js",
-    "dev": "ts-node index.ts",
-    "build": "tsc && shx chmod +x dist/*.js",
-    "prepare": "npm run build",
-    "watch": "tsc --watch",
-    "setup:test:db": "tsx scripts/setup-test-db.ts",
-    "pretest": "pnpm run setup:test:db",
-    "test": "pnpm run setup:test:db && vitest run",
-    "test:socket": "pnpm run setup:test:db && vitest run tests/integration/socket-connection.test.ts",
-    "test:watch": "pnpm run setup:test:db && vitest",
-    "test:coverage": "vitest run --coverage",
-    "test:unit": "vitest run --config vitest.unit.config.ts",
-    "test:integration": "vitest run --config vitest.integration.config.ts",
-    "test:e2e": "vitest run --config vitest.e2e.config.ts",
-    "stdio": "node dist/index.js --stdio",
-    "exec": " pnpm build && npx node --env-file=.env dist/index.js",
-    "lint": "npm run lint:eslint && npm run lint:markdown",
-    "lint:eslint": "eslint .",
-    "lint:markdown": "markdownlint .",
-    "lint:fix": "npm run lint:eslint:fix && npm run lint:markdown:fix",
-    "lint:eslint:fix": "eslint . --fix",
-    "lint:markdown:fix": "markdownlint . --fix"
-  },
   "dependencies": {
     "@ai-sdk/openai": "^1.3.22",
     "@modelcontextprotocol/sdk": "1.15.1",
@@ -83,5 +59,29 @@
     "mcp-patch",
     "mcp-options",
     "mcp-head"
-  ]
-}
+  ],
+  "scripts": {
+    "start": "node dist/index.js",
+    "dev": "ts-node index.ts",
+    "build": "tsc && npm run chmod:win",
+    "chmod:win": "node -e \"const os=require('os');if(os.platform()!=='win32'){const fs=require('fs');const dist='dist';const files=fs.readdirSync(dist).filter(f=>f.endsWith('.js'));files.forEach(f=>fs.chmodSync(dist+'/'+f,0o755));console.log('Made executable:',files.join(', '))}\"",
+    "watch": "tsc --watch",
+    "setup:test:db": "tsx scripts/setup-test-db.ts",
+    "pretest": "pnpm run setup:test:db",
+    "test": "pnpm run setup:test:db && vitest run",
+    "test:socket": "pnpm run setup:test:db && vitest run tests/integration/socket-connection.test.ts",
+    "test:watch": "pnpm run setup:test:db && vitest",
+    "test:coverage": "vitest run --coverage",
+    "test:unit": "vitest run --config vitest.unit.config.ts",
+    "test:integration": "vitest run --config vitest.integration.config.ts",
+    "test:e2e": "vitest run --config vitest.e2e.config.ts",
+    "stdio": "node dist/index.js --stdio",
+    "exec": " pnpm build && npx node --env-file=.env dist/index.js",
+    "lint": "npm run lint:eslint && npm run lint:markdown",
+    "lint:eslint": "eslint .",
+    "lint:markdown": "markdownlint .",
+    "lint:fix": "npm run lint:eslint:fix && npm run lint:markdown:fix",
+    "lint:eslint:fix": "eslint . --fix",
+    "lint:markdown:fix": "markdownlint . --fix"
+  }
+}