Backend: Crash in dev env for @ConfigLink (#1455)

Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>

1 files changed, 30 insertions, 1 deletions

diff --git a/src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt b/src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt
index 7f9b47d6c..93f40cae4 100644
--- a/src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt
+++ b/src/main/java/at/hannibal2/skyhanni/config/ConfigManager.kt
@@ -202,12 +202,27 @@ class ConfigManager {
         }
     }
 
+    // Some position elements dont need config links as they dont have a config option.
+    private val ignoredMissingConfigLinks = listOf(
+        // commands
+        "features.garden.GardenConfig.cropSpeedMeterPos",
+        "features.misc.MiscConfig.collectionCounterPos",
+        "features.misc.MiscConfig.lockedMouseDisplay",
+
+        // debug features
+        "features.dev.DebugConfig.trackSoundPosition",
+        "features.dev.DevConfig.debugPos",
+        "features.dev.DevConfig.debugLocationPos",
+        "features.dev.DevConfig.debugItemPos",
+    )
+
     private fun findPositionLinks(obj: Any?, slog: MutableSet<IdentityCharacteristics<Any>>) {
         if (obj == null) return
         if (!obj.javaClass.name.startsWith("at.hannibal2.skyhanni.")) return
         val ic = IdentityCharacteristics(obj)
         if (ic in slog) return
         slog.add(ic)
+        var missingConfigLink = false
         for (field in obj.javaClass.fields) {
             field.isAccessible = true
             if (field.type != Position::class.java) {
@@ -219,13 +234,27 @@ class ConfigManager {
                 if (LorenzUtils.isInDevEnvironment()) {
                     var name = "${field.declaringClass.name}.${field.name}"
                     name = name.replace("at.hannibal2.skyhanni.config.", "")
-                    println("WEE WOO WEE WOO HIER FEHLT EIN @CONFIGLINK: $name")
+                    if (name !in ignoredMissingConfigLinks) {
+                        println("WEE WOO WEE WOO HIER FEHLT EIN @CONFIGLINK: $name")
+                        missingConfigLink = true
+                    }
                 }
                 continue
             }
             val position = field.get(obj) as Position
             position.setLink(configLink)
         }
+        if (missingConfigLink) {
+            println("")
+            println("This crash is here to remind you to fix the missing @ConfigLink annotation over your new config position config element.")
+            println("")
+            println("Steps to fix:")
+            println("1. Search for `WEE WOO WEE WOO` in the console output.")
+            println("2. Either add the Config Link.")
+            println("3. Or add the name to ignoredMissingConfigLinks.")
+            println("")
+            LorenzUtils.shutdownMinecraft("Missing Config Link")
+        }
     }
 
     private fun firstLoadFile(file: File?, fileType: ConfigFileType, defaultValue: Any): Any {