search

derbyjs / racer

Realtime model synchronization engine for Node.js
1.19k stars 118 forks source link

Sourcemaps not working #175

Closed gurdasnijor closed 10 years ago

gurdasnijor commented 10 years ago

The Store::bundle function seems to strip out the sourcemap comment that gets emitted with the bundled code, making it pretty difficult to debug in devtools. It looks like a new parameter is added to the callback though for passing a caller the sourcemap though (so i'm assuming consuming code needs to now serve this *.js.map file with this content somehow now?

https://github.com/codeparty/racer/blob/master/lib/Store.js#L86

nateps commented 10 years ago

Correct. This isn't really a bug, just an internal API change.

The map file is returned as a separate argument instead of being included in the source like Browserify does. This way, you can still have source maps in production and not serve them with the main script file. Racer doesn't deal with serving the script bundle, so this is out of scope of what Racer should take care of.

I'm likely going to move this browserify function to its own repo anyway, since it is an annoyingly large dependency when you are using racer in server-side only scripts.