TypeScript icon indicating copy to clipboard operation
TypeScript copied to clipboard

Add option to include default typeRoots when overriding typeRoots

Open ide opened this issue 6 years ago • 21 comments

Search Terms

default typeRoots

Suggestion

By default, when typeRoots is unspecified, TypeScript looks for node_modules/@types in the project's directory and all ancestor directories, similar to Node's module resolution algorithm. This is covered in the tsconfig.json docs and is implemented here.

When specifying typeRoots in tsconfig.json, this overrides the default typeRoots entirely. You can include "node_modules/@types" in your typeRoots array, but that does not include ancestor directories.

This is a feature request to provide a way, such as a new option, to include the default type roots in addition to the overridden values.

Use Cases

The primary use case is within a repository using Yarn workspaces. Yarn hoists npm packages to the workspace root. This means you can end up with a directory hierarchy like this:

.
├── node_modules
│   └── @types
│       └── dependency
└── packages
    └── example
        ├── example.ts
        ├── package.json
        └── tsconfig.json

If tsconfig.json wants to include node_modules/@types in the workspace root (in addition to other type declaration directories specified in typeRoots), it needs to look like this:

{
  "compilerOptions": {
    "typeRoots": ["../../node_modules/@types", ...]
  }
}

Examples

This feature request is a quality-of-life improvement to instead allow for:

{
  "compilerOptions": {
    "typeRoots": [...],
    "includeDefaultTypeRoots": true
  }
}

The default value of this option would be false and would not be a breaking change.

Checklist

