@markuplint/rules 5.0.0-rc.7 → 5.0.1

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [5.0.1](https://github.com/markuplint/markuplint/compare/v5.0.0...v5.0.1) (2026-09-22)
+
+### Bug Fixes
+
+- **rules:** skip framework directives in no-event-handler-attr ([1da97ed](https://github.com/markuplint/markuplint/commit/1da97ed5ff6a17aeabd9dbe7077ebe057a3b674d)), closes [#4044](https://github.com/markuplint/markuplint/issues/4044)
+
+# [5.0.0](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.7...v5.0.0) (2026-09-11)
+
+### Bug Fixes
+
+- **rules:** add role predefined group to attr-order ([ed07019](https://github.com/markuplint/markuplint/commit/ed07019cafaf5dc85ce5c5e6f4ac56f908b93b54)), closes [#4033](https://github.com/markuplint/markuplint/issues/4033)
+
 # [5.0.0-rc.7](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.6...v5.0.0-rc.7) (2026-08-31)
 
 ### Bug Fixes

package/CLAUDE.md

@@ -2,16 +2,16 @@
 
 ## 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
+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)
+## Test ID Convention
 
-Every `test()` block in rule spec files — `src/<rule-name>/**/*.spec.ts` — MUST have a unique ID prefix:
+Every `test()` block in rule spec files — `src/<rule-name>/**/*.spec.ts` — has a unique ID prefix:
 
 ```
 [rule-name-category-NNN] description
@@ -37,7 +37,42 @@ Repo-wide meta specs directly under `src/` (e.g. `mirrors-parse-error-codes.spec
 - When adding a new test, assign the next available number in the appropriate category
 - 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)
+## Sample identifiers in specs
+
+<!-- cspell:ignore myform mymap -->
+
+Sample `id` / `name` values in test markup must be kebab-case (`my-form`), suffixed with a digit
+(`form1`), or a plain dictionary word (`target`). Never a camelCase coinage (`myform`, `mymap`,
+`targetEl`): CSpell decomposes compound words only when every part is at least three characters,
+so a two-letter prefix glued to a word is looked up whole, misses the dictionary, and fails the
+lint — on the `dev` line the same one-line fix has been made in four separate PRs (#3826, #3831,
+#3833, #3932). The v6 line has no CSpell, so a coinage introduced there surfaces only once the
+branch merges somewhere that does.
+
+## Writing a rule's README and `schema.json` description
+
+Both files feed the website build (`website/scripts/prebuild/rule-content.mts` injects the
+description into MDX), which breaks in two ways that no rule test catches:
+
+- **An attribute-bearing tag must be wrapped in backticks.** `escapeMdx` only escapes bare tags
+  (`<head>`); `<input type="button">` reaches the MDX parser as unclosed JSX and crashes the
+  Docusaurus build.
+- **A YAML `description:` may not start with a backtick** — `gray-matter` fails with "stream/document
+  separator expected". Wrap the whole value in single quotes instead:
+
+  ```yaml
+  description: '`<input type="button">` must not have a value attribute'
+  ```
+
+Deliberately malformed markup in an ❌ example needs `<!-- prettier-ignore -->` immediately before
+the fence: oxfmt formats ` ```html ` blocks as real HTML and will close the tag you left open,
+turning the incorrect example into a copy of the correct one. Apply it to `README.md` and
+`README.ja.md` separately, and read the diff after `yarn lint` to confirm the breakage survived.
+
+The CI website check only runs when a PR already touches `website/`, so a new rule can land with a
+broken description and detonate in someone else's PR. Run `yarn site:build` before pushing.
+
+## Assertion Convention
 
 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.)
 

package/lib/attr-order/index.d.ts

@@ -1,4 +1,4 @@
-type PredefinedGroup = 'global' | 'event' | 'aria' | 'data' | 'spread';
+type PredefinedGroup = 'global' | 'event' | 'aria' | 'role' | 'data' | 'spread';
 type SortOrder = 'alphabetical' | 'source-order' | string[];
 type OrderEntry = string | {
     name?: string;

package/lib/attr-order/index.js

@@ -112,6 +112,13 @@ eventAttrs) {
                         match: (name) => /^aria-.+$/.test(name),
                     };
                 }
+                case 'role': {
+                    return {
+                        entryIndex,
+                        order: groupOrder,
+                        match: (name) => name === 'role',
+                    };
+                }
                 case 'data': {
                     return {
                         entryIndex,

package/lib/index.d.ts

@@ -16,7 +16,7 @@ declare const rules: {
     readonly 'attr-order': Readonly<import("@markuplint/ml-core").RuleSeed<(string | {
         name?: string;
         pattern?: string;
-        group?: "data" | "aria" | "global" | "event" | "spread";
+        group?: "data" | "aria" | "global" | "event" | "role" | "spread";
         order?: string[] | "alphabetical" | "source-order";
     })[], {
         alphabetical?: boolean;

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

@@ -29,6 +29,13 @@ export default createRule({
             if (attr.ownerElement.elementType !== 'html') {
                 return;
             }
+            // Framework directives that resolve to an `onXxx` name (Vue `@click`, Alpine
+            // `x-on:click`, htmx `hx-on:click`, Svelte `on:click`) are excluded: they never
+            // reach the DOM as a literal inline-handler attribute, and each framework
+            // documents them as its recommended event-binding idiom.
+            if (attr.isDirective) {
+                return;
+            }
             const ignoreList = Array.isArray(attr.rule.options.ignore)
                 ? attr.rule.options.ignore
                 : attr.rule.options.ignore

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/rules",
-	"version": "5.0.0-rc.7",
+	"version": "5.0.1",
 	"description": "Built-in rules of markuplint",
 	"repository": {
 		"type": "git",
@@ -31,13 +31,13 @@
 		"./lib/permitted-contents/debug.js": "./lib/permitted-contents/debug.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/html-spec": "5.0.0-rc.7",
-		"@markuplint/ml-config": "5.0.0-rc.7",
-		"@markuplint/ml-core": "5.0.0-rc.7",
-		"@markuplint/ml-spec": "5.0.0-rc.7",
-		"@markuplint/selector": "5.0.0-rc.7",
-		"@markuplint/shared": "5.0.0-rc.7",
-		"@markuplint/types": "5.0.0-rc.7",
+		"@markuplint/html-spec": "5.0.1",
+		"@markuplint/ml-config": "5.0.1",
+		"@markuplint/ml-core": "5.0.1",
+		"@markuplint/ml-spec": "5.0.1",
+		"@markuplint/selector": "5.0.1",
+		"@markuplint/shared": "5.0.1",
+		"@markuplint/types": "5.0.1",
 		"@mdn/browser-compat-data": "^7.3.2",
 		"@types/debug": "4.1.13",
 		"ansi-colors": "4.1.3",
@@ -47,5 +47,5 @@
 		"image-size": "^2.0.2",
 		"type-fest": "5.6.0"
 	},
-	"gitHead": "8d7a3b801e0f5f88438c003e7eee2bd45e6aedd0"
+	"gitHead": "edbaeedca5588fb291ea9d4e21a4ab2bf41fc9cb"
 }