phernandez bfee8958ad docs(integrations): document the defensive getattr in the Hermes provider ...

Addresses the remaining review finding (deferred while #859 was open; now merged,
so the hermes module is on main and the comment lands cleanly here). The two
getattr sites read MCP SDK result/tool fields; `mcp` is an unpinned dependency
(plugin.yaml), and its CallToolResult / ListToolsResult / Tool shapes have varied
across SDK versions — so the defensive getattr is intentional, not the speculative
kind CLAUDE.md warns against. Added constraint comments at both sites explaining why.

No behavior change. `just package-check-hermes` passes (244 passed, 12 integration
tests skipped without BM_INTEGRATION).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>

2026-05-31 12:31:08 -05:00

..

hermes

docs(integrations): document the defensive getattr in the Hermes provider

2026-05-31 12:31:08 -05:00

openclaw

chore(integrations): consolidate agent packages

2026-05-30 14:16:04 -05:00