Join Declaration/Assignment about babel-transform-js-to-jsx HOT 7 CLOSED

@paulroper you had a possible solution for this? Want to PR it?

from babel-transform-js-to-jsx.

I didn't hit a solution for multi-line declarations unfortunately, only single line. 😞

from babel-transform-js-to-jsx.

Can you share the AST explorer sample you had? We can adapt it for multi-line and deal with scoping.

from babel-transform-js-to-jsx.

...I didn't save it because I dumb am 👍

from babel-transform-js-to-jsx.

I had a mess around in the AST Explorer again and came up with this:

export default function ({types: t}) {
  return {
    visitor: {      
      ExpressionStatement(path) {
        if (path.node.expression.type !== "AssignmentExpression") { return; }

        const left = path.node.expression.left;
        const binding = path.scope.bindings[left.name];

        if (binding && !binding.path.shouldSkip) {
          path.replaceWith(
            t.variableDeclaration("var", [t.VariableDeclarator(left, path.node.expression.right)])
          );

          binding.path.remove(); 
        }
      }
    }
  };
}

It'll turn:

var a, b, c;
a = 1;
b = 3;
c = 4;

Into:

var a = 1;
var b = 3;
var c = 4;

from babel-transform-js-to-jsx.

Great! Can you look into a good way to check the scope?

from babel-transform-js-to-jsx.

Check the scope of the transformed variable assignments?

from babel-transform-js-to-jsx.