Skip to content

Preserve all installation steps when converting documentation. - #67

Open
Floating-Y wants to merge 1 commit into
deeplethe:devfrom
Floating-Y:fix/docs-install-steps
Open

Floating-Y wants to merge 1 commit into
deeplethe:devfrom
Floating-Y:fix/docs-install-steps

Conversation

@Floating-Y

Copy link
Copy Markdown
Contributor

Why

CodeTabs conversion dropped Toast's Toaster setup, Attachment's Button command and Calendar's Button prerequisite.

What changes

Preserve Command and Manual as separate ordered block lists, reusing existing parsing and rendering for steps, prose, code, filenames and lazy source loading. Retain compact installation syntax. Add five regression tests to CI without changing upstream MDX, component behavior or generated artifacts.

How it was checked

  • Linux Node 24.21.0: npm ci, 13 Node tests, docs, typecheck and build passed. Five new tests fail against the old converter.
  • git diff --check and registry.json diff check passed.
  • Checked affected documentation in both themes, package-manager switching, copy success feedback and source display.
  • /dev/examples: 389 mounted, 0 failed.
  • Temporary consumer built and displayed toast.add() notifications with Toaster mounted; Calendar and Attachment rendered with Button present.

Limitations: Node 22 was not reproduced locally. Native Windows docs/build hit the existing path-to-slug issue; typecheck lacked the generated catalog. Consumer Button was manually copied, so its CLI installation path was not verified. Toast's upstream Manual also omits its existing Button prerequisite.

Before review

  • Every commit is signed off.
  • Typecheck and build pass in Linux as described above.
  • /dev/examples reports 0 failed.
  • Affected documentation inspected in both themes.

Signed-off-by: danwood <118035379+Floating-Y@users.noreply.github.com>

This branch has not been deployed

No deployments
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