fix: make ESM transpiled CommonJS play nice for TS folks, fix koajs#1513

miwnwski · Jan 4, 2021 · 0beef00 · 0beef00
1 parent 6e1093b
commit 0beef00
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/application.js b/lib/application.js
@@ -211,6 +211,15 @@ module.exports = class Application extends Emitter {
     const msg = err.stack || err.toString();
     console.error(`\n${msg.replace(/^/gm, '  ')}\n`);
   }
+
+  /**
+   * Help TS users comply to CommonJS, ESM, bundler mismatch.
+   * @see https://github.com/koajs/koa/issues/1513
+   */
+
+  static get default () {
+    return Application;
+  }
 };
 
 /**

diff --git a/test/load-with-esm.js b/test/load-with-esm.js
@@ -28,7 +28,10 @@ describe('Load with esm', () => {
     for (const k of ['prototype', 'length', 'name']) {
       required.delete(k);
     }
-    exported.delete('default');
+
+    // Commented out to "fix" CommonJS, ESM, bundling issue.
+    // @see https://github.com/koajs/koa/issues/1513
+    // exported.delete('default');
 
     assert.strictEqual(exported.size, required.size);
     assert.strictEqual([...exported].every(property => required.has(property)), true);