wikilint 2.4.5 → 2.5.0

package/dist/base.d.ts

@@ -10,18 +10,28 @@ export interface Config {
     readonly variants: string[];
     readonly excludes?: string[];
 }
-export type Severity = 'error' | 'warning';
-export type Rule = 'bold-header' | 'format-leakage' | 'fostered-content' | 'h1' | 'illegal-attr' | 'insecure-style' | 'invalid-gallery' | 'invalid-imagemap' | 'invalid-invoke' | 'lonely-apos' | 'lonely-bracket' | 'lonely-http' | 'nested-link' | 'no-arg' | 'no-duplicate' | 'no-ignored' | 'obsolete-attr' | 'obsolete-tag' | 'parsing-order' | 'pipe-like' | 'table-layout' | 'tag-like' | 'unbalanced-header' | 'unclosed-comment' | 'unclosed-quote' | 'unclosed-table' | 'unescaped' | 'unknown-page' | 'unmatched-tag' | 'unterminated-url' | 'url-encoding' | 'var-anchor' | 'void-ext';
+export declare namespace LintError {
+    type Severity = 'error' | 'warning';
+    type Rule = 'bold-header' | 'format-leakage' | 'fostered-content' | 'h1' | 'illegal-attr' | 'insecure-style' | 'invalid-gallery' | 'invalid-imagemap' | 'invalid-invoke' | 'lonely-apos' | 'lonely-bracket' | 'lonely-http' | 'nested-link' | 'no-arg' | 'no-duplicate' | 'no-ignored' | 'obsolete-attr' | 'obsolete-tag' | 'parsing-order' | 'pipe-like' | 'table-layout' | 'tag-like' | 'unbalanced-header' | 'unclosed-comment' | 'unclosed-quote' | 'unclosed-table' | 'unescaped' | 'unknown-page' | 'unmatched-tag' | 'unterminated-url' | 'url-encoding' | 'var-anchor' | 'void-ext';
+    interface Fix {
+        readonly range: [number, number];
+        text: string;
+    }
+}
 export interface LintError {
-    readonly rule: Rule;
-    readonly message: string;
-    readonly severity: Severity;
-    readonly startIndex: number;
-    readonly endIndex: number;
-    readonly startLine: number;
-    readonly startCol: number;
-    readonly endLine: number;
-    readonly endCol: number;
+    rule: LintError.Rule;
+    message: string;
+    severity: LintError.Severity;
+    startIndex: number;
+    endIndex: number;
+    startLine: number;
+    startCol: number;
+    endLine: number;
+    endCol: number;
+    fix?: LintError.Fix;
+    suggestions?: (LintError.Fix & {
+        desc: string;
+    })[];
 }
 /** 类似Node */
 export interface AstNode {

package/dist/bin/cli.js

@@ -3,10 +3,12 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const fs_1 = require("fs");
 const path_1 = require("path");
+const chalk = require("chalk");
 const Parser = require("../index");
 const man = `
 Available options:
 -c, --config <path or preset config>    Choose parser's configuration
+--fix                                   Automatically fix problems
 -h, --help                              Print available options
 -i, --include                           Parse for inclusion
 -l, --lang                              Choose i18n language
@@ -14,8 +16,8 @@ Available options:
 -s, --strict                            Exit when there is an error or warning
                                         Override -q or --quiet
 -v, --version                           Print package version
-`, preset = new Set(['default', 'zhwiki', 'moegirl', 'llwiki']), { argv } = process, files = [];
-let include = false, quiet = false, strict = false, exit = false, nErr = 0, nWarn = 0, option, config, lang;
+`, preset = new Set((0, fs_1.readdirSync)('./config').filter(file => file.endsWith('.json')).map(file => file.slice(0, -5))), { argv } = process, files = [];
+let include = false, quiet = false, strict = false, exit = false, fixing = false, nErr = 0, nWarn = 0, nFixableErr = 0, nFixableWarn = 0, option, config, lang;
 /**
  * throw if `-c` or `--config` option is incorrect
  * @throws `Error` unrecognized config input
@@ -33,12 +35,12 @@ const throwOnConfig = () => {
  * @param n number of items
  * @param word item name
  */
-const plural = (n, word) => `${n} ${word}${n > 1 ? 's' : ''}`;
+const plural = (n, word) => `${n} ${word}${n === 1 ? '' : 's'}`;
 /**
  * color the severity
  * @param severity problem severity
  */
-const coloredSeverity = (severity) => `\x1B[${severity === 'error' ? 31 : 33}m${severity.padEnd(7)}\x1B[0m`;
+const coloredSeverity = (severity) => chalk[severity === 'error' ? 'red' : 'yellow'](severity.padEnd(7));
 for (let i = 2; i < argv.length; i++) {
     option = argv[i];
     switch (option) {
@@ -47,6 +49,9 @@ for (let i = 2; i < argv.length; i++) {
             config = argv[++i];
             throwOnConfig();
             break;
+        case '--fix':
+            fixing = true;
+            break;
         case '-h':
         case '--help':
             console.log(man);
@@ -105,28 +110,46 @@ if (quiet && strict) {
     console.error('-s or --strict will override -q or --quiet\n');
 }
 for (const file of files) {
-    const wikitext = (0, fs_1.readFileSync)(file, 'utf8');
-    let problems = Parser.parse(wikitext, include).lint();
-    const errors = problems.filter(({ severity }) => severity === 'error'), { length: nLocalErr } = errors, nLocalWarn = problems.length - nLocalErr;
+    let wikitext = (0, fs_1.readFileSync)(file, 'utf8'), problems = Parser.parse(wikitext, include).lint();
+    if (fixing && problems.some(({ fix }) => fix)) {
+        // 倒序修复，跳过嵌套的修复
+        const fixable = problems.map(({ fix }) => fix).filter(Boolean)
+            .sort(({ range: [aFrom, aTo] }, { range: [bFrom, bTo] }) => aTo === bTo ? bFrom - aFrom : bTo - aTo);
+        let start = Infinity;
+        for (const { range: [from, to], text } of fixable) {
+            if (to <= start) {
+                wikitext = `${wikitext.slice(0, from)}${text}${wikitext.slice(to)}`;
+                start = from;
+            }
+        }
+        void fs_1.promises.writeFile(file, wikitext);
+        problems = Parser.parse(wikitext, include).lint();
+    }
+    const errors = problems.filter(({ severity }) => severity === 'error'), fixable = problems.filter(({ fix }) => fix), nLocalErr = errors.length, nLocalWarn = problems.length - nLocalErr, nLocalFixableErr = fixable.filter(({ severity }) => severity === 'error').length, nLocalFixableWarn = fixable.length - nLocalFixableErr;
     if (quiet) {
         problems = errors;
     }
     else {
         nWarn += nLocalWarn;
+        nFixableWarn += nLocalFixableWarn;
     }
     if (problems.length > 0) {
-        console.error('\x1B[4m%s\x1B[0m', (0, path_1.resolve)(file));
-        const { length: maxLineChars } = String(Math.max(...problems.map(({ startLine }) => startLine))), { length: maxColChars } = String(Math.max(...problems.map(({ startCol }) => startCol))), maxMessageChars = Math.max(...problems.map(({ message: { length } }) => length));
+        console.error(`\n${chalk.underline('%s')}`, (0, path_1.resolve)(file));
+        const maxLineChars = String(Math.max(...problems.map(({ startLine }) => startLine))).length, maxColChars = String(Math.max(...problems.map(({ startCol }) => startCol))).length, maxMessageChars = Math.max(...problems.map(({ message: { length } }) => length));
         for (const { rule, message, severity, startLine, startCol } of problems) {
-            console.error(`  \x1B[37m%s:%s\x1B[0m  %s  %s  \x1B[37m%s\x1B[0m`, String(startLine).padStart(maxLineChars), String(startCol).padEnd(maxColChars), coloredSeverity(severity), message.padEnd(maxMessageChars), rule);
+            console.error(`  ${chalk.dim('%s:%s')}  %s  %s  ${chalk.dim('%s')}`, String(startLine).padStart(maxLineChars), String(startCol).padEnd(maxColChars), coloredSeverity(severity), message.padEnd(maxMessageChars), rule);
         }
-        console.error();
     }
     nErr += nLocalErr;
+    nFixableErr += nLocalFixableErr;
     exit ||= Boolean(nLocalErr || strict && nLocalWarn);
 }
 if (nErr || nWarn) {
-    console.error('\x1B[1;31m%s\x1B[0m\n', `✖ ${plural(nErr + nWarn, 'problem')} (${plural(nErr, 'error')}, ${plural(nWarn, 'warning')})`);
+    console.error(chalk.red.bold('%s'), `\n✖ ${plural(nErr + nWarn, 'problem')} (${plural(nErr, 'error')}, ${plural(nWarn, 'warning')})`);
+    if (nFixableErr || nFixableWarn) {
+        console.error(chalk.red.bold('%s'), `  ${plural(nFixableErr, 'error')} and ${plural(nFixableWarn, 'warning')} potentially fixable with the \`--fix\` option.`);
+    }
+    console.error();
 }
 if (exit) {
     process.exitCode = 1;

package/dist/lib/text.js

@@ -80,8 +80,11 @@ class AstText extends node_1.AstNode {
      */
     lint(start = this.getAbsoluteIndex()) {
         const { data, parentNode, nextSibling, previousSibling } = this;
+        if (!parentNode) {
+            return [];
+        }
         const { NowikiToken } = require('../src/nowiki');
-        const { type, name } = parentNode, nowiki = name === 'nowiki' || name === 'pre';
+        const { type, name } = parentNode, nowiki = name === 'nowiki' || name === 'pre', isHtmlAttrVal = type === 'attr-value' && parentNode.parentNode.type !== 'ext-attr';
         let errorRegex;
         if (type === 'ext-inner' && (name === 'pre' || parentNode instanceof NowikiToken)) {
             errorRegex = new RegExp(`<\\s*(?:\\/\\s*)${nowiki ? '' : '?'}(${name})\\b`, 'giu');
@@ -89,7 +92,7 @@ class AstText extends node_1.AstNode {
         else if (type === 'free-ext-link'
             || type === 'ext-link-url'
             || type === 'image-parameter' && name === 'link'
-            || type === 'attr-value') {
+            || isHtmlAttrVal) {
             errorRegex = errorSyntaxUrl;
         }
         else {
@@ -121,14 +124,15 @@ class AstText extends node_1.AstNode {
             else if (char === ']' && (index || length > 1)) {
                 errorRegex.lastIndex--;
             }
-            const startIndex = start + index, endIndex = startIndex + length, rootStr = String(root), nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1], severity = length > 1 && (char !== '<' || !nowiki && /[\s/>]/u.test(nextChar ?? ''))
+            const startIndex = start + index, endIndex = startIndex + length, rootStr = String(root), nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1], severity = length > 1 && !(char === '<' && (nowiki || !/[\s/>]/u.test(nextChar ?? ''))
+                || isHtmlAttrVal && (char === '[' || char === ']'))
                 || char === '{' && (nextChar === char || previousChar === '-')
                 || char === '}' && (previousChar === char || nextChar === '-')
                 || char === '[' && (nextChar === char
                     || type === 'ext-link-text'
-                    || !data.slice(index + 1).trim() && nextType === 'free-ext-link')
+                    || nextType === 'free-ext-link' && !data.slice(index + 1).trim())
                 || char === ']' && (previousChar === char
-                    || !data.slice(0, index).trim() && previousType === 'free-ext-link')
+                    || previousType === 'free-ext-link' && !data.slice(0, index).includes(']'))
                 ? 'error'
                 : 'warning';
             const leftBracket = char === '{' || char === '[', rightBracket = char === ']' || char === '}';
@@ -149,8 +153,7 @@ class AstText extends node_1.AstNode {
                     }
                 }
             }
-            const lines = data.slice(0, index).split('\n'), startLine = lines.length + top - 1, line = lines[lines.length - 1], startCol = lines.length === 1 ? left + line.length : line.length;
-            errors.push({
+            const lines = data.slice(0, index).split('\n'), startLine = lines.length + top - 1, line = lines[lines.length - 1], startCol = lines.length === 1 ? left + line.length : line.length, e = {
                 rule: ruleMap[char],
                 message: index_1.default.msg('lonely "$1"', char === 'h' ? error : char),
                 severity,
@@ -160,7 +163,45 @@ class AstText extends node_1.AstNode {
                 endLine: startLine,
                 startCol,
                 endCol: startCol + length,
-            });
+            };
+            if (char === '<') {
+                e.suggestions = [
+                    {
+                        desc: 'escape',
+                        range: [startIndex, startIndex + 1],
+                        text: '&lt;',
+                    },
+                ];
+            }
+            else if (char === 'h'
+                && !(type === 'ext-link-text' || type === 'link-text')
+                && /[\p{L}\d_]/u.test(previousChar || '')) {
+                e.suggestions = [
+                    {
+                        desc: 'whitespace',
+                        range: [startIndex, startIndex],
+                        text: ' ',
+                    },
+                ];
+            }
+            else if (char === '[' && type === 'ext-link-text') {
+                const i = parentNode.getAbsoluteIndex() + String(parentNode).length;
+                e.suggestions = [
+                    {
+                        desc: 'escape',
+                        range: [i, i + 1],
+                        text: '&#93;',
+                    },
+                ];
+            }
+            else if (char === ']' && previousType === 'free-ext-link' && severity === 'error') {
+                const i = start - String(previousSibling).length;
+                e.fix = {
+                    range: [i, i],
+                    text: '[',
+                };
+            }
+            errors.push(e);
         }
         return errors;
     }

package/dist/src/arg.js

@@ -55,22 +55,40 @@ class ArgToken extends index_2.Token {
     }
     /** @override */
     lint(start = this.getAbsoluteIndex()) {
+        const { childNodes: [argName, argDefault, ...rest] } = this;
         if (!this.getAttribute('include')) {
-            return [(0, lint_1.generateForSelf)(this, { start }, 'no-arg', 'unexpected template argument')];
+            const e = (0, lint_1.generateForSelf)(this, { start }, 'no-arg', 'unexpected template argument');
+            if (argDefault) {
+                e.fix = {
+                    range: [start, e.endIndex],
+                    text: argDefault.text(),
+                };
+            }
+            return [e];
         }
-        const { childNodes: [argName, argDefault, ...rest] } = this, errors = argName.lint(start + 3);
+        const errors = argName.lint(start + 3);
         if (argDefault) {
             errors.push(...argDefault.lint(start + 4 + String(argName).length));
         }
         if (rest.length > 0) {
             const rect = { start, ...this.getRootNode().posFromIndex(start) };
             errors.push(...rest.map(child => {
-                const error = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'invisible content inside triple braces');
-                return {
-                    ...error,
-                    startIndex: error.startIndex - 1,
-                    startCol: error.startCol - 1,
-                };
+                const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'invisible content inside triple braces');
+                e.startIndex--;
+                e.startCol--;
+                e.suggestions = [
+                    {
+                        desc: 'remove',
+                        range: [e.startIndex, e.endIndex],
+                        text: '',
+                    },
+                    {
+                        desc: 'escape',
+                        range: [e.startIndex, e.startIndex + 1],
+                        text: '{{!}}',
+                    },
+                ];
+                return e;
             }));
         }
         return errors;

package/dist/src/attribute.js

@@ -261,11 +261,24 @@ class AttributeToken extends index_2.Token {
             const root = this.getRootNode();
             rect = { start, ...root.posFromIndex(start) };
             const e = (0, lint_1.generateForChild)(lastChild, rect, 'unclosed-quote', index_1.default.msg('unclosed $1', 'quotes'), 'warning');
-            errors.push({
-                ...e,
-                startIndex: e.startIndex - 1,
-                startCol: e.startCol - 1,
-            });
+            e.startIndex--;
+            e.startCol--;
+            const fix = {
+                range: [e.endIndex, e.endIndex],
+                text: this.#quotes[0],
+            };
+            if (lastChild.childNodes.some(child => child.type === 'text' && /\s/u.test(child.text()))) {
+                e.suggestions = [
+                    {
+                        desc: 'quote',
+                        ...fix,
+                    },
+                ];
+            }
+            else {
+                e.fix = fix;
+            }
+            errors.push(e);
         }
         const attrs = extAttrs[tag];
         if (attrs && !attrs.has(name)
@@ -286,7 +299,20 @@ class AttributeToken extends index_2.Token {
         }
         else if (name === 'tabindex' && typeof value === 'string' && value.trim() !== '0') {
             rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-            errors.push((0, lint_1.generateForChild)(lastChild, rect, 'illegal-attr', 'nonzero tabindex'));
+            const e = (0, lint_1.generateForChild)(lastChild, rect, 'illegal-attr', 'nonzero tabindex');
+            e.suggestions = [
+                {
+                    desc: 'remove',
+                    range: [start, e.endIndex],
+                    text: '',
+                },
+                {
+                    desc: '0 tabindex',
+                    range: [e.startIndex, e.endIndex],
+                    text: '0',
+                },
+            ];
+            errors.push(e);
         }
         return errors;
     }

package/dist/src/attributes.js

@@ -100,13 +100,23 @@ class AttributesToken extends index_2.Token {
         let rect;
         if (parentNode?.type === 'html' && parentNode.closing && this.text().trim()) {
             rect = { start, ...this.getRootNode().posFromIndex(start) };
-            errors.push((0, lint_1.generateForSelf)(this, rect, 'no-ignored', 'attributes of a closing tag'));
+            const e = (0, lint_1.generateForSelf)(this, rect, 'no-ignored', 'attributes of a closing tag');
+            e.fix = { range: [start, e.endIndex], text: '' };
+            errors.push(e);
         }
         for (let i = 0; i < length; i++) {
             const attr = childNodes[i];
             if (attr instanceof atom_1.AtomToken && attr.text().trim()) {
                 rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-                errors.push((0, lint_1.generateForChild)(attr, rect, 'no-ignored', 'containing invalid attribute'));
+                const e = (0, lint_1.generateForChild)(attr, rect, 'no-ignored', 'containing invalid attribute');
+                e.suggestions = [
+                    {
+                        desc: 'remove',
+                        range: [e.startIndex, e.endIndex],
+                        text: ' ',
+                    },
+                ];
+                errors.push(e);
             }
             else if (attr instanceof attribute_1.AttributeToken) {
                 const { name } = attr;

package/dist/src/converterFlags.js

@@ -56,7 +56,23 @@ class ConverterFlagsToken extends index_2.Token {
                 && !variantFlags.has(flag)
                 && !unknownFlags.has(flag)
                 && (variantFlags.size > 0 || !validFlags.has(flag))) {
-                errors.push((0, lint_1.generateForChild)(child, rect, 'no-ignored', 'invalid conversion flag'));
+                const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'invalid conversion flag');
+                if (variantFlags.size === 0 && definedFlags.has(flag.toUpperCase())) {
+                    e.fix = {
+                        range: [e.startIndex, e.endIndex],
+                        text: flag.toUpperCase(),
+                    };
+                }
+                else {
+                    e.suggestions = [
+                        {
+                            desc: 'remove',
+                            range: [e.startIndex, e.endIndex],
+                            text: '',
+                        },
+                    ];
+                }
+                errors.push(e);
             }
         }
         return errors;

package/dist/src/gallery.js

@@ -67,6 +67,18 @@ class GalleryToken extends index_2.Token {
                     endLine: startLine,
                     startCol,
                     endCol: startCol + length,
+                    suggestions: [
+                        {
+                            desc: 'remove',
+                            range: [start, start + length],
+                            text: '',
+                        },
+                        {
+                            desc: 'comment',
+                            range: [start, start + length],
+                            text: `<!--${str}-->`,
+                        },
+                    ],
                 });
             }
             else if (child.type !== 'noinclude' && child.type !== 'text') {

package/dist/src/html.js

@@ -114,6 +114,24 @@ class HtmlToken extends index_2.Token {
                     if (ancestor && magicWords.has(ancestor.name)) {
                         error.severity = 'warning';
                     }
+                    else {
+                        error.suggestions = [
+                            {
+                                desc: 'remove',
+                                range: [start, error.endIndex],
+                                text: '',
+                            },
+                        ];
+                    }
+                }
+                else if (msg === 'tag that is both closing and self-closing') {
+                    const { html: [, , voidTags] } = this.getAttribute('config');
+                    if (voidTags.includes(this.name)) {
+                        error.fix = {
+                            range: [start + 1, start + 2],
+                            text: '',
+                        };
+                    }
                 }
                 errors.push(error);
             }
@@ -131,7 +149,7 @@ class HtmlToken extends index_2.Token {
             refError ??= (0, lint_1.generateForSelf)(this, { start }, 'h1', '');
             errors.push({
                 ...refError,
-                rule: 'format-leakage',
+                rule: 'bold-header',
                 message: index_1.default.msg('bold in section header'),
                 severity: 'warning',
             });

package/dist/src/imageParameter.js

@@ -102,7 +102,12 @@ class ImageParameterToken extends index_2.Token {
     lint(start = this.getAbsoluteIndex()) {
         const errors = super.lint(start), { link, name } = this;
         if (name === 'invalid') {
-            errors.push((0, lint_1.generateForSelf)(this, { start }, 'invalid-gallery', 'invalid gallery image parameter'));
+            const e = (0, lint_1.generateForSelf)(this, { start }, 'invalid-gallery', 'invalid gallery image parameter');
+            e.fix = {
+                range: [start, start + e.endIndex],
+                text: '',
+            };
+            errors.push(e);
         }
         else if (typeof link === 'object' && link.encoded) {
             errors.push((0, lint_1.generateForSelf)(this, { start }, 'url-encoding', 'unnecessary URL encoding in an internal link'));

package/dist/src/link/base.js

@@ -82,14 +82,34 @@ class LinkBaseToken extends index_2.Token {
             rect ??= { start, ...this.getRootNode().posFromIndex(start) };
             errors.push((0, lint_1.generateForChild)(target, rect, 'url-encoding', 'unnecessary URL encoding in an internal link'));
         }
-        if ((linkType === 'link' || linkType === 'category')
-            && linkText?.childNodes.some(({ type, data }) => type === 'text' && data.includes('|'))) {
-            rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-            errors.push((0, lint_1.generateForChild)(linkText, rect, 'pipe-like', 'additional "|" in the link text', 'warning'));
+        if (linkType === 'link' || linkType === 'category') {
+            const textNode = linkText?.childNodes.find((c) => c.type === 'text' && c.data.includes('|'));
+            if (textNode) {
+                rect ??= { start, ...this.getRootNode().posFromIndex(start) };
+                const e = (0, lint_1.generateForChild)(linkText, rect, 'pipe-like', 'additional "|" in the link text', 'warning');
+                e.suggestions = [
+                    {
+                        desc: 'escape',
+                        range: [
+                            e.startIndex + textNode.getRelativeIndex(),
+                            e.startIndex + textNode.getRelativeIndex() + textNode.data.length,
+                        ],
+                        text: textNode.data.replace(/\|/gu, '|'),
+                    },
+                ];
+                errors.push(e);
+            }
         }
-        else if (linkType !== 'link' && fragment !== undefined) {
+        if (linkType !== 'link' && fragment !== undefined) {
             rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-            errors.push((0, lint_1.generateForChild)(target, rect, 'no-ignored', 'useless fragment'));
+            const e = (0, lint_1.generateForChild)(target, rect, 'no-ignored', 'useless fragment'), textNode = target.childNodes.find((c) => c.type === 'text' && c.data.includes('#'));
+            if (textNode) {
+                e.fix = {
+                    range: [e.startIndex + textNode.getRelativeIndex() + textNode.data.indexOf('#'), e.endIndex],
+                    text: '',
+                };
+            }
+            errors.push(e);
         }
         return errors;
     }

package/dist/src/magicLink.js

@@ -30,17 +30,41 @@ class MagicLinkToken extends index_2.Token {
             const refError = (0, lint_1.generateForChild)(child, rect, 'unterminated-url', '', 'warning');
             regexGlobal.lastIndex = 0;
             for (let mt = regexGlobal.exec(data); mt; mt = regexGlobal.exec(data)) {
-                const { index, 0: s } = mt, lines = data.slice(0, index).split('\n'), top = lines.length, left = lines[top - 1].length, startIndex = refError.startIndex + index, startLine = refError.startLine + top - 1, startCol = top === 1 ? refError.startCol + left : left;
-                errors.push({
+                const { index, 0: s } = mt, lines = data.slice(0, index).split('\n'), top = lines.length, left = lines[top - 1].length, startIndex = refError.startIndex + index, startLine = refError.startLine + top - 1, startCol = top === 1 ? refError.startCol + left : left, pipe = s.startsWith('|');
+                const e = {
                     ...refError,
-                    message: index_1.default.msg('$1 in URL', s.startsWith('|') ? '"|"' : 'full-width punctuation'),
+                    message: index_1.default.msg('$1 in URL', pipe ? '"|"' : 'full-width punctuation'),
                     startIndex,
                     endIndex: startIndex + s.length,
                     startLine,
                     endLine: startLine,
                     startCol,
                     endCol: startCol + s.length,
-                });
+                };
+                if (!pipe) {
+                    e.suggestions = [
+                        {
+                            desc: 'whitespace',
+                            range: [startIndex, startIndex],
+                            text: ' ',
+                        },
+                        {
+                            desc: 'escape',
+                            range: [startIndex, e.endIndex],
+                            text: encodeURI(s),
+                        },
+                    ];
+                }
+                else if (s.length === 1) {
+                    e.suggestions = [
+                        {
+                            desc: 'whitespace',
+                            range: [startIndex, startIndex + 1],
+                            text: ' ',
+                        },
+                    ];
+                }
+                errors.push(e);
             }
         }
         return errors;

package/dist/src/nested.js

@@ -50,7 +50,20 @@ class NestedToken extends index_2.Token {
                 return str && !/^<!--.*-->$/su.test(str);
             }).map(child => {
                 rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-                return (0, lint_1.generateForChild)(child, rect, 'no-ignored', index_1.default.msg('invalid content in <$1>', this.name));
+                const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', index_1.default.msg('invalid content in <$1>', this.name));
+                e.suggestions = [
+                    {
+                        desc: 'remove',
+                        range: [e.startIndex, e.endIndex],
+                        text: '',
+                    },
+                    {
+                        desc: 'comment',
+                        range: [e.startIndex, e.startIndex],
+                        text: `<!--${String(child)}-->`,
+                    },
+                ];
+                return e;
             }),
         ];
     }

package/dist/src/nowiki/comment.js

@@ -20,9 +20,15 @@ class CommentToken extends (0, hidden_1.hiddenToken)(base_1.NowikiBaseToken) {
     }
     /** @override */
     lint(start = this.getAbsoluteIndex()) {
-        return this.closed
-            ? []
-            : [(0, lint_1.generateForSelf)(this, { start }, 'unclosed-comment', index_1.default.msg('unclosed $1', 'HTML comment'))];
+        if (this.closed) {
+            return [];
+        }
+        const e = (0, lint_1.generateForSelf)(this, { start }, 'unclosed-comment', index_1.default.msg('unclosed $1', 'HTML comment'));
+        e.fix = {
+            range: [e.endIndex, e.endIndex],
+            text: '-->',
+        };
+        return [e];
     }
     /** @private */
     toString() {

package/dist/src/nowiki/index.js

@@ -10,9 +10,15 @@ class NowikiToken extends base_1.NowikiBaseToken {
     /** @override */
     lint(start = this.getAbsoluteIndex()) {
         const { name, firstChild: { data } } = this;
-        return (name === 'templatestyles' || name === 'section') && data
-            ? [(0, lint_1.generateForSelf)(this, { start }, 'void-ext', index_1.default.msg('nothing should be in <$1>', name))]
-            : super.lint(start);
+        if ((name === 'templatestyles' || name === 'section') && data) {
+            const e = (0, lint_1.generateForSelf)(this, { start }, 'void-ext', index_1.default.msg('nothing should be in <$1>', name));
+            e.fix = {
+                range: [start - 1, e.endIndex + name.length + 3],
+                text: '/>',
+            };
+            return [e];
+        }
+        return super.lint(start);
     }
 }
 exports.NowikiToken = NowikiToken;

package/dist/src/nowiki/quote.js

@@ -29,6 +29,13 @@ class QuoteToken extends base_1.NowikiBaseToken {
                 startCol: endCol - length,
                 endLine,
                 endCol,
+                suggestions: [
+                    {
+                        desc: 'escape',
+                        range: [startIndex, endIndex],
+                        text: '''.repeat(length),
+                    },
+                ],
             });
         }
         if (nextSibling?.type === 'text' && nextSibling.data.startsWith(`'`)) {
@@ -41,6 +48,13 @@ class QuoteToken extends base_1.NowikiBaseToken {
                 startLine,
                 startCol,
                 endCol: startCol + length,
+                suggestions: [
+                    {
+                        desc: 'escape',
+                        range: [startIndex, endIndex],
+                        text: '''.repeat(length),
+                    },
+                ],
             });
         }
         if (bold && this.closest('heading-title')) {

package/dist/src/paramTag/index.js

@@ -39,7 +39,15 @@ class ParamTagToken extends index_2.Token {
             return str && !(i >= 0 ? /^[a-z]+(?:\[\])?\s*(?:=|$)/iu : /^[a-z]+(?:\[\])?\s*=/iu).test(str);
         }).map(child => {
             rect ??= { start, ...this.getRootNode().posFromIndex(start) };
-            return (0, lint_1.generateForChild)(child, rect, 'no-ignored', index_1.default.msg('invalid parameter of <$1>', this.name));
+            const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', index_1.default.msg('invalid parameter of <$1>', this.name));
+            e.suggestions = [
+                {
+                    desc: 'remove',
+                    range: [e.startIndex, e.endIndex],
+                    text: '',
+                },
+            ];
+            return e;
         });
     }
 }

package/dist/src/parameter.js

@@ -61,14 +61,16 @@ class ParameterToken extends index_2.Token {
         const errors = super.lint(start), { firstChild } = this, link = new RegExp(`https?://${string_1.extUrlCharFirst}${string_1.extUrlChar}$`, 'iu').exec(firstChild.text())?.[0];
         if (link && new URL(link).search) {
             const e = (0, lint_1.generateForChild)(firstChild, { start }, 'unescaped', 'unescaped query string in an anonymous parameter');
-            errors.push({
-                ...e,
-                startIndex: e.endIndex,
-                endIndex: e.endIndex + 1,
-                startLine: e.endLine,
-                startCol: e.endCol,
-                endCol: e.endCol + 1,
-            });
+            e.startIndex = e.endIndex;
+            e.startLine = e.endLine;
+            e.startCol = e.endCol;
+            e.endIndex++;
+            e.endCol++;
+            e.fix = {
+                range: [e.startIndex, e.endIndex],
+                text: '{{=}}',
+            };
+            errors.push(e);
         }
         return errors;
     }

package/dist/src/table/td.js

@@ -81,7 +81,24 @@ class TdToken extends base_1.TableBaseToken {
             if (child.type === 'text') {
                 const { data } = child;
                 if (data.includes('|')) {
-                    errors.push((0, lint_1.generateForChild)(child, { start }, 'pipe-like', 'additional "|" in a table cell', data.includes('||') ? 'error' : 'warning'));
+                    const isError = data.includes('||'), e = (0, lint_1.generateForChild)(child, { start }, 'pipe-like', 'additional "|" in a table cell', isError ? 'error' : 'warning');
+                    if (isError) {
+                        const syntax = { caption: '|+', td: '|', th: '!' }[this.subtype];
+                        e.fix = {
+                            range: [e.startIndex, e.endIndex],
+                            text: data.replace(/\|\|/gu, `\n${syntax}`),
+                        };
+                    }
+                    else {
+                        e.suggestions = [
+                            {
+                                desc: 'escape',
+                                range: [e.startIndex, e.endIndex],
+                                text: data.replace(/\|/gu, '|'),
+                            },
+                        ];
+                    }
+                    errors.push(e);
                 }
             }
         }

package/dist/src/table/trBase.js

@@ -27,13 +27,11 @@ class TrBaseToken extends base_1.TableBaseToken {
             catch { }
         }
         const error = (0, lint_1.generateForChild)(inter, { start }, 'fostered-content', 'content to be moved out from the table');
-        errors.push({
-            ...error,
-            severity: first.type === 'template' ? 'warning' : 'error',
-            startIndex: error.startIndex + 1,
-            startLine: error.startLine + 1,
-            startCol: 0,
-        });
+        error.severity = first.type === 'template' ? 'warning' : 'error';
+        error.startIndex++;
+        error.startLine++;
+        error.startCol = 0;
+        errors.push(error);
         return errors;
     }
 }

