@markuplint/rules 2.0.0-rc.3 → 2.1.0

package/README.md

@@ -8,86 +8,54 @@
 
 ### Conformance checking
 
-| Avaiable | Docs | Docs(ja) | Rule                                              | Fixable |
-| -------- | ---- | -------- | ------------------------------------------------- | ------- |
-| ✅       | ✅   | ✅       | [attr-duplication](./src/attr-duplication/)       | 🚧      |
-| ✅       | ✅   | ✅       | [character-reference](./src/character-reference/) | 🚧      |
-| 🚧       | 🚧   | 🚧       | [comment](./src/comment/)                         |         |
-| ✅       | ✅   | ✅       | [deprecated-attr](./src/deprecated-attr/)         |         |
-| ✅       | ✅   | ✅       | [deprecated-element](./src/deprecated-element/)   |         |
-| ✅       | ✅   | ✅       | [doctype](./src/doctype/)                         | 🚧      |
-| ✅       | ✅   | ✅       | [id-duplication](./src/id-duplication/)           |         |
-| ✅       | ✅   | ✅       | [permitted-contents](./src/permitted-contents/)   |         |
-| ✅       | ✅   | ✅       | [required-attr](./src/required-attr/)             |         |
-| ✅       | ✅   | ✅       | [invalid-attr](./src/invalid-attr/)               |         |
+- [`attr-duplication`](./src/attr-duplication)
+- [`character-reference`](./src/character-reference)
+- [`deprecated-attr`](./src/deprecated-attr)
+- [`deprecated-element`](./src/deprecated-element)
+- [`disallowed-element`](./src/disallowed-element)
+- [`doctype`](./src/doctype)
+- [`id-duplication`](./src/id-duplication)
+- [`ineffective-attr`](./src/ineffective-attr)
+- [`invalid-attr`](./src/invalid-attr)
+- [`permitted-contents`](./src/permitted-contents)
+- [`required-attr`](./src/required-attr)
+- [`required-element`](./src/required-element)
 
 ### Accessibility
 
-| Avaiable | Docs | Docs(ja) | Rule                                                                  | Fixable |
-| -------- | ---- | -------- | --------------------------------------------------------------------- | ------- |
-| 🚧       | 🚧   | 🚧       | [accessible-text](./src/accessible-text/)                             |         |
-| 🚧       | 🚧   | 🚧       | [attr-role](./src/permitted-role/)                                    |         |
-| 🚧       | 🚧   | 🚧       | [heading-in-sectioning-content](./src/heading-in-sectioning-content/) |         |
-| 🚧       | 🚧   | 🚧       | [heading-in-sectioning-root](./src/heading-in-sectioning-root/)       |         |
-| 🚧       | 🚧   | 🚧       | [heading-levels-skipping](./src/heading-levels-skipping/)             |         |
-| 🚧       | 🚧   | 🚧       | [labeling-controls](./src/labeling-controls/)                         |         |
-| ✅       | ✅   | ✅       | [landmark-roles](./src/landmark-roles/)                               |         |
-| ✅       | ✅   | ✅       | [required-h1](./src/required-h1/)                                     |         |
+- [`landmark-roles`](./src/landmark-roles)
+- [`no-refer-to-non-existent-id`](./src/no-refer-to-non-existent-id)
+- [`required-h1`](./src/required-h1)
+- [`wai-aria`](./src/wai-aria)
 
-### Usability
+### Naming Convention
 
-| Avaiable | Docs | Docs(ja) | Rule                                  | Fixable |
-| -------- | ---- | -------- | ------------------------------------- | ------- |
-| 🚧       | 🚧   | 🚧       | [external-link](./src/external-link/) | 🚧      |
+- [`class-naming`](./src/class-naming)
 
-### Structure Design and Naming Convention
+### Maintenability
 
-| Avaiable | Docs | Docs(ja) | Rule                                                  | Fixable |
-| -------- | ---- | -------- | ----------------------------------------------------- | ------- |
-| ✅       | ✅   | ✅       | [class-naming](./src/class-naming/)                   |         |
-| 🚧       | 🚧   | 🚧       | [custom-element-naming](./src/custom-element-naming/) |         |
-| 🚧       | 🚧   | 🚧       | [data-attr-naming](./src/data-attr-naming/)           |         |
-| 🚧       | 🚧   | 🚧       | [required-element](./src/required-element/)           |         |
+- [`no-hard-code-id`](./src/no-hard-code-id)
+- [`no-use-event-handler-attr`](./src/no-use-event-handler-attr)
 
 ### Style
 
