On a messaging chat with display.interim_assistant_messages off, a top-level delegate_task still produced two messages: a "checks are running, I don't have the findings yet" reply and, later, the real answer. Top-level model delegations always run in the background (ea8a8b4af8), and the tool text tells the model to "give a one-line status and END YOUR TURN" (c89f3b8800). That status is the turn's final answer, not interim text, so no display setting held it back. New key delegation.gateway_mode: auto | inline | background (alias async). auto (default) joins the children inside the turn on a messaging chat whose resolved interim setting is off (global, display.platforms.<p> override or platform default). The parent waits in the tool call, gets the results there and sends one answer. API, local and webhook surfaces keep the background path under auto. Chats with interim messages on are unchanged: same schema bytes, same async dispatch. How it works: - The gateway resolves the mode from the turn's own profile-scoped config when it builds a session's agent (gateway/session_execution.py), and the agent's first tool build freezes it (like the one-shot pin). - An inline session carries the joined delegate_task variant the one-shot path already uses: "results return in this tool call", no live-control parameters, no END YOUR TURN. - Dispatch follows the delegate_task schema the session carries (session_joins_delegation), so schema and behaviour cannot disagree. The schema rides the session's tools[] pin: a config change applies from the next session (/new), as delegation.backend does, and a cache eviction or restart never flips a running chat. - Inline children run on the existing synchronous path: attached to the parent, so /stop and an interrupting follow-up stop them; the per-child heartbeat keeps the parent's activity fresh for the inactivity watchdog; the async completion registry is not used. Nested and one-shot delegation are unchanged. Refs #95522.
17 KiB
tools/ + toolsets.py + model_tools.py — model tools
Applies on top of the root AGENTS.md: settle the Footprint Ladder before adding anything here.
Most capabilities should NOT be core tools. Long-form: website/docs/developer-guide/adding-tools.md,
tools-runtime.md.
Registry and discovery
tools/registry.py has no deps and is imported by every tool file; each tools/*.py calls
registry.register() at import time; model_tools.py imports the registry and triggers discovery
(discover_builtin_tools()), then run_agent.py, cli.py, batch_runner.py, environments/
consume it. Any tools/*.py with a top-level registry.register() is imported automatically — no
manual import list. A tool that is a whole package (tools/connectors/) registers from
tools/<pkg>/tool.py, the only file discovery scans inside a package; every sibling in the package
is a library by construction, and the package needs an __init__.py or discovery skips it with a
warning (setuptools would drop it from the wheel). The registry handles schema collection, dispatch (handle_function_call()),
availability (check_fn, TTL-cached process-wide), and error wrapping. All handlers return a JSON
string, except a tool that hands the model pixels: it may return the multimodal envelope
{"_multimodal": True, "content": [text, image_url…], "text_summary": str} (registry._normalize_handler_result
accepts exactly that shape). Its image path goes through vision_tools._native_tool_result_images — never a
private vision check.
Adding a core tool (2 files) — only when the user is explicitly contributing a core tool
For custom/local-only tools do NOT edit core: create ~/.hermes/plugins/<name>/plugin.yaml +
__init__.py and call ctx.register_tool(...); plugin toolsets are discovered automatically and
toggled without touching tools/ or toolsets.py (plugins/AGENTS.md).
tools/your_tool.py:from agent.secret_scope import get_secret from tools.registry import registry def check_requirements() -> bool: return bool(get_secret("EXAMPLE_API_KEY")) def example_tool(param: str, task_id: str = None) -> str: return json.dumps({"success": True, ...}) registry.register(name="example_tool", toolset="example", schema={"name": "example_tool", "description": "...", "parameters": {...}}, handler=lambda args, **kw: example_tool(param=args.get("param", ""), task_id=kw.get("task_id")), check_fn=check_requirements, requires_env=["EXAMPLE_API_KEY"])toolsets.py: add the name to_HERMES_CORE_TOOLS(all platforms) or a new toolset. Required — discovery registers the schema, but a tool is only exposed if a toolset names it._HERMES_CORE_TOOLSis the default bundle every platform's base toolset inherits, not dead code.
Rules for tool code:
- Schema descriptions must not name tools from other toolsets (a browser tool saying "prefer
web_search"). Those tools may be unavailable (missing key, disabled toolset) and the model
hallucinates calls to them. Cross-references are added dynamically in
get_tool_definitions()inmodel_tools.py— see thebrowser_cdp/execute_codepost-processing blocks. - Paths in schema descriptions use
display_hermes_home()(schema is built at import; under multiplex it shows the launch home, which is display-only). State files useget_hermes_home()at call time, neverPath.home()/.hermesand never a module constant, so each served profile gets its own state. - No
offset/limiton instructional tools (skills, prompts, playbooks) — models read page 1 and skip the rest (root rubric). check_fnanswers reachability/opt-in for the profile it runs under, never surface. Results are TTL-cached inregistry.py::_check_fn_cachekeyed byhermes_home_key(), and one process serves many sessions AND many profiles: a probe reads credentials throughagent.secret_scope.get_secret, never bareos.getenv(that answers with the launch profile's.envfor everyone). The registry classifies anUnscopedSecretErrorfromcurrent_secret_scope()at the catch site — a boot-time probe with no scope is DEBUG, not a traceback. GUI-only tools go in a named toolset (desktop_ui,project) folded in by_load_enabled_toolsets(platform)(root: capability is a property of the SESSION).- Agent-level tools (
todo,memory) are intercepted beforehandle_function_call()via theINLINE_TOOL_EXECUTORStable (agent/inline_tool_executors.py;agent/AGENTS.md). _last_resolved_tool_namesis a process-global inmodel_tools.py;_run_single_child()indelegate_tool.pysaves/restores it around child runs — readers may see it stale mid-delegation.- New tools integrate with existing setup UX (
hermes tools,hermes setup, auto-install) rather than a raw env var; secrets go inOPTIONAL_ENV_VARS(hermes_cli/AGENTS.md).
Toolsets (toolsets.py)
Single TOOLSETS dict. Keys today: browser, clarify, code_execution, cronjob, debugging, delegation, discord, discord_admin, feishu_doc, feishu_drive, file, image_gen, kanban, memory, messaging, moa, rl, safe, search, session_search, skills, terminal, todo, tts, video, vision, web, yuanbao (don't assert the list in tests). Per-platform enable/disable via
hermes tools (curses) or tools.<platform>.enabled/disabled in config.yaml. browser_exec is the
one core browser tool (beside browser_cdp / browser_dialog / browser_vault, all CDP).
Browser runtime
browser_exec (browser_use_cli.py) drives a browser over CDP through browser-harness. Where that
CDP endpoint comes from is browser_session.get_session_info(cache_key): operator BU_CDP_* env (in
browser_exec) > CDP override (live /browser connect > BROWSER_CDP_URL > browser.cdp_url) >
hybrid ::local sidecar > cloud provider (failure → local session marked fallback_from_cloud) >
real profile (consented) > Lightpanda > the packaged Chromium from browser_launcher.launch_chromium
(one browser per cache key, --remote-debugging-port=0 + DevToolsActivePort). State lives in
browser_session.py; siblings late-import it inside functions. Every launched Chromium is stopped
parent-first and recorded in a private 0700 dir (a throwaway profile's in
<HERMES_HOME>/cache/browser/chromium-records; a persistent profile's per OS user under the host lock dir,
one for every home pinning it) so a later process reaps a crashed owner's browser after checking pid + start
time + its --user-data-dir argv token; the reaper walks every served home and every shared profile.
browser_tool.py and browser_tool_install.py are frozen old-updater stubs only (see their
docstrings); never grow them.
Backends and providers inside tools/
Several tools front pluggable backends: terminal environments in tools/environments/ (local,
docker, ssh, modal, daytona; terminal_tool_backends.py, tool_backend_helpers.py),
browser (browser_launcher.py + browser_session*.py: lifecycle, cloud, cdp, real_profile, sandbox), MCP
client (mcp_tool_*.py: config, discovery, transport, registration, content, errors), TTS
(tts_tool_providers.py, tts_command_provider.py), skills hub sources (skills_hub_official.py
OptionalSkillSource). Adding a backend = a new sibling or provider entry in the existing table,
never an elif on a backend name (root shape rules). Remote-backend file visibility problems are
fixed at the mount, not by adding a tool.
Native vision embeds are history, not one-shot payloads. vision_tools.py::_vision_analyze_native
(and the browser screenshot twin in browser_use_cli.py) bake the image
into a tool result that is re-sent on every later API call. Size and repeat policy live in
vision_tools_history_budget.py (config section vision: embed_target_bytes, max_calls_per_image);
the repeat counter is keyed on (session id, resolved source) so region crops share their file's count,
and its default cap applies only inside agent.delegation_context.is_delegated_child_process_context().
Put new embed-cost rules there, never a second counter in a tool.
Every spawn goes through one env builder. environments/local.py::build_subprocess_env (+
hermes_constants.apply_subprocess_home_env, env_passthrough.py::resolve_passthrough_value) is
how a terminal, execute_code, background process, delegation child, ACP or MCP stdio child gets
its environment; a child that acts FOR the served profile (hermes -p X workers, key_cmd
helpers, browser drivers, Bot Chat relay turns) uses environments/local.py:: served_profile_child_env(target_home=, inherit_credentials=): launch-profile .env /
TERMINAL_* residue dropped (strip_launch_profile_env), the target home pinned, only the
target's own secrets overlaid. os.environ.copy() / dict(os.environ) pins the launch profile;
contextvars do not cross process boundaries, so resolve before Popen. A child's
UnscopedSecretError is a spawn-site bug, never a reason to add environ fallthrough. New threads
from scoped code use agent.memory_provider.spawn_context_thread (a bare threading.Thread
drops the scope). MCP trust is a per-profile record: mcp_tool_registration.py:: _record_scope_trust keys trust on the home; a secondary never adopts the launch profile's trust
for a same-named server, and mcp_tool_handlers.py::_trust_gate_check consults the calling
session's profile.
Background-process teardown signals the parent first. process_registry_termination.py:: ProcessTerminationMixin._terminate_host_pid snapshots the descendants, SIGTERMs only the recorded parent, waits
terminal.daemon_term_grace_seconds for it to exit and reap its own children, then SIGTERMs the
snapshot survivors and SIGKILLs whatever ignored both (so a supervisor that reaps its tree — a
Chromium/Electron browser reaping its zygotes, a shell trap — exits cleanly, while a shell whose
children ignore SIGHUP still leaves no orphan). Never SIGTERM descendants before the parent: killing
a browser's zygote mid-shutdown turns exit 0 into a SIGTRAP core dump. _stop_systemd_unit (scope
teardown, kills the worker cgroup) runs only after that PID kill in kill(), or on a parent already
proven dead/recycled (session.exited, _signal_kill recycled-PID path, checkpoint recovery) —
it is never the first signal a live parent receives.
Delegation (tools/delegate_tool.py)
Spawns a subagent with isolated context + terminal session; the parent waits for the summary unless
background=true, which returns a delegation id and re-enters the result via the async-delegation
completion queue. Shapes: single (goal + optional context, toolsets) or batch (tasks: [...],
concurrency capped by delegation.max_concurrent_children, default 3). A background batch returns as ONE
completion by default; with delegation.independent_completions it is split into completion units
(delegate_tool_dispatch._units_of): tasks sharing a group join and report together; each ungrouped
task reports alone as it finishes. Units of one call share ONE pool slot (slot_key in
async_delegation._dispatch) — never count units against capacity; the executor is sized by live UNITS
and the stall clock arms when the runner starts, so a queued unit is never judged stalled. Roles: leaf (default;
no delegate_task, clarify, memory, send_message, cronjob; keeps execute_code) and
orchestrator (keeps delegate_task; gated by delegation.orchestrator_enabled, bounded by
delegation.max_spawn_depth, default 2). Config knobs under delegation::
max_concurrent_children, independent_completions, gateway_mode, max_spawn_depth, child_timeout_seconds, orchestrator_enabled, subagent_auto_approve, inherit_mcp_toolsets, max_iterations, backend, backends. Routes: delegation.backend picks a
delegation provider (agent/delegation_provider.py, ctx.register_delegation_provider; its settings live in
delegation.backends.<name>) that answers each task's route (tools/delegate_tool_routes.py: model/provider fields
resolved like the delegation section, service_tier checked like /fast in delegate_tool_tuning.py, or profile →
the child is built, leases and converses inside that profile's scope) and may add enum-checked per-task schema params; it
never owns lifecycle. The default strategy is the bundled plugins/delegation/default plugin (no core special case
beyond selecting it). Fails closed: an unusable backend is a tool error naming it (never a silent default), an
unroutable task is refused alone (TaskRoute.error), as is a task sending a parameter its backend lacks. A session routes on the backend its delegate_task was built from (agent._delegation_backend, pinned with its tools), so a save applies at /new or --now (agent/delegation_backends.py). Internal credentials_cfg callers (/review) and the lifecycle API bypass it. Child processes: a child's background
processes are killed at its teardown and their notices are suppressed in the parent; process_manage(action="handoff")
(children only) flips ProcessSession.owner_task_id to the parent under the registry lock
(process_registry.transfer_ownership) so the completion routes and reaps by the new owner; un-handed leftovers land on
the result as orphaned_processes, exited-but-never-read notify processes as unread_completions (_ChildRun.account_background_processes, before cleanup kills them). Child kernels: a child's execute_code kernels are keyed <parent-owner>::child::<child-session-id>, pinned against the max_session_kernels LRU cap while the child runs and disposed by cleanup (code_kernel.shutdown_kernels_for_delegated_child) — never let a finished child's kernel squat the cap. output_schema: a miss after the one retry keeps status: completed with the raw text in summary plus schema_valid: false / schema_errors / schema_note — never discard a child's result. Durability: background
delegation is process-local; work that must survive restart uses cronjob or
terminal(background=True, notify_on_complete=True). heartbeat=N on the same spawn emits a heartbeat event (type on the completion queue; delta output only, HEARTBEAT_MIN_SECONDS floor, one daemon timer thread for all sessions in process_registry.py::ProcessRegistry._heartbeat_loop) — every surface that renders watch_match must render heartbeat (process_registry_notifications.py, gateway/run.py::_drain_gateway_watch_events, tui_gateway/session_notifications.py::_DEDUP_EXTRA_FIELDS). Stop fan-out: a turn's hard interrupt reaches only
_active_children; background units are detached at dispatch and survive it on purpose (Teknium ruling: a
background subagent, /review included, is independent of the chat that spawned it). Turn stops — Ctrl+C,
Esc, the Desktop/TUI Stop button, a message that interrupts, session.interrupt, ACP cancel — never call
async_delegation.interrupt_for_session. Only the explicit /stop command (gateway busy AND idle paths, CLI
and Desktop process stop) and session close/reset end them. Depth>0 delegations are always synchronous
(_model_background_value), so a stop recurses through the child's own _active_children fan-out; an
interrupted entry's summary is the child's last real assistant text (_build_result_entry). API: website/docs/developer-guide/subagent-lifecycle-api.md.
Tests
tests/tools/. Test the handler through the registry (real dispatch), not the bare function only;
assert contracts ("every registered tool has a toolset", "no schema description names a tool from
another toolset") rather than tool counts. Approval/security-boundary tools are E2E'd with real
imports against a temp HERMES_HOME (see tests/tools/test_approval_config_readonly.py).
Surface capability is a property of the SESSION, never of the process env
A tool that works only because of who is on the other end (desktop panes, in-app browser,
message reactions, Projects) must resolve availability from the session's own source, not
from an env var on the backend. Client and backend are separate machines: the desktop app may
drive a locally spawned backend, one over SSH, one behind URL + token, or Hermes Cloud, and
only the first two carry HERMES_DESKTOP=1. An env-keyed gate is a silent no-op on the other
topologies — the tool is stripped from the schema while the platform hint tells the model it
is "inside the Hermes desktop app". The pattern:
- The toolset is the surface gate. Keep such tools off
_HERMES_CORE_TOOLSand in a named toolset (desktop_ui,project); the GUI gateway's_load_enabled_toolsets(platform)folds it in when the session's platform says GUI. One resolver, every topology. check_fnanswers reachability or opt-in, not surface. "Is the bridge wired?" — fine. "Was I spawned by Electron?" — not.check_fnresults are TTL-cached process-wide (tools/registry.py); a per-session answer does not belong there.- Ask which identity you mean.
HERMES_DESKTOP=1legitimately means "this backend was spawned by the app" (cron ticker, web-dist handling). It does NOT mean "a GUI is watching"; the embedded terminal pane (hermes --tuiagainst that backend) is the counterexample.
Test: if the capability still makes sense with the client on another machine, it is session-scoped. Assert the GUI session gets the tool with the env var absent.