fix: handle drive-relative paths on windows

[Andrej730]

Resolves #196

Just when I thought I learned all Windows quirks, I've found one more...

Turned out, on Windows L: is not the same thing as L:\. It's called drive-relative path (see https://learn.microsoft.com/en-us/dotnet/standard/io/file-path-formats#apply-the-current-directory) and L: ends up being replace with cwd.

import { resolve } from "node:path";

// cwd: L:\oblivion\tinyglobby
console.log("cwd:", resolve("."));
// resolve("L:"): L:\oblivion\tinyglobby
console.log('resolve("L:"):', resolve("L:"));
// resolve("L:/"): L:\
console.log('resolve("L:/"):', resolve("L:/"));
// resolve("L:\\"): L:\
console.log('resolve("L:\\"):', resolve("L:\\"));

In my case I've stumbled upon, I had L:\test1\repro.js running L:\test\*.py pattern while being in L:\test1 as cwd.
So naturally, glob have found a common root to start from - L:, but when absolute is set to true.

The call chain goes like this

glob calls .withPromise

tinyglobby/src/index.ts

Line 75 in 5779202

return crawler ? formatPaths(await crawler.withPromise(), relative) : [];

Which constructs a Walker.

https://github.com/thecodrr/fdir/blob/790c182eb54809ea7bc14f1bcb40acce89506bb4/src/api/async.ts#L21

Which calls normalizePath.

https://github.com/thecodrr/fdir/blob/790c182eb54809ea7bc14f1bcb40acce89506bb4/src/api/walker.ts#L40

Which calls resolve on root, but only in case if resolvePaths is set.

https://github.com/thecodrr/fdir/blob/790c182eb54809ea7bc14f1bcb40acce89506bb4/src/utils.ts#L33-L39

Which is set, when constructing options for a crawler, from absolute option:

tinyglobby/src/crawler.ts

Line 74 in 5779202

resolvePaths: absolute,

Since L: root comes from normalizePattern call, I've added a check to ensure it appends / in case it's drive-relative path to make it just abs path at drive root and added a test.

I've created a workflow to reproduce the issue before and after the fix https://github.com/Andrej730/tinyglobby/actions/runs/24131845264/job/70410134306

@SuperchupuDev can you please take a look?

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/tinyglobby@197

commit: f8c72fc

[SuperchupuDev]

in addition to my comments, also remember to run biome so that ci is green 👍

[SuperchupuDev]

thank you for reporting the issue and sending a fix!!!! 👍