Clean up tunnel cap for release

Author: robotgryphon

src/main/java/dev/compactmods/machines/core/Tunnels.java

@@ -55,7 +55,7 @@ public static TunnelDefinition getDefinition(ResourceLocation id) {
     // ================================================================================================================
     //   TUNNEL TYPE DEFINITIONS
     // ================================================================================================================
-    public static final RegistryObject<TunnelDefinition> ITEM_IN_DEF = DEFINITIONS.register("item_in", ItemImportTunnel::new);
+    // public static final RegistryObject<TunnelDefinition> ITEM_IN_DEF = DEFINITIONS.register("item_in", ItemImportTunnel::new);
 
     // ================================================================================================================
     //   TUNNEL BLOCKS / TILES

src/main/java/dev/compactmods/machines/rooms/capability/RoomChunkCapabilities.java

@@ -10,56 +10,41 @@
 import net.minecraftforge.common.util.LazyOptional;
 
 public class RoomChunkCapabilities implements IRoomCapabilities {
-    private final HashMap<TunnelDefinition, HashSet<Direction>> capabilities;
-    private final HashMap<TunnelDefinition, HashMap<Direction, LazyOptional<?>>> sidedCache;
+    private final HashMap<TunnelDefinition, HashSet<Direction>> placedTunnels;
+    // private final HashMap<TunnelDefinition, HashMap<T, TunnelSideCache<T>>> sidedCache;
 
     public RoomChunkCapabilities(LevelChunk chunk) {
-        this.capabilities = new HashMap<>();
-        this.sidedCache = new HashMap<>();
+        this.placedTunnels = new HashMap<>();
+        // this.sidedCache = new HashMap<>();
     }
 
     @Override
     public <CapType, TunnType extends TunnelDefinition> void addCapability(TunnType tunnel, Capability<CapType> capability, CapType instance, Direction side) {
-        capabilities.putIfAbsent(tunnel, new HashSet<>());
-        sidedCache.putIfAbsent(tunnel, new HashMap<>());
+        placedTunnels.putIfAbsent(tunnel, new HashSet<>());
 
-        var sides = capabilities.get(tunnel);
+        var sides = placedTunnels.get(tunnel);
         if (!sides.isEmpty() && sides.contains(side))
             return;
 
         sides.add(side);
-        var cache = sidedCache.get(tunnel);
-        cache.put(side, LazyOptional.of(() -> instance));
     }
 
     @Override
     public <CapType, TunnType extends TunnelDefinition> void removeCapability(TunnType tunnel, Capability<CapType> capability, Direction side) {
-        if (!capabilities.containsKey(tunnel))
+        if (!placedTunnels.containsKey(tunnel))
             return;
 
-        var sides = capabilities.get(tunnel);
-        if (sides.contains(side)) {
-            sides.remove(side);
-            if (sidedCache.containsKey(tunnel)) {
-                var cache = sidedCache.get(tunnel);
-                if (cache.containsKey(side)) {
-                    cache.get(side).invalidate();
-                    cache.remove(side);
-                }
-            }
-        }
+        var sides = placedTunnels.get(tunnel);
+        sides.remove(side);
     }
 
     @Override
     public <CapType, TunnType extends TunnelDefinition> LazyOptional<CapType> getCapability(TunnType tunnel, Capability<CapType> capability, Direction side) {
         if (side == null)
             return LazyOptional.empty();
 
-        if (capabilities.containsKey(tunnel)) {
-            // get sides tunnel applied to
-            var c = capabilities.get(tunnel);
-            if (c.contains(side) && sidedCache.containsKey(tunnel))
-                return sidedCache.get(tunnel).get(side).cast();
+        if (placedTunnels.containsKey(tunnel) && placedTunnels.get(tunnel).contains(side)) {
+            // TODO: delegate capability lookup back to tunnel definition, with storage
         }
 
         return LazyOptional.empty();