feat: support shorthand JSX fragments (JamesHenry#15)

Author: wsabransky

lib/ast-node-types.js

@@ -62,13 +62,16 @@ module.exports = {
   ImportSpecifier: 'ImportSpecifier',
   JSXAttribute: 'JSXAttribute',
   JSXClosingElement: 'JSXClosingElement',
+  JSXClosingFragment: 'JSXClosingFragment',
   JSXElement: 'JSXElement',
   JSXEmptyExpression: 'JSXEmptyExpression',
   JSXExpressionContainer: 'JSXExpressionContainer',
+  JSXFragment: 'JSXFragment',
   JSXIdentifier: 'JSXIdentifier',
   JSXMemberExpression: 'JSXMemberExpression',
   JSXNamespacedName: 'JSXNamespacedName',
   JSXOpeningElement: 'JSXOpeningElement',
+  JSXOpeningFragment: 'JSXOpeningFragment',
   JSXSpreadAttribute: 'JSXSpreadAttribute',
   JSXSpreadChild: 'JSXSpreadChild',
   JSXText: 'JSXText',

lib/convert.js

@@ -1976,6 +1976,15 @@ module.exports = function convert(config) {
 
       break;
 
+    case SyntaxKind.JsxFragment:
+      Object.assign(result, {
+        type: AST_NODE_TYPES.JSXFragment,
+        openingElement: convertChild(node.openingElement),
+        closingElement: convertChild(node.closingElement),
+        children: node.children.map(convertChild)
+      });
+      break;
+
     case SyntaxKind.JsxSelfClosingElement: {
       /**
        * Convert SyntaxKind.JsxSelfClosingElement to SyntaxKind.JsxOpeningElement,
@@ -2015,6 +2024,18 @@ module.exports = function convert(config) {
       });
       break;
 
+    case SyntaxKind.JsxOpeningFragment:
+      Object.assign(result, {
+        type: AST_NODE_TYPES.JSXOpeningFragment,
+        selfClosing: false
+      });
+      break;
+    case SyntaxKind.JsxClosingFragment:
+      Object.assign(result, {
+        type: AST_NODE_TYPES.JSXClosingFragment
+      });
+      break;
+
     case SyntaxKind.JsxExpression: {
       const eloc = ast.getLineAndCharacterOfPosition(result.range[0] + 1);
       const expression = node.expression

tests/ast-alignment/fixtures-to-test.js

@@ -19,6 +19,12 @@ const jsxFilesWithKnownIssues = require('../jsx-known-issues').map(f =>
  */
 jsxFilesWithKnownIssues.push('invalid-no-tag-name');
 
+/**
+ * JSX shorthand fragments are not supported by Babylon
+ */
+jsxFilesWithKnownIssues.push('shorthand-fragment');
+jsxFilesWithKnownIssues.push('shorthand-fragment-with-child');
+
 /**
  * Custom constructs/concepts used in this file:
  *

tests/fixtures/jsx/invalid-shorthand-fragment-no-closing.src.js

@@ -0,0 +1 @@
+<><div></div>

tests/fixtures/jsx/shorthand-fragment-with-child.src.js

@@ -0,0 +1 @@
+<><div /></>

tests/fixtures/jsx/shorthand-fragment.src.js

@@ -0,0 +1 @@
+<></>