@angular-eslint/eslint-plugin-template 16.0.4-alpha.7 → 16.0.4-alpha.9

package/dist/configs/all.json

@@ -24,6 +24,7 @@
     "@angular-eslint/template/no-interpolation-in-attributes": "error",
     "@angular-eslint/template/no-negated-async": "error",
     "@angular-eslint/template/no-positive-tabindex": "error",
+    "@angular-eslint/template/prefer-self-closing-tags": "error",
     "@angular-eslint/template/role-has-required-aria": "error",
     "@angular-eslint/template/table-scope": "error",
     "@angular-eslint/template/use-track-by-function": "error",

package/dist/index.js

@@ -52,6 +52,7 @@ const no_inline_styles_1 = __importStar(require("./rules/no-inline-styles"));
 const no_interpolation_in_attributes_1 = __importStar(require("./rules/no-interpolation-in-attributes"));
 const no_negated_async_1 = __importStar(require("./rules/no-negated-async"));
 const no_positive_tabindex_1 = __importStar(require("./rules/no-positive-tabindex"));
+const prefer_self_closing_tags_1 = __importStar(require("./rules/prefer-self-closing-tags"));
 const role_has_required_aria_1 = __importStar(require("./rules/role-has-required-aria"));
 const table_scope_1 = __importStar(require("./rules/table-scope"));
 const use_track_by_function_1 = __importStar(require("./rules/use-track-by-function"));
@@ -87,6 +88,7 @@ module.exports = {
         [no_interpolation_in_attributes_1.RULE_NAME]: no_interpolation_in_attributes_1.default,
         [no_negated_async_1.RULE_NAME]: no_negated_async_1.default,
         [no_positive_tabindex_1.RULE_NAME]: no_positive_tabindex_1.default,
+        [prefer_self_closing_tags_1.RULE_NAME]: prefer_self_closing_tags_1.default,
         [role_has_required_aria_1.RULE_NAME]: role_has_required_aria_1.default,
         [table_scope_1.RULE_NAME]: table_scope_1.default,
         [use_track_by_function_1.RULE_NAME]: use_track_by_function_1.default,

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

@@ -0,0 +1,53 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RULE_NAME = exports.MESSAGE_ID = void 0;
+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");
+exports.MESSAGE_ID = 'preferSelfClosingTags';
+exports.RULE_NAME = 'prefer-self-closing-tags';
+exports.default = (0, create_eslint_rule_1.createESLintRule)({
+    name: exports.RULE_NAME,
+    meta: {
+        type: 'layout',
+        docs: {
+            description: 'Ensures that self-closing tags are used for elements with a closing tag but no content.',
+            recommended: false,
+        },
+        fixable: 'code',
+        schema: [],
+        messages: {
+            [exports.MESSAGE_ID]: 'Use self-closing tags for elements with a closing tag but no content.',
+        },
+    },
+    defaultOptions: [],
+    create(context) {
+        const parserServices = (0, utils_1.getTemplateParserServices)(context);
+        return {
+            Element$1({ children, name, startSourceSpan, endSourceSpan, }) {
+                // Ignore native elements.
+                if ((0, get_dom_elements_1.getDomElements)().has(name)) {
+                    return;
+                }
+                const noContent = !children.length ||
+                    children.every((node) => {
+                        const text = node.value;
+                        // If the node has no value, or only whitespace,
+                        // we can consider it empty.
+                        return (typeof text === 'string' && text.replace(/\n/g, '').trim() === '');
+                    });
+                const noCloseTag = !endSourceSpan ||
+                    (startSourceSpan.start.offset === endSourceSpan.start.offset &&
+                        startSourceSpan.end.offset === endSourceSpan.end.offset);
+                if (!noContent || noCloseTag) {
+                    return;
+                }
+                context.report({
+                    loc: parserServices.convertNodeSourceSpanToLoc(endSourceSpan),
+                    messageId: exports.MESSAGE_ID,
+                    fix: (fixer) => fixer.replaceTextRange([startSourceSpan.end.offset - 1, endSourceSpan.end.offset], ' />'),
+                });
+            },
+        };
+    },
+});

package/dist/rules/valid-aria.js

@@ -98,6 +98,12 @@ function isNil(value) {
 function isString(value) {
     return typeof value == 'string';
 }
+function isMixed(value) {
+    return isString(value) && value === 'mixed';
+}
+function isTristate(value) {
+    return isMixed(value) || isBooleanLike(value) || isNil(value);
+}
 function isValidAriaPropertyValue({ allowundefined, type, values }, attributeValue) {
     if (allowundefined && isNil(attributeValue))
         return true;
@@ -105,7 +111,7 @@ function isValidAriaPropertyValue({ allowundefined, type, values }, attributeVal
         case 'boolean':
             return isBooleanLike(attributeValue);
         case 'tristate':
-            return isBooleanLike(attributeValue) || isNil(attributeValue);
+            return isTristate(attributeValue);
         case 'id':
         case 'idlist':
             return true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-eslint/eslint-plugin-template",
-  "version": "16.0.4-alpha.7+580c842",
+  "version": "16.0.4-alpha.9+6d26c59",
   "description": "ESLint plugin for Angular Templates",
   "license": "MIT",
   "main": "dist/index.js",
@@ -17,8 +17,8 @@
     "LICENSE"
   ],
   "dependencies": {
-    "@angular-eslint/bundled-angular-compiler": "16.0.4-alpha.7+580c842",
-    "@angular-eslint/utils": "16.0.4-alpha.7+580c842",
+    "@angular-eslint/bundled-angular-compiler": "16.0.4-alpha.9+6d26c59",
+    "@angular-eslint/utils": "16.0.4-alpha.9+6d26c59",
     "@typescript-eslint/type-utils": "5.59.8",
     "@typescript-eslint/utils": "5.59.8",
     "aria-query": "5.1.3",
@@ -31,5 +31,5 @@
     "eslint": "^7.20.0 || ^8.0.0",
     "typescript": "*"
   },
-  "gitHead": "580c8422835ec29eeeb3d196c934bf18e243dc09"
+  "gitHead": "6d26c590e4b15e0b28a6ff7467560537e2b9b92d"
 }