Skip to content

feat(hiagent): restructure into knowledge-scoped MCP server with capa… - #430

Merged
cuericlee merged 1 commit into
volcengine:mainfrom
yooyea:pr/hiagent-knowledge-mcp
Sep 8, 2026
Merged

feat(hiagent): restructure into knowledge-scoped MCP server with capa…#430
cuericlee merged 1 commit into
volcengine:mainfrom
yooyea:pr/hiagent-knowledge-mcp

Conversation

@yooyea

@yooyea yooyea commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

What & Why

首版 HiAgent MCP Server(server/mcp_server_hiagent/,4 个工具)已合入 #424
本 PR 将其重构为聚焦「知识库(知识引擎)」的独立 server:删除旧
mcp_server_hiagent/,新增 mcp_server_hiagent_knowledge/

Changes

  • 目录/包/命令改名:mcp_server_hiagentmcp_server_hiagent_knowledge
  • 工具集扩展为 11 个:
    • 知识库发现:list_datasetsget_dataset
    • 检索取证:search_knowledgegrep_knowledge_chunkslist_document_infoslist_document_chunks
    • Wiki 溯源:search_wikiread_wiki_pageread_wiki_source_chunkread_wiki_source_doc
    • 运维自检:health_check
  • 薄适配架构:MCP 层仅校验必填字段与类型,数值范围/枚举/跨字段规则交由后端服务(错误原样透传)
  • 工具描述优化:以"建议/提示"语气传达工具间的接力关系,便于消费方 Agent 编排
  • 提供中英双份 README 与完整单元测试

Testing

  • 72 个单元测试全部通过(pytest,无需网络/凭证)
  • 已通过 MCP Inspector 对 11 个工具做真实端到端调用验证

@yooyea
yooyea force-pushed the pr/hiagent-knowledge-mcp branch 2 times, most recently from 23095b0 to d8f4c16 Compare September 7, 2026 15:12

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

必须修】uv.lock 未随包重命名更新
新目录 uv.lock 是旧 lock 直接复制,第 683 行仍是 name = "mcp-server-hiagent",与 pyproject.toml 的新包名 mcp-server-hiagent-knowledge 不一致。需在新目录执行 uv lock 重新生成。

@cuericlee cuericlee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@yooyea
yooyea force-pushed the pr/hiagent-knowledge-mcp branch from d8f4c16 to 5d12b88 Compare September 8, 2026 07:20
@cuericlee
cuericlee merged commit 0be1ba5 into volcengine:main Sep 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants