{"repo":"NousResearch/hermes-agent","public_id":"95976","display_label":"Skill update failures in background review fork","summary":"These issues and fixes center on a bug where the background review fork cannot update skills, caused by conflicts between read-before-write guards and skill_view dedup stubs. Several related patches address seeding context and removing or correcting the dedup stub behavior in the fork.","subsystem":null,"counts":{"members":5,"open_issues":0,"open_issues_distinct":0,"open_prs":0,"closed_issues":1,"merged_prs":2,"attempt_prs":2},"landed_fix":{"status":"merged_to_main","basis":"structural","pr":115299,"merged_at":"2026-09-18T19:03:59+00:00"},"resolution":{"state":"resolved","by":"pr-115299"},"freshness":{"state":"verified","last_verified_at":"2026-09-18T19:03:59+00:00","is_stale":false},"grouping":{"backbone":"STRUCTURAL_SPANNING","link_density":0.4},"fix_edges":[{"issue_number":95976,"pr_number":112196,"tier":"verified","basis":"superseded_ref","confidence":0.9,"recoverable_from_timeline":false,"pr_merged_at":"2026-09-16T02:01:12+00:00"},{"issue_number":95976,"pr_number":115299,"tier":"verified","basis":"closing_ref","confidence":0.95,"recoverable_from_timeline":true,"pr_merged_at":"2026-09-18T19:03:59+00:00"},{"issue_number":111604,"pr_number":112196,"tier":"verified","basis":"closing_ref","confidence":0.95,"recoverable_from_timeline":true,"pr_merged_at":"2026-09-16T02:01:12+00:00"}],"members":[{"kind":"issue","number":95976,"title":"Background review fork fails to update skills 100% of the time — read-before-write guard conflicts with skill_view repeat-view dedup","url":null,"state":"closed"},{"kind":"pr","number":95993,"title":"fix(skills): the dedup stub must mark the read for the review fork guard","url":null,"state":"closed","close_reason":{"refs":[{"kind":"pr","value":"#115299","number":115299},{"kind":"pr","value":"#95976","number":95976},{"kind":"pr","value":"#95975","number":95975}],"category":"superseded_by_pr","implication":"superseded"},"merged":false},{"kind":"pr","number":107734,"title":"fix(curator): seed the review read-mark store in the fork's context","url":null,"state":"closed","close_reason":{"refs":[{"kind":"commit","value":"8b5856261462","number":0},{"kind":"pr","value":"#112196","number":112196}],"category":"superseded_by_pr","implication":"superseded"},"merged":false},{"kind":"pr","number":112196,"title":"fix(curator): consolidation pass can patch skills again (#111604, salvage #111605)","url":null,"state":"merged","close_reason":{"category":"merged"},"merged":true,"is_fix_pr":true,"fix_tier":"verified","files_changed":["agent/curator.py","tests/agent/test_curator.py"]},{"kind":"pr","number":115299,"title":"Background review fork can update skills again: no dedup stub in the fork (#95976, salvage #95975)","url":null,"state":"merged","close_reason":{"category":"merged"},"merged":true,"is_fix_pr":true,"fix_tier":"verified","files_changed":["contributors/emails/[redacted:email]","tests/tools/test_file_read_guards.py","tests/tools/test_skill_view_dedup.py","tools/file_tools.py","tools/skills_tool.py"]}],"trust":{"default_tier":"verified","includes_suggested":false}}