Handle default parameters with flow types

Input:

const myObj = {
  method: function (foo: number = 10) {
    return foo;
  },
};

Tranform:

module.exports = (file, api, options) => {
  const j = api.jscodeshift;
  const printOptions = options.printOptions || {quote: 'single'};
  const root = j(file.source);


  const canBeSimplified = (key, value) => {
    return key.type === 'Identifier' && value.type === 'FunctionExpression';
  };

  root
    .find(j.Property)
    .filter(p => canBeSimplified(p.value.key, p.value.value))
    .forEach(p => {
      p.value.method = true;
    });

  return root.toSource(printOptions);
};

Current (broken) Output:

const myObj = {
  method(= 10) {
    return foo;
  },
};

Expected output:

const myObj = {
  method(foo: number = 10) {
    return foo;
  },
};

[ForbesLindesay]

Yes, you can see what I’m working on at https://github.com/cpojer/js-codemod/pull/52

I’m trying to get it accurate enough to run at Facebook.

[hzoo]

lol what! I encountered this issue too and mentioned it in the babel slack - made https://github.com/babel/babylon/issues/67

Oh probably since we are making the same thing! (except with babel/jscodeshift)