Skip to content

fix(memory): replace --resume with transcript-based background updater#151

Merged
dean0x merged 1 commit intomainfrom
fix/working-memory-updater
Mar 20, 2026
Merged

fix(memory): replace --resume with transcript-based background updater#151
dean0x merged 1 commit intomainfrom
fix/working-memory-updater

Conversation

@dean0x
Copy link
Owner

@dean0x dean0x commented Mar 20, 2026

Summary

  • Rewrote background-memory-update: Replaced broken claude --resume approach with transcript reading (~/.claude/projects/{encoded-cwd}/{session_id}.jsonl) + fresh claude -p --model haiku. The --resume approach has been broken for 17+ days — the model responds conversationally instead of calling Write.
  • Removed PROJECT-PATTERNS.md: Redundant with decisions.md + pitfalls.md from the knowledge-persistence skill. Removed from background updater, session-start hook, migration logic, and all documentation.
  • Added verbose logging: Transcript extraction, extracted text, and full prompt are logged to .memory/.working-memory-update.log for debugging.

Test plan

  • npm run build passes (35 skills, 17 plugins)
  • npm test passes (250 tests)
  • grep -r "PROJECT-PATTERNS" scripts/ src/ returns nothing
  • grep "resume" scripts/hooks/background-memory-update — only in comment
  • Manual: exit a session, check .memory/.working-memory-update.log shows successful write
  • Manual: start new session, verify SessionStart injects fresh memory

The background working memory updater has been broken for 17+ days because
`claude --resume` loads the full conversation, causing the model to respond
conversationally instead of calling Write. Replace with transcript reading
+ fresh `claude -p` invocation, and remove the redundant PROJECT-PATTERNS.md
system (superseded by knowledge-persistence skill's decisions.md + pitfalls.md).
@dean0x dean0x merged commit baf0226 into main Mar 20, 2026
3 of 4 checks passed
@dean0x dean0x deleted the fix/working-memory-updater branch March 20, 2026 07:53
dean0x pushed a commit that referenced this pull request Mar 20, 2026
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