@markuplint/ml-core 5.0.0-alpha.0 → 5.0.0-alpha.2

package/lib/ml-dom/token/token.d.ts

@@ -2,7 +2,7 @@ import type { MLASTToken } from '@markuplint/ml-ast';
 /**
  * Represents a single token in the markuplint AST.
  * Wraps an AST token with positional information (line, column, offset)
- * and provides both raw and fixed string representations.
+ * and provides the raw string representation.
  *
  * @template A - The AST token type this token wraps
  */
@@ -40,12 +40,6 @@ export declare class MLToken<A extends MLASTToken = MLASTToken> {
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
      */
     get endOffset(): number;
-    /**
-     * The fixed (potentially modified) string content of this token.
-     *
-     * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     */
-    get fixed(): string;
     /**
      * The original raw string content of this token from the source.
      *
@@ -71,19 +65,10 @@ export declare class MLToken<A extends MLASTToken = MLASTToken> {
      */
     get startOffset(): number;
     /**
-     * Replaces the fixed content of this token with the given string,
-     * used when applying lint fixes.
-     *
-     * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     * @param raw - The new string content to set as the fixed value
-     */
-    fix(raw: string): void;
-    /**
-     * Returns the string representation of this token.
+     * Returns the raw string representation of this token.
      *
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     * @param fixed - When true, returns the fixed content; otherwise returns the original raw content
      * @returns The string content of this token
      */
-    toString(fixed?: boolean): string;
+    toString(): string;
 }

package/lib/ml-dom/token/token.js

