@angular-eslint/bundled-angular-compiler 18.1.1-alpha.0 → 18.1.1-alpha.1

package/dist/index.js

@@ -1,7 +1,7 @@
 'use strict';
 
 /**
- * @license Angular v18.0.5
+ * @license Angular v18.1.2
  * (c) 2010-2024 Google LLC. https://angular.io/
  * License: MIT
  */
@@ -6552,6 +6552,45 @@ class AbstractJsEmitterVisitor extends AbstractEmitterVisitor {
         this.visitAllObjects((param) => ctx.print(null, param.name), params, ctx, ',');
     }
 }
+
+/**
+ * @fileoverview
+ * A module to facilitate use of a Trusted Types policy within the JIT
+ * compiler. It lazily constructs the Trusted Types policy, providing helper
+ * utilities for promoting strings to Trusted Types. When Trusted Types are not
+ * available, strings are used as a fallback.
+ * @security All use of this module is security-sensitive and should go through
+ * security review.
+ */
+/**
+ * The Trusted Types policy, or null if Trusted Types are not
+ * enabled/supported, or undefined if the policy has not been created yet.
+ */
+let policy;
+/**
+ * Returns the Trusted Types policy, or null if Trusted Types are not
+ * enabled/supported. The first call to this function will create the policy.
+ */
+function getPolicy() {
+    if (policy === undefined) {
+        const trustedTypes = _global['trustedTypes'];
+        policy = null;
+        if (trustedTypes) {
+            try {
+                policy = trustedTypes.createPolicy('angular#unsafe-jit', {
+                    createScript: (s) => s,
+                });
+            }
+            catch {
+                // trustedTypes.createPolicy throws if called with a name that is
+                // already registered, even in report-only mode. Until the API changes,
+                // catch the error not to break the applications functionally. In such
+                // cases, the code will fall back to using strings.
+            }
+        }
+    }
+    return policy;
+}
 /**
  * Unsafely promote a string to a TrustedScript, falling back to strings when
  * Trusted Types are not available.
@@ -6560,7 +6599,7 @@ class AbstractJsEmitterVisitor extends AbstractEmitterVisitor {
  * interpreted and executed as a script by a browser, e.g. when calling eval.
  */
 function trustedScriptFromString(script) {
-    return script;
+    return getPolicy()?.createScript(script) || script;
 }
 /**
  * Unsafely call the Function constructor with the given string arguments.
@@ -7161,8 +7200,8 @@ class ShadowCss {
          * animation declaration (with possibly multiple animation definitions)
          *
          * The regular expression can be divided in three parts
-         *  - (^|\s+)
-         *    simply captures how many (if any) leading whitespaces are present
+         *  - (^|\s+|,)
+         *    captures how many (if any) leading whitespaces are present or a comma
          *  - (?:(?:(['"])((?:\\\\|\\\2|(?!\2).)+)\2)|(-?[A-Za-z][\w\-]*))
          *    captures two different possible keyframes, ones which are quoted or ones which are valid css
          * idents (custom properties excluded)
@@ -7170,7 +7209,7 @@ class ShadowCss {
          *    simply matches the end of the possible keyframe, valid endings are: a comma, a space, a
          * semicolon or the end of the string
          */
