Handlebars: Dynamic requires are not currently supported by rollup-plugin-commonjs

[jasonwaters]

This is probably the wrong place to ask this, but I'm getting this error when trying to use handlebars in my rollupjs build:

Dynamic requires are not currently supported by rollup-plugin-commonjs

I'm trying to figure out how I can include Handlebars import Handlebars from 'handlebars'; for use in my project using rollupjs. Any ideas?

[michal-perlakowski]

I'm getting the same error with Sinon.js.

[thisconnect]

This is is far as I know:

commonjs allows to do conditional requires

if (something){
  dep = require('foo')
}

es modules are static and do NOT allow conditional imports. So the example can not be converted to imports/exports

Looking at the handlebars.js source code... They moved partly to es modules, but index.js requires some dist/cjs files.

Wild guess: Could you try to import from the es modules directly? Something like import Handlebars from 'handlebars/lib/handlebars.js';

[krstffr]

I also have this error using mongoose!

I've created a minimal repo with the bug for reproduction.

EDIT: Here's the code throwing the error in mongoose: https://github.com/Automattic/mongoose/blob/67c465aac5c864c3004d11d49934605037c8f520/lib/drivers/index.js#L8

[seangwright]

I ran into this issue and I'm also using Typescript (so I wanted to still be able to use @types/handlebars).

I created a typings.d.ts along side the files that use import Handlebars from 'handlebars/lib/handlebars.js';. It contains the following lines

declare module "handlebars/lib/handlebars" {
    export default Handlebars;
}

The reason I'm doing this is I'm trying to use ES Modules natively in the browser (with Chrome 61) but I want my node_modules dependencies to be bundled separately from my application code as ES Modules, which was the approach recommended in this youtube video.

Here is a demo app I made showing the workarounds/steps needed to use ES Modules in the browser without transpiling/polyfilling them
https://github.com/sgwatgit/ts-to-es2015

[franciscop]

@thisconnect that worked surprisingly well for me! Thanks!

[svedova]

For me it worked to include this option in rollup.config.js:

commonJs({ 
  ignore: ["conditional-runtime-dependency"] 
})

[soullivaneuh]

I ran into this issue just by importing casual:

import casual from 'casual';

If I remove this line, everything works. I'm not very sure of what to do.

I tried #131 (comment) without any luck.

[shellscape]

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. 🍺