[DRAFT] Turbopack: Always encode turbopack:///[fs]/ urls as valid URLs

[bgw]

We have a mix of some code that treats turbopack:/// urls as proper URLs and attempts to percent encode/decode them, and some code that assumes it's just a dumb prefixed file path.

They look and smell like URLs, and the source map spec says they should be URLs, so let's treat them as actual URLs everywhere.

TODO

• Replace remaining callsites of the urlencoding crate with percent-encoding so that we're only using one encoding library.
• Adjust changes from Turbopack: sourcemaps for JSON modules #84611
• Make sure I didn't horribly break everything.

[bgw]

• [DRAFT] Turbopack: Always encode turbopack:///[fs]/ urls as valid URLs #84362 👈 (View in Graphite)
• canary

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #84362 will not alter performance

_{Comparing bgw/turbopack-urls-are-urls (5f6f616) with canary (b90559c)}

Summary

✅ 17 untouched
⏩ 3 skipped¹

Footnotes

1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[mischnic]

#84611 added another place that will have to be adjusted