fix: missing mcp logger config (#4413)

* fix: mcp logger

* fix: lint

Author: czy88840616

packages/mcp/src/configuration.ts

@@ -19,6 +19,13 @@ import { MidwayMCPFramework } from './framework';
             messages: '/messages',
           },
         },
+        midwayLogger: {
+          clients: {
+            mcpLogger: {
+              fileLogName: 'midway-mcp.log',
+            },
+          },
+        },
       },
     },
   ],

packages/mcp/test/index.test.ts

@@ -674,4 +674,164 @@ describe('/test/index.test.ts', () => {
       }
     }
   });
+
+  it('should test Tool with complex parameters', async () => {
+    const clientManager = new SSEMCPClientManager();
+    let app: any;
+
+    try {
+      // Start the app server with Tool that requires complex parameters
+      @mcp.Tool('complexParamTool', {
+        description: 'Tool that accepts complex parameters including nested objects and arrays',
+        inputSchema: {
+          user: z.object({
+            name: z.string().describe('User name'),
+            age: z.number().describe('User age'),
+            email: z.string().email().describe('User email')
+          }).describe('User information'),
+          preferences: z.array(z.string()).describe('User preferences'),
+          metadata: z.object({
+            source: z.string().describe('Data source'),
+            timestamp: z.string().optional().describe('Optional timestamp')
+          }).optional().describe('Optional metadata'),
+          action: z.enum(['create', 'update', 'delete']).describe('Action to perform')
+        }
+      })
+      class ComplexParamTool implements mcp.IMcpTool {
+        name = 'complexParamTool';
+
+        async execute(args: any): Promise<CallToolResult> {
+          const { user, preferences, metadata, action } = args;
+
+          const result = {
+            action,
+            user: `${user.name} (${user.age}, ${user.email})`,
+            preferences: preferences.join(', '),
+            metadata: metadata ? `Source: ${metadata.source}${metadata.timestamp ? `, Time: ${metadata.timestamp}` : ''}` : 'No metadata'
+          };
+
+          return {
+            content: [{
+              type: 'text',
+              text: `Action: ${result.action}\nUser: ${result.user}\nPreferences: ${result.preferences}\nMetadata: ${result.metadata}`
+            }]
+          };
+        }
+      }
+
+      app = await createApp({
+        baseDir: null,
+        imports: [
+          express,
+          mcp,
+        ],
+        preloadModules: [
+          ComplexParamTool,
+        ],
+        globalConfig: {
+          express: {
+            keys: ['test'],
+            port: 7010,
+          },
+          mcp: {
+            serverInfo: {
+              name: 'test-complex-param-mcp',
+              version: '1.0.0',
+            },
+            transportType: 'stream-http',
+          }
+        },
+        loggerFactory: new DefaultConsoleLoggerFactory(),
+      });
+
+      // Create StreamHTTP MCP client
+      const client = await clientManager.createTestStreamHTTPClient('http://localhost:7010/mcp', 'test-complex-param-client');
+
+      // List tools
+      const { tools } = await client.listTools();
+      const complexTool = tools.find(tool => tool.name === 'complexParamTool');
+
+      if (!complexTool) {
+        throw new Error('complexParamTool should be available');
+      }
+
+      // Test with all parameters
+      const fullResult = await client.callTool({
+        name: 'complexParamTool',
+        arguments: {
+          user: {
+            name: 'John Doe',
+            age: 30,
+            email: 'john@example.com'
+          },
+          preferences: ['coding', 'reading', 'gaming'],
+          metadata: {
+            source: 'api',
+            timestamp: '2023-01-01T00:00:00Z'
+          },
+          action: 'create'
+        }
+      });
+
+      if (!fullResult.content || !fullResult.content[0]?.text?.includes('Action: create') ||
+          !fullResult.content[0]?.text?.includes('User: John Doe (30, john@example.com)') ||
+          !fullResult.content[0]?.text?.includes('Preferences: coding, reading, gaming') ||
+          !fullResult.content[0]?.text?.includes('Source: api, Time: 2023-01-01T00:00:00Z')) {
+        throw new Error('Tool should return expected result with full parameters');
+      }
+
+      // Test with optional parameter omitted
+      const partialResult = await client.callTool({
+        name: 'complexParamTool',
+        arguments: {
+          user: {
+            name: 'Jane Smith',
+            age: 25,
+            email: 'jane@example.com'
+          },
+          preferences: ['music', 'travel'],
+          action: 'update'
+        }
+      });
+
+      if (!partialResult.content || !partialResult.content[0]?.text?.includes('Action: update') ||
+          !partialResult.content[0]?.text?.includes('User: Jane Smith (25, jane@example.com)') ||
+          !partialResult.content[0]?.text?.includes('Preferences: music, travel') ||
+          !partialResult.content[0]?.text?.includes('Metadata: No metadata')) {
+        throw new Error('Tool should work with optional parameters omitted');
+      }
+
+      // Test with partial metadata (timestamp omitted)
+      const partialMetadataResult = await client.callTool({
+        name: 'complexParamTool',
+        arguments: {
+          user: {
+            name: 'Bob Wilson',
+            age: 35,
+            email: 'bob@example.com'
+          },
+          preferences: ['sports'],
+          metadata: {
+            source: 'database'
+          },
+          action: 'delete'
+        }
+      });
+
+      if (!partialMetadataResult.content || !partialMetadataResult.content[0]?.text?.includes('Action: delete') ||
+          !partialMetadataResult.content[0]?.text?.includes('User: Bob Wilson (35, bob@example.com)') ||
+          !partialMetadataResult.content[0]?.text?.includes('Preferences: sports') ||
+          !partialMetadataResult.content[0]?.text?.includes('Source: database') ||
+          partialMetadataResult.content[0]?.text?.includes('Time:')) {
+        throw new Error('Tool should work with partial metadata (timestamp omitted)');
+      }
+
+    } finally {
+      // Ensure cleanup always happens
+      await clientManager.closeAllClients();
+      if (app) {
+        await close(app);
+      }
+    }
+  });
 })

