feat: make the Ivy compiler the default for ngc (angular#32219)

This commit switches the default value of the enableIvy flag to true. Applications that run ngc will now by default receive an Ivy build! This does not affect the way Bazel builds in the Angular repo work, since those are still switched based on the value of the --define=compile flag. Additionally, projects using @angular/bazel still use View Engine builds by default. Since most of the Angular repo tests are still written against View Engine (particularly because we still publish VE packages to NPM), this switch also requires lots of `enableIvy: false` flags in tsconfigs throughout the repo. Congrats to the team for reaching this milestone! PR Close angular#32219
devversion · Aug 20, 2019 · ec4381d · ec4381d
1 parent 2b64031
commit ec4381d
Show file tree

Hide file tree

Showing 43 changed files with 146 additions and 48 deletions.
diff --git a/aio/content/examples/upgrade-phonecat-1-typescript/tsconfig.json b/aio/content/examples/upgrade-phonecat-1-typescript/tsconfig.json
@@ -6,7 +6,10 @@
     "sourceMap": true,
     "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
-    "lib": [ "es2015", "dom" ],
+    "lib": [
+      "es2015",
+      "dom"
+    ],
     "noImplicitAny": true,
     "skipLibCheck": true,
     "suppressImplicitAnyIndexErrors": true
@@ -16,5 +19,8 @@
     "node_modules/*",
     "**/*-aot.ts",
     "aot/**/*"
-  ]
-}
+  ],
+  "angularCompilerOptions": {
+    "enableIvy": false
+  }
+}
diff --git a/aio/content/examples/upgrade-phonecat-2-hybrid/tsconfig-aot.json b/aio/content/examples/upgrade-phonecat-2-hybrid/tsconfig-aot.json
@@ -19,6 +19,7 @@
   ],
 
   "angularCompilerOptions": {
-    "skipMetadataEmit" : true
+    "skipMetadataEmit" : true,
+    "enableIvy": false,
  }
 }
diff --git a/aio/content/examples/upgrade-phonecat-2-hybrid/tsconfig.json b/aio/content/examples/upgrade-phonecat-2-hybrid/tsconfig.json
@@ -6,7 +6,10 @@
     "sourceMap": true,
     "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
-    "lib": [ "es2015", "dom" ],
+    "lib": [
+      "es2015",
+      "dom"
+    ],
     "noImplicitAny": true,
     "skipLibCheck": true,
     "suppressImplicitAnyIndexErrors": true
@@ -15,5 +18,8 @@
   "exclude": [
     "node_modules/*",
     "**/*-aot.ts"
-  ]
-}
+  ],
+  "angularCompilerOptions": {
+    "enableIvy": false
+  }
+}
diff --git a/aio/content/examples/upgrade-phonecat-3-final/tsconfig.json b/aio/content/examples/upgrade-phonecat-3-final/tsconfig.json
@@ -6,7 +6,10 @@
     "sourceMap": true,
     "emitDecoratorMetadata": true,
     "experimentalDecorators": true,
-    "lib": [ "es2015", "dom" ],
+    "lib": [
+      "es2015",
+      "dom"
+    ],
     "noImplicitAny": true,
     "skipLibCheck": true,
     "suppressImplicitAnyIndexErrors": true
@@ -15,5 +18,8 @@
   "exclude": [
     "node_modules/*",
     "**/*-aot.ts"
-  ]
-}
+  ],
+  "angularCompilerOptions": {
+    "enableIvy": false
+  }
+}
diff --git a/aio/content/examples/visual-studio-2015/tsconfig.json b/aio/content/examples/visual-studio-2015/tsconfig.json
@@ -13,5 +13,8 @@
       "node_modules/@types"
     ]
   },
-  "compileOnSave": true
-}
+  "compileOnSave": true,
+  "angularCompilerOptions": {
+    "enableIvy": false
+  }
+}
diff --git a/aio/tools/examples/shared/boilerplate/cli/tsconfig.json b/aio/tools/examples/shared/boilerplate/cli/tsconfig.json
@@ -18,5 +18,8 @@
       "es2018",
       "dom"
     ]
