Problem with esrever dependency

Do you want to request a feature or report a bug?

Bug

What’s the current behavior?

Slate: “slate”: “^0.58.1”, “slate-history”: “^0.58.1”, “slate-hyperscript”: “^0.58.1”, “slate-react”: “^0.58.1”

Browser: Chrome OS: Mac / Windows / Linux

I am getting an error when trying to build my project:

Error: 'reverse' is not exported by node_modules/esrever/esrever.js, imported by node_modules/slate/dist/index.es.js

It is coming from this line:

import { reverse } from 'esrever';

I am able to fix the issue manually by changing this line to:

import esrever from 'esrever';
const { reverse } from esrever;

This seems functionally the same to me, but it works.

What’s the expected behavior?

The dependency should provide the reverse function and build correctly.

Any help would be greatly appreciated 👍

[TheSpyder]

This is probably the wrong place to ask - as I said I don’t use rollup anymore - please try Slack.

Also note I did say

The rollup commonjs plugin has fixed this issue, so namedExports is no longer required. Upgrading will show a warning.

[TheSpyder]

Briefly skimming through and trying esbuild makes me think this would not be a trivial switch, would it?

It depends what you’re doing. It was easy for my team, but the project wasn’t really leveraging advanced rollup capabilities (no React, and we output ES6 so we don’t need babel at all).

The insane performance difference was worth any headaches - we bundle in about 150 milliseconds.