[memory/RAG] Missing README.md in services/memory/ — violates AGENTS.md feature directory requirement #742

Closed
opened 2026-06-03 00:16:01 +02:00 by sleepy · 0 comments
Owner

AGENTS.md violation: Every feature directory MUST contain a README.md with what it owns, what it does NOT own, key entry points, external dependencies, and test locations.

services/memory/ has 8 Python files but no README.md.

Files in scope:

  • services/memory/memory.py — MemoryManager (JSON file storage)
  • services/memory/memory_extractor.py — LLM-based fact extraction + audit
  • services/memory/memory_vector.py — ChromaDB vector store for memories
  • services/memory/service.py — MemoryService thin wrapper
  • services/memory/skill_extractor.py — LLM skill extraction from agent runs
  • services/memory/skill_format.py — SKILL.md parser/writer
  • services/memory/skills.py — SkillsManager (disk-backed SKILL.md storage)
  • services/memory/__init__.py — Public exports

Action: Create services/memory/README.md documenting ownership, entry points, and dependencies.

**AGENTS.md violation**: Every feature directory MUST contain a `README.md` with what it owns, what it does NOT own, key entry points, external dependencies, and test locations. `services/memory/` has 8 Python files but no `README.md`. **Files in scope**: - `services/memory/memory.py` — MemoryManager (JSON file storage) - `services/memory/memory_extractor.py` — LLM-based fact extraction + audit - `services/memory/memory_vector.py` — ChromaDB vector store for memories - `services/memory/service.py` — MemoryService thin wrapper - `services/memory/skill_extractor.py` — LLM skill extraction from agent runs - `services/memory/skill_format.py` — SKILL.md parser/writer - `services/memory/skills.py` — SkillsManager (disk-backed SKILL.md storage) - `services/memory/__init__.py` — Public exports **Action**: Create `services/memory/README.md` documenting ownership, entry points, and dependencies.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
sleepy/odysseus#742
No description provided.