@chrrxs/robloxstudio-mcp-inspector 2.22.1 → 2.22.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp-inspector",
-  "version": "2.22.1",
+  "version": "2.22.2",
   "description": "Read-only MCP server for inspecting and debugging Roblox Studio from AI coding tools",
   "main": "dist/index.js",
   "type": "module",

package/studio-plugin/MCPInspectorPlugin.rbxmx

@@ -843,19 +843,33 @@ local function processRequest(request)
 	end
 end
 local function sendResponse(conn, requestId, responseData)
-	pcall(function()
-		HttpService:RequestAsync({
-			Url = `{conn.serverUrl}/response`,
+	local responseUrl = `{conn.serverUrl}/response`
+	local encodeOk, encoded = pcall(function()
+		return HttpService:JSONEncode({
+			requestId = requestId,
+			response = responseData,
+		})
+	end)
+	local body = if encodeOk then encoded else HttpService:JSONEncode({
+		requestId = requestId,
+		error = `Plugin response serialization failed: {tostring(encoded)}`,
+	})
+	if not encodeOk then
+		warn(`[robloxstudio-mcp] Failed to serialize response {requestId}: {tostring(encoded)}`)
+	end
+	local requestOk, requestResult = pcall(function()
+		return HttpService:RequestAsync({
+			Url = responseUrl,
 			Method = "POST",
 			Headers = {
 				["Content-Type"] = "application/json",
 			},
-			Body = HttpService:JSONEncode({
-				requestId = requestId,
-				response = responseData,
-			}),
+			Body = body,
 		})
 	end)
+	if not requestOk or not requestResult.Success then
+		warn(`[robloxstudio-mcp] Failed to deliver response {requestId}: {HttpDiagnostics.formatRequestFailure(responseUrl, requestOk, requestResult)}`)
+	end
 end
 local function getConnectionStatus()
 	local conn = State.getActiveConnection()
@@ -1425,9 +1439,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.22.1" is replaced with the package version at package time
+	-- "2.22.2" is replaced with the package version at package time
 	-- (scripts/build-plugin.mjs injectVersion), so a release bump also restamps.
-	return `{tostring(h)}-2.22.1`
+	return `{tostring(h)}-2.22.2`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10437,6 +10451,41 @@ end
 local function nowSec()
 	return DateTime.now().UnixTimestampMillis / 1000
 end
+-- Studio occasionally exposes binary-bearing Output messages through
+-- LogService (for example, plugin hydration diagnostics containing raw CSG
+-- data). HttpService:JSONEncode rejects those strings outright. Preserve all
+-- valid UTF-8 verbatim and make only malformed bytes JSON-safe and visible.
+local function escapeInvalidUtf8(msg)
+	local valid = utf8.len(msg)
+	-- Roblox currently returns nil (not the false declared by @rbxts/types)
+	-- when it encounters a malformed sequence. A numeric result is the only
+	-- portable success discriminator across both representations.
+	if type(valid) == "number" then
+		return msg
+	end
+	local parts = {}
+	local cursor = 1
+	while cursor <= #msg do
+		local suffixValid, invalidPosition = utf8.len(msg, cursor)
+		if type(suffixValid) == "number" then
+			local _arg0 = string.sub(msg, cursor)
+			table.insert(parts, _arg0)
+			break
+		end
+		if not (type(invalidPosition) == "number") then
+			break
+		end
+		if invalidPosition > cursor then
+			local _arg0 = string.sub(msg, cursor, invalidPosition - 1)
+			table.insert(parts, _arg0)
+		end
+		local invalidByte = string.byte(msg, invalidPosition)
+		local _arg0 = string.format("\\x%02X", invalidByte)
+		table.insert(parts, _arg0)
+		cursor = invalidPosition + 1
+	end
+	return table.concat(parts, "")
+end
 local function dropOldestUntilFits(incomingBytes)
 	while #entries > 0 and (totalBytes + incomingBytes > MAX_BYTES or #entries >= HARD_ENTRY_CAP) do
 		local dropped = table.remove(entries, 1)
@@ -10448,13 +10497,14 @@ local function pushEntry(msg, t, ts)
 	if ts == nil then
 		ts = nowSec()
 	end
-	local bytes = #msg
+	local safeMessage = escapeInvalidUtf8(msg)
+	local bytes = #safeMessage
 	dropOldestUntilFits(bytes)
 	local _arg0 = {
 		seq = nextSeq,
 		ts = ts,
 		level = levelTag(t),
-		message = msg,
+		message = safeMessage,
 	}
 	table.insert(entries, _arg0)
 	nextSeq += 1
@@ -10763,7 +10813,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.22.1"
+local CURRENT_VERSION = "2.22.2"
 local PLUGIN_VARIANT = "inspector"
 local BASE_PORT = 58741
 local function createConnection(port)

package/studio-plugin/MCPPlugin.rbxmx

@@ -843,19 +843,33 @@ local function processRequest(request)
 	end
 end
 local function sendResponse(conn, requestId, responseData)
-	pcall(function()
-		HttpService:RequestAsync({
-			Url = `{conn.serverUrl}/response`,
+	local responseUrl = `{conn.serverUrl}/response`
+	local encodeOk, encoded = pcall(function()
+		return HttpService:JSONEncode({
+			requestId = requestId,
+			response = responseData,
+		})
+	end)
+	local body = if encodeOk then encoded else HttpService:JSONEncode({
+		requestId = requestId,
+		error = `Plugin response serialization failed: {tostring(encoded)}`,
+	})
+	if not encodeOk then
+		warn(`[robloxstudio-mcp] Failed to serialize response {requestId}: {tostring(encoded)}`)
+	end
+	local requestOk, requestResult = pcall(function()
+		return HttpService:RequestAsync({
+			Url = responseUrl,
 			Method = "POST",
 			Headers = {
 				["Content-Type"] = "application/json",
 			},
-			Body = HttpService:JSONEncode({
-				requestId = requestId,
-				response = responseData,
-			}),
+			Body = body,
 		})
 	end)
+	if not requestOk or not requestResult.Success then
+		warn(`[robloxstudio-mcp] Failed to deliver response {requestId}: {HttpDiagnostics.formatRequestFailure(responseUrl, requestOk, requestResult)}`)
+	end
 end
 local function getConnectionStatus()
 	local conn = State.getActiveConnection()
@@ -1425,9 +1439,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.22.1" is replaced with the package version at package time
+	-- "2.22.2" is replaced with the package version at package time
 	-- (scripts/build-plugin.mjs injectVersion), so a release bump also restamps.
-	return `{tostring(h)}-2.22.1`
+	return `{tostring(h)}-2.22.2`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10437,6 +10451,41 @@ end
 local function nowSec()
 	return DateTime.now().UnixTimestampMillis / 1000
 end
+-- Studio occasionally exposes binary-bearing Output messages through
+-- LogService (for example, plugin hydration diagnostics containing raw CSG
+-- data). HttpService:JSONEncode rejects those strings outright. Preserve all
+-- valid UTF-8 verbatim and make only malformed bytes JSON-safe and visible.
+local function escapeInvalidUtf8(msg)
+	local valid = utf8.len(msg)
+	-- Roblox currently returns nil (not the false declared by @rbxts/types)
+	-- when it encounters a malformed sequence. A numeric result is the only
+	-- portable success discriminator across both representations.
+	if type(valid) == "number" then
+		return msg
+	end
+	local parts = {}
+	local cursor = 1
+	while cursor <= #msg do
+		local suffixValid, invalidPosition = utf8.len(msg, cursor)
+		if type(suffixValid) == "number" then
+			local _arg0 = string.sub(msg, cursor)
+			table.insert(parts, _arg0)
+			break
+		end
+		if not (type(invalidPosition) == "number") then
+			break
+		end
+		if invalidPosition > cursor then
+			local _arg0 = string.sub(msg, cursor, invalidPosition - 1)
+			table.insert(parts, _arg0)
+		end
+		local invalidByte = string.byte(msg, invalidPosition)
+		local _arg0 = string.format("\\x%02X", invalidByte)
+		table.insert(parts, _arg0)
+		cursor = invalidPosition + 1
+	end
+	return table.concat(parts, "")
+end
 local function dropOldestUntilFits(incomingBytes)
 	while #entries > 0 and (totalBytes + incomingBytes > MAX_BYTES or #entries >= HARD_ENTRY_CAP) do
 		local dropped = table.remove(entries, 1)
@@ -10448,13 +10497,14 @@ local function pushEntry(msg, t, ts)
 	if ts == nil then
 		ts = nowSec()
 	end
-	local bytes = #msg
+	local safeMessage = escapeInvalidUtf8(msg)
+	local bytes = #safeMessage
 	dropOldestUntilFits(bytes)
 	local _arg0 = {
 		seq = nextSeq,
 		ts = ts,
 		level = levelTag(t),
-		message = msg,
+		message = safeMessage,
 	}
 	table.insert(entries, _arg0)
 	nextSeq += 1
@@ -10763,7 +10813,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.22.1"
+local CURRENT_VERSION = "2.22.2"
 local PLUGIN_VARIANT = "main"
 local BASE_PORT = 58741
 local function createConnection(port)

package/studio-plugin/src/modules/Communication.ts

@@ -197,14 +197,27 @@ function processRequest(request: RequestPayload): unknown {
 }
 
 function sendResponse(conn: Connection, requestId: string, responseData: unknown) {
-	pcall(() => {
-		HttpService.RequestAsync({
-			Url: `${conn.serverUrl}/response`,
-			Method: "POST",
-			Headers: { "Content-Type": "application/json" },
-			Body: HttpService.JSONEncode({ requestId, response: responseData }),
+	const responseUrl = `${conn.serverUrl}/response`;
+	const [encodeOk, encoded] = pcall(() => HttpService.JSONEncode({ requestId, response: responseData }));
+	const body = encodeOk
+		? encoded
+		: HttpService.JSONEncode({
+			requestId,
+			error: `Plugin response serialization failed: ${tostring(encoded)}`,
 		});
-	});
+	if (!encodeOk) {
+		warn(`[robloxstudio-mcp] Failed to serialize response ${requestId}: ${tostring(encoded)}`);
+	}
+
+	const [requestOk, requestResult] = pcall(() => HttpService.RequestAsync({
+		Url: responseUrl,
+		Method: "POST",
+		Headers: { "Content-Type": "application/json" },
+		Body: body,
+	}));
+	if (!requestOk || !requestResult.Success) {
+		warn(`[robloxstudio-mcp] Failed to deliver response ${requestId}: ${HttpDiagnostics.formatRequestFailure(responseUrl, requestOk, requestResult)}`);
+	}
 }
 
 function getConnectionStatus(): string {

package/studio-plugin/src/modules/RuntimeLogBuffer.ts

@@ -46,6 +46,37 @@ function nowSec(): number {
 	return DateTime.now().UnixTimestampMillis / 1000;
 }
 
+// Studio occasionally exposes binary-bearing Output messages through
+// LogService (for example, plugin hydration diagnostics containing raw CSG
+// data). HttpService:JSONEncode rejects those strings outright. Preserve all
+// valid UTF-8 verbatim and make only malformed bytes JSON-safe and visible.
+function escapeInvalidUtf8(msg: string): string {
+	const [valid] = utf8.len(msg);
+	// Roblox currently returns nil (not the false declared by @rbxts/types)
+	// when it encounters a malformed sequence. A numeric result is the only
+	// portable success discriminator across both representations.
+	if (typeIs(valid, "number")) return msg;
+
+	const parts: string[] = [];
+	let cursor = 1;
+	while (cursor <= msg.size()) {
+		const [suffixValid, invalidPosition] = utf8.len(msg, cursor);
+		if (typeIs(suffixValid, "number")) {
+			parts.push(string.sub(msg, cursor));
+			break;
+		}
+		if (!typeIs(invalidPosition, "number")) break;
+
+		if (invalidPosition > cursor) {
+			parts.push(string.sub(msg, cursor, invalidPosition - 1));
+		}
+		const [invalidByte] = string.byte(msg, invalidPosition);
+		parts.push(string.format("\\x%02X", invalidByte));
+		cursor = invalidPosition + 1;
+	}
+	return parts.join("");
+}
+
 function dropOldestUntilFits(incomingBytes: number): void {
 	while (
 		entries.size() > 0 &&
@@ -58,13 +89,14 @@ function dropOldestUntilFits(incomingBytes: number): void {
 }
 
 function pushEntry(msg: string, t: Enum.MessageType, ts = nowSec()): void {
-	const bytes = msg.size();
+	const safeMessage = escapeInvalidUtf8(msg);
+	const bytes = safeMessage.size();
 	dropOldestUntilFits(bytes);
 	entries.push({
 		seq: nextSeq,
 		ts,
 		level: levelTag(t),
-		message: msg,
+		message: safeMessage,
 	});
 	nextSeq += 1;
 	totalBytes += bytes;