@markuplint/ml-core 5.0.0-rc.1 → 5.0.0-rc.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-rc.2](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.1...v5.0.0-rc.2) (2026-04-15)
+
+### Bug Fixes
+
+- **ml-core:** propagate base rule disable to virtual rules in nodeRules/childNodeRules ([06fa269](https://github.com/markuplint/markuplint/commit/06fa2695d43671adc1ebcadf1304aca44fd10919)), closes [#3578](https://github.com/markuplint/markuplint/issues/3578)
+
+- build!: remove ESLint and replace with oxlint ([1e0a337](https://github.com/markuplint/markuplint/commit/1e0a337707f76b903b16beeeb8c4d4fc0d8fc9e4))
+- feat(ml-core)!: remove deprecated getIndent function ([7ab1a2d](https://github.com/markuplint/markuplint/commit/7ab1a2dae8f36002f4e5f3108a400bb9095db319))
+
+### Features
+
+- **ml-core:** propagate option overrides to virtual rules in nodeRules/childNodeRules ([9cb9521](https://github.com/markuplint/markuplint/commit/9cb9521fc1303c132f547ffd1e8c82b3b9099bef))
+
+### BREAKING CHANGES
+
+- ESLint is no longer used. Use oxlint instead.
+- getIndent() has been removed from the public API.
+
 # [5.0.0-rc.1](https://github.com/markuplint/markuplint/compare/v5.0.0-rc.0...v5.0.0-rc.1) (2026-03-27)
 
 ### Bug Fixes

package/lib/index.d.ts

@@ -5,7 +5,6 @@ export { enableDebug } from './debug.js';
 export { computeCursorOffset } from './cursor-offset.js';
 export { applyFixes } from './fix-applier.js';
 export type { FixResult } from './fix-applier.js';
-export { getIndent } from './ml-dom/helper/get-indent.js';
 export * from './convert-ruleset.js';
 export * from './ml-core.js';
 export * from './ml-dom/index.js';

package/lib/index.js

@@ -3,7 +3,6 @@ export { Ruleset } from './ruleset/index.js';
 export { enableDebug } from './debug.js';
 export { computeCursorOffset } from './cursor-offset.js';
 export { applyFixes } from './fix-applier.js';
-export { getIndent } from './ml-dom/helper/get-indent.js';
 export * from './convert-ruleset.js';
 export * from './ml-core.js';
 export * from './ml-dom/index.js';

package/lib/ml-core.js

@@ -67,6 +67,8 @@ export class MLCore {
             rules: resolvedRules,
             nodeRules: nodeRuleResult.transformedNodeRules,
             childNodeRules: childNodeRuleResult.transformedNodeRules,
+            baseRuleToVirtualNames: buildBaseRuleToVirtualNames(namedRulesResult.virtualRules),
+            mappingErrors: [],
         };
         this.#disabledNamespaces = extractDisabledNamespaces(resolvedRules);
         this.#configErrors.push(...namedRulesResult.errors, ...nodeRuleResult.errors, ...childNodeRuleResult.errors);
@@ -100,7 +102,7 @@ export class MLCore {
         this.#locale = locale ?? this.#locale;
         this.#schemas = schemas ?? this.#schemas;
         this.#configErrors = [...(configErrors ?? [])];
-        const baseRules = rules?.slice() ?? this.#rules.filter(r => !r.baseRuleId);
+        const baseRules = rules ? [...rules] : this.#rules.filter(r => !r.baseRuleId);
         // Use pre-expansion originals as fallback when ruleset is not provided
         const incomingNodeRules = ruleset?.nodeRules ?? this.#originalNodeRules;
         const incomingChildNodeRules = ruleset?.childNodeRules ?? this.#originalChildNodeRules;
@@ -123,6 +125,8 @@ export class MLCore {
             rules: resolvedRules,
             nodeRules: nodeRuleResult.transformedNodeRules,
             childNodeRules: childNodeRuleResult.transformedNodeRules,
+            baseRuleToVirtualNames: buildBaseRuleToVirtualNames(namedRulesResult.virtualRules),
+            mappingErrors: [],
         };
         this.#disabledNamespaces = extractDisabledNamespaces(resolvedRules);
         this.#configErrors.push(...namedRulesResult.errors, ...nodeRuleResult.errors, ...childNodeRuleResult.errors);
@@ -243,6 +247,9 @@ export class MLCore {
                 tagNameCaseSensitive: this.#parser.tagNameCaseSensitive,
                 pretenders: this.#pretenders,
             });
+            // Collect errors from rule mapping (e.g., invalid wildcard usage)
+            this.#configErrors.push(...this.#ruleset.mappingErrors);
+            this.#ruleset.mappingErrors.length = 0;
         }
         catch (error) {
             if (error instanceof ParserError) {
@@ -426,6 +433,29 @@ function extractDisabledNamespaces(rules) {
         .filter(([key, value]) => key.endsWith('/*') && value === false)
         .map(([key]) => key.slice(0, -1)); // "a11y/*" → "a11y/"
 }
+/**
+ * Builds a mapping from base rule names to virtual rule names.
+ * Used by nodeRules/childNodeRules to propagate settings (especially `false`)
+ * to virtual rules created by NamedRuleGroups.
+ */
+function buildBaseRuleToVirtualNames(
+// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
+virtualRules) {
+    const map = new Map();
+    for (const vRule of virtualRules) {
+        if (!vRule.baseRuleId) {
+            continue;
+        }
+        const list = map.get(vRule.baseRuleId);
+        if (list) {
+            list.push(vRule.name);
+        }
+        else {
+            map.set(vRule.baseRuleId, [vRule.name]);
+        }
+    }
+    return map;
+}
 /**
  * Collects all `FixData` from violations that have a fix callback result.
  *

package/lib/ml-dom/node/document.js

@@ -2399,6 +2399,24 @@ export class MLDocument extends MLParentNode {
                     if (rule == null) {
                         continue;
                     }
+                    // Namespace wildcard (e.g., "a11y/*": false)
+                    if (ruleName.endsWith('/*')) {
+                        if (rule !== false) {
+                            ruleset.mappingErrors.push(new Error(`Namespace wildcard "${ruleName}" only accepts false; use a specific rule name for options`));
+                            continue;
+                        }
+                        const nsPrefix = ruleName.slice(0, -1); // "a11y/*" → "a11y/"
+                        for (const key of Object.keys(ruleset.rules)) {
+                            if (key.startsWith(nsPrefix)) {
+                                ruleMapper.set(node, key, {
+                                    from: 'nodeRules',
+                                    specificity: matches.specificity,
+                                    rule: false,
+                                });
+                            }
+                        }
+                        continue;
+                    }
                     const convertedRule = exchangeValueOnRule(rule, matches.data ?? {});
                     if (convertedRule === undefined) {
                         continue;
@@ -2411,6 +2429,19 @@ export class MLDocument extends MLParentNode {
                         specificity: matches.specificity,
                         rule: mergedRule,
                     });
+                    // Propagate to virtual rules that wrap this base rule
+                    const virtualNames = ruleset.baseRuleToVirtualNames.get(ruleName);
+                    if (virtualNames) {
+                        for (const vName of virtualNames) {
+                            const vGlobalRule = ruleset.rules[vName];
+                            const vMergedRule = vGlobalRule == null ? convertedRule : mergeRule(vGlobalRule, convertedRule);
+                            ruleMapper.set(node, vName, {
+                                from: 'nodeRules',
+                                specificity: matches.specificity,
+                                rule: vMergedRule,
+                            });
+                        }
+                    }
                 }
             }
             // overwrite rule to child node
@@ -2442,6 +2473,26 @@ export class MLDocument extends MLParentNode {
                         if (rule == null) {
                             continue;
                         }
+                        // Namespace wildcard (e.g., "a11y/*": false)
+                        if (ruleName.endsWith('/*')) {
+                            if (rule !== false) {
+                                ruleset.mappingErrors.push(new Error(`Namespace wildcard "${ruleName}" only accepts false; use a specific rule name for options`));
+                                continue;
+                            }
+                            const nsPrefix = ruleName.slice(0, -1); // "a11y/*" → "a11y/"
+                            for (const key of Object.keys(ruleset.rules)) {
+                                if (key.startsWith(nsPrefix)) {
+                                    for (const descendant of targetDescendants) {
+                                        ruleMapper.set(descendant, key, {
+                                            from: 'childNodeRules',
+                                            specificity: matches.specificity,
+                                            rule: false,
+                                        });
+                                    }
+                                }
+                            }
+                            continue;
+                        }
                         const convertedRule = exchangeValueOnRule(rule, matches.data ?? {});
                         if (convertedRule === undefined) {
                             continue;
@@ -2456,6 +2507,21 @@ export class MLDocument extends MLParentNode {
                                 rule: mergedRule,
                             });
                         }
+                        // Propagate to virtual rules that wrap this base rule
+                        const virtualNames = ruleset.baseRuleToVirtualNames.get(ruleName);
+                        if (virtualNames) {
+                            for (const vName of virtualNames) {
+                                const vGlobalRule = ruleset.rules[vName];
+                                const vMergedRule = vGlobalRule == null ? convertedRule : mergeRule(vGlobalRule, convertedRule);
+                                for (const descendant of targetDescendants) {
+                                    ruleMapper.set(descendant, vName, {
+                                        from: 'childNodeRules',
+                                        specificity: matches.specificity,
+                                        rule: vMergedRule,
+                                    });
+                                }
+                            }
+                        }
                     }
                 }
             }

package/lib/ruleset/index.d.ts

@@ -4,14 +4,26 @@ import type { ChildNodeRule, Config, NodeRule, Rules } from '@markuplint/ml-conf
  * rule overrides extracted from a markuplint {@link Config}.
  */
 export declare class Ruleset {
+    /**
+     * Maps base rule names to their virtual rule names created by NamedRuleGroups.
+     * For example, if `a11y/wai-aria` wraps `wai-aria`, this maps `"wai-aria"` → `["a11y/wai-aria"]`.
+     * Used by nodeRules/childNodeRules to propagate settings to virtual rules.
+     */
+    readonly baseRuleToVirtualNames: ReadonlyMap<string, readonly string[]>;
     /** Rule overrides that apply to child nodes matching specific selectors */
     readonly childNodeRules: readonly ChildNodeRule[];
+    /**
+     * Errors collected during rule mapping (e.g., invalid wildcard usage).
+     * Consumed by MLCore and reported as config-error violations.
+     */
+    readonly mappingErrors: Error[];
     /** Rule overrides that apply to nodes matching specific selectors */
     readonly nodeRules: readonly NodeRule[];
     /** The global rule definitions */
     readonly rules: Rules;
     /**
      * @param config - The markuplint configuration to extract rules from
+     * @param baseRuleToVirtualNames - Mapping from base rule names to virtual rule names
      */
-    constructor(config: Config);
+    constructor(config: Config, baseRuleToVirtualNames?: ReadonlyMap<string, readonly string[]>);
 }

package/lib/ruleset/index.js

@@ -3,18 +3,31 @@
  * rule overrides extracted from a markuplint {@link Config}.
  */
 export class Ruleset {
+    /**
+     * Maps base rule names to their virtual rule names created by NamedRuleGroups.
+     * For example, if `a11y/wai-aria` wraps `wai-aria`, this maps `"wai-aria"` → `["a11y/wai-aria"]`.
+     * Used by nodeRules/childNodeRules to propagate settings to virtual rules.
+     */
+    baseRuleToVirtualNames;
     /** Rule overrides that apply to child nodes matching specific selectors */
     childNodeRules;
+    /**
+     * Errors collected during rule mapping (e.g., invalid wildcard usage).
+     * Consumed by MLCore and reported as config-error violations.
+     */
+    mappingErrors = [];
     /** Rule overrides that apply to nodes matching specific selectors */
     nodeRules;
     /** The global rule definitions */
     rules;
     /**
      * @param config - The markuplint configuration to extract rules from
+     * @param baseRuleToVirtualNames - Mapping from base rule names to virtual rule names
      */
-    constructor(config) {
+    constructor(config, baseRuleToVirtualNames) {
         this.rules = config.rules ?? {};
         this.nodeRules = config.nodeRules ?? [];
         this.childNodeRules = config.childNodeRules ?? [];
+        this.baseRuleToVirtualNames = baseRuleToVirtualNames ?? new Map();
     }
 }

package/package.json

@@ -1,8 +1,12 @@
 {
 	"name": "@markuplint/ml-core",
-	"version": "5.0.0-rc.1",
+	"version": "5.0.0-rc.2",
 	"description": "The core module of markuplint",
-	"repository": "git@github.com:markuplint/markuplint.git",
+	"repository": {
+		"type": "git",
+		"url": "https://github.com/markuplint/markuplint.git",
+		"directory": "packages/@markuplint/ml-core"
+	},
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
 	"license": "MIT",
 	"engines": {
@@ -30,20 +34,20 @@
 		"./lib/configs.js": "./lib/configs.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/config-presets": "5.0.0-rc.1",
-		"@markuplint/html-parser": "5.0.0-rc.1",
-		"@markuplint/html-spec": "5.0.0-rc.1",
-		"@markuplint/i18n": "5.0.0-rc.1",
-		"@markuplint/ml-ast": "5.0.0-rc.1",
-		"@markuplint/ml-config": "5.0.0-rc.1",
-		"@markuplint/ml-spec": "5.0.0-rc.1",
-		"@markuplint/parser-utils": "5.0.0-rc.1",
-		"@markuplint/selector": "5.0.0-rc.1",
-		"@markuplint/shared": "5.0.0-rc.1",
+		"@markuplint/config-presets": "5.0.0-rc.2",
+		"@markuplint/html-parser": "5.0.0-rc.2",
+		"@markuplint/html-spec": "5.0.0-rc.2",
+		"@markuplint/i18n": "5.0.0-rc.2",
+		"@markuplint/ml-ast": "5.0.0-rc.2",
+		"@markuplint/ml-config": "5.0.0-rc.2",
+		"@markuplint/ml-spec": "5.0.0-rc.2",
+		"@markuplint/parser-utils": "5.0.0-rc.2",
+		"@markuplint/selector": "5.0.0-rc.2",
+		"@markuplint/shared": "5.0.0-rc.2",
 		"@types/debug": "4.1.13",
 		"debug": "4.4.3",
 		"is-plain-object": "5.0.0",
 		"type-fest": "5.5.0"
 	},
-	"gitHead": "0d6b4324d9a7d6b9e1ba57d4a57e45d36975cba9"
+	"gitHead": "e43763858d9234c417053becc73dbd088c1e7ea6"
 }

package/lib/ml-dom/helper/get-indent.d.ts

@@ -1,20 +0,0 @@
-import type { MLNode } from '../node/node.js';
-import type { MLText } from '../node/text.js';
-/**
- * Computes the indentation preceding the given node by analyzing
- * the whitespace in adjacent text nodes.
- *
- * @deprecated
- * @param node - The node whose indentation to determine
- * @returns An indentation object describing the whitespace, or null if no indentation is found
- */
-export declare function getIndent(node: MLNode<any, any>): MLDOMIndentation | null;
-declare class MLDOMIndentation {
-    #private;
-    readonly line: number;
-    constructor(originTextNode: MLText<any, any>, raw: string, line: number, parentNode: MLNode<any, any>);
-    get raw(): string;
-    get type(): 'tab' | 'space' | 'mixed' | 'none';
-    get width(): number;
-}
-export {};

package/lib/ml-dom/helper/get-indent.js

@@ -1,90 +0,0 @@
-/**
- * Computes the indentation preceding the given node by analyzing
- * the whitespace in adjacent text nodes.
- *
- * @deprecated
- * @param node - The node whose indentation to determine
- * @returns An indentation object describing the whitespace, or null if no indentation is found
- */
-export function getIndent(
-// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-node) {
-    const prevToken = node.prevToken;
-    if (!prevToken) {
-        return null;
-    }
-    if (node.is(node.TEXT_NODE)) {
-        if (node.isRawTextElementContent()) {
-            return null;
-        }
-        const matched = node.raw.match(/^([\t\v\f\r \u00A0\u1680\u2000-\u200A\u2028\u2029\u202F\u205F\u3000\uFEFF]*\n\s*)\S+/);
-        if (matched) {
-            const spaces = matched[1];
-            if (spaces) {
-                const spaceLines = spaces.split(/\r?\n/);
-                const line = spaceLines.length + node.startLine - 1;
-                const lastSpace = spaceLines.pop();
-                if (lastSpace != null) {
-                    return new MLDOMIndentation(node, lastSpace, line, node);
-                }
-            }
-        }
-        return null;
-    }
-    if (!prevToken.is(node.TEXT_NODE)) {
-        return null;
-    }
-    // One or more newlines and zero or more spaces or tabs.
-    // Or, If textNode is first token and that is filled spaces, tabs and newlines only.
-    const matched = isFirstToken(prevToken)
-        ? prevToken.raw.match(/^(?:[\t ]*\r?\n)*([\t ]*)$/)
-        : prevToken.raw.match(/\r?\n([\t ]*)$/);
-    if (matched) {
-        // Spaces will include empty string.
-        const spaces = matched[1];
-        if (spaces != null) {
-            return new MLDOMIndentation(prevToken, spaces, node.startLine, node);
-        }
-    }
-    return null;
-}
-class MLDOMIndentation {
-    #raw;
-    line;
-    #node;
-    #parent;
-    constructor(
-    // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-    originTextNode, raw, line, 
-    // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-    parentNode) {
-        this.line = line;
-        this.#node = originTextNode;
-        this.#parent = parentNode;
-        this.#raw = raw;
-    }
-    get raw() {
-        if (!this.#parent.is(this.#parent.TEXT_NODE) && this.line !== this.#node.endLine) {
-            return '';
-        }
-        return this.#raw;
-    }
-    get type() {
-        if (!this.#parent.is(this.#parent.TEXT_NODE) && this.line !== this.#node.endLine) {
-            return 'none';
-        }
-        const raw = this.#raw;
-        return raw === '' ? 'none' : /^\t+$/.test(raw) ? 'tab' : /^[^\t]+$/.test(raw) ? 'space' : 'mixed';
-    }
-    get width() {
-        if (!this.#parent.is(this.#parent.TEXT_NODE) && this.line !== this.#node.endLine) {
-            return 0;
-        }
-        return this.#raw.length;
-    }
-}
-function isFirstToken(
-// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-node) {
-    return !node.prevToken;
-}