Flow annotation is incorrectly removed?

[kangax]

require('babel-core').transform('export default function(foo: string, bar: string = baz()) { }', {plugins:['syntax-flow'],passPerPreset:true,presets:['babili']}).code

gives me this:

export default function (a: string, b = baz()) {}

Shouldn't this be:

export default function (a: string, b: string = baz()) {}

/cc @boopathi

[boopathi]

Interesting that it didn't happen for a - mangler replaces the identifier and will create a new identifier and this one won't have the type information.

Just curious - why would you want a type annotation for minified code - don't we assume the output is JavaScript? If we do some unsafe transformations, types are going to mismatch somewhere and flow is going to throw an error on the transformed code.

[kangax]

I stumbled at this when running a smoke test (on Babel repo, actually). It was erroring out without 'syntax-flow' plugin so once I added that I noticed this inconsistency in output. So this is purely theoretical at this point but still shows that something is off.

[boopathi]

With #390, all bindings should get flow removed, as we were doing previously binding.identifier.name = newName. Now we replace the identifier using path. So flow annotation should be removed.

[kangax]

Hm, I still get export default function (foo: string, bar: string = baz()) {} even on latest master