Allow 'verbatimModuleSyntax' with transpileModule (#53240)

Co-authored-by: Daniel Rosenwasser <DanielRosenwasser@users.noreply.github.com>
microsoft · Mar 14, 2023 · ae1b3db · ae1b3db
1 parent 2378c0b
commit ae1b3db
Show file tree

Hide file tree

Showing 280 changed files with 1,437 additions and 73 deletions.
diff --git a/src/services/transpile.ts b/src/services/transpile.ts
@@ -43,6 +43,12 @@ export interface TranspileOutput {
     sourceMapText?: string;
 }
 
+const optionsRedundantWithVerbatimModuleSyntax = new Set([
+    "isolatedModules",
+    "preserveValueImports",
+    "importsNotUsedAsValues"
+]);
+
 /*
  * This function will compile source text from 'input' argument using specified compiler options.
  * If not options are provided - it will use a set of default compiler options.
@@ -66,6 +72,11 @@ export function transpileModule(input: string, transpileOptions: TranspileOption
     }
 
     for (const option of transpileOptionValueCompilerOptions) {
+        // Do not set redundant config options if `verbatimModuleSyntax` was supplied.
+        if (options.verbatimModuleSyntax && optionsRedundantWithVerbatimModuleSyntax.has(option.name)) {
+            continue;
+        }
+
         options[option.name] = option.transpileOptionValue;
     }
 

diff --git a/src/testRunner/unittests/services/transpile.ts b/src/testRunner/unittests/services/transpile.ts
diff --git a/...tly serialize metadata when transpile with CommonJS option (verbatimModuleSyntax=true).js b/...tly serialize metadata when transpile with CommonJS option (verbatimModuleSyntax=true).js
diff --git a/... metadata when transpile with CommonJS option (verbatimModuleSyntax=true).oldTranspile.js b/... metadata when transpile with CommonJS option (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...elide import equals referenced only by export type (verbatimModuleSyntax=true).errors.txt b/...elide import equals referenced only by export type (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import IFoo = Namespace.IFoo;export type { IFoo };
diff --git a/...oes not elide import equals referenced only by export type (verbatimModuleSyntax=true).js b/...oes not elide import equals referenced only by export type (verbatimModuleSyntax=true).js
diff --git a/...equals referenced only by export type (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...equals referenced only by export type (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import IFoo = Namespace.IFoo;export type { IFoo };
diff --git a/... import equals referenced only by export type (verbatimModuleSyntax=true).oldTranspile.js b/... import equals referenced only by export type (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...uals referenced only by type only export specifier (verbatimModuleSyntax=true).errors.txt b/...uals referenced only by type only export specifier (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import IFoo = Namespace.IFoo;export { type IFoo };
diff --git a/...mport equals referenced only by type only export specifier (verbatimModuleSyntax=true).js b/...mport equals referenced only by type only export specifier (verbatimModuleSyntax=true).js
diff --git a/...ed only by type only export specifier (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...ed only by type only export specifier (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import IFoo = Namespace.IFoo;export { type IFoo };
diff --git a/...referenced only by type only export specifier (verbatimModuleSyntax=true).oldTranspile.js b/...referenced only by type only export specifier (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...e/transpile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).errors.txt b/...e/transpile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    var x: string = 0;
diff --git a/...reference/transpile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).js b/...reference/transpile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).js
diff --git a/...oes not generate semantic diagnostics (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...oes not generate semantic diagnostics (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    var x: string = 0;
diff --git a/...nspile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).oldTranspile.js b/...nspile/Does not generate semantic diagnostics (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...transpile/Does not support setting isolatedModules (verbatimModuleSyntax=true).errors.txt b/...transpile/Does not support setting isolatedModules (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,8 @@
+error TS5104: Option 'isolatedModules' is redundant and cannot be specified with option 'verbatimModuleSyntax'.
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5104: Option 'isolatedModules' is redundant and cannot be specified with option 'verbatimModuleSyntax'.
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== input.js (0 errors) ====
+    x;
diff --git a/...ference/transpile/Does not support setting isolatedModules (verbatimModuleSyntax=true).js b/...ference/transpile/Does not support setting isolatedModules (verbatimModuleSyntax=true).js
diff --git a/...s not support setting isolatedModules (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...s not support setting isolatedModules (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,8 @@
+error TS5104: Option 'isolatedModules' is redundant and cannot be specified with option 'verbatimModuleSyntax'.
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5104: Option 'isolatedModules' is redundant and cannot be specified with option 'verbatimModuleSyntax'.
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== input.js (0 errors) ====
+    x;
diff --git a/...pile/Does not support setting isolatedModules (verbatimModuleSyntax=true).oldTranspile.js b/...pile/Does not support setting isolatedModules (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...ranspile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).errors.txt b/...ranspile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,9 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+
+    var a;
+    export { a as alias };
+    export * as alias from './file';
diff --git a/...erence/transpile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).js b/...erence/transpile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).js
diff --git a/...rt star as ns conflict does not crash (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...rt star as ns conflict does not crash (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,9 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+
+    var a;
+    export { a as alias };
+    export * as alias from './file';
diff --git a/...ile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).oldTranspile.js b/...ile/Export star as ns conflict does not crash (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...transpile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).errors.txt b/...transpile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,9 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+file.ts(1,1): error TS1434: Unexpected keyword or identifier.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (1 errors) ====
+    a b
+    ~
+!!! error TS1434: Unexpected keyword or identifier.
diff --git a/...ference/transpile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).js b/...ference/transpile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).js
diff --git a/...erates expected syntactic diagnostics (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...erates expected syntactic diagnostics (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,9 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+file.ts(1,1): error TS1434: Unexpected keyword or identifier.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (1 errors) ====
+    a b
+    ~
+!!! error TS1434: Unexpected keyword or identifier.
diff --git a/...pile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).oldTranspile.js b/...pile/Generates expected syntactic diagnostics (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...nerates no diagnostics for missing file references (verbatimModuleSyntax=true).errors.txt b/...nerates no diagnostics for missing file references (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,7 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    /// <reference path="file2.ts" />
+    var x = 0;
diff --git a/...spile/Generates no diagnostics for missing file references (verbatimModuleSyntax=true).js b/...spile/Generates no diagnostics for missing file references (verbatimModuleSyntax=true).js
diff --git a/...agnostics for missing file references (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...agnostics for missing file references (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,7 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    /// <reference path="file2.ts" />
+    var x = 0;
diff --git a/...es no diagnostics for missing file references (verbatimModuleSyntax=true).oldTranspile.js b/...es no diagnostics for missing file references (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...enerates no diagnostics for missing module imports (verbatimModuleSyntax=true).errors.txt b/...enerates no diagnostics for missing module imports (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import {a} from "module2";
diff --git a/...nspile/Generates no diagnostics for missing module imports (verbatimModuleSyntax=true).js b/...nspile/Generates no diagnostics for missing module imports (verbatimModuleSyntax=true).js
diff --git a/...iagnostics for missing module imports (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...iagnostics for missing module imports (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    import {a} from "module2";
diff --git a/...tes no diagnostics for missing module imports (verbatimModuleSyntax=true).oldTranspile.js b/...tes no diagnostics for missing module imports (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...anspile/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).errors.txt b/...anspile/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    var x = 0;
diff --git a/...rence/transpile/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).js b/...rence/transpile/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).js
diff --git a/...ates no diagnostics with valid inputs (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...ates no diagnostics with valid inputs (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    var x = 0;
diff --git a/...le/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).oldTranspile.js b/...le/Generates no diagnostics with valid inputs (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...s/reference/transpile/Infer correct file extension (verbatimModuleSyntax=true).errors.txt b/...s/reference/transpile/Infer correct file extension (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    const fn = <T>(a: T) => a
diff --git a/...baselines/reference/transpile/Infer correct file extension (verbatimModuleSyntax=true).js b/...baselines/reference/transpile/Infer correct file extension (verbatimModuleSyntax=true).js
diff --git a/...ranspile/Infer correct file extension (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...ranspile/Infer correct file extension (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file.ts (0 errors) ====
+    const fn = <T>(a: T) => a
diff --git a/...erence/transpile/Infer correct file extension (verbatimModuleSyntax=true).oldTranspile.js b/...erence/transpile/Infer correct file extension (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...anspile/No extra errors for file without extension (verbatimModuleSyntax=true).errors.txt b/...anspile/No extra errors for file without extension (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,7 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file (0 errors) ====
+    "use strict";
+    var x = 0;
diff --git a/...rence/transpile/No extra errors for file without extension (verbatimModuleSyntax=true).js b/...rence/transpile/No extra errors for file without extension (verbatimModuleSyntax=true).js
diff --git a/...tra errors for file without extension (verbatimModuleSyntax=true).oldTranspile.errors.txt b/...tra errors for file without extension (verbatimModuleSyntax=true).oldTranspile.errors.txt
@@ -0,0 +1,7 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== file (0 errors) ====
+    "use strict";
+    var x = 0;
diff --git a/...le/No extra errors for file without extension (verbatimModuleSyntax=true).oldTranspile.js b/...le/No extra errors for file without extension (verbatimModuleSyntax=true).oldTranspile.js
diff --git a/...eference/transpile/Support options with lib values (verbatimModuleSyntax=true).errors.txt b/...eference/transpile/Support options with lib values (verbatimModuleSyntax=true).errors.txt
@@ -0,0 +1,6 @@
+error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+
+
+!!! error TS5107: Option 'target=ES3' is deprecated and will stop functioning in TypeScript 5.5. Specify compilerOption '"ignoreDeprecations": "5.0"' to silence this error.
+==== input.js (0 errors) ====
+    const a = 10;