Some NPM packages don't work

[bmcmahen]

I've come across a few npm modules that don't appear to work correctly. Notably, event-emitter has issues importing es5-ext/object/assign, and debug doesn't appear to work with the chrome dev-tools. I suspect that small alterations to the module themselves (to take into account yet another working environment) may be necessary, but there could also be issues with the packager, too.

[novadev94]

I can't get it work with lodash too.

var _ = require('lodash'); and var lodash = require('lodash'); both return an empty object.

Take a look at the bundle file generated by packager, the code for lodash is there.

[jgable]

I've also had a hard time getting some modules to work. I was able to get lodash to work by using the lodash-node version. But, things like Backbone are not.

It looks like the packager thinks the exported function is a factory for creating the module, but the factory (in lodash) is not actually detecting a node environment and instead the export ends up being undefined.

I'll keep digging in the bundle and report if I find anything useful.

[nicinabox]

I had to use the lodash-cli to build a node version for React: lodash exports=node

[jgable]

It looks like, at least in the case of backbone (and probably regular underscore/lodash), that it checks for a define && define.amd to determine whether to use AMD versus commonJS style exports.

Why does the packager set define.amd if it's preferring commonJS exports? Is it just because that's what the polyfill had initially? Can we remove that?

I guess, the follow up question would be, if we do allow AMD style defines, why aren't they working for lodash, et. al.

[jgable]

@nicinabox The lodash-node package on npm should work for you without making a custom build; at least it has for me so far. npm install lodash-node --save and then var _ = require('lodash-node');.

[nicinabox]

I think it's been deprecated though.

[jgable]

When I comment out this define.amd = {} I am able to load lodash and backbone (well, backbone is a little trickier because it depends on underscore) without a problem.

[vjeux]

For the amd issue, here's our code responsible for it. https://github.com/facebook/react-native/blob/master/packager/react-packager/src/DependencyResolver/haste/polyfills/require.js#L597 I'm unsure why we define this global.

[vjeux]

Woops @jgable beat me to it!

[siuying]

package he shows same issue.

    // Some AMD build optimizers, like r.js, check for specific condition patterns
    // like the following:
    if (
        typeof define == 'function' &&
        typeof define.amd == 'object' &&
        define.amd
    ) {
        define(function() {
            return he;
        });
    }   else if (freeExports && !freeExports.nodeType) {
        if (freeModule) { // in Node.js or RingoJS v0.8.0+
            freeModule.exports = he;
        } else { // in Narwhal or RingoJS v0.7.0-
            for (var key in he) {
                has(he, key) && (freeExports[key] = he[key]);
            }
        }
    } else { // in Rhino or a web browser
        root.he = he;
    }

[OlivierFortin]

I have the same issue with async,

var async = require('async');
console.log(_);// {}