wikiparser-node 1.13.9 → 1.14.0

package/dist/addon/transclude.js

@@ -94,7 +94,16 @@ transclude_1.TranscludeToken.prototype.fixDuplication =
             if (args.length <= 1) {
                 continue;
             }
-            const values = Map.groupBy(args, (arg) => arg.getValue().trim());
+            const values = new Map();
+            for (const arg of args) {
+                const val = arg.getValue().trim();
+                if (values.has(val)) {
+                    values.get(val).push(arg);
+                }
+                else {
+                    values.set(val, [arg]);
+                }
+            }
             let noMoreAnon = anonCount === 0 || !key.trim() || isNaN(key);
             const emptyArgs = values.get('') ?? [], duplicatedArgs = [...values].filter(([val, { length }]) => val && length > 1).flatMap(([, curArgs]) => {
                 const anonIndex = noMoreAnon ? -1 : curArgs.findIndex(({ anon }) => anon);

package/dist/base.d.ts

@@ -63,9 +63,7 @@ export interface LintError {
     endLine: number;
     endCol: number;
     fix?: LintError.Fix;
-    suggestions?: (LintError.Fix & {
-        desc: string;
-    })[];
+    suggestions?: LintError.Fix[];
 }
 export type AST = Record<string, string | number | boolean> & {
     range: [number, number];

package/dist/lib/text.js

@@ -148,7 +148,7 @@ class AstText extends node_1.AstNode {
             return [];
         }
         errorRegex.lastIndex = 0;
-        const errors = [], nextType = nextSibling?.type, nextName = nextSibling?.name, previousType = previousSibling?.type, root = this.getRootNode(), { ext, html } = root.getAttribute('config'), { top, left } = root.posFromIndex(start), tags = new Set(['onlyinclude', 'noinclude', 'includeonly', ext, html, disallowedTags].flat(2));
+        const errors = [], nextType = nextSibling?.type, nextName = nextSibling?.name, previousType = previousSibling?.type, root = this.getRootNode(), rootStr = root.toString(), { ext, html } = root.getAttribute('config'), { top, left } = root.posFromIndex(start), tags = new Set(['onlyinclude', 'noinclude', 'includeonly', ext, html, disallowedTags].flat(2));
         for (let mt = errorRegex.exec(data); mt; mt = errorRegex.exec(data)) {
             const [, tag, prefix] = mt;
             let { index } = mt, error = mt[0].toLowerCase();
@@ -168,7 +168,7 @@ class AstText extends node_1.AstNode {
             else if (char === ']' && (index || length > 1)) {
                 errorRegex.lastIndex--;
             }
-            const startIndex = start + index, endIndex = startIndex + length, rootStr = root.toString(), nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1], severity = length > 1 && !(char === '<' && !/[\s/>]/u.test(nextChar ?? '')
+            const startIndex = start + index, endIndex = startIndex + length, nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1], severity = length > 1 && !(char === '<' && !/[\s/>]/u.test(nextChar ?? '')
                 || isHtmlAttrVal && (char === '[' || char === ']')
                 || magicLink && type === 'parameter-value')
                 || char === '{' && (nextChar === char || previousChar === '-')
@@ -213,39 +213,31 @@ class AstText extends node_1.AstNode {
                 endCol: startCol + length,
             };
             if (char === '<') {
-                e.suggestions = [
-                    {
-                        desc: 'escape',
-                        range: [startIndex, startIndex + 1],
-                        text: '&lt;',
-                    },
-                ];
+                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: ' ',
-                    },
-                ];
+                e.suggestions = [{ desc: 'whitespace', range: [startIndex, startIndex], text: ' ' }];
             }
             else if (char === '[' && type === 'ext-link-text') {
                 const i = parentNode.getAbsoluteIndex() + parentNode.toString().length;
-                e.suggestions = [
-                    {
-                        desc: 'escape',
-                        range: [i, i + 1],
-                        text: '&#93;',
-                    },
-                ];
+                e.suggestions = [{ desc: 'escape', range: [i, i + 1], text: '&#93;' }];
             }
             else if (char === ']' && previousType === 'free-ext-link' && severity === 'error') {
                 const i = start - previousSibling.toString().length;
                 e.fix = { range: [i, i], text: '[', desc: 'left bracket' };
             }
+            else if (magicLink) {
+                e.suggestions = [
+                    ...mt[0] === error
+                        ? []
+                        : [{ desc: 'uppercase', range: [startIndex, endIndex], text: error }],
+                    ...nextChar === ':' || nextChar === '：'
+                        ? [{ desc: 'whitespace', range: [endIndex, endIndex + 1], text: ' ' }]
+                        : [],
+                ];
+            }
             errors.push(e);
         }
         return errors;

package/dist/src/arg.js

@@ -95,7 +95,7 @@ class ArgToken extends index_2.Token {
         if (!this.getAttribute('include')) {
             const e = (0, lint_1.generateForSelf)(this, { start }, 'no-arg', 'unexpected template argument');
             if (argDefault) {
-                e.fix = { range: [start, e.endIndex], text: argDefault.text(), desc: 'expand' };
+                e.suggestions = [{ range: [start, e.endIndex], text: argDefault.text(), desc: 'expand' }];
             }
             return [e];
         }
@@ -110,16 +110,8 @@ class ArgToken extends index_2.Token {
                 e.startIndex--;
                 e.startCol--;
                 e.suggestions = [
-                    {
-                        desc: 'remove',
-                        range: [e.startIndex, e.endIndex],
-                        text: '',
-                    },
-                    {
-                        desc: 'escape',
-                        range: [e.startIndex, e.startIndex + 1],
-                        text: '{{!}}',
-                    },
+                    { desc: 'remove', range: [e.startIndex, e.endIndex], text: '' },
+                    { desc: 'escape', range: [e.startIndex, e.startIndex + 1], text: '{{!}}' },
                 ];
                 return e;
             }));

