{"repo":"NousResearch/hermes-agent","public_id":"75618","display_label":"Background-review read-mark thread-safety bug and fixes","summary":"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.","subsystem":"agent","counts":{"members":5,"open_issues":0,"open_issues_distinct":0,"open_prs":0,"closed_issues":1,"merged_prs":1,"attempt_prs":3},"landed_fix":{"status":"merged_to_main","basis":"structural","pr":94511},"resolution":{"state":"resolved","by":"pr-94511"},"freshness":{"state":"verified","last_verified_at":"2026-08-25T06:55:40+00:00","is_stale":false},"grouping":{"backbone":"STRUCTURAL_SPANNING","link_density":0.4},"fix_edges":[{"issue_number":73965,"pr_number":94511,"tier":"verified","basis":"superseded_ref","confidence":0.9,"recoverable_from_timeline":false},{"issue_number":75618,"pr_number":94511,"tier":"verified","basis":"closing_ref","confidence":0.95,"recoverable_from_timeline":true},{"issue_number":94324,"pr_number":94511,"tier":"verified","basis":"superseded_ref","confidence":0.9,"recoverable_from_timeline":false}],"members":[{"kind":"issue","number":75618,"title":"skill_manage always refuses background-review patches: ContextVar read-mark doesn't survive per-call thread dispatch","url":null,"state":"closed"},{"kind":"pr","number":73975,"title":"fix(skills): preserve review read marks across tool contexts","url":null,"state":"closed","close_reason":{"refs":[{"kind":"commit","value":"335c60ecdd76a7b8b9d6fa0332ac15e58059963b","number":0},{"kind":"pr","value":"#94511","number":94511},{"kind":"pr","value":"#75618","number":75618}],"category":"merged_via","implication":"superseded"},"merged":false},{"kind":"pr","number":75661,"title":"fix(skills): scope background-review read marks per review fork","url":null,"state":"closed","close_reason":{"refs":[{"kind":"commit","value":"335c60ecdd76a7b8b9d6fa0332ac15e58059963b","number":0},{"kind":"pr","value":"#94511","number":94511},{"kind":"pr","value":"#73975","number":73975}],"category":"merged_via","implication":"superseded"},"merged":false},{"kind":"pr","number":83321,"title":"fix(skills): replace ContextVar with thread-safe set for background review marks","url":null,"state":"closed","close_reason":{"refs":[{"kind":"commit","value":"335c60ecdd76a7b8b9d6fa0332ac15e58059963b","number":0},{"kind":"pr","value":"#75618","number":75618},{"kind":"pr","value":"#94511","number":94511},{"kind":"pr","value":"#73975","number":73975}],"category":"merged_via","implication":"superseded"},"merged":false},{"kind":"pr","number":94511,"title":"fix(skills): background-review curator patches no longer refused after skill_view (#75618, salvage #73975)","url":null,"state":"merged","close_reason":{"category":"merged","implication":"superseded"},"merged":true,"is_fix_pr":true,"fix_tier":"verified","files_changed":["tests/tools/test_skill_manager_tool.py","tools/skill_manager_tool.py"]}],"trust":{"default_tier":"verified","includes_suggested":false}}