-| Avaiable | Docs | Docs(ja) | Rule                                                        | Fixable |
-| -------- | ---- | -------- | ----------------------------------------------------------- | ------- |
-| ✅       | ✅   | ✅       | [attr-equal-space-after](./src/attr-equal-space-after/)     | ✅      |
-| ✅       | ✅   | ✅       | [attr-equal-space-before](./src/attr-equal-space-before/)   | ✅      |
-| ✅       | ✅   | ✅       | [attr-spacing](./src/attr-spacing/)                         | ✅      |
-| ✅       | ✅   | ✅       | [attr-value-quotes](./src/attr-value-quotes/)               | ✅      |
-| ✅       | ✅   | ✅       | [case-sensitive-attr-name](./src/case-sensitive-attr-name/) | ✅      |
-| ✅       | ✅   | ✅       | [case-sensitive-tag-name](./src/case-sensitive-tag-name/)   | ✅      |
-| 🚧       | 🚧   | 🚧       | [comment-spasing](./src/comment-spasing/)                   | 🚧      |
-| 🚧       | 🚧   | 🚧       | [event-attr](./src/event-attr/)                             |         |
-| ✅       | ✅   | ✅       | [indentation](./src/indentation/)                           | ✅      |
-| 🚧       | 🚧   | 🚧       | [indentation-attr](./src/indentation-attr/)                 | 🚧      |
-| 🚧       | 🚧   | 🚧       | [path](./src/path/)                                         |         |
-| 🚧       | 🚧   | ✅       | [self-closing-tag](./src/self-closing-tag/)                 | 🚧      |
-| 🚧       | 🚧   | 🚧       | [tag-omission](./src/tag-omission/)                         | 🚧      |
+- [`attr-equal-space-after`](./src/attr-equal-space-after)
+- [`attr-equal-space-before`](./src/attr-equal-space-before)
+- [`attr-spacing`](./src/attr-spacing)
+- [`attr-value-quotes`](./src/attr-value-quotes)
+- [`case-sensitive-attr-name`](./src/case-sensitive-attr-name)
+- [`case-sensitive-tag-name`](./src/case-sensitive-tag-name)
+- [`end-tag`](./src/end-tag)
+- [`indentation`](./src/indentation)
+- [`no-boolean-attr-value`](./src/no-boolean-attr-value)
+- [`no-default-value`](./src/no-default-value)
 
 ## Install
 
