add no-export-default-of-import-star eslint rule

[anonrig]

Fixes a portion of #5844

Adds a rule to disallow certain usages across the node.js directory.

[codspeed-hq]

Merging this PR will degrade performance by 18.54%

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 138 untouched benchmarks
⏩ 38 skipped benchmarks¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
❌	jsonResponse[Response]	34.9 µs	42.8 µs	-18.54%
⚡	simpleStringBody[Response]	21.7 µs	19.2 µs	+12.93%

---

_{Comparing yagiz/fix-default-export-start (5c7bc07) with main (5f00731)}

Footnotes

1. 38 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[jasnell]

Ordinarily this could be a potentially breaking change but I think we may have lucked out. If any of the internal modules had default exports then changing things like this could have broken existing code... for instance, suppose the internal_timers_promises.ts had a default export, existing code could have been doing something like...

import ntp from 'node:timers/promises';
ntp.default.whatever();

But in each of the changed cases there are no default exports. I think we're likely safe!

[vicb]

Woohoo, I verified that the PR fixes the issue with Next 16.1 🎉

Thanks @anonrig and @jasnell !

Could you please add a test in this PR (move the one I added in #5847 here instead)

We should probably update the PR title to reflect the main fix.

I also added a couple minor comments

Feel free to discard my review when comments are addressed

Addressed Victor's feedback

[vicb]

Woohoo, thanks !