path: add escapeGlob and unescapeGlob

[Xstoudi]

path: add escapeGlob and unescapeGlob

Expose minimatch escaping and unescaping capabilities on path module.

Fixes: #61258

This is an implem proposal fix #61258 using minimatch that is already-vendored. Also feel free to -1 the change if you think it doesn't have its place there.

[nodejs-github-bot]

Review requested:

• @nodejs/path

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.70%. Comparing base (ce2ec3d) to head (3671060).
⚠️ Report is 649 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #61269      +/-   ##
==========================================
+ Coverage   88.54%   89.70%   +1.15%     
==========================================
  Files         704      695       -9     
  Lines      208734   214198    +5464     
  Branches    40271    41020     +749     
==========================================
+ Hits       184823   192145    +7322     
+ Misses      15932    14114    -1818     
+ Partials     7979     7939      -40     

Files with missing lines	Coverage Δ
lib/internal/fs/glob.js	91.69% <100.00%> (+0.31%)	⬆️
lib/path.js	97.51% <100.00%> (+0.02%)	⬆️

... and 381 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

The notable-change PRs with changes that should be highlighted in changelogs. label has been added by @avivkeller.

Please suggest a text for the release notes if you'd like to include a more detailed summary, then proceed to update the PR description with the text or a link to the notable change suggested text comment. Otherwise, the commit will be placed in the Other Notable Changes section.

[Xstoudi]

Hey, no interest?

@nodejs/path

[targos]

@nodejs/path @nodejs/fs

[TheOneTheOnlyJJ]

I am wondering, why not just expose minimatch? It's already vendored, right?

[Xstoudi]

Vendoring is an implem detail, not a commitment and I'd prefer to not couple the Node's API to a third-party dependency.

Goal here is to provide stable, cross-platform glob semantic in Node rather than exposing raw minimatch behavior. windowsPathsNoEscape and magicalBraces are good illustrations of this.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/72403/