fix(engine): replace str::replace with cow_replace in module loader

[iammdzaidalam]

Closes #5140

Replaces str::replace with cow_utils::CowUtils::cow_replace in the module loader to satisfy clippy::disallowed_methods and avoid unnecessary allocation.

Follows the pattern introduced in #4123.

[jedel1043]

It would be better to use the trait directly instead of importing

cow_utils::CowUtils::cow_replace(specifier, '/', "\\")

Also, using cow_replace then into_owned immediately after is... an interesting decision, because it's basically nullifying the use of cow_replace.

[iammdzaidalam]

Thanks, resolved 👍

[jedel1043]

LGTM

[github-actions]

Test262 conformance changes

Test result	main count	PR count	difference
Total	52,963	52,963	0
Passed	50,073	50,073	0
Ignored	2,072	2,072	0
Failed	818	818	0
Panics	0	0	0
Conformance	94.54%	94.54%	0.00%

Tested main commit: 50f0103d9188d33862efbc38397a81e497973632
Tested PR commit: 7c5509a9d66cd0c12ebf0b44cea4e2b6bea46408
Compare commits: 50f0103...7c5509a

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.35%. Comparing base (6ddc2b4) to head (7c5509a).
⚠️ Report is 889 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #5141       +/-   ##
===========================================
+ Coverage   47.24%   59.35%   +12.11%     
===========================================
  Files         476      580      +104     
  Lines       46892    63181    +16289     
===========================================
+ Hits        22154    37502    +15348     
- Misses      24738    25679      +941     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.