package/dist/src/tagPair/include.js

@@ -22,9 +22,15 @@ class IncludeToken extends (0, hidden_1.hiddenToken)(index_2.TagPairToken) {
     }
     /** @override */
     lint(start = this.getAbsoluteIndex()) {
-        return this.closed
-            ? []
-            : [(0, lint_1.generateForSelf)(this, { start }, 'unclosed-comment', index_1.default.msg('unclosed $1', `<${this.name}>`))];
+        if (this.closed) {
+            return [];
+        }
+        const e = (0, lint_1.generateForSelf)(this, { start }, 'unclosed-comment', index_1.default.msg('unclosed $1', `<${this.name}>`));
+        e.fix = {
+            range: [e.endIndex, e.endIndex],
+            text: `</${this.name}>`,
+        };
+        return [e];
     }
 }
 exports.IncludeToken = IncludeToken;

package/dist/src/transclude.js

@@ -166,7 +166,14 @@ class TranscludeToken extends index_2.Token {
         const title = this.#getTitle();
         if (title.fragment !== undefined) {
             rect = { start, ...this.getRootNode().posFromIndex(start) };
-            errors.push((0, lint_1.generateForChild)(childNodes[type === 'template' ? 0 : 1], rect, 'no-ignored', 'useless fragment'));
+            const child = childNodes[type === 'template' ? 0 : 1], e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'useless fragment'), textNode = child.childNodes.find((c) => c.type === 'text' && c.data.includes('#'));
+            if (textNode) {
+                e.fix = {
+                    range: [e.startIndex + textNode.getRelativeIndex() + textNode.data.indexOf('#'), e.endIndex],
+                    text: '',
+                };
+            }
+            errors.push(e);
         }
         if (!title.valid) {
             rect ??= { start, ...this.getRootNode().posFromIndex(start) };

package/dist/util/diff.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.info = exports.error = exports.diff = exports.cmd = void 0;
 const fs = require("fs/promises");
 const child_process_1 = require("child_process");
+const chalk = require("chalk");
 process.on('unhandledRejection', e => {
     console.error(e);
 });
@@ -72,11 +73,11 @@ const diff = async (oldStr, newStr, uid = -1) => {
 exports.diff = diff;
 /** @implements */
 const error = (msg, ...args) => {
-    console.error('\x1B[31m%s\x1B[0m', msg, ...args);
+    console.error(chalk.red(msg), ...args);
 };
 exports.error = error;
 /** @implements */
 const info = (msg, ...args) => {
-    console.info('\x1B[32m%s\x1B[0m', msg, ...args);
+    console.info(chalk.green(msg), ...args);
 };
 exports.info = info;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wikilint",
-	"version": "2.4.5",
+	"version": "2.5.0",
 	"description": "A Node.js linter for MediaWiki markup",
 	"keywords": [
 		"mediawiki",
@@ -51,6 +51,7 @@
 		"@typescript-eslint/eslint-plugin": "^6.19.1",
 		"@typescript-eslint/parser": "^6.19.1",
 		"ajv-cli": "^5.0.0",
+		"chalk": "^4.1.2",
 		"eslint": "^8.56.0",
 		"eslint-plugin-es-x": "^7.5.0",
 		"eslint-plugin-eslint-comments": "^3.2.0",