Skip to content

Conversation

@Asbestosstar
Copy link
Contributor

Early Stacktrace Reader.
Still needs some work, but this is a basic stacktrace reader which reads all the stacktraces and can either find spongemixin conficts or list jars in a stacktrace from the supported logging sites or file upload.

In order to activate it you must include it as a LogAnalyserModule in your GitHub Addon.https://github.com/MangoRageBot/MangoBotGithub/blame/master/src/main/java/org/mangorage/mangobotgithub/core/PasteRequestModule.java#L37

Early Stacktrace Reader
@Asbestosstar
Copy link
Contributor Author

imagen
imagen
imagen
imagen
imagen
This a a stacktrace reader for the log analyser. it will allow stacktraces to be read. It looks for

  • Broken SpongeMixin Configs
  • ModLauncher services+SpongeMixins in Stacktrace
  • Jar Files in lines
  • If there is no modfile in the lines it checks for modids (this is common in dev enviornments and dev orientated launchers)
  • If that does not exist it checks for packages in that line
  • It also checks for class not found
  • It seperates between Fatal and nonFatal Traces saying to prioritise the Fatal ones
  • It has a level and line number system, ones with higher level should be prioritised over ones with lower level, in the same level ones with lower ln should be prioritised as specified.
  • It has denylists of prefixes for packages to stop loader packages and modids and jars from being included in the output.
  • It works with the same paste sites and file upload as LogAnalyser

It still needs some testing and some more real world use and some beautification but it should eventually be possible to have a stacktrace reader like this one.

@RealMangorage
Copy link
Member

Can you add it to the org.mangorage.mangobot.modules.logs.LogAnalyserModule#MAIN

And then I can pull it in.

Thanks!

@RealMangorage RealMangorage merged commit 8c92625 into MangoRageBot:master Mar 20, 2025
1 check passed
RealMangorage added a commit that referenced this pull request Mar 20, 2025
RealMangorage added a commit that referenced this pull request Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants