-
Notifications
You must be signed in to change notification settings - Fork 40
添加 Nacos MCP 客户端和服务端模块 #32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
- 将 spring.ai.openai.chat.api-key 更名为 spring.ai.openai.api-key - 统一 API 密钥配置,解决潜在的配置错误问题
- 新增 spring-ai-observability 模块,包含 metric 和 tracing 两个子模块 - 实现了对 AI 模型调用的监控、追踪和可观测性管理 - 集成了 OpenAI 客户端、Redis VectorStore 和 Zipkin 分布式追踪 - 提供了聊天、图像生成、文本嵌入等功能的 REST 接口 - 添加了 Prometheus 指标和健康检查支持
- 将 image 方法的返回类型从 void改为 String - 移除了 HttpServletResponse 参数 - 直接返回图像 URL 字符串,而不是通过输出流传输图像数据
- 在 metric 和 tracing模块的 README 文件中,将 application.yml 的路径从绝对路径修改为相对路径 - 优化文档的可读性和通用性,避免特定于某个用户的路径
- 新增 Nacos MCP 客户端模块,实现基于 Nacos 的服务发现和 MCP 协议通信 - 新增 Nacos MCP 服务端模块,提供天气查询工具服务 - 添加相关配置文件、控制器、服务和工具类 - 更新项目结构,增加新模块的引用
glmapper
requested changes
Jun 30, 2025
...mcp/nacos-mcp-client/src/main/java/com/glmapper/ai/nacos/client/NacoseClientApplication.java
Outdated
Show resolved
Hide resolved
- 移除了 Nacos客户端和服务器应用中的未使用导入 - 删除了 Nacos 客户端应用中的冗余代码片段 - 清理了 Nacos 服务器应用中的多余导入和代码
- 将 OpenAI API 密钥配置从硬编码改为使用环境变量 - 提高了配置的灵活性和安全性
- 将 NacoseClientApplication 重命名为 NacosClientApplication - 将 NacoseServerApplication 重命名为 NacosServerApplication - 更新相关 README 文件中的类名引用
glmapper
approved these changes
Jul 1, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
#2