fix!: handle glob patterns ending with /** in CopyRspackPlugin

[inottn]

Summary

Attention: This PR includes a breaking change, but it is necessary to align with webpack's behavior.

fix #8801

A glob pattern ending with /** should match all files within a directory, not just the directory itself.
Since the standard glob only matches directories (refer to rust-lang/glob#129), we append /* to align with webpack's behavior.

Checklist

• Tests updated (or not required).
• Documentation updated (or not required).

[netlify]

✅ Deploy Preview for rspack canceled.

Built without sensitive environment variables

Name	Link
🔨 Latest commit	60ca36f
🔍 Latest deploy log	https://app.netlify.com/sites/rspack/deploys/676792adb0339100089c62c4

[codspeed-hq]

CodSpeed Performance Report

Merging #8803 will not alter performance

_{Comparing inottn:fix/copy (60ca36f) with main (ed67e97)}

Summary

✅ 1 untouched benchmarks

[hardfist]

if the glob library behaves different from webpack's glob library, maybe we should consider align glob crate behavior to globby, I'm not sure the fast-glob behaves the same as globby @shulaoda can you help check?, if it behaves the same maybe we use fast-glob in copyRspackPlugin.

BTW this is actually a breaking and should be emphasized in notice

[shulaoda]

I'm not sure the fast-glob behaves the same as globby

Yes, it aligns with the glob pattern used by webpack and includes corresponding test cases.

see https://github.com/shulaoda/fast-glob/blob/2fa17673ba4ba599a7ad21221867dc71557be68b/tests/test.rs#L8

[inottn]

BTW this is actually a breaking and should be emphasized in notice

Okay, sometimes it's hard to distinguish between fixing and breaking. I'll revise the corresponding description later to emphasize that.

[inottn]

@shulaoda fast-glob's API looks very concise. Can you help me confirm if it supports the globOptions configuration for use in the CopyRspackPlugin?

[hardfist]

Okay, sometimes it's hard to distinguish between fixing and breaking. I'll revise the corresponding description later to emphasize that.

yeah this is tricky maybe we should have some label like “alignment breaking” which means the small
breaking introduced for alignment which is allowed in minor release

[shulaoda]

@shulaoda fast-glob's API looks very concise. Can you help me confirm if it supports the globOptions configuration for use in the CopyRspackPlugin?

Yes, fast-glob only supports simple matching and doesn't support additional extensions, as these could lead to performance degradation. For more details, you can refer to #7556 related attempts made by @SyMind.