# NousResearch/hermes-agent · Request middleware chaining drops earlier rewrites

## Open now

- Nothing open on GitHub.

A bug where registering multiple \`llm_request\` or \`tool_request\` middleware callbacks causes earlier rewrites to be lost, along with fixes addressing middleware composition and mutation leaks.

## TL;DR

- 4 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 2 PR(s) closed without merging.
- A fix was merged to main in PR #133159. Verify it addresses your case.
- **Recommended:** read PR #133159 (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: `resolved` · deciding PR(s): `pr-133159`. This is distinct from the fix status above. Weigh it yourself.

## How this cluster was grouped

- Link evidence (`STRUCTURAL_SPANNING`): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
- Pairwise link density `0.50`: 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.

## Closed / duplicate issues

- #128638: \[Bug\]: a second \`llm_request\` / \`tool_request\` middleware drops the first one's rewrite

## Prior & competing PRs

- **#133159: Request middleware rewrites now compose instead of the last one winning (#128638, salvage #128704)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `hermes_cli/middleware.py`, `hermes_cli/plugins_dispatch.py`, `tests/hermes_cli/test_plugins_middleware_chain.py`
- **#128643: fix(plugins): chain request middleware so each callback sees the previous rewrite**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `#133159`, `#128704`
- **#128704: fix(plugins): compose request middleware without mutation leaks**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `#133159`

## Fix status

- Merged to main in PR #133159 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-10-05`.