feat: allow sort files

[userquin]

DO NOT MERGE YET: we need to await tinyglobby release 🤞 (SuperchupuDev/tinyglobby#168)

I need to check when sortFilesByGlobPrecedence should be called, I'm not sure how dirs and globs are "connected" here, this comment confuse me https://github.com/unplugin/unplugin-vue-components/blob/main/src/types.ts#L94. I guess we also need to check if globs there from raw options before calling sortFilesByGlobPrecedence here https://github.com/unplugin/unplugin-vue-components/pull/885/files#diff-c02b4d62dc54667a8a1243f8ddc5ae0a604cc5d7218d94fd69bfda1c8514ce1fR215-R222, this should be:

  private* preparePaths(): Generator<string, undefined, void> {
    if (this.options.sortByGlob && this.hasGlobs) {
      yield* sortFilesByGlobPrecedence(this, this._componentPaths)
    }
    else {
      yield* this._componentPaths
    }
  }

where hasGlobs should be initialized in the constructor using this.hasGlobs = rawOptions.globs?.lenght > 0

SuperchupuDev/tinyglobby#166

closes #831

[bolt-new-by-stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[userquin]

We need to change the local tinyglobby tgz copy with pkg-pr-new from this PR (in a few minutes I'll send an update ✔️ done): SuperchupuDev/tinyglobby#168

and once released 🤞 just update the dependency.

[userquin]

@antfu any chance to add pkg-pr-new in the repo? I can send a PR to add it, I only need pkg-pr-new installed, oh wait, this is unplugin, it should be ready iirc

[userquin]

missing * before sort

[userquin]

arggg the linter removing it, I need to check @antfu/eslint-config to disable the rule.

Looks like this works, luckely the object-shorthand rule not being applied ;) (check the screenshot below):

sort: function* (_root, files): Generator<string, undefined, void> {
  yield* files.sort((a, b) => b.localeCompare(a))
},

[antfu]

Let's put it in the pnpm-workspace.yaml

[userquin]

Done

[userquin]

Add some warning here about performance when enabling this option, will be O(F * M) where F is the number of files and M the "picomatch" search patterns logic, check the sortFilesByGlobPrecedence function.

[userquin]

Maybe we can add built-in asc/desc sort support, changing sort to:

sort?: 'asc' | 'desc' | ((root: string, files: string[]) => Generator<string, undefined, void>)

[userquin]

If asc/desc here and sortByGlob enabled just show a warning disabling sortByGlob