Skip to content

docs: fix Hermes integration - use mcp_servers instead of skills config - #954

Open
richardiitse wants to merge 1 commit into
MemPalace:developfrom
richardiitse:fix/hermes-guide-mcp-integration
Open

docs: fix Hermes integration - use mcp_servers instead of skills config#954
richardiitse wants to merge 1 commit into
MemPalace:developfrom
richardiitse:fix/hermes-guide-mcp-integration

Conversation

@richardiitse

Copy link
Copy Markdown

Summary

Fixes the Hermes Agent integration guide which documented an incorrect configuration approach.

Problem

The HERMES_MEMPALACE_GUIDE.md documented a skills: configuration method, but Hermes Agent only exposes tools via:

  1. SKILL.md (YAML frontmatter + Markdown) — properly parsed, tools exposed
  2. skill.py (Python class) — loaded as metadata only, not exposed as tools

The documented skills: approach falls into category 2, meaning no mempalace_* tools would actually appear in the agent's tool list despite the skill being loaded.

Solution

  1. New integrations/hermes/SKILL.md — MCP-based skill file in the Hermes-integrations pattern, documents all 33 available tools and usage protocol
  2. Rewritten HERMES_MEMPALACE_GUIDE.md — replaces skills: config with correct mcp_servers: configuration, full tool reference, and practical usage examples

Changes

  • integrations/hermes/SKILL.md (new) — Hermes skill file for MCP-based integration
  • HERMES_MEMPALACE_GUIDE.md (rewritten) — corrected integration instructions + full tool reference

Testing

The fix was validated on a live Hermes + MemPalace installation:

  • MCP server starts correctly
  • All 33 mempalace_* tools are discovered and available
  • Semantic search, knowledge graph, and drawer operations all functional

Closes #953

@igorls igorls added the documentation Improvements or additions to documentation label Apr 24, 2026
@igorls
igorls changed the base branch from main to develop May 17, 2026 19:35
@igorls

igorls commented Jun 24, 2026

Copy link
Copy Markdown
Member

Thanks for the Hermes docs contribution. I reviewed this along with #1684 during the agent-support pass.

The correction from skills config to mcp_servers is useful, but this doc is now stale against the current project:

  • The MCP tool count and examples need to match the current reference docs.
  • We should avoid a root-level HERMES_MEMPALACE_GUIDE.md; this should live under integrations/hermes/ or website/guide/ with the rest of the integration docs.
  • The setup should use the current recommended command/config style and mention local-first/privacy constraints precisely.
  • If feat(integrations): Hermes memory provider (revives #3, ABC-current + dim-fix) #1684 is revived, this guide should be folded into that integration work rather than landing as a separate parallel Hermes doc.

Happy to merge a refreshed Hermes guide once it is aligned with the current MCP surface and location conventions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: HERMES_MEMPALACE_GUIDE.md describes incorrect Hermes integration method

2 participants