Skip to content

chapters: redraw the flowcharts with Unicode box-drawing#42

Merged
yagop merged 2 commits into
mainfrom
chapters-unicode-diagrams
Jun 17, 2026
Merged

chapters: redraw the flowcharts with Unicode box-drawing#42
yagop merged 2 commits into
mainfrom
chapters-unicode-diagrams

Conversation

@yagop

@yagop yagop commented Jun 17, 2026

Copy link
Copy Markdown
Owner

Now that the commit hook permits non-ASCII inside fenced blocks (and the convention allows it), redraw all four chapter diagrams with box-drawing characters for a cleaner picture.

chapter diagram change
1 request/response boxes +/-/|┌─┐ │ └─┘ (pure char-swap, every column width unchanged)
2 streaming event sequence |/v connectors → /
3 the read/build/call/emit loop ->; the / connector realigned to sit under "call Claude"
5 the tool round-trip redrawn as a vertical / flow - this also fixes the orphaned two-line -> version that was still on main (my earlier vertical redraw, 26326f3, was stranded on the issue-5-05-tools branch and never merged)

Verification

  • No ASCII connectors (\|, v, +-, ->) remain in any diagram.
  • bunx tsc --noEmit clean; bun x vitepress build passes.
  • Rendered each diagram and eyeballed alignment (per the new review-checklist item): box right-edges line up, / sit under the node they descend from.
  • The commit itself passed the now fence-aware check-commit-chars hook - a real end-to-end test that Unicode inside ``` fences is allowed while prose stays ASCII.

🤖 Generated with Claude Code

yagop and others added 2 commits June 17, 2026 09:29
Now that the commit hook permits non-ASCII inside fenced blocks, redraw
the four ASCII diagrams with box-drawing characters for a cleaner picture:
- ch1: request/response boxes (┌─┐ │ └─┘) - char-swap, widths unchanged.
- ch2: streaming event sequence - │/▼ connectors.
- ch3: the read/build/call/emit loop - → arrows, │/▼ realigned under
  "call Claude".
- ch5: the tool round-trip - redrawn as a vertical │/▼ flow (also fixes
  the orphaned two-line "->" version that was on main).

All four render aligned; tsc and vitepress build pass.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The vertical │/▼ flow was left-aligned at each node's first character, so
the arrows did not sit under the nodes. Center the nodes around a common
column and put the connectors on that center line.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yagop yagop merged commit 7998729 into main Jun 17, 2026
1 check 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.

1 participant