Skip to content

docs: add LLM equivalents for Server SDK diagrams - #612

Merged
Devon-White merged 6 commits into
mainfrom
Devon/388-sdk-diagram-equivalents
Aug 21, 2026
Merged

docs: add LLM equivalents for Server SDK diagrams#612
Devon-White merged 6 commits into
mainfrom
Devon/388-sdk-diagram-equivalents

Conversation

@Devon-White

@Devon-White Devon-White commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Stacks on #611. Covers all 46 raster diagram references across 30 Server SDK guide pages: human Frames are excluded from Markdown exports, 38 diagrams receive Mermaid equivalents, five receive structured-Markdown equivalents, and three Dashboard captures rely on complete adjacent numbered instructions. Adds a diagram-specific authoring rule and local/manual preview acceptance cases. The export checker is intentionally not wired into CI.

Validation:

  • manually compared all 46 source assets with their LLM-view content
  • yarn test:scripts
  • yarn fern-md-check (all 2,824 MDX files valid)
  • git diff --check
  • yarn check-llm-exports after the Fern preview deploys

Part of #388.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

@Devon-White
Devon-White force-pushed the Devon/388-sdk-diagram-equivalents branch from 4c28711 to 4775e03 Compare August 19, 2026 17:29
@Devon-White
Devon-White force-pushed the Devon/388-sdk-diagram-equivalents branch 2 times, most recently from b6c26a4 to 9ddd417 Compare August 19, 2026 18:22
@Devon-White
Devon-White force-pushed the Devon/388-sdk-diagram-equivalents branch from 9ddd417 to 94f7de0 Compare August 19, 2026 19:07
hey-august
hey-august previously approved these changes Aug 21, 2026

@hey-august hey-august left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

  • All 46 diagram references are inside <llms-ignore> (38 Mermaid diagrams, 5 structured-Markdown equivalents, 3 covered by adjacent steps)
  • Image equivalents look good
  • All 38 Mermaid fences have correct syntax.
  • Manifest routes match page slugs.

I fixed a casing error in the call-transfer table and the manifest. It said final=true when it was supposed to be final=True.

@Devon-White
Devon-White dismissed hey-august’s stale review August 21, 2026 18:57

The merge-base changed after approval.

Base automatically changed from Devon/388-callouts-widgets to main August 21, 2026 19:07
@Devon-White
Devon-White force-pushed the Devon/388-sdk-diagram-equivalents branch from 9d91396 to 9f6018b Compare August 21, 2026 19:16

@hey-august hey-august left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving again.

Post-rebase content verified identical to what was reviewed. The mdx-diagrams.md merge with main's themed-SVG rule resolves cleanly.

@Devon-White
Devon-White merged commit 1cc8957 into main Aug 21, 2026
2 checks passed
@Devon-White
Devon-White deleted the Devon/388-sdk-diagram-equivalents branch August 21, 2026 20:55
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.

2 participants