browserify / browserify/webworkify

Deduped modules don't get included in worker blob

Open
#41 1 comment 1 reaction 0 assignees View on GitHub
bug
Dominant language
JavaScript
Stars
419
Forks
34
PR merge metrics
No merged PRs in 30d

Description

This causes an exception at runtime when the missing module is required.

The `resolveSources` function tries to recursively find all dependencies, but it doesn't know about modules that are deduped, which have an entry in the modules map that looks like this:

```js
"/Users/jhiesey/Projects/node/stream-http/node_modules/inherits/inherits_browser.js": [function(require, module, exports) {
arguments[4]["/Users/jhiesey/Projects/node/airtap/node_modules/inherits/inherits_browser.js"][0].apply(exports, arguments)
}
, {}]
```

The module the dedupe entry points to ("/Users/jhiesey/Projects/node/airtap/node_modules/inherits/inherits_browser.js") needs to be included too, but `resovleSources` doesn't know that, so the dedupe entry throws at runtime.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.