External dependencies are not bundled in the UMD build

Current Behavior

My project has a few dependencies (qs, fast-text-encoding and es-cookie). When I build the UMD bundle, the dependencies are not bundled together with my code.

(function (global, factory) {
	typeof exports === 'object' && typeof module !== 'undefined' ? factory(exports, require('fast-text-encoding'), require('qs'), require('es-cookie')) :
	typeof define === 'function' && define.amd ? define(['exports', 'fast-text-encoding', 'qs', 'es-cookie'], factory) :
	(global = global || self, factory(global.createAuth0Client = {}, null, global.qs, global.Cookies));
}(this, function (exports, fastTextEncoding, qs, Cookies) { 'use strict';
  // ... some code
}));

Expected behavior

I expect all my dependencies to be bundled together with my code

Suggested solution(s)

Maybe it’s a rollup config? I have no idea 😦

Your environment

Software	Version(s)
TSDX	0.8.0
TypeScript	3.5.3
Browser	n/a
npm/Yarn	npm 6.9.0
Operating System	windows 10

[agilgur5]

@jimmyn config.external = (id) => false; should work. as should delete config.external. In full form:

tsdx.config.js:

module.exports = {
  rollup(config) {
    if (config.output.format === 'umd') {
      delete config.external;
    }
    return config;
  }
}

What other error did you get? If this were supported internally it would basically look the same.

---

I’m not totally sure why it’s not the default behavior – I think that’s just historically adopted from microbundle’s behavior (c.f. https://github.com/developit/microbundle/issues/95)

I do think it could make sense to change the default as some other tools do bundle UMD by default. However, there are ways to “import” other libraries into UMD, pending your environment (UMD can be used in Node too). In the browser, you could get your dependencies from CDN to prevent duplication. If we made this the default for UMD, it would make it more difficult to get back the current behavior for external, vs. removing external is fairly easy to configure right now.

[jimmyn]

@agilgur5 your solution works fine for me, thanks. Maybe it is worth mentioning in the documentation somewhere.