-This package is **default** used that dependenced by [`markuplint`](https://www.npmjs.com/package/markuplint/).
-
-Prerequisites: [Node.js](https://nodejs.org) (Version 12.4.0 or later)
+Generally, you **don't have to install** this package because markuplint dependents it defaultly.
 
 ```sh
-$ npm install @markuplint/rules
-
-$ yarn add @markuplint/rules
-```
-
-## Contributing
+$ npm install -D @markuplint/rules
 
+$ yarn add -D @markuplint/rules
 ```
-$ git clone git@github.com:markuplint/markuplint.git -b main
-$ yarn
-$ yarn build
-$ yarn test
-```
-
----
-
-Copyright © 2021 markuplint. Under the MIT License.

package/lib/attr-check.js

@@ -28,7 +28,7 @@ function attrCheck(t, name, value, isCustomRule, spec) {
         (0, debug_1.log)('The "%s" attribute DOES\'NT EXIST in the spec', name);
         return {
             invalidType: 'non-existent',
-            message: t('{0} is {1:c}', t('the "{0}" {1}', name, 'attribute'), 'disallowed'),
+            message: t('{0} is {1:c}', t('the "{0*}" {1}', name, 'attribute'), 'disallowed'),
         };
     }
     const nameCaseSensitive = /[A-Z]/.test(spec.name);
@@ -36,7 +36,7 @@ function attrCheck(t, name, value, isCustomRule, spec) {
         (0, debug_1.log)('The "%s" attribute name is unmatched in case-sensitive', name);
         return {
             invalidType: 'non-existent',
-            message: t('{0} is {1:c}', t('the "{0}" {1}', name, 'attribute'), 'disallowed') +
+            message: t('{0} is {1:c}', t('the "{0*}" {1}', name, 'attribute'), 'disallowed') +
                 t('. ') +
                 t('Did you mean "{0*}"?', spec.name),
         };

package/lib/case-sensitive-attr-name/index.d.ts

@@ -1,3 +1,7 @@
-export declare type Value = 'no-upper' | 'no-lower';
-declare const _default: import("@markuplint/ml-core").RuleSeed<Value, null>;
+export declare type Value = 'lower' | 'upper';
+/**
+ * @deprecated
+ */
+export declare type Value_v1 = 'no-upper' | 'no-lower';
+declare const _default: import("@markuplint/ml-core").RuleSeed<Value | Value_v1, null>;
 export default _default;

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

@@ -3,15 +3,27 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const ml_core_1 = require("@markuplint/ml-core");
 exports.default = (0, ml_core_1.createRule)({
     defaultServerity: 'warning',
-    defaultValue: 'no-upper',
+    defaultValue: 'lower',
     async verify({ document, report, t }) {
         await document.walkOn('Element', async (node) => {
             if (node.isForeignElement || node.isCustomElement) {
                 return;
             }
+            let value;
+            // Convert deprecated value
+            switch (node.rule.value) {
+                case 'no-lower':
+                    value = 'upper';
+                    break;
+                case 'no-upper':
+                    value = 'lower';
+                    break;
+                default:
+                    value = node.rule.value;
+            }
             const ms = node.rule.severity === 'error' ? 'must' : 'should';
-            const deny = node.rule.value === 'no-upper' ? /[A-Z]/ : /[a-z]/;
-            const cases = node.rule.value === 'no-upper' ? 'lower' : 'upper';
+            const deny = value === 'lower' ? /[A-Z]/ : /[a-z]/;
+            const cases = value === 'lower' ? 'lower' : 'upper';
             const message = t(`{0} ${ms} be {1}`, t('{0} of {1}', 'attribute names', 'HTML elements'), `${cases}case`);
             const attrSpecs = (0, ml_core_1.getAttrSpecs)(node.nameWithNS, document.specs);
             for (const attr of node.attributes) {
@@ -50,6 +62,18 @@ exports.default = (0, ml_core_1.createRule)({
                 return;
             }
             const attrSpecs = (0, ml_core_1.getAttrSpecs)(node.nameWithNS, document.specs);
+            let value;
+            // Convert deprecated value
+            switch (node.rule.value) {
+                case 'no-lower':
+                    value = 'upper';
+                    break;
+                case 'no-upper':
+                    value = 'lower';
+                    break;
+                default:
+                    value = node.rule.value;
+            }
             if (node.attributes) {
                 for (const attr of node.attributes) {
                     if (attr.attrType === 'ps-attr') {
@@ -69,7 +93,7 @@ exports.default = (0, ml_core_1.createRule)({
                             continue;
                         }
                     }
-                    if (node.rule.value === 'no-upper') {
+                    if (value === 'lower') {
                         attr.name.fix(attr.name.raw.toLowerCase());
                     }
                     else {

package/lib/class-naming/index.js

@@ -25,7 +25,7 @@ exports.default = (0, ml_core_1.createRule)({
                     if (!classPatterns.some(pattern => (0, helpers_1.match)(className, pattern))) {
                         report({
                             scope: node,
-                            message: t('{0} is unmatched with the below patterns: {1}', t('the "{0}" {1}', className, 'class name'), `"${classPatterns.join('", "')}"`),
+                            message: t('{0} is unmatched with the below patterns: {1}', t('the "{0*}" {1}', className, 'class name'), `"${classPatterns.join('", "')}"`),
                             line: classAttr.line,
                             col: classAttr.col,
                             raw: classAttr.raw,

package/lib/deprecated-attr/index.js

@@ -15,7 +15,7 @@ exports.default = (0, ml_core_1.createRule)({
                     return;
                 }
                 if (attrSpec.deprecated || attrSpec.obsolete) {
-                    const message = t('{0} is {1:c}', t('the "{0}" {1}', name.potential, 'attribute'), attrSpec.obsolete ? 'obsolete' : 'deprecated');
+                    const message = t('{0} is {1:c}', t('the "{0*}" {1}', name.potential, 'attribute'), attrSpec.obsolete ? 'obsolete' : 'deprecated');
                     report({
                         scope: element,
                         message,

package/lib/deprecated-element/index.js

@@ -13,7 +13,7 @@ exports.default = (0, ml_core_1.createRule)({
             }
             const spec = (0, ml_spec_1.getSpecByTagName)(el.nameWithNS, html_spec_1.default);
             if (spec && (spec.obsolete || spec.deprecated || spec.nonStandard)) {
-                const message = t('{0} is {1:c}', t('the "{0}" {1}', el.nodeName, 'element'), spec.deprecated ? 'deprecated' : spec.obsolete ? 'obsolete' : 'non-standard');
+                const message = t('{0} is {1:c}', t('the "{0*}" {1}', el.nodeName, 'element'), spec.deprecated ? 'deprecated' : spec.obsolete ? 'obsolete' : 'non-standard');
                 report({
                     scope: el,
                     message,

package/lib/end-tag/index.d.ts

@@ -0,0 +1,2 @@
+declare const _default: import("@markuplint/ml-core").RuleSeed<boolean, null>;
+export default _default;

package/lib/end-tag/index.js

@@ -0,0 +1,43 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const ml_core_1 = require("@markuplint/ml-core");
+const voidElements = [
+    'area',
+    'base',
+    'br',
+    'col',
+    'embed',
+    'hr',
+    'img',
+    'input',
+    'link',
+    'meta',
+    'param',
+    'source',
+    'track',
+    'wbr',
+];
+exports.default = (0, ml_core_1.createRule)({
+    defaultServerity: '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())) {
+                return;
+            }
+            if (el.closeTag != null) {
+                return;
+            }
+            if (document.endTag === 'xml' && ((_a = el.selfClosingSolidus) === null || _a === void 0 ? void 0 : _a.raw)) {
+                return;
+            }
+            report({
+                scope: el,
+                message: t('Missing {0}', t('the {0}', 'end tag')),
+            });
+        });
+    },
+});

package/lib/helpers.d.ts

@@ -1,7 +1,7 @@
 import type { Log } from './debug';
 import type { Translator } from '@markuplint/i18n';
 import type { Element, RuleConfigValue } from '@markuplint/ml-core';
-import type { ARIRRoleAttribute, Attribute, PermittedRoles } from '@markuplint/ml-spec';
+import type { ARIRRoleAttribute, Attribute, MLMLSpec, PermittedRoles } 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,7 @@ 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(nameWithNS: string): import("@markuplint/ml-spec").ElementSpec | null;
+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,11 +31,11 @@ export declare function isValidAttr(t: Translator, name: string, value: string,
     } | undefined;
 };
 export declare function toNormalizedValue(value: string, spec: Attribute): string;
-export declare function ariaSpec(): {
+export declare function ariaSpec(specs: Readonly<MLMLSpec>): {
     roles: ARIRRoleAttribute[];
     ariaAttrs: import("@markuplint/ml-spec").ARIAAttribute[];
 };
-export declare function getRoleSpec(roleName: string): {
+export declare function getRoleSpec(specs: Readonly<MLMLSpec>, roleName: string): {
     name: string;
     isAbstract: boolean;
     statesAndProps: import("@markuplint/ml-spec").ARIARoleOwnedPropOrState[];
@@ -48,9 +48,9 @@ export declare function getRoleSpec(roleName: string): {
  * - If `true`, this mean is "Any".
  * - If `false`, this mean is "No".
  */
-export declare function getPermittedRoles(el: Element<any, any>): PermittedRoles;
-export declare function getImplicitRole(el: Element<any, any>): string | false;
-export declare function getComputedRole(el: Element<any, any>): {
+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;
@@ -63,7 +63,7 @@ export declare function getComputedRole(el: Element<any, any>): {
  * @param tokenEnum
  */
 export declare function checkAriaValue(type: string, value: string, tokenEnum: string[]): boolean;
-export declare function checkAria(attrName: string, currentValue: string, role?: string): {
+export declare function checkAria(specs: Readonly<MLMLSpec>, attrName: string, currentValue: string, role?: string): {
     currentValue: string;
     isValid: boolean;
 } | {

package/lib/helpers.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 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;
-const html_spec_1 = require("@markuplint/html-spec");
 const attr_check_1 = require("./attr-check");
 function attrMatches(node, condition) {
     if (!condition) {
@@ -75,8 +74,8 @@ exports.rePCENChar = [
     '[\uFDF0-\uFFFD]',
     '[\uD800-\uDBFF][\uDC00-\uDFFF]',
 ].join('|');
-function htmlSpec(nameWithNS) {
-    const spec = html_spec_1.specs.find(spec => spec.name === nameWithNS);
+function htmlSpec(specs, nameWithNS) {
+    const spec = specs.specs.find(spec => spec.name === nameWithNS);
     return spec || null;
 }
 exports.htmlSpec = htmlSpec;
@@ -88,7 +87,7 @@ function isValidAttr(t, name, value, isDynamicValue, node, attrSpecs, log) {
     if (!invalid && spec && spec.condition && !node.hasSpreadAttr && !attrMatches(node, spec.condition)) {
         invalid = {
             invalidType: 'non-existent',
-            message: t('{0} is {1}', t('the "{0}" {1}', name, 'attribute'), 'disallowed'),
+            message: t('{0} is {1}', t('the "{0*}" {1}', name, 'attribute'), 'disallowed'),
         };
     }
     if (invalid && invalid.invalidType === 'invalid-value' && isDynamicValue) {
@@ -126,21 +125,21 @@ function toNormalizedValue(value, spec) {
     return normalized;
 }
 exports.toNormalizedValue = toNormalizedValue;
-function ariaSpec() {
-    const roles = html_spec_1.def['#roles'];
-    const ariaAttrs = html_spec_1.def['#ariaAttrs'];
+function ariaSpec(specs) {
+    const roles = specs.def['#roles'];
+    const ariaAttrs = specs.def['#ariaAttrs'];
     return {
         roles,
         ariaAttrs,
     };
 }
 exports.ariaSpec = ariaSpec;
-function getRoleSpec(roleName) {
-    const role = getRoleByName(roleName);
+function getRoleSpec(specs, roleName) {
+    const role = getRoleByName(specs, roleName);
     if (!role) {
         return null;
     }
-    const superClassRoles = recursiveTraverseSuperClassRoles(roleName);
+    const superClassRoles = recursiveTraverseSuperClassRoles(specs, roleName);
     return {
         name: role.name,
         isAbstract: !!role.isAbstract,
@@ -149,22 +148,22 @@ function getRoleSpec(roleName) {
     };
 }
 exports.getRoleSpec = getRoleSpec;
-function getRoleByName(roleName) {
-    const roles = html_spec_1.def['#roles'];
+function getRoleByName(specs, roleName) {
+    const roles = specs.def['#roles'];
     const role = roles.find(r => r.name === roleName);
     return role;
 }
-function getSuperClassRoles(roleName) {
-    const role = getRoleByName(roleName);
-    return ((role === null || role === void 0 ? void 0 : role.generalization.map(roleName => getRoleByName(roleName)).filter((role) => !!role)) || null);
+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(roleName) {
+function recursiveTraverseSuperClassRoles(specs, roleName) {
     const roles = [];
-    const superClassRoles = getSuperClassRoles(roleName);
+    const superClassRoles = getSuperClassRoles(specs, roleName);
     if (superClassRoles) {
         roles.push(...superClassRoles);
         for (const superClassRole of superClassRoles) {
-            const ancestorRoles = recursiveTraverseSuperClassRoles(superClassRole.name);
+            const ancestorRoles = recursiveTraverseSuperClassRoles(specs, superClassRole.name);
             roles.push(...ancestorRoles);
         }
     }
@@ -177,10 +176,10 @@ function recursiveTraverseSuperClassRoles(roleName) {
  * - If `true`, this mean is "Any".
  * - If `false`, this mean is "No".
  */
-function getPermittedRoles(el) {
+function getPermittedRoles(specs, el) {
     var _a;
-    const implicitRole = getImplicitRole(el);
-    const spec = (_a = htmlSpec(el.nodeName)) === null || _a === void 0 ? void 0 : _a.permittedRoles;
+    const implicitRole = getImplicitRole(specs, el);
+    const spec = (_a = htmlSpec(specs, el.nodeName)) === null || _a === void 0 ? void 0 : _a.permittedRoles;
     if (!spec) {
         return true;
     }
@@ -209,9 +208,9 @@ function mergeRoleList(implicitRole, permittedRoles) {
     }
     return permittedRoles;
 }
-function getImplicitRole(el) {
+function getImplicitRole(specs, el) {
     var _a;
-    const implicitRole = (_a = htmlSpec(el.nodeName)) === null || _a === void 0 ? void 0 : _a.implicitRole;
+    const implicitRole = (_a = htmlSpec(specs, el.nodeName)) === null || _a === void 0 ? void 0 : _a.implicitRole;
     if (!implicitRole) {
         return false;
     }
@@ -225,7 +224,7 @@ function getImplicitRole(el) {
     return implicitRole.role;
 }
 exports.getImplicitRole = getImplicitRole;
-function getComputedRole(el) {
+function getComputedRole(specs, el) {
     const roleAttrTokens = el.getAttributeToken('role');
     const roleAttr = roleAttrTokens[0];
     if (roleAttr) {
@@ -235,7 +234,7 @@ function getComputedRole(el) {
             isImplicit: false,
         };
     }
-    const implicitRole = getImplicitRole(el);
+    const implicitRole = getImplicitRole(specs, el);
     if (implicitRole) {
         return {
             name: implicitRole,
@@ -287,8 +286,8 @@ function checkAriaValue(type, value, tokenEnum) {
     return true;
 }
 exports.checkAriaValue = checkAriaValue;
-function checkAria(attrName, currentValue, role) {
-    const ariaAttrs = html_spec_1.def['#ariaAttrs'];
+function checkAria(specs, attrName, currentValue, role) {
+    const ariaAttrs = specs.def['#ariaAttrs'];
     const aria = ariaAttrs.find(a => a.name === attrName);
     if (!aria) {
         return {

package/lib/id-duplication/index.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const ml_core_1 = require("@markuplint/ml-core");
 exports.default = (0, ml_core_1.createRule)({
     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 message = t('{0} is {1:c}', t('{0} of {1}', t('the {0}', 'value'), t('the "{0*}" {1}', 'id', 'attribute')), 'duplicated');
         const idStack = [];
         await document.walkOn('Element', async (node) => {
             const idAttrs = node.getAttributeToken('id');

package/lib/index.d.ts

@@ -4,7 +4,7 @@ declare const _default: {
     '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, 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-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>;
@@ -14,6 +14,7 @@ declare const _default: {
     doctype: import("packages/@markuplint/ml-core/lib").RuleSeed<"always", {
         denyObsolateType: 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>;

package/lib/index.js

@@ -14,6 +14,7 @@ const deprecated_attr_1 = (0, tslib_1.__importDefault)(require("./deprecated-att
 const deprecated_element_1 = (0, tslib_1.__importDefault)(require("./deprecated-element"));
 const disallowed_element_1 = (0, tslib_1.__importDefault)(require("./disallowed-element"));
 const doctype_1 = (0, tslib_1.__importDefault)(require("./doctype"));
+const end_tag_1 = (0, tslib_1.__importDefault)(require("./end-tag"));
 const id_duplication_1 = (0, tslib_1.__importDefault)(require("./id-duplication"));
 const indentation_1 = (0, tslib_1.__importDefault)(require("./indentation"));
 const ineffective_attr_1 = (0, tslib_1.__importDefault)(require("./ineffective-attr"));
@@ -43,6 +44,7 @@ exports.default = {
     'deprecated-element': deprecated_element_1.default,
     'disallowed-element': disallowed_element_1.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,

package/lib/ineffective-attr/index.js

@@ -29,7 +29,7 @@ exports.default = (0, ml_core_1.createRule)({
                         raw: attr.raw,
                         line: attr.startLine,
                         col: attr.startCol,
-                        message: t('{0} is {1:c}', t('the "{0}" {1}', name, 'attribute'), 'ineffective') +
+                        message: t('{0} is {1:c}', t('the "{0*}" {1}', name, 'attribute'), 'ineffective') +
                             t('. ') +
                             t("It doesn't need {0}", t('the {0}', 'attribute')),
                     });

package/lib/invalid-attr/index.js

@@ -18,7 +18,7 @@ exports.default = (0, ml_core_1.createRule)({
                 const attrName = attr.getName();
                 const name = attrName.potential;
                 if (!node.isCustomElement && attr.attrType === 'html-attr' && attr.candidate) {
-                    const message = t('{0} is {1:c}', t('the "{0}" {1}', attrName.raw, 'attribute'), 'disallowed') +
+                    const message = t('{0} is {1:c}', t('the "{0*}" {1}', attrName.raw, 'attribute'), 'disallowed') +
                         t('. ') +
                         t('Did you mean "{0}"?', attr.candidate);
                     report({
@@ -43,6 +43,9 @@ exports.default = (0, ml_core_1.createRule)({
                 let invalid = false;
                 const customRule = node.rule.option.attrs ? node.rule.option.attrs[name] : null;
                 if (customRule) {
+                    if (attr.attrType === 'html-attr' && attr.isDynamicValue) {
+                        continue;
+                    }
                     if ('enum' in customRule) {
                         invalid = (0, attr_check_1.attrCheck)(t, name.toLowerCase(), value, true, {
                             name,
@@ -56,7 +59,7 @@ exports.default = (0, ml_core_1.createRule)({
                         if (!(0, helpers_1.match)(value, customRule.pattern)) {
                             invalid = {
                                 invalidType: 'invalid-value',
-                                message: t('{0} is unmatched with the below patterns: {1}', t('the "{0}" {1}', name, 'attribute'), customRule.pattern),
+                                message: t('{0} is unmatched with the below patterns: {1}', t('the "{0*}" {1}', name, 'attribute'), customRule.pattern),
                             };
                         }
                     }
@@ -66,7 +69,7 @@ exports.default = (0, ml_core_1.createRule)({
                     else if ('disallowed' in customRule && customRule.disallowed) {
                         invalid = {
                             invalidType: 'non-existent',
-                            message: t('{0} is disallowed', t('the "{0}" {1}', name, 'attribute')),
+                            message: t('{0} is disallowed', t('the "{0*}" {1}', name, 'attribute')),
                         };
                     }
                 }
@@ -87,6 +90,10 @@ exports.default = (0, ml_core_1.createRule)({
                             break;
                         }
                         case 'non-existent': {
+                            const spec = (0, helpers_1.htmlSpec)(document.specs, node.nameWithNS);
+                            if (spec === null || spec === void 0 ? void 0 : spec.possibleToAddProperties) {
+                                continue;
+                            }
                             report({
                                 scope: node,
                                 message: invalid.message,

package/lib/landmark-roles/index.js

@@ -78,7 +78,7 @@ exports.default = (0, ml_core_1.createRule)({
                 if (el.isDescendantByUUIDList(uuidList)) {
                     report({
                         scope: el,
-                        message: t('{0} should be {1}', t('the "{0}" {1}', role, 'role'), 'top level'),
+                        message: t('{0} should be {1}', t('the "{0*}" {1}', role, 'role'), 'top level'),
                     });
                 }
             }

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

@@ -10,7 +10,7 @@ exports.default = (0, ml_core_1.createRule)({
                 return;
             }
             for (const attr of el.attributes) {
-                if (attr.attrType === 'ps-attr') {
+                if (attr.attrType === 'ps-attr' || attr.isDynamicValue) {
                     continue;
                 }
                 const name = attr.getName().potential;
@@ -36,7 +36,7 @@ exports.default = (0, ml_core_1.createRule)({
                         raw: extraRaw,
                         line: attr.spacesBeforeEqual.startLine,
                         col: attr.spacesBeforeEqual.startCol,
-                        message: t('{0} is {1}', t('the "{0}" {1}', name, 'attribute'), t('a {0}', 'boolean attribute')) +
+                        message: t('{0} is {1}', t('the "{0*}" {1}', name, 'attribute'), t('a {0}', 'boolean attribute')) +
                             t('. ') +
                             t("It doesn't need {0}", t('the {0}', 'value')),
                     });

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

@@ -4,7 +4,7 @@ const ml_core_1 = require("@markuplint/ml-core");
 const helpers_1 = require("../helpers");
 exports.default = (0, ml_core_1.createRule)({
     async verify({ document, report, t }) {
-        const { ariaAttrs } = (0, helpers_1.ariaSpec)();
+        const { ariaAttrs } = (0, helpers_1.ariaSpec)(document.specs);
         const idList = new Set();
         let hasDynamicId = false;
         await document.walkOn('Element', el => {
@@ -34,7 +34,7 @@ exports.default = (0, ml_core_1.createRule)({
                 return;
             }
             for (const attr of el.attributes) {
-                if (attr.attrType !== 'html-attr') {
+                if (attr.attrType !== 'html-attr' || attr.isDynamicValue) {
                     continue;
                 }
                 const name = attr.getName().potential;