Skip to content

feat(memory): add long-term memory system#93

Open
jexShain wants to merge 1 commit intoAI-Shell-Team:mainfrom
jexShain:worktree-long-term-memory
Open

feat(memory): add long-term memory system#93
jexShain wants to merge 1 commit intoAI-Shell-Team:mainfrom
jexShain:worktree-long-term-memory

Conversation

@jexShain
Copy link
Copy Markdown
Collaborator

@jexShain jexShain commented Apr 3, 2026

Summary

  • Add persistent memory system with SQLite FTS5 full-text search and Markdown storage
  • Integrate recall (pre-AI) and retain (post-AI) hooks into AIHandler
  • Add MemoryTool for LLM explicit memory management and MemoryConfig

Test plan

  • Run uv run pytest tests/test_memory_manager.py tests/test_memory_config.py tests/test_memory_models.py tests/test_memory_tool.py
  • Verify memory persists across shell sessions
  • Test recall/retain cycle in AI conversations

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Thanks for the pull request. A maintainer will review it when available.

Please keep the PR focused, explain the why in the description, and make sure local checks pass before requesting review.

Contribution guide: https://github.com/AI-Shell-Team/aish/blob/main/CONTRIBUTING.md

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

This pull request description looks incomplete. Please update the missing sections below before review.

Missing items:

  • User-visible Changes
  • Compatibility
  • Testing
  • Change Type
  • Scope

…nd LLM tool integration

Implement persistent memory for aish sessions:
- MemoryManager with SQLite FTS5 full-text search and Markdown storage
- MemoryConfig integrated into ConfigModel
- MemoryTool for LLM explicit memory management
- recall (pre-AI) and retain (post-AI) hooks in AIHandler
- Async I/O and config dir support
@jexShain jexShain force-pushed the worktree-long-term-memory branch from 1db0e79 to 6e9ad6d Compare April 3, 2026 08:44
@github-actions github-actions bot removed the i18n label Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant