@angular-eslint/eslint-plugin-template 19.2.1-alpha.4 → 19.2.1

package/dist/rules/prefer-self-closing-tags.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prefer-self-closing-tags.d.ts","sourceRoot":"","sources":["../../src/rules/prefer-self-closing-tags.ts"],"names":[],"mappings":"AASA,MAAM,MAAM,OAAO,GAAG,EAAE,CAAC;AACzB,MAAM,MAAM,UAAU,GAAG,uBAAuB,CAAC;AACjD,eAAO,MAAM,SAAS,6BAA6B,CAAC;;AAEpD,wBAiJG"}
+{"version":3,"file":"prefer-self-closing-tags.d.ts","sourceRoot":"","sources":["../../src/rules/prefer-self-closing-tags.ts"],"names":[],"mappings":"AAUA,MAAM,MAAM,OAAO,GAAG,EAAE,CAAC;AACzB,MAAM,MAAM,UAAU,GAAG,uBAAuB,CAAC;AACjD,eAAO,MAAM,SAAS,6BAA6B,CAAC;;AAEpD,wBA+JG"}

package/dist/rules/prefer-self-closing-tags.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.RULE_NAME = void 0;
+const bundled_angular_compiler_1 = require("@angular-eslint/bundled-angular-compiler");
 const utils_1 = require("@angular-eslint/utils");
 const create_eslint_rule_1 = require("../utils/create-eslint-rule");
 const get_dom_elements_1 = require("../utils/get-dom-elements");
@@ -41,30 +42,40 @@ exports.default = (0, create_eslint_rule_1.createESLintRule)({
             },
         };
         function processElementOrTemplateNode(node) {
-            const { startSourceSpan, endSourceSpan } = node;
+            const { children, startSourceSpan, endSourceSpan } = node;
             if (!endSourceSpan ||
                 (startSourceSpan.start.offset === endSourceSpan.start.offset &&
                     startSourceSpan.end.offset === endSourceSpan.end.offset)) {
                 // No close tag.
                 return;
             }
+            const someChildNodesHaveContent = children.some((node) => {
+                // If the node is only whitespace, we can consider it empty.
+                //
+                // We need to look at the text from the source code, rather
+                // than the `TmplAstText.value` property. The `value` property
+                // contains the HTML-decoded value, so if the raw text contains
+                // ` `, that is decoded to a space, but we don't want to
+                // treat that as empty text.
+                return (!(node instanceof bundled_angular_compiler_1.TmplAstText) ||
+                    context.sourceCode.text
+                        .slice(node.sourceSpan.start.offset, node.sourceSpan.end.offset)
+                        .trim() !== '');
+            });
             // If the element only contains whitespace, we can consider it
-            // empty. We cannot use the children to reliably determine whether
-            // the element only contains whitespace because the template
-            // parser will HTML-decoded the text. For example, if the raw
-            // content contains ` `, that is decoded to a space, but
-            // we need to treat that as content that should be kept.
+            // empty.
             //
-            // Likewise, if the node only contains comments, those comments
+            // If the node only contains comments, those comments
             // will not appear in the syntax tree, which results in the
             // content appearing empty.
             //
             // So instead of using the syntax tree, we'll look at the
             // source code and get the text that appears between the
             // start element and the end element.
-            if (context.sourceCode.text
+            const nodeHasContent = context.sourceCode.text
                 .slice(startSourceSpan.end.offset, endSourceSpan.start.offset)
-                .trim() !== '') {
+                .trim() !== '';
+            if (nodeHasContent || someChildNodesHaveContent) {
                 // The element has content.
                 return;
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/eslint-plugin-template",
-  "version": "19.2.1-alpha.4",
+  "version": "19.2.1",
   "description": "ESLint plugin for Angular Templates",
   "license": "MIT",
   "main": "dist/index.js",
@@ -20,13 +20,13 @@
   "dependencies": {
     "aria-query": "5.3.2",
     "axobject-query": "4.1.0",
-    "@angular-eslint/bundled-angular-compiler": "19.2.1-alpha.4",
-    "@angular-eslint/utils": "19.2.1-alpha.4"
+    "@angular-eslint/bundled-angular-compiler": "19.2.1",
+    "@angular-eslint/utils": "19.2.1"
   },
   "devDependencies": {
     "@types/aria-query": "5.0.4",
-    "@angular-eslint/template-parser": "19.2.1-alpha.4",
-    "@angular-eslint/test-utils": "19.2.1-alpha.4"
+    "@angular-eslint/template-parser": "19.2.1",
+    "@angular-eslint/test-utils": "19.2.1"
   },
   "peerDependencies": {
     "@typescript-eslint/types": "^7.11.0 || ^8.0.0",