Skip to content

Remove the vendored swiML stylesheet pipeline#62

Merged
hazzery merged 2 commits into
bartneck:masterfrom
cjdyer:refactor/remove-vendored-swiml-css
May 20, 2026
Merged

Remove the vendored swiML stylesheet pipeline#62
hazzery merged 2 commits into
bartneck:masterfrom
cjdyer:refactor/remove-vendored-swiml-css

Conversation

@cjdyer

@cjdyer cjdyer commented May 13, 2026

Copy link
Copy Markdown

This PR simplifies how SwimDSL handles swiML.css by relying on the stylesheet URL emitted by the upstream swiML transform instead of rewriting preview output to a local generated file.

Changes:

  • Stop rewriting rendered HTML from the upstream stylesheet URL to ./swiML.css
  • Remove the prepare-time CSS/font download pipeline
  • Drop the unused css and @types/css dev dependencies
  • Document that prepare now refreshes only the upstream XSLT/SEF assets
  • Mark local public/swiML.css and public/fonts/ artifacts as legacy ignored assets for existing worktrees

Why:

  • Reduces architectural confusion for new contributors by making it clear that the stylesheet is owned upstream
  • Removes repo-local tooling that existed only to support the stylesheet rewrite
  • Shrinks dependency and maintenance surface area

@cjdyer

cjdyer commented May 13, 2026

Copy link
Copy Markdown
Author

For an exported html document.

Before:
image

After:
image

@hazzery hazzery self-requested a review May 14, 2026 06:49

@hazzery hazzery 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.

We manually tested on the branches with Cory's branch and all seemed well

@hazzery hazzery merged commit e48671b into bartneck:master May 20, 2026
1 check passed
@cjdyer cjdyer deleted the refactor/remove-vendored-swiml-css branch May 20, 2026 11:49
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