Add a setting to InventoryTweaks to make input handling happen every frame instead of every tick.

Closes #6065

Author: Wide-Cat

src/main/java/meteordevelopment/meteorclient/mixin/MinecraftClientAccessor.java

@@ -72,4 +72,7 @@ public interface MinecraftClientAccessor {
     @Mutable
     @Accessor("apiServices")
     void meteor$setApiServices(ApiServices apiServices);
+
+    @Invoker("handleInputEvents")
+    void meteor$handleInputEvents();
 }

src/main/java/meteordevelopment/meteorclient/mixin/MinecraftClientMixin.java

@@ -24,6 +24,7 @@
 import meteordevelopment.meteorclient.mixininterface.IMinecraftClient;
 import meteordevelopment.meteorclient.systems.config.Config;
 import meteordevelopment.meteorclient.systems.modules.Modules;
+import meteordevelopment.meteorclient.systems.modules.misc.InventoryTweaks;
 import meteordevelopment.meteorclient.systems.modules.movement.GUIMove;
 import meteordevelopment.meteorclient.systems.modules.player.FastUse;
 import meteordevelopment.meteorclient.systems.modules.player.Multitask;
@@ -90,6 +91,9 @@ public abstract class MinecraftClientMixin implements IMinecraftClient {
     @Mutable
     private Framebuffer framebuffer;
 
+    @Shadow
+    protected abstract void handleBlockBreaking(boolean breaking);
+
     @Inject(method = "<init>", at = @At("TAIL"))
     private void onInit(CallbackInfo info) {
         MeteorClient.INSTANCE.onInitializeClient();
@@ -279,6 +283,31 @@ private boolean hasOutlineModifyIsOutline(boolean original, Entity entity) {
         return esp.getColor(entity) != null || original;
     }
 
+
+    // faster inputs
+
+    @Unique
+    private boolean isBreaking = false;
+
+    @WrapWithCondition(method = "tick", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/MinecraftClient;handleInputEvents()V"))
+    private boolean wrapHandleInputEvents(MinecraftClient instance) {
+        return !Modules.get().get(InventoryTweaks.class).frameInput();
+    }
+
+    @WrapWithCondition(method = "handleInputEvents", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/MinecraftClient;handleBlockBreaking(Z)V"))
+    private boolean wrapHandleBlockBreaking(MinecraftClient instance, boolean breaking) {
+        isBreaking = breaking;
+        return !Modules.get().get(InventoryTweaks.class).frameInput();
+    }
+
+    @Inject(method = "tick", at = @At(value = "INVOKE", target = "Lnet/minecraft/client/MinecraftClient;handleInputEvents()V", shift = At.Shift.AFTER))
+    private void afterHandleInputEvents(CallbackInfo ci) {
+        if (!Modules.get().get(InventoryTweaks.class).frameInput()) return;
+
+        handleBlockBreaking(isBreaking);
+        isBreaking = false;
+    }
+
     // Interface
 
     @Override

src/main/java/meteordevelopment/meteorclient/mixin/RenderSystemMixin.java

@@ -7,6 +7,8 @@
 
 import com.mojang.blaze3d.systems.RenderSystem;
 import meteordevelopment.meteorclient.renderer.MeshUniforms;
+import meteordevelopment.meteorclient.systems.modules.Modules;
+import meteordevelopment.meteorclient.systems.modules.misc.InventoryTweaks;
 import meteordevelopment.meteorclient.utils.render.postprocess.ChamsShader;
 import meteordevelopment.meteorclient.utils.render.postprocess.OutlineUniforms;
 import meteordevelopment.meteorclient.utils.render.postprocess.PostProcessShader;
@@ -15,6 +17,8 @@
 import org.spongepowered.asm.mixin.injection.Inject;
 import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
 
+import static meteordevelopment.meteorclient.MeteorClient.mc;
+
 @Mixin(RenderSystem.class)
 public abstract class RenderSystemMixin {
     @Inject(method = "flipFrame", at = @At("TAIL"))
@@ -23,5 +27,8 @@ public abstract class RenderSystemMixin {
         PostProcessShader.flipFrame();
         ChamsShader.flipFrame();
         OutlineUniforms.flipFrame();
+
+        if (Modules.get() == null || mc.player == null) return;
+        if (Modules.get().get(InventoryTweaks.class).frameInput()) ((MinecraftClientAccessor) mc).meteor$handleInputEvents();
     }
 }

src/main/java/meteordevelopment/meteorclient/systems/modules/misc/InventoryTweaks.java

@@ -70,6 +70,21 @@ public class InventoryTweaks extends Module {
         .build()
     );
 
+    private final Setting<Boolean> uncapBundleScrolling = sgGeneral.add(new BoolSetting.Builder()
+        .name("uncap-bundle-scrolling")
+        .description("Whether to uncap the bundle scrolling feature to let you select any item.")
+        .defaultValue(true)
+        .build()
+    );
+
+    private final Setting<Boolean> frameInput = sgGeneral.add(new BoolSetting.Builder()
+        .name("frame-input-handling")
+        .description("Changes input handling to work every frame instead of every tick. A very minor effect but may\n" +
+            "make inputs feel smoother, especially in laggy environments. Will flag anticheats that check packet order (Grim).")
+        .defaultValue(false)
+        .build()
+    );
+
     // Sorting
 
     private final Setting<Boolean> sortingEnabled = sgSorting.add(new BoolSetting.Builder()
@@ -104,13 +119,6 @@ public class InventoryTweaks extends Module {
         .build()
     );
 
-    private final Setting<Boolean> uncapBundleScrolling = sgGeneral.add(new BoolSetting.Builder()
-        .name("uncap-bundle-scrolling")
-        .description("Whether to uncap the bundle scrolling feature to let you select any item.")
-        .defaultValue(true)
-        .build()
-    );
-
     // Anti drop
 
     private final Setting<List<Item>> antiDropItems = sgAntiDrop.add(new ItemListSetting.Builder()
@@ -466,6 +474,10 @@ public boolean uncapBundleScrolling() {
         return isActive() && uncapBundleScrolling.get();
     }
 
+    public boolean frameInput() {
+        return isActive() && frameInput.get();
+    }
+
     public boolean canSteal(ScreenHandler handler) {
         try {
             return (stealScreens.get().contains(handler.getType()));