package/dist/src/attribute.js

@@ -180,16 +180,10 @@ let AttributeToken = (() => {
                 const e = (0, lint_1.generateForChild)(lastChild, rect, 'unclosed-quote', index_1.default.msg('unclosed $1', 'quotes'), 'warning');
                 e.startIndex--;
                 e.startCol--;
-                const fix = { range: [e.endIndex, e.endIndex], text: this.#quotes[0], desc: 'close' };
-                if (lastChild.childNodes.some(({ type: t, data }) => t === 'text' && /\s/u.test(data))) {
-                    e.suggestions = [fix];
-                }
-                else {
-                    e.fix = fix;
-                }
+                e.suggestions = [{ range: [e.endIndex, e.endIndex], text: this.#quotes[0], desc: 'close' }];
                 errors.push(e);
             }
-            const attrs = sharable_1.extAttrs[tag], attrs2 = sharable_1.htmlAttrs[tag];
+            const attrs = sharable_1.extAttrs[tag], attrs2 = sharable_1.htmlAttrs[tag], { length } = this.toString();
             if (!attrs?.has(name)
                 && !attrs2?.has(name)
                 // 不是未定义的扩展标签或包含嵌入的HTML标签
@@ -197,7 +191,10 @@ let AttributeToken = (() => {
                 && (type === 'ext-attr' && !attrs2
                     || !/^(?:xmlns:[\w:.-]+|data-(?!ooui|mw|parsoid)[^:]*)$/u.test(name)
                         && (tag === 'meta' || tag === 'link' || !sharable_1.commonHtmlAttrs.has(name)))) {
-                errors.push((0, lint_1.generateForChild)(firstChild, rect, 'illegal-attr', 'illegal attribute name'));
+                errors.push({
+                    ...(0, lint_1.generateForChild)(firstChild, rect, 'illegal-attr', 'illegal attribute name'),
+                    suggestions: [{ desc: 'remove', range: [start, start + length], text: '' }],
+                });
             }
             else if (sharable_1.obsoleteAttrs[tag]?.has(name)) {
                 errors.push((0, lint_1.generateForChild)(firstChild, rect, 'obsolete-attr', 'obsolete attribute', 'warning'));
@@ -208,16 +205,8 @@ let AttributeToken = (() => {
             else if (name === 'tabindex' && typeof value === 'string' && value !== '0') {
                 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',
-                    },
+                    { desc: 'remove', range: [start, start + length], text: '' },
+                    { desc: '0 tabindex', range: [e.startIndex, e.endIndex], text: '0' },
                 ];
                 errors.push(e);
             }

package/dist/src/attributes.js

@@ -168,8 +168,11 @@ class AttributesToken extends index_2.Token {
     lint(start = this.getAbsoluteIndex(), re) {
         const errors = super.lint(start, re), { parentNode, childNodes } = this, attrs = new Map(), duplicated = new Set(), rect = new rect_1.BoundingRect(this, start);
         if (parentNode?.type === 'html' && parentNode.closing && this.text().trim()) {
-            const e = (0, lint_1.generateForSelf)(this, rect, 'no-ignored', 'attributes of a closing tag');
-            e.fix = { range: [start, e.endIndex], text: '', desc: 'remove' };
+            const e = (0, lint_1.generateForSelf)(this, rect, 'no-ignored', 'attributes of a closing tag'), index = parentNode.getAbsoluteIndex();
+            e.suggestions = [
+                { desc: 'remove', range: [start, e.endIndex], text: '' },
+                { desc: 'open', range: [index + 1, index + 2], text: '' },
+            ];
             errors.push(e);
         }
         for (const attr of childNodes) {
@@ -187,20 +190,27 @@ class AttributesToken extends index_2.Token {
                 const str = attr.text().trim();
                 if (str) {
                     const e = (0, lint_1.generateForChild)(attr, rect, 'no-ignored', 'containing invalid attribute', /[\p{L}\d]/u.test(str) ? 'error' : 'warning');
-                    e.suggestions = [
-                        {
-                            desc: 'remove',
-                            range: [e.startIndex, e.endIndex],
-                            text: ' ',
-                        },
-                    ];
+                    e.suggestions = [{ desc: 'remove', range: [e.startIndex, e.endIndex], text: ' ' }];
                     errors.push(e);
                 }
             }
         }
         if (duplicated.size > 0) {
             for (const key of duplicated) {
-                errors.push(...attrs.get(key).map(attr => (0, lint_1.generateForChild)(attr, rect, 'no-duplicate', index_1.default.msg('duplicated $1 attribute', key))));
+                const pairs = attrs.get(key).map(attr => {
+                    const value = attr.getValue();
+                    return [attr, value === true ? '' : value];
+                });
+                errors.push(...pairs.map(([attr, value], i) => {
+                    const e = (0, lint_1.generateForChild)(attr, rect, 'no-duplicate', index_1.default.msg('duplicated $1 attribute', key)), remove = { desc: 'remove', range: [e.startIndex, e.endIndex], text: '' };
+                    if (!value || pairs.slice(0, i).some(([, v]) => v === value)) {
+                        e.fix = remove;
+                    }
+                    else {
+                        e.suggestions = [remove];
+                    }
+                    return e;
+                }));
             }
         }
         return errors;

package/dist/src/converterFlags.js

@@ -91,13 +91,7 @@ class ConverterFlagsToken extends index_2.Token {
                     e.fix = { range: [e.startIndex, e.endIndex], text: flag.toUpperCase(), desc: 'uppercase' };
                 }
                 else {
-                    e.suggestions = [
-                        {
-                            desc: 'remove',
-                            range: [e.startIndex - (i && 1), e.endIndex],
-                            text: '',
-                        },
-                    ];
+                    e.suggestions = [{ desc: 'remove', range: [e.startIndex - (i && 1), e.endIndex], text: '' }];
                 }
                 errors.push(e);
             }

package/dist/src/gallery.js

@@ -92,16 +92,8 @@ class GalleryToken extends index_2.Token {
                     startCol,
                     endCol: startCol + length,
                     suggestions: [
-                        {
-                            desc: 'remove',
-                            range: [start, endIndex],
-                            text: '',
-                        },
-                        {
-                            desc: 'comment',
-                            range: [start, endIndex],
-                            text: `<!--${str}-->`,
-                        },
+                        { desc: 'remove', range: [start, endIndex], text: '' },
+                        { desc: 'comment', range: [start, endIndex], text: `<!--${str}-->` },
                     ],
                 });
             }

package/dist/src/heading.js

@@ -135,21 +135,55 @@ let HeadingToken = (() => {
         }
         /** @private */
         lint(start = this.getAbsoluteIndex(), re) {
-            const errors = super.lint(start, re), { firstChild, level } = this, innerStr = firstChild.toString(), quotes = firstChild.childNodes.filter((0, debug_1.isToken)('quote')), boldQuotes = quotes.filter(({ bold }) => bold), italicQuotes = quotes.filter(({ italic }) => italic), rect = new rect_1.BoundingRect(this, start);
+            const errors = super.lint(start, re), { firstChild, level } = this, innerStr = firstChild.toString(), unbalancedStart = innerStr.startsWith('='), unbalanced = unbalancedStart || innerStr.endsWith('='), quotes = firstChild.childNodes.filter((0, debug_1.isToken)('quote')), boldQuotes = quotes.filter(({ bold }) => bold), italicQuotes = quotes.filter(({ italic }) => italic), rect = new rect_1.BoundingRect(this, start);
             if (this.level === 1) {
-                errors.push((0, lint_1.generateForChild)(firstChild, rect, 'h1', '<h1>'));
+                const e = (0, lint_1.generateForChild)(firstChild, rect, 'h1', '<h1>');
+                if (!unbalanced) {
+                    e.suggestions = [{ desc: 'h2', range: [e.startIndex, e.endIndex], text: `=${innerStr}=` }];
+                }
+                errors.push(e);
             }
-            if (innerStr.startsWith('=') || innerStr.endsWith('=')) {
-                errors.push((0, lint_1.generateForChild)(firstChild, rect, 'unbalanced-header', index_1.default.msg('unbalanced $1 in a section header', '"="')));
+            if (unbalanced) {
+                const e = (0, lint_1.generateForChild)(firstChild, rect, 'unbalanced-header', index_1.default.msg('unbalanced $1 in a section header', '"="'));
+                if (innerStr === '=') {
+                    //
+                }
+                else if (unbalancedStart) {
+                    const [extra] = /^=+/u.exec(innerStr);
+                    e.suggestions = [
+                        { desc: `h${level}`, range: [e.startIndex, e.startIndex + extra.length], text: '' },
+                        { desc: `h${level + extra.length}`, range: [e.endIndex, e.endIndex], text: extra },
+                    ];
+                }
+                else {
+                    const extra = /[^=](=+)$/u.exec(innerStr)[1];
+                    e.suggestions = [
+                        { desc: `h${level}`, range: [e.endIndex - extra.length, e.endIndex], text: '' },
+                        { desc: `h${level + extra.length}`, range: [e.startIndex, e.startIndex], text: extra },
+                    ];
+                }
+                errors.push(e);
             }
             if (this.closest('html-attrs,table-attrs')) {
                 errors.push((0, lint_1.generateForSelf)(this, rect, 'parsing-order', 'section header in a HTML tag'));
             }
+            const rootStr = this.getRootNode().toString();
             if (boldQuotes.length % 2) {
-                errors.push((0, lint_1.generateForChild)(boldQuotes[boldQuotes.length - 1], { ...rect, start: start + level, left: rect.left + level }, 'format-leakage', index_1.default.msg('unbalanced $1 in a section header', 'bold apostrophes')));
+                const e = (0, lint_1.generateForChild)(boldQuotes[boldQuotes.length - 1], { ...rect, start: start + level, left: rect.left + level }, 'format-leakage', index_1.default.msg('unbalanced $1 in a section header', 'bold apostrophes')), end = start + level + innerStr.length;
+                if (rootStr.slice(e.endIndex, end).trim()) {
+                    e.suggestions = [{ desc: 'close', range: [end, end], text: "'''" }];
+                }
+                else {
+                    e.fix = { desc: 'remove', range: [e.startIndex, e.endIndex], text: '' };
+                }
+                errors.push(e);
             }
             if (italicQuotes.length % 2) {
-                errors.push((0, lint_1.generateForChild)(italicQuotes[italicQuotes.length - 1], { start: start + level }, 'format-leakage', index_1.default.msg('unbalanced $1 in a section header', 'italic apostrophes')));
+                const e = (0, lint_1.generateForChild)(italicQuotes[italicQuotes.length - 1], { start: start + level }, 'format-leakage', index_1.default.msg('unbalanced $1 in a section header', 'italic apostrophes')), end = start + level + innerStr.length;
+                e.fix = rootStr.slice(e.endIndex, end).trim()
+                    ? { desc: 'close', range: [end, end], text: "''" }
+                    : { desc: 'remove', range: [e.startIndex, e.endIndex], text: '' };
+                errors.push(e);
             }
             return errors;
         }
@@ -205,7 +239,7 @@ let HeadingToken = (() => {
             if (headings?.has(lcId)) {
                 let i = 2;
                 for (; headings.has(`${lcId}_${i}`); i++) {
-                    // pass
+                    //
                 }
                 id = `${id}_${i}`;
                 headings.add(`${lcId}_${i}`);

package/dist/src/html.js

@@ -182,10 +182,15 @@ let HtmlToken = (() => {
         lint(start = this.getAbsoluteIndex(), re) {
             const errors = super.lint(start, re), rect = new rect_1.BoundingRect(this, start);
             if (this.name === 'h1' && !this.closing) {
-                errors.push((0, lint_1.generateForSelf)(this, rect, 'h1', '<h1>'));
+                errors.push({
+                    ...(0, lint_1.generateForSelf)(this, rect, 'h1', '<h1>'),
+                    suggestions: [{ desc: 'h2', range: [start + 2, start + 3], text: '2' }],
+                });
             }
             if (this.closest('table-attrs')) {
-                errors.push((0, lint_1.generateForSelf)(this, rect, 'parsing-order', 'HTML tag in table attributes'));
+                const e = (0, lint_1.generateForSelf)(this, rect, 'parsing-order', 'HTML tag in table attributes');
+                e.fix = { desc: 'remove', range: [start, e.endIndex], text: '' };
+                errors.push(e);
             }
             try {
                 this.findMatchingTag();
@@ -193,38 +198,53 @@ let HtmlToken = (() => {
             catch (e) {
                 if (e instanceof SyntaxError) {
                     const { message } = e;
-                    const msg = message.split(':')[0].toLowerCase(), error = (0, lint_1.generateForSelf)(this, rect, 'unmatched-tag', msg);
-                    if (msg === 'unclosed tag' && !this.closest('heading-title')) {
-                        if (formattingTags.has(this.name)) {
-                            const childNodes = this.parentNode?.childNodes, i = childNodes?.indexOf(this);
-                            if (!childNodes?.slice(0, i).some(({ type, name }) => type === 'html' && name === this.name)) {
+                    const msg = message.split(':')[0].toLowerCase(), error = (0, lint_1.generateForSelf)(this, rect, 'unmatched-tag', msg), noSelfClosing = {
+                        desc: 'no self-closing',
+                        range: [error.endIndex - 2, error.endIndex - 1],
+                        text: '',
+                    };
+                    switch (msg) {
+                        case 'unclosed tag': {
+                            const childNodes = this.parentNode?.childNodes;
+                            if (formattingTags.has(this.name)
+                                && childNodes?.slice(0, childNodes.indexOf(this))
+                                    .some(({ type, name }) => type === 'html' && name === this.name)) {
+                                error.suggestions = [{ desc: 'close', range: [start + 1, start + 1], text: '/' }];
+                            }
+                            else if (!this.closest('heading-title')) {
                                 error.severity = 'warning';
                             }
+                            break;
                         }
-                        else {
-                            error.severity = 'warning';
+                        case 'unmatched closing tag': {
+                            const ancestor = this.closest('magic-word');
+                            if (ancestor && magicWords.has(ancestor.name)) {
+                                error.severity = 'warning';
+                            }
+                            else {
+                                error.suggestions = [{ desc: 'remove', range: [start, error.endIndex], text: '' }];
+                            }
+                            break;
                         }
-                    }
-                    else if (msg === 'unmatched closing tag') {
-                        const ancestor = this.closest('magic-word');
-                        if (ancestor && magicWords.has(ancestor.name)) {
-                            error.severity = 'warning';
+                        case 'tag that is both closing and self-closing': {
+                            const { html: [normalTags, , voidTags] } = this.getAttribute('config'), open = { desc: 'open', range: [start + 1, start + 2], text: '' };
+                            if (voidTags.includes(this.name)) {
+                                error.fix = open;
+                            }
+                            else if (normalTags.includes(this.name)) {
+                                error.fix = noSelfClosing;
+                            }
+                            else {
+                                error.suggestions = [open, noSelfClosing];
+                            }
+                            break;
                         }
-                        else {
+                        case 'invalid self-closing tag':
                             error.suggestions = [
-                                {
-                                    desc: 'remove',
-                                    range: [start, error.endIndex],
-                                    text: '',
-                                },
+                                noSelfClosing,
+                                { desc: 'close', range: [error.endIndex - 2, error.endIndex], text: `></${this.name}>` },
                             ];
-                        }
-                    }
-                    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: '', desc: 'open' };
-                        }
+                        // no default
                     }
                     errors.push(error);
                 }

package/dist/src/imageParameter.js

@@ -185,7 +185,9 @@ class ImageParameterToken extends index_2.Token {
             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'));
+            const e = (0, lint_1.generateForSelf)(this, { start }, 'url-encoding', 'unnecessary URL encoding in an internal link');
+            e.suggestions = [{ desc: 'decode', range: [start, e.endIndex], text: (0, string_1.rawurldecode)(this.text()) }];
+            errors.push(e);
         }
         return errors;
     }

package/dist/src/imagemap.js

@@ -101,7 +101,14 @@ class ImagemapToken extends index_2.Token {
             errors.push(...this.childNodes.filter(child => {
                 const str = child.toString().trim();
                 return child.type === 'noinclude' && str && !str.startsWith('#');
-            }).map(child => (0, lint_1.generateForChild)(child, rect, 'invalid-imagemap', 'invalid link in <imagemap>')));
+            }).map(child => {
+                const e = (0, lint_1.generateForChild)(child, rect, 'invalid-imagemap', 'invalid link in <imagemap>');
+                e.suggestions = [
+                    { desc: 'remove', range: [e.startIndex - 1, e.endIndex], text: '' },
+                    { desc: 'comment', range: [e.startIndex, e.startIndex], text: '# ' },
+                ];
+                return e;
+            }));
         }
         else {
             errors.push((0, lint_1.generateForSelf)(this, rect, 'invalid-imagemap', '<imagemap> without an image'));

package/dist/src/index.d.ts

@@ -3,7 +3,7 @@ import { AstElement } from '../lib/element';
 import { AstText } from '../lib/text';
 import type { LintError, TokenTypes } from '../base';
 import type { Title } from '../lib/title';
-import type { AstNodes, ExtToken, IncludeToken, HtmlToken, TranscludeToken, CommentToken, FileToken, LinkToken, RedirectTargetToken, ExtLinkToken, MagicLinkToken, ImageParameterToken } from '../internal';
+import type { AstNodes, IncludeToken, HtmlToken, ExtToken, TranscludeToken, CommentToken, FileToken, LinkToken, RedirectTargetToken, ExtLinkToken, MagicLinkToken, ImageParameterToken } from '../internal';
 import { Ranges } from '../lib/ranges';
 import { AstRange } from '../lib/range';
 import type { Range } from '../lib/ranges';

package/dist/src/index.js

@@ -446,47 +446,35 @@ class Token extends element_1.AstElement {
         index_1.default.viewOnly = true;
         let errors = super.lint(start, re);
         if (this.type === 'root') {
-            const record = new Map(), selector = 'category,html-attr#id,ext-attr#id,table-attr#id,ext-attr#name';
+            const record = new Map(), selector = 'category,html-attr#id,ext-attr#id,table-attr#id';
             for (const cat of this.querySelectorAll(selector)) {
                 let key;
                 if (cat.type === 'category') {
                     key = cat.name;
                 }
                 else {
-                    const value = cat.getValue(), attrs = cat.parentNode;
-                    if (cat.name === 'id') {
-                        key = `#${value === true ? '' : value}`;
+                    const value = cat.getValue();
+                    if (value && value !== true) {
+                        key = `#${value}`;
                     }
-                    else if (cat.tag === 'ref' && value !== true && value
-                        && attrs.parentNode.innerText) {
-                        const group = attrs.getAttr('group');
-                        key = `${typeof group === 'string' && group || ' '}#${value}`;
+                }
+                if (key) {
+                    const thisCat = record.get(key);
+                    if (thisCat) {
+                        thisCat.add(cat);
                     }
                     else {
-                        continue;
+                        record.set(key, new Set([cat]));
                     }
                 }
-                const thisCat = record.get(key);
-                if (thisCat) {
-                    thisCat.add(cat);
-                }
-                else {
-                    record.set(key, new Set([cat]));
-                }
             }
             for (const [key, value] of record) {
                 if (value.size > 1 && !key.startsWith('#mw-customcollapsible-')) {
-                    const isCat = !key.includes('#'), msg = `duplicated ${isCat ? 'category' : 'id/name'}`, severity = key.startsWith('#') ? 'warning' : 'error';
+                    const isCat = !key.startsWith('#'), msg = `duplicated ${isCat ? 'category' : 'id'}`, severity = isCat ? 'error' : 'warning';
                     errors.push(...[...value].map(cat => {
                         const e = (0, lint_1.generateForSelf)(cat, { start: cat.getAbsoluteIndex() }, 'no-duplicate', msg, severity);
                         if (isCat) {
-                            e.suggestions = [
-                                {
-                                    desc: 'remove',
-                                    range: [e.startIndex, e.endIndex],
-                                    text: '',
-                                },
-                            ];
+                            e.suggestions = [{ desc: 'remove', range: [e.startIndex, e.endIndex], text: '' }];
                         }
                         return e;
                     }));

package/dist/src/link/base.js

@@ -3,13 +3,14 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.LinkBaseToken = void 0;
 const lint_1 = require("../../util/lint");
 const constants_1 = require("../../util/constants");
+const string_1 = require("../../util/string");
 const rect_1 = require("../../lib/rect");
 const index_1 = require("../../index");
 const index_2 = require("../index");
 const atom_1 = require("../atom");
 /* NOT FOR BROWSER */
 const debug_1 = require("../../util/debug");
-const string_1 = require("../../util/string");
+const string_2 = require("../../util/string");
 const html_1 = require("../../util/html");
 /* NOT FOR BROWSER END */
 /**
@@ -156,7 +157,9 @@ class LinkBaseToken extends index_2.Token {
             errors.push((0, lint_1.generateForChild)(target, rect, 'unknown-page', 'template in an internal link target', 'warning'));
         }
         if (encoded) {
-            errors.push((0, lint_1.generateForChild)(target, rect, 'url-encoding', 'unnecessary URL encoding in an internal link'));
+            const e = (0, lint_1.generateForChild)(target, rect, 'url-encoding', 'unnecessary URL encoding in an internal link');
+            e.suggestions = [{ desc: 'decode', range: [e.startIndex, e.endIndex], text: (0, string_1.rawurldecode)(target.text()) }];
+            errors.push(e);
         }
         if (type === 'link' || type === 'category') {
             const textNode = linkText?.childNodes.find((c) => c.type === 'text' && c.data.includes('|'));
@@ -221,7 +224,7 @@ class LinkBaseToken extends index_2.Token {
     setFragment(fragment) {
         const { type, name } = this;
         if (fragment === undefined || isLink(type)) {
-            fragment &&= (0, string_1.encode)(fragment);
+            fragment &&= (0, string_2.encode)(fragment);
             this.setTarget(`${name}${fragment === undefined ? '' : `#${fragment}`}`);
         }
     }
@@ -252,7 +255,7 @@ class LinkBaseToken extends index_2.Token {
             const { link, length, lastChild, type } = this, title = link.getTitleAttr();
             return (0, html_1.font)(this, `<a${link.interwiki && ' class="extiw"'} href="${link.getUrl()}"${title && ` title="${title}"`}>${type === 'link' && length > 1
                 ? lastChild.toHtmlInternal({ ...opt, nowrap: true })
-                : (0, string_1.sanitize)(this.innerText)}</a>`);
+                : (0, string_2.sanitize)(this.innerText)}</a>`);
         }
         return '';
     }

package/dist/src/link/file.js

@@ -150,7 +150,11 @@ class FileToken extends base_1.LinkBaseToken {
          * @param msg 消息键
          * @param p1 替换$1
          */
-        const generate = (msg, p1) => (arg) => (0, lint_1.generateForChild)(arg, rect, 'no-duplicate', index_1.default.msg(`${msg} image $1 parameter`, p1));
+        const generate = (msg, p1) => (arg) => {
+            const e = (0, lint_1.generateForChild)(arg, rect, 'no-duplicate', index_1.default.msg(`${msg} image $1 parameter`, p1));
+            e.suggestions = [{ desc: 'remove', range: [e.startIndex - 1, e.endIndex], text: '' }];
+            return e;
+        };
         for (const key of keys) {
             if (key === 'invalid' || key === 'width' && unscaled) {
                 continue;

package/dist/src/link/galleryImage.js

@@ -108,7 +108,9 @@ let GalleryImageToken = (() => {
         lint(start = this.getAbsoluteIndex(), re) {
             const errors = super.lint(start, re), { ns, interwiki } = this.getAttribute('title');
             if (interwiki || ns !== 6) {
-                errors.push((0, lint_1.generateForSelf)(this, { start }, 'invalid-gallery', 'invalid gallery image'));
+                const e = (0, lint_1.generateForSelf)(this, { start }, 'invalid-gallery', 'invalid gallery image');
+                e.suggestions = [{ desc: 'prefix', range: [start, start], text: 'File:' }];
+                errors.push(e);
             }
             return errors;
         }