make line starter cache not regenerate by doing anything

Author: GeorgeRNG

src/main/java/dev/dfonline/codeclient/CodeClient.java

@@ -47,6 +47,7 @@
 import net.fabricmc.loader.api.ModContainer;
 import net.minecraft.block.Blocks;
 import net.minecraft.block.entity.BlockEntityType;
+import net.minecraft.block.entity.SignText;
 import net.minecraft.client.MinecraftClient;
 import net.minecraft.client.gui.DrawContext;
 import net.minecraft.client.gui.screen.ChatScreen;
@@ -57,10 +58,13 @@
 import net.minecraft.client.render.VertexConsumerProvider;
 import net.minecraft.client.util.math.MatrixStack;
 import net.minecraft.item.ItemStack;
+import net.minecraft.nbt.NbtCompound;
+import net.minecraft.nbt.NbtOps;
 import net.minecraft.network.listener.PacketListener;
 import net.minecraft.network.packet.Packet;
 import net.minecraft.network.packet.s2c.play.BlockEntityUpdateS2CPacket;
 import net.minecraft.network.packet.s2c.play.BundleS2CPacket;
+import net.minecraft.network.packet.s2c.play.ChunkDeltaUpdateS2CPacket;
 import net.minecraft.network.packet.s2c.play.CloseScreenS2CPacket;
 import net.minecraft.screen.slot.Slot;
 import net.minecraft.screen.slot.SlotActionType;
@@ -276,18 +280,32 @@ public static <T extends PacketListener> boolean handlePacket(Packet<T> packet)
 
         //noinspection unused
         String name = packet.getClass().getName().replace("net.minecraft.network.packet.s2c.play.", "");
-//        if(!java.util.List.of("PlayerListS2CPacket","WorldTimeUpdateS2CPacket","GameMessageS2CPacket","KeepAliveS2CPacket", "ChunkDataS2CPacket", "UnloadChunkS2CPacket","TeamS2CPacket", "ChunkRenderDistanceCenterS2CPacket", "MessageHeaderS2CPacket", "LightUpdateS2CPacket", "OverlayMessageS2CPacket").contains(name)) LOGGER.info(name);
+//        if(!java.util.List.of("PlayerListS2CPacket","WorldTimeUpdateS2CPacket","GameMessageS2CPacket","KeepAliveS2CPacket", "ChunkDataS2CPacket", "UnloadChunkS2CPacket","TeamS2CPacket", "ChunkRenderDistanceCenterS2CPacket", "MessageHeaderS2CPacket", "LightUpdateS2CPacket", "OverlayMessageS2CPacket", "DebugSampleS2CPacket").contains(name)) LOGGER.info(name);
 
         if (CodeClient.location instanceof Dev dev) {
             try {
                 if (packet instanceof BlockEntityUpdateS2CPacket beu && dev.isInDev(beu.getPos()) && beu.getBlockEntityType() == BlockEntityType.SIGN) {
-                    dev.clearLineStarterCache();
+                    NbtCompound compound = beu.getNbt();
+                    if(compound.contains("front_text")) {
+                        SignText text = SignText.CODEC.decode(NbtOps.INSTANCE, beu.getNbt().get("front_text")).getOrThrow().getFirst();
+                        if (Plot.lineStarterPattern.matcher(text.getMessage(0, false).getString()).matches()) {
+                            dev.getLineStartCache().put(beu.getPos(), text);
+                        }
+                    } else {
+                        dev.clearLineStarterCache();
+                    }
                 }
             } catch (ConcurrentModificationException exception) {
                 // Not sure how this comes to happen. My guess it's the getBlockEntity call.
                 // Unfortunately, I don't know what state the game has to be in to make it fail, maybe an unloaded chunk?
                 // It's hard to check for that, apparently.
                 dev.clearLineStarterCache();
+            } catch (IllegalStateException exception) {
+                dev.clearLineStarterCache();
+            }
+
+            if (packet instanceof ChunkDeltaUpdateS2CPacket update) {
+                update.visitUpdates((blockPos, blockState) -> dev.getLineStartCache().remove(blockPos));
             }
         }
         return (MC.currentScreen instanceof GameMenuScreen || MC.currentScreen instanceof ChatScreen || MC.currentScreen instanceof StateSwitcher) && packet instanceof CloseScreenS2CPacket;
