@markuplint/vue-parser 4.0.0-alpha.2 → 4.0.0-alpha.4

package/lib/attr.js

@@ -1,10 +1,10 @@
-const duplicatableAttrs = ['class', 'style'];
+const duplicatableAttrs = new Set(['class', 'style']);
 export function attr(
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
 attr) {
     if (attr.type !== 'html-attr') {
         const name = attr.potentialName;
-        if (duplicatableAttrs.includes(name)) {
+        if (duplicatableAttrs.has(name)) {
             attr.isDuplicatable = true;
         }
         return attr;
@@ -30,7 +30,7 @@ attr) {
          */
         const [, directive, potentialName, modifier] = attr.name.raw.match(/^(v-bind:|:)([^.]+)(?:\.([^.]+))?$/i) ?? [];
         if (directive && potentialName) {
-            if (duplicatableAttrs.includes(potentialName.toLowerCase())) {
+            if (duplicatableAttrs.has(potentialName.toLowerCase())) {
                 attr.isDuplicatable = true;
             }
             if (!modifier) {
@@ -50,6 +50,7 @@ attr) {
                         _modifier: modifier,
                     };
                 }
+                /* eslint-disable unicorn/no-useless-switch-case */
                 // TODO: Supporting for `prop` and `camel` https://github.com/markuplint/markuplint/pull/681
                 case '.prop':
                 case '.camel':
@@ -57,12 +58,13 @@ attr) {
                     const name = `v-bind:${potentialName}${modifier ?? ''}`;
                     return {
                         ...attr,
-                        potentialName: attr.name.raw !== name ? name : undefined,
+                        potentialName: attr.name.raw === name ? undefined : name,
                         isDirective: true,
                         // @ts-ignore
                         _modifier: modifier,
                     };
                 }
+                /* eslint-enable unicorn/no-useless-switch-case */
             }
         }
     }
