Skip to content

fix(hermes): advertise memory tools before gateway init#220

Open
RerankerGuo wants to merge 1 commit into
TencentCloud:mainfrom
RerankerGuo:fix/hermes-tools-auto-discovery
Open

fix(hermes): advertise memory tools before gateway init#220
RerankerGuo wants to merge 1 commit into
TencentCloud:mainfrom
RerankerGuo:fix/hermes-tools-auto-discovery

Conversation

@RerankerGuo

Copy link
Copy Markdown
Contributor

Description | 描述

Advertise the Hermes memory_tencentdb search tool schemas before Gateway initialization so Hermes can build its tool routing table correctly in zero-config auto-discovery mode.

Runtime tool calls still go through the existing Gateway availability checks and return a transient unavailable error while the sidecar is starting or disconnected.

Related Issue | 关联 Issue

Fix #219

Change Type | 修改类型

  • Bug fix | Bug 修复
  • New feature | 新功能
  • Documentation update | 文档更新
  • Code optimization | 代码优化

Self-test Checklist | 自测清单

  • Verified locally | 本地验证通过
  • No existing features affected | 无影响现有功能

Additional Notes | 其他说明

Verified with:

  • npm test
  • npm run build
  • Python smoke test for pre-initialize get_tool_schemas() with no Gateway env vars

python3 -m pytest hermes-plugin/memory/memory_tencentdb/tests/test_memory_tencentdb_recovery.py could not be run in this environment because pytest is not installed.

Signed-off-by: Ziyang Guo <121015044+RerankerGuo@users.noreply.github.com>
@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

Thanks for the PR! We'll review the tool registration fix.

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.

memory_tencentdb 被注册到了Hermes,但是可用工具为0

2 participants