Using Acorn plugins doesn't work as specified by README about rollup-plugin-cleanup

[jcowman2]

Thanks for this, @aMarCruz! Following those steps fixed the issue in my project.

I'm looking forward to seeing your permanent solution.

[aMarCruz]

@jcowman2 , rollup-plugin-cleanup v3.1.0 is in npm.

The solution was to write js-cleanup to replace acorn.
Details in the Changelog.

Please ping me for any issue with this.

[jcowman2]

Confirmed, everything is working in the new plugin version. Thanks again, @aMarCruz 😄

[aMarCruz]

@jcowman2 , I will check it this weekend,
thanks for your feedback.

[aMarCruz]

@jcowman2 , I'm working on this issue right now.
The current rollup 0.66.6 (branch master) is using acorn@^5.7.0 and embeding acorn-dynamic-import@^3.0.0 and acorn-import-meta@^0.2.1 so I will do it the same.

Anyway, my solution to support acorn plugins was not good 'cause it is at very low level, I think something like the rollup's "acornInjectPlugins" option is better, perhaps in automatic mode.

Also, it is possible to use Rollup's internal acorn instance to parse the code, which would solve compatibility problems (if it works for rollup, works for cleanup) ...but only in latest versions. Maybe it can be used with an option like useInternalAcorn.

In the meantime, please remove "acorn" from your devDependencies (it is already installed by rollup) and downgrade dynamic import to "^3.0.0" to use acorn-dynamic-import/lib/inject.

Note: fot this, it is better to edit the package.json, delete node_modules and the package-lock.json or yarn.lock and reinstall.

I will leave this issue open until a rn-cleanup version has been published that solves these problems, hope in a few days.