@chrrxs/robloxstudio-mcp-inspector 2.9.1 → 2.10.1

package/dist/index.js

@@ -109,10 +109,12 @@ function createHttpServer(tools, bridge, allowedTools, serverConfig) {
       bridge.updateInstanceActivity(instanceId);
     }
     let callerRole = "edit";
+    let knownInstance = false;
     if (instanceId) {
       const inst = bridge.getInstances().find((i) => i.instanceId === instanceId);
       if (inst) {
         callerRole = inst.role;
+        knownInstance = true;
       }
     }
     if (!isMCPServerActive()) {
@@ -120,6 +122,7 @@ function createHttpServer(tools, bridge, allowedTools, serverConfig) {
         error: "MCP server not connected",
         pluginConnected: true,
         mcpConnected: false,
+        knownInstance,
         request: null
       });
       return;
@@ -131,6 +134,7 @@ function createHttpServer(tools, bridge, allowedTools, serverConfig) {
         requestId: pendingRequest.requestId,
         mcpConnected: true,
         pluginConnected: true,
+        knownInstance,
         proxyInstanceCount: proxyInstances.size
       });
     } else {
@@ -138,6 +142,7 @@ function createHttpServer(tools, bridge, allowedTools, serverConfig) {
         request: null,
         mcpConnected: true,
         pluginConnected: true,
+        knownInstance,
         proxyInstanceCount: proxyInstances.size
       });
     }
@@ -340,6 +345,7 @@ var init_http_server = __esm({
       start_playtest: (tools, body) => tools.startPlaytest(body.mode, body.numPlayers),
       stop_playtest: (tools) => tools.stopPlaytest(),
       get_playtest_output: (tools, body) => tools.getPlaytestOutput(body.target),
+      get_runtime_logs: (tools, body) => tools.getRuntimeLogs(body.target, body.since, body.tail, body.filter),
       get_connected_instances: (tools) => tools.getConnectedInstances(),
       export_build: (tools, body) => tools.exportBuild(body.instancePath, body.outputId, body.style),
       create_build: (tools, body) => tools.createBuild(body.id, body.style, body.palette, body.parts, body.bounds),
@@ -1206,6 +1212,37 @@ function luaLongQuote(s) {
 ${s}
 ]${eq}]`;
 }
+function buildModuleScriptInvokeWrapper(opts) {
+  const wrapped = `return ((function()
+${opts.userCode}
+end)())`;
+  return `
+local HttpService = game:GetService("HttpService")
+local bf = game:GetService("${opts.service}"):FindFirstChild("${opts.bridgeName}")
+if not bf then
+	return HttpService:JSONEncode({
+		bridge = "missing",
+		error = ${luaLongQuote(opts.missingError)},
+	})
+end
+local USER_CODE = ${luaLongQuote(wrapped)}
+local m = Instance.new("ModuleScript")
+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) })
+end
+m.Parent = workspace
+local ok, result = bf:Invoke(m)
+m:Destroy()
+return HttpService:JSONEncode({
+	bridge = "ok",
+	ok = ok,
+	result = if result == nil then nil else tostring(result),
+})
+`;
+}
 function parseBridgeResponse(response) {
   const r = response;
   if (r && typeof r.returnValue === "string") {
@@ -1765,23 +1802,12 @@ ${code}`
         if (!code) {
           throw new Error("Code is required for eval_server_runtime");
         }
-        const wrapper = `
-local HttpService = game:GetService("HttpService")
-local bf = game:GetService("ServerScriptService"):FindFirstChild("${SERVER_LOCAL_NAME}")
-if not bf then
-	return HttpService:JSONEncode({
-		bridge = "missing",
-		error = "ServerEvalBridge not installed. Bridges are auto-installed at start_playtest and removed at stop_playtest. Start a playtest before calling eval_server_runtime.",
-	})
-end
-local USER_CODE = ${luaLongQuote(code)}
-local ok, result = bf:Invoke(USER_CODE)
-return HttpService:JSONEncode({
-	bridge = "ok",
-	ok = ok,
-	result = if result == nil then nil else tostring(result),
-})
-`;
+        const wrapper = buildModuleScriptInvokeWrapper({
+          service: "ServerScriptService",
+          bridgeName: SERVER_LOCAL_NAME,
+          missingError: "ServerEvalBridge not installed. Bridges are auto-installed at start_playtest and removed at stop_playtest. Start a playtest before calling eval_server_runtime.",
+          userCode: code
+        });
         const response = await this.client.request("/api/execute-luau", { code: wrapper }, "server");
         return {
           content: [
@@ -1800,32 +1826,12 @@ return HttpService:JSONEncode({
         if (!clientTarget.startsWith("client-")) {
           throw new Error(`eval_client_runtime requires target=client-N (got: ${clientTarget})`);
         }
-        const wrapper = `
-local HttpService = game:GetService("HttpService")
-local bf = game:GetService("ReplicatedStorage"):FindFirstChild("${CLIENT_LOCAL_NAME}")
-if not bf then
-	return HttpService:JSONEncode({
-		bridge = "missing",
-		error = "ClientEvalBridge not installed. Bridges are auto-installed at start_playtest and removed at stop_playtest. Start a playtest before calling eval_client_runtime.",
-	})
-end
-local USER_CODE = ${luaLongQuote(code)}
-local m = Instance.new("ModuleScript")
-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) })
-end
-m.Parent = workspace
-local ok, result = bf:Invoke(m)
-m:Destroy()
-return HttpService:JSONEncode({
-	bridge = "ok",
-	ok = ok,
-	result = if result == nil then nil else tostring(result),
-})
-`;
+        const wrapper = buildModuleScriptInvokeWrapper({
+          service: "ReplicatedStorage",
+          bridgeName: CLIENT_LOCAL_NAME,
+          missingError: "ClientEvalBridge not installed. Bridges are auto-installed at start_playtest and removed at stop_playtest. Start a playtest before calling eval_client_runtime.",
+          userCode: code
+        });
         const response = await this.client.request("/api/execute-luau", { code: wrapper }, clientTarget);
         return {
           content: [
@@ -1836,6 +1842,70 @@ return HttpService:JSONEncode({
           ]
         };
       }
+      async getRuntimeLogs(target, since, tail, filter) {
+        const tgt = target ?? "all";
+        const data = {};
+        if (since !== void 0)
+          data.since = since;
+        if (tail !== void 0)
+          data.tail = tail;
+        if (filter !== void 0)
+          data.filter = filter;
+        if (tgt !== "all") {
+          const response = await this.client.request("/api/get-runtime-logs", data, tgt);
+          return {
+            content: [{ type: "text", text: JSON.stringify(response) }]
+          };
+        }
+        const targets = this.bridge.getInstances().filter((i) => i.role !== "edit-proxy").map((i) => i.role);
+        const responses = await Promise.allSettled(targets.map(async (t) => {
+          const r = await this.client.request("/api/get-runtime-logs", data, t);
+          return { ...r, peer: t };
+        }));
+        const merged = [];
+        const perPeerNextSince = {};
+        const perPeerErrors = {};
+        let totalDropped = 0;
+        for (const r of responses) {
+          if (r.status !== "fulfilled")
+            continue;
+          const v = r.value;
+          const peer = v.peer ?? "unknown";
+          if (v.error) {
+            perPeerErrors[peer] = v.error;
+            continue;
+          }
+          if (v.nextSince !== void 0)
+            perPeerNextSince[peer] = v.nextSince;
+          totalDropped += v.totalDropped ?? 0;
+          for (const e of v.entries ?? []) {
+            merged.push({ ...e, peer });
+          }
+        }
+        merged.sort((a, b) => a.ts !== b.ts ? a.ts - b.ts : a.seq - b.seq);
+        const DEDUP_WINDOW = 2;
+        const deduped = [];
+        for (const e of merged) {
+          const isDup = deduped.some((d) => d.message === e.message && d.level === e.level && Math.abs(d.ts - e.ts) <= DEDUP_WINDOW && d.peer !== e.peer);
+          if (!isDup)
+            deduped.push(e);
+        }
+        let final = deduped;
+        if (tail !== void 0 && deduped.length > tail) {
+          final = deduped.slice(deduped.length - tail);
+        }
+        const body = {
+          entries: final,
+          totalDropped,
+          perPeerNextSince
+        };
+        if (Object.keys(perPeerErrors).length > 0) {
+          body.perPeerErrors = perPeerErrors;
+        }
+        return {
+          content: [{ type: "text", text: JSON.stringify(body) }]
+        };
+      }
       async startPlaytest(mode, numPlayers) {
         if (mode !== "play" && mode !== "run") {
           throw new Error('mode must be "play" or "run"');
@@ -2740,13 +2810,20 @@ var init_bridge_service = __esm({
     BridgeService = class {
       pendingRequests = /* @__PURE__ */ new Map();
       instances = /* @__PURE__ */ new Map();
-      nextClientIndex = 1;
       requestTimeout = 3e4;
       registerInstance(instanceId, role) {
         let assignedRole = role;
         if (role === "client") {
-          assignedRole = `client-${this.nextClientIndex}`;
-          this.nextClientIndex++;
+          const used = /* @__PURE__ */ new Set();
+          for (const inst of this.instances.values()) {
+            const match = inst.role.match(/^client-(\d+)$/);
+            if (match)
+              used.add(Number(match[1]));
+          }
+          let idx = 1;
+          while (used.has(idx))
+            idx++;
+          assignedRole = `client-${idx}`;
         }
         this.instances.set(instanceId, {
           instanceId,
@@ -3832,7 +3909,7 @@ var init_definitions = __esm({
       {
         name: "eval_server_runtime",
         category: "write",
-        description: "Execute Luau on the server peer in the running game's Script VM (shares require cache with user game scripts). Use this instead of execute_luau target=server when you need to see runtime-mutated module state. Auto-installed at start_playtest, removed at stop_playtest. Requires ServerScriptService.LoadStringEnabled=true.",
+        description: "Execute Luau on the server peer in the running game's Script VM (shares require cache with user game scripts). Use this instead of execute_luau target=server when you need to see runtime-mutated module state. Auto-installed at start_playtest, removed at stop_playtest.",
         inputSchema: {
           type: "object",
           properties: {
@@ -3847,7 +3924,7 @@ var init_definitions = __esm({
       {
         name: "eval_client_runtime",
         category: "write",
-        description: "Execute Luau on a client peer in the running game's LocalScript VM (shares require cache with user game scripts). Use this instead of execute_luau target=client-N when you need to see runtime-mutated module state. Auto-installed at start_playtest, removed at stop_playtest. Does not require LoadStringEnabled.",
+        description: "Execute Luau on a client peer in the running game's LocalScript VM (shares require cache with user game scripts). Use this instead of execute_luau target=client-N when you need to see runtime-mutated module state. Auto-installed at start_playtest, removed at stop_playtest.",
         inputSchema: {
           type: "object",
           properties: {
@@ -3956,6 +4033,32 @@ var init_definitions = __esm({
           }
         }
       },
+      {
+        name: "get_runtime_logs",
+        category: "read",
+        description: "Read the in-memory log buffer captured by the plugin on each peer's LogService.MessageOut. Each peer (edit, server, client-N) captures ~64 KB of recent prints; oldest entries drop when over budget. Drop-oldest semantics preserve the recent tail, unlike get_console_output's 10 KB drop-newest cap. Caveat: peer tag reflects which peer's plugin captured the entry, not which peer's script originated it - LogService reflects prints across peers in Studio Play and origin is undetectable from inside MessageOut. target=all (default) merges all peers and dedups same-message-and-level entries captured within 2s across different peers.",
+        inputSchema: {
+          type: "object",
+          properties: {
+            target: {
+              type: "string",
+              description: 'Peer to read from: "edit", "server", "client-N", or "all" (default). "all" merges all peers and dedups cross-peer reflections within a 2s window.'
+            },
+            since: {
+              type: "number",
+              description: "Return only entries with seq > since. Pass back the previous response's nextSince (single-peer) or perPeerNextSince entry (target=all) for incremental polling."
+            },
+            tail: {
+              type: "number",
+              description: "Return only the last N entries after since/filter is applied."
+            },
+            filter: {
+              type: "string",
+              description: "Plain substring matched against each entry's message (no pattern semantics; literal text). Applied after since, before tail."
+            }
+          }
+        }
+      },
       // === Multi-Instance ===
       {
         name: "get_connected_instances",

package/package.json

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