docs: restructure README per guidelines + abstract architecture diagram#9
Merged
Merged
Conversation
Reorder to pitch → quickstart → depth → reference and apply the project's README style guide: - Centered masthead: tagline, inline nav bar, dynamic version badge - Convert status note to a > [!WARNING] maturity callout - Collapsible reference sections (CLI Commands, Configuration, Files on disk, Architecture, Development), each with a one-line TLDR - Add annotated config.toml example alongside the env-var table - Add collapsible ASCII architecture diagram - Add Credits section and a contrib.rocks contributors footer Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
Replace the source-tree "Workspace layout" listing (main.rs, daemon.rs, tunnel.rs, …) with an abstract component diagram keyed on responsibility (control surface, supervisor, tunnel transport, child supervision). The diagram now captures architectural essence that survives renames and refactors rather than enumerating files that move around. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
Hand-authored docs/architecture.svg captures the data flow at the role level (AI client → backend gateway → outbound WebSocket tunnel → daemon → local MCP servers) with no file/folder names, so it survives refactors. Embedded near the top of the README for first impressions; the collapsible ASCII diagram stays as the text-renderable fallback. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
Void-black background, electric-cyan (#C3FFFD) 2px wireframe boxes with solid corner tags (40/10 rule, alternating placement), white primary + graphene-grey secondary type, cyan grid markers, dashed machine boundary, and a stacked MCP-servers box. Diagram stays abstract (roles, not files). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
Represent the user's machine as a wireframe laptop and place the AI client inside it — the client runs locally and may call a remote LLM API. Split the diagram into a REMOTE (cloud) band (Edison gateway + optional third-party LLM API) and the local laptop (AI client, stdiod, MCP servers), so the real loop is visible: local client -> remote gateway -> outbound tunnel back down to the local daemon -> local MCP servers. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
The laptop already conveys what is local; nodes drawn outside it (Edison gateway, third-party LLM API) read as remote without an explicit boundary. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
Drop the solid cyan hinge bar — it read like a Hackbox tag and blurred the line between the laptop container and the active component boxes. The laptop is now a plain grey wireframe; Hackbox tags stay reserved for components. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
A sharp-cornered grey wireframe rectangle still mirrored the Hackbox shape. Redraw the laptop with rounded corners, a subtle dark fill (a physical surface the component boxes sit on), a camera dot, and a base deck — so it no longer reads as a Hackbox container. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014U6fYNnTcpivxcwHZGE5gh
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
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.
Summary
Applies the project's README style guide and makes the architecture documentation resistant to refactoring.
README (
README.md)Reordered to the guideline's invariant pitch → quickstart → depth → reference and layered on the style patterns:
•-separated nav bar, centered badge row. The version badge is now dynamic (readsCargo.tomllive via shields.io) instead of a hardcodedv0.0.1; added a repo-size badge.> [!WARNING]maturity callout — converted the old bold "Status:" line into the proper GitHub callout.<details>, each opening with a one-lineTLDR:. The top-level scan now fits ~one screen.config.tomlwith inline comments alongside the env-var table (self-documenting schema).All commands/flags were verified against the source (
-f/--follow,-n/--lines,server add --command/--arg/--working-dir/--display-name,uninstall --purge).Architecture (
ARCHITECTURE.md)Replaced the source-file tree ("Workspace layout":
main.rs,daemon.rs,tunnel.rs, …) with an abstract Components diagram keyed on responsibility (control surface, supervisor, tunnel transport, child supervision). The diagram now captures architectural essence that survives renames and refactors rather than enumerating files that move around.Deliberately omitted
Guideline sections that would have been fabrication for this project: structured Exit Codes table (the binary returns
anyhow::Result, exit 0/1 only), a self-updating leaderboard asset, and ✅/❌ Use Cases workflow branches (stdiod is set-up-once infra, not an iterative authoring loop). A🤖 Agent Quickstartwas skipped as well — stdiod is operated by developers, not driven by coding agents.Note
This branch was based on a local
mainthat is ahead oforigin/main, so the diff also includes pre-existing release-workflow and dependency-bump commits (release.yml,Cargo.toml/Cargo.lock) that were already on the branch and are not part of this docs work. They will drop out oncemaincatches up.🤖 Generated with Claude Code
Generated by Claude Code
Summary by cubic
Restructured the README to follow the pitch → quickstart → depth → reference flow and embedded a branded, abstract architecture SVG.
ARCHITECTURE.mdis now role-based, MCP-agnostic, and links to JSON Schema–generated wire types; the diagram now uses a rounded, dark laptop device surface to clarify local vs remote.Cargo.toml) and repo-size badge.[!WARNING]callout and moved details behind collapsible sections with TLDRs; added a concise CLI table, annotatedconfig.toml, Files on disk, Architecture, Development, Credits, and a contrib.rocks footer.docs/architecture.svgnear the top, styled to the Edison Hackbox brand; shows a rounded, dark laptop device surface (local), the Edison backend gateway, an optional remote LLM API, and the real loop via one outbound WebSocket tunnel; kept the ASCII diagram as a fallback and dropped the redundant cloud boundary.ARCHITECTURE.md: replaced the file-tree with a role-based overview (control surface, supervisor, tunnel transport, child supervision); called out MCP-agnostic framing and linked the JSON Schema source of truth (schema/tunnel-protocol.json) for generated wire types.Written for commit ecfba66. Summary will update on new commits.