Merged
Conversation
Contributor
Author
|
/next |
Contributor
|
🎉 PR Next publish successful! 3.8.3-next-1743647111.0 |
Contributor
## Walkthrough
本次更改为 MCP 服务器管理相关接口和类新增了按名称检索服务器及获取客户端实例的能力。具体来说，在多个接口和类中分别添加了 `$getMCPServerByName`、`getServerByName` 和 `getClient` 方法。更新后的接口允许通过服务器名称检索对应的 MCP 服务器，同时可获取与服务器关联的客户端对象，现有功能得以保持，功能范围得到扩展。此外，SSEMCPServer 构造函数新增了可选的传输选项参数，用于配置 SSE 客户端传输。
## Changes
| 文件 | 变更摘要 |
| --- | --- |
| `packages/ai-native/src/common/index.ts` | 更新 mcp-server-manager 模块的导入语句，引入 `IMCPServer`；在 `ISumiMCPServerBackend` 接口中新增 `$getMCPServerByName(name: string)` 方法 |
| `packages/ai-native/src/common/mcp-server-manager.ts`<br>`packages/ai-native/src/common/types.ts`<br>`packages/ai-native/src/node/mcp-server-manager-impl.ts` | 在 `IMCPServer` 接口中新增 `getClient(): Client \| null` 方法；在 `MCPServerManager` 接口及其实现中新增 `getServerByName(name: string): IMCPServer \| undefined` 方法；新增接口 `SSEClientTransportOptions` 并在 `SSEMCPServerDescription` 中新增可选属性 `transportOptions` |
| `packages/ai-native/src/node/mcp-server.sse.ts`<br>`packages/ai-native/src/node/mcp-server.stdio.ts`<br>`packages/ai-native/src/node/mcp/sumi-mcp-server.ts` | `SSEMCPServer` 构造函数新增可选参数 `options?: SSEClientTransportOptions`，并在 `start` 方法中使用该选项实例化 `SSEClientTransport`；`SSEMCPServer`、`StdioMCPServer`、`BuiltinMCPServer` 类中新增 `getClient(): Client \| null` 方法；`SumiMCPServerBackend` 类新增 `$getMCPServerByName(name: string)` 方法 |
## Sequence Diagram(s)
```mermaid
sequenceDiagram
participant C as 调用方
participant B as SumiMCPServerBackend
participant M as MCPServerManagerImpl
participant S as IMCPServer
C->>B: 调用 $getMCPServerByName(name)
B->>M: 调用 getServerByName(name)
M-->>B: 返回 IMCPServer 实例或 undefined
B-->>C: 返回检索结果Possibly related PRs
Suggested reviewers
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
packages/ai-native/src/node/mcp-server.sse.ts (1)
30-32: 实现访问客户端实例的方法新增的
getClient()方法为外部提供了访问 MCP 服务器客户端实例的能力，这是一个很好的扩展。考虑在返回
this.client前检查其初始化状态，因为当服务器未启动时，this.client可能为undefined。接口定义的返回类型是Client | null，但实际实现可能返回undefined。getClient(): Client | null { - return this.client; + return this.client || null; }packages/ai-native/src/node/mcp-server.stdio.ts (1)
39-41: 实现访问客户端实例的方法新增的
getClient()方法为外部提供了访问 MCP 服务器客户端实例的能力，这是一个很好的扩展。考虑在返回
this.client前检查其初始化状态，因为当服务器未启动时，this.client可能为undefined。接口定义的返回类型是Client | null，但实际实现可能返回undefined。getClient(): Client | null { - return this.client; + return this.client || null; }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
packages/ai-native/src/common/index.ts(2 hunks)packages/ai-native/src/common/mcp-server-manager.ts(2 hunks)packages/ai-native/src/node/mcp-server-manager-impl.ts(1 hunks)packages/ai-native/src/node/mcp-server.sse.ts(1 hunks)packages/ai-native/src/node/mcp-server.stdio.ts(1 hunks)packages/ai-native/src/node/mcp/sumi-mcp-server.ts(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (6)
- GitHub Check: build-windows
- GitHub Check: unittest (macos-latest, 18.x, jsdom)
- GitHub Check: unittest (ubuntu-latest, 18.x, jsdom)
- GitHub Check: unittest (ubuntu-latest, 18.x, node)
- GitHub Check: unittest (macos-latest, 18.x, node)
- GitHub Check: ubuntu-latest, Node.js 20.x
🔇 Additional comments (7)
packages/ai-native/src/node/mcp-server-manager-impl.ts (1)
95-97: 根据名称获取服务器的实现
getServerByName方法实现简洁明了，与整体架构设计一致。这个添加使得通过服务器名称直接检索服务器实例变得更加方便。packages/ai-native/src/common/mcp-server-manager.ts (2)
11-11: 接口定义 - 获取客户端方法为
IMCPServer接口添加getClient()方法是一个很好的扩展，提供了访问客户端实例的统一接口。返回类型定义为Client | null表示客户端可能不存在的情况。确保所有实现此接口的类都正确处理返回类型，特别是区分
null和undefined的情况。
36-36: 接口定义 - 根据名称获取服务器方法为
MCPServerManager接口添加getServerByName方法是一个有用的扩展，使得通过名称检索服务器变得更加直接。返回类型IMCPServer | undefined适当地表示了找不到服务器的情况。packages/ai-native/src/node/mcp/sumi-mcp-server.ts (2)
63-65: 新增按名称检索MCP服务器的方法实现该方法提供了按名称检索特定MCP服务器的能力，使客户端可以通过服务器名称获取对应的服务器实例。实现简洁有效，直接委托给mcpServerManager的getServerByName方法。
239-241: 为BuiltinMCPServer实现getClient方法为BuiltinMCPServer类实现了IMCPServer接口中定义的getClient方法。由于内置服务器不关联客户端实例，因此返回null是合理的。这与接口规范一致，保持了实现的完整性。
packages/ai-native/src/common/index.ts (2)
18-18: 更新导入声明以包含IMCPServer接口导入声明已更新，增加了对IMCPServer接口的导入，这是为了支持新添加的$getMCPServerByName方法的返回类型。更改简洁且必要。
154-154: 在ISumiMCPServerBackend接口中添加按名称获取MCP服务器的方法该方法增强了ISumiMCPServerBackend接口的功能，允许客户端通过名称检索特定的MCP服务器实例。方法签名清晰地表明如果找不到具有给定名称的服务器，将返回undefined。此更新保持了接口的一致性和功能性。
Contributor
Walkthrough此次提交涉及多个文件的更新，主要新增了通过名称获取 MCP 服务器实例的方法以及客户端获取方法。改动范围涵盖接口声明和类实现，包括在 Changes
Sequence Diagram(s)sequenceDiagram
participant Backend as SumiMCPServerBackend
participant Manager as MCPServerManager
participant Server as IMCPServer
Backend->>Manager: 调用 getServerByName(name)
Manager->>Server: 查询服务器实例
Server-->>Manager: 返回实例或 undefined
Manager-->>Backend: 返回实例或 undefined
Possibly related PRs
Suggested reviewers
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
yarn install v1.22.22 ✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Contributor
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
packages/ai-native/src/common/mcp-server-manager.ts (2)
11-11: 添加了 getClient 方法，但缺少文档说明此方法允许获取与 MCP 服务器关联的客户端实例，但缺少文档说明什么情况下会返回 null。建议添加注释解释该方法的用途和返回值的可能情况。
+ /** + * 获取与服务器关联的客户端实例。 + * 如果没有关联的客户端，则返回 null。 + */ getClient(): Client | null;
36-36: 添加了 getServerByName 方法，但缺少文档说明此方法允许通过名称检索特定的 MCP 服务器实例，但缺少文档说明当找不到指定名称的服务器时会发生什么情况。建议添加注释解释该方法的用途和可能的返回值。
+ /** + * 通过名称获取 MCP 服务器实例。 + * @param name 服务器名称 + * @returns 如果找到，则返回对应的服务器实例；否则返回 undefined + */ getServerByName(name: string): IMCPServer | undefined;packages/ai-native/src/node/mcp/sumi-mcp-server.ts (2)
63-65: 新增的 $getMCPServerByName 方法缺少文档说明这个新方法用于通过名称获取 MCP 服务器实例，但缺少文档说明其用途和行为。建议添加 JSDoc 注释以提高代码的可读性和可维护性。
+ /** + * 通过名称获取 MCP 服务器实例。 + * @param name 服务器名称 + * @returns 如果找到，则返回对应的服务器实例；否则返回 undefined + */ $getMCPServerByName(name: string) { return this.mcpServerManager.getServerByName(name); }
239-241: 内置服务器的 getClient 方法总是返回 nullBuiltinMCPServer 类中实现的 getClient 方法总是返回 null，这符合接口要求，但缺少注释解释为什么内置服务器没有关联的客户端。建议添加注释说明这一设计决策。
+ /** + * 获取与服务器关联的客户端实例。 + * 对于内置服务器，始终返回 null，因为它不需要外部客户端。 + * @returns null + */ getClient(): Client | null { return null; }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
packages/ai-native/src/common/index.ts(2 hunks)packages/ai-native/src/common/mcp-server-manager.ts(2 hunks)packages/ai-native/src/node/mcp-server-manager-impl.ts(1 hunks)packages/ai-native/src/node/mcp-server.sse.ts(1 hunks)packages/ai-native/src/node/mcp-server.stdio.ts(1 hunks)packages/ai-native/src/node/mcp/sumi-mcp-server.ts(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (6)
- GitHub Check: build-windows
- GitHub Check: unittest (macos-latest, 18.x, jsdom)
- GitHub Check: unittest (ubuntu-latest, 18.x, jsdom)
- GitHub Check: unittest (ubuntu-latest, 18.x, node)
- GitHub Check: unittest (macos-latest, 18.x, node)
- GitHub Check: ubuntu-latest, Node.js 20.x
🔇 Additional comments (5)
packages/ai-native/src/node/mcp-server.sse.ts (1)
30-32: 新增的 getClient 方法实现合理此方法提供了一种获取 SSEMCPServer 实例中 client 对象的方式，使外部组件能够直接访问和使用底层的 MCP 客户端。
建议考虑添加一些文档注释来说明此方法的用途和返回值的可能状态，特别是返回 null 的情况。
packages/ai-native/src/node/mcp-server.stdio.ts (1)
39-41: 新增的 getClient 方法实现合理此方法提供了一种获取 StdioMCPServer 实例中 client 对象的方式，与 SSEMCPServer 中的实现保持一致，增强了接口的一致性。
建议同样添加文档注释来说明此方法的用途和返回值的可能状态，特别是何时可能返回 null。
packages/ai-native/src/node/mcp-server-manager-impl.ts (1)
95-97: 新增的 getServerByName 方法实现合理此方法通过名称检索 IMCPServer 实例，增强了服务器管理器的功能。实现简洁明了，使用 Map 的 get 方法高效查询。
建议添加简单的文档注释说明方法用途，尤其是 undefined 返回值的含义和处理方式。
packages/ai-native/src/common/index.ts (2)
154-154: 新增的 $getMCPServerByName 方法声明合理此接口方法声明允许在 ISumiMCPServerBackend 中根据名称获取 MCP 服务器实例，是对现有功能的良好补充。
建议添加 JSDoc 注释以说明参数和返回值，特别是解释返回 undefined 的情况及调用方应如何处理。
18-18: 导入 IMCPServer 类型的更改适当地更新了导入语句以包含 IMCPServer 类型，这是对新增 $getMCPServerByName 方法的必要支持。
Contributor
Author
|
/next |
Contributor
|
🎉 PR Next publish successful! 3.8.3-next-1744856097.0 |
Member
|
CI 修一下 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4505 +/- ##
==========================================
- Coverage 53.13% 53.12% -0.01%
==========================================
Files 1665 1665
Lines 102648 102707 +59
Branches 22216 22240 +24
==========================================
+ Hits 54543 54568 +25
- Misses 40019 40047 +28
- Partials 8086 8092 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Member
|
/next |
Contributor
|
🎉 PR Next publish successful! 3.8.3-next-1745295615.0 |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Types
Background or solution
Changelog
Summary by CodeRabbit