Skip to content

docs: add ADR-001 memory layer evaluation and selection #178

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Aureliolo merged 4 commits into from
Mar 8, 2026
Merged

docs: add ADR-001 memory layer evaluation and selection #178

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
fa20c33
docs: add ADR-001 memory layer evaluation and selection (#39)
Aureliolo Mar 8, 2026
c21eff4
docs: update README with current milestone status and memory decision
Aureliolo Mar 8, 2026
cf62ea4
Revert "docs: update README with current milestone status and memory …
Aureliolo Mar 8, 2026
248e9e2
fix: address 25 PR review items from local agents and external reviewers
Aureliolo Mar 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion DESIGN_SPEC.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2326,7 +2326,7 @@ Run: ai-company start acme-corp
| **Language** | Python 3.14+ | Best AI/ML ecosystem, all major frameworks use it, LiteLLM/MCP and memory layer candidates all Python-native. PEP 649 native lazy annotations, PEP 758 except syntax. |
| **API Framework** | FastAPI | Async-native, WebSocket support, auto OpenAPI docs, high performance, type-safe with Pydantic |
| **LLM Abstraction** | LiteLLM | 100+ providers, unified API, built-in cost tracking, retries/fallbacks |
| **Agent Memory** | TBD (candidates: Mem0, Zep, Letta, Cognee, custom) + SQLite | Memory layer library TBD after evaluation. SQLite for structured data. Upgrade to Postgres later |
| **Agent Memory** | Mem0 (initial) → custom stack (future) + SQLite | Mem0 in-process as initial backend behind pluggable `MemoryBackend` protocol (ADR-001). Qdrant embedded + SQLite for persistence. Custom stack (Neo4j + Qdrant external) as future upgrade. Config-driven backend selection |

Copilot AI Mar 8, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This table row now documents a concrete Agent Memory decision, but the surrounding section header still reads “Technology Stack (Candidates - TBD After Research)”. Consider updating the section title (or adding a short note) so the spec doesn’t imply the tech stack is still TBD after the ADR decision.

Copilot uses AI. Check for mistakes.
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Outdated
| **Message Bus** | Internal (async queues) → Redis | Start with Python asyncio queues, upgrade to Redis for multi-process/distributed |
| **Task Queue** | Internal → Celery/Redis | Start simple, scale with Celery when needed |
| **Database** | SQLite → PostgreSQL | Start lightweight, migrate to Postgres for production/multi-user |
Expand Down
Loading