New: Create option to enable JSXText node type (fixes #266)

Author: soda0289

lib/ast-converter.js

@@ -56,7 +56,8 @@ module.exports = (ast, extra) => {
         parent: null,
         ast,
         additionalOptions: {
-            errorOnUnknownASTType: extra.errorOnUnknownASTType || false
+            errorOnUnknownASTType: extra.errorOnUnknownASTType || false,
+            useJSXTextNode: extra.useJSXTextNode || false
         }
     });
 

lib/convert.js

@@ -1613,16 +1613,25 @@ module.exports = function convert(config) {
 
         }
 
+        /**
+         * The JSX AST changed the node type for string literals
+         * inside a JSX Element from `Literal` to `JSXText`. We
+         * provide a flag to support both types until `Literal`
+         * node type is deprecated in ESLint v5.
+         */
         case SyntaxKind.JsxText: {
-            Object.assign(result, {
-                type: AST_NODE_TYPES.Literal,
-                value: ast.text.slice(node.pos, node.end),
-                raw: ast.text.slice(node.pos, node.end)
-            });
-
             const start = node.getFullStart();
             const end = node.getEnd();
 
+            const type = (additionalOptions.useJSXTextNode)
+                ? AST_NODE_TYPES.JSXText : AST_NODE_TYPES.Literal;
+
+            Object.assign(result, {
+                type,
+                value: ast.text.slice(start, end),
+                raw: ast.text.slice(start, end)
+            });
+
             result.loc = nodeUtils.getLocFor(start, end, ast);
             result.range = [start, end];
 

parser.js

@@ -48,7 +48,8 @@ function resetExtra() {
         tolerant: false,
         errors: [],
         strict: false,
-        ecmaFeatures: {}
+        ecmaFeatures: {},
+        useJSXTextNode: false
     };
 }
 
@@ -104,6 +105,10 @@ function parse(code, options) {
             extra.errorOnUnknownASTType = true;
         }
 
+        if (typeof options.useJSXTextNode === "boolean" && options.useJSXTextNode) {
+            extra.useJSXTextNode = true;
+        }
+
     }
 
     // Even if jsx option is set in typescript compiler, filename still has to