Skip to content

fix: use file:// URLs when dynamically importing local paths#4217

Merged
erickzhao merged 1 commit intonextfrom
fix/use-file-protocol-for-local-imports
Apr 16, 2026
Merged

fix: use file:// URLs when dynamically importing local paths#4217
erickzhao merged 1 commit intonextfrom
fix/use-file-protocol-for-local-imports

Conversation

@erikian
Copy link
Copy Markdown
Member

@erikian erikian commented Apr 16, 2026

  • I have read the contribution documentation for this project.
  • I agree to follow the code of conduct that this project follows, as appropriate.
  • The changes are appropriately documented (if applicable).
  • The changes have sufficient test coverage (if applicable).
  • The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:

Follow-up to #4214, there are other dynamic imports failing on Windows right now.

@erikian erikian requested a review from a team as a code owner April 16, 2026 14:33
@github-actions github-actions Bot added the next label Apr 16, 2026
This prevents the following error when importing absolute file paths on Windows:

```
Error [ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only URLs with a scheme in: file, data, and node are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. Received protocol 'c:'
```
@erikian erikian force-pushed the fix/use-file-protocol-for-local-imports branch from f32e8f7 to 41f4e7b Compare April 16, 2026 16:01
@erickzhao erickzhao merged commit 077ae4a into next Apr 16, 2026
19 of 20 checks passed
@erickzhao erickzhao deleted the fix/use-file-protocol-for-local-imports branch April 16, 2026 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants