# NousResearch/hermes-agent · Shell hooks not registered across multiple entry points

## Open now

- 1 issue, 1 PR open on GitHub.
- #41457: Shell hooks not registered in desktop (TUI gateway) and ACP adapter — pre_tool_call block hooks silently ignored
- PR #41464: fix: register shell hooks in TUI gateway and ACP adapter (issue #41457)

A recurring bug where shell hooks defined in config.yaml (e.g. pre_tool_call, on_session_finalize) are never registered in the TUI gateway, ACP adapter, hermes serve, and kanban-worker chat contexts. The cluster contains the original bug reports and numerous fix attempts targeting these entry points.

## TL;DR

- 11 member(s): 1 open issue(s), 3 closed issue(s), 1 open PR(s), 1 merged PR(s), 5 PR(s) closed without merging.
- A fix was merged to main in PR #104870. Verify it addresses your case.
- **Recommended:** read PR #104870 (and any competing PRs below) before filing. Confirm the merged change covers your symptom.

## Maintainer resolution verdict

- Internal-tracker verdict over this cluster's issue members: `unresolved`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_PARTIAL`): some members are connected by structural evidence (shared changed files, or a closing/superseded reference); the rest are joined by an automated relevance judgement over their text.
- Members outside that structural core may belong to a different underlying problem.
- Pairwise link density `0.20`: the share of member pairs that carry a link, counting a structural link in full and a relevance-only link at 0.36. `1.00` means every pair is linked.

## Open related issues

- #41457: Shell hooks not registered in desktop (TUI gateway) and ACP adapter — pre_tool_call block hooks silently ignored

## Closed / duplicate issues

- #43823: \[Bug\]: Shell hooks (config.yaml hooks:) are never registered in the desktop TUI entry point
- #61806: shell hooks (pre_tool_call) never registered in \`hermes serve\` — Desktop app / dashboard API sessions bypass all hook policies
- #67053: Shell hooks register in the launcher but never dispatch in interactive \`hermes chat\` — tui_gateway daemon doesn't call register_from_config

## Prior & competing PRs

- **#41548: fix(#41457): register shell hooks in TUI gateway and ACP adapter at startup**. closed without merge · fix tier `suggested`
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `#41457`, `#41464`
  - _suggested-tier fix detail withheld; re-request with `?tier=all` to include it._
- **#41555: fix(security): register shell hooks in TUI gateway and ACP adapter entry points**. closed without merge · fix tier `suggested`
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#41464`, `#41457`
  - _suggested-tier fix detail withheld; re-request with `?tier=all` to include it._
- **#43826: fix(tui_gateway): register shell hooks at desktop/TUI startup**. closed without merge · fix tier `suggested`
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#13854`
  - _suggested-tier fix detail withheld; re-request with `?tier=all` to include it._
- **#104870: fix(desktop): configured shell hooks run for chat agents**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `evals/desktop_bug_campaign/hooks_live.py`, `tests/tui_gateway/test_profile_shell_hooks.py`, `tui_gateway/server.py`, `website/docs/user-guide/features/hooks.md`
- **#41464: fix: register shell hooks in TUI gateway and ACP adapter (issue #41457)**. open
  - maintainer direction: category `uncategorized`; implication `needs_llm`
  - files: `acp_adapter/session.py`, `tui_gateway/entry.py`
- **#43828: fix(tui): register shell hooks at TUI gateway startup**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#43826`, `#13854`
- **#67084: fix(tui_gateway): register shell hooks in _make_agent so they dispatch interactively**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `d87d7657c0791c7038f2e5dfffb036516cba512c`, `#104870`, `#57020`, `#69825`

## Fix status

- Merged to main in PR #104870 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-09-07`.
- Subsystem: `apps/desktop`