Add Hermes Agent provider#386
Conversation
- Add Hermes Agent feature entry under Added (CLI) in CHANGELOG - Map delegate_task → Agent, vision_analyze → Vision, process → Bash, browser_get_images → Browser, skills_list → Skill - Add comment explaining composio MCP prefix precedence in mapToolName
buildPeriodData and per-session output use raw totalOutputTokens without adding reasoning tokens. buildJsonReport already handles this correctly via billableOutputTokens(). This is a pre-existing gap that also affects Claude — not introduced by the Hermes provider.
|
Thanks for contributing this. The provider structure is solid and test coverage is good. Found a few things that need fixing before we can merge: Must fix1. Cost fallback bug ( const actualCost = row.actual_cost_usd ?? row.estimated_cost_usd ?? 0
const costUSD =
(row.actual_cost_usd ?? 0) > 0 ? row.actual_cost_usd!
: (row.estimated_cost_usd ?? 0) > 0 ? row.estimated_cost_usd!
: calculatedCost2. The 3. Bumping from 3 to 4 forces a full cache rebuild for every user on upgrade (all providers, not just hermes). The cache schema did not change. The Should fix4. Discovery query has no LIMIT ( The 5. trimmed.replace(/^[/\\]/, '')Missing the |
Hey! I added Hermes Agent support to CodeBurn because I wanted to use it with my own Hermes sessions, and figured I’d share it in case it’s useful here too. No pressure at all if this isn’t the direction you want to take it.
Summary
Testing
npm test -- tests/providers/hermes.test.tsI also checked it against my real local Hermes data before opening this.
Closes #368