Add new option resolveBareImports

Author: jakebailey

src/compiler/checker.ts

@@ -1502,6 +1502,7 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
     var noImplicitThis = getStrictOptionValue(compilerOptions, "noImplicitThis");
     var useUnknownInCatchVariables = getStrictOptionValue(compilerOptions, "useUnknownInCatchVariables");
     var exactOptionalPropertyTypes = compilerOptions.exactOptionalPropertyTypes;
+    var resolveBareImports = compilerOptions.resolveBareImports;
 
     var checkBinaryExpression = createCheckBinaryExpression();
     var emitResolver = createResolver();
@@ -47126,6 +47127,9 @@ export function createTypeChecker(host: TypeCheckerHost): TypeChecker {
                     }
                 }
             }
+            else if (resolveBareImports && !importClause) {
+                void resolveExternalModuleName(node, node.moduleSpecifier);
+            }
         }
         checkImportAttributes(node);
     }

src/compiler/commandLineParser.ts

@@ -1192,6 +1192,14 @@ const commandOptionsWithoutBuild: CommandLineOption[] = [
         category: Diagnostics.Modules,
         description: Diagnostics.Conditions_to_set_in_addition_to_the_resolver_specific_defaults_when_resolving_imports,
     },
+    {
+        name: "resolveBareImports",
+        type: "boolean",
+        affectsModuleResolution: true,
+        category: Diagnostics.Modules,
+        description: Diagnostics.Resolve_bare_imports,
+        defaultValueDescription: false,
+    },
 
     // Source Maps
     {

src/compiler/diagnosticMessages.json

@@ -6384,6 +6384,10 @@
         "category": "Message",
         "code": 6805
     },
+    "Resolve bare imports.": {
+        "category": "Message",
+        "code": 6806
+    },
 
     "one of:": {
         "category": "Message",

src/compiler/types.ts

@@ -7349,6 +7349,7 @@ export interface CompilerOptions {
     target?: ScriptTarget;
     traceResolution?: boolean;
     useUnknownInCatchVariables?: boolean;
+    resolveBareImports?: boolean;
     resolveJsonModule?: boolean;
     types?: string[];
     /** Paths used to compute primary types search locations */

src/testRunner/compilerRunner.ts

@@ -150,6 +150,7 @@ class CompilerTest {
         "noUncheckedIndexedAccess",
         "preserveConstEnums",
         "removeComments",
+        "resolveBareImports",
         "resolveJsonModule",
         "resolvePackageJsonExports",
         "resolvePackageJsonImports",

tests/baselines/reference/api/typescript.d.ts

@@ -6953,6 +6953,7 @@ declare namespace ts {
         target?: ScriptTarget;
         traceResolution?: boolean;
         useUnknownInCatchVariables?: boolean;
+        resolveBareImports?: boolean;
         resolveJsonModule?: boolean;
         types?: string[];
         /** Paths used to compute primary types search locations */

tests/baselines/reference/config/initTSConfig/Default initialized TSConfig/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with --help/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with --watch/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with advanced options/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with boolean value compiler options/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with enum value compiler options/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with files options/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with incorrect compiler option value/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with incorrect compiler option/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with list compiler options with enum value/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/initTSConfig/Initialized TSConfig with list compiler options/tsconfig.json

@@ -39,6 +39,7 @@
     // "resolvePackageJsonExports": true,                /* Use the package.json 'exports' field when resolving package imports. */
     // "resolvePackageJsonImports": true,                /* Use the package.json 'imports' field when resolving imports. */
     // "customConditions": [],                           /* Conditions to set in addition to the resolver-specific defaults when resolving imports. */
+    // "resolveBareImports": true,                       /* Resolve bare imports. */
     // "resolveJsonModule": true,                        /* Enable importing .json files. */
     // "allowArbitraryExtensions": true,                 /* Enable importing files with any extension, provided a declaration file is present. */
     // "noResolve": true,                                /* Disallow 'import's, 'require's or '<reference>'s from expanding the number of files TypeScript should add to a project. */

tests/baselines/reference/config/showConfig/Shows tsconfig for single option/resolveBareImports/tsconfig.json

@@ -0,0 +1,5 @@
+{
+    "compilerOptions": {
+        "resolveBareImports": true
+    }
+}

tests/baselines/reference/resolveBareImports(module=commonjs).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=commonjs).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=commonjs).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=false).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=false).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=false).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=true).errors.txt

@@ -0,0 +1,16 @@
+resolveBareImports.ts(1,8): error TS2307: Cannot find module 'does-not-exist' or its corresponding type declarations.
+resolveBareImports.ts(2,8): error TS2307: Cannot find module './does-not-exist-either' or its corresponding type declarations.
+resolveBareImports.ts(3,8): error TS2307: Cannot find module './does-not-exist-either.js' or its corresponding type declarations.
+
+
+==== resolveBareImports.ts (3 errors) ====
+    import "does-not-exist";
+           ~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module 'does-not-exist' or its corresponding type declarations.
+    import "./does-not-exist-either";
+           ~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module './does-not-exist-either' or its corresponding type declarations.
+    import "./does-not-exist-either.js";
+           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module './does-not-exist-either.js' or its corresponding type declarations.
+    

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=true).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=true).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=commonjs,resolvebareimports=true).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=nodenext).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=false).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=false).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=false).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=true).errors.txt

@@ -0,0 +1,16 @@
+resolveBareImports.ts(1,8): error TS2307: Cannot find module 'does-not-exist' or its corresponding type declarations.
+resolveBareImports.ts(2,8): error TS2307: Cannot find module './does-not-exist-either' or its corresponding type declarations.
+resolveBareImports.ts(3,8): error TS2307: Cannot find module './does-not-exist-either.js' or its corresponding type declarations.
+
+
+==== resolveBareImports.ts (3 errors) ====
+    import "does-not-exist";
+           ~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module 'does-not-exist' or its corresponding type declarations.
+    import "./does-not-exist-either";
+           ~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module './does-not-exist-either' or its corresponding type declarations.
+    import "./does-not-exist-either.js";
+           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+!!! error TS2307: Cannot find module './does-not-exist-either.js' or its corresponding type declarations.
+    

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=true).js

@@ -0,0 +1,14 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+//// [resolveBareImports.ts]
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+
+
+//// [resolveBareImports.js]
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+require("does-not-exist");
+require("./does-not-exist-either");
+require("./does-not-exist-either.js");

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=true).symbols

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+

tests/baselines/reference/resolveBareImports(module=nodenext,resolvebareimports=true).types

@@ -0,0 +1,8 @@
+//// [tests/cases/compiler/resolveBareImports.ts] ////
+
+=== resolveBareImports.ts ===
+
+import "does-not-exist";
+import "./does-not-exist-either";
+import "./does-not-exist-either.js";
+