fix(store): keep hand-written MEMORY.md index lines

[kuitos]

Problem

MemoryStore.save() rewrote a memory's MEMORY.md line on every save, even when someone had written that line by hand. The line was rebuilt as - [<name>](<file>) — <description>, and a re-save by the main agent, an extraction fork or auto-dream replaced the user's own title and hook with it (#49: "it also rewrote … the whole MEMORY.md, which replaced our hand-written index lines").

Reproduced with MemoryStore on main (ef15ddb). The test seeds a memory, writes its index line by hand, then re-saves it with a new description:

# main@ef15ddb, after the re-save
## Working with me
- [terse-answers](terse-answers.md) — User wants terse answers without trailing summaries

# this branch, after the re-save
## Working with me
- [Terse please](terse-answers.md) — no recaps at the end

Fix

• save() adds a pointer line only when the index has none for the file.
• An existing line is replaced only when it is exactly the line the plugin would have generated from the memory's previous name and description, i.e. the plugin wrote it and nobody edited it. Any other line is kept verbatim.
• findIndexPointerLine() (store/indexFile.ts) finds the line that points at a file. All edits still go through upsertIndexLine / removeIndexLine.
• A save whose content is unchanged still adds a pointer when the index has none.
• AGENTS.md (anti-patterns) and the README compatibility row document the rule.

Tests

• test/store/MemoryStore.test.ts ("MemoryStore index lines written by hand (Memories deleted or rewritten without being asked (read-only questions, extraction, auto-dream) #49)"):
  • a hand-written line survives an identical re-save, a new description and a new body;
  • a line still exactly as the plugin generated it follows a new name or description;
  • a memory without a pointer gains one without its file being rewritten;
  • a pointer written before the memory exists is kept;
  • re-saving every memory, as a consolidation pass does, leaves a hand-written index (with headings) byte-identical.
• test/store/indexFile.test.ts: findIndexPointerLine.
• bun run lint, bun run typecheck, bun test and bun run build pass.

Real-environment verification (OpenCode 2.0.22, isolated DB/config, memory agents on opencode/gpt-6-luna)

Auto-dream run over 5 seeded memories, with a hand-written MEMORY.md under ## headings, a duplicate pair and an expired sprint goal. Measured on the top of this stack, since the dream prompt changes in #53:

• main@ef15ddb, 2/2 runs: every surviving index line was replaced with a [slug] — description line. [Terse please] and [Real DB in tests] were lost.
• Stack, 3/3 runs:
  • the hand-written lines of all 3 surviving memories were kept byte for byte;
  • only the lines of the deleted memories were removed (a merged duplicate and the expired sprint goal);
  • this held even when the dream re-saved the merged memory with a new description.

Refs #49

🤖 Generated with Claude Code

[github-actions]

🎉 This PR is included in version 2.1.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