# NousResearch/hermes-agent · Background-review read-mark thread-safety bug and fixes

## Open now

- Nothing open on GitHub.

A bug where background-review read marks stored in a ContextVar are lost across per-call thread dispatch, plus two fix attempts replacing or scoping that storage with a thread-safe alternative.

## 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 #94511. Verify it addresses your case.
- **Recommended:** read PR #94511 (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-94511`. 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

- #75618: skill_manage always refuses background-review patches: ContextVar read-mark doesn't survive per-call thread dispatch

## Prior & competing PRs

- **#94511: fix(skills): background-review curator patches no longer refused after skill_view (#75618, salvage #73975)**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`; implication `superseded`
  - files: `tests/tools/test_skill_manager_tool.py`, `tools/skill_manager_tool.py`
- **#73975: fix(skills): preserve review read marks across tool contexts**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `335c60ecdd76a7b8b9d6fa0332ac15e58059963b`, `#94511`, `#75618`
- **#75661: fix(skills): scope background-review read marks per review fork**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `335c60ecdd76a7b8b9d6fa0332ac15e58059963b`, `#94511`, `#73975`
- **#83321: fix(skills): replace ContextVar with thread-safe set for background review marks**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `335c60ecdd76a7b8b9d6fa0332ac15e58059963b`, `#75618`, `#94511`, `#73975`

## Fix status

- Merged to main in PR #94511 (cited above). No release data is tracked. Verify downstream.
- Fix last verified (merged) `2026-08-25`.
- Subsystem: `agent`