feat: migrate deprecated diagram to new yuml API - #190
Merged
Merged
Conversation
The current diagram has a 'PLEASE MIGRATE' hint on it. URL migration (yuml.me → app.yuml.me v1) - Base URL: https://yuml.me/diagram/class/ → https://app.yuml.me/diagram/v1/class/plain/ - Added .svg suffix required by v1 - Old style/direction params (;plain;dir:TB;scale:60) replaced by inline @direction LR DSL metadata and ?dir query param (which turned out not to work — only the inline DSL directive does) - DSL is now encodeURIComponent-encoded before embedding in the URL (raw [, ], {, } caused 500 errors in v1) Node disambiguation fix - Old code used a literal %20 in the DSL to give entity set boxes a different name (Books ) from same-named entity type boxes (Books). v1 normalizes/trims whitespace in node names, collapsing them together. - Replaced with U+200C (ZWNJ), an invisible zero-width character that v1 treats as part of the name, keeping the nodes distinct while rendering identically. Legend - @direction LR embedded in DSL for the horizontal layout - Rendered via <img width="450"> for a smaller display size ER diagram - Rendered as a markdown linked image [](url) so clicking opens the full-size SVG
chgeo
marked this pull request as ready for review
September 16, 2026 17:20
daogrady
self-requested a review
September 16, 2026 17:39
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.

The current diagram has a 'PLEASE MIGRATE' hint on it.
So:
Changes in detail:
URL migration (yuml.me → app.yuml.me v1)
Node disambiguation fix
Legend
@directionLR embedded in DSL for the horizontal layout<img width="450">for a smaller display sizeER diagram
Example from the test base:
test/lib/compile/data/example.openapi3.jsonNew:
Old