Skip to content

Bundle ranges-apply into CJS dist#39

Merged
avigoldman merged 2 commits intomainfrom
fix-ranges-apply-bundling
Feb 19, 2026
Merged

Bundle ranges-apply into CJS dist#39
avigoldman merged 2 commits intomainfrom
fix-ranges-apply-bundling

Conversation

@avigoldman
Copy link
Copy Markdown
Contributor

Follow-up to #37. ranges-apply and its deps (ranges-merge, tiny-invariant) are ESM-only packages. Without bundling them, CJS consumers (like server-procedures in parcel) fail at runtime with import errors.

Adds noExternal: ["ranges-apply", "ranges-merge", "tiny-invariant"] to the tsup configs for html-email-formatter, html-preserve-comment-whitespace, and html-process-conditional-comments.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Feb 19, 2026

🦋 Changeset detected

Latest commit: f240699

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
@ciolabs/html-email-formatter Patch
@ciolabs/html-preserve-comment-whitespace Patch
@ciolabs/html-process-conditional-comments Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@avigoldman avigoldman merged commit 99b5892 into main Feb 19, 2026
1 check passed
@github-actions github-actions Bot mentioned this pull request Feb 19, 2026
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