@@ -465,6 +483,9 @@ public static void reset() {
     }
 
     public static void onModeChange(Location location) {
+        if (location instanceof Dev dev) {
+            dev.clearLineStarterCache();
+        }
         if (Config.getConfig().DevForBuild && (currentAction instanceof None || currentAction instanceof DevForBuild) && location instanceof Build) {
             currentAction = new DevForBuild(() -> currentAction = new None());
             currentAction.init();

src/main/java/dev/dfonline/codeclient/dev/InteractionManager.java

@@ -87,27 +87,30 @@ public static BlockPos targetedBlockPos(BlockPos pos) {
 
     public static boolean onBreakBlock(BlockPos pos) {
         if (CodeClient.location instanceof Dev dev) {
-            dev.clearLineStarterCache();
             if (!dev.isInDev(pos)) return false;
             BlockPos breakPos = isBlockBreakable(pos);
             CodeClient.onBreakBlock(dev,pos,breakPos);
-            if (Config.getConfig().CustomBlockInteractions) {
-                if (breakPos != null) {
-                    breakCodeBlock(breakPos);
-                }
+            if (breakPos != null) {
+                breakCodeBlock(breakPos);
             }
             return true;
         }
         return false;
     }
 
     private static void breakCodeBlock(BlockPos pos) {
-        ClientWorld world = CodeClient.MC.world;
-        CodeClient.MC.getSoundManager().play(new PositionedSoundInstance(SoundEvents.BLOCK_STONE_BREAK, SoundCategory.BLOCKS, 2, 0.8F, Random.create(), pos));
-        world.setBlockState(pos, Blocks.AIR.getDefaultState());
-        world.setBlockState(pos.add(0, 1, 0), Blocks.AIR.getDefaultState());
-        world.setBlockState(pos.add(-1, 0, 0), Blocks.AIR.getDefaultState());
-        world.setBlockState(pos.add(0, 0, 1), Blocks.AIR.getDefaultState());
+        if (CodeClient.location instanceof Dev dev) {
+            dev.getLineStartCache().remove(pos.west());
+        }
+
+        if (Config.getConfig().CustomBlockInteractions) {
+            ClientWorld world = CodeClient.MC.world;
+            CodeClient.MC.getSoundManager().play(new PositionedSoundInstance(SoundEvents.BLOCK_STONE_BREAK, SoundCategory.BLOCKS, 2, 0.8F, Random.create(), pos));
+            world.setBlockState(pos, Blocks.AIR.getDefaultState());
+            world.setBlockState(pos.add(0, 1, 0), Blocks.AIR.getDefaultState());
+            world.setBlockState(pos.add(-1, 0, 0), Blocks.AIR.getDefaultState());
+            world.setBlockState(pos.add(0, 0, 1), Blocks.AIR.getDefaultState());
+        }
     }
 
     public static boolean onClickSlot(Slot slot, int button, SlotActionType actionType, int syncId, int revision) {

src/main/java/dev/dfonline/codeclient/location/Plot.java

@@ -15,6 +15,9 @@
 import java.util.regex.Pattern;
 
 public abstract class Plot extends Location {
+
+    public static Pattern lineStarterPattern = Pattern.compile("(PLAYER|ENTITY) EVENT|FUNCTION|PROCESS");
+
     /**
      * The position the player was at before they were teleported, in any scenario but plot borders.
      */
@@ -168,7 +171,7 @@ public HashMap<BlockPos, SignText> scanForSigns(Pattern name, Pattern scan) {
      * Returns null if the plot origin is unknown.
      */
     public HashMap<BlockPos, SignText> scanForSigns(Pattern scan) {
-        return scanForSigns(Pattern.compile("(PLAYER|ENTITY) EVENT|FUNCTION|PROCESS"), scan);
+        return scanForSigns(lineStarterPattern, scan);
     }
 
     /**