@chrrxs/robloxstudio-mcp 3.1.0 → 3.1.1

package/dist/index.js

@@ -6116,7 +6116,7 @@ var TOOL_HANDLERS = {
   manage_instance: (tools, body) => tools.manageInstance(body),
   solo_playtest: (tools, body) => tools.soloPlaytest(body.action, body.mode, body.timeout, body.instance_id),
   multiplayer_playtest: (tools, body) => tools.multiplayerPlaytest(body.action, body.numPlayers, body.target, body.testArgs, body.value, body.timeout, body.instance_id),
-  get_runtime_logs: (tools, body) => tools.getRuntimeLogs(body.instance_id, body.multiplayer_group_id, body.cursor, body.cursor_by_instance, body.tail, body.filter),
+  get_runtime_logs: (tools, body, context) => tools.getRuntimeLogs(body.instance_id, body.multiplayer_group_id, body.cursor, body.cursor_by_instance, body.tail, body.filter, context?.signal),
   capture_script_profiler: (tools, body) => tools.captureScriptProfiler(body.target, {
     duration_ms: body.duration_ms,
     frequency: body.frequency,
@@ -9506,6 +9506,7 @@ var MAX_DEVICE_MATRIX_ENTRIES = 6;
 var MAX_NETWORK_PACKET_LOSS_PERCENT = 0.5;
 var GREP_SCRIPTS_TIMEOUT_MS = 12e4;
 var MAX_GREP_PATTERN_UTF8_BYTES = 4096;
+var RUNTIME_LOG_PEER_TIMEOUT_MS = 5e3;
 var STUDIO_ASSISTANT_SOURCE_IMAGE_LABEL = "Studio Assistant Source Image";
 var CREATOR_STORE_SEARCH_TYPES = /* @__PURE__ */ new Set([
   "Audio",
@@ -11367,7 +11368,7 @@ var RobloxStudioTools = class {
       ]
     };
   }
-  async getRuntimeLogs(instance_id, multiplayer_group_id, cursor, cursor_by_instance, tail, filter) {
+  async getRuntimeLogs(instance_id, multiplayer_group_id, cursor, cursor_by_instance, tail, filter, signal) {
     if (instance_id !== void 0 && multiplayer_group_id !== void 0) {
       throw new Error("get_runtime_logs accepts only one of instance_id or multiplayer_group_id.");
     }
@@ -11511,7 +11512,7 @@ var RobloxStudioTools = class {
         if (filter !== void 0)
           data.filter = filter;
         try {
-          const responseValue = await this.client.request("/api/get-runtime-logs", data, peer.peerId);
+          const responseValue = await this.client.request("/api/get-runtime-logs", data, peer.peerId, RUNTIME_LOG_PEER_TIMEOUT_MS, signal);
           if (typeof responseValue !== "object" || responseValue === null || Array.isArray(responseValue)) {
             return {
               peerId: peer.peerId,
@@ -11538,6 +11539,8 @@ var RobloxStudioTools = class {
             nextSince: response.nextSince
           };
         } catch (error) {
+          if (signal?.aborted)
+            throw error;
           return { peerId: peer.peerId, role: peer.role, error: errorMessage(error) };
         }
       }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "MCP server for testing, debugging, and controlling Roblox Studio from AI coding tools",
   "main": "dist/index.js",
   "type": "module",

package/studio-plugin/MCPPlugin.rbxmx

@@ -1472,9 +1472,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "3.1.0" is replaced with the package version at package time
+	-- "3.1.1" is replaced with the package version at package time
 	-- (scripts/build-plugin.mjs injectVersion), so a release bump also restamps.
-	return `{tostring(h)}-3.1.0`
+	return `{tostring(h)}-3.1.1`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -9736,7 +9736,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "3.1.0"
+local CURRENT_VERSION = "3.1.1"
 local PLUGIN_VARIANT = "main"
 local BASE_PORT = 58741
 local function createConnection(port)
@@ -10013,6 +10013,8 @@ local INITIAL_RESPONSE_RETRY_DELAY_SECONDS = 0.5
 local MAX_RESPONSE_RETRY_DELAY_SECONDS = 5
 local MAX_TERMINAL_RESPONSES = 256
 local STREAM_SILENCE_TIMEOUT_SECONDS = 20
+-- The bridge accepts request bodies up to 50 MiB; leave room for its envelope.
+local MAX_STREAM_FRAME_BYTES = 64 * 1024 * 1024
 local options
 local active = false
 local shutdownSuspended = false
@@ -10026,15 +10028,13 @@ local pendingResponses = {}
 local terminalResponseIds = {}
 local terminalResponseOrder = {}
 local readyFailureLogKeys = {}
-local function decodeMessage(message)
-	-- Studio versions in the supported channel have surfaced either the SSE
-	-- data payload or the complete single-line `data:` frame. The bridge emits
-	-- one JSON data line per event, so normalize both forms before decoding.
-	local payload = message
-	local normalized = (string.gsub((string.gsub(message, "\r\n", "\n")), "\r", "\n"))
-	if string.sub(normalized, 1, 5) == "data:" then
-		payload = (string.gsub((string.gsub(string.sub(normalized, 6), "^%s+", "")), "%s+$", ""))
-	end
+local streamLineParts = {}
+local streamDataLines = {}
+local streamFrameBytes = 0
+local streamLineHasContent = false
+local streamDiscardingFrame = false
+local streamSkipLf = false
+local function decodeMessage(payload)
 	local decodeOk, decoded = pcall(function()
 		return HttpService:JSONDecode(payload)
 	end)
@@ -10135,9 +10135,123 @@ local function decodeMessage(message)
 	end
 	return nil
 end
+local function resetStreamFrame()
+	streamLineParts = {}
+	streamDataLines = {}
+	streamFrameBytes = 0
+	streamLineHasContent = false
+	streamDiscardingFrame = false
+end
+local function finishStreamLine(events)
+	if not streamLineHasContent then
+		if not streamDiscardingFrame and #streamDataLines > 0 then
+			local event = decodeMessage(table.concat(streamDataLines, "\n"))
+			if event ~= nil then
+				table.insert(events, event)
+			end
+		end
+		resetStreamFrame()
+		return nil
+	end
+	if not streamDiscardingFrame then
+		local line = table.concat(streamLineParts, "")
+		if string.sub(line, 1, 5) == "data:" then
+			local start = if string.sub(line, 6, 6) == " " then 7 else 6
+			local _streamDataLines = streamDataLines
+			local _arg0 = string.sub(line, start)
+			table.insert(_streamDataLines, _arg0)
+		elseif line == "data" then
+			table.insert(streamDataLines, "")
+		end
+	end
+	streamLineParts = {}
+	streamLineHasContent = false
+end
+-- MessageReceived may coalesce frames (notably server/client fanout) or split
+-- them across callbacks. Callback boundaries are not SSE event boundaries.
+local function decodeMessages(message)
+	local events = {}
+	local messageBytes = #message
+	if messageBytes == 0 then
+		return events
+	end
+	-- Some Studio versions deliver the decoded SSE data or a complete data line
+	-- without delimiters. Only recognize those forms with no unfinished frame;
+	-- once framing starts, callback boundaries never terminate partial data.
+	if streamFrameBytes == 0 and not streamDiscardingFrame and messageBytes <= MAX_STREAM_FRAME_BYTES then
+		local first = string.sub(message, 1, 1)
+		if first == "{" or first == " " or first == "\t" or first == "\r" or first == "\n" then
+			local event = decodeMessage(message)
+			if event ~= nil then
+				streamSkipLf = false
+				table.insert(events, event)
+				return events
+			end
+			if first == "{" then
+				return events
+			end
+		end
+		if string.sub(message, 1, 5) == "data:" and (string.find(message, "[\r\n]")) == nil then
+			local event = decodeMessage(string.sub(message, 6))
+			if event ~= nil then
+				streamSkipLf = false
+				table.insert(events, event)
+				return events
+			end
+		end
+	end
+	local offset = 1
+	while offset <= messageBytes do
+		if streamSkipLf then
+			streamSkipLf = false
+			local _message = message
+			local _offset = offset
+			local _offset_1 = offset
+			if string.sub(_message, _offset, _offset_1) == "\n" then
+				offset += 1
+				continue
+			end
+		end
+		local _message = message
+		local _offset = offset
+		local newline = string.find(_message, "[\r\n]", _offset)
+		local lineEnd = if newline == nil then messageBytes + 1 else newline
+		local partBytes = lineEnd - offset
+		if partBytes > 0 then
+			streamLineHasContent = true
+		end
+		if not streamDiscardingFrame then
+			streamFrameBytes += partBytes + (if newline == nil then 0 else 1)
+			if streamFrameBytes > MAX_STREAM_FRAME_BYTES then
+				-- Keep scanning delimiters but retain no oversized frame bytes.
+				streamDiscardingFrame = true
+				streamLineParts = {}
+				streamDataLines = {}
+			elseif partBytes > 0 then
+				-- Join only on a line boundary, not every callback (large scripts
+				-- can arrive in many fragments).
+				local _streamLineParts = streamLineParts
+				local _message_1 = message
+				local _offset_1 = offset
+				local _arg1 = lineEnd - 1
+				local _arg0 = string.sub(_message_1, _offset_1, _arg1)
+				table.insert(_streamLineParts, _arg0)
+			end
+		end
+		if newline == nil then
+			break
+		end
+		finishStreamLine(events)
+		streamSkipLf = string.sub(message, newline, newline) == "\r"
+		offset = newline + 1
+	end
+	return events
+end
 local function closeCurrentStream()
 	local current = streamClient
 	streamClient = nil
+	resetStreamFrame()
+	streamSkipLf = false
 	for _, connection in streamConnections do
 		connection:Disconnect()
 	end
@@ -10616,30 +10730,33 @@ function connect(expectedGeneration)
 			if not active or generation ~= expectedGeneration or streamClient ~= createdClient then
 				return nil
 			end
-			local event = decodeMessage(message)
-			if event == nil then
-				return nil
-			end
-			lastValidEventAt = tick()
-			if event.kind == "heartbeat" then
-				invokeCallback("event stream heartbeat", function()
-					return currentOptions.onHeartbeat(event.timestamp)
+			for _, event in decodeMessages(message) do
+				-- Status callbacks can refresh/stop the stream. The remainder of
+				-- that callback belongs to the old connection, not its successor.
+				if not active or generation ~= expectedGeneration or streamClient ~= createdClient then
+					return nil
+				end
+				lastValidEventAt = tick()
+				if event.kind == "heartbeat" then
+					invokeCallback("event stream heartbeat", function()
+						return currentOptions.onHeartbeat(event.timestamp)
+					end)
+					continue
+				end
+				if event.kind == "cancel" then
+					cancelRequest(event)
+					continue
+				end
+				if event.kind == "request" then
+					dispatchRequest(event)
+					continue
+				end
+				invokeCallback("event stream status", function()
+					return currentOptions.onStatus(event)
 				end)
-				return nil
-			end
-			if event.kind == "cancel" then
-				cancelRequest(event)
-				return nil
-			end
-			if event.kind == "request" then
-				dispatchRequest(event)
-				return nil
-			end
-			invokeCallback("event stream status", function()
-				return currentOptions.onStatus(event)
-			end)
-			if not event.knownPeer then
-				refresh()
+				if not event.knownPeer then
+					refresh()
+				end
 			end
 		end), createdClient.Error:Connect(function(statusCode, message)
 			if not active or generation ~= expectedGeneration or streamClient ~= createdClient then