# NousResearch/hermes-agent · Bound command-substitution nesting depth in approval scanner

## Open now

- 1 issue, 1 PR open on GitHub.
- #137066: \[Bug\]: Deeply nested shell command substitutions crash the pre-exec command-approval scanner with RecursionError (the parser guard bounds size, not nesting depth)
- PR #137894: fix(tools): bound nested approval scan depth

One bug report describes deeply nested shell command substitutions causing a RecursionError in the pre-exec command-approval scanner, noting the existing guard limits size but not nesting depth. Two pull requests are titled as changes that bound command-substitution nesting in the approval guard.

## TL;DR

- 4 member(s): 1 open issue(s), 0 closed issue(s), 1 open PR(s), 0 merged PR(s), 2 PR(s) closed without merging.
- No fix has been merged to main; one or more PRs were attempted (see below).
- **Recommended:** review the prior/competing PR(s) below. A fix was attempted but not merged; your case may extend or revive that work.

## 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_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.

## Open related issues

- #137066: \[Bug\]: Deeply nested shell command substitutions crash the pre-exec command-approval scanner with RecursionError (the parser guard bounds size, not nesting depth)

## Prior & competing PRs

- **#137071: fix(approval): bound command substitution nesting**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `#134471`, `#137071`
- **#137092: fix(tools): bound command-substitution nesting depth in the approval guard**. closed without merge
  - maintainer direction: category `superseded_by_pr`; implication `superseded`; refs `#137071`, `#137066`
- **#137894: fix(tools): bound nested approval scan depth**. open
  - maintainer direction: category `silent_close`; implication `active`
  - files: `tests/tools/test_approval_nested_depth.py`, `tools/approval_detection.py`

## Fix status

- No merged fix on record.
- Subsystem: `agent`