Skip to content

Let the owner forget one memory everywhere it is stored - #5

Merged
alexeybe1kin merged 1 commit into
mainfrom
feat/forget-memory
Sep 25, 2026
Merged

alexeybe1kin merged 1 commit into
mainfrom
feat/forget-memory

Conversation

@alexeybe1kin

Copy link
Copy Markdown
Collaborator

Part of Conker-AI/conker#44. Needs Conker-AI/memorygate#1. Details: docs/memory-forget.md.

What changes

  • Preview: GET /memory/forget/{id} returns the exact current text and revision.
  • Forget: POST /memory/forget forgets that revision through MemoryGate's correction capability. It's idempotent by request id; if the memory changed, it returns 409.
  • Pi's copies: cached per-turn memory packages that quoted the memory are cleared. The source chat is kept.
  • Access: owner-control routes only; through the browser the write needs the password.

Verification

  • 8 new tests; full suite 1113 passed on Linux; auth 19/19 and audit 28/28 mutants caught.
  • Cross-repo, in one process: Pi's real client against MemoryGate's real corrections router. Preview, forget and replay work; the forgotten text survives in no table.

🤖 Generated with Claude Code

- GET /memory/forget/{id} shows the exact current text and revision;
  POST /memory/forget forgets that revision through MemoryGate's
  correction capability (Conker-AI/memorygate#1), idempotent by request id.
- Pi then clears every cached per-turn memory package that quoted it.
  The source conversation is kept.
- Owner-control routes: through the browser the write needs the owner
  password. Outcomes map to 404/409/503; a lost reply is reported as
  unknown and is safe to repeat.

Tests: 8 new; full suite 1113 passed on Linux; auth 19/19 and audit
28/28 mutants caught. Cross-repo check against MemoryGate's real router:
nothing of the forgotten text remained in any table.

Refs Conker-AI/conker#44

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@alexeybe1kin
alexeybe1kin merged commit cb426ba into main Sep 25, 2026
3 checks passed
@alexeybe1kin
alexeybe1kin deleted the feat/forget-memory branch September 25, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant