Set default types array to []; support "*" wildcard
#63054
+4,629
−59,159
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
typescript-bot
added
Author: Team
For Uncommitted Bug
Collaborator
|
Looks like you're introducing a change to the public API surface area. If this includes breaking changes, please document them on our wiki's API Breaking Changes page. Also, please make sure @DanielRosenwasser and @RyanCavanaugh are aware of the changes, just as a heads up. |
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request implements a breaking change to TypeScript's types compiler option, setting its default to an empty array ([]) instead of automatically including all packages from node_modules/@types. A wildcard ("*") is introduced to allow users to opt into the old "include all" behavior for backward compatibility.
Changes:
- Modified
getAutomaticTypeDirectiveNamesto return[]by default whentypesis undefined and support"*"wildcard expansion - Added
usesWildcardTypeshelper function to centralize wildcard detection logic - Updated error messages to suggest adding types to the
typesfield in tsconfig when wildcard is not present - Updated 29+ tests to explicitly use
"types": ["*"]where automatic type discovery was relied upon
Reviewed changes
Copilot reviewed 171 out of 265 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/compiler/moduleNameResolver.ts | Core logic: changes getAutomaticTypeDirectiveNames to default to [] and support wildcard expansion |
| src/compiler/utilities.ts | Adds usesWildcardTypes helper to check if types array includes "*" |
| src/compiler/checker.ts | Updates error messages to conditionally suggest adding types to tsconfig |
| src/compiler/programDiagnostics.ts | Updates file include diagnostics to handle wildcard |
| src/compiler/watch.ts | Updates watch mode file include reasons for wildcard |
| src/jsTyping/jsTyping.ts | Updates type discovery to use wildcard check |
| tests/cases/fourslash/server/* | Adds "types": ["*"] to tests requiring auto-discovery |
| tests/cases/conformance/* | Adds // @types: * directive to compiler tests |
| tests/baselines/reference/* | Updates baselines for new error messages and behavior |
jakebailey
reviewed
Jan 28, 2026
Member
jakebailey
left a comment
jakebailey
left a comment
There was a problem hiding this comment.
Seems like some tests are still left.
I do also wonder if we need to be forcing this on for inferred projects in tsserver...
...rence/tsbuild/moduleResolution/impliedNodeFormat-differs-between-projects-for-shared-file.js
Show resolved
Hide resolved
...rence-resolution-uses-correct-options-for-different-resolution-options-referenced-project.js
Outdated
Show resolved
Hide resolved
.../incremental/react-jsx-emit-mode-with-no-backing-types-found-doesn't-crash-under---strict.js
Outdated
Show resolved
Hide resolved
...-something-in-node_modules-or-@types-when-there-is-no-notification-from-fs-for-index-file.js
Outdated
Show resolved
Hide resolved
...reference/tscWatch/resolutionCache/works-when-module-resolution-changes-to-ambient-module.js
Outdated
Show resolved
Hide resolved
...ce/tsserver/autoImportProvider/Auto-importable-file-is-in-inferred-project-until-imported.js
Show resolved
Hide resolved
Member
|
@typescript-bot test top800 |
Collaborator
Collaborator
|
@jakebailey Here are the results of running the top 800 repos with tsc comparing Something interesting changed - please have a look. Details
|
Collaborator
|
@jakebailey Here are some more interesting changes from running the top 800 repos suite Details
|
Collaborator
Collaborator
Collaborator
|
@jakebailey Here are some more interesting changes from running the top 800 repos suite Details
|
Collaborator
|
@jakebailey Here are some more interesting changes from running the top 800 repos suite Details
|
Collaborator
Collaborator
Collaborator
|
@jakebailey Here are some more interesting changes from running the top 800 repos suite Details
|
sheetalkamat
requested changes
Jan 28, 2026
...rence-resolution-uses-correct-options-for-different-resolution-options-referenced-project.js
Show resolved
Hide resolved
github-project-automation
bot
moved this from Not started
to Waiting on author
in PR Backlog
jakebailey
reviewed
Jan 29, 2026
...s/reference/tscWatch/resolutionCache/works-when-included-file-with-ambient-module-changes.js
Show resolved
Hide resolved
...-something-in-node_modules-or-@types-when-there-is-no-notification-from-fs-for-index-file.js
Show resolved
Hide resolved
...rence/tsbuild/moduleResolution/impliedNodeFormat-differs-between-projects-for-shared-file.js
Outdated
Show resolved
Hide resolved
...reference/tscWatch/resolutionCache/works-when-module-resolution-changes-to-ambient-module.js
Show resolved
Hide resolved
...solutionCache/works-when-renaming-node_modules-folder-that-already-contains-@types-folder.js
Show resolved
Hide resolved
Co-authored-by: Sheetal Nandi <shkamat@microsoft.com>
| { | ||
| "compilerOptions": { | ||
| "strict": true, | ||
| "module": "node16" |
| errorCodeCannotFindModule, | ||
| Diagnostics.Could_not_find_a_declaration_file_for_module_0_1_implicitly_has_an_any_type.code, | ||
| Diagnostics.Cannot_find_name_0_Do_you_need_to_install_type_definitions_for_node_Try_npm_i_save_dev_types_Slashnode.code, | ||
| Diagnostics.Cannot_find_name_0_Do_you_need_to_install_type_definitions_for_node_Try_npm_i_save_dev_types_Slashnode_and_then_add_node_to_the_types_field_in_your_tsconfig.code, |
Member
There was a problem hiding this comment.
Doesnt this need to fix tsconfig to add type to types array ?
| /home/src/tslibs/TS/Lib/lib.d.ts Text-1 "interface Boolean {}\ninterface Function {}\ninterface CallableFunction {}\ninterface NewableFunction {}\ninterface IArguments {}\ninterface Number { toExponential: any; }\ninterface Object {}\ninterface RegExp {}\ninterface String { charAt: any; }\ninterface Array<T> { length: number; [n: number]: T; }\ninterface ReadonlyArray<T> {}\ndeclare const console: { log(msg: any): void; };" | ||
| /users/username/projects/project/app.ts SVC-1-0 "let x = 1;" | ||
| /users/username/projects/project/node_modules/@types/lib1/index.d.ts Text-1 "export let a: number" |
Member
There was a problem hiding this comment.
need to add wildcard types to this test ?
| @@ -98,55 +98,18 @@ Info seq [hh:mm:ss:mss] event: | |||
| Info seq [hh:mm:ss:mss] DirectoryWatcher:: Added:: WatchInfo: /user/username/projects/myproject/test 1 undefined Config: /user/username/projects/myproject/test/tsconfig.json WatchType: Wild card directory | |||
| Info seq [hh:mm:ss:mss] Elapsed:: *ms DirectoryWatcher:: Added:: WatchInfo: /user/username/projects/myproject/test 1 undefined Config: /user/username/projects/myproject/test/tsconfig.json WatchType: Wild card directory | |||
| Info seq [hh:mm:ss:mss] Starting updateGraphWorker: Project: /user/username/projects/myproject/test/tsconfig.json | |||
| Info seq [hh:mm:ss:mss] ======== Resolving type reference directive 'UpperCasePackage', containing file '/user/username/projects/myproject/test/__inferred type names__.ts', root directory '/user/username/projects/myproject/lib/@types,/user/username/projects/myproject/lib/@app'. ======== | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Redo of #63031 with slightly more manageable commit history