fix: make glob pattern supports nested paths

[OnlyWick]

Description

closes: #19605

This fix supports matching nested paths by adding /** in the path.

I think it would be better to convert it like this.

before:

./foo/${filePath}   =>   ./foo/*
./foo/${dir}/index.js   =>   ./foo/*/index.js
./foo/${dir}.js   =>   ./foo/*.js
./foo/${dir}${file}.js   =>   ./foo/*.js
${file}.js   =>   *.js

after:

./foo/${filePath}   =>   ./foo/**
./foo/${dir}/index.js   =>   ./foo/**/index.js
./foo/${dir}.js   =>   ./foo/**/*.js
./foo/${dir}${file}.js   =>   ./foo/**/*.js
${file}.js   =>   *.js

The execution result is as follows.

Before

import micromatch from "micromatch";

const globs = ["/foo/*", "/foo/*.js", "/foo/*/foo.js"];
const paths = ["/foo/foo.js", "/foo/dir/foo.js"];

for (const glob of globs) {
  const matcher = micromatch.matcher(glob);
  const result = paths.map((p) => [p, matcher(p)]);

  console.log(`# ${glob}`);
  for (const res of result) {
    console.log(`- ${res[0]}: ${res[1]}`);
  }
  console.log();
}

# /foo/*
- /foo/foo.js: true
- /foo/dir/foo.js: false

# /foo/*.js
- /foo/foo.js: true
- /foo/dir/foo.js: false

# /foo/*/foo.js
- /foo/foo.js: false
- /foo/dir/foo.js: true

import micromatch from "micromatch";

const globs = ["/foo/**", "/foo/**/*.js", "/foo/**/foo.js"];
const paths = ["/foo/foo.js", "/foo/dir/foo.js"];

for (const glob of globs) {
  const matcher = micromatch.matcher(glob);
  const result = paths.map((p) => [p, matcher(p)]);

  console.log(`# ${glob}`);
  for (const res of result) {
    console.log(`- ${res[0]}: ${res[1]}`);
  }
  console.log();
}

# /foo/**
- /foo/foo.js: true
- /foo/dir/foo.js: true

# /foo/**/*.js
- /foo/foo.js: true
- /foo/dir/foo.js: true

# /foo/**/foo.js
- /foo/foo.js: true
- /foo/dir/foo.js: true

[OnlyWick]

It seems to be a Windows path issue🤔. I don't have a computer with Windows.

[hi-ogawa]

Hey, thanks for the PR 👋

Choosing * over **/* was a deliberate decision to match import(`./some/${file}.js`) globbing behavior as discussed in #18194. So, we don't consider the current behavior as a bug. We'll need to discuss whether changing it back would make sense.

[OnlyWick]

OK, it's really frustrating when recursive directory matching is not supported.

[patak-dev]

Let's close this for now as it isn't clear we are going to change this feature. We should get a discussion going with use cases before to gather feedback from the ecosystem if we intend to change this.