Implemented "Click Window" for 1.18 (now sending the new required fields), formated few files with Visual Studio formatting. PS: left debug for people to see, I'll remove it once everything is working

MinecraftClient/Inventory/Container.cs

@@ -25,6 +25,11 @@ namespace MinecraftClient.Inventory
         /// </summary>
         public string Title;
 
+        /// <summary>
+        /// state of container
+        /// </summary>
+        public int StateID;
+
         /// <summary>
         /// Container Items
         /// </summary>

MinecraftClient/McClient.cs

@@ -1244,7 +1244,7 @@ namespace MinecraftClient
                 item = inventories[windowId].Items[slotId];
 
             // Inventory update must be after sending packet
-            bool result = handler.SendWindowAction(windowId, slotId, action, item);
+            bool result = handler.SendWindowAction(windowId, slotId, action, item, inventories[windowId].Items, inventories[windowId].StateID);
 
             // Update our inventory base on action type
             var inventory = GetInventory(windowId);
@@ -2030,11 +2030,12 @@ namespace MinecraftClient
         /// </summary>
         /// <param name="inventoryID">Inventory ID</param>
         /// <param name="itemList">Item list, key = slot ID, value = Item information</param>
-        public void OnWindowItems(byte inventoryID, Dictionary<int, Inventory.Item> itemList)
+        public void OnWindowItems(byte inventoryID, Dictionary<int, Inventory.Item> itemList, int stateId)
         {
             if (inventories.ContainsKey(inventoryID))
             {
                 inventories[inventoryID].Items = itemList;
+                inventories[inventoryID].StateID = stateId;
                 DispatchBotEvent(bot => bot.OnInventoryUpdate(inventoryID));
             }
         }

MinecraftClient/Protocol/Handlers/DataTypes.cs

@@ -1028,6 +1028,23 @@ namespace MinecraftClient.Protocol.Handlers
             return slotData.ToArray();
         }
 
+        public string ByteArrayToString(byte[] ba)
+        {
+            return BitConverter.ToString(ba).Replace("-", " ");
+        }
+
+        public byte[] GetSlotsArray(Dictionary<int, Item> items, ItemPalette itemPalette)
+        {
+            byte[] slotsArray = new byte[items.Count];
+
+            foreach (KeyValuePair<int, Item> item in items)
+            {
+                slotsArray = ConcatBytes(slotsArray, GetShort((short)item.Key), GetItemSlot(item.Value, itemPalette));
+            }
+
+            return slotsArray;
+        }
+
         /// <summary>
         /// Get protocol block face from Direction
         /// </summary>

MinecraftClient/Protocol/Handlers/Protocol16.cs

@@ -94,7 +94,8 @@ namespace MinecraftClient.Protocol.Handlers
             int nbr = 0;
             switch (id)
             {
-                case 0x00: byte[] keepalive = new byte[5] { 0, 0, 0, 0, 0 };
+                case 0x00:
+                    byte[] keepalive = new byte[5] { 0, 0, 0, 0, 0 };
                     Receive(keepalive, 1, 4, SocketFlags.None);
                     handler.OnServerKeepAlive();
                     Send(keepalive); break;
@@ -183,11 +184,13 @@ namespace MinecraftClient.Protocol.Handlers
                 case 0xCF: if (protocolversion > 51) { readNextString(); readData(1); readNextString(); } readData(4); break;
                 case 0xD0: if (protocolversion > 51) { readData(1); readNextString(); } break;
                 case 0xD1: if (protocolversion > 51) { readNextTeamData(); } break;
-                case 0xFA: string channel = readNextString();
+                case 0xFA:
+                    string channel = readNextString();
                     byte[] payload = readNextByteArray();
                     handler.OnPluginChannelMessage(channel, payload);
                     break;
-                case 0xFF: string reason = readNextString();
+                case 0xFF:
+                    string reason = readNextString();
                     handler.OnConnectionLost(ChatBot.DisconnectReason.InGameKick, reason); break;
                 default: return false; //unknown packet!
             }
@@ -717,7 +720,7 @@ namespace MinecraftClient.Protocol.Handlers
             return false; //Currently not implemented
         }
 
-        public bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item)
+        public bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item, Dictionary<int, Item> Items, int stateId)
         {
             return false; //Currently not implemented
         }
