Skip to content

fs,path: key glob matcher cache by platform#64571

Open
Archkon wants to merge 1 commit into
nodejs:mainfrom
Archkon:path
Open

fs,path: key glob matcher cache by platform#64571
Archkon wants to merge 1 commit into
nodejs:mainfrom
Archkon:path

Conversation

@Archkon

@Archkon Archkon commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Fixes: #64570

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/path

@nodejs-github-bot nodejs-github-bot added fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. labels Jul 18, 2026
@Archkon Archkon changed the title fs: key glob matcher cache by platform fs,path: key glob matcher cache by platform Jul 18, 2026
@codecov

codecov Bot commented Jul 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.15%. Comparing base (4a5eb1c) to head (f688be4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #64571   +/-   ##
=======================================
  Coverage   90.15%   90.15%           
=======================================
  Files         744      744           
  Lines      242517   242518    +1     
  Branches    45688    45688           
=======================================
+ Hits       218642   218650    +8     
- Misses      15358    15361    +3     
+ Partials     8517     8507   -10     
Files with missing lines Coverage Δ
lib/internal/fs/glob.js 91.27% <100.00%> (+<0.01%) ⬆️

... and 30 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.

@Archkon

Archkon commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

@lpinca Sorry to ping you and could you apply request ci for this pr? Thanks!

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Jul 21, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 21, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Archkon

Archkon commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

@lpinca I don't know what is wrong with jenkins ci tests so could you take a look at these failed test ?Should I resolve these problems ?

@Archkon

Archkon commented Jul 25, 2026

Copy link
Copy Markdown
Contributor Author

@Renegade334 @avivkeller Could you take a look at these failed tests ?

@avivkeller

Copy link
Copy Markdown
Member
02:04:39 CONFLICT (add/add): Merge conflict in Makefile
02:04:39 error: could not apply 61890720c8a... add readme and initial code
02:04:39 hint: Resolve all conflicts manually, mark them as resolved with
02:04:40 hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
02:04:40 hint: You can instead skip this commit: run "git rebase --skip".
02:04:40 hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
02:04:40 Could not apply 61890720c8a... add readme and initial code
02:04:40 Build step 'Execute shell' marked build as failure

@Archkon please rebase, looks like CI needs you to

Glob matchers capture the path platform when they are created. Include
the platform in the cache key so path.posix and path.win32 do not share
incompatible matchers for the same pattern.

Signed-off-by: Archkon <180910180+Archkon@users.noreply.github.com>
@Archkon

Archkon commented Jul 26, 2026

Copy link
Copy Markdown
Contributor Author

@avivkeller Done,and Maybe you need to apply request-ci again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

path: matchesGlob() cache does not distinguish path platforms

6 participants