Skip to content

docs: add architecture diagram to README#59

Merged
LanNguyenSi merged 1 commit into
masterfrom
docs/readme-architecture-diagram
Jun 25, 2026
Merged

docs: add architecture diagram to README#59
LanNguyenSi merged 1 commit into
masterfrom
docs/readme-architecture-diagram

Conversation

@LanNguyenSi

Copy link
Copy Markdown
Owner

Adds a Mermaid diagram to the README so the system is legible at a glance, matching the depsight docs reference.

  • Data-flow: hook/MCP entry to router.ts to memory/loader.ts (fed by the markdown corpus, synced by agent-memory-sync / digested by memory-digest-cli) to sqlite-vec index to topic/tool/confidence gates to inject-or-stay-empty.

Verified locally: the Mermaid parses and renders under GitHub's default theme, and every node maps to a real module/file in the repo. No manual styling (keeps with the no-slop convention).

Adds a Mermaid architecture/data-flow diagram to the README so the
system is legible at a glance, matching the depsight docs reference.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@LanNguyenSi LanNguyenSi added review:tests-pass merge-approval gate prerequisite review:checklist-complete merge-approval gate prerequisite review:comments-resolved merge-approval gate prerequisite review:scope-matches-task merge-approval gate prerequisite review:evidence-logged merge-approval gate prerequisite labels Jun 25, 2026
@LanNguyenSi LanNguyenSi merged commit 172ca20 into master Jun 25, 2026
5 of 7 checks passed
@LanNguyenSi LanNguyenSi deleted the docs/readme-architecture-diagram branch June 25, 2026 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review:checklist-complete merge-approval gate prerequisite review:comments-resolved merge-approval gate prerequisite review:evidence-logged merge-approval gate prerequisite review:scope-matches-task merge-approval gate prerequisite review:tests-pass merge-approval gate prerequisite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants