StackDeobfuscator
Madis0 opened this issue ยท 3 comments
CurseForge link
https://www.curseforge.com/minecraft/mc-mods/stackdeobf
CurseForge Mod Distribution
Allowed
Modrinth link
https://modrinth.com/mod/stackdeobf
Source/other link
https://github.com/booky10/StackDeobfuscator
Mod file size
149.64 KiB + ca 7 MB downloaded afterwards (Yarn)
License
(any other license)
What it does
Maps the class names in crashes with their actual value
Why should it be in the modpack
Looks very useful for determining the cause of crashes, perhaps even moreso than MixinTrace. Could be more like #439 without the cons.
Iris dev endorsement (of the theoretical idea, not actually tested):
Seems good, since it's just converting mappings
ETF dev:
On the one hand I love it, on the other hand the standing out of non obfuscated text in stack traces is probably the single most helpful thing in identifying mod issues, if you need to identify a vanilla class then the troubleshooting is already out of the scope of a typical end user
Why shouldn't it be in the modpack
- ETF dev's quote brings a good point, mod issues are usually visible in between the obfuscated classes
- Untested, unsure how much it helps in practice
- Unsure if it really helps mod devs or may cause problems too, have to be very careful
Additional details
If this proves very useful, I might even reconsider #440
Useful for mod devs to debug mixin related issues in their mods as the class name can help them figure out which mixin of theirs is causing problems
Users who add other mods on top of FO can benefit from this even more, if those mods use a lot of mixins
I've now tested the dev's web version of the tool out and it seems to work well, though that makes me think whether there is currently a need for this in the mod form.
The mod technically just replaces words (instead of adding new content to file like MixinTrace), and users are expected to post most crashlogs to the FO Discord anyway, so logs can be copied there if relevant.
Additionally, it may not be needed in a lot of times per ETF dev's comment, as mods' mixin names are already visible by default.