fix: return absolute paths if one of the globs is absolute

[brillout]

Proposal for #6.

[brillout]

I'll add docs to the readme if we end up merging this.

[brillout]

I updated the PR to remove the filePathsRelativeToRoot option.

I don't see a use case for configuring this.

Also, the user can always change the glob to fit what he wants:

// If the user wants the file paths to be relative to root:
import.meta.importGlob('/path/to/*.js')
// Post-transform
{ "/path/to/some.js": () => import(/* ... */) }

// If the user wants the file paths to be relative to `__dirname`:
import.meta.importGlob('../../path/to/*.js')
// Post-transform
{ "../../path/to/some.js": () => import(/* ... */) }

[antfu]

BTW, we have this vitejs/vite#6526 in Vite, which I haven't implemented in this plugin. Are you aware of it, and what do you think how it should affects this implementation?

[antfu]

For multiple globs, do you think it makes sense for us to return the absolute paths if one of the globs is absolute?

import.meta.importGlob(['/path/to/*.js', './*.ts')
// -->
{ 
  "/path/to/some.js": () => import(/* ... */),
  "/src/foo.ts": () => import(/* ... */)
}

[brillout]

For multiple globs, do you think it makes sense for us to return the absolute paths if one of the globs is absolute?

import.meta.importGlob(['/path/to/*.js', './*.ts')
// -->
{ 
  "/path/to/some.js": () => import(/* ... */),
  "/src/foo.ts": () => import(/* ... */)
}

I'm not super happy about this, but I don't see an alternative.

[brillout]

BTW, we have this vitejs/vite#6526 in Vite, which I haven't implemented in this plugin. Are you aware of it, and what do you think how it should affects this implementation

👍. I'd say: it's a nice-to-have we can implement later.

I'd propose to get vite-plugin-glob out to more users, experiment, then we can see how/if we want to implement such nice-to-haves.

[antfu]

I will make a tweet later :)

[brillout]

Exciting 😀.

Btw, I'm going to release the new vps major 0.4.0 as soon as we merge the file extension restoration thing. (It's the only blocker for vps.)