@markuplint/rules 2.4.3 → 3.0.0-alpha.1

package/lib/doctype/index.js

@@ -4,16 +4,16 @@ const ml_core_1 = require("@markuplint/ml-core");
 exports.default = (0, ml_core_1.createRule)({
     defaultValue: 'always',
     defaultOptions: {
-        denyObsolateType: true,
+        denyObsoleteType: true,
     },
-    verify(context) {
-        if (context.document.isFragment) {
+    verify({ document, report, t }) {
+        if (document.isFragment) {
             return;
         }
-        const doctype = context.document.doctype;
+        const doctype = document.doctype;
         if (!doctype) {
-            context.report({
-                message: context.translate('Require {0}', 'doctype'),
+            report({
+                message: t('Require {0}', 'doctype'),
                 line: 1,
                 col: 1,
                 raw: '',
@@ -21,9 +21,9 @@ exports.default = (0, ml_core_1.createRule)({
             return;
         }
         if ((doctype.name.toLowerCase() === 'html' && doctype.publicId) || doctype.systemId) {
-            context.report({
+            report({
                 scope: doctype,
-                message: context.translate('Never {0} {1}', 'declarate', 'obsolete doctype'),
+                message: t('Never {0} {1}', 'declarate', 'obsolete doctype'),
             });
         }
     },

package/lib/end-tag/index.js

@@ -18,14 +18,17 @@ const voidElements = [
     'wbr',
 ];
 exports.default = (0, ml_core_1.createRule)({
-    defaultServerity: 'warning',
+    defaultSeverity: 'warning',
     async verify({ document, report, t }) {
         if (document.endTag === 'never') {
             return;
         }
         await document.walkOn('Element', el => {
             var _a;
-            if (voidElements.some(vE => vE === el.nodeName.toLowerCase())) {
+            if (el.isOmitted) {
+                return;
+            }
+            if (voidElements.some(vE => vE === el.localName)) {
                 return;
             }
             if (el.closeTag != null) {

package/lib/helpers.d.ts

@@ -1,7 +1,7 @@
 import type { Log } from './debug';
 import type { Translator } from '@markuplint/i18n';
 import type { Element, RuleConfigValue, Document } from '@markuplint/ml-core';
-import type { ARIRRoleAttribute, Attribute, MLMLSpec, PermittedRoles } from '@markuplint/ml-spec';
+import type { ARIAVersion, Attribute, MLMLSpec } from '@markuplint/ml-spec';
 export declare function attrMatches<T extends RuleConfigValue, R>(node: Element<T, R>, condition: Attribute['condition']): boolean;
 export declare function match(needle: string, pattern: string): boolean;
 /**
@@ -20,7 +20,6 @@ export declare function match(needle: string, pattern: string): boolean;
  * Originally, it is not possible to define a name including ASCII upper alphas in the custom element, but it is not treated as illegal by the HTML parser.
  */
 export declare const rePCENChar: string;
-export declare function htmlSpec(specs: Readonly<MLMLSpec>, nameWithNS: string): import("@markuplint/ml-spec").ElementSpec | null;
 export declare function isValidAttr(t: Translator, name: string, value: string, isDynamicValue: boolean, node: Element<any, any>, attrSpecs: Attribute[], log?: Log): false | {
     invalidType: "non-existent" | "invalid-value";
     message: string;
@@ -31,30 +30,6 @@ export declare function isValidAttr(t: Translator, name: string, value: string,
     } | undefined;
 };
 export declare function toNormalizedValue(value: string, spec: Attribute): string;
-export declare function ariaSpec(specs: Readonly<MLMLSpec>): {
-    roles: ARIRRoleAttribute[];
-    ariaAttrs: import("@markuplint/ml-spec").ARIAAttribute[];
-};
-export declare function getRoleSpec(specs: Readonly<MLMLSpec>, roleName: string): {
-    name: string;
-    isAbstract: boolean;
-    accessibleNameRequired: boolean;
-    statesAndProps: import("@markuplint/ml-spec").ARIARoleOwnedPropOrState[];
-    superClassRoles: ARIRRoleAttribute[];
-} | null;
-/**
- * Getting permitted ARIA roles.
- *
- * - If an array, it is role list.
- * - If `true`, this mean is "Any".
- * - If `false`, this mean is "No".
- */
-export declare function getPermittedRoles(specs: Readonly<MLMLSpec>, el: Element<any, any>): PermittedRoles;
-export declare function getImplicitRole(specs: Readonly<MLMLSpec>, el: Element<any, any>): string | false;
-export declare function getComputedRole(specs: Readonly<MLMLSpec>, el: Element<any, any>): {
-    name: string;
-    isImplicit: boolean;
-} | null;
 /**
  *
  * @see https://www.w3.org/TR/wai-aria-1.2/#propcharacteristic_value
@@ -64,7 +39,7 @@ export declare function getComputedRole(specs: Readonly<MLMLSpec>, el: Element<a
  * @param tokenEnum
  */
 export declare function checkAriaValue(type: string, value: string, tokenEnum: string[]): boolean;
-export declare function checkAria(specs: Readonly<MLMLSpec>, attrName: string, currentValue: string, role?: string): {
+export declare function checkAria(specs: Readonly<MLMLSpec>, attrName: string, currentValue: string, version: ARIAVersion, role?: string): {
     currentValue: string;
     isValid: boolean;
 } | {
@@ -85,5 +60,17 @@ export declare function checkAria(specs: Readonly<MLMLSpec>, attrName: string, c
     valueDescriptions?: Record<string, string> | undefined;
 };
 export declare function accnameMayBeMutable(el: Element<any, any>, document: Document<any, any>): boolean;
-export declare function getOwnedLabel<V extends RuleConfigValue, O>(el: Element<V, O>, docuemnt: Document<V, O>): import("packages/@markuplint/ml-core/lib/ml-dom/tokens/abstract-element").default<V, O, import("packages/@markuplint/ml-ast/src").MLASTElement | import("packages/@markuplint/ml-ast/src").MLASTOmittedElement> | null;
+export declare function getOwnedLabel<V extends RuleConfigValue, O>(el: Element<V, O>, document: Document<V, O>): Element<V, O> | null;
 export declare function decodeCharRef(characterReference: string): string;
+export declare class Collection<T> {
+    #private;
+    constructor(...items: (T | null | undefined)[]);
+    [Symbol.iterator](): Iterator<T>;
+    add(...items: (T | null | undefined)[]): void;
+    toArray(): readonly T[];
+}
+declare type Writeable<T> = {
+    -readonly [P in keyof T]: T[P];
+};
+export declare function deepCopy<T>(value: T): Writeable<T>;
+export {};

package/lib/helpers.js

@@ -1,33 +1,19 @@
 "use strict";
+var _Collection_items;
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.decodeCharRef = exports.getOwnedLabel = exports.accnameMayBeMutable = exports.checkAria = exports.checkAriaValue = exports.getComputedRole = exports.getImplicitRole = exports.getPermittedRoles = exports.getRoleSpec = exports.ariaSpec = exports.toNormalizedValue = exports.isValidAttr = exports.htmlSpec = exports.rePCENChar = exports.match = exports.attrMatches = void 0;
+exports.deepCopy = exports.Collection = exports.decodeCharRef = exports.getOwnedLabel = exports.accnameMayBeMutable = exports.checkAria = exports.checkAriaValue = exports.toNormalizedValue = exports.isValidAttr = exports.rePCENChar = exports.match = exports.attrMatches = void 0;
+const tslib_1 = require("tslib");
+const ml_core_1 = require("@markuplint/ml-core");
+// @ts-ignore
+const structured_clone_1 = tslib_1.__importDefault(require("@ungap/structured-clone"));
 const html_entities_1 = require("html-entities");
 const attr_check_1 = require("./attr-check");
 function attrMatches(node, condition) {
     if (!condition) {
         return true;
     }
-    let matched = false;
-    if ('self' in condition && condition.self) {
-        const condSelector = Array.isArray(condition.self) ? condition.self.join(',') : condition.self;
-        matched = node.matches(condSelector);
-    }
-    if ('ancestor' in condition && condition.ancestor) {
-        let _node = node.parentNode;
-        while (_node) {
-            if (_node.type === 'Element') {
-                const condSelector = Array.isArray(condition.ancestor)
-                    ? condition.ancestor.join(',')
-                    : condition.ancestor;
-                if (_node.matches(condSelector)) {
-                    matched = true;
-                    break;
-                }
-            }
-            _node = _node.parentNode;
-        }
-    }
-    return matched;
+    const condSelector = Array.isArray(condition) ? condition.join(',') : condition;
+    return node.matches(condSelector);
 }
 exports.attrMatches = attrMatches;
 function match(needle, pattern) {
@@ -75,11 +61,6 @@ exports.rePCENChar = [
     '[\uFDF0-\uFFFD]',
     '[\uD800-\uDBFF][\uDC00-\uDFFF]',
 ].join('|');
-function htmlSpec(specs, nameWithNS) {
-    const spec = specs.specs.find(spec => spec.name === nameWithNS);
-    return spec || null;
-}
-exports.htmlSpec = htmlSpec;
 function isValidAttr(t, name, value, isDynamicValue, node, attrSpecs, log) {
     let invalid = false;
     const spec = attrSpecs.find(s => s.name.toLowerCase() === name.toLowerCase());
@@ -126,126 +107,6 @@ function toNormalizedValue(value, spec) {
     return normalized;
 }
 exports.toNormalizedValue = toNormalizedValue;
-function ariaSpec(specs) {
-    const roles = specs.def['#roles'];
-    const ariaAttrs = specs.def['#ariaAttrs'];
-    return {
-        roles,
-        ariaAttrs,
-    };
-}
-exports.ariaSpec = ariaSpec;
-function getRoleSpec(specs, roleName) {
-    const role = getRoleByName(specs, roleName);
-    if (!role) {
-        return null;
-    }
-    const superClassRoles = recursiveTraverseSuperClassRoles(specs, roleName);
-    return {
-        name: role.name,
-        isAbstract: !!role.isAbstract,
-        accessibleNameRequired: role.accessibleNameRequired,
-        statesAndProps: role.ownedAttribute,
-        superClassRoles,
-    };
-}
-exports.getRoleSpec = getRoleSpec;
-function getRoleByName(specs, roleName) {
-    const roles = specs.def['#roles'];
-    const role = roles.find(r => r.name === roleName);
-    return role;
-}
-function getSuperClassRoles(specs, roleName) {
-    const role = getRoleByName(specs, roleName);
-    return ((role === null || role === void 0 ? void 0 : role.generalization.map(roleName => getRoleByName(specs, roleName)).filter((role) => !!role)) || null);
-}
-function recursiveTraverseSuperClassRoles(specs, roleName) {
-    const roles = [];
-    const superClassRoles = getSuperClassRoles(specs, roleName);
-    if (superClassRoles) {
-        roles.push(...superClassRoles);
-        for (const superClassRole of superClassRoles) {
-            const ancestorRoles = recursiveTraverseSuperClassRoles(specs, superClassRole.name);
-            roles.push(...ancestorRoles);
-        }
-    }
-    return roles;
-}
-/**
- * Getting permitted ARIA roles.
- *
- * - If an array, it is role list.
- * - If `true`, this mean is "Any".
- * - If `false`, this mean is "No".
- */
-function getPermittedRoles(specs, el) {
-    var _a;
-    const implicitRole = getImplicitRole(specs, el);
-    const spec = (_a = htmlSpec(specs, el.nodeName)) === null || _a === void 0 ? void 0 : _a.permittedRoles;
-    if (!spec) {
-        return true;
-    }
-    if (spec.conditions) {
-        for (const { condition, roles } of spec.conditions) {
-            if (el.matches(condition)) {
-                return mergeRoleList(implicitRole, roles);
-            }
-        }
-    }
-    if (implicitRole && Array.isArray(spec.roles)) {
-        return [implicitRole, ...spec.roles];
-    }
-    if (implicitRole && spec.roles === false) {
-        return [implicitRole];
-    }
-    return mergeRoleList(implicitRole, spec.roles);
-}
-exports.getPermittedRoles = getPermittedRoles;
-function mergeRoleList(implicitRole, permittedRoles) {
-    if (implicitRole && Array.isArray(permittedRoles)) {
-        return [implicitRole, ...permittedRoles];
-    }
-    if (implicitRole && permittedRoles === false) {
-        return [implicitRole];
-    }
-    return permittedRoles;
-}
-function getImplicitRole(specs, el) {
-    var _a;
-    const implicitRole = (_a = htmlSpec(specs, el.nodeName)) === null || _a === void 0 ? void 0 : _a.implicitRole;
-    if (!implicitRole) {
-        return false;
-    }
-    if (implicitRole.conditions) {
-        for (const { condition, role } of implicitRole.conditions) {
-            if (el.matches(condition)) {
-                return role;
-            }
-        }
-    }
-    return implicitRole.role;
-}
-exports.getImplicitRole = getImplicitRole;
-function getComputedRole(specs, el) {
-    const roleAttrTokens = el.getAttributeToken('role');
-    const roleAttr = roleAttrTokens[0];
-    if (roleAttr) {
-        const roleName = roleAttr.getValue().potential.trim().toLowerCase();
-        return {
-            name: roleName,
-            isImplicit: false,
-        };
-    }
-    const implicitRole = getImplicitRole(specs, el);
-    if (implicitRole) {
-        return {
-            name: implicitRole,
-            isImplicit: true,
-        };
-    }
-    return null;
-}
-exports.getComputedRole = getComputedRole;
 /**
  *
  * @see https://www.w3.org/TR/wai-aria-1.2/#propcharacteristic_value
@@ -288,8 +149,8 @@ function checkAriaValue(type, value, tokenEnum) {
     return true;
 }
 exports.checkAriaValue = checkAriaValue;
-function checkAria(specs, attrName, currentValue, role) {
-    const ariaAttrs = specs.def['#ariaAttrs'];
+function checkAria(specs, attrName, currentValue, version, role) {
+    const ariaAttrs = (0, ml_core_1.ariaSpecs)(specs, version).props;
     const aria = ariaAttrs.find(a => a.name === attrName);
     if (!aria) {
         return {
@@ -319,29 +180,54 @@ function accnameMayBeMutable(el, document) {
     if (el.hasMutableAttributes() || el.hasMutableChildren(true)) {
         return true;
     }
-    const ownedLable = getOwnedLabel(el, document);
-    if (ownedLable && (ownedLable.hasMutableAttributes() || ownedLable.hasMutableChildren(true))) {
+    const ownedLabel = getOwnedLabel(el, document);
+    if (ownedLabel && (ownedLabel.hasMutableAttributes() || ownedLabel.hasMutableChildren(true))) {
         return true;
     }
     return false;
 }
 exports.accnameMayBeMutable = accnameMayBeMutable;
 const labelable = ['button', 'input:not([type=hidden])', 'meter', 'output', 'progress', 'select', 'textarea'];
-function getOwnedLabel(el, docuemnt) {
+function getOwnedLabel(el, document) {
     if (!labelable.some(cond => el.matches(cond))) {
         return null;
     }
-    let ownedLable = el.closest('label');
-    if (!ownedLable) {
+    let ownedLabel = el.closest('label');
+    if (!ownedLabel) {
         const id = el.getAttribute('id');
         if (id) {
-            ownedLable = docuemnt.querySelector(`label[for="${id}"]`);
+            ownedLabel = document.querySelector(`label[for="${id}"]`);
         }
     }
-    return ownedLable;
+    return ownedLabel;
 }
 exports.getOwnedLabel = getOwnedLabel;
 function decodeCharRef(characterReference) {
     return (0, html_entities_1.decode)(characterReference);
 }
 exports.decodeCharRef = decodeCharRef;
+class Collection {
+    constructor(...items) {
+        _Collection_items.set(this, new Set());
+        this.add(...items);
+    }
+    [(_Collection_items = new WeakMap(), Symbol.iterator)]() {
+        return tslib_1.__classPrivateFieldGet(this, _Collection_items, "f").values();
+    }
+    add(...items) {
+        for (const item of items) {
+            if (item == null) {
+                continue;
+            }
+            tslib_1.__classPrivateFieldGet(this, _Collection_items, "f").add(item);
+        }
+    }
+    toArray() {
+        return Object.freeze(Array.from(tslib_1.__classPrivateFieldGet(this, _Collection_items, "f")));
+    }
+}
+exports.Collection = Collection;
+function deepCopy(value) {
+    return (0, structured_clone_1.default)(value);
+}
+exports.deepCopy = deepCopy;

package/lib/id-duplication/index.js

@@ -8,13 +8,11 @@ exports.default = (0, ml_core_1.createRule)({
         await document.walkOn('Element', node => {
             const idAttrs = node.getAttributeToken('id');
             for (const idAttr of idAttrs) {
-                if (idAttr.attrType === 'ps-attr' ||
-                    (idAttr.attrType === 'html-attr' && idAttr.isDynamicValue) ||
-                    (idAttr.attrType === 'html-attr' && idAttr.isDirective)) {
+                if (idAttr.isDynamicValue || idAttr.isDirective) {
                     continue;
                 }
-                const id = idAttr.getValue();
-                if (idStack.includes(id.raw)) {
+                const id = idAttr.value;
+                if (idStack.includes(id)) {
                     report({
                         scope: node,
                         message,
@@ -23,7 +21,7 @@ exports.default = (0, ml_core_1.createRule)({
                         raw: idAttr.raw,
                     });
                 }
-                idStack.push(id.raw);
+                idStack.push(id);
             }
         });
     },

package/lib/index.d.ts

@@ -1,10 +1,7 @@
 declare const _default: {
     'attr-duplication': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
-    'attr-equal-space-after': import("packages/@markuplint/ml-core/lib").RuleSeed<"always" | "never" | "always-single-line" | "never-single-line", null>;
-    'attr-equal-space-before': import("packages/@markuplint/ml-core/lib").RuleSeed<"always" | "never" | "always-single-line" | "never-single-line", null>;
-    'attr-spacing': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, import("./attr-spacing").AttrSpasingOptions>;
     'attr-value-quotes': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./attr-value-quotes").Type, null>;
-    'case-sensitive-attr-name': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./case-sensitive-attr-name").Value | import("./case-sensitive-attr-name").Value_v1, null>;
+    'case-sensitive-attr-name': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./case-sensitive-attr-name").Value, null>;
     'case-sensitive-tag-name': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./case-sensitive-tag-name").Value, null>;
     'character-reference': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
     'class-naming': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./class-naming").Value, null>;
@@ -12,11 +9,10 @@ declare const _default: {
     'deprecated-element': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
     'disallowed-element': import("packages/@markuplint/ml-core/lib").RuleSeed<string[], null>;
     doctype: import("packages/@markuplint/ml-core/lib").RuleSeed<"always", {
-        denyObsolateType: boolean;
+        denyObsoleteType: boolean;
     }>;
     'end-tag': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, null>;
     'id-duplication': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
-    indentation: import("packages/@markuplint/ml-core/lib").RuleSeed<import("./indentation").Value, import("./indentation").IndentationOptions>;
     'ineffective-attr': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
     'invalid-attr': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, {
         attrs?: Record<string, {
@@ -31,38 +27,33 @@ declare const _default: {
         ignoreAttrNamePrefix?: string | string[] | undefined;
     }>;
     'landmark-roles': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, {
-        ignoreRoles?: (("banner" | "main" | "complementary" | "contentinfo") | "form" | "navigation" | "region")[] | undefined;
-        labelEachArea?: boolean | undefined;
+        ignoreRoles: (("banner" | "main" | "complementary" | "contentinfo") | "form" | "navigation" | "region")[];
+        labelEachArea: boolean;
     }>;
     'no-boolean-attr-value': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
     'no-default-value': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
     'no-hard-code-id': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
-    'no-refer-to-non-existent-id': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, null>;
+    'no-refer-to-non-existent-id': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-config/src").RuleConfigValue, {
+        ariaVersion: import("packages/@markuplint/ml-spec/lib").ARIAVersion;
+    }>;
     'no-use-event-handler-attr': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, {
         ignore?: string | string[] | undefined;
     }>;
-    'permitted-contents': import("packages/@markuplint/ml-core/lib").RuleSeed<import("packages/@markuplint/ml-spec/lib").PermittedStructuresSchema[], {
-        ignoreHasMutableChildren: boolean;
+    'permitted-contents': import("packages/@markuplint/ml-core/lib").RuleSeed<import("./permitted-contents/types").TagRule[], import("./permitted-contents/types").Options>;
+    'require-accessible-name': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, {
+        ariaVersion: import("packages/@markuplint/ml-spec/lib").ARIAVersion;
     }>;
-    'require-accessible-name': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, null>;
     'required-attr': import("packages/@markuplint/ml-core/lib").RuleSeed<string | (string | {
         name: string;
         value: string | string[];
     })[], null>;
     'required-element': import("packages/@markuplint/ml-core/lib").RuleSeed<string[], {
-        ignoreHasMutableContents?: boolean | undefined;
+        ignoreHasMutableContents: boolean;
     }>;
     'required-h1': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, import("./required-h1").Options>;
     'use-list': import("packages/@markuplint/ml-core/lib").RuleSeed<string[], {
         spaceNeededBullets?: string[] | undefined;
     }>;
-    'wai-aria': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, {
-        checkingValue?: boolean | undefined;
-        checkingDeprecatedProps?: boolean | undefined;
-        permittedAriaRoles?: boolean | undefined;
-        disallowSetImplicitRole?: boolean | undefined;
-        disallowSetImplicitProps?: boolean | undefined;
-        disallowDefaultValue?: boolean | undefined;
-    }>;
+    'wai-aria': import("packages/@markuplint/ml-core/lib").RuleSeed<boolean, import("./wai-aria/types").Options>;
 };
 export default _default;

package/lib/index.js

@@ -2,9 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const tslib_1 = require("tslib");
 const attr_duplication_1 = tslib_1.__importDefault(require("./attr-duplication"));
-const attr_equal_space_after_1 = tslib_1.__importDefault(require("./attr-equal-space-after"));
-const attr_equal_space_before_1 = tslib_1.__importDefault(require("./attr-equal-space-before"));
-const attr_spacing_1 = tslib_1.__importDefault(require("./attr-spacing"));
 const attr_value_quotes_1 = tslib_1.__importDefault(require("./attr-value-quotes"));
 const case_sensitive_attr_name_1 = tslib_1.__importDefault(require("./case-sensitive-attr-name"));
 const case_sensitive_tag_name_1 = tslib_1.__importDefault(require("./case-sensitive-tag-name"));
@@ -16,7 +13,6 @@ const disallowed_element_1 = tslib_1.__importDefault(require("./disallowed-eleme
 const doctype_1 = tslib_1.__importDefault(require("./doctype"));
 const end_tag_1 = tslib_1.__importDefault(require("./end-tag"));
 const id_duplication_1 = tslib_1.__importDefault(require("./id-duplication"));
-const indentation_1 = tslib_1.__importDefault(require("./indentation"));
 const ineffective_attr_1 = tslib_1.__importDefault(require("./ineffective-attr"));
 const invalid_attr_1 = tslib_1.__importDefault(require("./invalid-attr"));
 const landmark_roles_1 = tslib_1.__importDefault(require("./landmark-roles"));
@@ -34,9 +30,6 @@ const use_list_1 = tslib_1.__importDefault(require("./use-list"));
 const wai_aria_1 = tslib_1.__importDefault(require("./wai-aria"));
 exports.default = {
     'attr-duplication': attr_duplication_1.default,
-    'attr-equal-space-after': attr_equal_space_after_1.default,
-    'attr-equal-space-before': attr_equal_space_before_1.default,
-    'attr-spacing': attr_spacing_1.default,
     'attr-value-quotes': attr_value_quotes_1.default,
     'case-sensitive-attr-name': case_sensitive_attr_name_1.default,
     'case-sensitive-tag-name': case_sensitive_tag_name_1.default,
@@ -48,7 +41,6 @@ exports.default = {
     doctype: doctype_1.default,
     'end-tag': end_tag_1.default,
     'id-duplication': id_duplication_1.default,
-    indentation: indentation_1.default,
     'ineffective-attr': ineffective_attr_1.default,
     'invalid-attr': invalid_attr_1.default,
     'landmark-roles': landmark_roles_1.default,

package/lib/ineffective-attr/index.js

@@ -2,38 +2,30 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const ml_core_1 = require("@markuplint/ml-core");
 exports.default = (0, ml_core_1.createRule)({
-    defaultServerity: 'warning',
+    defaultSeverity: 'warning',
     async verify({ document, report, t }) {
-        await document.walkOn('Element', el => {
-            const attrSpec = (0, ml_core_1.getAttrSpecs)(el.nameWithNS, document.specs);
+        await document.walkOn('Attr', attr => {
+            const attrSpec = (0, ml_core_1.getAttrSpecs)(attr.ownerElement, document.specs);
             if (!attrSpec) {
                 return;
             }
-            for (const attr of el.attributes) {
-                if (attr.attrType === 'ps-attr') {
-                    continue;
-                }
-                const name = attr.getName().potential;
-                const spec = attrSpec.find(s => s.name === name);
-                if (!spec) {
-                    continue;
-                }
-                const ineffectiveConditions = spec.ineffective
-                    ? Array.isArray(spec.ineffective)
-                        ? spec.ineffective
-                        : [spec.ineffective]
-                    : [];
-                if (ineffectiveConditions.some(selector => el.matches(selector))) {
-                    report({
-                        scope: el,
-                        raw: attr.raw,
-                        line: attr.startLine,
-                        col: attr.startCol,
-                        message: t('{0} is {1:c}', t('the "{0*}" {1}', name, 'attribute'), 'ineffective') +
-                            t('. ') +
-                            t("It doesn't need {0}", t('the {0}', 'attribute')),
-                    });
-                }
+            const name = attr.name;
+            const spec = attrSpec.find(s => s.name === name);
+            if (!spec) {
+                return;
+            }
+            const ineffectiveConditions = spec.ineffective
+                ? Array.isArray(spec.ineffective)
+                    ? spec.ineffective
+                    : [spec.ineffective]
+                : [];
+            if (ineffectiveConditions.some(selector => attr.ownerElement.matches(selector))) {
+                report({
+                    scope: attr,
+                    message: t('{0} is {1:c}', t('the "{0*}" {1}', name, 'attribute'), 'ineffective') +
+                        t('. ') +
+                        t("It doesn't need {0}", t('the {0}', 'attribute')),
+                });
             }
         });
     },