Skip to content

docs: add architecture overview to API reference - #620

Merged
msiemens merged 2 commits into
msiemens:masterfrom
uadhran:docs/api-architecture-overview
Aug 10, 2026
Merged

docs: add architecture overview to API reference#620
msiemens merged 2 commits into
msiemens:masterfrom
uadhran:docs/api-architecture-overview

Conversation

@uadhran

@uadhran uadhran commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Adds a high-level architecture section at the top of the API reference so readers can understand how TinyDB's core classes relate before diving into autodoc details.

Changes

  • New Architecture Overview section in docs/api.rst covering the three layers (core API, persistence, supporting components), typical data flow, and a link to extend.rst for contributors
  • New class relationship diagram at docs/_static/architecture.svg

Verification

  • uv run py.test -v --cov=tinydb — not run (docs-only change)
  • sphinx-build -b html docs/ _build/html — build succeeds; architecture.svg is copied into the output

Related: #575

@msiemens

msiemens commented Aug 6, 2026

Copy link
Copy Markdown
Owner

Thanks @uadhran! I really like the idea but the SVG diagram seems to have some overlapping:

image

Could you fix that? I really like using Mermaid for diagrams like that, it takes care of the overlapping too. Ideally, you could commit the mermaid code along with the generated SVG

Add a high-level class diagram and data-flow summary at the top of the
API docs so contributors can understand how TinyDB's core classes
relate before diving into the autodoc sections.

Related: msiemens#575
Replace the hand-drawn SVG (overlapping labels) with a Mermaid-generated
diagram. Commit architecture.mmd as the source of truth alongside the SVG.
@uadhran

uadhran commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Fixed the overlapping diagram by regenerating it from Mermaid.

  • Added docs/_static/architecture.mmd (source)
  • Regenerated docs/_static/architecture.svg from that source
  • Also rebased onto latest master

Regenerate with:

npx @mermaid-js/mermaid-cli -i docs/_static/architecture.mmd -o docs/_static/architecture.svg -b transparent

@uadhran
uadhran force-pushed the docs/api-architecture-overview branch from 2a65a64 to 4c66a26 Compare August 10, 2026 05:33
@msiemens
msiemens merged commit f16aec4 into msiemens:master Aug 10, 2026
31 checks passed
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