Regression @rollup/plugin-commonjs v13.0.0

[MatthiasKainer]

• Rollup Plugin Name: @rollup/plugin-commonjs
• Rollup Plugin Version: v13.0.0
• Rollup Version: 1.32.1
• Operating System (or Browser): any
• Node Version: any

How Do We Reproduce?

https://repl.it/repls/PhonyUsefulNanotechnology#output/bundle.js

Have a file like

import {LitElement, html} from "lit-element"
import { useState } from "lit-element-state-decoupler";
import {useOnce} from "lit-element-effect";

export class Element extends LitElement {
  render() {
    const {getState, publish} = useState(this, false)
    useOnce(this, () => publish(true))

    return html`<div>Called: ${getState()}<div>`
  }
}

Expected Behavior

The imports are resolved correctly. In the bundle that would create a result:

class Element extends LitElement {
  render() {
    const {getState, publish} = index.useState(this, false);
    index$1.useOnce(this, () => publish(true));

    return html`<div>Called: ${getState()}<div>`
  }
}

(Note the index$1 on useOnce)

Actual Behavior

useOnce is retrieved from index, which obviously fails:

class Element extends LitElement {
  render() {
    const {getState, publish} = index.useState(this, false);
    index.useOnce(this, () => publish(true));

    return html`<div>Called: ${getState()}<div>`
  }
}

Workaround

Use version 12.0.0 of @rollup/plugin-commonjs instead of v13.0.0

[vladshcherbin]

I've got same thing with another library, also with index near this line:

console.log(1, lib)

var index = /*@__PURE__*/unwrapExports(lib);

console.log(2, index)

I believe exports are unwrapped incorrectly, here is result:

From source code function checks for default export and uses it, however object can have other keys which will be removed.

Bundling results in an application error shown above. Workaround is to use 12.0.0.

Will be probably fixed with #465

[lukastaegert]

I started #481 to move all interop issues including this one forward.

[lukastaegert]

I can confirm that #501 solves this issue.