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 on https://opencollective.com/typescript-eslint.

✅ Deploy Preview for typescript-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 your Netlify site settings.

@marekdedic How's this approach look to you?

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 6f3bdbf. 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 💌 from NxCloud.

Codecov Report

Merging #6866 (6f3bdbf) into v6 (a35a2bb) will decrease coverage by 0.02%.
The diff coverage is 70.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 the ts.getEffectiveTypeRoots() function, which I didn't know before - quick googling led me here - I understand this description so that this would exclude all types declared in some @types/... package - however, if I understand it correctly, the type roots don't include folders for packages that ship their own types - so for example types from the vue package would still be found. Is that correct or am I missing something?

Oh actually, I see isSourceFileFromExternalLibrary there as well (previously thought that was the default 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 from node_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 on program.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] unless typeRoots is specified in the tsconfig.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 - if program.isSourceFileFromExternalLibrary(declaration) supports custom node_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 the v6 branch 🙃 it'll be recreated 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! 😊