# NousResearch/hermes-agent · Temp path alias mismatch in verification cleanup logic

## Open now

- 1 issue, 2 PRs open on GitHub.
- #70797: Test platform: test_nonrecursive_verification_artifact_cleanup_is_not_dangerous fails on macOS since introduction (literal /tmp operand vs canonical-only exemption)
- PR #70415: fix(approval): recognize the raw temp-dir alias in verification cleanup
- PR #106198: fix(approval): make temp-artifact exemption work on Windows

A test failure on macOS exposed that the verification cleanup code did not recognize the raw /tmp alias alongside the canonical temp directory path. The accompanying fixes address both the cleanup logic and the test's path handling.

## TL;DR

- 5 member(s): 1 open issue(s), 0 closed issue(s), 2 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.40`: 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

- #70797: Test platform: test_nonrecursive_verification_artifact_cleanup_is_not_dangerous fails on macOS since introduction (literal /tmp operand vs canonical-only exemption)

## Prior & competing PRs

- **#70415: fix(approval): recognize the raw temp-dir alias in verification cleanup**. open
  - maintainer direction: category `uncategorized`; implication `needs_llm`
  - files: `tests/tools/test_approval.py`, `tools/approval.py`
- **#76742: fix(test): use canonical temp path for verification cleanup exemption**. closed without merge
  - maintainer direction: category `partial_fix`; implication `needs_llm`; refs `5cf226824d3d`, `#70797`, `#76742`, `#70415`, `#55946`, `#374151`
- **#94846: fix(mcp-oauth): recover from peer-rotated refresh tokens instead of clearing the session**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `8ad7ae06e7`, `#111186`
- **#106198: fix(approval): make temp-artifact exemption work on Windows**. open
  - maintainer direction: category `wontfix_bydesign`; implication `wontfix`
  - files: `tests/tools/test_approval.py`, `tools/approval_detection.py`

## Fix status

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