@chrrxs/robloxstudio-mcp-inspector 2.11.3 → 2.11.4

package/dist/index.js

@@ -1261,7 +1261,28 @@ m.Parent = workspace
 local bridgeOk, inner = bf:Invoke(m)
 m:Destroy()
 if not bridgeOk then
-	return HttpService:JSONEncode({ bridge = "ok", ok = false, error = tostring(inner) })
+	local errMsg = tostring(inner)
+	-- pcall(require, payload) collapses parse/compile failures into the
+	-- canned engine string below. The real parser diagnostic was emitted
+	-- to LogService just before. Walk GetLogHistory backward for the most
+	-- recent ERR entry tagged at our payload path and substitute.
+	if errMsg == "Requested module experienced an error while loading" then
+		-- The parser diagnostic is emitted to LogService on the next
+		-- engine frame, not synchronously with pcall(require). task.wait(0)
+		-- yields too early; 50ms is enough to let the frame complete and
+		-- the message land in GetLogHistory.
+		task.wait(0.05)
+		local LogService = game:GetService("LogService")
+		local hist = LogService:GetLogHistory()
+		for i = #hist, 1, -1 do
+			local e = hist[i]
+			if e.messageType == Enum.MessageType.MessageError and string.sub(e.message, 1, 27) == "Workspace.__MCPEvalPayload:" then
+				errMsg = e.message
+				break
+			end
+		end
+	end
+	return HttpService:JSONEncode({ bridge = "ok", ok = false, error = errMsg })
 end
 -- inner is the {ok, value, output} table from our IIFE. Defensive: if it's
 -- somehow not a table (caller bypassed the wrapper), fall back to old shape.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp-inspector",
-  "version": "2.11.3",
+  "version": "2.11.4",
   "description": "Read-only MCP Server for Roblox Studio (fork of boshyxd/robloxstudio-mcp-inspector with per-peer execute_luau fixes baked in)",
   "main": "dist/index.js",
   "type": "module",
@@ -37,7 +37,7 @@
     "url": "https://github.com/chrrxs/robloxstudio-mcp/issues"
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^0.6.0",
+    "@modelcontextprotocol/sdk": "^1.27.1",
     "cors": "^2.8.5",
     "express": "^4.18.2",
     "node-fetch": "^3.3.2",

package/studio-plugin/MCPInspectorPlugin.rbxmx

@@ -2827,7 +2827,9 @@ return {
             <string name="Name">MetadataHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
 local TS = require(script.Parent.Parent.Parent.include.RuntimeLib)
-local CollectionService = TS.import(script, script.Parent.Parent.Parent, "node_modules", "@rbxts", "services").CollectionService
+local _services = TS.import(script, script.Parent.Parent.Parent, "node_modules", "@rbxts", "services")
+local CollectionService = _services.CollectionService
+local LogService = _services.LogService
 local Utils = TS.import(script, script.Parent.Parent, "Utils")
 local Recording = TS.import(script, script.Parent.Parent, "Recording")
 local ChangeHistoryService = game:GetService("ChangeHistoryService")
@@ -3314,7 +3316,26 @@ end)())`
 		end)
 		m:Destroy()
 		if not okReq then
-			error(tostring(reqResult))
+			local errMsg = tostring(reqResult)
+			-- pcall(require, m) collapses parse/compile failures into the
+			-- canned engine string below. Walk LogService backward for the
+			-- real diagnostic, which was emitted to MessageOut just before.
+			if errMsg == "Requested module experienced an error while loading" then
+				-- The parser diagnostic is emitted to LogService on the next
+				-- engine frame, not synchronously with pcall(require). task.wait(0)
+				-- yields too early; 50ms is enough to let the frame complete and
+				-- the message land in GetLogHistory.
+				task.wait(0.05)
+				local hist = LogService:GetLogHistory()
+				for i = #hist - 1, 0, -1 do
+					local e = hist[i + 1]
+					if e.messageType == Enum.MessageType.MessageError and string.sub(e.message, 1, 31) == "Workspace.__MCPExecLuauPayload:" then
+						errMsg = e.message
+						break
+					end
+				end
+			end
+			error(errMsg)
 		end
 		return reqResult
 	end
@@ -6215,7 +6236,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.11.3"
+local CURRENT_VERSION = "2.11.4"
 local MAX_CONNECTIONS = 5
 local BASE_PORT = 58741
 local activeTabIndex = 0

package/studio-plugin/MCPPlugin.rbxmx

@@ -2827,7 +2827,9 @@ return {
             <string name="Name">MetadataHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
 local TS = require(script.Parent.Parent.Parent.include.RuntimeLib)
-local CollectionService = TS.import(script, script.Parent.Parent.Parent, "node_modules", "@rbxts", "services").CollectionService
+local _services = TS.import(script, script.Parent.Parent.Parent, "node_modules", "@rbxts", "services")
+local CollectionService = _services.CollectionService
+local LogService = _services.LogService
 local Utils = TS.import(script, script.Parent.Parent, "Utils")
 local Recording = TS.import(script, script.Parent.Parent, "Recording")
 local ChangeHistoryService = game:GetService("ChangeHistoryService")
@@ -3314,7 +3316,26 @@ end)())`
 		end)
 		m:Destroy()
 		if not okReq then
