One-com/assetgraph — pull request #125 by Munter (2013-10-20 13:31:22+00:00)
Title: Optionalarguments in requirejs commonjs comatible syntax

RequireJS allows for commonjs compatible syntax like so:
```js
define(function(require, exports, module) {
//Put traditional CommonJS module content here
});
```

Currently this is tested in https://github.com/One-com/assetgraph/blob/master/test/JavaScriptRequireJsCommonJsCompatibilityRequire-test.js

However it is also allowed to leave out the `exports` and `module` arguments in the define call, like so:
``` js
define(function (require) {
var foo = require('foo');

//Define this module as exporting a function
return function () {
foo.doSomething();
};
});
```

This makes the `liftUpJavaScriptRequireJsCommonJsCompatibilityRequire` transform fail.

This patch should fix that.