refactor: replace globby with faster alternative

[SuperchupuDev]

globby has a count of 23 subdependencies, which is far from ideal for something as simple as globbing. This PR switches to a lighter and faster alternative using fdir and picomatch. They have both a total of zero (0) subdependencies, making this not just a faster approach, but also one that's way lighter.

This makes tsup go from 102 subdependencies down to just 88.

Note: The patterns at

tsup/src/utils.ts

Lines 69 to 77 in 57a372b

	const matchPatterns: string[] = []
	const ignorePatterns: string[] = []
	for (const pattern of patterns) {
	if (pattern.startsWith('!') && pattern[1] !== '(') {
	ignorePatterns.push(pattern.slice(1))
	} else {
	matchPatterns.push(pattern)
	}
	}

are filtered this way to properly match what globby (or more specifically, fast-glob which is used by globby) does. It has some behavior around negated patterns no other globbing library has.

Relevant prior work: dotenvx/dotenvx#278

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
tsup	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 19, 2024 2:53pm

[SuperchupuDev]

tests are failing on windows but not on linux...weird

[sxzz]

I think we can use fdir and picomatch to create a new library for lightweight glob, enhancing maintainability.

[SuperchupuDev]

the implementation here isnt big really, there's a pr in fdir to add a glob function method though (requiring a matcher function) that maybe could be used in the future. i dont really see a need of making it a lib if sometimes fdir is called in a way picomatch (therefore globbing) isnt even needed

[github-actions]

🎉 This issue has been resolved in version 8.2.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[lenovouser]

I am getting this now after upgrading to 8.2.1

[SuperchupuDev]

i am aware, when conflicts were solved, it looks like picomatch was updated to v4 which fdir's package.json doesn't allow. to fix this tsup must downgrade to picomatch v3 or fdir must release a new version which contains a fix i submitted a few days ago thecodrr/fdir#101