# NousResearch/hermes-agent · Fix journey memory edit/delete file mutation safety

## Open now

- Nothing open on GitHub.

A bug was identified where editing or deleting a Journey memory would rewrite the entire file without locking, silently discarding concurrent changes or hand-edited content. These items address routing such operations through a locked mutation path and ensuring the correct card is targeted when the list shifts.

## TL;DR

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

- #119668: \[Bug\]: a Journey memory edit/delete rewrites the whole file unlocked — concurrent memories and hand-edited content are silently lost

## Prior & competing PRs

- **#121338: fix(journey): route memory edit/delete through MemoryStore._mutate**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `agent/learning_mutations.py`, `tests/agent/test_learning_mutations.py`, `tools/memory_tool_store.py`
- **#121455: fix(memory): journey edit/delete targets the clicked card after the list shifts**. merged to main · fix tier `verified`
  - maintainer direction: category `merged`
  - files: `agent/learning_graph.py`, `agent/learning_graph_render.py`, `agent/learning_mutations.py`, `apps/desktop/src/app/starmap/star-map.tsx`, `apps/desktop/src/types/hermes.ts`, `contributors/emails/[redacted:email]`, `hermes_cli/journey.py`, `tests/agent/test_learning_graph_render.py`, `tests/agent/test_learning_mutations.py`, `website/docs/user-guide/features/memory.md`
- **#119670: fix(memory): journey edits go through the locked mutate, not a bare rewrite**. closed without merge
  - maintainer direction: category `merged_via`; implication `superseded`; refs `3da4a42359`, `#121455`, `#121338`

## Fix status

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