Default Source Map Preprocessor

[schneems]

An alternative to #205 we don't set any sort of a "default" source map if one is already provided via another preprocessor like coffee or sass. We also set the default to be a line for line match between original and generated.

I'm not sure why the url for babel is different test is different in test_source_maps.rb#111 and test_source_maps.rb#161, or how we lost the digest on test_source_maps.rb#L119.

Thanks for some implementation ideas in: 9e119cd

[elia]

👍

[lucasmazza]

@schneems any ideas on how this patch might affect the //= require processing? I'm getting some Sprockets::FileNotFound on //= requires that are pointing to assets with different extensions, like when you require normalize.css inside application.scss, while same extension requires (scss to scss) are working as usual.

//= require normalize/normalize.css
@import 'foo/bar';

The same issue happens when requiring a .js file inside an .es6 asset. I couldn't figure out why this might be happening based on the diff here but I tested against the previous commit (723c94c) and the pipeline works as expected.

[schneems]

IDK, can you give me an example app?

[lucasmazza]

@schneems I'll see if I can isolate it and replicate in a new app