Either export for Node/CommonJS or RequireJS, not both

At the moment the following happens:

/* export Chess object if using node or any other CommonJS compatible
 * environment */
if (typeof exports !== 'undefined') exports.Chess = Chess;
/* export Chess object for any RequireJS compatible environment */
if (typeof define !== 'undefined') define( function () { return Chess;  });

When using only Node this works fine, because the second statement is not executed. When using Webpack, however, both statements are executed (because it supports both if I’m correct), which means that the first statement is overwritten (at least it seems like that) and, hence, is therefore not longer compatible with code that is written for Node. And the code works without Webpack. Note that I don’t have this problem when using chess.js as a direct dependency, only when chess.js is an indirect dependency.

Therefore, I would suggest to turn these two separate ifs into in an if-else.

[maxwellhaydn]

My current (ugly) workaround:

const { isNode } = require('browser-or-node');

let Chess;

if (isNode) {
    const chess = require('chess.js');
    Chess = chess.Chess;
}
else {
    Chess = require('chess.js');
}

[maxwellhaydn]

It turns out there’s an easy way to fix this with Webpack. Simply disable the AMD parser for chess.js in your Webpack config:

module: {
    rules: [
    ...
        {
            test: require.resolve('chess.js'),
            parser: {
                amd: false
            }
        }
    ]
}

Now you can run

const { Chess } = require('chess.js');

in both your Node code and your Webpack-bundled code.