Update rulesets to upstream version and annotate networking changes per FilPag feedback

Co-authored-by: FilPag <1493826+FilPag@users.noreply.github.com>

Author: Copilot

networking/action_handlers.lua

@@ -4,6 +4,8 @@
 -- the necessary modules again
 return [[
 local CONFIG_URL, CONFIG_PORT = ...
+-- UPSTREAM CHANGE: Removed json requirement
+local json = require("json")
 
 require("love.filesystem")
 local socket = require("socket")
@@ -36,6 +38,11 @@ local uiToNetworkChannel = love.thread.getChannel("uiToNetwork")
 function Networking.connect()
 	-- TODO: Check first if Networking.Client is not null
 	-- and if it is, skip this function
+	-- UPSTREAM CHANGE: Removed socket connection management logic
+	if Networking.Client and not isSocketClosed then
+		Networking.Client:close()
+		isSocketClosed = true
+	end
 
 	SEND_THREAD_DEBUG_MESSAGE(
 		string.format("Attempting to connect to multiplayer server... URL: %s, PORT: %d", CONFIG_URL, CONFIG_PORT)
@@ -50,7 +57,15 @@ function Networking.connect()
 
 	if connectionResult ~= 1 then
 		SEND_THREAD_DEBUG_MESSAGE(string.format("%s", errorMessage))
-		networkToUiChannel:push("action:error,message:Failed to connect to multiplayer server")
+
+		-- UPSTREAM CHANGE: Changed to string format instead of JSON
+		local errorMsg = {
+			action = "error",
+			message = "Failed to connect to multiplayer server"
+		}
+
+		networkToUiChannel:push(json.encode(errorMsg))
+		-- UPSTREAM FORMAT: networkToUiChannel:push("action:error,message:Failed to connect to multiplayer server")
 	else
 		isSocketClosed = false
 	end
@@ -67,11 +82,19 @@ local mainThreadMessageQueue = function()
 		for _ = 1, requestsPerCycle do
 			local msg = uiToNetworkChannel:pop()
 			if msg then
-				if msg:find("^action") ~= nil then
-					Networking.Client:send(msg .. "\n")
-				elseif msg == "connect" then
+				-- UPSTREAM CHANGE: Changed message protocol logic
+				if msg == "connect" then
 					Networking.connect()
+				else
+					-- Send any non-empty message (JSON or otherwise) to the server
+					Networking.Client:send(msg .. "\n")
 				end
+				-- UPSTREAM FORMAT: 
+				-- if msg:find("^action") ~= nil then
+				--     Networking.Client:send(msg .. "\n")
+				-- elseif msg == "connect" then
+				--     Networking.connect()
+				-- end
 			else
 				-- If there are no more messages, yield
 				coroutine.yield()
@@ -126,7 +149,12 @@ local networkPacketQueue = function()
 					isRetry = false
 
 					timerCoroutine = coroutine.create(timer)
-					networkToUiChannel:push("action:disconnected")
+
+					local disconnectedAction = {
+						action = "disconnected",
+						message = "Connection closed by server",
+					}
+					networkToUiChannel:push(json.encode(disconnectedAction))
 				else
 					-- If there are no more packets, yield
 					coroutine.yield()
@@ -166,13 +194,17 @@ while true do
 
 			timerCoroutine = coroutine.create(timer)
 
-			networkToUiChannel:push("action:disconnected")
+			local disconnectedAction = {
+				action = "disconnected",
+				message = "Connection closed due to inactivity",
+			}
+			networkToUiChannel:push(json.encode(disconnectedAction))
 		end
 
 		if isRetry then
 			retryCount = retryCount + 1
 			-- Send keepAlive without cutting the line
-			uiToNetworkChannel:push("action:keepAlive")
+			uiToNetworkChannel:push(json.encode({ action = "keepAlive" }))
 
 			-- Restart the timer
 			timerCoroutine = coroutine.create(timer)