Skip to content

Fix project memory auto-consolidation#51

Merged
chandra447 merged 1 commit into
chandra447:mainfrom
ffrappo:fix/project-memory-consolidation
May 23, 2026
Merged

Fix project memory auto-consolidation#51
chandra447 merged 1 commit into
chandra447:mainfrom
ffrappo:fix/project-memory-consolidation

Conversation

@ffrappo
Copy link
Copy Markdown
Contributor

@ffrappo ffrappo commented May 23, 2026

Summary

  • wire auto-consolidation into the project-scoped MemoryStore
  • tell child consolidation runs to use target='project' for project memory
  • include project memory in /memory-consolidate when a project store is available
  • add coverage for project consolidation prompts and manual command behavior

Why

Project memory writes route to projectStore, but only the global store had a consolidator. When project memory hit the character limit, auto-consolidate was skipped and the memory tool returned the capacity error.

Test

  • npm run check -- --pretty false
  • node --import tsx --test tests/handlers/auto-consolidate.test.ts
  • npm test

chandra447
chandra447 approved these changes May 23, 2026
@chandra447
Copy link
Copy Markdown
Owner

chandra447 commented May 23, 2026

Thank you for contributing this and for actively using the extension — really appreciated. 🙏

Strengths:

  • Adds project-aware consolidation target routing (target = "project")
  • Wires consolidator for projectStore in src/index.ts
  • Extends /memory-consolidate to include project scope output
  • Adds targeted tests for project prompt labeling/targeting and command integration

Non-blocking nits: (can you please open a follow-up PR )

  1. entriesForTarget() currently maps non-user targets to getMemoryEntries(). If failure is ever routed here, failure entries are not read explicitly. Either narrow supported target values or add explicit failure handling + a test.
  2. Optional regression test: validate /memory-consolidate output when projectStore is absent, to lock legacy behavior.

@chandra447 chandra447 merged commit 1580762 into chandra447:main May 23, 2026
3 checks passed
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.

2 participants