fix: Windows file:\/\// -> file:[\/]+

[laggingreflex]

On Windows (edit: it seems this can be a case on other OSes too) there seems to be file:/// i.e. 3 forward slashes, whereas here it was only accounting for 2, leaving in an extra /

Fixes #41

Checklist

• npm test, tests passing

  Weren't passing for me before this fix either.

npm run test:snap (to update the snapshot) shouldn't be needed
tests and/or benchmarks are included
~~ documentation is changed or added~~

[demurgos]

This is more complicated than this, the path <-> URL conversion escapes some characters. For Windows, there are also server paths (UNC paths) and long paths (\\?\).

I wrote a utility library to convert between File URIs and system paths (furi) to solve this c8 issue. While writing it, I found some issues in Node support for file urls so I was waiting for it. I still need to add support for UNC paths (and some doc) but it should already be usable to fix this issue.

[laggingreflex]

#46 fixes it better. Closing