wikilint 2.20.2 → 2.20.3

package/dist/bin/config.js

@@ -116,7 +116,6 @@ exports.default = async (site, url, force, internal) => {
             ...ns.map(([id, canonical]) => [canonical.toLowerCase(), Number(id)]),
             ...namespacealiases.filter(({ id }) => filterGadget(id)).map(({ id, alias }) => [alias.toLowerCase(), id]),
         ]),
-        functionHook: [...functionhooks.map(s => s.toLowerCase()), 'msgnw'],
         articlePath: articlepath,
     };
     config.doubleUnderscore[0] = [];
@@ -124,6 +123,9 @@ exports.default = async (site, url, force, internal) => {
     Object.assign(config.parserFunction[0], (0, cm_1.getConfig)(magicwords, ({ name }) => name === 'msgnw'));
     config.parserFunction[2] = getAliases(magicwords, new Set(['msg', 'raw']));
     config.parserFunction[3] = getAliases(magicwords, new Set(['subst', 'safesubst']));
+    if (!mwConfig.functionHooks) {
+        Object.assign(config, { functionHook: [...functionhooks.map(s => s.toLowerCase()), 'msgnw'] });
+    }
     if (!mwConfig.variableIDs) {
         const { query: { variables } } = await (await fetch(`${url}/api.php?${new URLSearchParams({ ...params, siprop: 'variables' }).toString()}`)).json();
         Object.assign(config, { variable: [...new Set([...variables, '='])] });

package/dist/lib/text.js

@@ -101,6 +101,7 @@ class AstText extends node_1.AstNode {
                 isHtmlAttrVal = true;
             }
             else if (tag === 'ref' && (grandName === 'name' || grandName === 'extends' || grandName === 'follow')
+                || grandName === 'group' && (tag === 'ref' || tag === 'references')
                 || tag === 'choose' && (grandName === 'before' || grandName === 'after')) {
                 return [];
             }
@@ -116,7 +117,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(), rootStr = root.toString(), { ext, html } = root.getAttribute('config'), { top, left } = root.posFromIndex(start), tags = new Set([
+        const errors = [], nextType = nextSibling?.type, nextName = nextSibling?.name, previousType = previousSibling?.type, root = this.getRootNode(), rootStr = root.toString(), { ext, html, variants } = root.getAttribute('config'), { top, left } = root.posFromIndex(start), tags = new Set([
             'onlyinclude',
             'noinclude',
             'includeonly',
@@ -135,7 +136,8 @@ class AstText extends node_1.AstNode {
                 error = error.slice(length);
             }
             error = error.toLowerCase();
-            const { 0: char, length } = error, magicLink = char === 'r' || char === 'p' || char === 'i';
+            const [char] = error, magicLink = char === 'r' || char === 'p' || char === 'i';
+            let { length } = error;
             if (char === '<' && !tags.has(tag.toLowerCase())
                 || char === '[' && type === 'ext-link-text' && (/&(?:rbrack|#93|#x5[Dd];);/u.test(data.slice(index + 1))
                     || nextSibling?.is('ext') && nextName === 'nowiki'
@@ -146,12 +148,14 @@ class AstText extends node_1.AstNode {
             else if (char === ']' && (index || length > 1)) {
                 errorRegex.lastIndex--;
             }
-            const startIndex = start + index, endIndex = startIndex + length, nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1], severity = length > 1 && !(char === '<' && (/^<\s/u.test(error) || !/[\s/>]/u.test(nextChar ?? '') || disallowedTags.has(tag))
+            let startIndex = start + index, endIndex = startIndex + length;
+            const nextChar = rootStr[endIndex], previousChar = rootStr[startIndex - 1];
+            let severity = length > 1 && !(char === '<' && (/^<\s/u.test(error) || !/[\s/>]/u.test(nextChar ?? '') || disallowedTags.has(tag))
                 || isHtmlAttrVal && (char === '[' || char === ']')
                 || magicLink && type === 'parameter-value'
                 || /^(?:rfc|pmid|isbn)$/iu.test(error))
-                || char === '{' && (nextChar === char || previousChar === '-')
-                || char === '}' && (previousChar === char || nextChar === '-')
+                || char === '{' && (nextChar === char || previousChar === '-' && variants.length > 0)
+                || char === '}' && (previousChar === char || nextChar === '-' && variants.length > 0)
                 || char === '[' && (type === 'ext-link-text' || nextType === 'free-ext-link' && !data.slice(index + 1).trim())
                 || char === ']' && previousType === 'free-ext-link'
                     && !data.slice(0, index).includes(']')
@@ -178,9 +182,26 @@ class AstText extends node_1.AstNode {
             if (magicLink) {
                 error = error.toUpperCase();
             }
+            else if (error === '{' && previousChar === '-' && severity === 'error') {
+                severity = 'warning';
+                if (index > 0) {
+                    error = '-{';
+                    index--;
+                    startIndex--;
+                    length = 2;
+                }
+            }
+            else if (error === '}' && nextChar === '-' && severity === 'error') {
+                severity = 'warning';
+                if (index < data.length - 1) {
+                    error = '}-';
+                    endIndex++;
+                    length = 2;
+                }
+            }
             const pos = this.posFromIndex(index), { line: startLine, character: startCol } = (0, lint_1.getEndPos)(top, left, pos.top + 1, pos.left), e = {
                 rule: ruleMap[char],
-                message: index_1.default.msg('lonely "$1"', magicLink || char === 'h' ? error : char),
+                message: index_1.default.msg('lonely "$1"', magicLink || char === 'h' || error === '-{' || error === '}-' ? error : char),
                 severity,
                 startIndex,
                 endIndex,

package/dist/src/arg.js

@@ -114,13 +114,6 @@ let ArgToken = (() => {
         /** @private */
         lint(start = this.getAbsoluteIndex(), re) {
             const { childNodes: [argName, argDefault, ...rest] } = this;
-            if (!this.getAttribute('include')) {
-                const e = (0, lint_1.generateForSelf)(this, { start }, 'no-arg', 'unexpected template argument');
-                if (argDefault) {
-                    e.suggestions = [{ range: [start, e.endIndex], text: argDefault.text(), desc: 'expand' }];
-                }
-                return [e];
-            }
             argName.setAttribute('aIndex', start + 3);
             const errors = argName.lint(start + 3, re);
             if (argDefault) {
@@ -144,6 +137,13 @@ let ArgToken = (() => {
                     return e;
                 }));
             }
+            if (!this.getAttribute('include')) {
+                const e = (0, lint_1.generateForSelf)(this, { start }, 'no-arg', 'unexpected template argument', 'warning');
+                if (argDefault) {
+                    e.suggestions = [{ range: [start, e.endIndex], text: argDefault.text(), desc: 'expand' }];
+                }
+                errors.push(e);
+            }
             return errors;
         }
     };

package/dist/src/link/base.js

@@ -157,7 +157,7 @@ let LinkBaseToken = (() => {
                 }
             }
             if (fragment !== undefined && !isLink(type)) {
-                const e = (0, lint_1.generateForChild)(target, rect, 'no-ignored', 'useless fragment'), j = target.childNodes.findIndex(c => c.type === 'text' && c.data.includes('#')), textNode = target.childNodes[j];
+                const e = (0, lint_1.generateForChild)(target, rect, 'no-ignored', 'useless fragment', 'warning'), j = target.childNodes.findIndex(c => c.type === 'text' && c.data.includes('#')), textNode = target.childNodes[j];
                 if (textNode) {
                     e.fix = {
                         range: [

package/dist/src/transclude.js

@@ -274,7 +274,7 @@ let TranscludeToken = (() => {
                 const child = childNodes[invoke ? 1 : 0], i = child.childNodes
                     .findIndex(c => c.type === 'text' && (0, string_1.decodeHtml)(c.data).includes('#')), textNode = child.childNodes[i];
                 if (textNode) {
-                    const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'useless fragment');
+                    const e = (0, lint_1.generateForChild)(child, rect, 'no-ignored', 'useless fragment', 'warning');
                     e.fix = {
                         range: [
                             e.startIndex + child.getRelativeIndex(i) + textNode.data.indexOf('#'),

package/i18n/zh-hans.json

@@ -4,7 +4,7 @@
 	"additional \"|\" in a table cell": "表格单元格中多余的\"|\"",
 	"additional \"|\" in the link text": "链接文本中多余的\"|\"",
 	"attributes of a closing tag": "位于闭合标签的属性",
-	"bold": "粗体单引号",
+	"bold apostrophes": "粗体单引号",
 	"bold in section header": "段落标题中的粗体",
 	"conflicting image $1 parameter": "冲突的图片$1参数",
 	"containing invalid attribute": "包含无效属性",
@@ -35,7 +35,7 @@
 	"invalid parameter of <$1>": "<$1>的无效参数",
 	"invalid self-closing tag": "无效自封闭标签",
 	"invisible content inside triple braces": "三重括号内的不可见部分",
-	"italic": "斜体单引号",
+	"italic apostrophes": "斜体单引号",
 	"lonely \"$1\"": "孤立的\"$1\"",
 	"missing module function": "缺少模块函数",
 	"nonzero tabindex": "不为0的tabindex",

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "wikilint",
-	"version": "2.20.2",
+	"version": "2.20.3",
 	"description": "A Node.js linter for MediaWiki markup",
 	"keywords": [
 		"mediawiki",
@@ -66,7 +66,7 @@
 		]
 	},
 	"dependencies": {
-		"@bhsd/common": "^0.9.4",
+		"@bhsd/common": "^0.10.0",
 		"vscode-languageserver-types": "^3.17.5"
 	},
 	"optionalDependencies": {