@chrrxs/robloxstudio-mcp-inspector 2.11.1 → 2.11.3

package/dist/index.js

@@ -1217,7 +1217,28 @@ ${s}
 }
 function buildModuleScriptInvokeWrapper(opts) {
   const wrapped = `return ((function()
+	local __mcp_output = {}
+	local __mcp_real_print = print
+	local __mcp_real_warn = warn
+	local print = function(...)
+		__mcp_real_print(...)
+		local args = {...}
+		local parts = table.create(#args)
+		for i, a in ipairs(args) do parts[i] = tostring(a) end
+		table.insert(__mcp_output, table.concat(parts, "\\t"))
+	end
+	local warn = function(...)
+		__mcp_real_warn(...)
+		local args = {...}
+		local parts = table.create(#args)
+		for i, a in ipairs(args) do parts[i] = tostring(a) end
+		table.insert(__mcp_output, "[warn] " .. table.concat(parts, "\\t"))
+	end
+	local function __mcp_run()
 ${opts.userCode}
+	end
+	local ok, errOrValue = xpcall(__mcp_run, debug.traceback)
+	return { ok = ok, value = errOrValue, output = __mcp_output }
 end)())`;
   return `
 local HttpService = game:GetService("HttpService")
@@ -1234,15 +1255,29 @@ m.Name = "__MCPEvalPayload"
 local okSet, setErr = pcall(function() m.Source = USER_CODE end)
 if not okSet then
 	m:Destroy()
-	return HttpService:JSONEncode({ bridge = "ok", ok = false, result = "ModuleScript Source set failed: " .. tostring(setErr) })
+	return HttpService:JSONEncode({ bridge = "ok", ok = false, error = "ModuleScript Source set failed: " .. tostring(setErr) })
 end
 m.Parent = workspace
-local ok, result = bf:Invoke(m)
+local bridgeOk, inner = bf:Invoke(m)
 m:Destroy()
+if not bridgeOk then
+	return HttpService:JSONEncode({ bridge = "ok", ok = false, error = tostring(inner) })
+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.
+if typeof(inner) ~= "table" then
+	return HttpService:JSONEncode({
+		bridge = "ok",
+		ok = true,
+		result = if inner == nil then nil else tostring(inner),
+	})
+end
 return HttpService:JSONEncode({
 	bridge = "ok",
-	ok = ok,
-	result = if result == nil then nil else tostring(result),
+	ok = inner.ok == true,
+	result = if inner.ok and inner.value ~= nil then tostring(inner.value) else nil,
+	error = if not inner.ok then tostring(inner.value) else nil,
+	output = inner.output or {},
 })
 `;
 }
@@ -1928,23 +1963,9 @@ ${code}`
         };
       }
       async stopPlaytest() {
-        let hasProxy = this.bridge.getInstances().some((i) => i.role === "edit-proxy");
-        if (!hasProxy) {
-          const deadline = Date.now() + 1500;
-          while (!hasProxy && Date.now() < deadline) {
-            await new Promise((r) => setTimeout(r, 150));
-            hasProxy = this.bridge.getInstances().some((i) => i.role === "edit-proxy");
-          }
-        }
-        const target = hasProxy ? "edit-proxy" : "edit";
-        const response = await this.client.request("/api/stop-playtest", {}, target);
+        const response = await this.client.request("/api/stop-playtest", {}, "edit");
         return {
-          content: [
-            {
-              type: "text",
-              text: JSON.stringify(response)
-            }
-          ]
+          content: [{ type: "text", text: JSON.stringify(response) }]
         };
       }
       async getPlaytestOutput(target) {
@@ -3087,14 +3108,42 @@ var init_proxy_bridge_service = __esm({
   "../core/dist/proxy-bridge-service.js"() {
     "use strict";
     init_bridge_service();
-    ProxyBridgeService = class extends BridgeService {
+    ProxyBridgeService = class _ProxyBridgeService extends BridgeService {
       primaryBaseUrl;
       proxyInstanceId;
       proxyRequestTimeout = 3e4;
+      cachedInstances = [];
+      refreshTimer;
+      static REFRESH_INTERVAL_MS = 1e3;
       constructor(primaryBaseUrl) {
         super();
         this.primaryBaseUrl = primaryBaseUrl;
         this.proxyInstanceId = uuidv42();
+        this.refreshInstances();
+        this.refreshTimer = setInterval(() => this.refreshInstances(), _ProxyBridgeService.REFRESH_INTERVAL_MS);
+      }
+      async refreshInstances() {
+        try {
+          const res = await fetch(`${this.primaryBaseUrl}/instances`);
+          if (!res.ok)
+            return;
+          const body = await res.json();
+          if (Array.isArray(body.instances)) {
+            this.cachedInstances = body.instances;
+          }
+        } catch {
+        }
+      }
+      getInstances() {
+        return this.cachedInstances;
+      }
+      /** Called when this proxy is being discarded (e.g. promotion to primary
+          replaced it). Stops the background refresh so it doesn't leak. */
+      stop() {
+        if (this.refreshTimer) {
+          clearInterval(this.refreshTimer);
+          this.refreshTimer = void 0;
+        }
       }
       async sendRequest(endpoint, data, target = "edit") {
         const controller = new AbortController();
@@ -3222,8 +3271,12 @@ var init_server = __esm({
             const candidateApp = createHttpServer(candidateTools, candidateBridge, this.allowedToolNames, this.config);
             try {
               const result = await listenWithRetry(candidateApp, host, basePort, 1);
+              const oldBridge = this.bridge;
               this.bridge = candidateBridge;
               this.tools = candidateTools;
+              if (oldBridge instanceof ProxyBridgeService) {
+                oldBridge.stop();
+              }
               httpHandle = result.server;
               boundPort = result.port;
               primaryApp = candidateApp;
@@ -3286,6 +3339,9 @@ var init_server = __esm({
           clearInterval(cleanupInterval);
           if (promotionInterval)
             clearInterval(promotionInterval);
+          if (this.bridge instanceof ProxyBridgeService) {
+            this.bridge.stop();
+          }
           await this.server.close().catch(() => {
           });
           if (httpHandle)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp-inspector",
-  "version": "2.11.1",
+  "version": "2.11.3",
   "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",