@markuplint/rules 4.0.0-alpha.1 → 4.0.0-alpha.10

package/.eslintrc

@@ -0,0 +1,5 @@
+{
+	"rules": {
+		"import/no-default-export": 0
+	}
+}

package/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2017-2019 Yusuke Hirao
+Copyright (c) 2017-2024 Yusuke Hirao
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

package/lib/attr-check.js

@@ -71,10 +71,10 @@ export function attrCheck(t, name, value, isCustomRule, spec) {
             };
         }
     });
-    if (invalidList.some(i => i === false)) {
+    if (invalidList.includes(false)) {
         return false;
     }
-    const invalid = invalidList.find(i => i);
+    const invalid = invalidList.find(Boolean);
     return invalid ?? false;
 }
 export function valueCheck(t, name, value, type) {

package/lib/attr-duplication/index.js

@@ -1,5 +1,8 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     async verify({ document, report, t }) {
         const message = t('{0} is {1:c}', t('the {0}', 'attribute name'), 'duplicated');
         await document.walkOn('Element', node => {

package/lib/attr-value-quotes/index.js

@@ -4,6 +4,9 @@ const quoteList = {
     single: "'",
 };
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     defaultValue: 'double',
     async verify({ document, report, t }) {
@@ -26,7 +29,6 @@ export default createRule({
             const quote = quoteList[attr.rule.value];
             if (quote && attr.startQuote && attr.startQuote.raw !== quote) {
                 attr.startQuote.fix(quote);
-                // TODO: attr.endQuote = new MLToken(quote);
                 attr.endQuote?.fix(quote);
             }
         });

package/lib/case-sensitive-attr-name/index.js

@@ -1,5 +1,8 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     defaultValue: 'lower',
     async verify({ document, report, t }) {

package/lib/case-sensitive-tag-name/index.js

@@ -1,5 +1,8 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     defaultValue: 'lower',
     async verify({ document, report, t }) {

package/lib/character-reference/index.js

@@ -1,11 +1,14 @@
 import { createRule, getLocationFromChars } from '@markuplint/ml-core';
 const defaultChars = ['"', '&', '<', '>'];
-const ignoreParentElement = ['script', 'style'];
+const ignoreParentElement = new Set(['script', 'style']);
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     async verify({ document, report, t }) {
         const targetNodes = [];
         await document.walkOn('Text', node => {
-            if (node.parentNode && ignoreParentElement.includes(node.parentNode.nodeName.toLowerCase())) {
+            if (node.parentNode && ignoreParentElement.has(node.parentNode.nodeName.toLowerCase())) {
                 return;
             }
             const severity = node.rule.severity;
@@ -41,14 +44,14 @@ export default createRule({
             if (!('scope' in targetNode && 'line' in targetNode && targetNode.line != null)) {
                 continue;
             }
-            const escapedText = targetNode.raw.replace(/&(?:[a-z]+|#[0-9]+|#x[0-9a-f]+);/gi, $0 => '*'.repeat($0.length));
-            getLocationFromChars(defaultChars, escapedText, targetNode.line, targetNode.col).forEach(location => {
+            const escapedText = targetNode.raw.replaceAll(/&(?:[a-z]+|#\d+|#x[\da-f]+);/gi, $0 => '*'.repeat($0.length));
+            for (const location of getLocationFromChars(defaultChars, escapedText, targetNode.line, targetNode.col)) {
                 report({
                     scope: targetNode.scope,
                     message: targetNode.message,
                     ...location,
                 });
-            });
+            }
         }
     },
 });

package/lib/class-naming/index.js

@@ -2,6 +2,9 @@ import { createRule } from '@markuplint/ml-core';
 import { toNoEmptyStringArrayFromStringOrArray } from '@markuplint/shared';
 import { match } from '../helpers.js';
 export default createRule({
+    meta: {
+        category: 'naming-convention',
+    },
     defaultSeverity: 'warning',
     defaultValue: null,
     async verify({ document, report, t }) {
@@ -14,9 +17,9 @@ export default createRule({
                 }
                 const classAttr = attr.valueNode;
                 const classList = attr.value
-                    .split(/\s+/g)
+                    .split(/\s+/)
                     .map(c => c.trim())
-                    .filter(c => c);
+                    .filter(Boolean);
                 for (const className of classList) {
                     if (!classPatterns.some(pattern => match(className, pattern))) {
                         report({

package/lib/create-message.d.ts

@@ -3,4 +3,4 @@ import type { AttributeType } from '@markuplint/ml-spec';
 import type { UnmatchedResult } from '@markuplint/types';
 import type { ReadonlyDeep } from 'type-fest';
 export declare function createMessageValueExpected(t: Translator, baseTarget: string, type: ReadonlyDeep<AttributeType>, matches: UnmatchedResult): string;
-export declare function __createMessageValueExpected(t: Translator, baseTarget: string, expected: string | null, matches: Pick<UnmatchedResult, 'partName' | 'reason' | 'raw' | 'candidate' | 'ref' | 'extra'>): string;
+export declare function __createMessageValueExpected(t: Translator, baseTarget: string, expected: string | null, matches: Pick<UnmatchedResult, 'partName' | 'reason' | 'raw' | 'candidate' | 'ref' | 'extra' | 'fallbackTo'>): string;

package/lib/create-message.js

@@ -13,7 +13,7 @@ export function createMessageValueExpected(t, baseTarget, type, matches) {
     }
     const expected = createExpectedObject(tokenType, matches, t);
     const message = [listDescriptionPart, __createMessageValueExpected(t, target, expected, matches)]
-        .filter(s => s)
+        .filter(Boolean)
         .join(t('. '));
     return message;
 }
@@ -23,6 +23,7 @@ export function __createMessageValueExpected(t, baseTarget, expected, matches) {
     let expectPart;
     let unnecessaryPart;
     let candidatePart;
+    let fallbackToPart;
     let expectOrNeed = 'expects';
     if (matches.partName) {
         if (matches.partName === 'the content of the list') {
@@ -135,11 +136,11 @@ export function __createMessageValueExpected(t, baseTarget, expected, matches) {
                         if (lte != null && gte === lte) {
                             expectedDigits = t('{0} digits', gte);
                         }
-                        else if (lte != null) {
-                            expectedDigits = t('{0} to {1} digits', gte, lte);
+                        else if (lte == null) {
+                            expectedDigits = t('{0} or more digits', gte);
                         }
                         else {
-                            expectedDigits = t('{0} or more digits', gte);
+                            expectedDigits = t('{0} to {1} digits', gte, lte);
                         }
                         if (!expected) {
                             expected = expectedDigits;
@@ -187,7 +188,12 @@ export function __createMessageValueExpected(t, baseTarget, expected, matches) {
     if (matches.candidate) {
         candidatePart = t('Did you mean "{0*}"?', matches.candidate);
     }
-    let message = [reasonPart, unnecessaryPart, expectPart, candidatePart].filter(s => s).join(t('. '));
+    if (matches.fallbackTo) {
+        fallbackToPart = t('The user agent will automatically use "{0*}" instead', matches.fallbackTo);
+    }
+    let message = [reasonPart, unnecessaryPart, expectPart, candidatePart, fallbackToPart]
+        .filter(Boolean)
+        .join(t('. '));
     if (matches.ref) {
         message += ` (${matches.ref})`;
     }
@@ -220,19 +226,24 @@ function createExpectedObject(type, matches, t) {
 }
 function expectValueToWord(t, expect, type) {
     switch (expect.type) {
-        case 'common':
+        case 'common': {
             return expect.value;
-        case 'const':
+        }
+        case 'const': {
             return expect.value ? `%${expect.value}%` : '';
-        case 'format':
+        }
+        case 'format': {
             return t('the {0} format', expect.value);
-        case 'regexp':
+        }
+        case 'regexp': {
             return t('{0} ({1})', 'regular expression', expect.value);
-        case 'syntax':
+        }
+        case 'syntax': {
             if (isKeyword(type) && type[0] === '<') {
                 return t('the CSS Syntax "{0}"', expect.value);
             }
             return t('{0} syntax', expect.value);
+        }
     }
 }
 function createExpectedNumber(t, type) {

package/lib/deprecated-attr/index.js

@@ -1,5 +1,8 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     async verify({ document, report, t }) {
         await document.walkOn('Attr', attr => {
             const attrSpecs = getAttrSpecs(attr.ownerElement, document.specs);

package/lib/deprecated-element/index.js

@@ -1,5 +1,8 @@
 import { createRule, getSpec } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     async verify({ document, report, t }) {
         await document.walkOn('Element', el => {
             if (!(el.namespaceURI === 'http://www.w3.org/1999/xhtml' ||
@@ -9,7 +12,7 @@ export default createRule({
             }
             const spec = getSpec(el, document.specs.specs);
             if (spec && (spec.obsolete != null || spec.deprecated || spec.nonStandard)) {
-                const message = t('{0} is {1:c}', t('the "{0*}" {1}', el.localName, 'element'), spec.deprecated ? 'deprecated' : spec.obsolete != null ? 'obsolete' : 'non-standard');
+                const message = t('{0} is {1:c}', t('the "{0*}" {1}', el.localName, 'element'), spec.deprecated ? 'deprecated' : spec.obsolete == null ? 'non-standard' : 'obsolete');
                 report({
                     scope: el,
                     message,

package/lib/disallowed-element/index.js

@@ -1,16 +1,19 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     defaultValue: [],
     async verify({ document, report, t }) {
         for (const query of document.rule.value) {
             const elements = document.querySelectorAll(query);
-            elements.forEach(el => {
+            for (const el of elements) {
                 const message = t('{0} is disallowed', t('the "{0*}" {1}', query, 'element'));
                 report({
                     scope: el,
                     message,
                 });
-            });
+            }
         }
         await document.walkOn('Element', el => {
             if (el.rule.value === document.rule.value) {
@@ -18,13 +21,13 @@ export default createRule({
             }
             for (const query of el.rule.value) {
                 const elements = el.querySelectorAll(query);
-                elements.forEach(el => {
+                for (const el of elements) {
                     const message = t('{0} is disallowed', t('the "{0*}" {1}', query, 'element'));
                     report({
                         scope: el,
                         message,
                     });
-                });
+                }
             }
         });
     },

package/lib/doctype/index.js

@@ -1,5 +1,8 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     defaultValue: 'always',
     defaultOptions: {
         denyObsoleteType: true,

package/lib/end-tag/index.js

@@ -1,6 +1,9 @@
 import { createRule } from '@markuplint/ml-core';
 import { isVoidElement } from '@markuplint/ml-spec';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         if (document.endTag === 'never') {

package/lib/helpers.js

@@ -1,5 +1,9 @@
+var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (receiver, state, kind, f) {
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a getter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot read private member from an object whose class did not declare it");
+    return kind === "m" ? f : kind === "a" ? f.call(receiver) : f ? f.value : state.get(receiver);
+};
 var _Collection_items;
-import { __classPrivateFieldGet } from "tslib";
 // @ts-ignore
 import structuredClone from '@ungap/structured-clone';
 import { attrCheck } from './attr-check.js';
@@ -13,7 +17,7 @@ node, condition) {
     return node.matches(condSelector);
 }
 export function match(needle, pattern) {
-    const matches = pattern.match(/^\/(.*)\/(i|g|m)*$/);
+    const matches = pattern.match(/^\/(.*)\/([gim])*$/);
     if (matches && matches[1]) {
         const re = matches[1];
         const flag = matches[2];
@@ -87,7 +91,7 @@ export function toNormalizedValue(value, spec) {
     }
     if (typeof spec.type === 'string') {
         if (spec.type[0] === '<') {
-            normalized = normalized.toLowerCase().trim().replace(/\s+/g, ' ');
+            normalized = normalized.toLowerCase().trim().replaceAll(/\s+/g, ' ');
         }
     }
     else {
@@ -99,10 +103,10 @@ export function toNormalizedValue(value, spec) {
                 normalized = normalized.trim();
             }
             if (spec.type.separator === 'space') {
-                normalized = normalized.replace(/\s+/g, ' ');
+                normalized = normalized.replaceAll(/\s+/g, ' ');
             }
             if (spec.type.separator === 'comma') {
-                normalized = normalized.replace(/\s*,\s*/g, ',');
+                normalized = normalized.replaceAll(/\s*,\s*/g, ',');
             }
         }
     }
@@ -157,7 +161,7 @@ export class Collection {
         }
     }
     toArray() {
-        return Object.freeze(Array.from(__classPrivateFieldGet(this, _Collection_items, "f")));
+        return Object.freeze([...__classPrivateFieldGet(this, _Collection_items, "f")]);
     }
 }
 export function deepCopy(value) {

package/lib/id-duplication/index.js

@@ -1,5 +1,8 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     async verify({ document, report, t }) {
         const message = t('{0} is {1:c}', t('{0} of {1}', t('the {0}', 'value'), t('the "{0*}" {1}', 'id', 'attribute')), 'duplicated');
         const idStack = [];

package/lib/index.d.ts

@@ -72,7 +72,7 @@ declare const rules: {
     }>>;
     readonly 'no-hard-code-id': Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, undefined>>;
     readonly 'no-refer-to-non-existent-id': Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, {
-        ariaVersion: "1.1" | "1.2" | "1.3";
+        ariaVersion: "1.2" | "1.1" | "1.3";
         fragmentRefersNameAttr: boolean;
     }>>;
     readonly 'no-use-event-handler-attr': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {
@@ -81,7 +81,7 @@ declare const rules: {
     readonly 'permitted-contents': Readonly<import("@markuplint/ml-core").RuleSeed<import("./permitted-contents/types.js").TagRule[], import("./permitted-contents/types.js").Options>>;
     readonly 'placeholder-label-option': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, undefined>>;
     readonly 'require-accessible-name': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {
-        ariaVersion: "1.1" | "1.2" | "1.3";
+        ariaVersion: "1.2" | "1.1" | "1.3";
     }>>;
     readonly 'require-datetime': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {
         langs?: import("./require-datetime/types.js").Lang[] | undefined;
@@ -96,6 +96,10 @@ declare const rules: {
     readonly 'required-h1': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, import("./required-h1/index.js").Options>>;
     readonly 'use-list': Readonly<import("@markuplint/ml-core").RuleSeed<readonly string[], {
         spaceNeededBullets?: string[] | undefined;
+        noPrev?: boolean | undefined;
+        prevElement?: boolean | undefined;
+        prevComment?: boolean | undefined;
+        prevCodeBlock?: boolean | undefined;
     }>>;
     readonly 'wai-aria': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, import("./wai-aria/types.js").Options>>;
 };

package/lib/ineffective-attr/index.js

@@ -1,6 +1,9 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 import { toNoEmptyStringArrayFromStringOrArray } from '@markuplint/shared';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         await document.walkOn('Attr', attr => {

package/lib/invalid-attr/index.js

@@ -4,6 +4,9 @@ import { log as ruleLog } from '../debug.js';
 import { isValidAttr, match } from '../helpers.js';
 const log = ruleLog.extend('invalid-attr');
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     defaultOptions: {},
     async verify({ document, report, t }) {
         await document.walkOn('Attr', attr => {

package/lib/label-has-control/index.js

@@ -1,6 +1,9 @@
 import { createRule } from '@markuplint/ml-core';
 const controlSelector = ["input:not([type='hidden' i])", 'select', 'textarea'].join(',');
 export default createRule({
+    meta: {
+        category: 'a11y',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         await document.walkOn('Element', el => {

package/lib/landmark-roles/index.js

@@ -10,6 +10,9 @@ const selectors = {
 };
 const topLevelRoles = ['banner', 'main', 'complementary', 'contentinfo'];
 export default createRule({
+    meta: {
+        category: 'a11y',
+    },
     defaultSeverity: 'warning',
     defaultOptions: {
         ignoreRoles: [],
@@ -20,13 +23,13 @@ export default createRule({
             return;
         }
         const roles = {
-            complementary: Array.from(document.querySelectorAll(selectors.complementary.join(','))),
-            contentinfo: Array.from(document.querySelectorAll(selectors.contentinfo.join(','))),
-            form: Array.from(document.querySelectorAll(selectors.form.join(','))),
-            banner: Array.from(document.querySelectorAll(selectors.banner.join(','))),
-            main: Array.from(document.querySelectorAll(selectors.main.join(','))),
-            navigation: Array.from(document.querySelectorAll(selectors.navigation.join(','))),
-            region: Array.from(document.querySelectorAll(selectors.region.join(','))),
+            complementary: [...document.querySelectorAll(selectors.complementary.join(','))],
+            contentinfo: [...document.querySelectorAll(selectors.contentinfo.join(','))],
+            form: [...document.querySelectorAll(selectors.form.join(','))],
+            banner: [...document.querySelectorAll(selectors.banner.join(','))],
+            main: [...document.querySelectorAll(selectors.main.join(','))],
+            navigation: [...document.querySelectorAll(selectors.navigation.join(','))],
+            region: [...document.querySelectorAll(selectors.region.join(','))],
         };
         /**
          * `<header>`
@@ -40,7 +43,7 @@ export default createRule({
          * > - nav
          * > - section
          */
-        const headers = Array.from(document.querySelectorAll('header')).filter(header => {
+        const headers = [...document.querySelectorAll('header')].filter(header => {
             return !header.closest('article, aside, main, nav, section');
         });
         roles.banner.push(...headers);
@@ -56,7 +59,7 @@ export default createRule({
          * > - nav
          * > - section
          */
-        const footers = Array.from(document.querySelectorAll('footer')).filter(footer => {
+        const footers = [...document.querySelectorAll('footer')].filter(footer => {
             return !footer.closest('article, aside, main, nav, section');
         });
         roles.contentinfo.push(...footers);
@@ -106,9 +109,7 @@ export default createRule({
 function landmarkRoleElementUUIDList(
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
 roleset) {
-    return Object.values(roleset)
-        .map(elements => elements.map(element => element.uuid))
-        .flat();
+    return Object.values(roleset).flatMap(elements => elements.map(element => element.uuid));
 }
 function hasLabel(
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types

package/lib/no-boolean-attr-value/index.js

@@ -1,5 +1,8 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         await document.walkOn('Attr', attr => {

package/lib/no-default-value/index.js

@@ -1,6 +1,9 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 import { toNormalizedValue } from '../helpers.js';
 export default createRule({
+    meta: {
+        category: 'style',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         await document.walkOn('Attr', attr => {

package/lib/no-empty-palpable-content/index.js

@@ -1,6 +1,9 @@
 import { createRule } from '@markuplint/ml-core';
 import { isNothingContentModel, isPalpableElement } from '@markuplint/ml-spec';
 export default createRule({
+    meta: {
+        category: 'validation',
+    },
     defaultSeverity: 'warning',
     defaultOptions: {
         extendsExposableElements: true,
@@ -28,7 +31,7 @@ export default createRule({
             if (el.rule.options.ignoreIfAriaBusy && el.getAttribute('aria-busy') === 'true') {
                 return;
             }
-            const isEmpty = Array.from(el.childNodes).every(node => node.is(node.TEXT_NODE) && node.isWhitespace());
+            const isEmpty = [...el.childNodes].every(node => node.is(node.TEXT_NODE) && node.isWhitespace());
             if (isEmpty) {
                 report({
                     scope: el,

package/lib/no-hard-code-id/index.js

@@ -1,5 +1,8 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
+    meta: {
+        category: 'maintainability',
+    },
     defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         if (!document.isFragment) {

package/lib/no-refer-to-non-existent-id/index.d.ts

@@ -1,5 +1,5 @@
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, {
-    ariaVersion: "1.1" | "1.2" | "1.3";
+    ariaVersion: "1.2" | "1.1" | "1.3";
     fragmentRefersNameAttr: boolean;
 }>>;
 export default _default;

package/lib/no-refer-to-non-existent-id/index.js

@@ -3,6 +3,9 @@ import { ARIA_RECOMMENDED_VERSION } from '@markuplint/ml-spec';
 import { decodeEntities, decodeHref } from '@markuplint/shared';
 const HYPERLINK_SELECTOR = 'a[href], area[href]';
 export default createRule({
+    meta: {
+        category: 'a11y',
+    },
     defaultOptions: {
         ariaVersion: ARIA_RECOMMENDED_VERSION,
         fragmentRefersNameAttr: false,
@@ -16,10 +19,10 @@ export default createRule({
         if (isMutable) {
             return;
         }
-        document.querySelectorAll('[id]').forEach(el => {
+        for (const el of document.querySelectorAll('[id]')) {
             const attr = el.getAttributeNode('id');
             if (!attr) {
-                return;
+                continue;
             }
             if (attr.isDynamicValue) {
                 hasDynamicId = true;
@@ -27,14 +30,14 @@ export default createRule({
             if (attr.valueType !== 'code') {
                 idList.add(decodeEntities(attr.value));
             }
-        });
+        }
         if (hasDynamicId) {
             return;
         }
-        document.querySelectorAll('[name]').forEach(el => {
+        for (const el of document.querySelectorAll('[name]')) {
             const attr = el.getAttributeNode('name');
             if (!attr) {
-                return;
+                continue;
             }
             if (attr.isDynamicValue) {
                 hasDynamicName = true;
@@ -42,7 +45,7 @@ export default createRule({
             if (attr.valueType !== 'code') {
                 nameList.add(decodeEntities(attr.value));
             }
-        });
+        }
         await document.walkOn('Attr', attr => {
             const attrSpec = getAttrSpecs(attr.ownerElement, document.specs);
             if (!attrSpec) {
@@ -75,7 +78,7 @@ export default createRule({
                     const refs = value
                         .split(spec.type.separator === 'space' ? /\s/ : ',')
                         .map(id => id.trim())
-                        .filter(_ => _);
+                        .filter(Boolean);
                     for (const ref of refs) {
                         if (!idList.has(ref)) {
                             report({
@@ -105,7 +108,7 @@ export default createRule({
                     const refs = value
                         .split(/\s/)
                         .map(id => id.trim())
-                        .filter(_ => _);
+                        .filter(Boolean);
                     for (const ref of refs) {
                         if (!idList.has(ref)) {
                             report({

package/lib/no-use-event-handler-attr/index.js

@@ -1,6 +1,9 @@
 import { createRule } from '@markuplint/ml-core';
 import { match } from '../helpers.js';
 export default createRule({
+    meta: {
+        category: 'maintainability',
+    },
     defaultSeverity: 'warning',
     defaultOptions: {},
     async verify({ document, report, t }) {