# NousResearch/hermes-agent · Test mock Popen streams causing non-EOF spin issue

## Open now

- Nothing open on GitHub.

These issues and PRs address a problem where mocked kernel streams in local-env tests return non-EOF MagicMock values instead of terminating, causing reader threads to spin indefinitely. The work involves fixing fake Popen fixtures to properly end kernel streams with EOF.

## 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 #116852. Verify it addresses your case.
- **Recommended:** read PR #116852 (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-116852`. 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

- #115912: tests: local-env Popen mock feeds non-EOF MagicMock values to kernel read1 readers

## Prior & competing PRs

- **#116852: fix(tests): local-env Popen fixture ends the kernel streams, reader threads no longer spin on MagicMock (#115912, salvage #115917)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `plugin-catalog/intelligent-tool-break.yaml`, `tests/tools/test_local_env_blocklist.py`
- **#115917: fix(tests): terminate mocked kernel streams**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `d0b71c921d71`, `#116852`, `#115912`, `#115917`
- **#115919: test(code-exec): feed read1() EOF from the fake Popen in the PYTHONPATH test**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#115917`

## Fix status

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