Rollup mangles class names, breaking IDE code-complete

[cowwoc]

Version 0.41.6
Webstorm 2017.1.1 (IDE)

1. Every time a module imports a class and re-exports it, rollup mangles the name of the exported class. I am expecting the name to remain unchanged.
2. Testcase
3. The main entry point is supposed to return a function createParent() that returns an instance of type Parent.
4. Unfortunately, rollup mangles the class name of Parent into Parent$1 so when the IDE parses jsdoc it sees that createParent() returns Parent but it cannot find class Parent anywhere. As a result, code-complete does not show Parent.childMethod().
5. I am not aware of any workaround for this issue. I am forced to use this design to deal with circular dependencies between modules and these cannot be removed.

[kzc]

Workaround - instead of

export default Parent;

use

export { Parent as default };

[cowwoc]

Thanks for the workaround! I really appreciate it.

1. So is this a confirmed bug?
2. Out of curiosity, what is the difference between the two syntaxes? Is this a rollup implementation detail, or are they conceptually different according to the ES6 specification?

[kzc]

I don't know if it's a bug, nor am I very well versed on what the ES6 spec says on the matter.

[kzc]

It appears to be ES2015 spec compliant.

Your example is re-exporting a live binding which is why it requires the export { Parent as default } syntax.

See:

• http://stackoverflow.com/questions/39276608/is-there-a-difference-between-export-default-x-and-export-x-as-default/39277065#39277065

• Default import/export is not treated as live binding Default import/export is not treated as live binding #1078

• [BREAKING] fix behaviour of export { foo as default } [BREAKING] fix behaviour of export { foo as default } #1166

[cowwoc]

FYI, I published a related bug report for Babel: babel/babel#5660

[cowwoc]

@kzc Thanks for the references. Rollup's behavior makes a lot more sense now.

[fregante]

Was this also solved by https://youtrack.jetbrains.com/issue/WEB-26724?

[cowwoc]

@bfred-it yes