# NousResearch/hermes-agent · Background review memory access scoping fixes

## Open now

- Nothing open on GitHub.

These issues and PRs concern a bug where the background review whitelist was applied per-tool rather than per-operation, allowing skill reviews to silently delete memory entries. The work focuses on scoping memory access permissions to the specific trigger that fired the background review.

## TL;DR

- 5 member(s): 0 open issue(s), 1 closed issue(s), 0 open PR(s), 1 merged PR(s), 3 PR(s) closed without merging.
- A fix was merged to main in PR #106310. Verify it addresses your case.
- **Recommended:** read PR #106310 (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-106310`. 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.

## Closed / duplicate issues

- #105921: Background review whitelist is per-tool, not per-operation: skill review can silently delete memory entries

## Prior & competing PRs

- **#106310: fix(agent): a skills-only background review can no longer delete memory entries (#105921; salvage #105944, dup #105949)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `agent/background_review.py`, `agent/turn_context.py`, `run_agent.py`, `tests/agent/test_background_review_memory_scope.py`, `tests/agent/test_refine_snapshot_isolation.py`, `tests/tools/test_memory_tool.py`, `tests/tools/test_skill_provenance.py`, `tools/memory_tool.py`, `tools/skill_provenance.py`
- **#105944: fix(agent): scope background review memory access to its trigger (#105921)**. closed without merge
  - maintainer direction: category `implemented_elsewhere`; implication `superseded`; refs `e333113871`, `#106310`
- **#105949: fix(agent): scope background-review memory whitelist to the trigger that fired**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#105944`, `#105921`, `#105944`, `#105944`
- **#105961: fix(review): scope background tools to trigger**. closed without merge
  - maintainer direction: category `duplicate_of`; implication `duplicate`; refs `#105944`, `#105944`, `#105949`, `#105944`

## Fix status

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