Right, `function (err)` above will not be triggered from errors from load.fromText(). Looks like what should happen around this line in requirejs:

https://github.com/jrburke/requirejs/blob/master/require.js#L1020

is for requirejs to call an errback internally if one is available. I opened jrburke/requirejs#583 to track that change. With that, I expect that to close the loop around this function(err) section. However, doing work around calling load.error for CoffeeScript compile errors should still be done.

I'm happy to have the changes, so feel free to keep going with it. I am starting a new process where I try to close things more aggressively, just to keep the open count smaller, particularly when the issues are more grayer than this one and have uncertain outcomes. But even for items like these, I would prefer to close first if not fully workable, but then reopen later after discussion and more fixes done.