Skip to content

[WIP] feat: memory store#1266

Draft
supreme-gg-gg wants to merge 6 commits intokagent-dev:mainfrom
supreme-gg-gg:feat/memory
Draft

[WIP] feat: memory store#1266
supreme-gg-gg wants to merge 6 commits intokagent-dev:mainfrom
supreme-gg-gg:feat/memory

Conversation

@supreme-gg-gg
Copy link
Contributor

This is work in progress and meant for community feedback only. For description see #1256. I documented the changes and limitations of the design here and will be updated as more changes are made.

Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
@github-actions github-actions bot added the enhancement-proposal Indicates that this PR is for an enhancement proposal label Feb 4, 2026
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
Signed-off-by: Jet Chiang <pokyuen.jetchiang-ext@solo.io>
@dimetron
Copy link
Contributor

dimetron commented Feb 5, 2026

Hi @supreme-gg-gg

  • Do you plan to add GenerateMemories func to do LLM based extraction ?
  • How I can configure default InMemoryMemoryService, VertexAiMemoryBankService options ?

@supreme-gg-gg
Copy link
Contributor Author

  • Do you plan to add GenerateMemories func to do LLM based extraction ?

Yea there is a LLM based extraction step here:

async def _summarize_session_content_async(

However there isn't a "consolidation" step which means before inserting memory directly after extraction, find relevant entries in the db, show them to another LLM and ask it to update / add these entries, this is commonly done in e.g. Vertex AI memory bank. I marked that in the limitations section but that will be more complicated to do.

  • How I can configure default InMemoryMemoryService, VertexAiMemoryBankService options ?

With this PR if you enable memory on declarative agents you only use the kagent memory store, for those two options you can use BYO for now. But I'm working on refactoring the interface for extending to other memory stores I'll also think about if we can allow things like Vertex memory service.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement-proposal Indicates that this PR is for an enhancement proposal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants