Fixes an issue where ts-plugin crashes on invalid TSX data. Closes #7

Author: Havunen

src/index.ts

@@ -387,7 +387,7 @@ export default () => {
                 }
             }
 
-            if (vChildren && vChildren !== null && childrenResults.foundText) {
+            if (vChildren && childrenResults.foundText) {
                 vChildren = transformTextNodes(vChildren)
             }
 
@@ -576,7 +576,7 @@ export default () => {
                         propsPropertyAssignments.push(
                             factory.createPropertyAssignment(
                                 factory.createStringLiteral(svgAttributes[propName]),
-                                getValue(initializer, visitor, factory)
+                                initializer ? getValue(initializer, visitor, factory) : factory.createTrue()
                             )
                         )
                     } else {
@@ -614,10 +614,10 @@ export default () => {
                                 childrenKnown = true
                                 break
                             case 'ref':
-                                ref = getValue(initializer, visitor, factory)
+                                ref = initializer ? getValue(initializer, visitor, factory) : null
                                 break
                             case 'key':
-                                key = getValue(initializer, visitor, factory)
+                                key = initializer ? getValue(initializer, visitor, factory) : null
                                 break
                             case PROP_ReCreate:
                                 hasReCreateFlag = true

src/utils/getValue.ts

@@ -5,6 +5,9 @@ export default function getValue(node, visitor, factory) {
         return factory.createStringLiteral(node.text);
     }
     if (node.kind === SyntaxKind.JsxExpression) {
+        if (!node.expression) {
+            return factory.createNull();
+        }
         return visitor(node.expression);
     }
 }

src/utils/isNodeNull.ts

@@ -4,11 +4,17 @@ export default function isNodeNull(node) {
     if (!node) {
         return true;
     }
+
+    if (node.kind === SyntaxKind.NullKeyword) {
+        return true;
+    }
+
     if (
         node.kind === SyntaxKind.ArrayLiteralExpression &&
         node.elements.length === 0
     ) {
         return true;
     }
+
     return node.text === "null";
 }

tests/cases/empty.tsx

@@ -0,0 +1 @@
+<div ref={}/>

tests/cases/empty_2.tsx

@@ -0,0 +1 @@
+<div ref={} a={1} className="test"/>

tests/cases/empty_3.tsx

@@ -0,0 +1 @@
+<NewComponent ref={} a={} className={} class={} a={} b c/>

tests/cases/empty_4.tsx

@@ -0,0 +1 @@
+<svg key={} dominantBaseline clipRule={} capHeight={3}/>

tests/references/empty.jsx

@@ -0,0 +1,3 @@
+var $inferno = require("inferno");
+var createVNode = $inferno.createVNode;
+createVNode(1, "div");

tests/references/empty_2.jsx

@@ -0,0 +1,3 @@
+var $inferno = require("inferno");
+var createVNode = $inferno.createVNode;
+createVNode(1, "div", "test", null, 1, { "a": 1 });

tests/references/empty_3.jsx

@@ -0,0 +1,3 @@
+var $inferno = require("inferno");
+var createComponentVNode = $inferno.createComponentVNode;
+createComponentVNode(2, NewComponent, { "a": null, "className": null, "class": null, "a": null, "b": true, "c": true });