[Refactor] no-extraneous-dependencies improve performance

[meowtec]

In my project this will reduce the time from

import/no-extraneous-dependencies         |  3442.375 |     5.9%

to

import/no-extraneous-dependencies         |  1903.296 |     3.5%

[codecov]

Codecov Report

Base: 95.20% // Head: 95.15% // Decreases project coverage by -0.05% ⚠️

Coverage data is based on head (7f251b2) compared to base (d7c4f94).
Patch coverage: 85.71% of modified lines in pull request are covered.

❗ Current head 7f251b2 differs from pull request most recent head 9b9ae94. Consider uploading reports for the commit 9b9ae94 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2374      +/-   ##
==========================================
- Coverage   95.20%   95.15%   -0.06%     
==========================================
  Files          66       66              
  Lines        2818     2828      +10     
  Branches      948      949       +1     
==========================================
+ Hits         2683     2691       +8     
- Misses        135      137       +2     

Impacted Files	Coverage Δ
src/rules/no-extraneous-dependencies.js	98.16% <85.71%> (-1.84%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[ljharb]

what happens if the package.json file has invalid syntax, or doesn't exist?

[ljharb]

i very intentionally had all of these pass all three arguments, for consistency. if we're changing that, then this only takes 1 argument.

Suggested change

	if (isAbsolute(name, settings)) { return 'absolute'; }
	if (isAbsolute(name)) { return 'absolute'; }

(similar feedback on all the others)

[ljharb]

this change moves isBuiltIn below absolute/relativeToParent/index/relativeToSibling - this might be a breaking change for anyone relying on the import/core-modules setting, which currently takes precedence.

[meowtec]

Could we make this change in next major version?

[ljharb]

There's unlikely to be a major version any time soon, if ever.

[ljharb]

@meowtec i'd love to incorporate these speed improvements if we can avoid the breaking change - can you update this PR to do that?

[ljharb]

I've reordered the commits, and added "clear the cache on program exit" so that long-running eslint processes won't have a memory leak. I plan to pull the cache commit into main directly, and leave this PR containing just the "lazy resolve" commit so we can keep working on it.