@angular/core 21.0.0-next.1 → 21.0.0-next.10

package/schematics/bundles/signal-input-migration.cjs

@@ -1,29 +1,26 @@
 'use strict';
 /**
- * @license Angular v21.0.0-next.1
+ * @license Angular v21.0.0-next.10
  * (c) 2010-2025 Google LLC. https://angular.io/
  * License: MIT
  */
 'use strict';
 
-var migrate_ts_type_references = require('./migrate_ts_type_references-BVSg43hf.cjs');
+var migrate_ts_type_references = require('./migrate_ts_type_references-UGIUl7En.cjs');
 var ts = require('typescript');
-require('os');
-var project_tsconfig_paths = require('./project_tsconfig_paths-D7xzGqRi.cjs');
-var index$1 = require('./index-jjHOgYYs.cjs');
-require('path');
+require('@angular/compiler-cli');
+var migrations = require('@angular/compiler-cli/private/migrations');
 require('node:path');
-var project_paths = require('./project_paths-T_M15e2g.cjs');
-var index = require('./index-BdH2rlWJ.cjs');
+var project_paths = require('./project_paths-DvD50ouC.cjs');
+var index = require('./index-B7I9sIUx.cjs');
 var assert = require('assert');
-var apply_import_manager = require('./apply_import_manager-yycO3l8f.cjs');
+var apply_import_manager = require('./apply_import_manager-1Zs_gpB6.cjs');
 require('@angular-devkit/core');
 require('node:path/posix');
 require('./leading_space-D9nQ8UQC.cjs');
-require('fs');
-require('module');
-require('url');
 require('@angular-devkit/schematics');
