@markuplint/rules 5.0.0-rc.2 → 5.0.0-rc.5

package/CHANGELOG.md

@@ -3,6 +3,104 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [5.0.0-rc.5](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.4...v5.0.0-rc.5) (2026-08-28)
+
+### Bug Fixes
+
+- **html-spec:** div in dl allows exactly one dt+dd group per HTML LS ([6f344bb](https://github.com/markuplint/markuplint/commit/6f344bbc72bc82dbe2d1ebf05600a5fce9838de1))
+- **html-spec:** forbid aria-expanded / aria-pressed on summary in details ([#3927](https://github.com/markuplint/markuplint/issues/3927)) ([92efa50](https://github.com/markuplint/markuplint/commit/92efa500ff6033d0bb82f816cbc27090afb5dbfe))
+- **rules:** apply always-matching check to srcset-less sources ([2b6960d](https://github.com/markuplint/markuplint/commit/2b6960d59f6be3b35cc44466d549813c9c14f2c3))
+- **rules:** apply deprecated-element and no-unsupported-features to pretendered components ([9becedd](https://github.com/markuplint/markuplint/commit/9becedd2e6403721fe57cc4624673d46343b811b)), closes [#3740](https://github.com/markuplint/markuplint/issues/3740)
+- **rules:** flag placeholder-label-option on empty required select ([#3936](https://github.com/markuplint/markuplint/issues/3936)) ([8a80802](https://github.com/markuplint/markuplint/commit/8a80802fb27010f4cba5a604857ee85608398caa))
+- **rules:** heading-levels must only fire when preceded by another heading ([34d380c](https://github.com/markuplint/markuplint/commit/34d380c4ada1315a389a18357feabb93daab2c93))
+- **rules:** keep transparent element itself in parent content-model check ([#3960](https://github.com/markuplint/markuplint/issues/3960)) ([88c0412](https://github.com/markuplint/markuplint/commit/88c04125a0a582436d865c7463c3d52b528e529e)), closes [#3928](https://github.com/markuplint/markuplint/issues/3928) [#3928](https://github.com/markuplint/markuplint/issues/3928) [#3928](https://github.com/markuplint/markuplint/issues/3928) [#3838](https://github.com/markuplint/markuplint/issues/3838)
+- **rules:** log when the permitted-contents transparent pattern cap is exceeded ([3d26ed4](https://github.com/markuplint/markuplint/commit/3d26ed4018a6a219fe0f69864d2d1e39cd3130c7)), closes [#3895](https://github.com/markuplint/markuplint/issues/3895)
+- **rules:** only require aria-valuenow on focusable separators ([fe97162](https://github.com/markuplint/markuplint/commit/fe971625962a25540a1cd0aff790fd882541da30)), closes [#3682](https://github.com/markuplint/markuplint/issues/3682)
+- **rules:** preserve [invalid-attr-valid-007] structure and mark breaking change inline ([de27044](https://github.com/markuplint/markuplint/commit/de27044fd64526ddcdf345795dca17ebb8456cf1))
+- **rules:** quote input-button-non-empty-value description in YAML frontmatter ([d289824](https://github.com/markuplint/markuplint/commit/d289824ed6b394d4a130de3f6ac819e568b06761))
+- **rules:** rename test fixture id "myform" to "form1" for cspell ([a9a1194](https://github.com/markuplint/markuplint/commit/a9a11943809e434d81a724aad516f18e29df794b))
+- **rules:** replace "unparseable" with "unparsable" for cspell ([fb63696](https://github.com/markuplint/markuplint/commit/fb6369612fcdae0f4f12a3581e467fe92faaad96))
+- **rules:** script-content guards Tier 1 errors and covers scopes specifier-map branches ([7004652](https://github.com/markuplint/markuplint/commit/700465223ad504b7b014d0314fe6a1c456ad20f7))
+- **rules:** skip condition-disallow when a referenced attr value is dynamic ([f04557d](https://github.com/markuplint/markuplint/commit/f04557d2f5526ad0b47a3ffc2691ccba9a67fa49))
+- **rules:** srcset-sizes-constraint now covers source-in-picture for w-without-sizes ([fb442a6](https://github.com/markuplint/markuplint/commit/fb442a6c3a9e2d4e20f1adcad1723621fbaa3c2c))
+- **rules:** strip only ASCII whitespace in always-matching media check ([e309e97](https://github.com/markuplint/markuplint/commit/e309e97bc4ba0bb2cf66b2c07a17c826b544482c))
+- **rules:** surface disallowed-element reason via reasonOnly (close [#3815](https://github.com/markuplint/markuplint/issues/3815)) ([#3986](https://github.com/markuplint/markuplint/issues/3986)) ([0142cec](https://github.com/markuplint/markuplint/commit/0142cec667f70fee086f2a6e06d7a26e66bda380))
+- **rules:** tighten new conformance rules from QA review ([3096626](https://github.com/markuplint/markuplint/commit/3096626bcf34c1897c8fb77cc0ff6a4f3d4c9fb4))
+- **rules:** wrap attr-bearing tags in description so MDX accepts them ([1ad1225](https://github.com/markuplint/markuplint/commit/1ad12256f67246d6ec18d5b97078ef1e2a85fd85))
+- **types): strict charset=utf-8; feat(rules:** usemap-references-map ([#3969](https://github.com/markuplint/markuplint/issues/3969)) ([c63070e](https://github.com/markuplint/markuplint/commit/c63070e29ccb283da7468b2fc67db372ebfcf42a)), closes [#3945](https://github.com/markuplint/markuplint/issues/3945) [#3966](https://github.com/markuplint/markuplint/issues/3966) [#3966](https://github.com/markuplint/markuplint/issues/3966) [#3928](https://github.com/markuplint/markuplint/issues/3928)
+
+### Code Refactoring
+
+- **rules:** redesign v5 rule system — naming, splits, specConformance ([#3989](https://github.com/markuplint/markuplint/issues/3989)) ([e925565](https://github.com/markuplint/markuplint/commit/e925565ce537848d7d1573369723cbce724a841b)), closes [#4](https://github.com/markuplint/markuplint/issues/4) [#aside-conditional-role-mapping-aria-13](https://github.com/markuplint/markuplint/issues/aside-conditional-role-mapping-aria-13)
+
+- feat(rules)!: enforce ARIA naming prohibition on autonomous custom elements ([7e01a4b](https://github.com/markuplint/markuplint/commit/7e01a4b691df643255ad5b1e6542688198b01a01))
+- fix(rules)!: scope invalid-attr virtual rules to narrow checks (#3803) ([77acb1d](https://github.com/markuplint/markuplint/commit/77acb1d1ac7cadd35203fa209ca016f802775b59)), closes [#3803](https://github.com/markuplint/markuplint/issues/3803)
+
+### Features
+
+- **config-presets:** forbid <base> after <link> or <script> in <head> ([#3925](https://github.com/markuplint/markuplint/issues/3925)) ([ceb892d](https://github.com/markuplint/markuplint/commit/ceb892d64772d459a6bd9564684218e3afbdec2e))
+- **html-spec:** enforce X-UA-Compatible content="IE=edge" ([#3929](https://github.com/markuplint/markuplint/issues/3929)) ([2638fba](https://github.com/markuplint/markuplint/commit/2638fba826a489ad745ee30c4b8ed195b2362403)), closes [#3928](https://github.com/markuplint/markuplint/issues/3928) [#3928](https://github.com/markuplint/markuplint/issues/3928) [#3734](https://github.com/markuplint/markuplint/issues/3734)
+- **rules:** add form-attr-references-form rule ([6b541f0](https://github.com/markuplint/markuplint/commit/6b541f032b76c3712c99ea35596b6b0aa79b6137))
+- **rules:** add input-button-non-empty-value rule ([2cc73dd](https://github.com/markuplint/markuplint/commit/2cc73ddd0f874fae9faf005f498d73fa364b682c))
+- **rules:** add input-file-empty-value rule ([228cbd7](https://github.com/markuplint/markuplint/commit/228cbd752956d3df8f525e4f19f9278a44d87160))
+- **rules:** add input-list-references-datalist rule ([#3931](https://github.com/markuplint/markuplint/issues/3931)) ([bf4ef54](https://github.com/markuplint/markuplint/commit/bf4ef54a1b2937ecbe05fbe5121ddfe199781a95))
+- **rules:** add itemprop-requires-itemscope rule ([2b4cba4](https://github.com/markuplint/markuplint/commit/2b4cba4a3afe205f1b4c96744e8d5115ea3bc0ee)), closes [#3852](https://github.com/markuplint/markuplint/issues/3852)
+- **rules:** add label-for-references-labelable rule ([#3932](https://github.com/markuplint/markuplint/issues/3932)) ([3713e6b](https://github.com/markuplint/markuplint/commit/3713e6b435a76fe03a941a36a5e33c0ab06c9a80)), closes [#3918](https://github.com/markuplint/markuplint/issues/3918)
+- **rules:** add label-no-multiple-controls rule ([5da3f85](https://github.com/markuplint/markuplint/commit/5da3f8523dd6ffd9f44ea73d9012952aad85d821))
+- **rules:** add map-id-name-match rule ([1472daf](https://github.com/markuplint/markuplint/commit/1472daf62470bf56c4cb326ab47dc43ba87a8cb3))
+- **rules:** add meter-value-bounds rule ([82cb9a0](https://github.com/markuplint/markuplint/commit/82cb9a01edaf574f5b3b1e8e8ce531d0fab21f1c))
+- **rules:** add no-extra-selected-options rule ([3ea75ac](https://github.com/markuplint/markuplint/commit/3ea75ac0b6850c36d5419924b18c1002dfb864a9))
+- **rules:** add progress-value-bounds rule ([#3926](https://github.com/markuplint/markuplint/issues/3926)) ([1e259ec](https://github.com/markuplint/markuplint/commit/1e259ec9929ceb3c7ac5864ce2807420646e9602))
+- **rules:** add script-content rule for `<script type=importmap>` JSON validation ([c635970](https://github.com/markuplint/markuplint/commit/c6359707af6b37abb0bca130e880cb1e9b5b9005))
+- **rules:** add wai-aria-tab-requires-tabpanel rule ([#3955](https://github.com/markuplint/markuplint/issues/3955)) ([eac9abe](https://github.com/markuplint/markuplint/commit/eac9abef20ef304c3da2114849686b9cf0733942))
+- **rules:** character-reference consumes parse5 malformed-reference codes via parseErrors hook ([a33bb4e](https://github.com/markuplint/markuplint/commit/a33bb4e6674c23ae06b6a12d45ace1c2e954b3be)), closes [#3844](https://github.com/markuplint/markuplint/issues/3844)
+- **rules:** declare mirrorsParseErrorCodes on attr-duplication / doctype / no-orphaned-end-tag ([06893f0](https://github.com/markuplint/markuplint/commit/06893f07584be548c9719153cfae288bd2fdf83e)), closes [#3844](https://github.com/markuplint/markuplint/issues/3844)
+- **rules:** detect always-matching source in srcset-sizes-constraint ([a11199c](https://github.com/markuplint/markuplint/commit/a11199cb13011a775630788a80820ade189b21bd))
+- **rules:** disallow aria-\* on elements where html-spec says properties: false ([1850ee3](https://github.com/markuplint/markuplint/commit/1850ee3a854d25fd5291d4c2a555b8be3f99f245)), closes [#3735](https://github.com/markuplint/markuplint/issues/3735) [#3735](https://github.com/markuplint/markuplint/issues/3735)
+- **rules:** extend label-no-multiple-controls for external labeled control ([#3933](https://github.com/markuplint/markuplint/issues/3933)) ([392f051](https://github.com/markuplint/markuplint/commit/392f051c0d7ee8be8f3dc12afae6a902c6c8ea2b))
+- **rules:** report HTML LS table model errors in table-row-column-alignment ([#3953](https://github.com/markuplint/markuplint/issues/3953)) ([bacdcd6](https://github.com/markuplint/markuplint/commit/bacdcd697f76300153388f5b1cabaa8504f08d1f)), closes [#3916](https://github.com/markuplint/markuplint/issues/3916) [#3915](https://github.com/markuplint/markuplint/issues/3915) [#3916](https://github.com/markuplint/markuplint/issues/3916) [#3915](https://github.com/markuplint/markuplint/issues/3915)
+- **rules:** script-content validates the integrity top-level key ([cc1a050](https://github.com/markuplint/markuplint/commit/cc1a0504ab0664c96ae326e6c5580d4fe04fed58))
+- **rules:** surface parse5-silent HTML LS parse errors (close nu-only umbrella [#3943](https://github.com/markuplint/markuplint/issues/3943)) ([#3980](https://github.com/markuplint/markuplint/issues/3980)) ([89951fa](https://github.com/markuplint/markuplint/commit/89951fa274007d56370510cb0cf11aead808ce13))
+- **rules:** validate `<script type="speculationrules">` content ([d000a84](https://github.com/markuplint/markuplint/commit/d000a84badfc359c11258f8d46230c44a254cc05)), closes [#3882](https://github.com/markuplint/markuplint/issues/3882)
+- **types:** enforce element-context autocomplete constraints (select webauthn, input[type=hidden] on/off) ([#3930](https://github.com/markuplint/markuplint/issues/3930)) ([dc51d46](https://github.com/markuplint/markuplint/commit/dc51d460731541c963c9aa8bdf407c20974dced4))
+- **types:** reject general-enclosed matches in MediaQueryList and SourceSizeList ([#3934](https://github.com/markuplint/markuplint/issues/3934)) ([2fc8615](https://github.com/markuplint/markuplint/commit/2fc8615c475540d4e5b85661baa1491d791cb661))
+- **types:** validate CSP3 grammar for meta[content][http-equiv=content-security-policy] ([#3982](https://github.com/markuplint/markuplint/issues/3982)) ([b2d1d84](https://github.com/markuplint/markuplint/commit/b2d1d84ea92c754fe2dbf54950ae8fdb74fe51eb)), closes [#3942](https://github.com/markuplint/markuplint/issues/3942) [#3942](https://github.com/markuplint/markuplint/issues/3942) [#3942](https://github.com/markuplint/markuplint/issues/3942) [#3946](https://github.com/markuplint/markuplint/issues/3946)
+
+### BREAKING CHANGES
+
+- **rules:** with no alias coverage.
+- **rules:** is visible in the test source, not only in git history.
+
+Also drop the duplicate `//user:pass@sample.com/path/to` case from
+[invalid-attr-invalid-044]; valid-007 now carries the canonical
+assertion for that URL, while invalid-044 covers the empty-userinfo
+variant (`http://@host`) and the other six URL LS categories.
+
+- <my-widget aria-label="x"> now reports an error
+  under wai-aria-disallowed-props. Add an explicit role
+  (<my-widget role="button" aria-label="x">) or remove the naming
+  attribute. Affects autonomous custom elements only;
+  customised-built-in elements (<button is="x-y">) are unaffected.
+
+Source: https://w3c.github.io/html-aria/#docconformance
+
+Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
+
+- Named rules wrapping `invalid-attr` (e.g., `a11y/no-accesskey`)
+  only perform their configured allow/disallow checks and no longer fall back to
+  HTML-spec validation. To retain spec validation, extend `markuplint:html-standard`
+  or set `"invalid-attr": true` in your config.
+
+Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
+
+# [5.0.0-rc.4](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.3...v5.0.0-rc.4) (2026-04-19)
+
+**Note:** Version bump only for package @markuplint/rules
+
+# [5.0.0-rc.3](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.2...v5.0.0-rc.3) (2026-04-19)
+
+**Note:** Version bump only for package @markuplint/rules
+
 # [5.0.0-rc.2](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.1...v5.0.0-rc.2) (2026-04-15)
 
 ### Bug Fixes

package/CLAUDE.md

@@ -1,13 +1,24 @@
 # @markuplint/rules
 
+## Adding a New Rule (checklist — steps 2–4 have NO enforcing test; forgetting them fails silently)
+
+1. `src/<rule-name>/`: `index.ts`, `meta.ts`, `schema.json`, spec file(s), `README.md` **and** `README.ja.md` — both languages are website source and MUST stay in sync; updating only one is a recurring mistake
+   - `meta.ts`'s `category` must be one of the v5 9-category scheme: `syntax`, `structure`, `attributes`, `references`, `forms`, `a11y`, `style`, `maintainability`, `compat` (see the [Categories section](https://markuplint.dev/docs/migration/v4-to-v5/rules/rule-names#categories) of the website's rule-names guide for what each covers)
+   - `meta.ts` should also declare `specConformance` (`sources`/`level`/`cites`) once you know the rule's governing spec and requirement strength — `level: 'must'` or `'should'` requires a non-empty `cites`. The rollout across pre-existing rules is incremental and partial by design (enforced only for rules that declare it — see `spec-conformance.spec.ts` and `@markuplint/config-presets`'s `html-standard-entries.spec.ts`), but a **new** rule should declare it from the start
+2. Register the rule in `src/index.ts` (import + registry entry)
+3. Add a `$ref` entry to `packages/@markuplint/rules/schema.json` — this file is a **manually maintained registry** (no generator produces it, despite looking generated)
+4. If the rule belongs in a preset, update `packages/@markuplint/config-presets/src/`
+
 ## Test ID Convention (MANDATORY)
 
-Every `test()` block in `src/**/*.spec.ts` MUST have a unique ID prefix:
+Every `test()` block in rule spec files — `src/<rule-name>/**/*.spec.ts` — MUST have a unique ID prefix:
 
 ```
 [rule-name-category-NNN] description
 ```
 
+Repo-wide meta specs directly under `src/` (e.g. `mirrors-parse-error-codes.spec.ts`) are NOT rule specs and are exempt.
+
 ### Categories
 
 | Category     | When to use                                               |
@@ -24,4 +35,22 @@ Every `test()` block in `src/**/*.spec.ts` MUST have a unique ID prefix:
 - Numbers are sequential per category per file, starting at `001`
 - Multiple tests for the same issue: `[rule-name-issue-NNNN-001]`, `[rule-name-issue-NNNN-002]`
 - When adding a new test, assign the next available number in the appropriate category
-- Run `/list-rule-test` (or `node .claude/commands/scripts/list-rule-test.mjs --no-id`) to check for missing IDs
+- Run `/list-rule-test` (or `node .claude/skills/list-rule-test/scripts/list-rule-test.mjs --no-id`) to check for missing IDs — the only expected hits are the exempt meta specs noted above
+
+## Assertion Convention (MANDATORY)
+
+When asserting reported violations in rule specs, use `toStrictEqual` with the exact violation object `{ severity, line, col, message, raw }`. Never loose or partial matchers for violation objects. (Assertions on other values — counts, fixed code strings, registry invariants — use whatever matcher fits.)
+
+## parse5 mirror declaration (`meta.mirrorsParseErrorCodes`)
+
+When a rule's detection covers a parse5 `onParseError` code, declare the code in the rule's
+`meta.ts` via `mirrorsParseErrorCodes` so ml-core suppresses the duplicate `parse-error`
+channel violation. The declaration criterion (rule scope must be at least as broad as the
+parse5 event) and the ruleset dual-lookup semantics are documented in JSDoc:
+`@markuplint/ml-core` `src/ml-rule/types.ts` (the meta field) and
+`src/ml-core.ts` `#pushNonFatalParseErrors`. Canonical example of a rule that actively
+consumes `document.parseErrors` and reports them under its own ruleId: `no-malformed-character-reference`.
+
+Enforced by tests: `src/mirrors-parse-error-codes.spec.ts` (no duplicate declarations across
+rules) and `@markuplint/html-parser`'s `parse-error-code-sync.spec.ts` (compile-time alignment
+with parse5's code list).

package/lib/{wai-aria-deprecated-role → aria-prop-requires-role}/index.d.ts

@@ -1,4 +1,3 @@
 import type { Options } from '../wai-aria/types.js';
-/** Warns when a deprecated WAI-ARIA role is used. */
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<boolean, Options>>;
 export default _default;

package/lib/{wai-aria-no-global-prop → aria-prop-requires-role}/index.js

@@ -3,7 +3,6 @@ import { ARIA_RECOMMENDED_VERSION } from '@markuplint/ml-spec';
 import { checkingNoGlobalProp } from '../wai-aria/checkings/no-global-prop.js';
 import { defaultOptions } from '../wai-aria/default-options.js';
 import meta from './meta.js';
-/** Warns when a non-global ARIA property is used on an element without an explicit role. */
 export default createRule({
     meta,
     defaultOptions,

package/lib/attr-check.d.ts

@@ -2,65 +2,17 @@ import type { Translator } from '@markuplint/i18n';
 import type { Attribute as AttrSpec, AttributeType } from '@markuplint/ml-spec';
 import type { Type } from '@markuplint/types';
 import type { ReadonlyDeep } from 'type-fest';
-/**
- * Discriminated union tag representing the kind of attribute invalidity.
- */
 type InvalidTYpe = 'non-existent' | 'invalid-value' | 'disallowed-attr';
-/**
- * Describes a single attribute validation failure, including the kind
- * of invalidity, a human-readable message, and an optional source location.
- *
- * @template T - The specific invalidity tag (defaults to any `InvalidTYpe`)
- */
 type Invalid<T extends InvalidTYpe = InvalidTYpe> = {
     invalidType: T;
     message: string;
     loc?: Loc;
 };
-/**
- * Source location information pointing to the invalid portion of an attribute value.
- */
 type Loc = {
     raw: string;
     line: number;
     col: number;
 };
-/**
- * Validates an attribute against its specification. Used by the `invalid-attr`
- * and `wai-aria` rules.
- *
- * Performs the following checks in order:
- * 1. Skips `data-*`, `aria-*`/`role`, and `adapt-*` attributes (unless `isCustomRule` is `true`)
- * 2. Verifies the attribute exists in the spec
- * 3. Checks case-sensitive name matching
- * 4. Checks whether the attribute is marked as `noUse` (disallowed)
- * 5. Safety-net fallback for `ConditionalAttributeType[]` (#3685); in normal flow,
- *    `isValidAttr()` resolves conditional types before calling this function (#3598)
- * 6. Validates the attribute value against all declared types
- *
- * @param t - The i18n translator for generating localized error messages
- * @param name - The attribute name to check
- * @param value - The attribute value to validate
- * @param isCustomRule - When `true`, skips the built-in bypass for `data-*`, `aria-*`, and `adapt-*` attributes
- * @param spec - The attribute specification to validate against; if absent, the attribute is considered non-existent
- * @param allAttrNames - Optional list of all valid attribute names for the element;
- *   used to suggest a similar attribute name via Levenshtein distance when the attribute is non-existent
- * @returns `false` if the attribute is valid, a single `Invalid` object for existence/disallowed errors,
- *   or an array of `Invalid<'invalid-value'>` objects for value validation failures
- */
 export declare function attrCheck(t: Translator, name: string, value: string, isCustomRule: boolean, spec?: AttrSpec, allAttrNames?: readonly string[]): Invalid | Invalid<'invalid-value'>[] | false;
-/**
- * Validates an attribute value against a single attribute type definition.
- * Returns `false` if the value is valid, or a tuple of `[message, location]`
- * describing the mismatch.
- *
- * Boolean attributes are always considered valid (their mere presence is sufficient).
- *
- * @param t - The i18n translator for generating localized error messages
- * @param name - The attribute name (used in error messages)
- * @param value - The attribute value to validate
- * @param type - The attribute type definition to validate against
- * @returns `false` if the value is valid, or a `[message, location]` tuple on failure
- */
 export declare function valueCheck(t: Translator, name: string, value: string, type: ReadonlyDeep<AttributeType | Type>): [string, Loc] | false;
 export {};

package/lib/attr-check.js

@@ -3,29 +3,6 @@ import { toNonNullableArrayFromItemOrArray } from '@markuplint/shared';
 import { check, getCandidate } from '@markuplint/types';
 import { createMessageValueExpected } from './create-message.js';
 import { log } from './debug.js';
-/**
- * Validates an attribute against its specification. Used by the `invalid-attr`
- * and `wai-aria` rules.
- *
- * Performs the following checks in order:
- * 1. Skips `data-*`, `aria-*`/`role`, and `adapt-*` attributes (unless `isCustomRule` is `true`)
- * 2. Verifies the attribute exists in the spec
- * 3. Checks case-sensitive name matching
- * 4. Checks whether the attribute is marked as `noUse` (disallowed)
- * 5. Safety-net fallback for `ConditionalAttributeType[]` (#3685); in normal flow,
- *    `isValidAttr()` resolves conditional types before calling this function (#3598)
- * 6. Validates the attribute value against all declared types
- *
- * @param t - The i18n translator for generating localized error messages
- * @param name - The attribute name to check
- * @param value - The attribute value to validate
- * @param isCustomRule - When `true`, skips the built-in bypass for `data-*`, `aria-*`, and `adapt-*` attributes
- * @param spec - The attribute specification to validate against; if absent, the attribute is considered non-existent
- * @param allAttrNames - Optional list of all valid attribute names for the element;
- *   used to suggest a similar attribute name via Levenshtein distance when the attribute is non-existent
- * @returns `false` if the attribute is valid, a single `Invalid` object for existence/disallowed errors,
- *   or an array of `Invalid<'invalid-value'>` objects for value validation failures
- */
 export function attrCheck(t, name, value, isCustomRule, spec, allAttrNames) {
     if (!isCustomRule) {
         if (/^data-.+$/.test(name)) {
@@ -98,19 +75,6 @@ export function attrCheck(t, name, value, isCustomRule, spec, allAttrNames) {
     }
     return [...invalidMap.values()];
 }
-/**
- * Validates an attribute value against a single attribute type definition.
- * Returns `false` if the value is valid, or a tuple of `[message, location]`
- * describing the mismatch.
- *
- * Boolean attributes are always considered valid (their mere presence is sufficient).
- *
- * @param t - The i18n translator for generating localized error messages
- * @param name - The attribute name (used in error messages)
- * @param value - The attribute value to validate
- * @param type - The attribute type definition to validate against
- * @returns `false` if the value is valid, or a `[message, location]` tuple on failure
- */
 export function valueCheck(t, name, value, type) {
     if (type === 'Boolean') {
         // Valid because an attribute is exist

package/lib/attr-eligibility.d.ts

@@ -0,0 +1,41 @@
+import type { Translator } from '@markuplint/i18n';
+import type { PlainData } from '@markuplint/ml-config';
+import type { Element, RuleConfigValue } from '@markuplint/ml-core';
+import type { Attribute } from '@markuplint/ml-spec';
+/**
+ * Result of resolving whether an attribute name is known to the spec and,
+ * if so, whether it currently applies. Shared by `no-unknown-attr` (acts on
+ * `unknown`/`case-mismatch`), `no-disallowed-attr` (acts on `no-use`/
+ * `condition-not-met`), and `no-invalid-attr-value` (acts on `ok`, using the
+ * resolved `spec.type` for value-checking). Splitting this into one function
+ * keeps the three rules' notion of "what the spec says about this name" in
+ * sync without re-deriving it three times.
+ */
+export type AttrEligibility = {
+    readonly status: 'exempt';
+} | {
+    readonly status: 'unknown';
+    readonly candidate: string | undefined;
+} | {
+    readonly status: 'case-mismatch';
+    readonly expectedName: string;
+} | {
+    readonly status: 'no-use';
+} | {
+    readonly status: 'condition-not-met';
+} | {
+    readonly status: 'ok';
+    readonly spec: Attribute;
+};
+/**
+ * Resolves an attribute name against the element's `attrSpecs`, following
+ * the same order the pre-split `invalid-attr` rule used: existence, then
+ * case-sensitivity, then `noUse`, then `condition` (with
+ * `ConditionalAttributeType[]` resolved to a concrete type first, per #3598).
+ */
+export declare function resolveAttrEligibility<T extends RuleConfigValue, O extends PlainData>(name: string, node: Element<T, O>, attrSpecs: readonly Attribute[]): AttrEligibility;
+/**
+ * Formats the "did you mean" suffix shared by `no-unknown-attr`'s two typo
+ * paths (spec-driven candidate, case-sensitivity mismatch).
+ */
+export declare function suggestionSuffix(t: Translator, candidate: string | undefined): string;

package/lib/attr-eligibility.js

@@ -0,0 +1,60 @@
+import { isConditionalAttributeTypeArray } from '@markuplint/ml-spec';
+import { getCandidate } from '@markuplint/types';
+import { attrMatches, conditionDependsOnDynamicAttr } from './helpers.js';
+/**
+ * Attribute namespaces that are exempt from spec-based name/eligibility
+ * checking regardless of what `attrSpecs` says: `data-*` and `adapt-*` are
+ * any-type by definition, and ARIA (`aria-*` / `role`) is validated by the
+ * dedicated ARIA rules, not here.
+ */
+const EXEMPT_NAME_PATTERN = /^(?:data-.+|adapt-.+|aria-.+|role)$/;
+/**
+ * Resolves an attribute name against the element's `attrSpecs`, following
+ * the same order the pre-split `invalid-attr` rule used: existence, then
+ * case-sensitivity, then `noUse`, then `condition` (with
+ * `ConditionalAttributeType[]` resolved to a concrete type first, per #3598).
+ */
+export function resolveAttrEligibility(name, 
+// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
+node, attrSpecs) {
+    if (EXEMPT_NAME_PATTERN.test(name)) {
+        return { status: 'exempt' };
+    }
+    let spec = attrSpecs.find(s => s.name.toLowerCase() === name.toLowerCase());
+    if (!spec) {
+        const candidate = getCandidate(name, attrSpecs.map(s => s.name));
+        return { status: 'unknown', candidate };
+    }
+    const nameCaseSensitive = /[A-Z]/.test(spec.name);
+    if (nameCaseSensitive && name !== spec.name) {
+        return { status: 'case-mismatch', expectedName: spec.name };
+    }
+    if (spec.noUse) {
+        return { status: 'no-use' };
+    }
+    // Resolve ConditionalAttributeType[] to a concrete type based on element matching (#3598).
+    if (isConditionalAttributeTypeArray(spec.type)) {
+        const matched = spec.type.find(entry => {
+            const cond = typeof entry.condition === 'string' ? entry.condition : entry.condition.join(',');
+            return node.matches(cond);
+        });
+        // Fallback to 'Any' when no condition matches: input types without an explicit
+        // entry (text, search, tel, password, hidden, checkbox, radio, file, submit,
+        // image, reset, button) have no value constraints per the HTML spec.
+        spec = { ...spec, type: matched ? matched.type : 'Any' };
+    }
+    if (spec.condition != null &&
+        !node.hasSpreadAttr &&
+        !conditionDependsOnDynamicAttr(node, spec.condition) &&
+        !attrMatches(node, spec.condition)) {
+        return { status: 'condition-not-met' };
+    }
+    return { status: 'ok', spec };
+}
+/**
+ * Formats the "did you mean" suffix shared by `no-unknown-attr`'s two typo
+ * paths (spec-driven candidate, case-sensitivity mismatch).
+ */
+export function suggestionSuffix(t, candidate) {
+    return candidate ? t('. ') + t('Did you mean "{0*}"?', candidate) : '';
+}

package/lib/attr-value-quotes/index.d.ts

@@ -6,12 +6,5 @@ export type Quote = '"' | "'";
 export type QuoteMap = {
     [P in Type]: Quote;
 };
-/**
- * Rule that enforces consistent quotation marks around attribute values.
- *
- * Verifies that all attribute values use the configured quote style (double
- * or single). Attributes without values, dynamic values, and directives are
- * skipped. Includes an auto-fix that replaces mismatched quotes.
- */
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<Type, undefined>>;
 export default _default;

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

@@ -7,13 +7,6 @@ const quoteList = {
     double: '"',
     single: "'",
 };
-/**
- * Rule that enforces consistent quotation marks around attribute values.
- *
- * Verifies that all attribute values use the configured quote style (double
- * or single). Attributes without values, dynamic values, and directives are
- * skipped. Includes an auto-fix that replaces mismatched quotes.
- */
 export default createRule({
     meta: meta,
     defaultSeverity: 'warning',

package/lib/attr-value-quotes/meta.d.ts

@@ -1,4 +1,3 @@
-/** Rule metadata for `attr-value-quotes`: categorized as a style rule. */
 declare const _default: {
     readonly category: "style";
     readonly fixable: true;

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

@@ -1,4 +1,3 @@
-/** Rule metadata for `attr-value-quotes`: categorized as a style rule. */
 export default {
     category: 'style',
     fixable: true,

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

@@ -1,12 +1,4 @@
 /** The enforced letter case for attribute names. */
 export type Value = 'lower' | 'upper';
-/**
- * Rule that enforces consistent letter case for HTML attribute names.
- *
- * Reports attribute names that do not match the configured case (lower or upper)
- * on HTML elements. Foreign elements and case-sensitive attributes (per spec)
- * are excluded. Includes an auto-fix that converts attribute names to the
- * configured case.
- */
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<Value, undefined>>;
 export default _default;

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

@@ -1,13 +1,5 @@
 import { createRule, getAttrSpecs } from '@markuplint/ml-core';
 import meta from './meta.js';
-/**
- * Rule that enforces consistent letter case for HTML attribute names.
- *
- * Reports attribute names that do not match the configured case (lower or upper)
- * on HTML elements. Foreign elements and case-sensitive attributes (per spec)
- * are excluded. Includes an auto-fix that converts attribute names to the
- * configured case.
- */
 export default createRule({
     meta: meta,
     defaultSeverity: 'warning',

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

@@ -1,4 +1,3 @@
-/** Rule metadata for `case-sensitive-attr-name`: categorized as a style rule. */
 declare const _default: {
     readonly category: "style";
     readonly fixable: true;

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

@@ -1,4 +1,3 @@
-/** Rule metadata for `case-sensitive-attr-name`: categorized as a style rule. */
 export default {
     category: 'style',
     fixable: true,

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

@@ -1,12 +1,4 @@
 /** The enforced letter case for tag names. */
 export type Value = 'lower' | 'upper';
-/**
- * Rule that enforces consistent letter case for HTML tag names.
- *
- * Reports opening and closing tag names that do not match the configured
- * case (lower or upper) on HTML elements. Skipped when the document is
- * tag-name-case-sensitive or for foreign elements. Includes an auto-fix
- * that converts tag names to the configured case.
- */
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<Value, undefined>>;
 export default _default;

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

@@ -1,13 +1,5 @@
 import { createRule } from '@markuplint/ml-core';
 import meta from './meta.js';
-/**
- * Rule that enforces consistent letter case for HTML tag names.
- *
- * Reports opening and closing tag names that do not match the configured
- * case (lower or upper) on HTML elements. Skipped when the document is
- * tag-name-case-sensitive or for foreign elements. Includes an auto-fix
- * that converts tag names to the configured case.
- */
 export default createRule({
     meta: meta,
     defaultSeverity: 'warning',

package/lib/case-sensitive-tag-name/meta.d.ts

@@ -1,4 +1,3 @@
-/** Rule metadata for `case-sensitive-tag-name`: categorized as a style rule. */
 declare const _default: {
     readonly category: "style";
     readonly fixable: true;

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

@@ -1,4 +1,3 @@
-/** Rule metadata for `case-sensitive-tag-name`: categorized as a style rule. */
 export default {
     category: 'style',
     fixable: true,

package/lib/class-naming/index.d.ts

@@ -1,12 +1,4 @@
 /** One or more regex or glob patterns that class names must match, or `null` to disable. */
 export type Value = string | string[] | null;
-/**
- * Rule that validates class attribute values against configured naming patterns.
- *
- * Splits the `class` attribute into individual class names and checks each
- * one against the provided string or regex patterns. Reports any class name
- * that does not match at least one of the configured patterns. Dynamic
- * values are excluded from the check.
- */
 declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<Value, undefined>>;
 export default _default;

package/lib/class-naming/index.js

@@ -2,14 +2,6 @@ import { createRule } from '@markuplint/ml-core';
 import { toNoEmptyStringArrayFromStringOrArray } from '@markuplint/shared';
 import { match } from '../helpers.js';
 import meta from './meta.js';
-/**
- * Rule that validates class attribute values against configured naming patterns.
- *
- * Splits the `class` attribute into individual class names and checks each
- * one against the provided string or regex patterns. Reports any class name
- * that does not match at least one of the configured patterns. Dynamic
- * values are excluded from the check.
- */
 export default createRule({
     meta: meta,
     defaultSeverity: 'warning',

package/lib/class-naming/meta.d.ts

@@ -1,5 +1,4 @@
-/** Rule metadata for `class-naming`: categorized as a naming-convention rule. */
 declare const _default: {
-    readonly category: "naming-convention";
+    readonly category: "style";
 };
 export default _default;

package/lib/class-naming/meta.js

@@ -1,4 +1,3 @@
-/** Rule metadata for `class-naming`: categorized as a naming-convention rule. */
 export default {
-    category: 'naming-convention',
+    category: 'style',
 };

package/lib/consistent-table-row-length/index.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Flags rows whose width disagrees with the table's base column count.
+ *
+ * Unlike the sibling splits of `table-row-column-alignment`, this is not a table model error:
+ * HTML LS §4.9.12.1 Step 7 grows `xwidth` to fit a wider row, and a narrower row is only a table
+ * model error where {@link import('../no-empty-table-track/index.js')} already catches it (a
+ * column no cell ever anchors to). A ragged table is nevertheless almost always a mistake, which
+ * is why this check exists and why it stays a `warning` rather than the `error` of its siblings.
+ *
+ * Skips a table that `no-table-cell-overlap` already reports on — once two cells claim the same
+ * slot, row widths derived from the grid model are no longer reliable.
+ */
+declare const _default: Readonly<import("@markuplint/ml-core").RuleSeed<boolean, undefined>>;
+export default _default;