# NousResearch/hermes-agent · AIAgent.close leaks Codex subprocess session

## Open now

- Nothing open on GitHub.

A bug was reported and multiple fixes proposed to ensure AIAgent.close() properly closes the _codex_session, preventing Codex app-server subprocess leaks.

## TL;DR

- 6 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 2 merged PR(s), 3 PR(s) closed without merging.
- A fix was merged to main in PR #115778. Verify it addresses your case.
- **Recommended:** read PR #115778 (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-115778`. 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.33`: 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

- #66671: \[Bug\]: session.close leaks the Codex app-server process tree — AIAgent.close never closes _codex_session

## Prior & competing PRs

- **#115778: fix(agent): evicted Codex-route agents no longer leak their app-server child on soft release (#72548, #66671)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `hermes_cli/cli_agent_setup_mixin.py`, `hermes_cli/cli_chat_turn_mixin.py`, `hermes_cli/cli_commands_mixin.py`, `hermes_cli/cli_model_switch_mixin.py`, `run_agent.py`, `tests/agent/test_codex_app_server_lifecycle.py`, `tests/hermes_cli/test_cli_retire_agent.py`
- **#121345: test: native provider E2E suites: Bedrock Converse, Gemini, Vertex, Codex app-server and Copilot ACP wire contracts proven against validating fakes**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `tests/e2e/core/providers/__init__.py`, `tests/e2e/core/providers/_native_helpers.py`, `tests/e2e/core/providers/test_native_bedrock_converse_faults.py`, `tests/e2e/core/providers/test_native_bedrock_converse_turns.py`, `tests/e2e/core/providers/test_native_codex_app_server.py`, `tests/e2e/core/providers/test_native_codex_app_server_faults.py`, `tests/e2e/core/providers/test_native_copilot_acp.py`, `tests/e2e/core/providers/test_native_copilot_acp_errors.py`, `tests/e2e/core/providers/test_native_copilot_acp_streaming.py`, `tests/e2e/core/providers/test_native_gemini_compaction.py`, `tests/e2e/core/providers/test_native_gemini_errors.py`, `tests/e2e/core/providers/test_native_gemini_schema.py`, `tests/e2e/core/providers/test_native_gemini_tools.py`, `tests/e2e/core/providers/test_native_vertex_errors.py`, `tests/e2e/core/providers/test_native_vertex_recovery.py`, `tests/e2e/core/providers/test_native_vertex_tools.py`, `tests/fakes/providers/__init__.py`, `tests/fakes/providers/bedrock_converse.py`, `tests/fakes/providers/codex_app_server.py`, `tests/fakes/providers/copilot_acp.py` (+2 more)
- **#66678: fix(agent): close _codex_session in AIAgent.close() to prevent subprocess leaks (#66671)**. closed without merge
  - maintainer direction: category `uncategorized`; implication `needs_llm`
- **#66865: fix(agent): close _codex_session in AIAgent.close() to prevent Codex subprocess leak (#66671)**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#66678`, `#66678`
- **#66904: fix(agent): close Codex app-server subprocess on AIAgent.close() (#66671)**. closed without merge
  - maintainer direction: category `stale_abandoned`; implication `active`; refs `#66678`, `#55450`, `#66678`

## Fix status

- Merged to main in PR #115778 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-09-24`.
- Subsystem: `agent`