Skip to content

fix: use browser-compatible url imports#338

Merged
erunion merged 1 commit into
mainfrom
hugo/fix-browser-url-import
Jun 25, 2026
Merged

fix: use browser-compatible url imports#338
erunion merged 1 commit into
mainfrom
hugo/fix-browser-url-import

Conversation

@HugoHSun

@HugoHSun HugoHSun commented Jun 25, 2026

Copy link
Copy Markdown

🧰 Changes

Switches the url imports back to the bare Node module specifier instead of node:url.

The package already relied on consumers polyfilling Node’s url module for browser builds. After the move to tsdown, the published output started preserving node:url, which browser bundlers may not resolve through existing url fallbacks.

This keeps the runtime behavior the same while restoring compatibility for browser-bundled consumers like @readme/oas-to-snippet.

@HugoHSun HugoHSun marked this pull request as ready for review June 25, 2026 09:49
@HugoHSun HugoHSun requested a review from erunion as a code owner June 25, 2026 09:49
@HugoHSun HugoHSun requested a review from darrenyong June 25, 2026 09:49
@erunion erunion merged commit 68cb361 into main Jun 25, 2026
14 checks passed
@erunion erunion deleted the hugo/fix-browser-url-import branch June 25, 2026 18:56
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