My suggestion meets these guidelines:

  • [x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
  • [x] This wouldn't change the runtime behavior of existing JavaScript code
  • [x] This could be implemented without emitting different JS based on the types of the expressions
  • [x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
  • [x] This feature would agree with the rest of TypeScript's Design Goals.

ide avatar Apr 11 '19 01:04 ide

We need this exact same thing for the exact same reasons. I was looking for a way to say, "In addition to the normal @types folders resolution that Typescript does, please also add the following folders." There is no way to do this that I am aware of.

Taytay avatar May 19 '20 16:05 Taytay

This would be useful for me too. Specifically, if in addition to the default @types org I could specify an additional org to check.

I would like to include some custom global utility types in each of my typescript projects that extends a particular tsconfig. These global utility types would be available as an npm package at a path “@myorg/global-types”. I would include this package as a dev dependency. I would then configure the tsconfig typeroots to look first in “@myorg” then fallback to “@types”. I would then add “global-types” to the types array in the tsconfig and all would be well.

I’m also open to being told if I’m missing something. Right now, I use my yarn magic to fake like my types are in @types even tho they are not. This works, but would break in the event that I used one of my packages (which are all git submodules) without the mono-repo package surrounding it - which will eventually be a problem.

nashspence avatar May 28 '20 00:05 nashspence

Another way to support this could be to add some kind of glob support to typeRoots, so you could write this:

{
  "compilerOptions": {
    "typeRoots": ["./node_modules/@types/**", ...]
  }
}

dummdidumm avatar Aug 14 '20 15:08 dummdidumm

@dummdidumm it seems a bit over complicated... I have never seen that type of glob. But this PR could be quite easy, in the code, they are looking after each ancestor node_modules like the node behavior, just adding a condition here and modifying the config schema. "includeDefaultTypeRoots" is a bit long (but "allowSyntheticDefaultImports" too) so maybe "extendsTypeRoots" ?

eytienne avatar Aug 14 '20 20:08 eytienne

This is now a much more serious issue with the introduction of Pnp. Without this feature, there is no way to include additional typeRoots on top of the pnp roots without overriding them completely.

yinzara avatar Aug 29 '20 07:08 yinzara

I have a fix for this on my fork, but I cannot PR it due to this issue not being on the backlog: https://github.com/jonrimmer/TypeScript/tree/include-default-type-roots-option

@RyanCavanaugh What feedback are you waiting for on this issue?

jonrimmer avatar Mar 03 '21 15:03 jonrimmer

Running into the same issue when we want to use a private NPM Types module in a Serverless mono-repo

https://stackoverflow.com/questions/66495073/automatically-import-typescript-namespace-from-namespace-type-only-npm-module-in

Borduhh avatar Mar 05 '21 15:03 Borduhh

+1, I'm shipping a typescript source repository and it would be easier if I could specify the types it defines internally rather than needing to build + load the types separately

psalv avatar Apr 06 '21 15:04 psalv

Bump.

@RyanCavanaugh - How can any of us be helpful here in moving this forward?

lobsterkatie avatar Apr 27 '21 18:04 lobsterkatie

This would be a very helpful feature.

tzachbon avatar Aug 30 '21 07:08 tzachbon

+1, This would really be useful.

anurag-roy avatar Sep 30 '21 15:09 anurag-roy

@RyanCavanaugh Any updates on this?

anurag-roy avatar Sep 30 '21 15:09 anurag-roy

nth-ing that this is a big problem when using yarn pnp in a monorepo.

fiveable-jferg avatar Nov 19 '21 01:11 fiveable-jferg

Any news on this?

Maximaximum avatar Dec 21 '21 17:12 Maximaximum

I would highly appreciate this feature too ...

stefancplace avatar Feb 18 '22 10:02 stefancplace

same :+1:

gwro avatar Mar 04 '22 12:03 gwro

same 👍

shrinktofit avatar Apr 25 '22 13:04 shrinktofit

+1, awaiting this good feature

SantyWang avatar Apr 25 '22 13:04 SantyWang

same 👍

nacho4d avatar Aug 01 '22 15:08 nacho4d

+1

fsmiamoto avatar Aug 06 '22 18:08 fsmiamoto

Can you please stop typing no value comments like "+1"? It certainly doesn't help the development process of this issue and you're sending useless notifications, emails to people who subscribed to this thread and waiting actual feedback.

Use 👍🏻 reaction instead.

kutsan avatar Sep 22 '22 11:09 kutsan

@RyanCavanaugh pinging you as you originally added the labels for this issue.

Could you consider re-evaluating this and maybe adding it to the backlog? This feature would be nice to have for monorepo setups.

My case is for using pnpm workspaces where I have <root>/node_modules but also <root>/packages/<package>/node_modules. I would like to define my default typeRoots in my root tsconfig and have my packages add their node_modules/@types to their own tsconfig (which extends the base one).

nVitius avatar Dec 12 '22 20:12 nVitius

For people who have this issue in Monorepo context, you can "put" your package in a @types organisation with something like this:

{
  "name": "@types/my-monorepo-global-types"
}

and it'll be picked up by typescript automatically without having to override the typeRoots option.

s0ber avatar Feb 16 '23 11:02 s0ber

That's a fantastic idea @s0ber to avoid having to define typeRoots!

Pro tip: if you intend on applying these types globally in your monorepo, do not build the @types/my-monorepo-global-types package. Let the declaration files (*.d.ts) load from src itself - otherwise, you will bump into this problem: Cannot find type definition file for ... The file is in the program because: Entry point for implicit type library.

This is what I have in package.json:

{
  "name": "@types/my-monorepo-global-types",
  "main": "./src/index.d.ts",
  "types": "./src/index.d.ts"
}

And these are the contents of my src directory:

./src
├── assets.d.ts
├── index.d.ts
└── jazzicon.d.ts

PaulRBerg avatar Jun 05 '23 09:06 PaulRBerg

I am trying to add custom Playwright matchers to a SvelteKit repo. The docs say to add global.d.ts using either typeRoots or include.

Using typeRoots will break resolution of every other type because of this issue. Because SvelteKit generates its own tsconfig in .svelte-kit/tsconfig.json and uses "extends": "./.svelte-kit/tsconfig.json" to include it, I can't use include without copying the include section from that file into the main tsconfig.json.

This means I somehow have to be aware when SvelteKit has changed what it generates so I can re-copy it. Brittle.

Why on earth, if tsconfig.json allows extends, does it not merge things rather than just clobbering them?

The solution from @s0ber is great for monorepo users, but not much use for anyone else. Please could we have a works-for-everyone fix for this?

(Update: I have just noticed that I can include global.d.ts using files, which SvelteKit doesn't set, so that's the simple solution! Not sure why the Playwright docs don't mention this option :)

unikitty37 avatar Jul 28 '23 08:07 unikitty37