Support default exports in loaders about remix HOT 4 CLOSED

I hadn't ever seen it before either but when I looked up how to do module.exports in the TypeScript docs that's how it said it should be done. AFAICT export = is the TypeScript equivalent for ESM export default. In TypeScript, export default is actually a named export called default.

from remix.

Ya, just confirmed. If I use export default loader I get exports.default = loader in my .js file. But export = loader gives me module.exports = loader in the CommonJS output.

from remix.

Any chance the code that requires the loaders could handle this case (kinda like babel's interop require thing):

function _interopRequireDefault(obj) {
  return obj && obj.__esModule ? obj : { default: obj };
}

I do this in babel-plugin-macros: https://github.com/kentcdodds/babel-plugin-macros/blob/3263dbe8f9153090249fc2b8dcc4a5c6d61b6625/src/index.js#L84-L88

function interopRequire(path) {
  const o = require(path)
  return o && o.__esModule && o.default ? o.default : o
}

That works for babel because they set the __esModule property. TypeScript doesn't do this, so it'd probably be better to go with something like this:

function interopRequire(path) {
  const o = require(path)
  return typeof o === 'object' && o.default ? o.default : o
}

I think that would be pretty safe and would allow us to avoid the odd syntax.

from remix.

This is no longer relevant

from remix.