Repository navigation
docs(diagrams): fix what the rendered pages showed - #61
Merged
Merged
Conversation
The diagrams had been checked for geometry only: overlapping boxes and connectors crossing them. Rendered with the draw.io CLI, the thirteen pages showed defects that check cannot see, because it does not know where draw.io places a connector's label or how a stencil stretches. - Sequences: every actor was stretched to its lifeline's 160 px width and flattened to 46 px; actors now use a 30 px lifeline in proportion, and message labels wrap to their own arrow's length instead of 36 characters. - Economical infrastructure: the alias-records label sat on the Route 53 label, two labels were squeezed between workload boxes, and three labels landed on lane titles or borders. - Full infrastructure: the egress connectors ran through the Transit Gateway label, and the frontend route and the todos connector crossed auth-api. - Views and flows: tighter labels, a compact platform overview, a controls note sized to its text, and solid arrows for the sequential pipeline. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.
What changes
The four draw.io documents are corrected against their rendered output. No content changes; the same diagrams now read correctly.
Why
The maintainer asked whether the diagrams had been checked visually, pointing at the elongated actors. They had not: microservice-app-docs#57 to #60 were checked for geometry only, and #60 says so. A geometry check cannot see where draw.io places a connector's label or how a stencil stretches to its cell.
Tasks
No task register covers documentation in this repository. This corrects the work closed under microservice-app-docs#52, which is noted there.
Neither box applies literally: nothing is ticked here.
How it is verified
Every page was rendered with the draw.io CLI (
drawio --export --format png --page-index N) and inspected: two infrastructure pages, three views, two flows, and six sequences — thirteen in all. The first pass found the defects above. After the fixes, all thirteen were re-rendered and inspected again, and none remains.The geometry check still passes on every page, and the sequence renderer still asserts that each page's message count matches its Mermaid block:
No Terraform, tfvars, or Kubernetes manifest changes here.
Risk and rollback
None to the running system. Revert the commit.
What this PR does not do
It does not change what any diagram says. It does not commit the rendered PNGs or the generators.
🤖 Generated with Claude Code