Backend: Event Predicates

[ItsEmpa]

What

This PR changes the way in which conditions are checked when invoking events, only looking at which conditions will have to be checked when the listener is first initialized, and storing the conditions as lists of lambdas. it also makes it so that conditions such as onlyOnSkyblock and onlyOnIsland(s) are only calculated once every tick, which should help with performance a bit on events that get called a lot (such as render events getting called every frame).

This PR also indirectly fixes the issue found and fixed in #3411, because it only accesses the values of the HandleEvent object once.

Did some testing while in hub 1, this is the difference i found when profiling the previous and the new shouldInvoke function was of more or less ~66%

Profiling Images

before

after

Changelog Improvements

• Improved performance overall slightly. - Empa, Nessiesson

Changelog Technical Details

• Changed pre-conditions of events to be cached per tick and only checked when necessary. - Empa, ThatGravyBoat

[github-actions]

One or more Detekt Failures were detected:

• EventHandler.kt#L10: Unused import
• RepoPatternManager.kt#L4: Unused import
• ConfigUtils.kt#L4: Unused import
• ConfigManager.kt#L4: Unused import

[hannibal002]

waiting on changes above

[github-actions]

One or more Detekt Failures were detected:

• ConfigUtils.kt#L4: Unused import