@@ -759,7 +762,8 @@ namespace MinecraftClient.Protocol.Handlers
         /// <param name="data">packet Data</param>
         public bool SendPluginChannelPacket(string channel, byte[] data)
         {
-            try {
+            try
+            {
                 byte[] channelLength = BitConverter.GetBytes((short)channel.Length);
                 Array.Reverse(channelLength);
 

MinecraftClient/Protocol/Handlers/Protocol18.cs

@@ -850,7 +850,7 @@ namespace MinecraftClient.Protocol.Handlers
                                     if (item != null)
                                         inventorySlots[slotId] = item;
                                 }
-                                handler.OnWindowItems(windowId, inventorySlots);
+                                handler.OnWindowItems(windowId, inventorySlots, stateId);
                             }
                             break;
                         case PacketTypesIn.SetSlot:
@@ -1895,7 +1895,7 @@ namespace MinecraftClient.Protocol.Handlers
             catch (ObjectDisposedException) { return false; }
         }
 
-        public bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item)
+        public bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item, Dictionary<int, Item> items, int stateId)
         {
             try
             {
@@ -1931,14 +1931,51 @@ namespace MinecraftClient.Protocol.Handlers
                 }
 
                 List<byte> packet = new List<byte>();
+
+                log.Info("Window id: " + windowId + " - State id: " + stateId + " - Slot id: " + slotId + " - Mode: " + mode);
+                log.Info("Bytes > " + (byte)windowId + " - State id: " + dataTypes.ByteArrayToString(dataTypes.GetVarInt(stateId)) + " - Slot id: " + dataTypes.ByteArrayToString(dataTypes.GetVarInt(slotId)) + " - Mode: " + dataTypes.ByteArrayToString(dataTypes.GetVarInt(mode)));
+
                 packet.Add((byte)windowId);
+
+                // 1.18+
+                if (protocolversion > MC1171Version)
+                {
+                    packet.AddRange(dataTypes.GetVarInt(stateId));
                     packet.AddRange(dataTypes.GetShort((short)slotId));
+                }
+                // 1.17.1
+                else if (protocolversion == MC1171Version)
+                {
+                    packet.AddRange(dataTypes.GetShort((short)slotId));
+                    packet.AddRange(dataTypes.GetVarInt(stateId));
+                }
+                // Older
+                else
+                {
+                    packet.AddRange(dataTypes.GetShort((short)slotId));
+                }
+
                 packet.Add(button);
                 if (protocolversion < MC117Version) packet.AddRange(dataTypes.GetShort(actionNumber));
-                if (protocolversion >= MC19Version)
+                if (protocolversion >= MC1165Version)
                     packet.AddRange(dataTypes.GetVarInt(mode));
-                else packet.Add(mode);
+
+                // 1.17+
+                if (protocolversion >= MC117Version)
+                {
+                    byte[] arrayOfSlots = dataTypes.GetSlotsArray(items, itemPalette);
+
+                    log.Info("Length: " + sizeof(byte) * arrayOfSlots.Length);
+                    log.Info("Array: " + dataTypes.ByteArrayToString(arrayOfSlots));
+
+                    packet.AddRange(dataTypes.GetVarInt(sizeof(byte) * arrayOfSlots.Length));
+                    packet.AddRange(arrayOfSlots);
+                }
+
                 packet.AddRange(dataTypes.GetItemSlot(item, itemPalette));
+
+                log.Info("Packet data: " + dataTypes.ByteArrayToString(packet.ToArray()));
+
                 SendPacket(PacketTypesOut.ClickWindow, packet);
                 return true;
             }

MinecraftClient/Protocol/IMinecraftCom.cs

@@ -165,7 +165,7 @@ namespace MinecraftClient.Protocol
         /// <param name="buttom">Action to perform</param>
         /// <param name="item">Item in the clicked slot</param>
         /// <returns>True if packet was successfully sent</returns>
-        bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item);
+        bool SendWindowAction(int windowId, int slotId, WindowActionType action, Item item, Dictionary<int, Item> Items, int stateId);
 
         /// <summary>
         /// Request Creative Mode item creation into regular/survival Player Inventory

MinecraftClient/Protocol/IMinecraftComHandler.cs

@@ -263,7 +263,8 @@ namespace MinecraftClient.Protocol
         /// </summary>
         /// <param name="inventoryID">Inventory ID</param>
         /// <param name="itemList">Item list</param>
-        void OnWindowItems(byte inventoryID, Dictionary<int, Item> itemList);
+        /// <param name="stateId">State ID</param>
+        void OnWindowItems(byte inventoryID, Dictionary<int, Item> itemList, int stateId);
 
         /// <summary>
         /// Called when a single slot has been updated inside an inventory