Cody Casterline

Results 183 comments of Cody Casterline

I see this seemingly-related line over in the rollup `node-resolve` plugin: https://github.com/rollup/plugins/blame/master/packages/node-resolve/src/index.js#L15 ```javascript const ES6_BROWSER_EMPTY = '\0node-resolve:empty.js'; ``` I'm not quite sure what's going on there, but I think from...

Aha, interesting. Over in https://github.com/rollup/rollup/pull/3342 some tests were recently updated to no longer use `:` in that `\0`-prefixed string. https://github.com/rollup/rollup/pull/3342/files#diff-478c0f90009219e922bc7f42abeb14c8b74fea727890acb28787d768caf1edbdL6 ```diff - var _virtual_entry1 = exports('default', "\u0000virtual:entry-1"); + var _virtualEntry1...

I've been doing a bit of investigation on my snowpack-v3 branch, which introduces another invalid file name: ``` C:\Users\Cody\code\feoblog\web-client\build\_snowpack\pkg\common>dir /r polyfill-node Volume in drive C has no label. Volume Serial...

OK, I think I've got it. rollup has this: ```js export function sanitizeFileName(name: string): string { return name.replace(/[\0?*]/g, '_'); } ``` which strips the `\0` characters, but leaves in the...

And it's not that simple, because while the function is called `sanitizeFileName` it's actually being used to sanitize full paths, which may contain a drive letter prefix like `C:`. I've...

Wooo, looks like this might be fixed soon. 🤞 https://github.com/snowpackjs/snowpack/pull/3606

So that fix got released in 3.8.5. I'm trying out 3.8.8 but ran into this blocker: https://github.com/snowpackjs/snowpack/issues/3712

> Wooo, looks like this might be fixed soon. 🤞 Nope. Have to wait for new releases of esinstall and snowpack. See: https://github.com/snowpackjs/snowpack/pull/3606#issuecomment-917779619

Opened a new bug for Snowpack to request they actually bump the dependency version so we can use the fix. :p

Well, we're coming up on a year of this issue being open. I think I'm going to try some Snowpack alternatives. Vite looks like a good one to try first:...