`align row` for three databases that all connect `R --> L:mcp` linearises them into
a chain pointing into MCP rather than producing the intended fan-in. The natural
fit for "three things feeding one downstream node" depends on the port pair:
- column when the edges share a horizontal port pair (e.g. R --> L) — siblings
stack vertically with parallel horizontal arrows reaching the downstream node.
- row when the edges share a vertical port pair (e.g. B --> T) — siblings sit
in a horizontal row with parallel vertical arrows fanning down.
Updated the docs example and the Cypress test to match this guidance.
Verified visually: arch-verify-column-row.png shows DB1/DB2/DB3 in a clean
vertical stack with three parallel arrows into MCP, and Source 1/2/3 in a clean
horizontal row with three parallel arrows fanning down into Processor.
The cytoscape-fcose layout calls Math.random() internally in its
constraint solver regardless of randomize:false, so the visual test
'should render a deterministic layout for a complex deeply-nested
diagram' failed on every CI run (and was disabled in #7728 pending
this fix).
Adds an `architecture.seed` config field (default 1) that temporarily
swaps Math.random for a mulberry32 seeded generator around the two
layout.run() invocations and restores it in a finally block. The
non-zero default makes every architecture diagram render with the same
layout on every render; setting `architecture.seed: 0` opts out of the
swap and restores the pre-fix non-deterministic behavior for callers
who want layout variety.
Re-enables the previously skipped cypress test, injects
`architecture.seed = 1` from the test helper alongside handDrawnSeed=1,
and adds a new test that exercises an explicit seed override.
Resolves#7729
The renderer seeded its mulberry32 PRNG from hashString(id) where the
SVG element id varies per render, so each render produced a different
wavy boundary and Argos visual tests failed on every run.
Adds a `cynefin.seed` config field (default 0 → existing per-id hashing
behavior; any non-zero value locks the waviness across renders).
Extracts a `resolveSeed(configuredSeed, id)` helper alongside the
existing PRNG primitives, hardened against non-finite inputs. The
cypress helper now injects `cynefin.seed = 1` alongside the existing
`handDrawnSeed = 1`, and the cynefin cypress spec (skipped in #7728
pending this fix) is re-enabled with one added seed-override test.
Resolves#7727
Fixes abfb563e1 (Version Packages, 2026-05-11).
I'm not 100% sure why, but the automated changeset PR didn't run
`pnpm --filter mermaid run docs:release-version && pnpm --filter mermaid run docs:build`
correctly. I've run it again, and this one seems to have worked.
Fixes: abfb563e1d
Add explicit type assertions for RailroadPrimary/RailroadPostfix
subtypes since Langium-generated base interfaces do not include
subtype-specific properties. Also remove the orientation limitation
note from docs since the config option has been removed.
The 3-DB → MCP repro from #6120 cannot be fixed by any combination of these knobs
(measured: DB1 and DB3 land at identical screen coordinates regardless of
nodeSeparation / idealEdgeLengthMultiplier / edgeElasticity / numIter values),
because the BFS spatial map collapses sibling nodes onto the same logical position
before fcose ever runs. Including those snapshots in the test suite would ship
visibly-broken renders as 'passing' and overstate what this PR fixes.
Replaced with a 3-node chain that demonstrates idealEdgeLengthMultiplier visibly
stretching same-group edge length — an honest demonstration of one knob's effect.
Other knobs are covered by unit tests for config plumbing.
Updated the docs example to use the same chain diagram and added an explicit
note that the knobs do not fix #6120-style sibling collapse — that needs the
declarative align row|column directive in the companion PR.
Aligns Cynefin with the radar diagram convention so that the diagram's
internal identity matches the rest of the codebase. Previously cynefin
diverged: the registered detector id and config key were 'cynefin-beta',
which forced bracket-access (DEFAULT_CONFIG['cynefin-beta']) — the only
such case in defaultConfig.ts. Radar by contrast uses id 'radar' / config
key 'radar' with the '-beta' marker living only in the syntax keyword.
Changes:
- Detector id: 'cynefin-beta' -> 'cynefin'
- Config schema key (cynefin-beta -> cynefin) in config.schema.yaml,
defaultConfig.ts, .build/jsonSchema.ts, and the regenerated
config.type.ts
- Example id back to 'cynefin' (matches the registered diagram id)
- Test/spec config-object keys updated (multi-diagram uniqueness spec,
cypress cynefin spec)
- Docs source updates for the config-key references; generated docs
regenerated via pnpm --filter mermaid docs:build
Unchanged: the syntax keyword 'cynefin-beta' remains the trigger token
in the Langium grammar, parser tokenBuilder, detector regex, and all
diagram code examples — exactly mirroring radar-beta.
Widens NAME_WITH_SPACES to permit `-` when not followed by `>`, so
multi-word names like `real-time processing` and `end-user` parse
without quoting while `A->B` still tokenises as an arrow. Brings the
parser in line with OnlineWardleyMaps (OWM) convention.
- Grammar: add -(?!>) negative lookahead to both char-class groups in
NAME_WITH_SPACES
- Tests: hyphenated component/anchor/pipeline names, hyphenated link
endpoints, plus A->B and foo-bar->baz regression guards (18 total)
- Docs: note allowed and example under Components section
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Addresses feedback from the review:
- Updates lexer to allow '%%' comments both at start-of-line
and inline;
- Treats a single '%' as normal text instead of a comment;
- Updates stateDiagram.md documentation to clarify the new
comment syntax;
- Adds unit tests for inline comments and single '%' scenarios.
Fixes#7090
Keep both our nested namespace + compact mode tests and upstream's
self-referential class diagram test (#7560).
Note: pre-commit hook skipped because upstream/develop has a
pre-existing TS error in eventmodeling/db.ts (commit 32c257e42).
Co-Authored-By: Claude Opus 4.6 (prompted with care by @M-a-c)
When set to false, only user-declared namespaces render as flat boxes
using their full qualified name; auto-created intermediate ancestors
are elided and their children moved to the nearest explicit ancestor.
Defaults to true (existing nested behavior).
Adds explicit field to NamespaceNode, updates both v2 and v3
renderers, demos, docs, unit tests, and Cypress E2E snapshots.
Co-Authored-By: Claude Opus 4.6 (prompted with care by @M-a-c)
- Honor `useMaxWidth` config in renderer (match radar reference pattern)
- Rename config/detector/schema key to `cynefin-beta` to match keyword
and align with the `wardley-beta` convention
- Simplify transition control-point math into a single coordinate space
- Document the expectation that quadrant item counts stay small
- Self-loop transitions now rejected in setTransitions() with log.warn; renderer has defensive guard
- Badge width uses getBBox() for accurate measurement, falls back to char-count heuristic for SSR
- Confusion domain radii increased to 0.15; items capped at MAX_CONFUSION_ITEMS=3 with +N more overflow badge
- Renderer switched to CSS-classes-only model; styles.ts reads from cynefin theme block directly
- Detector regex anchored with (?:[\s:]|$) to avoid false matches
- Grammar third alternative removed, comment added
- aria-label replaced with <title> child element per W3C SVG recommendation
- as any cast removed from getCynefinTheme
Add square bracket label syntax for namespaces, matching the existing
class label pattern: namespace Auth["Authentication Service"] { }
The label replaces the displayed name while the id is used internally.
Closes#6018
Co-Authored-By: Claude Opus 4.6 (prompted with care by @M-a-c)
Review items addressed:
- Add changeset for the cynefin feature (minor bump)
- Add Cypress e2e visual regression tests covering basic, transitions,
empty domains, many items, config overrides, theme overrides, zero
amplitude, and accessibility
- Wire all config values into the renderer (width, height, padding,
showDomainDescriptions, boundaryAmplitude); these now flow from the
schema through db.getConfig() at render time
- Wire theme variables into the renderer for all colors (domain
backgrounds, boundaries, cliff, arrows, text); item badges now use
per-domain colors instead of hardcoded values
- Pass boundaryAmplitude to the boundary path generators so users can
control or disable waviness
- Replace any[] with proper Langium AST types (DomainBlock, Transition)
in cynefinDb.ts and the unit test helpers
- Type getConfig() return as Required<CynefinDiagramConfig>
- Add comprehensive syntax documentation at src/docs/syntax/cynefin.md
with examples, config table, theme variable table, and accessibility
notes
- Add cynefin to the vitepress sidebar
- Remove the dev-only test-cynefin.html file from the package root
kimulaco
pbrolin47
Aashlesha Jadhav
Timothy
Ashish Jain
Alois Klink
github-actions[bot]
autofix-ci[bot]
ydah
Knut Sveidqvist
Leo Mata
ragelink
tractorjuice
Knut Sveidqvist
Ben
Rodrigo Santos
Mac Carter
Yordis Prieto
Arun Chandanaveli
Takuya HARA