linaria fails to process imported modules under certain conditions (Node is to blame)

[dkamyshov]

Hi! Thank you for the great package.

Summary

linaria fails to process imported modules under certain conditions.

The problem is actually with Node, not linaria, see this issue: nodejs/help#2094

Conditions

1. The module in question must be executed in strict mode.
2. The module must export a function.
3. The function must be assigned to an exports variable.

This may seem like a rare scenario, but I guess there are quite a few modules on NPM that use exports this way.

Solution

Wrap the module with an additional function wrapper, like so:

(function (exports) {
  // code
})(exports);

I decided to go just with exports because:

• it solves the problem
• changes are minimal
• other variables (module, etc) are not affected

Test plan

• yarn test
• yarn link against this repo https://github.com/dkamyshov/es5-ext-linaria-vm-reproduction

Notes

I would greatly appreciate a new release (like v1.3.2) if this PR gets merged.

[callstack-bot]

Hey @dkamyshov, thank you for your pull request 🤗.
The coverage report for this branch can be viewed here.