-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generate 'reverse sourcemap' from unminified output, that can be used directly with the original minified bundle #66
Comments
I can see the value of this. But |
I experimented with this a while ago. reverse-sourcemap.mp4With some changes it should also be possible to make it work for modules of an unpacked bundles.
Then you might as well get the original source code ^^ https://github.com/Anthonyzou/Sourcemap-Unpack |
@pionxzh Oh wow.. really? This was the only issue I could find mentioning it, and while it was 'closed as completed', it didn't really suggest that it actually was implemented anywhere: It also seems that But I'm not sure if
@j4k0xb I was more thinking in the case of a common library such as React, bundled into the web app I'm reversing. I could go and manually look at React's source code/etc because it's open source. If the sourcemaps contain the full source I could also extract them from there. But I'm sort of less interested in getting the original source code of the (likely open source anyway) bundled module, as I am in being able to identify all of the bits related to that lib and 'reduce the noise' in the rest of the code I'm actually trying to look at; while also enriching the 'downstream app' with proper names for any functions/etc exported from that common lib that might have been mangled in minification process. |
I also saw that issue and assumed it's closed as wont fix. I have left a comment in facebook/jscodeshift#322 to check its status. But I'm afraid that there are still
|
I just checked the implementation in jscodeshift, and it seems like it might be possible to generate the sourcemap via recast. But this would be low-priority for me; PRs are welcome if anyone is interested. |
@pionxzh It looks like at least part of
@pionxzh Exciting!
@pionxzh nods, totally understandable |
The text was updated successfully, but these errors were encountered: