esm output provides wrong import statement

[TrySound]

There is in the end unused and not existing import. Seems like generated by mistake
https://unpkg.com/[email protected]/dist/es/WindowScroller/utils/onScroll.js

import { bpfrpt_proptype_WindowScroller } from '../WindowScroller.js';

/cc @brigand

[brigand]

Hey, this is expected behavior of the plugin, but since bpfrpt_proptype_WindowScroller isn't used anywhere, we'd ideally remove it. However, this requires some form of escape-analysis, which is a bit tricky. Created an issue.

If a quick fix is needed, you can add this line to the file:

'no babel-plugin-flow-react-proptypes';

[remorses]

I cannot build the package with esbuild because of this issue

error: No matching export for import "bpfrpt_proptype_WindowScroller"
    75 │ import { bpfrpt_proptype_WindowScroller } from '../WindowScroller.js';

[YurySolovyov]

For esbuild, i've redirected imports to umd and it seems to work

const resolveFixup = {
  name: 'resolve-fixup',
  setup(build) {
    build.onResolve({ filter: /react-virtualized/ }, async args => {
      return {
        path: path.resolve('./node_modules/react-virtualized/dist/umd/react-virtualized.js'),
      }
    })
  },
};

esbuild.build({
  plugins: [
    resolveFixup,
    // your plugins
  ],
});

[lucantini]

@YurySolovyov This is a good patch fix for this issue, but this needs to be resolved in a later release of react-virtualized!

[kitsune7]

For anyone using Vite v2.3.0 or newer, you can apply the esbuild plugin that @YurySolovyov wrote above by modifying your Vite config to include the following:

export default defineConfig({
  optimizeDeps: {
    esbuildOptions: {
      plugins: [resolveFixup]
    }
  }
});

[lucantini]

This problem also happens when building with rollup. Can we remove this particular import since this variable doesn't seem to exist?

[jasonwilliams]

This is still an issue for us when building with esbuild.

[intafon]

Yeh, I worked through several issues I had trying out esbuild and using a lerna monorepo, and after all those were resolved I hit this. Bumping this in hopes it gets attention. :-)

[jasonwilliams]

@intafon if you need a quick fix #1212 (comment) solution worked for me.

In terms of this being fixed in general, it's already been fixed and merged but it seems to be stuck on being published. 2e962d8

[VladBrok]

It's still an issue in 2023 vite v4.3.8

[VladBrok]

please fix (((

[kaiyoma]

Also hitting this issue. Please fix!

[H0JLuk]

Hotfix for this issue has been resolved above at the comment
comment

[abemedia]

I wrote an esbuild plugin to fix this: https://npmjs.com/package/esbuild-plugin-react-virtualized