browserify / browserify/webworkify

Deduped modules don't get included in worker blob

Aperta
#41 1 commento 1 reazione 0 assegnatari Vedi su GitHub
bug
Lingua principale
JavaScript
Stelle
419
Fork
34
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia individuando la funzione resolveSources e tracciando il modo in cui attraversa la mappa modules, quindi esamina la voce dedupe e il percorso del suo modulo di destinazione. Riproduci l’eccezione di runtime del worker con una dipendenza deduplicata; il lavoro è completato quando il modulo referenziato è incluso nel blob del worker e richiederlo ha esito positivo.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
build-system
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
42/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.