@@ -2,12 +2,11 @@ import { getEndCol, getEndLine } from '@markuplint/parser-utils/location';
 /**
  * Represents a single token in the markuplint AST.
  * Wraps an AST token with positional information (line, column, offset)
- * and provides both raw and fixed string representations.
+ * and provides the raw string representation.
  *
  * @template A - The AST token type this token wraps
  */
 export class MLToken {
-    #fixed;
     #raw;
     /**
      * The unique identifier for this token.
@@ -25,7 +24,6 @@ export class MLToken {
     constructor(astToken) {
         this._astToken = astToken;
         this.#raw = astToken.raw;
-        this.#fixed = astToken.raw;
         this.uuid = astToken.uuid;
     }
     /**
@@ -34,7 +32,7 @@ export class MLToken {
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
      */
     get endCol() {
-        return getEndCol(this.fixed, this.startCol);
+        return getEndCol(this.raw, this.startCol);
     }
     /**
      * The ending line number (1-based) of this token in the source.
@@ -42,7 +40,7 @@ export class MLToken {
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
      */
     get endLine() {
-        return getEndLine(this.fixed, this.startLine);
+        return getEndLine(this.raw, this.startLine);
     }
     /**
      * The ending character offset (0-based) of this token in the source.
@@ -50,15 +48,7 @@ export class MLToken {
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
      */
     get endOffset() {
-        return this.startOffset + this.fixed.length;
-    }
-    /**
-     * The fixed (potentially modified) string content of this token.
-     *
-     * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     */
-    get fixed() {
-        return this.#fixed;
+        return this.startOffset + this.raw.length;
     }
     /**
      * The original raw string content of this token from the source.
@@ -93,23 +83,12 @@ export class MLToken {
         return this._astToken.offset;
     }
     /**
-     * Replaces the fixed content of this token with the given string,
-     * used when applying lint fixes.
+     * Returns the raw string representation of this token.
      *
      * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     * @param raw - The new string content to set as the fixed value
-     */
-    fix(raw) {
-        this.#fixed = raw;
-    }
-    /**
-     * Returns the string representation of this token.
-     *
-     * @implements `@markuplint/ml-core` API: `MLDOMToken`
-     * @param fixed - When true, returns the fixed content; otherwise returns the original raw content
      * @returns The string content of this token
      */
-    toString(fixed = false) {
-        return fixed ? this.#fixed : this.#raw;
+    toString() {
+        return this.#raw;
     }
 }

package/lib/ml-rule/index.d.ts

@@ -1,3 +1,4 @@
 export * from './create-rule.js';
 export * from './ml-rule.js';
+export * from './rule-fixer.js';
 export * from './types.js';

package/lib/ml-rule/index.js

@@ -1,3 +1,4 @@
 export * from './create-rule.js';
 export * from './ml-rule.js';
+export * from './rule-fixer.js';
 export * from './types.js';

package/lib/ml-rule/ml-rule-context.d.ts

@@ -8,40 +8,12 @@ export declare class MLRuleContext<T extends RuleConfigValue, O extends PlainDat
     readonly locale: string;
     readonly translate: Translator;
     constructor(document: MLDocument<T, O>, locale: LocaleSet);
-    get reports(): ({
-        message: string;
-        line: number;
-        col: number;
-        raw: string;
-    } | {
-        message: string;
-        scope: import("@markuplint/ml-config").Scope<T, O>;
-    } | {
-        message: string;
-        scope: import("@markuplint/ml-config").Scope<T, O>;
-        line: number;
-        col: number;
-        raw: string;
-    })[];
+    get reports(): Report<T, O>[];
     provide(): {
         document: MLDocument<T, O>;
         translate: Translator;
         t: Translator;
-        reports: ({
-            message: string;
-            line: number;
-            col: number;
-            raw: string;
-        } | {
-            message: string;
-            scope: import("@markuplint/ml-config").Scope<T, O>;
-        } | {
-            message: string;
-            scope: import("@markuplint/ml-config").Scope<T, O>;
-            line: number;
-            col: number;
-            raw: string;
-        })[];
+        reports: Report<T, O>[];
         report: {
             (report: Report<T, O>): undefined;
             (report: CheckerReport<T, O>): boolean;

package/lib/ml-rule/ml-rule-context.js

@@ -3,6 +3,7 @@ export class MLRuleContext {
     document;
     locale;
     #reports = [];
+    #reportKeys = new Set();
     translate;
     constructor(
     // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
@@ -12,10 +13,7 @@ export class MLRuleContext {
         this.locale = locale.locale;
     }
     get reports() {
-        return this.#reports.map(report => ({
-            ...report,
-            message: finish(report.message, this.locale),
-        }));
+        return this.#reports;
     }
     provide() {
         return {
@@ -38,8 +36,13 @@ export class MLRuleContext {
         this._push(report);
     }
     _push(report) {
-        if (!this.#reports.some(r => is(r, report))) {
-            this.#reports.push(report);
+        const key = reportKey(report);
+        if (!this.#reportKeys.has(key)) {
+            this.#reportKeys.add(key);
+            this.#reports.push({
+                ...report,
+                message: finish(report.message, this.locale),
+            });
         }
     }
 }
@@ -51,15 +54,12 @@ function finish(message, locale = 'en') {
     }
     return message;
 }
-function is(r1, r2) {
-    if ('col' in r1 && 'col' in r2) {
-        return r1.col === r2.col && r1.line === r2.line && r1.message === r2.message && r1.raw === r2.raw;
+function reportKey(report) {
+    if ('col' in report && report.col != null) {
+        return `${report.line}:${report.col}:${report.message}:${report.raw}`;
     }
-    if ('scope' in r1) {
-        if (!('scope' in r2)) {
-            return false;
-        }
-        return r1.scope === r2.scope && r1.message === r2.message;
+    if ('scope' in report) {
+        return `${report.scope.startLine}:${report.scope.startCol}:${report.message}`;
     }
-    return false;
+    return report.message;
 }

package/lib/ml-rule/ml-rule.d.ts

@@ -12,7 +12,7 @@ import type { GlobalRuleInfo, SpecConformance, PlainData, Rule, RuleConfigValue,
 export declare class MLRule<T extends RuleConfigValue, O extends PlainData = undefined> {
     #private;
     /**
-     * For virtual rules, the name of the base rule whose verify/fix logic is reused.
+     * For virtual rules, the name of the base rule whose verify logic is reused.
      * When set, violations report this as `ruleId` for backwards compatibility.
      */
     readonly baseRuleId?: string;
@@ -37,24 +37,19 @@ export declare class MLRule<T extends RuleConfigValue, O extends PlainData = und
         readonly groupName?: string;
     });
     /**
-     * Creates a virtual rule that reuses this rule's verify/fix logic
+     * Creates a virtual rule that reuses this rule's verify logic
      * under a different name (alias). Used by named nodeRules to produce
      * independent rule instances that can be individually configured.
      *
      * @param aliasName - The alias name (must contain `/`)
      * @param options - Override options for the virtual rule
-     * @returns A new MLRule instance sharing the same verify/fix logic
+     * @returns A new MLRule instance sharing the same verify logic
      */
     createAlias(aliasName: string, options?: {
         readonly defaultSeverity?: Severity;
         readonly specConformance?: SpecConformance;
         readonly groupName?: string;
     }): MLRule<T, O>;
-    /**
-     * The following getter is unused internally,
-     * only for extending from 3rd party library
-     */
-    protected get f(): RuleSeed<T, O>['fix'];
     /**
      * The following getter is unused internally,
      * only for extending from 3rd party library
@@ -78,12 +73,13 @@ export declare class MLRule<T extends RuleConfigValue, O extends PlainData = und
      */
     optimizeOption(configSettings: Rule<T, O> | null | undefined): RuleInfo<T, O>;
     /**
-     * Executes this rule's verify (and optionally fix) function against a document,
+     * Executes this rule's verify function against a document,
      * then collects and returns the resulting violations.
+     * When `fix` is true, fix callbacks on reports are executed to produce {@link FixData}.
      *
      * @param document - The parsed document to verify
      * @param locale - The locale set for translating violation messages
-     * @param fix - Whether to also run the fix function
+     * @param fix - Whether to execute fix callbacks and attach FixData to violations
      * @returns An array of violations found by this rule
      */
     verify(document: MLDocument<T, O>, locale: LocaleSet, fix: boolean): Promise<Violation[]>;

package/lib/ml-rule/ml-rule.js

@@ -1,7 +1,9 @@
-import { deleteUndefProp } from '@markuplint/ml-config';
 // @ts-ignore
 import { isPlainObject } from 'is-plain-object';
+import { RuleFixer } from './rule-fixer.js';
 import { MLRuleContext } from './ml-rule-context.js';
+// Stateless — safe to share across all rule instances
+const sharedFixer = new RuleFixer();
 /**
  * Represents a single markuplint rule that can verify documents and report violations.
  *
@@ -10,14 +12,13 @@ import { MLRuleContext } from './ml-rule-context.js';
  */
 export class MLRule {
     /**
-     * For virtual rules, the name of the base rule whose verify/fix logic is reused.
+     * For virtual rules, the name of the base rule whose verify logic is reused.
      * When set, violations report this as `ruleId` for backwards compatibility.
      */
     baseRuleId;
     defaultOptions;
     defaultSeverity;
     defaultValue;
-    #f;
     /**
      * For multi-entry named nodeRules, the group name shared by all derived virtual rules.
      * Allows `rules["groupName"]: false` to disable all rules in the group.
@@ -42,16 +43,15 @@ export class MLRule {
         this.defaultValue = (o.defaultValue === undefined ? true : o.defaultValue);
         this.defaultOptions = o.defaultOptions;
         this.#v = o.verify;
-        this.#f = o.fix;
     }
     /**
-     * Creates a virtual rule that reuses this rule's verify/fix logic
+     * Creates a virtual rule that reuses this rule's verify logic
      * under a different name (alias). Used by named nodeRules to produce
      * independent rule instances that can be individually configured.
      *
      * @param aliasName - The alias name (must contain `/`)
      * @param options - Override options for the virtual rule
-     * @returns A new MLRule instance sharing the same verify/fix logic
+     * @returns A new MLRule instance sharing the same verify logic
      */
     createAlias(aliasName, options) {
         return new MLRule({
@@ -63,16 +63,8 @@ export class MLRule {
             defaultValue: this.defaultValue,
             defaultOptions: this.defaultOptions,
             verify: this.#v,
-            fix: this.#f,
         });
     }
-    /**
-     * The following getter is unused internally,
-     * only for extending from 3rd party library
-     */
-    get f() {
-        return this.#f;
-    }
     /**
      * The following getter is unused internally,
      * only for extending from 3rd party library
@@ -135,12 +127,13 @@ export class MLRule {
         };
     }
     /**
-     * Executes this rule's verify (and optionally fix) function against a document,
+     * Executes this rule's verify function against a document,
      * then collects and returns the resulting violations.
+     * When `fix` is true, fix callbacks on reports are executed to produce {@link FixData}.
      *
      * @param document - The parsed document to verify
      * @param locale - The locale set for translating violation messages
-     * @param fix - Whether to also run the fix function
+     * @param fix - Whether to execute fix callbacks and attach FixData to violations
      * @returns An array of violations found by this rule
      */
     async verify(
@@ -150,13 +143,19 @@ export class MLRule {
         const context = new MLRuleContext(document, locale);
         const providableContext = context.provide();
         await this.#v(providableContext);
-        if (this.#f && fix) {
-            await this.#f(providableContext);
-        }
         const ruleId = this.baseRuleId ?? this.name;
         // Only include name and specConformance for virtual rules (named nodeRules)
         const aliasName = this.baseRuleId ? this.name : undefined;
-        const violation = context.reports.map(report => {
+        const violations = context.reports.map(report => {
+            // Execute fix callback if fix mode is enabled
+            let fixData;
+            if (fix && report.fix) {
+                const edits = report.fix(sharedFixer);
+                const editArray = Array.isArray(edits) ? edits : [edits];
+                if (editArray.length > 0) {
+                    fixData = { edits: editArray };
+                }
+            }
             if ('scope' in report) {
                 let line = report.scope.startLine;
                 let col = report.scope.startCol;
@@ -166,36 +165,36 @@ export class MLRule {
                     col = report.col;
                     raw = report.raw;
                 }
-                const violation = {
+                return {
                     severity: report.scope.rule.severity,
                     message: report.message,
                     line,
                     col,
                     raw,
                     ruleId,
-                    name: aliasName,
-                    specConformance: this.specConformance,
-                    reason: report.scope.rule.reason ?? document.rule.reason,
+                    ...(aliasName != null && { name: aliasName }),
+                    ...(this.specConformance != null && { specConformance: this.specConformance }),
+                    ...((report.scope.rule.reason ?? document.rule.reason)
+                        ? { reason: report.scope.rule.reason ?? document.rule.reason }
+                        : {}),
+                    ...(fixData != null && { fix: fixData }),
                 };
-                deleteUndefProp(violation);
-                return violation;
             }
-            const violation = {
+            return {
                 severity: document.rule.severity,
                 message: report.message,
                 line: report.line,
                 col: report.col,
                 raw: report.raw,
                 ruleId,
-                name: aliasName,
-                specConformance: this.specConformance,
-                reason: document.rule.reason,
+                ...(aliasName != null && { name: aliasName }),
+                ...(this.specConformance != null && { specConformance: this.specConformance }),
+                ...(document.rule.reason ? { reason: document.rule.reason } : {}),
+                ...(fixData != null && { fix: fixData }),
             };
-            deleteUndefProp(violation);
-            return violation;
         });
         document.setRule(null);
-        return violation;
+        return violations;
     }
     _optimize(rules, ruleName) {
         const rule = (rules?.[ruleName] ?? false);

package/lib/ml-rule/rule-fixer.d.ts

@@ -0,0 +1,25 @@
+import type { IRuleFixer, TextEdit } from '@markuplint/ml-config';
+/**
+ * Stateless implementation of {@link IRuleFixer}.
+ * Provides helper methods for building {@link TextEdit} objects
+ * inside rule fix callbacks.
+ */
+export declare class RuleFixer implements IRuleFixer {
+    replaceText(token: {
+        readonly startOffset: number;
+        readonly raw: string;
+    }, text: string): TextEdit;
+    replaceRange(range: readonly [number, number], text: string): TextEdit;
+    insertBefore(token: {
+        readonly startOffset: number;
+    }, text: string): TextEdit;
+    insertAfter(token: {
+        readonly startOffset: number;
+        readonly raw: string;
+    }, text: string): TextEdit;
+    remove(token: {
+        readonly startOffset: number;
+        readonly raw: string;
+    }): TextEdit;
+    removeRange(range: readonly [number, number]): TextEdit;
+}

package/lib/ml-rule/rule-fixer.js

@@ -0,0 +1,32 @@
+/**
+ * Stateless implementation of {@link IRuleFixer}.
+ * Provides helper methods for building {@link TextEdit} objects
+ * inside rule fix callbacks.
+ */
+export class RuleFixer {
+    replaceText(token, text) {
+        return {
+            range: [token.startOffset, token.startOffset + token.raw.length],
+            text,
+        };
+    }
+    replaceRange(range, text) {
+        return { range, text };
+    }
+    insertBefore(token, text) {
+        return { range: [token.startOffset, token.startOffset], text };
+    }
+    insertAfter(token, text) {
+        const end = token.startOffset + token.raw.length;
+        return { range: [end, end], text };
+    }
+    remove(token) {
+        return {
+            range: [token.startOffset, token.startOffset + token.raw.length],
+            text: '',
+        };
+    }
+    removeRange(range) {
+        return { range, text: '' };
+    }
+}

package/lib/ml-rule/types.d.ts

@@ -3,7 +3,7 @@ import type { Attr, Element } from '../ml-dom/index.js';
 import type { Translator } from '@markuplint/i18n';
 import type { PlainData, Report, RuleConfigValue, Severity } from '@markuplint/ml-config';
 /**
- * The definition of a markuplint rule, including verification logic, optional fix logic,
+ * The definition of a markuplint rule, including verification logic
  * and default configuration values.
  *
  * @template T - The type of the rule's configuration value (defaults to boolean)
@@ -17,7 +17,6 @@ export type RuleSeed<T extends RuleConfigValue = boolean, O extends PlainData =
     readonly defaultValue?: T;
     readonly defaultOptions?: O;
     verify(context: ReturnType<MLRuleContext<T, O>['provide']>): void | Promise<void>;
-    fix?(context: ReturnType<MLRuleContext<T, O>['provide']>): void | Promise<void>;
 };
 /**
  * A generic checker function that produces a violation report from parameters.

package/lib/test/index.d.ts

@@ -1,7 +1,6 @@
 import type { MLElement } from '../ml-dom/node/element.js';
 import type { MLNode } from '../ml-dom/node/node.js';
-import type { MLToken } from '../ml-dom/token/token.js';
-import type { MLASTNode, MLASTToken, MLParser } from '@markuplint/ml-ast';
+import type { MLASTNode, MLParser } from '@markuplint/ml-ast';
 import type { Config, PlainData, Pretender, RuleConfigValue } from '@markuplint/ml-config';
 import type { MLMLSpec } from '@markuplint/ml-spec';
 import { MLDocument } from '../ml-dom/node/document.js';
@@ -38,14 +37,6 @@ export declare function createTestDocument<T extends RuleConfigValue = any, O ex
  * @returns A readonly array of all nodes in the parsed document
  */
 export declare function createTestNodeList(sourceCode: string, options?: CreateTestOptions): readonly MLNode<any, any, MLASTNode>[];
-/**
- * Parses markup source code and returns the flat list of tokens.
- *
- * @param sourceCode - The markup source code to parse
- * @param options - Options for parser, config, specs, and pretenders
- * @returns A readonly array of all tokens in the parsed document
- */
-export declare function createTestTokenList(sourceCode: string, options?: CreateTestOptions): readonly MLToken<MLASTToken>[];
 /**
  * Parses markup source code and returns the first element node.
  * Throws if the source does not produce an element as its first node.

package/lib/test/index.js

@@ -35,17 +35,6 @@ export function createTestNodeList(sourceCode, options) {
     const document = createTestDocument(sourceCode, options);
     return document.nodeList;
 }
-/**
- * Parses markup source code and returns the flat list of tokens.
- *
- * @param sourceCode - The markup source code to parse
- * @param options - Options for parser, config, specs, and pretenders
- * @returns A readonly array of all tokens in the parsed document
- */
-export function createTestTokenList(sourceCode, options) {
-    const document = createTestDocument(sourceCode, options);
-    return document.getTokenList();
-}
 /**
  * Parses markup source code and returns the first element node.
  * Throws if the source does not produce an element as its first node.

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/ml-core",
-	"version": "5.0.0-alpha.0",
+	"version": "5.0.0-alpha.2",
 	"description": "The core module of markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
@@ -30,20 +30,20 @@
 		"./lib/configs.js": "./lib/configs.browser.js"
 	},
 	"dependencies": {
-		"@markuplint/config-presets": "5.0.0-alpha.0",
-		"@markuplint/html-parser": "5.0.0-alpha.0",
-		"@markuplint/html-spec": "5.0.0-alpha.0",
-		"@markuplint/i18n": "5.0.0-alpha.0",
-		"@markuplint/ml-ast": "5.0.0-alpha.0",
-		"@markuplint/ml-config": "5.0.0-alpha.0",
-		"@markuplint/ml-spec": "5.0.0-alpha.0",
-		"@markuplint/parser-utils": "5.0.0-alpha.0",
-		"@markuplint/selector": "5.0.0-alpha.0",
-		"@markuplint/shared": "5.0.0-alpha.0",
+		"@markuplint/config-presets": "5.0.0-alpha.2",
+		"@markuplint/html-parser": "5.0.0-alpha.2",
+		"@markuplint/html-spec": "5.0.0-alpha.2",
+		"@markuplint/i18n": "5.0.0-alpha.2",
+		"@markuplint/ml-ast": "5.0.0-alpha.2",
+		"@markuplint/ml-config": "5.0.0-alpha.2",
+		"@markuplint/ml-spec": "5.0.0-alpha.2",
+		"@markuplint/parser-utils": "5.0.0-alpha.2",
+		"@markuplint/selector": "5.0.0-alpha.2",
+		"@markuplint/shared": "5.0.0-alpha.2",
 		"@types/debug": "4.1.12",
 		"debug": "4.4.3",
 		"is-plain-object": "5.0.0",
 		"type-fest": "5.4.4"
 	},
-	"gitHead": "13dcfc84ec83d87360c720e253383b60767e1b56"
+	"gitHead": "31ccf1e81443ea3f93597d287595211f1823ddcf"
 }