Supporting Source Maps

[valtido]

I think this line of code should change:
https://github.com/sintaxi/terraform/blob/master/lib/javascript/processors/coffee.js#L7

From: 
var script = cs.compile(fileContents.toString(), { bare: true })
To:
var useMaps = process.env.coffee_map || process.env.NODE_ENV=="dev"?true:false;
var script = cs.compile(fileContents.toString(), { bare: true , map : useMaps })

I have not tested this, but I think then we can do something like

coffee_map=true harp server app.js
//or maybe
NODE_ENV=dev harp server app.js

I wanted to make the change, but not sure, I should at this point in time. Any thoughts, or am I completely getting this wrong?

[kennethormandy]

Hey, thanks for opening an issue!

Terraform / Harp already supports the Node Environment variables, yeah, so that’d be the way to go. I’m not the best person to ask about how this should be implemented, but we definitely would like to see source maps for all the preprocessors per sintaxi/harp#74

If you’re interested in contribute that, we’d be super grateful and can put together a better explanation of the expected behaviour.