@markuplint/rules 5.0.0-alpha.0 → 5.0.0-alpha.2

package/CHANGELOG.md

@@ -3,6 +3,24 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [5.0.0-alpha.2](https://github.com/markuplint/markuplint/compare/v5.0.0-alpha.1...v5.0.0-alpha.2) (2026-02-23)
+
+### Features
+
+- **rules:** add inline fix callbacks to three rules ([16f9600](https://github.com/markuplint/markuplint/commit/16f9600a1f495046541c195859e218ca6dd61eca))
+
+# [5.0.0-alpha.1](https://github.com/markuplint/markuplint/compare/v5.0.0-alpha.0...v5.0.0-alpha.1) (2026-02-22)
+
+- fix(rules)!: remove complementary from top-level landmark check ([4643651](https://github.com/markuplint/markuplint/commit/4643651d256276ae0473d4756e72277d59398e3d))
+
+### BREAKING CHANGES
+
+- The landmark-roles rule no longer checks that
+  complementary landmarks are top-level. This aligns with axe-core's
+  deprecation of landmark-complementary-is-top-level.
+
+Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
+
 # [5.0.0-alpha.0](https://github.com/markuplint/markuplint/compare/v4.14.1...v5.0.0-alpha.0) (2026-02-20)
 
 ### Bug Fixes

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

@@ -25,21 +25,23 @@ export default createRule({
                 return;
             }
             const quote = attr.startQuote?.raw;
-            if (quote !== quoteList[attr.rule.value]) {
+            const expectedQuote = quoteList[attr.rule.value];
+            if (quote !== expectedQuote) {
                 report({
                     scope: attr,
                     message,
+                    fix: fixer => {
+                        const edits = [];
+                        if (attr.startQuote) {
+                            edits.push(fixer.replaceText(attr.startQuote, expectedQuote));
+                        }
+                        if (attr.endQuote) {
+                            edits.push(fixer.replaceText(attr.endQuote, expectedQuote));
+                        }
+                        return edits;
+                    },
                 });
             }
         });
     },
-    async fix({ document }) {
-        await document.walkOn('Attr', attr => {
-            const quote = quoteList[attr.rule.value];
-            if (quote && attr.startQuote && attr.startQuote.raw !== quote) {
-                attr.startQuote.fix(quote);
-                attr.endQuote?.fix(quote);
-            }
-        });
-    },
 });

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

@@ -39,44 +39,17 @@ export default createRule({
                 }
             }
             if (deny.test(name)) {
+                const nameNode = attr.nameNode;
+                const fixedName = value === 'lower' ? name.toLowerCase() : name.toUpperCase();
                 report({
                     scope: attr,
-                    line: attr.nameNode?.startLine,
-                    col: attr.nameNode?.startCol,
-                    raw: attr.nameNode?.raw,
+                    line: nameNode?.startLine,
+                    col: nameNode?.startCol,
+                    raw: nameNode?.raw,
                     message,
+                    fix: nameNode ? fixer => fixer.replaceText(nameNode, fixedName) : undefined, // eslint-disable-line @typescript-eslint/strict-boolean-expressions
                 });
             }
         });
     },
-    async fix({ document }) {
-        await document.walkOn('Attr', attr => {
-            const el = attr.ownerElement;
-            if (el.isForeignElement || el.elementType !== 'html') {
-                return;
-            }
-            const attrSpecs = getAttrSpecs(el, document.specs);
-            const value = attr.rule.value;
-            /**
-             * Ignore when it has the potential name,
-             * it Interprets `tabIndex` to `tabindex` in JSX for example.
-             */
-            if (attr.nameNode?.raw !== attr.name) {
-                return;
-            }
-            const name = attr.name;
-            if (attrSpecs) {
-                const spec = attrSpecs.find(spec => spec.name === name);
-                if (spec && spec.caseSensitive) {
-                    return;
-                }
-            }
-            if (value === 'lower') {
-                attr.nameNode.fix(name.toLowerCase());
-            }
-            else {
-                attr.nameNode.fix(name.toUpperCase());
-            }
-        });
-    },
 });

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

@@ -22,19 +22,24 @@ export default createRule({
             }
             const ms = el.rule.severity === 'error' ? 'must' : 'should';
             const deny = el.rule.value === 'lower' ? /[A-Z]/ : /[a-z]/;
+            const fixedName = el.rule.value === 'lower' ? el.rawName.toLowerCase() : el.rawName.toUpperCase();
             const message = t(`{0} ${ms} be {1}`, t('{0} of {1}', 'tag names', 'HTML elements'), `${el.rule.value}case`);
             if (deny.test(el.rawName)) {
                 const loc = el.getNameLocation();
+                const nameOffset = loc.offset + el.tagOpenChar.length;
                 report({
                     scope: el,
                     message,
                     line: loc.line,
                     col: loc.col,
                     raw: el.rawName,
+                    fix: fixer => fixer.replaceText({ startOffset: nameOffset, raw: el.rawName }, fixedName),
                 });
             }
             const closeTag = el.closeTag;
-            if (closeTag && deny.test(closeTag.raw)) {
+            if (closeTag && deny.test(closeTag.rawName)) {
+                const closeNameOffset = closeTag.startOffset + el.tagOpenChar.length + '/'.length;
+                const fixedCloseName = el.rule.value === 'lower' ? closeTag.rawName.toLowerCase() : closeTag.rawName.toUpperCase();
                 report({
                     scope: {
                         rule: el.rule,
@@ -43,27 +48,9 @@ export default createRule({
                         raw: closeTag.raw,
                     },
                     message,
+                    fix: fixer => fixer.replaceText({ startOffset: closeNameOffset, raw: closeTag.rawName }, fixedCloseName),
                 });
             }
         });
     },
-    async fix({ document }) {
-        if (document.tagNameCaseSensitive) {
-            return;
-        }
-        await document.walkOn('Element', el => {
-            if (el.isForeignElement || el.elementType !== 'html') {
-                return;
-            }
-            const deny = el.rule.value === 'lower' ? /[A-Z]/ : /[a-z]/;
-            if (deny.test(el.nodeName)) {
-                if (el.rule.value === 'lower') {
-                    el.fixNodeName(el.nodeName.toLowerCase());
-                }
-                else {
-                    el.fixNodeName(el.nodeName.toUpperCase());
-                }
-            }
-        });
-    },
 });

package/lib/index.d.ts

@@ -44,7 +44,7 @@ declare const rules: {
     }>>;
     readonly 'label-has-control': Readonly<import("@markuplint/ml-core").RuleSeed<import("@markuplint/ml-core").RuleConfigValue, undefined>>;
     readonly 'landmark-roles': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {
-        ignoreRoles: (("banner" | "main" | "complementary" | "contentinfo") | "form" | "navigation" | "region")[];
+        ignoreRoles: (("banner" | "main" | "contentinfo") | "complementary" | "form" | "navigation" | "region")[];
         labelEachArea: boolean;
     }>>;
     readonly 'link-types': Readonly<import("@markuplint/ml-core").RuleSeed<boolean, {

package/lib/landmark-roles/index.d.ts

@@ -8,13 +8,13 @@ type Options = {
     labelEachArea: boolean;
 };
 /** Landmark roles that should appear at the top level of the document. */
-type TopLevelRoles = 'banner' | 'main' | 'complementary' | 'contentinfo';
+type TopLevelRoles = 'banner' | 'main' | 'contentinfo';
 /** All recognized ARIA landmark roles for this rule. */
-type Roles = TopLevelRoles | 'form' | 'navigation' | 'region';
+type Roles = TopLevelRoles | 'complementary' | 'form' | 'navigation' | 'region';
 /**
  * Rule that validates proper usage of ARIA landmark roles.
  *
- * Checks that `banner`, `main`, `complementary`, and `contentinfo` landmarks are
+ * Checks that `banner`, `main`, and `contentinfo` landmarks are
  * top-level (not nested inside other landmarks), and that when duplicate landmarks
  * of the same role exist, each has a unique accessible name.
  */

package/lib/landmark-roles/index.js

@@ -11,11 +11,11 @@ const selectors = {
     region: ['[role="region"]', 'section[aria-labelledby]', 'section[aria-label]', 'section[title]'],
 };
 /** Roles that are required to be top-level landmarks per WAI-ARIA practices. */
-const topLevelRoles = ['banner', 'main', 'complementary', 'contentinfo'];
+const topLevelRoles = ['banner', 'main', 'contentinfo'];
 /**
  * Rule that validates proper usage of ARIA landmark roles.
  *
- * Checks that `banner`, `main`, `complementary`, and `contentinfo` landmarks are
+ * Checks that `banner`, `main`, and `contentinfo` landmarks are
  * top-level (not nested inside other landmarks), and that when duplicate landmarks
  * of the same role exist, each has a unique accessible name.
  */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/rules",
-	"version": "5.0.0-alpha.0",
+	"version": "5.0.0-alpha.2",
 	"description": "Built-in rules of markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
@@ -27,12 +27,12 @@
 		"./lib/permitted-contents/debug.js": "./lib/permitted-contents/debug.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/html-spec": "5.0.0-alpha.0",
-		"@markuplint/ml-core": "5.0.0-alpha.0",
-		"@markuplint/ml-spec": "5.0.0-alpha.0",
-		"@markuplint/selector": "5.0.0-alpha.0",
-		"@markuplint/shared": "5.0.0-alpha.0",
-		"@markuplint/types": "5.0.0-alpha.0",
+		"@markuplint/html-spec": "5.0.0-alpha.2",
+		"@markuplint/ml-core": "5.0.0-alpha.2",
+		"@markuplint/ml-spec": "5.0.0-alpha.2",
+		"@markuplint/selector": "5.0.0-alpha.2",
+		"@markuplint/shared": "5.0.0-alpha.2",
+		"@markuplint/types": "5.0.0-alpha.2",
 		"@mdn/browser-compat-data": "^7.3.2",
 		"@types/debug": "4.1.12",
 		"ansi-colors": "4.1.3",
@@ -42,5 +42,5 @@
 		"image-size": "^2.0.2",
 		"type-fest": "5.4.4"
 	},
-	"gitHead": "13dcfc84ec83d87360c720e253383b60767e1b56"
+	"gitHead": "31ccf1e81443ea3f93597d287595211f1823ddcf"
 }