Skip to content

migrate from glob β†’ tinyglobbyΒ #869

@wbolster

Description

@wbolster

hi πŸ‘‹πŸΌ,

it would be great if this repo could migrate from glob β†’ tinyglobby.

the glob dependency is old: pinned to 10.x while upstream is at 13.x (see glob on npm), and recently had security issues, e.g. #822 which was not important for this project but affects many downstreams b/c npm audit raises alarm bells about it, etc.

glob has good modern replacements, specifically tinyglobby which is lean and fast (see e.g. https://e18e.dev/blog/tinyglobby-migration and e18e/ecosystem-issues#218), stable, popular, e.g. used by vite (which even indirectly exposes it in its API; see e.g. these docs).

there are only 3 very similar call sites in this code base, e.g.

                  await glob(globAssets, {
                    absolute: true,
                    nodir: true, // We need individual files for preparation
                    ignore: options.sourcemaps?.ignore,
                  })

… which all seem to have direct tinyglobby equivalents: absolute and ignore can be used as-is, and nodir β†’ onlyFiles.

thanks in advance for your consideration!

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

Status

No status

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions