mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-05-02 16:57:36 +08:00
Broad drift audit against origin/main (b52b63396).
Reference pages (most user-visible drift):
- slash-commands: add /busy, /curator, /footer, /indicator, /redraw, /steer
that were missing; drop non-existent /terminal-setup; fix /q footnote
(resolves to /queue, not /quit); extend CLI-only list with all 24
CLI-only commands in the registry
- cli-commands: add dedicated sections for hermes curator / fallback /
hooks (new subcommands not previously documented); remove stale
hermes honcho standalone section (the plugin registers dynamically
via hermes memory); list curator/fallback/hooks in top-level table;
fix completion to include fish
- toolsets-reference: document the real 52-toolset count; split browser
vs browser-cdp; add discord / discord_admin / spotify / yuanbao;
correct hermes-cli tool count from 36 to 38; fix misleading claim
that hermes-homeassistant adds tools (it's identical to hermes-cli)
- tools-reference: bump tool count 55 -> 68; add 7 Spotify, 5 Yuanbao,
2 Discord toolsets; move browser_cdp/browser_dialog to their own
browser-cdp toolset section
- environment-variables: add 40+ user-facing HERMES_* vars that were
undocumented (--yolo, --accept-hooks, --ignore-*, inference model
override, agent/stream/checkpoint timeouts, OAuth trace, per-platform
batch tuning for Telegram/Discord/Matrix/Feishu/WeCom, cron knobs,
gateway restart/connect timeouts); dedupe the Cron Scheduler section;
replace stale QQ_SANDBOX with QQ_PORTAL_HOST
User-guide (top level):
- cli.md: compression preserves last 20 turns, not 4 (protect_last_n: 20)
- configuration.md: display.platforms is the canonical per-platform
override key; tool_progress_overrides is deprecated and auto-migrated
- profiles.md: model.default is the config key, not model.model
- sessions.md: CLI/TUI session IDs use 6-char hex, gateway uses 8
- checkpoints-and-rollback.md: destructive-command list now matches
_DESTRUCTIVE_PATTERNS (adds rmdir, cp, install, dd)
- docker.md: the container runs as non-root hermes (UID 10000) via
gosu; fix install command (uv pip); add missing --insecure on the
dashboard compose example (required for non-loopback bind)
- security.md: systemctl danger pattern also matches 'restart'
- index.md: built-in tool count 47 -> 68
- integrations/index.md: 6 STT providers, 8 memory providers
- integrations/providers.md: drop fictional dashscope/qwen aliases
Features:
- overview.md: 9 image models (not 8), 9 TTS providers (not 5),
8 memory providers (Supermemory was missing)
- tool-gateway.md: 9 image models
- tools.md: extend common-toolsets list with search / messaging /
spotify / discord / debugging / safe
- fallback-providers.md: add 6 real providers from PROVIDER_REGISTRY
(lmstudio, kimi-coding-cn, stepfun, alibaba-coding-plan,
tencent-tokenhub, azure-foundry)
- plugins.md: Available Hooks table now includes on_session_finalize,
on_session_reset, subagent_stop
- built-in-plugins.md: add the 7 bundled plugins the page didn't
mention (spotify, google_meet, three image_gen providers, two
dashboard examples)
- web-dashboard.md: add --insecure and --tui flags
- cron.md: hermes cron create takes positional schedule/prompt, not
flags
Messaging:
- telegram.md: TELEGRAM_WEBHOOK_SECRET is now REQUIRED when
TELEGRAM_WEBHOOK_URL is set (gateway refuses to start without it
per GHSA-3vpc-7q5r-276h). Biggest user-visible drift in the batch.
- discord.md: HERMES_DISCORD_TEXT_BATCH_SPLIT_DELAY_SECONDS default
is 2.0, not 0.1
- dingtalk.md: document DINGTALK_REQUIRE_MENTION /
FREE_RESPONSE_CHATS / MENTION_PATTERNS / HOME_CHANNEL /
ALLOW_ALL_USERS that the adapter supports
- bluebubbles.md: drop fictional BLUEBUBBLES_SEND_READ_RECEIPTS env
var; the setting lives in platforms.bluebubbles.extra only
- qqbot.md: drop dead QQ_SANDBOX; add real QQ_PORTAL_HOST and
QQ_GROUP_ALLOWED_USERS
- wecom-callback.md: replace 'hermes gateway start' (service-only)
with 'hermes gateway' for first-time setup
Developer-guide:
- architecture.md: refresh tool/toolset counts (61/52), terminal
backend count (7), line counts for run_agent.py (~13.7k), cli.py
(~11.5k), main.py (~10.4k), setup.py (~3.5k), gateway/run.py
(~12.2k), mcp_tool.py (~3.1k); add yuanbao adapter, bump platform
adapter count 18 -> 20
- agent-loop.md: run_agent.py line count 10.7k -> 13.7k
- tools-runtime.md: add vercel_sandbox backend
- adding-tools.md: remove stale 'Discovery import added to
model_tools.py' checklist item (registry auto-discovery)
- adding-platform-adapters.md: mark send_typing / get_chat_info as
concrete base methods; only connect/disconnect/send are abstract
- acp-internals.md: ACP sessions now persist to SessionDB
(~/.hermes/state.db); acp.run_agent call uses
use_unstable_protocol=True
- cron-internals.md: gateway runs scheduler in a dedicated background
thread via _start_cron_ticker, not on a maintenance cycle; locking
is cross-process via fcntl.flock (Unix) / msvcrt.locking (Windows)
- gateway-internals.md: gateway/run.py ~12k lines
- provider-runtime.md: cron DOES support fallback (run_job reads
fallback_providers from config)
- session-storage.md: SCHEMA_VERSION = 11 (not 9); add migrations
10 and 11 (trigram FTS, inline-mode FTS5 re-index); add
api_call_count column to Sessions DDL; document messages_fts_trigram
and state_meta in the architecture tree
- context-compression-and-caching.md: remove the obsolete 'context
pressure warnings' section (warnings were removed for causing
models to give up early)
- context-engine-plugin.md: compress() signature now includes
focus_topic param
- extending-the-cli.md: _build_tui_layout_children signature now
includes model_picker_widget; add to default layout
Also fixed three pre-existing broken links/anchors the build warned
about (docker.md -> api-server.md, yuanbao.md -> cron-jobs.md and
tips#background-tasks, nix-setup.md -> #container-aware-cli).
Regenerated per-skill pages via website/scripts/generate-skill-docs.py
so catalog tables and sidebar are consistent with current SKILL.md
frontmatter.
docusaurus build: clean, no broken links or anchors.
199 lines
5.9 KiB
Markdown
199 lines
5.9 KiB
Markdown
---
|
|
sidebar_position: 2
|
|
title: "Adding Tools"
|
|
description: "How to add a new tool to Hermes Agent — schemas, handlers, registration, and toolsets"
|
|
---
|
|
|
|
# Adding Tools
|
|
|
|
Before writing a tool, ask yourself: **should this be a [skill](creating-skills.md) instead?**
|
|
|
|
Make it a **Skill** when the capability can be expressed as instructions + shell commands + existing tools (arXiv search, git workflows, Docker management, PDF processing).
|
|
|
|
Make it a **Tool** when it requires end-to-end integration with API keys, custom processing logic, binary data handling, or streaming (browser automation, TTS, vision analysis).
|
|
|
|
## Overview
|
|
|
|
Adding a tool touches **2 files**:
|
|
|
|
1. **`tools/your_tool.py`** — handler, schema, check function, `registry.register()` call
|
|
2. **`toolsets.py`** — add tool name to `_HERMES_CORE_TOOLS` (or a specific toolset)
|
|
|
|
Any `tools/*.py` file with a top-level `registry.register()` call is auto-discovered at startup — no manual import list required.
|
|
|
|
## Step 1: Create the Tool File
|
|
|
|
Every tool file follows the same structure:
|
|
|
|
```python
|
|
# tools/weather_tool.py
|
|
"""Weather Tool -- look up current weather for a location."""
|
|
|
|
import json
|
|
import os
|
|
import logging
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
|
|
# --- Availability check ---
|
|
|
|
def check_weather_requirements() -> bool:
|
|
"""Return True if the tool's dependencies are available."""
|
|
return bool(os.getenv("WEATHER_API_KEY"))
|
|
|
|
|
|
# --- Handler ---
|
|
|
|
def weather_tool(location: str, units: str = "metric") -> str:
|
|
"""Fetch weather for a location. Returns JSON string."""
|
|
api_key = os.getenv("WEATHER_API_KEY")
|
|
if not api_key:
|
|
return json.dumps({"error": "WEATHER_API_KEY not configured"})
|
|
try:
|
|
# ... call weather API ...
|
|
return json.dumps({"location": location, "temp": 22, "units": units})
|
|
except Exception as e:
|
|
return json.dumps({"error": str(e)})
|
|
|
|
|
|
# --- Schema ---
|
|
|
|
WEATHER_SCHEMA = {
|
|
"name": "weather",
|
|
"description": "Get current weather for a location.",
|
|
"parameters": {
|
|
"type": "object",
|
|
"properties": {
|
|
"location": {
|
|
"type": "string",
|
|
"description": "City name or coordinates (e.g. 'London' or '51.5,-0.1')"
|
|
},
|
|
"units": {
|
|
"type": "string",
|
|
"enum": ["metric", "imperial"],
|
|
"description": "Temperature units (default: metric)",
|
|
"default": "metric"
|
|
}
|
|
},
|
|
"required": ["location"]
|
|
}
|
|
}
|
|
|
|
|
|
# --- Registration ---
|
|
|
|
from tools.registry import registry
|
|
|
|
registry.register(
|
|
name="weather",
|
|
toolset="weather",
|
|
schema=WEATHER_SCHEMA,
|
|
handler=lambda args, **kw: weather_tool(
|
|
location=args.get("location", ""),
|
|
units=args.get("units", "metric")),
|
|
check_fn=check_weather_requirements,
|
|
requires_env=["WEATHER_API_KEY"],
|
|
)
|
|
```
|
|
|
|
### Key Rules
|
|
|
|
:::danger Important
|
|
- Handlers **MUST** return a JSON string (via `json.dumps()`), never raw dicts
|
|
- Errors **MUST** be returned as `{"error": "message"}`, never raised as exceptions
|
|
- The `check_fn` is called when building tool definitions — if it returns `False`, the tool is silently excluded
|
|
- The `handler` receives `(args: dict, **kwargs)` where `args` is the LLM's tool call arguments
|
|
:::
|
|
|
|
## Step 2: Add to a Toolset
|
|
|
|
In `toolsets.py`, add the tool name:
|
|
|
|
```python
|
|
# If it should be available on all platforms (CLI + messaging):
|
|
_HERMES_CORE_TOOLS = [
|
|
...
|
|
"weather", # <-- add here
|
|
]
|
|
|
|
# Or create a new standalone toolset:
|
|
"weather": {
|
|
"description": "Weather lookup tools",
|
|
"tools": ["weather"],
|
|
"includes": []
|
|
},
|
|
```
|
|
|
|
## ~~Step 3: Add Discovery Import~~ (No longer needed)
|
|
|
|
Tool modules with a top-level `registry.register()` call are auto-discovered by `discover_builtin_tools()` in `tools/registry.py`. No manual import list to maintain — just create your file in `tools/` and it's picked up at startup.
|
|
|
|
## Async Handlers
|
|
|
|
If your handler needs async code, mark it with `is_async=True`:
|
|
|
|
```python
|
|
async def weather_tool_async(location: str) -> str:
|
|
async with aiohttp.ClientSession() as session:
|
|
...
|
|
return json.dumps(result)
|
|
|
|
registry.register(
|
|
name="weather",
|
|
toolset="weather",
|
|
schema=WEATHER_SCHEMA,
|
|
handler=lambda args, **kw: weather_tool_async(args.get("location", "")),
|
|
check_fn=check_weather_requirements,
|
|
is_async=True, # registry calls _run_async() automatically
|
|
)
|
|
```
|
|
|
|
The registry handles async bridging transparently — you never call `asyncio.run()` yourself.
|
|
|
|
## Handlers That Need task_id
|
|
|
|
Tools that manage per-session state receive `task_id` via `**kwargs`:
|
|
|
|
```python
|
|
def _handle_weather(args, **kw):
|
|
task_id = kw.get("task_id")
|
|
return weather_tool(args.get("location", ""), task_id=task_id)
|
|
|
|
registry.register(
|
|
name="weather",
|
|
...
|
|
handler=_handle_weather,
|
|
)
|
|
```
|
|
|
|
## Agent-Loop Intercepted Tools
|
|
|
|
Some tools (`todo`, `memory`, `session_search`, `delegate_task`) need access to per-session agent state. These are intercepted by `run_agent.py` before reaching the registry. The registry still holds their schemas, but `dispatch()` returns a fallback error if the intercept is bypassed.
|
|
|
|
## Optional: Setup Wizard Integration
|
|
|
|
If your tool requires an API key, add it to `hermes_cli/config.py`:
|
|
|
|
```python
|
|
OPTIONAL_ENV_VARS = {
|
|
...
|
|
"WEATHER_API_KEY": {
|
|
"description": "Weather API key for weather lookup",
|
|
"prompt": "Weather API key",
|
|
"url": "https://weatherapi.com/",
|
|
"tools": ["weather"],
|
|
"password": True,
|
|
},
|
|
}
|
|
```
|
|
|
|
## Checklist
|
|
|
|
- [ ] Tool file created with handler, schema, check function, and registration
|
|
- [ ] Added to appropriate toolset in `toolsets.py`
|
|
- [ ] Handler returns JSON strings, errors returned as `{"error": "..."}`
|
|
- [ ] Optional: API key added to `OPTIONAL_ENV_VARS` in `hermes_cli/config.py`
|
|
- [ ] Optional: Added to `toolset_distributions.py` for batch processing
|
|
- [ ] Tested with `hermes chat -q "Use the weather tool for London"`
|