feat(tsconfig): support JSONC comments in tsconfig.json files

[xiaoxiaojx]

Summary
Fixes #475

What kind of change does this PR introduce?
feat

Did you add tests for your changes?
Yes

Does this PR introduce a breaking change?
No

If relevant, what needs to be documented once your changes are merged or what have you already documented?
Nothing

[codecov]

Codecov Report

❌ Patch coverage is 74.74747% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.67%. Comparing base (f7f15d9) to head (3cb56d5).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
lib/util/strip-json-comments.js	73.03%	20 Missing and 4 partials ⚠️
lib/util/fs.js	88.88%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #476      +/-   ##
==========================================
- Coverage   93.42%   92.67%   -0.76%     
==========================================
  Files          49       50       +1     
  Lines        2388     2483      +95     
  Branches      719      751      +32     
==========================================
+ Hits         2231     2301      +70     
- Misses        128      149      +21     
- Partials       29       33       +4     

Flag	Coverage Δ
integration	92.67% <74.74%> (-0.76%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[alexander-akait]

The main problem here what readJson will not cache fs calls, because we have options here, we cache only calls without options, maybe we need to introduce readJsonc

[xiaoxiaojx]

Here, we cache the fs calls. We’re not passing the arguments through to FileSystem — this is a readJson helper wrapped in utils. I’m not sure whether we still need to implement a readJsonc method in FileSystem.
https://github.com/webpack/enhanced-resolve/pull/476/changes#diff-68a304bd1f2cab3fd1377549ad55b1bd26e250aa52bf9afbba06a6da4280876fR41

[alexander-akait]

Oh, I see, thanks

[alexander-akait]

@xiaoxiaojx let's rebase, wrong order merge 😞 and we can merge