# NousResearch/hermes-agent · Shell quoting fix in CLI approvals test

## Open now

- 1 issue, 2 PRs open on GitHub.
- #120931: hermes approvals test drops shell quoting, so the dry-run verdict can diverge from the runtime verdict
- PR #120932: fix(cli): approvals test re-quotes argv so the dry-run verdict matches the runtime
- PR #120962: fix(cli): re-quote approvals test command with shlex.join

These items address a shell quoting issue in the CLI approvals test where the dry-run verdict could diverge from the runtime verdict. The proposed fix involves re-quoting the argv using shlex.join to keep the two verdicts consistent.

## TL;DR

- 3 member(s): 1 open issue(s), 0 closed issue(s), 2 open PR(s), 0 merged PR(s), 0 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.67`: 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

- #120931: hermes approvals test drops shell quoting, so the dry-run verdict can diverge from the runtime verdict

## Prior & competing PRs

- **#120932: fix(cli): approvals test re-quotes argv so the dry-run verdict matches the runtime**. open
  - maintainer direction: category `superseded_by_pr`; implication `superseded`
  - files: `hermes_cli/approvals_test.py`, `tests/hermes_cli/test_approvals_test.py`
- **#120962: fix(cli): re-quote approvals test command with shlex.join**. open
  - maintainer direction: category `uncategorized`; implication `needs_llm`; refs `#120931`, `#120931`
  - files: `hermes_cli/approvals_test.py`, `tests/hermes_cli/test_approvals_test.py`

## Fix status

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