-  }
+  },
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
 }
diff --git a/aio/tsconfig.json b/aio/tsconfig.json
@@ -32,5 +32,8 @@
     "out-tsc",
     "scripts",
     "tools"
-  ]
+  ],
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
 }
diff --git a/integration/bazel-schematics/disable-ivy.js b/integration/bazel-schematics/disable-ivy.js
@@ -0,0 +1,17 @@
+/**
+ * @license
+ * Copyright Google Inc. All Rights Reserved.
+ *
+ * Use of this source code is governed by an MIT-style license that can be
+ * found in the LICENSE file at https://angular.io/license
+ */
+
+const fs = require('fs');
+const configPath = 'demo/tsconfig.json';
+const config = {
+  ...JSON.parse(fs.readFileSync(configPath, 'utf8')),
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
+};
+fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
diff --git a/integration/bazel-schematics/test.sh b/integration/bazel-schematics/test.sh
@@ -18,13 +18,15 @@ function installLocalPackages() {
   yarn add "${local_packages[@]}"
 }
 
+
 function testBazel() {
   # Set up
   bazel version
   ng version
   rm -rf demo
   # Create project
   ng new demo --collection=@angular/bazel --routing --skip-git --skip-install --style=scss
+  node ./disable-ivy.js
   cd demo
   installLocalPackages
   ng generate component widget --style=css

diff --git a/integration/bazel/angular-metadata.tsconfig.json b/integration/bazel/angular-metadata.tsconfig.json
@@ -13,7 +13,8 @@
     "moduleResolution": "node"
   },
   "angularCompilerOptions": {
-    "enableSummariesForJit": true
+    "enableSummariesForJit": true,
+    "enableIvy": false
   },
   "include": [
     "node_modules/@angular/**/*"

diff --git a/integration/cli-hello-world/tsconfig.json b/integration/cli-hello-world/tsconfig.json
@@ -16,5 +16,8 @@
       "es2017",
       "dom"
     ]
-  }
+  },
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
 }
diff --git a/integration/dynamic-compiler/tsconfig.json b/integration/dynamic-compiler/tsconfig.json
@@ -21,6 +21,9 @@
     "types": [
     ]
   },
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
   "files": [
     "src/main.ts",
     "src/lazy.module.ts"

diff --git a/integration/hello_world__closure/tsconfig.json b/integration/hello_world__closure/tsconfig.json
@@ -2,7 +2,8 @@
   "angularCompilerOptions": {
     "annotationsAs": "static fields",
     "annotateForClosureCompiler": true,
-    "alwaysCompileGeneratedCode": true
+    "alwaysCompileGeneratedCode": true,
+    "enableIvy": false,
   },
 
   "compilerOptions": {

diff --git a/integration/hello_world__systemjs_umd/src/tsconfig.json b/integration/hello_world__systemjs_umd/src/tsconfig.json
@@ -9,5 +9,8 @@
     "lib": [ "es2015", "dom" ],
     "noImplicitAny": true,
     "suppressImplicitAnyIndexErrors": true
-  }
+  },
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
 }
diff --git a/integration/i18n/tsconfig.json b/integration/i18n/tsconfig.json
@@ -2,7 +2,8 @@
   "angularCompilerOptions": {
     "annotationsAs": "static fields",
     "annotateForClosureCompiler": true,
-    "alwaysCompileGeneratedCode": true
+    "alwaysCompileGeneratedCode": true,
+    "enableIvy": false,
   },
 
   "compilerOptions": {

diff --git a/integration/injectable-def/tsconfig-app.json b/integration/injectable-def/tsconfig-app.json
@@ -10,6 +10,9 @@
     "types": ["node"],
     "rootDir": "."
   },
+  "angularCompilerOptions": {
+    "enableIvy": false,
+  },
   "files": [
     "src/app.ts",
     "src/main.ts"

diff --git a/integration/injectable-def/tsconfig-lib1.json b/integration/injectable-def/tsconfig-lib1.json
@@ -17,6 +17,7 @@
   "angularCompilerOptions": {
     "skipTemplateCodegen": true,
     "flatModuleOutFile": "index.js",
-    "flatModuleId": "lib1_built"
+    "flatModuleId": "lib1_built",
+    "enableIvy": false,
   }
 }
