Making sourcetype unambiguous to support commonjs in addition

Hi,

I understand that the configuration is mainly setup to support es6 module syntax. I think it would be great to additionally not break working commonjs.

Webpack itself supports it to a certain degree: You can mix require and export. You can’t mix import and module.exports. https://github.com/webpack/webpack/issues/4039#issuecomment-273804003

The problem is that babel-transform is transforming require statements to imports: https://stackoverflow.com/a/56283408/1602476

This could be prevented (see stackoverflow answer) by setting:

"sourceType": "unambiguous"

Would this be something that could potentially be supported? Since I think it would not break existing setups, but would also allow commonjs to be supported additionally.

[KeKs0r]

I found a workaround by creating a .babelrc:

{
  "sourceType": "unambiguous"
}

and its picked up by babel-transform, which resolves the issue. I think this can be closed and potntially just documented, that .babelrc can be used with this plugin.

[Zambonilli]

For anyone else hitting this, you can manually update your version of serverless-bundle and add sourceType: “unambiguous” to the babel-loader config here.

https://github.com/AnomalyInnovations/serverless-bundle/blob/master/src/webpack.config.js#L157