framework icon indicating copy to clipboard operation
framework copied to clipboard

Feature request: Include sourcemaps in built jspm packages (external .map files)

Open sylvain-hamel opened this issue 10 years ago • 5 comments

When I debug my application, it would be very useful to be able to step into the Aurelia code with the source maps. It's also a great way to learn the framework.

sylvain-hamel avatar Oct 16 '15 21:10 sylvain-hamel

Trying to debug Aurelia but can't find the sourcemaps published anywhere. What's the recommended approach to do this?

It seems that sourcemaps are not being generated at all? I'm not familiar with Babel but it seems that the sourceMap option being used here is not a valid Babel option so it is just being ignored. There's a sourceMap**s** but in order to generate a .map file I had to use gulp-sourcemaps as described on the gulp-babel instructions.

Having to tweak all the modules and build them seems to be a lot of work just to get debug working correctly - am I missing something?

Also, if using gulp-sourcemaps is the correct way of producing the sourcemaps during the build I'm happy to create PRs with this change.

renataogarcia avatar Jun 30 '16 02:06 renataogarcia

If you can generate a single PR, we can merge that and then apply it throughout. At some point the configuration must have changed or gotten messed up as we migrated our build setup over many months.

EisenbergEffect avatar Jun 30 '16 03:06 EisenbergEffect

I was hoping to help out with this, but the pr opened against this issue was closed due to "cleaning of the code we have to do. The cleaning relates to issues with the output of the Babel transpiler." Is there an issue tracking this cleaning or any specifics around what this cleaning entails that would allow work to be done here?

Rleahy22 avatar Oct 04 '16 01:10 Rleahy22

stale since 2016 still relevant?

Alexander-Taran avatar Mar 12 '18 01:03 Alexander-Taran

Let's keep this open. It's difficult to do in this our current version but I bet we can do it for vNext.

EisenbergEffect avatar Mar 12 '18 01:03 EisenbergEffect