Value imported in a dependency from web_modules/common is undefined

[gunnnnii]

Hello, I want to lead by mentioning that the dev experience with Snowpack has mostly been phenomenal! Huge props to all of you working on this.

I think I may have come across a bug, although I'm fairly new to Snowpack so it could very well be a problem with my setup.
The issue that I have is an imported variable in a dependency ends up being undefined.

Specifically, I'm trying to import a plugin for Uppy. I've tried multiple plugins and their all failing with the same error:
Uncaught TypeError: Cannot read property 'Plugin' of undefined
(Plugin is a class in @uppy/core that all plugins extend)

And this is the line it fails on: var Plugin = lib$1.__moduleExports.Plugin;

Looking around web_modules, I notice something I think is odd. lib$1 is not a module, but an imported value from a file in /common, import { l as lib$1, T as Translator } from '../common/index-40e5907d.js';. So referring to __moduleExports seems wrong.

in index-40e5907d.js we have the following:

var lib$1 = function (opts) {
  return new Uppy(opts);
}; // Expose class constructor.

var Uppy_1 = Uppy;
var Plugin_1 = Plugin;
var debugLogger_1 = debugLogger$1;
lib$1.Uppy = Uppy_1;
lib$1.Plugin = Plugin_1;
lib$1.debugLogger = debugLogger_1;

export {
  Plugin_1 as P,
  Translator as T,
  Uppy_1 as U,
  debugLogger_1 as d,
  lib$1 as l,
};

Here is a repo with my setup: https://github.com/gunnnnii/snowpack-import-errors

I'm not sure what I should be looking for to debug this, so I hope what I've written up is any help.. Snowpack doesn't log any errors, all I see is the typeerror in the browser console.

[thezimmee]

@gunnnnii I am experiencing the same issue and would love to know if you were able to resolve this?

[FredKSchott]

Ah @gunnnnii sorry we missed this one! We recently upgrade to a version of Rollup that we had hoped would solve this problem for us.

@thezimmee it looks like you are also seeing this issue in Uppy, do you mind creating a new test (test/integration/package-uppy) so that we can have a breaking test to fix? You could basically copy the package-tippyjs test directory, but replacing src/index.js with:

import { UppyProvider, UppyFactory, useUppyContext } from './UppyContext';
import Uppy from '@uppy/core';
import DragDrop, { DragDropOptions } from '@uppy/drag-drop';
import '@uppy/core/dist/style.css';
import '@uppy/drag-drop/dist/style.css';

And updating the dependencies in the test. package.json. Any help would be greatly appreciated!

[thezimmee]

Sorry @FredKSchott I should have been more clear. My issue was not in Uppy, but I was getting a similar Uncaught TypeError: Cannot read property x of undefined error and wondered if there was a fix that might apply to my situation. I since transitioned to the full build using rollup, so I'm not in a place where I could provide a test case for you (sorry!). We have a complex monorepo setup which may have contributed. I plan on transitioning back to snowpack in a bit bc I really love it, when I do I'll post back if/when I run into this again.