@@ -85,12 +87,12 @@ attr) {
         /**
          * `v-slot`
          */
-        const [, , slotName] = attr.name.raw.match(/^(v-slot:|#)(.+)$/i) ?? [];
+        const slotName = (attr.name.raw.match(/^(v-slot:|#)(.+)$/i) ?? [])[2];
         const name = `v-slot:${slotName}`;
         if (slotName) {
             return {
                 ...attr,
-                potentialName: attr.name.raw !== name ? name : undefined,
+                potentialName: attr.name.raw === name ? undefined : name,
                 isDirective: true,
             };
         }
@@ -98,7 +100,7 @@ attr) {
     /**
      * If directives
      */
-    if (/^v-/.test(attr.name.raw)) {
+    if (attr.name.raw.startsWith('v-')) {
         return {
             ...attr,
             isDirective: true,

package/lib/parse.js

@@ -7,26 +7,26 @@ export const parse = (rawCode, options) => {
     try {
         ast = vueParse(rawCode);
     }
-    catch (err) {
-        if (err instanceof SyntaxError && 'lineNumber' in err && 'column' in err) {
+    catch (error) {
+        if (error instanceof SyntaxError && 'lineNumber' in error && 'column' in error) {
             throw new ParserError(
             // @ts-ignore
-            err.message, {
+            error.message, {
                 // @ts-ignore
-                line: err.lineNumber,
+                line: error.lineNumber,
                 // @ts-ignore
-                col: err.column,
+                col: error.column,
                 raw: '',
             });
         }
         return {
             nodeList: [],
             isFragment: true,
-            parseError: err instanceof Error ? err.message : new Error(`${err}`).message,
+            parseError: error instanceof Error ? error.message : new Error(`${error}`).message,
         };
     }
     const nodeList = ast.templateBody
-        ? flattenNodes(traverse(ast.templateBody, null, rawCode, options), rawCode)
+        ? flattenNodes(traverse(ast.templateBody, null, rawCode, ast.templateBody.comments, options), rawCode)
         : [];
     // Remove `</template>`
     const templateEndTagIndex = nodeList.findIndex(node => /\s*<\/\s*template\s*>\s*/i.test(node.raw));
@@ -46,17 +46,50 @@ export const parse = (rawCode, options) => {
 };
 function traverse(
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-rootNode, parentNode = null, rawHtml, options) {
+rootNode, parentNode = null, rawHtml, 
+// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
+comments, options) {
     const nodeList = [];
     if (rootNode.type !== 'VElement') {
         return [];
     }
     let prevNode = null;
     for (const vNode of rootNode.children) {
-        const node = nodeize(vNode, prevNode, parentNode, rawHtml, options);
+        const node = nodeize(vNode, prevNode, parentNode, rawHtml, comments, options);
         if (!node) {
             continue;
         }
+        const lastOffset = prevNode?.endOffset ?? parentNode?.endOffset ?? 0;
+        const betweenComment = comments.find(comment => {
+            return lastOffset <= comment.range[0] && comment.range[1] <= node.startOffset;
+        });
+        if (betweenComment) {
+            const comment = {
+                uuid: uuid(),
+                raw: `<!--${betweenComment.value}-->`,
+                startOffset: betweenComment.range[0],
+                endOffset: betweenComment.range[1],
+                startLine: betweenComment.loc.start.line,
+                endLine: betweenComment.loc.end.line,
+                startCol: betweenComment.loc.start.column + 1,
+                endCol: betweenComment.loc.end.column + 1,
+                nodeName: '#comment',
+                type: 'comment',
+                parentNode,
+                prevNode,
+                nextNode: node,
+                isFragment: false,
+                isGhost: false,
+            };
+            comment.parentNode = parentNode;
+            comment.prevNode = prevNode;
+            comment.nextNode = node;
+            if (prevNode) {
+                prevNode.nextNode = comment;
+            }
+            nodeList.push(comment);
+            prevNode = comment;
+        }
         if (prevNode) {
             if (node.type !== 'endtag') {
                 prevNode.nextNode = node;
@@ -74,7 +107,9 @@ originNode,
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
 prevNode, 
 // eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
-parentNode, rawHtml, options) {
+parentNode, rawHtml, 
+// eslint-disable-next-line @typescript-eslint/prefer-readonly-parameter-types
+comments, options) {
     const nextNode = null;
     const startOffset = originNode.range[0];
     const endOffset = originNode.range[1];
@@ -201,7 +236,7 @@ parentNode, rawHtml, options) {
             if (endTag) {
                 endTag.pearNode = startTag;
             }
-            startTag.childNodes = traverse(originNode, startTag, rawHtml, options);
+            startTag.childNodes = traverse(originNode, startTag, rawHtml, comments, options);
             return startTag;
         }
     }

package/lib/vue-parser/index.d.ts

@@ -2,3 +2,4 @@ import * as VueESLintParser from 'vue-eslint-parser';
 export type VueTokens = VueESLintParser.AST.ESLintProgram;
 export default function vueParse(vueTemplate: string): VueTokens;
 export type ASTNode = VueESLintParser.AST.VElement | VueESLintParser.AST.VText | VueESLintParser.AST.VExpressionContainer;
+export type ASTComment = VueESLintParser.AST.Token & VueESLintParser.AST.HasLocation;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@markuplint/vue-parser",
-	"version": "4.0.0-alpha.2",
+	"version": "4.0.0-alpha.4",
 	"description": "Vue parser for markuplint",
 	"repository": "git@github.com:markuplint/markuplint.git",
 	"author": "Yusuke Hirao <yusukehirao@me.com>",
@@ -21,11 +21,10 @@
 		"clean": "tsc --build --clean"
 	},
 	"dependencies": {
-		"@markuplint/html-parser": "4.0.0-alpha.2",
-		"@markuplint/ml-ast": "4.0.0-alpha.2",
-		"@markuplint/parser-utils": "4.0.0-alpha.2",
-		"tslib": "^2.6.2",
+		"@markuplint/html-parser": "4.0.0-alpha.4",
+		"@markuplint/ml-ast": "4.0.0-alpha.4",
+		"@markuplint/parser-utils": "4.0.0-alpha.4",
 		"vue-eslint-parser": "^9.3.1"
 	},
-	"gitHead": "51ad52c760435641f9bff8685707d8178b0787eb"
+	"gitHead": "991b3aef77fde42c79343ee8c807257a35c589d7"
 }