site/docs/extensions/mcp.md

@@ -37,13 +37,24 @@ $ npm i @midwayjs/mcp@4 --save
 $ npm i @modelcontextprotocol/sdk zod --save
 ```
 
+:::warning
+**⚠️ Zod 版本兼容性重要提示**
+
+MCP SDK (`@modelcontextprotocol/sdk`) 当前使用 Zod v3.x 版本。为确保兼容性，您的项目**必须**使用 Zod 3.x 版本，不要使用 Zod v4.x 或更高版本，否则会导致类型兼容性问题。
+
+**推荐安装特定版本**：
+```bash
+$ npm i zod@^3.24.1 --save
+```
+:::
+
 如果使用 `sse` 或 `stream-http` 传输类型，还需要安装 HTTP 框架组件（选择其中一个）：
 
 ```bash
 # Express 框架
 $ npm i @midwayjs/express@4 --save
 
-# 或者 Koa 框架  
+# 或者 Koa 框架
 $ npm i @midwayjs/koa@4 --save
 
 # 或者 Egg.js 框架
@@ -57,7 +68,7 @@ $ npm i @midwayjs/web@4 --save
   "dependencies": {
     "@midwayjs/mcp": "^4.0.0",
     "@modelcontextprotocol/sdk": "^1.19.0",
-    "zod": "^3.24.0",
+    "zod": "^3.24.1",
     "@midwayjs/express": "^4.0.0"
   }
 }
@@ -192,6 +203,21 @@ export default {
 **向下兼容说明**：当服务器配置为 `stream-http` 传输类型时，传统的 SSE 客户端仍然可以通过 `/sse` 端点正常连接和通信。
 :::
 
+### 日志配置
+
+MCP 组件默认志输出，可以通过 `mcpLogger` 进一步配置日志详情。
+
+```typescript
+// src/config/config.default.ts
+export default {
+  midwayLogger: {
+    mcpLogger: {
+      // ...
+    }
+  },
+}
+```
+
 ## 使用方式
 
 ### 工具（Tools）

site/i18n/en/docusaurus-plugin-content-docs/current/extensions/mcp.md

@@ -37,6 +37,17 @@ $ npm i @midwayjs/mcp@4 --save
 $ npm i @modelcontextprotocol/sdk zod --save
 ```
 
+:::warning
+Important Zod compatibility notice
+
+The MCP SDK (`@modelcontextprotocol/sdk`) currently uses Zod v3.x. To ensure compatibility, your project must use Zod 3.x. Do not use Zod v4.x or higher, otherwise type compatibility issues may occur.
+
+Recommended specific version:
+```bash
+$ npm i zod@^3.24.1 --save
+```
+:::
+
 If you use the `sse` or `stream-http` transport, you also need to install one of the HTTP framework components (choose one):
 
 ```bash
@@ -57,7 +68,7 @@ Or add the following dependencies to `package.json` and reinstall.
   "dependencies": {
     "@midwayjs/mcp": "^4.0.0",
     "@modelcontextprotocol/sdk": "^1.19.0",
-    "zod": "^3.24.0",
+    "zod": "^3.24.1",
     "@midwayjs/express": "^4.0.0"
   }
 }
@@ -192,6 +203,21 @@ Assuming the server runs at `http://localhost:3000`, the default endpoints are:
 Backward compatibility: When the server is configured with the `stream-http` transport, legacy SSE clients can still connect and communicate via the `/sse` endpoint.
 :::
 
+### Logging configuration
+
+The MCP component outputs logs by default. You can further configure logging details via `mcpLogger`.
+
+```typescript
+// src/config/config.default.ts
+export default {
+  midwayLogger: {
+    mcpLogger: {
+      // ...
+    }
+  },
+}
+```
+
 ## Usage
 
 ### Tools