@markuplint/rules 4.0.0-alpha.3 → 4.0.0-dev.28

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/character-reference/index.js

@@ -1,11 +1,11 @@
 import { createRule, getLocationFromChars } from '@markuplint/ml-core';
 const defaultChars = ['"', '&', '<', '>'];
-const ignoreParentElement = ['script', 'style'];
+const ignoreParentElement = new Set(['script', 'style']);
 export default createRule({
     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 +41,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

@@ -16,7 +16,7 @@ export default createRule({
                 const classList = attr.value
                     .split(/\s+/g)
                     .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.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;
 }
@@ -136,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;
@@ -191,7 +191,9 @@ export function __createMessageValueExpected(t, baseTarget, expected, matches) {
     if (matches.fallbackTo) {
         fallbackToPart = t('The user agent will automatically use "{0*}" instead', matches.fallbackTo);
     }
-    let message = [reasonPart, unnecessaryPart, expectPart, candidatePart, fallbackToPart].filter(s => s).join(t('. '));
+    let message = [reasonPart, unnecessaryPart, expectPart, candidatePart, fallbackToPart]
+        .filter(Boolean)
+        .join(t('. '));
     if (matches.ref) {
         message += ` (${matches.ref})`;
     }
@@ -224,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-element/index.js

@@ -9,7 +9,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

@@ -4,13 +4,13 @@ export default createRule({
     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 +18,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/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/index.d.ts

@@ -15,38 +15,38 @@ declare const rules: {
     readonly 'id-duplication': Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, undefined>>;
     readonly 'ineffective-attr': Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, undefined>>;
     readonly 'invalid-attr': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {
-        allowAttrs?: (string | {
+        allowAttrs?: Record<string, {
+            enum: [string, ...string[]];
+        } | {
+            pattern: string;
+        } | {
+            type: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
+        }> | (string | {
             name: string;
-            value: ({
+            value: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType | ({
                 enum: [string, ...string[]];
             } | {
                 pattern: string;
             } | {
                 type: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-            }) | import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-        })[] | Record<string, {
+            });
+        })[] | undefined;
+        disallowAttrs?: Record<string, {
             enum: [string, ...string[]];
         } | {
             pattern: string;
         } | {
             type: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-        }> | undefined;
-        disallowAttrs?: (string | {
+        }> | (string | {
             name: string;
-            value: ({
+            value: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType | ({
                 enum: [string, ...string[]];
             } | {
                 pattern: string;
             } | {
                 type: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-            }) | import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-        })[] | Record<string, {
-            enum: [string, ...string[]];
-        } | {
-            pattern: string;
-        } | {
-            type: import("packages/@markuplint/ml-spec/lib/index.js").AttributeType;
-        }> | undefined;
+            });
+        })[] | undefined;
         ignoreAttrNamePrefix?: string | string[] | undefined;
         allowToAddPropertiesForPretender?: boolean | undefined;
         attrs?: Record<string, ({
@@ -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;

package/lib/landmark-roles/index.js

@@ -20,13 +20,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 +40,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 +56,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 +106,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-empty-palpable-content/index.js

@@ -28,7 +28,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-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

@@ -16,10 +16,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 +27,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 +42,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 +75,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 +105,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/permitted-contents/choice.js

@@ -14,7 +14,7 @@ elements, specs, options, depth) {
         const result = order(some, collection.unmatched, specs, options, depth + 1);
         if (result.type === 'MATCHED' ||
             result.type === 'MATCHED_ZERO' ||
-            (result.type === 'UNEXPECTED_EXTRA_NODE' && result.matched.length >= 1)) {
+            (result.type === 'UNEXPECTED_EXTRA_NODE' && result.matched.length > 0)) {
             choiceLog('Results[%s]: %s', i, choiceLogString(pattern.choice, i));
             return {
                 type: result.type,

package/lib/permitted-contents/order.js

@@ -41,9 +41,9 @@ nodes, specs, options, depth) {
             if (!barelyMatchedResult) {
                 throw new Error('Unreachable code');
             }
-            orderLog('Result (%s): %s%s', result.type, collection.toString(true), barelyMatchedResult.hint.missing?.barelyMatchedElements != null
-                ? `; But ${barelyMatchedResult.hint.missing.barelyMatchedElements} elements hit out of pattern`
-                : '');
+            orderLog('Result (%s): %s%s', result.type, collection.toString(true), barelyMatchedResult.hint.missing?.barelyMatchedElements == null
+                ? ''
+                : `; But ${barelyMatchedResult.hint.missing.barelyMatchedElements} elements hit out of pattern`);
             return {
                 type: barelyMatchedResult.type,
                 matched: collection.matched,

package/lib/permitted-contents/represent-transparent-nodes.js

@@ -30,7 +30,7 @@ nodes, specs, options) {
         const collection = new Collection(getChildNodesWithoutWhitespaces(node));
         let unmatched;
         if (noTransparentModels.length > 0) {
-            const result = order(noTransparentModels, collection.unmatched, specs, options, Infinity);
+            const result = order(noTransparentModels, collection.unmatched, specs, options, Number.POSITIVE_INFINITY);
             unmatched = result.unmatched;
         }
         else {

package/lib/permitted-contents/transparent.js

@@ -7,7 +7,7 @@ nodes) {
         transparentLog('Skipped');
         return {
             type: nodes.length === 0 ? 'MATCHED_ZERO' : 'MATCHED',
-            matched: nodes.slice(),
+            matched: [...nodes],
             unmatched: [],
             zeroMatch: nodes.length === 0,
             query: 'transparent',
@@ -17,7 +17,7 @@ nodes) {
     transparentLog('Transparent model element is component root');
     return {
         type: 'MATCHED',
-        matched: nodes.slice(),
+        matched: [...nodes],
         unmatched: [],
         zeroMatch: false,
         query: 'transparent',

package/lib/permitted-contents/utils.js

@@ -1,5 +1,15 @@
+var __classPrivateFieldSet = (this && this.__classPrivateFieldSet) || function (receiver, state, value, kind, f) {
+    if (kind === "m") throw new TypeError("Private method is not writable");
+    if (kind === "a" && !f) throw new TypeError("Private accessor was defined without a setter");
+    if (typeof state === "function" ? receiver !== state || !f : !state.has(receiver)) throw new TypeError("Cannot write private member to an object whose class did not declare it");
+    return (kind === "a" ? f.call(receiver, value) : f ? f.value = value : state.set(receiver, value)), value;
+};
+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_locked, _Collection_matched, _Collection_nodes, _Collection_origin;
-import { __classPrivateFieldGet, __classPrivateFieldSet } from "tslib";
 import { createSelector } from '@markuplint/selector';
 import { bgGreen, green, bgRed, bgBlue, blue, bgMagenta, cyan } from './debug.js';
 import { transparentMode } from './represent-transparent-nodes.js';
@@ -11,7 +21,7 @@ el) {
     if (nodes) {
         return nodes;
     }
-    nodes = Array.from(el.childNodes).filter(node => {
+    nodes = [...el.childNodes].filter(node => {
         return !(node.is(node.TEXT_NODE) && node.isWhitespace());
     });
     getChildNodesWithoutWhitespacesCaches.set(el, nodes);
@@ -44,10 +54,8 @@ node, specs) {
         };
     }
     const not = selectorResult
-        .map(r => (r.matched ? [] : r.not ?? []))
-        .flat()
-        .map(descendants)
-        .flat()
+        .flatMap(r => (r.matched ? [] : r.not ?? []))
+        .flatMap(descendants)
         .shift();
     return {
         matched: false,
@@ -57,7 +65,7 @@ node, specs) {
 function descendants(
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
 selectorResult) {
-    let nodes = selectorResult.nodes.slice();
+    let nodes = [...selectorResult.nodes];
     while (selectorResult.has.length > 0) {
         for (const dep of selectorResult.has) {
             if (dep.has.length === 0) {
@@ -109,13 +117,13 @@ export function normalizeModel(pattern) {
     else if (isOneOrMore(pattern)) {
         model = pattern.oneOrMore;
         min = 1;
-        max = Math.max(pattern.max ?? Infinity, 1);
+        max = Math.max(pattern.max ?? Number.POSITIVE_INFINITY, 1);
         missingType = 'MISSING_NODE_ONE_OR_MORE';
     }
     else if (isZeroOrMore(pattern)) {
         model = pattern.zeroOrMore;
         min = 0;
-        max = Math.max(pattern.max ?? Infinity, 1);
+        max = Math.max(pattern.max ?? Number.POSITIVE_INFINITY, 1);
     }
     else {
         throw new Error('Unreachable code');
@@ -157,11 +165,11 @@ b) {
 }
 export function cleanObject(object) {
     const newObject = {};
-    Object.entries(object).forEach(([key, value]) => {
+    for (const [key, value] of Object.entries(object)) {
         if (value !== undefined) {
             newObject[key] = value;
         }
-    });
+    }
     return newObject;
 }
 export class Collection {
@@ -172,20 +180,20 @@ export class Collection {
         _Collection_matched.set(this, new Set());
         _Collection_nodes.set(this, void 0);
         _Collection_origin.set(this, void 0);
-        __classPrivateFieldSet(this, _Collection_origin, origin.slice(), "f");
+        __classPrivateFieldSet(this, _Collection_origin, [...origin], "f");
         __classPrivateFieldSet(this, _Collection_nodes, new Set(__classPrivateFieldGet(this, _Collection_origin, "f")), "f");
     }
     get matched() {
-        return Array.from(__classPrivateFieldGet(this, _Collection_matched, "f"));
+        return [...__classPrivateFieldGet(this, _Collection_matched, "f")];
     }
     get matchedCount() {
         return __classPrivateFieldGet(this, _Collection_matched, "f").size;
     }
     get nodes() {
-        return __classPrivateFieldGet(this, _Collection_origin, "f").slice();
+        return [...__classPrivateFieldGet(this, _Collection_origin, "f")];
     }
     get unmatched() {
-        return Array.from(__classPrivateFieldGet(this, _Collection_nodes, "f")).filter(n => !__classPrivateFieldGet(this, _Collection_matched, "f").has(n));
+        return [...__classPrivateFieldGet(this, _Collection_nodes, "f")].filter(n => !__classPrivateFieldGet(this, _Collection_matched, "f").has(n));
     }
     addMatched(
     // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
@@ -206,13 +214,14 @@ export class Collection {
         __classPrivateFieldSet(this, _Collection_locked, new Set(__classPrivateFieldGet(this, _Collection_matched, "f")), "f");
     }
     max(max) {
-        const sliced = Array.from(__classPrivateFieldGet(this, _Collection_matched, "f")).slice(max);
-        sliced.forEach(n => __classPrivateFieldGet(this, _Collection_matched, "f").delete(n));
+        const sliced = [...__classPrivateFieldGet(this, _Collection_matched, "f")].slice(max);
+        for (const n of sliced)
+            __classPrivateFieldGet(this, _Collection_matched, "f").delete(n);
     }
     toString(highlightExtraNodes = false) {
         const out = [];
         for (const n of __classPrivateFieldGet(this, _Collection_origin, "f")) {
-            const raw = n.is(n.TEXT_NODE) ? `:text(${n.raw.replace(/\n/g, '\\n')})` : n.raw;
+            const raw = n.is(n.TEXT_NODE) ? `:text(${n.raw.replaceAll('\n', '\\n')})` : n.raw;
             if (__classPrivateFieldGet(this, _Collection_locked, "f").has(n)) {
                 if (transparentMode.has(n)) {
                     out.push(bgBlue.bold(raw));

package/lib/placeholder-label-option/index.js

@@ -1,18 +1,18 @@
 import { createRule } from '@markuplint/ml-core';
 export default createRule({
     verify({ document, report, t }) {
-        document.querySelectorAll('select').forEach(select => {
+        for (const select of document.querySelectorAll('select')) {
             if (hasPlaceholderLabelOption(select)) {
-                return;
+                continue;
             }
             if (!needPlaceholderLabelOption(select)) {
-                return;
+                continue;
             }
             report({
                 scope: select,
                 message: t('need {0}', t('the {0}', 'placeholder label option')),
             });
-        });
+        }
     },
 });
 /**

package/lib/require-datetime/index.js

@@ -6,14 +6,14 @@ export default createRule({
         langs: undefined,
     },
     verify({ document, report, t }) {
-        document.querySelectorAll('time:not([datetime])').forEach(time => {
+        for (const time of document.querySelectorAll('time:not([datetime])')) {
             if (time.hasMutableChildren()) {
-                return;
+                continue;
             }
             const content = time.textContent.trim();
             const result = check(content, 'DateTime');
             if (result.matched) {
-                return;
+                continue;
             }
             const candidate = getCandidateDatetimeString(content, time.rule.options.langs);
             if (candidate) {
@@ -21,12 +21,12 @@ export default createRule({
                     scope: time,
                     message: t('need {0*}', `datetime="${candidate}"`),
                 });
-                return;
+                continue;
             }
             report({
                 scope: time,
                 message: t('need {0}', t('the "{0*}" {1}', 'datetime', 'attribute')),
             });
-        });
+        }
     },
 });

package/lib/require-datetime/utils.js

@@ -94,7 +94,7 @@ function parseTryMultipleLangs(content, langs, base) {
     for (const lang of langs) {
         const results = chrono[lang].casual.parse(content, base);
         // Is not multiple datetime contents
-        if (results.length < 1) {
+        if (results.length === 0) {
             continue;
         }
         const result = results[0];

package/lib/required-element/index.js

@@ -28,7 +28,7 @@ export default createRule({
                 return;
             }
             for (const query of el.rule.value) {
-                const exists = Array.from(el.children).find(child => child.matches(query));
+                const exists = [...el.children].find(child => child.matches(query));
                 if (!exists) {
                     const message = t('Require {0}', t('the "{0*}" {1}', query, 'element'));
                     report({

package/lib/use-list/index.js

@@ -76,7 +76,7 @@ export default createRule({
     },
 });
 function isMayListItem(text, bullets, spaceNeededBullets) {
-    const textArray = Array.from(text);
+    const textArray = [...text];
     const firstLetter = textArray[0] ?? '';
     const isBullet = bullets.includes(firstLetter);
     const needSpace = spaceNeededBullets.includes(firstLetter);

package/lib/wai-aria/checkings/implicit-props.js

@@ -34,13 +34,13 @@ export const checkingImplicitProps = ({ attr, propSpecs, attrSpecs }) => t => {
                 message: t('{0} contradicts {1}', t('the "{0*}" {1}', attr.name, `ARIA ${propSpec.type}`), t('the current "{0}" {1}', equivalentHtmlAttr.htmlAttrName, 'attribute')),
             };
         }
-        else if (value === 'true') {
-            if (!equivalentHtmlAttr.isNotStrictEquivalent && htmlAttrSpec?.type === 'Boolean') {
-                return {
-                    scope: attr,
-                    message: t('{0} contradicts {1}', t('the "{0*}" {1}', attr.name, `ARIA ${propSpec.type}`), t('the implicit "{0}" {1}', equivalentHtmlAttr.htmlAttrName, 'attribute')),
-                };
-            }
+        else if (value === 'true' &&
+            !equivalentHtmlAttr.isNotStrictEquivalent &&
+            htmlAttrSpec?.type === 'Boolean') {
+            return {
+                scope: attr,
+                message: t('{0} contradicts {1}', t('the "{0*}" {1}', attr.name, `ARIA ${propSpec.type}`), t('the implicit "{0}" {1}', equivalentHtmlAttr.htmlAttrName, 'attribute')),
+            };
         }
     }
 };

package/lib/wai-aria/checkings/presentational-children.js

@@ -33,7 +33,7 @@ export const checkingPresentationalChildren = ({ el }) => t => {
     if (!ancestor.role) {
         return;
     }
-    const hasAriaAttr = Array.from(el.attributes).some(attr => /^aria-|^role$/i.test(attr.name));
+    const hasAriaAttr = [...el.attributes].some(attr => /^aria-|^role$/i.test(attr.name));
     if (!hasAriaAttr) {
         return;
     }

package/lib/wai-aria/checkings/required-owned-elements.js

@@ -25,7 +25,7 @@ export const checkingRequiredOwnedElements = ({ el, role }) => t => {
         return;
     }
     // TODO: Needs to resolve `aria-own`
-    const children = Array.from(el.childNodes).map(child => {
+    const children = [...el.childNodes].map(child => {
         if (child.is(child.ELEMENT_NODE)) {
             if (child.matches('[aria-busy="true" i]')) {
                 return [child, 'BUSY'];
@@ -93,7 +93,7 @@ el) {
     if (el.isEmpty()) {
         return true;
     }
-    return Array.from(el.children).every(child => {
+    return [...el.children].every(child => {
         return ['script', 'template'].includes(child.localName);
     });
 }

package/lib/wai-aria/checkings/value.js

@@ -76,10 +76,10 @@ export function checkAriaValue(type, value, tokenEnum, booleanish) {
             return ['true', 'false', 'undefined'].includes(value);
         }
         case 'integer': {
-            return parseInt(value).toString() === value;
+            return Number.parseInt(value).toString() === value;
         }
         case 'number': {
-            return parseFloat(value).toString() === value;
+            return Number.parseFloat(value).toString() === value;
         }
     }
     // For skipping checking

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/rules",
-	"version": "4.0.0-alpha.3",
+	"version": "4.0.0-dev.28+0131de5e",
 	"description": "Built-in rules of markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
@@ -25,19 +25,18 @@
 		"./lib/permitted-contents/debug.js": "./lib/permitted-contents/debug.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/html-spec": "4.0.0-alpha.3",
-		"@markuplint/ml-core": "4.0.0-alpha.3",
-		"@markuplint/ml-spec": "4.0.0-alpha.3",
-		"@markuplint/selector": "4.0.0-alpha.3",
-		"@markuplint/shared": "4.0.0-alpha.3",
-		"@markuplint/types": "4.0.0-alpha.3",
-		"@types/debug": "^4.1.9",
+		"@markuplint/html-spec": "4.0.0-dev.28+0131de5e",
+		"@markuplint/ml-core": "4.0.0-dev.28+0131de5e",
+		"@markuplint/ml-spec": "4.0.0-dev.28+0131de5e",
+		"@markuplint/selector": "4.0.0-dev.28+0131de5e",
+		"@markuplint/shared": "4.0.0-dev.28+0131de5e",
+		"@markuplint/types": "4.0.0-dev.28+0131de5e",
+		"@types/debug": "^4.1.10",
 		"@ungap/structured-clone": "^1.2.0",
 		"ansi-colors": "^4.1.3",
 		"chrono-node": "^2.7.0",
 		"debug": "^4.3.4",
-		"tslib": "^2.6.2",
-		"type-fest": "^4.3.1"
+		"type-fest": "^4.5.0"
 	},
-	"gitHead": "380836f7adc1ff7e8eaf9d869e68d29eee8f3b7e"
+	"gitHead": "0131de5ea9dd6d3fd5472d7b414b66644c758881"
 }