Skip to content

feat: add on-demand memory storage and recall tools#193

Open
kyteinsky wants to merge 3 commits into
mainfrom
feat/tool-called-memory
Open

feat: add on-demand memory storage and recall tools#193
kyteinsky wants to merge 3 commits into
mainfrom
feat/tool-called-memory

Conversation

@kyteinsky

@kyteinsky kyteinsky commented Jun 18, 2026

Copy link
Copy Markdown
image image image image image

needs nextcloud/context_chat#247

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Assisted-by: Github Copilot:qwen-3-6-35b-a3b
Assisted-by: Github Copilot:claude-sonnet-4-6
@kyteinsky kyteinsky requested review from janepie and marcelklehr June 18, 2026 09:29
@kyteinsky

Copy link
Copy Markdown
Author

do we want specific tests for this feature in the integration tests?

@kyteinsky kyteinsky requested a review from julien-nc June 18, 2026 09:32
Comment thread ex_app/lib/all_tools/memory.py Outdated
Comment thread ex_app/lib/all_tools/memory.py
Comment thread ex_app/lib/all_tools/memory.py Outdated
Comment thread ex_app/lib/all_tools/memory.py Outdated
Comment thread ex_app/lib/all_tools/memory.py
Comment thread ex_app/lib/all_tools/memory.py
Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Signed-off-by: kyteinsky <kyteinsky@gmail.com>
@marcelklehr

Copy link
Copy Markdown
Member

Nice! We can now make the memory tools dependent on the assistant app being installed and it should be ok

Comment thread ex_app/lib/all_tools/memory.py
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