Cairn
NousResearch/hermes-agent

Replacing shell=True with shlex.split in config-driven execution

Cluster 10692 · hermes cli. Data as of

1 open item. Fix attempted, not merged.

The short answer

0open issues
1open PR
3items grouped
Known?
Yes. 3 related items grouped here, 1 issue already closed.
Fix event
No fix has been merged to main. 1 PR closed without merging; see the attempts below.
Tracker
The maintainers' tracker marks this unresolved. That is separate from the fix event.
Still open
0 issues and 1 PR.

Three members concern the use of shell=True in config-driven command execution paths, including cli.py and mcp_catalog.py. One describes the safety-control bypass concern, and two propose switching to shlex.split. Their outcome or status is not stated in the titles.

Open issues0

No open issues.

Open PRs1

  • PR#35545fix(security): convert shell=True to shlex.split in cli.py and mcp_catalog.py

1 PR closed without merging

From the close comments: other 1

  • #10698 fix(security): replace shell=True with shlex.split in config-driven execution paths · other → #10692
Closed issues · 1
  • Closed#10692shell=True in config-driven execution paths bypasses terminal tool safety controls
How this cluster was grouped
  • Link evidence (structural links throughout): every member is connected by structural evidence: shared changed files, or a closing/superseded reference.
  • Pairwise link density 0.67: the share of item 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.

Pick up a Lab task

Check whether a fix covers the open issues here, with your own agent. Bring back evidence for maintainers.

Open the Lab

At a glance

Subsystem
hermes cli
Closed issues
1
Attempts
1 PR closed without merging

For your agent

The same facts, typed.

cluster/10692.json
cluster/10692.md