docs: add comprehensive architecture document (ARCHITECTURE_v1.md)#207
Merged
forketyfork merged 3 commits intomainfrom Feb 8, 2026
Merged
docs: add comprehensive architecture document (ARCHITECTURE_v1.md)#207forketyfork merged 3 commits intomainfrom
forketyfork merged 3 commits intomainfrom
Conversation
Structured overview of the system architecture including component diagram (Mermaid), data flow diagrams, module boundary table for all 61 source files, and 12 architecture decision records in ADR format. https://claude.ai/code/session_013745dWh9pqJUxB6j82uhw9
Rename ARCHITECTURE_v1.md to ARCHITECTURE.md and restructure based on review feedback: add dependency direction rules, "Rules for New Code" section extracting key patterns from ADRs, and a "Where to Put New Code" decision table. Simplify the component diagram to show layer groups instead of individual files, and collapse the module boundary table from 50+ rows to ~18 by grouping ui/components/*, app/*, session/*, gfx/*, and shared utilities into single rows.
The app/* modules import c.zig directly for SDL type definitions in input handling, which is technically a cross-layer dependency. Document this as an explicit exception to the downward-only dependency rule.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Structured overview of the system architecture including component
diagram (Mermaid), data flow diagrams, module boundary table for all
61 source files, and 12 architecture decision records in ADR format.
https://claude.ai/code/session_013745dWh9pqJUxB6j82uhw9