Automatically patch the require.js module callbacks

[getsentry]

Automatically patch the require.js module callbacks

[SBoudrias]

Hi @mattrobenolt, tested this build this morning and this was working great on my app.

Although, I'm concerned by the fact you infer if define is a function, then require is a function. If I'm correct, only RequireJS AMD loader use the require function, other like curl, lsjs and dojo may break (didn't test it though). But anyway, I guess the point is mostly that inferring is a - most of the time - bad thing. Any though?

[mattrobenolt]

I'll look into this today with the other AMD libs and see if it'll hurt. Thanks for the feedback. :)

And, this is probably the direction things are going to go: #69 FYI

[mattrobenolt]

Ok, so according to the spec, you're right. It's not required to exist. I'll adjust this. http://wiki.commonjs.org/wiki/Modules/AsynchronousDefinition

[mattrobenolt]

This will be a plugin. So closing this.