fix(migrations): Fix typo for strict-template migration

Fix typo for strict-template migration

Author: aparzi

packages/core/schematics/BUILD.bazel

@@ -126,8 +126,8 @@ bundle_entrypoints = [
         "packages/core/schematics/migrations/http-xhr-backend/index.js",
     ],
     [
-        "strict-templates",
-        "packages/core/schematics/migrations/strict-template/index.js",
+        "strict-templates-default",
+        "packages/core/schematics/migrations/strict-templates-default/index.js",
     ],
     [
         "can-match-snapshot-required",
@@ -152,7 +152,7 @@ rollup.rollup(
         "//packages/core/schematics/migrations/change-detection-eager",
         "//packages/core/schematics/migrations/http-xhr-backend",
         "//packages/core/schematics/migrations/incremental-hydration",
-        "//packages/core/schematics/migrations/strict-template",
+        "//packages/core/schematics/migrations/strict-templates-default",
         "//packages/core/schematics/ng-generate/cleanup-unused-imports",
         "//packages/core/schematics/ng-generate/common-to-standalone-migration",
         "//packages/core/schematics/ng-generate/control-flow-migration",

packages/core/schematics/migrations.json

@@ -10,10 +10,10 @@
       "description": "Adds 'withXhr' to 'provideHttpClient' function calls when the 'HttpXhrBackend' is used. For more information see: https://angular.dev/api/common/http/withXhr",
       "factory": "./bundles/http-xhr-backend.cjs#migrate"
     },
-    "strict-template": {
+    "strict-templates-default": {
       "version": "22.0.0",
-      "description": "Adds 'strictTemplates: true' in tsconfig.json.",
-      "factory": "./bundles/strict-templates.cjs#migrate"
+      "description": "Adds 'strictTemplates: false' in tsconfig.json when not set.",
+      "factory": "./bundles/strict-templates-default.cjs#migrate"
     },
     "can-match-snapshot-required": {
       "version": "22.0.0",

packages/core/schematics/migrations/strict-template/index.ts

@@ -8,13 +8,16 @@ package(
 )
 
 ts_project(
-    name = "strict-template",
+    name = "strict-templates-default",
     srcs = glob(
         ["**/*.ts"],
         exclude = ["*.spec.ts"],
     ),
     deps = [
+        "//:node_modules/@angular-devkit/core",
         "//:node_modules/@angular-devkit/schematics",
+        "//:node_modules/@schematics/angular",
+        "//:node_modules/typescript",
         "//packages/core/schematics/utils",
     ],
 )
@@ -24,9 +27,11 @@ ts_project(
     testonly = True,
     srcs = glob(["*.spec.ts"]),
     deps = [
-        ":strict-template",
+        ":strict-templates-default",
         "//:node_modules/@angular-devkit/core",
         "//:node_modules/@angular-devkit/schematics",
+        "//:node_modules/@schematics/angular",
+        "//:node_modules/typescript",
         "//packages/core/schematics/utils",
     ],
 )

packages/core/schematics/migrations/strict-template/strict-template.spec.ts

@@ -0,0 +1,73 @@
+/**
+ * @license
+ * Copyright Google LLC 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.dev/license
+ */
+
+import {Rule} from '@angular-devkit/schematics';
+import {getProjectTsConfigPaths} from '../../utils/project_tsconfig_paths';
+import {JSONFile} from '@schematics/angular/utility/json-file';
+import ts from 'typescript';
+import {dirname, join} from 'node:path';
+
+function getResolvedAngularCompilerOptions(tree: any, tsconfigPath: string): Record<string, any> {
+  if (!tree.exists(tsconfigPath)) return {};
+
+  const sourceFile = ts.readJsonConfigFile(tsconfigPath, (path) => tree.readText(path));
+  const config = ts.convertToObject(sourceFile, []);
+
+  let angularOptions = config.angularCompilerOptions || {};
+
+  // Manually resolve inheritance for Angular-specific options.
+  // Since the TypeScript API doesn't perform a deep merge of custom/non-standard keys
+  // during config parsing, we must traverse the inheritance chain manually
+  if (config.extends) {
+    // Management extends property...
+    const parentPath = join(dirname(tsconfigPath), config.extends);
+
+    const parentOptions = getResolvedAngularCompilerOptions(tree, parentPath);
+
+    // Merge: the options of the current file overwrite those of the parent
+    angularOptions = {
+      ...parentOptions,
+      ...angularOptions,
+    };
+  }
+
+  return angularOptions;
+}
+
+/**
+ * Migration that adds `strictTemplates: false` to `tsconfig.json` files.
+ */
+export function migrate(): Rule {
+  return async (tree) => {
+    const {buildPaths, testPaths} = await getProjectTsConfigPaths(tree);
+    const allPaths = [...new Set([...buildPaths, ...testPaths])];
+
+    for (const tsconfigPath of allPaths) {
+      const json = new JSONFile(tree, tsconfigPath);
+      const compilerOptions = json.get(['compilerOptions']);
+
+      if (
+        !compilerOptions ||
+        typeof compilerOptions !== 'object' ||
+        Object.keys(compilerOptions).length === 0
+      ) {
+        continue;
+      }
+
+      const angularOptions = getResolvedAngularCompilerOptions(tree, tsconfigPath);
+
+      if (angularOptions['strictTemplates'] !== undefined) {
+        continue;
+      }
+
+      if (json.get(['angularCompilerOptions', 'strictTemplates']) === undefined) {
+        json.modify(['angularCompilerOptions', 'strictTemplates'], false);
+      }
+    }
+  };
+}