+require('./project_tsconfig_paths-CDVxT6Ov.cjs');
+require('@angular/compiler');
 
 /**
  * Class that holds information about a given directive and its input fields.
@@ -274,9 +271,9 @@ function prepareAnalysisInfo(userProgram, compiler, programAbsoluteRootPaths) {
         state.templateTypeChecker.generateAllTypeCheckBlocks();
     }
     const typeChecker = userProgram.getTypeChecker();
-    const reflector = new project_tsconfig_paths.TypeScriptReflectionHost(typeChecker);
-    const evaluator = new index$1.PartialEvaluator(reflector, typeChecker, null);
-    const dtsMetadataReader = new index$1.DtsMetadataReader(typeChecker, reflector);
+    const reflector = new migrations.TypeScriptReflectionHost(typeChecker);
+    const evaluator = new migrations.PartialEvaluator(reflector, typeChecker, null);
+    const dtsMetadataReader = new migrations.DtsMetadataReader(typeChecker, reflector);
     return {
         metaRegistry: metaReader,
         dtsMetadataReader,
@@ -337,7 +334,7 @@ function extractDtsInput(node, metadataReader) {
     // in the `.d.ts` aren't resolvable. This seems to be unexpected and shouldn't
     // result in the entire migration to be failing.
     try {
-        directiveMetadata = metadataReader.getDirectiveMetadata(new project_tsconfig_paths.Reference(node.parent));
+        directiveMetadata = metadataReader.getDirectiveMetadata(new migrations.Reference(node.parent));
     }
     catch (e) {
         console.error('Unexpected error. Gracefully ignoring.');
@@ -373,7 +370,7 @@ function extractSourceCodeInput(node, host, reflector, evaluator) {
     if (decorators === null) {
         return null;
     }
-    const ngDecorators = project_tsconfig_paths.getAngularDecorators(decorators, ['Input'], host.isMigratingCore);
+    const ngDecorators = migrations.getAngularDecorators(decorators, ['Input'], host.isMigratingCore);
     if (ngDecorators.length === 0) {
         return null;
     }
@@ -396,7 +393,15 @@ function extractSourceCodeInput(node, host, reflector, evaluator) {
                 isRequired = !!evaluatedInputOpts.get('required');
             }
             if (evaluatedInputOpts.has('transform') && evaluatedInputOpts.get('transform') != null) {
-                transformResult = parseTransformOfInput(evaluatedInputOpts, node, reflector);
+                const result = parseTransformOfInput(evaluatedInputOpts, node, reflector);
+                if (result === 'parsingError') {
+                    if (!host.config.bestEffortMode) {
+                        return null;
+                    }
+                }
+                else {
+                    transformResult = result;
+                }
             }
         }
     }
@@ -417,33 +422,33 @@ function extractSourceCodeInput(node, host, reflector, evaluator) {
  */
 function parseTransformOfInput(evaluatedInputOpts, node, reflector) {
     const transformValue = evaluatedInputOpts.get('transform');
-    if (!(transformValue instanceof project_tsconfig_paths.DynamicValue) && !(transformValue instanceof project_tsconfig_paths.Reference)) {
+    if (!(transformValue instanceof migrations.DynamicValue) && !(transformValue instanceof migrations.Reference)) {
         return null;
     }
     // For parsing the transform, we don't need a real reference emitter, as
     // the emitter is only used for verifying that the transform type could be
     // copied into e.g. an `ngInputAccept` class member.
-    const noopRefEmitter = new project_tsconfig_paths.ReferenceEmitter([
+    const noopRefEmitter = new migrations.ReferenceEmitter([
         {
             emit: () => ({
-                kind: project_tsconfig_paths.ReferenceEmitKind.Success,
-                expression: project_tsconfig_paths.NULL_EXPR,
+                kind: migrations.ReferenceEmitKind.Success,
+                expression: migrate_ts_type_references.NULL_EXPR,
                 importedFile: null,
             }),
         },
     ]);
     try {
-        return project_tsconfig_paths.parseDecoratorInputTransformFunction(node.parent, node.name.text, transformValue, reflector, noopRefEmitter, project_tsconfig_paths.CompilationMode.FULL, 
+        return migrations.parseDecoratorInputTransformFunction(node.parent, node.name.text, transformValue, reflector, noopRefEmitter, migrations.CompilationMode.FULL, 
         /* emitDeclarationOnly */ false);
     }
     catch (e) {
-        if (!(e instanceof project_tsconfig_paths.FatalDiagnosticError)) {
+        if (!(e instanceof migrations.FatalDiagnosticError)) {
             throw e;
         }
         // TODO: implement error handling.
         // See failing case: e.g. inherit_definition_feature_spec.ts
         console.error(`${e.node.getSourceFile().fileName}: ${e.toString()}`);
-        return null;
+        return 'parsingError';
     }
 }
 
@@ -648,7 +653,7 @@ function pass1__IdentifySourceFileAndDeclarationInputs(sf, host, checker, reflec
 function pass3__checkIncompatiblePatterns(host, inheritanceGraph, checker, groupedTsAstVisitor, knownInputs) {
     migrate_ts_type_references.checkIncompatiblePatterns(inheritanceGraph, checker, groupedTsAstVisitor, knownInputs, () => knownInputs.getAllInputContainingClasses());
     for (const input of knownInputs.knownInputIds.values()) {
-        const hostBindingDecorators = project_tsconfig_paths.getAngularDecorators(input.metadata.fieldDecorators, ['HostBinding'], host.isMigratingCore);
+        const hostBindingDecorators = migrations.getAngularDecorators(input.metadata.fieldDecorators, ['HostBinding'], host.isMigratingCore);
         if (hostBindingDecorators.length > 0) {
             knownInputs.markFieldIncompatible(input.descriptor, {
                 context: hostBindingDecorators[0].node,
@@ -686,7 +691,7 @@ function executeAnalysisPhase(host, knownInputs, result, { sourceFiles, fullProg
     fullProgramSourceFiles.forEach((sf) => 
     // Shim shim files. Those are unnecessary and might cause unexpected slowness.
     // e.g. `ngtypecheck` files.
-    !project_tsconfig_paths.isShim(sf) &&
+    !migrations.isShim(sf) &&
         pass1__IdentifySourceFileAndDeclarationInputs(sf, host, typeChecker, reflector, dtsMetadataReader, evaluator, knownInputs, result));
     const fieldNamesToConsiderForReferenceLookup = new Set();
     for (const input of knownInputs.knownInputIds.values()) {
@@ -961,6 +966,12 @@ function populateKnownInputsFromGlobalData(knownInputs, globalData) {
  * @returns Replacements for converting the input.
  */
 function convertToSignalInput(node, { resolvedMetadata: metadata, resolvedType, preferShorthandIfPossible, originalInputDecorator, initialValue, leadingTodoText, }, info, checker, importManager, result) {
+    // Check for 'this' references in initializer before doing anything else
+    if (node.initializer &&
+        (ts.isArrowFunction(node.initializer) || ts.isFunctionExpression(node.initializer)) &&
+        containsThisReferences(node.initializer)) {
+        return []; // Skip migration for this input by returning empty replacements
+    }
     let optionsLiteral = null;
     // We need an options array for the input because:
     //   - the input is either aliased,
@@ -1030,7 +1041,23 @@ function convertToSignalInput(node, { resolvedMetadata: metadata, resolvedType,
     if (!modifiersWithoutInputDecorator?.some((s) => s.kind === ts.SyntaxKind.ReadonlyKeyword)) {
         modifiersWithoutInputDecorator.push(ts.factory.createModifier(ts.SyntaxKind.ReadonlyKeyword));
     }
-    const newNode = ts.factory.createPropertyDeclaration(modifiersWithoutInputDecorator, node.name, undefined, undefined, inputInitializer);
+    // Skip migration if the input is a function that references class members via 'this'
+    if (inputInitializer &&
+        (ts.isArrowFunction(inputInitializer) || ts.isFunctionExpression(inputInitializer))) {
+        if (containsThisReferences(inputInitializer)) {
+            return []; // Skip migration for this input by returning empty replacements
+        }
+    }
+    let finalInitializer = inputInitializer;
+    if (inputInitializer === undefined) {
+        if (preferShorthandIfPossible === null) {
+            finalInitializer = ts.factory.createIdentifier('undefined');
+        }
+        else {
+            resolvedType = preferShorthandIfPossible.originalType;
+        }
+    }
+    const newNode = ts.factory.createPropertyDeclaration(modifiersWithoutInputDecorator, node.name, undefined, undefined, finalInitializer);
     const newPropertyText = result.printer.printNode(ts.EmitHint.Unspecified, newNode, node.getSourceFile());
     const replacements = [];
     if (leadingTodoText !== null) {
@@ -1074,6 +1101,43 @@ function extractTransformOfInput(transform, resolvedType, checker) {
         leadingTodoText,
     };
 }
+/**
+ * Checks if a function node contains any references to 'this'.
+ * This is used to skip migration for functions that reference class members.
+ */
+function containsThisReferences(node) {
+    let hasThis = false;
+    const visit = (node) => {
+        if (hasThis)
+            return;
+        if (node.kind === ts.SyntaxKind.ThisKeyword) {
+            hasThis = true;
+            return;
+        }
+        if (ts.isPropertyAccessExpression(node)) {
+            const expr = node.expression;
+            if (ts.isIdentifier(expr) && expr.text === 'this') {
+                hasThis = true;
+                return;
+            }
+        }
+        ts.forEachChild(node, visit);
+    };
+    if (ts.isArrowFunction(node) || ts.isFunctionExpression(node)) {
+        if (node.body) {
+            if (node.body.kind === ts.SyntaxKind.Block) {
+                node.body.statements.forEach(visit);
+            }
+            else {
+                visit(node.body);
+            }
+        }
+    }
+    else {
+        ts.forEachChild(node, visit);
+    }
+    return hasThis;
+}
 
 /**
  * Phase that migrates `@Input()` declarations to signal inputs and
@@ -1234,7 +1298,7 @@ function pass9__migrateTypeScriptTypeReferences(host, references, importManager,
  */
 function executeMigrationPhase(host, knownInputs, result, info) {
     const { typeChecker, sourceFiles } = info;
-    const importManager = new project_tsconfig_paths.ImportManager({
+    const importManager = new migrations.ImportManager({
         // For the purpose of this migration, we always use `input` and don't alias
         // it to e.g. `input_1`.
         generateUniqueIdentifier: () => null,

package/schematics/bundles/signal-queries-migration.cjs

@@ -1,28 +1,25 @@
 'use strict';
 /**
- * @license Angular v21.0.0-next.1
+ * @license Angular v21.0.0-next.10
  * (c) 2010-2025 Google LLC. https://angular.io/
  * License: MIT
  */
 'use strict';
 
-var project_tsconfig_paths = require('./project_tsconfig_paths-D7xzGqRi.cjs');
+var migrations = require('@angular/compiler-cli/private/migrations');
 var ts = require('typescript');
-require('os');
-var index$1 = require('./index-jjHOgYYs.cjs');
-require('path');
+require('@angular/compiler-cli');
 require('node:path');
-var project_paths = require('./project_paths-T_M15e2g.cjs');
-var apply_import_manager = require('./apply_import_manager-yycO3l8f.cjs');
-var migrate_ts_type_references = require('./migrate_ts_type_references-BVSg43hf.cjs');
+var project_paths = require('./project_paths-DvD50ouC.cjs');
+var apply_import_manager = require('./apply_import_manager-1Zs_gpB6.cjs');
+var migrate_ts_type_references = require('./migrate_ts_type_references-UGIUl7En.cjs');
 var assert = require('assert');
-var index = require('./index-BdH2rlWJ.cjs');
+var index = require('./index-B7I9sIUx.cjs');
+var compiler = require('@angular/compiler');
 require('@angular-devkit/core');
 require('node:path/posix');
-require('fs');
-require('module');
-require('url');
 require('@angular-devkit/schematics');
+require('./project_tsconfig_paths-CDVxT6Ov.cjs');
 require('./leading_space-D9nQ8UQC.cjs');
 
 /**
@@ -159,7 +156,7 @@ function computeReplacementsToMigrateQuery(node, metadata, importManager, info,
         }
     }
     if (metadata.queryInfo.read !== null) {
-        assert(metadata.queryInfo.read instanceof project_tsconfig_paths.WrappedNodeExpr);
+        assert(metadata.queryInfo.read instanceof compiler.WrappedNodeExpr);
         optionProperties.push(ts.factory.createPropertyAssignment('read', metadata.queryInfo.read.node));
     }
     if (metadata.queryInfo.descendants !== defaultDescendants) {
@@ -203,7 +200,7 @@ function computeReplacementsToMigrateQuery(node, metadata, importManager, info,
     // If the original property type and the read type are matching, we can rely
     // on the TS inference, instead of repeating types, like in `viewChild<Button>(Button)`.
     if (type !== undefined &&
-        resolvedReadType instanceof project_tsconfig_paths.WrappedNodeExpr &&
+        resolvedReadType instanceof compiler.WrappedNodeExpr &&
         ts.isIdentifier(resolvedReadType.node) &&
         ts.isTypeReferenceNode(type) &&
         ts.isIdentifier(type.typeName) &&
@@ -291,7 +288,7 @@ function extractSourceQueryDefinition(node, reflector, evaluator, info) {
         return null;
     }
     const decorators = reflector.getDecoratorsOfDeclaration(node) ?? [];
-    const ngDecorators = project_tsconfig_paths.getAngularDecorators(decorators, project_tsconfig_paths.queryDecoratorNames, /* isCore */ false);
+    const ngDecorators = migrations.getAngularDecorators(decorators, migrations.queryDecoratorNames, /* isCore */ false);
     if (ngDecorators.length === 0) {
         return null;
     }
@@ -318,10 +315,10 @@ function extractSourceQueryDefinition(node, reflector, evaluator, info) {
     }
     let queryInfo = null;
     try {
-        queryInfo = project_tsconfig_paths.extractDecoratorQueryMetadata(node, decorator.name, decorator.args ?? [], node.name.text, reflector, evaluator);
+        queryInfo = migrations.extractDecoratorQueryMetadata(node, decorator.name, decorator.args ?? [], node.name.text, reflector, evaluator);
     }
     catch (e) {
-        if (!(e instanceof project_tsconfig_paths.FatalDiagnosticError)) {
+        if (!(e instanceof migrations.FatalDiagnosticError)) {
             throw e;
         }
         console.error(`Skipping query: ${e.node.getSourceFile().fileName}: ${e.toString()}`);
@@ -532,7 +529,7 @@ function checkNonTsReferenceAccessesField(ref, fieldName) {
     if (ref.from.read !== readFromPath) {
         return null;
     }
-    if (!(parentRead instanceof project_tsconfig_paths.PropertyRead) || parentRead.name !== fieldName) {
+    if (!(parentRead instanceof compiler.PropertyRead) || parentRead.name !== fieldName) {
         return null;
     }
     return parentRead;
@@ -570,7 +567,7 @@ function checkNonTsReferenceCallsField(ref, fieldName) {
         return null;
     }
     const potentialCall = ref.from.readAstPath[accessIdx - 1];
-    if (potentialCall === undefined || !(potentialCall instanceof project_tsconfig_paths.Call)) {
+    if (potentialCall === undefined || !(potentialCall instanceof compiler.Call)) {
         return null;
     }
     return potentialCall;
@@ -740,8 +737,8 @@ class SignalQueriesMigration extends project_paths.TsurgeComplexMigration {
         }
         const { sourceFiles, program } = info;
         const checker = program.getTypeChecker();
-        const reflector = new project_tsconfig_paths.TypeScriptReflectionHost(checker);
-        const evaluator = new index$1.PartialEvaluator(reflector, checker, null);
+        const reflector = new migrations.TypeScriptReflectionHost(checker);
+        const evaluator = new migrations.PartialEvaluator(reflector, checker, null);
         const res = {
             knownQueryFields: {},
             potentialProblematicQueries: {},
@@ -789,7 +786,7 @@ class SignalQueriesMigration extends project_paths.TsurgeComplexMigration {
                 }
                 // Migrating fields with `@HostBinding` is incompatible as
                 // the host binding decorator does not invoke the signal.
-                const hostBindingDecorators = project_tsconfig_paths.getAngularDecorators(extractedQuery.fieldDecorators, ['HostBinding'], 
+                const hostBindingDecorators = migrations.getAngularDecorators(extractedQuery.fieldDecorators, ['HostBinding'], 
                 /* isCore */ false);
                 if (hostBindingDecorators.length > 0) {
                     markFieldIncompatibleInMetadata(res.potentialProblematicQueries, extractedQuery.id, migrate_ts_type_references.FieldIncompatibilityReason.SignalIncompatibleWithHostBinding);
@@ -928,10 +925,10 @@ class SignalQueriesMigration extends project_paths.TsurgeComplexMigration {
         const resourceLoader = info.ngCompiler?.['resourceManager'] ?? null;
         const { program, sourceFiles } = info;
         const checker = program.getTypeChecker();
-        const reflector = new project_tsconfig_paths.TypeScriptReflectionHost(checker);
-        const evaluator = new index$1.PartialEvaluator(reflector, checker, null);
+        const reflector = new migrations.TypeScriptReflectionHost(checker);
+        const evaluator = new migrations.PartialEvaluator(reflector, checker, null);
         const replacements = [];
-        const importManager = new project_tsconfig_paths.ImportManager();
+        const importManager = new migrations.ImportManager();
         const printer = ts.createPrinter();
         const filesWithSourceQueries = new Map();
         const filesWithIncompleteMigration = new Map();

package/schematics/bundles/signals.cjs

@@ -1,6 +1,6 @@
 'use strict';
 /**
- * @license Angular v21.0.0-next.1
+ * @license Angular v21.0.0-next.10
  * (c) 2010-2025 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -10,22 +10,19 @@ var schematics = require('@angular-devkit/schematics');
 var signalQueriesMigration = require('./signal-queries-migration.cjs');
 var signalInputMigration = require('./signal-input-migration.cjs');
 var outputMigration = require('./output-migration.cjs');
-require('./project_tsconfig_paths-D7xzGqRi.cjs');
+require('@angular/compiler-cli/private/migrations');
 require('typescript');
-require('os');
-require('fs');
-require('module');
-require('path');
-require('url');
-require('@angular-devkit/core');
-require('./index-jjHOgYYs.cjs');
+require('@angular/compiler-cli');
 require('node:path');
-require('./project_paths-T_M15e2g.cjs');
+require('./project_paths-DvD50ouC.cjs');
+require('@angular-devkit/core');
 require('node:path/posix');
-require('./apply_import_manager-yycO3l8f.cjs');
-require('./migrate_ts_type_references-BVSg43hf.cjs');
+require('./project_tsconfig_paths-CDVxT6Ov.cjs');
+require('./apply_import_manager-1Zs_gpB6.cjs');
+require('./migrate_ts_type_references-UGIUl7En.cjs');
 require('assert');
-require('./index-BdH2rlWJ.cjs');
+require('./index-B7I9sIUx.cjs');
+require('@angular/compiler');
 require('./leading_space-D9nQ8UQC.cjs');
 
 function migrate(options) {