Thanks for the PR,@RebeccaStevens!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

🙏Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently onhttps://opencollective.com/typescript-eslint.

✅ Deploy Preview fortypescript-eslint ready!

Name	Link
🔨 Latest commit	6f3bdbf
🔍 Latest deploy log	https://app.netlify.com/sites/typescript-eslint/deploys/64340e2515ccf900080f4b86
😎 Deploy Preview	https://deploy-preview-6866--typescript-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify site settings.

@marekdedic How's this approach look to you?

☁️ Nx Cloud Report

CI is running/has finished running commands for commit6f3bdbf. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Sent with 💌 fromNxCloud.

Codecov Report

Merging#6866 (6f3bdbf) intov6 (a35a2bb) willdecrease coverage by0.02%.
The diff coverage is70.00%.

@@            Coverage Diff             @@##               v6    #6866      +/-   ##==========================================- Coverage   87.46%   87.45%   -0.02%==========================================  Files         374      374                Lines       12879    12892      +13       Branches     3811     3813       +2     ==========================================+ Hits        11265    11275      +10- Misses       1229     1231       +2- Partials      385      386       +1

Hi, thanks for the alternative proposal!

Reading quickly through this, I see this mainly builds on thets.getEffectiveTypeRoots() function, which I didn't know before - quick googling led mehere - I understand this description so that this would exclude all types declared in some@types/... package - however, if I understand it correctly, the type rootsdon't include folders for packages that ship their own types - so for example types from thevue package would still be found. Is that correct or am I missing something?

Oh actually, I seeisSourceFileFromExternalLibrary there as well (previously thought that was thedefault library instead of external...) - reading the source code of that, I am confused - wouldn't this function alone do exactly what we want it to do - exclude stuff fromnode_modules? Or does that not include@types/... packages - then I'd understand the type roots bit :D

Sorry, if this does what I think it does then it's great, but as you see, I'm not that well versed with TS internals :/

This approach actually relies more onprogram.isSourceFileFromExternalLibrary(declaration).

ts.getEffectiveTypeRoots() is a little extra to support custom type root paths. The function will return[/absolute/path/to/project/node_modules/@types] unlesstypeRoots is specified in thetsconfig.json file.

Hmm, yes, that's what I thought in my second comment, sorry for the confusion. I am still not 100% why the type roots are needed - ifprogram.isSourceFileFromExternalLibrary(declaration) supports customnode_modules location, then typeRoots wouldn't be needed, right? And if it does, then this will be a half-measure -@types/ packages will be excluded, regular packages wouldn't, right?

Anyway, apart from this, LGTM.

Could you please add a test for this?

Oh, and just for the original question - IMHO, I prefer this to the alternative PR#6860

This was unintentionally auto-closed when we merged thev6 branch 🙃 it'll berecreated reopened.

👋 Hey@RebeccaStevens! Just checking in, is this still something you have time for? No worries if not - I just don't want to leave it hanging.

Closing this PR as it's been stale for a while without activity. Feel free to reopen@RebeccaStevens if you have time - but no worries if not!

If anybody wants to drive it forward, please do post your own PR - and if you use this as a start, consider adding a co-author at the end of your PR description. Thanks! 😊