-			error(tostring(reqResult))
+			local errMsg = tostring(reqResult)
+			-- pcall(require, m) collapses parse/compile failures into the
+			-- canned engine string below. Walk LogService backward for the
+			-- real diagnostic, which was emitted to MessageOut just before.
+			if errMsg == "Requested module experienced an error while loading" then
+				-- The parser diagnostic is emitted to LogService on the next
+				-- engine frame, not synchronously with pcall(require). task.wait(0)
+				-- yields too early; 50ms is enough to let the frame complete and
+				-- the message land in GetLogHistory.
+				task.wait(0.05)
+				local hist = LogService:GetLogHistory()
+				for i = #hist - 1, 0, -1 do
+					local e = hist[i + 1]
+					if e.messageType == Enum.MessageType.MessageError and string.sub(e.message, 1, 31) == "Workspace.__MCPExecLuauPayload:" then
+						errMsg = e.message
+						break
+					end
+				end
+			end
+			error(errMsg)
 		end
 		return reqResult
 	end
@@ -6215,7 +6236,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.11.3"
+local CURRENT_VERSION = "2.11.4"
 local MAX_CONNECTIONS = 5
 local BASE_PORT = 58741
 local activeTabIndex = 0

package/studio-plugin/src/modules/handlers/MetadataHandlers.ts

@@ -1,4 +1,4 @@
-import { CollectionService } from "@rbxts/services";
+import { CollectionService, LogService } from "@rbxts/services";
 import Utils from "../Utils";
 import Recording from "../Recording";
 
@@ -326,7 +326,31 @@ end)())`;
 		m.Parent = game.GetService("Workspace");
 		const [okReq, reqResult] = pcall(() => require(m));
 		m.Destroy();
-		if (!okReq) error(tostring(reqResult));
+		if (!okReq) {
+			let errMsg = tostring(reqResult);
+			// pcall(require, m) collapses parse/compile failures into the
+			// canned engine string below. Walk LogService backward for the
+			// real diagnostic, which was emitted to MessageOut just before.
+			if (errMsg === "Requested module experienced an error while loading") {
+				// The parser diagnostic is emitted to LogService on the next
+				// engine frame, not synchronously with pcall(require). task.wait(0)
+				// yields too early; 50ms is enough to let the frame complete and
+				// the message land in GetLogHistory.
+				task.wait(0.05);
+				const hist = LogService.GetLogHistory();
+				for (let i = hist.size() - 1; i >= 0; i--) {
+					const e = hist[i];
+					if (
+						e.messageType === Enum.MessageType.MessageError &&
+						string.sub(e.message, 1, 31) === "Workspace.__MCPExecLuauPayload:"
+					) {
+						errMsg = e.message;
+						break;
+					}
+				}
+			}
+			error(errMsg);
+		}
 		return reqResult as unknown as WrapperResult;
 	};