diff --git a/integration/injectable-def/tsconfig-lib2.json b/integration/injectable-def/tsconfig-lib2.json
@@ -17,6 +17,7 @@
   "angularCompilerOptions": {
     "skipTemplateCodegen": true,
     "flatModuleId": "lib2_built",
-    "flatModuleOutFile": "index.js"
+    "flatModuleOutFile": "index.js",
+    "enableIvy": false,
   }
 }
diff --git a/integration/ng_elements/tsconfig.json b/integration/ng_elements/tsconfig.json
@@ -2,7 +2,8 @@
   "angularCompilerOptions": {
     "annotationsAs": "static fields",
     "annotateForClosureCompiler": true,
-    "alwaysCompileGeneratedCode": true
+    "alwaysCompileGeneratedCode": true,
+    "enableIvy": false,
   },
 
   "compilerOptions": {

diff --git a/integration/platform-server/tsconfig.json b/integration/platform-server/tsconfig.json
@@ -1,7 +1,8 @@
 {
   "angularCompilerOptions": {
     "annotationsAs": "static fields",
-    "annotateForClosureCompiler": true
+    "annotateForClosureCompiler": true,
+    "enableIvy": false,
   },
 
   "compilerOptions": {

diff --git a/packages/bazel/src/schematics/ng-add/files/angular-metadata.tsconfig.json.template b/packages/bazel/src/schematics/ng-add/files/angular-metadata.tsconfig.json.template
@@ -12,6 +12,9 @@
     "module": "amd",
     "moduleResolution": "node"
   },
+  "angularCompilerOptions": {
+    "enableIvy": false
+  },
   "include": [
     "node_modules/@angular/**/*"
   ],

diff --git a/packages/compiler-cli/integrationtest/flat_module/tsconfig-build.json b/packages/compiler-cli/integrationtest/flat_module/tsconfig-build.json
@@ -2,7 +2,8 @@
   "angularCompilerOptions": {
     "flatModuleId": "flat_module",
     "flatModuleOutFile": "index.js",
-    "skipTemplateCodegen": true
+    "skipTemplateCodegen": true,
+    "enableIvy": false
   },
 
   "compilerOptions": {

diff --git a/packages/compiler-cli/integrationtest/ngtools_src/tsconfig-build.json b/packages/compiler-cli/integrationtest/ngtools_src/tsconfig-build.json
@@ -1,6 +1,7 @@
 {
   "angularCompilerOptions": {
-    "debug": true
+    "debug": true,
+    "enableIvy": false
   },
 
   "compilerOptions": {

diff --git a/packages/compiler-cli/integrationtest/third_party_src/tsconfig-build.json b/packages/compiler-cli/integrationtest/third_party_src/tsconfig-build.json
@@ -1,6 +1,7 @@
 {
   "angularCompilerOptions": {
-    "skipTemplateCodegen": true
+    "skipTemplateCodegen": true,
+    "enableIvy": false
   },
 
   "compilerOptions": {

diff --git a/packages/compiler-cli/integrationtest/tsconfig-build.json b/packages/compiler-cli/integrationtest/tsconfig-build.json
@@ -4,7 +4,8 @@
     "annotationsAs": "static fields",
     "debug": true,
     "enableSummariesForJit": true,
-    "i18nFormat": "xlf"
+    "i18nFormat": "xlf",
+    "enableIvy": false
   },
 
   "compilerOptions": {

diff --git a/packages/compiler-cli/integrationtest/tsconfig-xi18n.json b/packages/compiler-cli/integrationtest/tsconfig-xi18n.json
@@ -1,7 +1,8 @@
 {
   "angularCompilerOptions": {
     "debug": true,
-    "enableSummariesForJit": true
+    "enableSummariesForJit": true,
+    "enableIvy": false
   },
 
   "compilerOptions": {

diff --git a/packages/compiler-cli/src/main.ts b/packages/compiler-cli/src/main.ts
@@ -68,7 +68,8 @@ export function mainDiagnosticsForTest(
 }
 
 function createEmitCallback(options: api.CompilerOptions): api.TsEmitCallback|undefined {
-  const transformDecorators = !options.enableIvy && options.annotationsAs !== 'decorators';
+  const transformDecorators =
+      (options.enableIvy === false && options.annotationsAs !== 'decorators');
   const transformTypesToClosure = options.annotateForClosureCompiler;
   if (!transformDecorators && !transformTypesToClosure) {
     return undefined;
@@ -205,7 +206,7 @@ function reportErrorsAndExit(
   const errorsAndWarnings = filterErrorsAndWarnings(allDiagnostics);
   if (errorsAndWarnings.length) {
     const formatHost = getFormatDiagnosticsHost(options);
-    if (options && options.enableIvy === true) {
+    if (options && options.enableIvy !== false) {
       const ngDiagnostics = errorsAndWarnings.filter(api.isNgDiagnostic);
       const tsDiagnostics = errorsAndWarnings.filter(api.isTsDiagnostic);
       consoleError(replaceTsWithNgInErrors(

diff --git a/packages/compiler-cli/src/transformers/api.ts b/packages/compiler-cli/src/transformers/api.ts
@@ -186,10 +186,7 @@ export interface CompilerOptions extends ts.CompilerOptions {
 
   /**
    * Tells the compiler to generate definitions using the Render3 style code generation.
-   * This option defaults to `false`.
-   *
-   * Not all features are supported with this option enabled. It is only supported
-   * for experimentation and testing of Render3 style code generation.
+   * This option defaults to `true`.
    *
    * Acceptable values are as follows:
    *

diff --git a/packages/compiler-cli/src/transformers/program.ts b/packages/compiler-cli/src/transformers/program.ts
@@ -142,7 +142,7 @@ class AngularCompilerProgram implements Program {
     }
 
     this.loweringMetadataTransform =
-        new LowerMetadataTransform(options.enableIvy ? R3_LOWER_FIELDS : LOWER_FIELDS);
+        new LowerMetadataTransform(options.enableIvy !== false ? R3_LOWER_FIELDS : LOWER_FIELDS);
     this.metadataCache = this.createMetadataCache([this.loweringMetadataTransform]);
   }
 
@@ -262,7 +262,7 @@ class AngularCompilerProgram implements Program {
     emitCallback?: TsEmitCallback,
     mergeEmitResultsCallback?: TsMergeEmitResultsCallback,
   } = {}): ts.EmitResult {
-    if (this.options.enableIvy) {
+    if (this.options.enableIvy !== false) {
       throw new Error('Cannot run legacy compiler in ngtsc mode');
     }
     return this._emitRender2(parameters);
@@ -897,7 +897,7 @@ export function createProgram({rootNames, options, host, oldProgram}: {
   options: CompilerOptions,
   host: CompilerHost, oldProgram?: Program
 }): Program {
-  if (options.enableIvy === true) {
+  if (options.enableIvy !== false) {
     return new NgtscProgram(rootNames, options, host, oldProgram as NgtscProgram);
   } else {
     return new AngularCompilerProgram(rootNames, options, host, oldProgram);

diff --git a/packages/compiler-cli/test/extract_i18n_spec.ts b/packages/compiler-cli/test/extract_i18n_spec.ts
@@ -225,6 +225,9 @@ describe('extract_i18n command line', () => {
         "moduleResolution": "node",
         "lib": ["es6", "dom"],
         "typeRoots": ["node_modules/@types"]
+      },
+      "angularCompilerOptions": {
+        "enableIvy": false
       }
     }`);
   });

diff --git a/packages/compiler-cli/test/ngc_spec.ts b/packages/compiler-cli/test/ngc_spec.ts
@@ -59,6 +59,9 @@ describe('ngc transformer command-line', () => {
         "moduleResolution": "node",
         "lib": ["es6", "dom"],
         "typeRoots": ["node_modules/@types"]
+      },
+      "angularCompilerOptions": {
+        "enableIvy": false
       }
     }`);
   });