fs: fix buffer input with recursive mode on readdir

[himself65]

Fixes: #58892

[nodejs-github-bot]

Review requested:

• @nodejs/test_runner
• @nodejs/url

[codecov]

Codecov Report

❌ Patch coverage is 96.29630% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 89.94%. Comparing base (6ea421a) to head (841ee6c).
⚠️ Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
src/node_file.cc	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #59220      +/-   ##
==========================================
- Coverage   90.06%   89.94%   -0.13%     
==========================================
  Files         648      646       -2     
  Lines      191065   190666     -399     
  Branches    37456    37309     -147     
==========================================
- Hits       172092   171500     -592     
- Misses      11601    11876     +275     
+ Partials     7372     7290      -82     

Files with missing lines	Coverage Δ
lib/fs.js	98.18% <100.00%> (+<0.01%)	⬆️
lib/internal/fs/utils.js	99.38% <100.00%> (+0.01%)	⬆️
src/node_file.cc	75.54% <0.00%> (-0.32%)	⬇️

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

[himself65]

this might be not a good solution. let me find another way

[aduh95]

That's definitely a wrong solution, we need Buffer to support non-unicode paths

[himself65]

yeah, I see path = getValidatedPath(path); is a misleading code, the rest of the logic is pretending path is string but it's actually not. If I change getValidatedPath to support buffer + options.encoding I would guess it will have lot of performance regression

[himself65]

I think I find a good way