-        this._animationDeclarationKeyframesRe = /(^|\s+)(?:(?:(['"])((?:\\\\|\\\2|(?!\2).)+)\2)|(-?[A-Za-z][\w\-]*))(?=[,\s]|$)/g;
+        this._animationDeclarationKeyframesRe = /(^|\s+|,)(?:(?:(['"])((?:\\\\|\\\2|(?!\2).)+)\2)|(-?[A-Za-z][\w\-]*))(?=[,\s]|$)/g;
     }
     /*
      * Shim some cssText with the given selector. Returns cssText that can be included in the document
@@ -7327,7 +7366,7 @@ class ShadowCss {
      * @returns the updated css rule.
      **/
     _scopeAnimationRule(rule, scopeSelector, unscopedKeyframesSet) {
-        let content = rule.content.replace(/((?:^|\s+|;)(?:-webkit-)?animation(?:\s*):(?:\s*))([^;]+)/g, (_, start, animationDeclarations) => start +
+        let content = rule.content.replace(/((?:^|\s+|;)(?:-webkit-)?animation\s*:\s*),*([^;]+)/g, (_, start, animationDeclarations) => start +
             animationDeclarations.replace(this._animationDeclarationKeyframesRe, (original, leadingSpaces, quote = '', quotedName, nonQuotedName) => {
                 if (quotedName) {
                     return `${leadingSpaces}${this._scopeAnimationKeyframe(`${quote}${quotedName}${quote}`, scopeSelector, unscopedKeyframesSet)}`;
@@ -11117,6 +11156,12 @@ const CHAINABLE = new Set([
     Identifiers.twoWayListener,
     Identifiers.declareLet,
 ]);
+/**
+ * Chaining results in repeated call expressions, causing a deep AST of receiver expressions. To prevent running out of
+ * stack depth the maximum number of chained instructions is limited to this threshold, which has been selected
+ * arbitrarily.
+ */
+const MAX_CHAIN_LENGTH = 256;
 /**
  * Post-process a reified view compilation and convert sequential calls to chainable instructions
  * into chain calls.
@@ -11162,11 +11207,12 @@ function chainOperationsInList(opList) {
         }
         // This instruction can be chained. It can either be added on to the previous chain (if
         // compatible) or it can be the start of a new chain.
-        if (chain !== null && chain.instruction === instruction) {
+        if (chain !== null && chain.instruction === instruction && chain.length < MAX_CHAIN_LENGTH) {
             // This instruction can be added onto the previous chain.
             const expression = chain.expression.callFn(op.statement.expr.args, op.statement.expr.sourceSpan, op.statement.expr.pure);
             chain.expression = expression;
             chain.op.statement = expression.toStmt();
+            chain.length++;
             OpList.remove(op);
         }
         else {
@@ -11175,6 +11221,7 @@ function chainOperationsInList(opList) {
                 op,
                 instruction,
                 expression: op.statement.expr,
+                length: 1,
             };
         }
     }
@@ -12428,6 +12475,7 @@ function getScopeForView(view, parent) {
             kind: SemanticVariableKind.Identifier,
             name: null,
             identifier,
+            local: false,
         });
     }
     for (const op of view.create) {
@@ -12448,6 +12496,7 @@ function getScopeForView(view, parent) {
                             kind: SemanticVariableKind.Identifier,
                             name: null,
                             identifier: op.localRefs[offset].name,
+                            local: false,
                         },
                     });
                 }
@@ -12460,6 +12509,7 @@ function getScopeForView(view, parent) {
                         kind: SemanticVariableKind.Identifier,
                         name: null,
                         identifier: op.declaredName,
+                        local: false,
                     },
                 });
                 break;
@@ -17704,8 +17754,7 @@ class _Tokenizer {
         this._preserveLineEndings = options.preserveLineEndings || false;
         this._i18nNormalizeLineEndingsInICUs = options.i18nNormalizeLineEndingsInICUs || false;
         this._tokenizeBlocks = options.tokenizeBlocks ?? true;
-        // TODO(crisbeto): eventually set this to true.
-        this._tokenizeLet = options.tokenizeLet || false;
+        this._tokenizeLet = options.tokenizeLet ?? true;
         try {
             this._cursor.init();
         }
@@ -17903,8 +17952,11 @@ class _Tokenizer {
         const nameCursor = this._cursor.clone();
         let allowDigit = false;
         this._attemptCharCodeUntilFn((code) => {
-            // `@let` names can't start with a digit, but digits are valid anywhere else in the name.
-            if (isAsciiLetter(code) || code === $_ || (allowDigit && isDigit(code))) {
+            if (isAsciiLetter(code) ||
+                code === $$ ||
+                code === $_ ||
+                // `@let` names can't start with a digit, but digits are valid anywhere else in the name.
+                (allowDigit && isDigit(code))) {
                 allowDigit = true;
                 return false;
             }
@@ -22584,6 +22636,8 @@ function processLexicalScope(unit, ops, savedView) {
     // Since variables are generated in each view for the entire lexical scope (including any
     // identifiers from parent templates) only local variables need be considered here.
     const scope = new Map();
+    // Symbols defined within the current scope. They take precedence over ones defined outside.
+    const localDefinitions = new Map();
     // First, step through the operations list and:
     // 1) build up the `scope` mapping
     // 2) recurse into any listener functions
@@ -22592,6 +22646,17 @@ function processLexicalScope(unit, ops, savedView) {
             case OpKind.Variable:
                 switch (op.variable.kind) {
                     case SemanticVariableKind.Identifier:
+                        if (op.variable.local) {
+                            if (localDefinitions.has(op.variable.identifier)) {
+                                continue;
+                            }
+                            localDefinitions.set(op.variable.identifier, op.xref);
+                        }
+                        else if (scope.has(op.variable.identifier)) {
+                            continue;
+                        }
+                        scope.set(op.variable.identifier, op.xref);
+                        break;
                     case SemanticVariableKind.Alias:
                         // This variable represents some kind of identifier which can be used in the template.
                         if (scope.has(op.variable.identifier)) {
@@ -22630,7 +22695,10 @@ function processLexicalScope(unit, ops, savedView) {
                 // `expr` is a read of a name within the lexical scope of this view.
                 // Either that name is defined within the current view, or it represents a property from the
                 // main component context.
-                if (scope.has(expr.name)) {
+                if (localDefinitions.has(expr.name)) {
+                    return new ReadVariableExpr(localDefinitions.get(expr.name));
+                }
+                else if (scope.has(expr.name)) {
                     // This was a defined variable in the current scope.
                     return new ReadVariableExpr(scope.get(expr.name));
                 }
@@ -23841,6 +23909,7 @@ function generateLocalLetReferences(job) {
                 kind: SemanticVariableKind.Identifier,
                 name: null,
                 identifier: op.declaredName,
+                local: true,
             };
             OpList.replace(op, createVariableOp(job.allocateXrefId(), variable, new StoreLetExpr(op.target, op.value, op.sourceSpan), VariableFlags.None));
         }
@@ -27287,7 +27356,7 @@ function parseTemplate(template, templateUrl, options = {}) {
         ...options,
         tokenizeExpansionForms: true,
         tokenizeBlocks: options.enableBlockSyntax ?? true,
-        tokenizeLet: options.enableLetSyntax ?? false,
+        tokenizeLet: options.enableLetSyntax ?? true,
     });
     if (!options.alwaysAttemptHtmlToR3AstConversion &&
         parseResult.errors &&
@@ -28713,7 +28782,6 @@ function extractScopedNodeEntities(rootScope) {
 class ResourceLoader {
 }
 
-let enableLetSyntax = false;
 class CompilerFacadeImpl {
     constructor(jitEvaluator = new JitEvaluator()) {
         this.jitEvaluator = jitEvaluator;
@@ -28961,6 +29029,25 @@ function convertDirectiveFacadeToMetadata(facade) {
             });
         }
     }
+    const hostDirectives = facade.hostDirectives?.length
+        ? facade.hostDirectives.map((hostDirective) => {
+            return typeof hostDirective === 'function'
+                ? {
+                    directive: wrapReference(hostDirective),
+                    inputs: null,
+                    outputs: null,
+                    isForwardReference: false,
+                }
+                : {
+                    directive: wrapReference(hostDirective.directive),
+                    isForwardReference: false,
+                    inputs: hostDirective.inputs ? parseMappingStringArray(hostDirective.inputs) : null,
+                    outputs: hostDirective.outputs
+                        ? parseMappingStringArray(hostDirective.outputs)
+                        : null,
+                };
+        })
+        : null;
     return {
         ...facade,
         typeArgumentCount: 0,
@@ -28976,10 +29063,18 @@ function convertDirectiveFacadeToMetadata(facade) {
         providers: facade.providers != null ? new WrappedNodeExpr(facade.providers) : null,
         viewQueries: facade.viewQueries.map(convertToR3QueryMetadata),
         fullInheritance: false,
-        hostDirectives: convertHostDirectivesToMetadata(facade),
+        hostDirectives,
     };
 }
 function convertDeclareDirectiveFacadeToMetadata(declaration, typeSourceSpan) {
+    const hostDirectives = declaration.hostDirectives?.length
+        ? declaration.hostDirectives.map((dir) => ({
+            directive: wrapReference(dir.directive),
+            isForwardReference: false,
+            inputs: dir.inputs ? getHostDirectiveBindingMapping(dir.inputs) : null,
+            outputs: dir.outputs ? getHostDirectiveBindingMapping(dir.outputs) : null,
+        }))
+        : null;
     return {
         name: declaration.type.name,
         type: wrapReference(declaration.type),
@@ -28999,7 +29094,7 @@ function convertDeclareDirectiveFacadeToMetadata(declaration, typeSourceSpan) {
         fullInheritance: false,
         isStandalone: declaration.isStandalone ?? false,
         isSignal: declaration.isSignal ?? false,
-        hostDirectives: convertHostDirectivesToMetadata(declaration),
+        hostDirectives,
     };
 }
 function convertHostDeclarationToMetadata(host = {}) {
@@ -29013,25 +29108,17 @@ function convertHostDeclarationToMetadata(host = {}) {
         },
     };
 }
-function convertHostDirectivesToMetadata(metadata) {
-    if (metadata.hostDirectives?.length) {
-        return metadata.hostDirectives.map((hostDirective) => {
-            return typeof hostDirective === 'function'
-                ? {
-                    directive: wrapReference(hostDirective),
-                    inputs: null,
-                    outputs: null,
-                    isForwardReference: false,
-                }
-                : {
-                    directive: wrapReference(hostDirective.directive),
-                    isForwardReference: false,
-                    inputs: hostDirective.inputs ? parseMappingStringArray(hostDirective.inputs) : null,
-                    outputs: hostDirective.outputs ? parseMappingStringArray(hostDirective.outputs) : null,
-                };
-        });
+/**
+ * Parses a host directive mapping where each odd array key is the name of an input/output
+ * and each even key is its public name, e.g. `['one', 'oneAlias', 'two', 'two']`.
+ */
+function getHostDirectiveBindingMapping(array) {
+    let result = null;
+    for (let i = 1; i < array.length; i += 2) {
+        result = result || {};
+        result[array[i - 1]] = array[i];
     }
-    return null;
+    return result;
 }
 function convertOpaqueValuesToExpressions(obj) {
     const result = {};
@@ -29125,7 +29212,6 @@ function parseJitTemplate(template, typeName, sourceMapUrl, preserveWhitespaces,
     const parsed = parseTemplate(template, sourceMapUrl, {
         preserveWhitespaces,
         interpolationConfig,
-        enableLetSyntax,
     });
     if (parsed.errors !== null) {
         const errors = parsed.errors.map((err) => err.toString()).join(', ');
@@ -29355,7 +29441,7 @@ function publishFacade(global) {
  * @description
  * Entry point for all public APIs of the compiler package.
  */
-const VERSION = new Version('18.0.5');
+const VERSION = new Version('18.1.2');
 
 class CompilerConfig {
     constructor({ defaultEncapsulation = exports.ViewEncapsulation.Emulated, preserveWhitespaces, strictInjectionParameters, } = {}) {
@@ -30993,7 +31079,7 @@ const MINIMUM_PARTIAL_LINKER_DEFER_SUPPORT_VERSION = '18.0.0';
 function compileDeclareClassMetadata(metadata) {
     const definitionMap = new DefinitionMap();
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION$5));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', metadata.type);
     definitionMap.set('decorators', metadata.decorators);
@@ -31011,7 +31097,7 @@ function compileComponentDeclareClassMetadata(metadata, dependencies) {
     callbackReturnDefinitionMap.set('ctorParameters', metadata.ctorParameters ?? literal(null));
     callbackReturnDefinitionMap.set('propDecorators', metadata.propDecorators ?? literal(null));
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_DEFER_SUPPORT_VERSION));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', metadata.type);
     definitionMap.set('resolveDeferredDeps', compileComponentMetadataAsyncResolver(dependencies));
@@ -31106,7 +31192,7 @@ function createDirectiveDefinitionMap(meta) {
     const definitionMap = new DefinitionMap();
     const minVersion = getMinimumVersionForPartialOutput(meta);
     definitionMap.set('minVersion', literal(minVersion));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     // e.g. `type: MyDirective`
     definitionMap.set('type', meta.type.value);
     if (meta.isStandalone) {
@@ -31525,7 +31611,7 @@ const MINIMUM_PARTIAL_LINKER_VERSION$4 = '12.0.0';
 function compileDeclareFactoryFunction(meta) {
     const definitionMap = new DefinitionMap();
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION$4));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', meta.type.value);
     definitionMap.set('deps', compileDependencies(meta.deps));
@@ -31560,7 +31646,7 @@ function compileDeclareInjectableFromMetadata(meta) {
 function createInjectableDefinitionMap(meta) {
     const definitionMap = new DefinitionMap();
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION$3));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', meta.type.value);
     // Only generate providedIn property if it has a non-null value
@@ -31611,7 +31697,7 @@ function compileDeclareInjectorFromMetadata(meta) {
 function createInjectorDefinitionMap(meta) {
     const definitionMap = new DefinitionMap();
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION$2));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', meta.type.value);
     definitionMap.set('providers', meta.providers);
@@ -31644,7 +31730,7 @@ function createNgModuleDefinitionMap(meta) {
         throw new Error('Invalid path! Local compilation mode should not get into the partial compilation path');
     }
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION$1));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     definitionMap.set('type', meta.type.value);
     // We only generate the keys in the metadata if the arrays contain values.
@@ -31695,7 +31781,7 @@ function compileDeclarePipeFromMetadata(meta) {
 function createPipeDefinitionMap(meta) {
     const definitionMap = new DefinitionMap();
     definitionMap.set('minVersion', literal(MINIMUM_PARTIAL_LINKER_VERSION));
-    definitionMap.set('version', literal('18.0.5'));
+    definitionMap.set('version', literal('18.1.2'));
     definitionMap.set('ngImport', importExpr(Identifiers.core));
     // e.g. `type: MyPipe`
     definitionMap.set('type', meta.type.value);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/bundled-angular-compiler",
-  "version": "18.1.1-alpha.0",
+  "version": "18.1.1-alpha.1",
   "description": "A CJS bundled version of @angular/compiler",
   "license": "MIT",
   "main": "dist/index.js",