Replace OptionValueAccessor.java with an access widener to make it work fine on forge

Signed-off-by: Username404-59 <w.iron.zombie@gmail.com>

build.gradle.kts

@@ -3,6 +3,7 @@ import com.modrinth.minotaur.ModrinthExtension
 import com.modrinth.minotaur.dependencies.DependencyType
 import masecla.modrinth4j.model.version.ProjectVersion.VersionType
 import net.fabricmc.loom.LoomGradleExtension
+import net.fabricmc.loom.api.LoomGradleExtensionAPI
 
 buildscript {
     dependencies {
@@ -26,7 +27,7 @@ plugins {
 
 group = "fr.username404"
 version = "0.3.5"
-val groupAndName = "${rootProject.group}.${rootProject.name.toLowerCase()}"
+val groupAndName = "${rootProject.group}.${rootProject.name.lowercase()}"
 
 val javaVer: String = "21"
 val sourceJavaVer: String = javaVer
@@ -50,6 +51,9 @@ subprojects {
     apply(plugin = "dev.architectury.loom")
     apply(plugin = "org.jetbrains.kotlin.plugin.serialization")
     extensions.configure<LoomGradleExtension>("loom") {
+        if (this@subprojects.project.name != "common") {
+            accessWidenerPath.set(project(":common").extensions.getByType<LoomGradleExtensionAPI>().accessWidenerPath)
+        }
         mappingsDep = layered {
             silentMojangMappingsLicense()
             officialMojangMappings().parchment("org.parchmentmc.data:parchment-1.21:2024.06.23")
@@ -114,7 +118,7 @@ subprojects {
             val dictionariesDir = "$rootDir/obfuscation"
             dependsOn(shadowJar)
             injars(shadowJar)
-            outjars("$buildDir/shrinkedJar/${shadowJar.outputs.files.singleFile.name}")
+            outjars("${rootProject.layout.buildDirectory}/shrinkedJar/${shadowJar.outputs.files.singleFile.name}")
             keep("class $group.snowygui.mixins.* { * ; }")
             keep("class $group.snowygui.fabric.FabricInit")
             keep("class $group.snowygui.fabric.ModMenuConf")

common/build.gradle.kts

@@ -6,4 +6,8 @@ dependencies {
     }
 }
 
+loom {
+    accessWidenerPath = file("src/main/resources/${rootProject.name.lowercase()}.accessWidener")
+}
+
 tasks.getByName("shrinkJar").enabled = false

common/src/main/java/fr/username404/snowygui/mixins/OptionValueAccessor.java

@@ -1,11 +0,0 @@
-package fr.username404.snowygui.mixins;
-
-import net.minecraft.client.OptionInstance;
-import org.spongepowered.asm.mixin.Mixin;
-import org.spongepowered.asm.mixin.gen.Accessor;
-
-@Mixin(OptionInstance.class)
-public interface OptionValueAccessor {
-    @Accessor("value")
-    void setValue(Object value);
-}

common/src/main/kotlin/fr/username404/snowygui/gui/elements/ClickBox.kt

@@ -1,5 +1,6 @@
 package fr.username404.snowygui.gui.elements
 
+import com.mojang.blaze3d.vertex.BufferUploader
 import com.mojang.blaze3d.vertex.DefaultVertexFormat
 import com.mojang.blaze3d.vertex.VertexFormat
 import fr.username404.snowygui.Snowy.Companion.MissingComponent

common/src/main/kotlin/fr/username404/snowygui/gui/feature/ButtonImpl.kt

@@ -13,7 +13,7 @@ import net.minecraft.client.gui.GuiGraphics
 import kotlin.reflect.full.findAnnotation
 
 sealed class ButtonImpl: ColoredElement(0.0, 0.0, 73, 8, opacity = 0.60F) {
-    internal companion object {
+    companion object {
         private fun addButtons(vararg buttons: ButtonImpl) {
             buttons.groupBy { impl ->
                 ClickGui.clickBoxes.find { box ->

common/src/main/kotlin/fr/username404/snowygui/gui/feature/GammaBoost.kt

@@ -1,6 +1,5 @@
 package fr.username404.snowygui.gui.feature
 
-import fr.username404.snowygui.mixins.OptionValueAccessor
 import net.minecraft.client.Minecraft
 
 @ButtonInfo(Category.MISC)
@@ -10,11 +9,11 @@ object GammaBoost: ButtonImpl() {
     override fun execAction() {
         with(Minecraft.getInstance().options) {
             val gamma = gamma().get()
-            @Suppress("KotlinConstantConditions")
-            (gamma() as OptionValueAccessor).setValue(if (toggled) {
+            gamma().value =
+                if (toggled) {
                     if (gamma < boost) oldGamma = gamma
                         boost
-            } else oldGamma)
+                } else oldGamma
         }
     }
 }

common/src/main/resources/snowygui-common.mixins.json

@@ -2,13 +2,7 @@
   "required": true,
   "package": "fr.username404.snowygui.mixins",
   "compatibilityLevel": "JAVA_18",
-  "client": [
-    "KeysAccessor",
-    "KeyMappings",
-    "EndTickMixin",
-    "TitleScreenMixin",
-    "OptionValueAccessor"
-  ],
+  "client": [],
   "injectors": {
     "defaultRequire": 1
   },

common/src/main/resources/snowygui.accessWidener

@@ -0,0 +1,3 @@
+accessWidener v2 named
+accessible field net/minecraft/client/OptionInstance value Ljava/lang/Object;
+mutable field net/minecraft/client/OptionInstance value Ljava/lang/Object;

common/src/main/resources/snowygui.common.json

@@ -0,0 +1,3 @@
+{
+  "accessWidener": "snowygui.accessWidener"
+}

fabric/src/main/resources/fabric.mod.json

@@ -32,6 +32,7 @@
     "snowygui-common.mixins.json",
     "snowygui-fabric.mixins.json"
   ],
+  "accessWidener": "snowygui.accessWidener",
   "depends": {
     "fabricloader": ">=${fabric_loader}",
     "fabric-language-kotlin": ">=${fabric_kotlin}",

fabric/src/main/resources/snowygui-fabric.mixins.json

@@ -3,6 +3,10 @@
   "package": "fr.username404.snowygui.mixins",
   "compatibilityLevel": "JAVA_18",
   "client": [
+    "EndTickMixin",
+    "TitleScreenMixin",
+    "KeysAccessor",
+    "KeyMappings",
     "ZoomMixin",
     "OkZoomerAlternativeMixin"
   ],

neoforge/src/main/kotlin/fr/username404/snowygui/forge/EndTickHandler.kt

@@ -0,0 +1,11 @@
+package fr.username404.snowygui.forge
+
+import fr.username404.snowygui.EventSnowy
+import net.neoforged.bus.api.SubscribeEvent
+import net.neoforged.neoforge.client.event.ClientTickEvent
+
+object EndTickHandler: EventSnowy {
+    override val type: String = "EndTick"
+    @SubscribeEvent
+    fun handleEndTick(event: ClientTickEvent.Post) = fire<Any>()
+}

neoforge/src/main/kotlin/fr/username404/snowygui/forge/ForgeInit.kt

@@ -29,6 +29,8 @@ class ForgeInit(container: ModContainer): Snowy() {
     init {
         container.eventBus!!.run {
             addListener(this@ForgeInit::initSetup)
+            register(EndTickHandler)
+            register(MiscHandlers)
             register(HudHandler)
             register(ZoomHandler)
         }

neoforge/src/main/kotlin/fr/username404/snowygui/forge/MiscHandlers.kt

@@ -0,0 +1,22 @@
+package fr.username404.snowygui.forge
+
+import fr.username404.snowygui.ClickGui
+import fr.username404.snowygui.gui.feature.ButtonImpl
+import fr.username404.snowygui.misc.AddKeyMaps
+import net.neoforged.bus.api.SubscribeEvent
+import net.neoforged.neoforge.client.event.RegisterKeyMappingsEvent
+import net.neoforged.neoforge.client.event.RegisterMenuScreensEvent
+
+object MiscHandlers {
+    @SubscribeEvent
+    fun handleClickGuiInit(event: RegisterMenuScreensEvent) {
+        ClickGui.tick()
+        ButtonImpl.initButtons()
+    }
+    @SubscribeEvent
+    fun handleKeys(event: RegisterKeyMappingsEvent) {
+        AddKeyMaps.list.forEach {
+            event.register(it.key)
+        }
+    }
+}

neoforge/src/main/resources/META-INF/neoforge.mods.toml

@@ -14,14 +14,14 @@ A cross-platform mod offering a useful interface.
 logoFile = "icon.png"
 
 [[dependencies.snowygui]]
-modId = "cloth-config"
+modId = "cloth_config"
 mandatory = false
 versionRange = "[${clothconfig},)"
 ordering = "BEFORE"
 side = "CLIENT"
 
 [[dependencies.snowygui]]
-modId = "forge"
+modId = "neoforge"
 mandatory = true
 versionRange = "[${forge_version},)"
 ordering = "NONE"