fix: incorrect JSXAttribute position info

close #488, related #425
mdx-js · Dec 11, 2023 · 55130c7 · 55130c7
1 parent f6bcfb0
commit 55130c7
Show file tree

Hide file tree

Showing 3 changed files with 582 additions and 805 deletions.
diff --git a/packages/eslint-mdx/src/worker.ts b/packages/eslint-mdx/src/worker.ts
@@ -633,7 +633,7 @@ runAsWorker(
                   }
 
                   return {
-                    ...attrNamePos,
+                    ...normalizeNode(attrStart, lastAttrOffset + 1),
                     type: 'JSXAttribute',
                     name: {
                       ...attrNamePos,
@@ -713,11 +713,16 @@ runAsWorker(
 
         const expression = handleNode(node)
 
+        // keep for debugging
+        // if (+1 === 1) {
+        //   throw new SyntaxError(JSON.stringify({ node, expression }, null, 2))
+        // }
+
         if (expression) {
           body.push({
             ...normalizePosition(node.position),
             type: 'ExpressionStatement',
-            expression: handleNode(node) as Expression,
+            expression: expression as Expression,
           })
         }