fix: always use the regular function to avoid losing module context

[latin-1]

Using an arrow function here would cause the loss of context (this) inside the module factory, which may break some modules that rely on it.

[pmmmwh]

Hey, thanks for the PR. I'll review this tomorrow and also investigate a bit into the issue.

Is there concrete example demonstrating how arrow functions would break?

[latin-1]

Assume we have a module, x.js:

this.foo = "bar";

And now, we are trying to require it from node:

$ echo 'this.foo = "bar";' > x.js
$ node -e 'require("./x").foo' -p
bar

Webpack has a similar approach to Node.js by calling the module factory with this = module.exports. However, the react-refresh-webpack-plugin's runtime code breaks this treatment.

Currently, we are running into this bug with @mediapipe/face_mesh, which uses the same way to export something in both browsers and webpack.