This is primarily related to JourneyMap.
Your call to getKey in the constructor is breaking our KeyMapping logic due to our instance variables getting set in our constructor.

We set some variables in our constructor, those are checked on the getKey call, but those variables are not set when you call getKey which then throws an NPE during modloading. I can fix this on my end for our use case with a null check, but you should probably look at another approach so that you are not breaking other mods.

initial report:
TeamJM/journeymap#688

Also please verify if calling KeyMapping#setKey(Input) in KeyMapping#setKeyModifierAndCode(KeyModifier, Input) is safe for your implementation. Thanks for your report!

First of all, thank for your investigation on this iusse!

I sort of assuming KeyMapping#setKey(Input) to be a simple function and calling it in Mixin#setKeyAndCmbKeys(...) to make sure others who override setKey method can still receive the new bound key when bound key is changed. I used setKeyAndCmbKeys in constructor mainly for convenience, but you are right here, setKey may not be fully prepared before the constructor is finished, and calling it with exactly the same bound key in construct is also pointless. I will change the mixin implementation to only setup KeyMappingMixin#current_cmb_keys field in constructor and avoid the invocation of setKey in constructor. That should solve the problem.

That should be fine as long as it is not getting called in the constructor.

Thanks!

OK, thanks for your verification.