💡 Replace fast-glob by tinyglobby

[Jayllyz]

Suggest an idea for this project

Most of knip deps come from fast-glob.

https://npmgraph.js.org/?q=knip

https://bundlephobia.com/package/fast-glob vs https://bundlephobia.com/package/tinyglobby

repo: https://github.com/SuperchupuDev/tinyglobby

see: https://github.com/es-tooling/module-replacements/blob/main/docs/modules/glob.md

[webpro]

Interesting. At first glance, only one feature seem to be missing: a way to stop searching when one match is found:

knip/packages/knip/src/util/glob.ts

Lines 62 to 67 in a230582

	const firstGlob = async ({ cwd, patterns }: BaseGlobOptions) => {
	const stream = fg.globStream(patterns.map(removeProductionSuffix), { cwd, ignore: GLOBAL_IGNORE_PATTERNS });
	for await (const entry of stream) {
	return entry;
	}
	};

[webpro]

Related: #678 (would be great if an alternative contains the fix)

[SuperchupuDev]

just tested and it looks like tinyglobby does not have the bug described in #678

[webpro]

Thanks @SuperchupuDev! Now I'd be even happier to start using tinyglobby. Even though the globStream feature is currently used only once in this codebase, I do like to have it available (planning to use it more). Does not have to be part of tinyglobby. For sure I'll keep an eye on tinyglobby, thanks for building it!

[SuperchupuDev]

Thanks! The problem with globStream is that I have to wait until fdir adds support for streams, there's an open issue from years ago