@chrrxs/robloxstudio-mcp 2.15.0 → 2.15.2

package/studio-plugin/MCPInspectorPlugin.rbxmx

@@ -105,7 +105,9 @@ local MemoryHandlers = TS.import(script, script.Parent, "handlers", "MemoryHandl
 local SceneAnalysisHandlers = TS.import(script, script.Parent, "handlers", "SceneAnalysisHandlers")
 local CaptureHandlers = TS.import(script, script.Parent, "handlers", "CaptureHandlers")
 local InputHandlers = TS.import(script, script.Parent, "handlers", "InputHandlers")
+local EvalRuntimeHandlers = TS.import(script, script.Parent, "handlers", "EvalRuntimeHandlers")
 local LuauExec = TS.import(script, script.Parent, "LuauExec")
+local State = TS.import(script, script.Parent, "State")
 local StudioTestService = game:GetService("StudioTestService")
 -- Mirror of Communication.computeInstanceId() — duplicated here because the
 -- client broker runs in the play-server DM where it can't easily import from
@@ -166,6 +168,7 @@ local BROKER_OWNER_ATTRIBUTE = "__MCPBrokerOwner"
 -- cache / etc. lives there) so the server peer alone can't satisfy them.
 local CLIENT_BROKER_ALLOWED_ENDPOINTS = {
 	["/api/execute-luau"] = true,
+	["/api/eval-runtime"] = true,
 	["/api/get-runtime-logs"] = true,
 	["/api/get-memory-breakdown"] = true,
 	["/api/get-scene-analysis"] = true,
@@ -201,6 +204,8 @@ local function reRegisterProxy(proxyId, role)
 			placeName = resolvePlaceName(),
 			dataModelName = game.Name,
 			isRunning = RunService:IsRunning(),
+			pluginVersion = State.CURRENT_VERSION,
+			pluginVariant = State.PLUGIN_VARIANT,
 		})
 	end)
 end
@@ -244,8 +249,8 @@ local function handleGetRuntimeLogs(data)
 	local since = d.since
 	local tail = d.tail
 	local filter = d.filter
-	-- "client" is the generic peer tag; MCP-side aggregator overrides with
-	-- the specific role (e.g. "client-1") on target=all fan-out.
+	-- "client" is the generic capture tag; MCP-side aggregation overrides it
+	-- with the specific role (e.g. "client-1") for capturedBy.
 	return RuntimeLogBuffer.query({
 		since = since,
 		tail = tail,
@@ -363,6 +368,9 @@ local function setupClientBroker()
 		if payload and payload.endpoint == "/api/execute-luau" then
 			return handleExecuteLuau(payload.data)
 		end
+		if payload and payload.endpoint == "/api/eval-runtime" then
+			return EvalRuntimeHandlers.evalRuntime(payload.data or {})
+		end
 		-- Legacy: raw execute-luau payload at the top level.
 		return handleExecuteLuau(payload)
 	end
@@ -456,6 +464,8 @@ local function registerProxy(player, rf)
 		placeName = resolvePlaceName(),
 		dataModelName = game.Name,
 		isRunning = RunService:IsRunning(),
+		pluginVersion = State.CURRENT_VERSION,
+		pluginVariant = State.PLUGIN_VARIANT,
 	})
 	if not ok or not res or not res.Success then
 		warn(`[robloxstudio-mcp] proxy register failed for {player.Name}`)
@@ -557,6 +567,7 @@ local LogHandlers = TS.import(script, script.Parent, "handlers", "LogHandlers")
 local SerializationHandlers = TS.import(script, script.Parent, "handlers", "SerializationHandlers")
 local MemoryHandlers = TS.import(script, script.Parent, "handlers", "MemoryHandlers")
 local SceneAnalysisHandlers = TS.import(script, script.Parent, "handlers", "SceneAnalysisHandlers")
+local EvalRuntimeHandlers = TS.import(script, script.Parent, "handlers", "EvalRuntimeHandlers")
 -- Per-plugin-load random GUID. Used as the /poll URL param so the server
 -- can tell our polls apart from any other plugin's polls. Not user-facing —
 -- MCP tools and the LLM operate on instanceId (the place identifier).
@@ -584,6 +595,8 @@ end
 local instanceId = computeInstanceId()
 local assignedRole
 local duplicateInstanceRole = false
+local hasVersionMismatch = false
+local lastVersionMismatchWarningKey
 -- Cache the published place name from MarketplaceService:GetProductInfo so
 -- /ready can carry a friendly identifier (e.g. "Natural Disasters") distinct
 -- from game.Name (the DataModel name, often "Place1" in edit). We only fetch
@@ -661,6 +674,7 @@ local routeMap = {
 	["/api/get-tagged"] = MetadataHandlers.getTagged,
 	["/api/get-selection"] = MetadataHandlers.getSelection,
 	["/api/execute-luau"] = MetadataHandlers.executeLuau,
+	["/api/eval-runtime"] = EvalRuntimeHandlers.evalRuntime,
 	["/api/undo"] = MetadataHandlers.undo,
 	["/api/redo"] = MetadataHandlers.redo,
 	["/api/bulk-set-attributes"] = MetadataHandlers.bulkSetAttributes,
@@ -783,6 +797,8 @@ function sendReady(conn)
 					placeName = resolvePlaceName(),
 					dataModelName = game.Name,
 					isRunning = RunService:IsRunning(),
+					pluginVersion = State.CURRENT_VERSION,
+					pluginVariant = State.PLUGIN_VARIANT,
 					pluginReady = true,
 					timestamp = tick(),
 				}),
@@ -848,6 +864,23 @@ local function pollForRequests(connIndex)
 		local mcpConnected = data.mcpConnected == true
 		conn.lastHttpOk = true
 		conn.lastMcpOk = mcpConnected
+		local _condition = data.serverVersion
+		if _condition == nil then
+			_condition = "unknown"
+		end
+		local serverVersion = _condition
+		if data.versionMismatch == true then
+			hasVersionMismatch = true
+			local warningKey = `{State.CURRENT_VERSION}:{serverVersion}`
+			if lastVersionMismatchWarningKey ~= warningKey then
+				lastVersionMismatchWarningKey = warningKey
+				warn(`[MCPPlugin] Version mismatch: Studio plugin v{State.CURRENT_VERSION} / MCP v{serverVersion}. Run npx -y @chrrxs/robloxstudio-mcp@latest --auto-install-plugin and restart Studio.`)
+			end
+			UI.showBanner("version-mismatch", `Plugin v{State.CURRENT_VERSION} / MCP v{serverVersion} mismatch`)
+		elseif hasVersionMismatch then
+			hasVersionMismatch = false
+			UI.hideBanner("version-mismatch")
+		end
 		-- Server tells us when its in-memory instances map doesn't have us
 		-- (e.g. after an MCP process restart). Re-issue /ready immediately so
 		-- target=server/client-N start routing again. The throttle inside
@@ -860,12 +893,12 @@ local function pollForRequests(connIndex)
 			local el = ui
 			el.step1Dot.BackgroundColor3 = Color3.fromRGB(34, 197, 94)
 			el.step1Label.Text = "HTTP server (OK)"
-			local _condition = mcpConnected
-			if _condition then
+			local _condition_1 = mcpConnected
+			if _condition_1 then
 				local _value = (string.find(el.statusLabel.Text, "Connected"))
-				_condition = not (_value ~= 0 and _value == _value and _value)
+				_condition_1 = not (_value ~= 0 and _value == _value and _value)
 			end
-			if _condition then
+			if _condition_1 then
 				el.statusLabel.Text = "Connected"
 				el.statusLabel.TextColor3 = Color3.fromRGB(34, 197, 94)
 				el.statusIndicator.BackgroundColor3 = Color3.fromRGB(34, 197, 94)
@@ -896,11 +929,11 @@ local function pollForRequests(connIndex)
 					conn.mcpWaitStartTime = tick()
 				end
 				local _exp = tick()
-				local _condition_1 = conn.mcpWaitStartTime
-				if _condition_1 == nil then
-					_condition_1 = tick()
+				local _condition_2 = conn.mcpWaitStartTime
+				if _condition_2 == nil then
+					_condition_2 = tick()
 				end
-				local elapsed = _exp - _condition_1
+				local elapsed = _exp - _condition_2
 				el.troubleshootLabel.Visible = elapsed > 8
 				UI.startPulseAnimation()
 			end
@@ -1109,11 +1142,9 @@ local function checkForUpdates()
 			if _condition ~= "" and _condition then
 				local latestVersion = data.version
 				if Utils.compareVersions(State.CURRENT_VERSION, latestVersion) < 0 then
-					local ui = UI.getElements()
-					ui.updateBannerText.Text = `v{latestVersion} available - github.com/chrrxs/robloxstudio-mcp`
-					ui.updateBanner.Visible = true
-					ui.contentFrame.Position = UDim2.new(0, 8, 0, 92)
-					ui.contentFrame.Size = UDim2.new(1, -16, 1, -100)
+					if not hasVersionMismatch then
+						UI.showBanner("update", `v{latestVersion} available - github.com/chrrxs/robloxstudio-mcp`)
+					end
 				end
 			end
 		end
@@ -1218,9 +1249,10 @@ bf.Archivable = false\
 bf.Parent = ServerScriptService\
 bf.OnInvoke = function(payload)\
 	if typeof(payload) ~= "Instance" or not payload:IsA("ModuleScript") then\
-		return false, "payload must be a ModuleScript instance"\
+		return \{ ok = false, value = "payload must be a ModuleScript instance" \}\
 	end\
-	return pcall(require, payload)\
+	local ok, value = pcall(require, payload)\
+	return \{ ok = ok, value = value \}\
 end\
 `
 local CLIENT_BRIDGE_SOURCE = `\
@@ -1244,9 +1276,10 @@ bf.Archivable = false\
 bf.Parent = ReplicatedStorage\
 bf.OnInvoke = function(payload)\
 	if typeof(payload) ~= "Instance" or not payload:IsA("ModuleScript") then\
-		return false, "payload must be a ModuleScript instance"\
+		return \{ ok = false, value = "payload must be a ModuleScript instance" \}\
 	end\
-	return pcall(require, payload)\
+	local ok, value = pcall(require, payload)\
+	return \{ ok = ok, value = value \}\
 end\
 `
 -- Stamp written onto each installed bridge Script so we can tell whether the
@@ -1263,9 +1296,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.15.0" is replaced with the package version at package time
+	-- "2.15.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.15.0`
+	return `{tostring(h)}-2.15.2`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -2442,6 +2475,132 @@ return {
           </Properties>
         </Item>
         <Item class="ModuleScript" referent="9">
+          <Properties>
+            <string name="Name">EvalRuntimeHandlers</string>
+            <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
+local TS = require(script.Parent.Parent.Parent.include.RuntimeLib)
+local _services = TS.import(script, script.Parent.Parent.Parent, "node_modules", "@rbxts", "services")
+local LogService = _services.LogService
+local ReplicatedStorage = _services.ReplicatedStorage
+local RunService = _services.RunService
+local ServerScriptService = _services.ServerScriptService
+local BRIDGE_NAMES = TS.import(script, script.Parent.Parent, "EvalBridges").BRIDGE_NAMES
+local LuauExec = TS.import(script, script.Parent.Parent, "LuauExec")
+local PAYLOAD_INSTANCE_NAME = "__MCPEvalPayload"
+local function getBridgeConfig()
+	if not RunService:IsRunning() then
+		return {
+			error = "eval_*_runtime requires a running playtest.",
+		}
+	end
+	if RunService:IsServer() then
+		return {
+			service = ServerScriptService,
+			bridgeName = BRIDGE_NAMES.serverLocal,
+			missingError = "ServerEvalBridge not found. The bridge runs inside the play DM, so a playtest must be running. The bridge installs automatically (including for manually-started playtests); if a playtest is running and you still see this, reconnect the plugin in the edit window so the bridge reinstalls, then start the playtest again.",
+		}
+	end
+	return {
+		service = ReplicatedStorage,
+		bridgeName = BRIDGE_NAMES.clientLocal,
+		missingError = "ClientEvalBridge not found. The bridge runs inside the play DM, so a playtest must be running. The bridge installs automatically (including for manually-started playtests); if a playtest is running and you still see this, reconnect the plugin in the edit window so the bridge reinstalls, then start the playtest again.",
+	}
+end
+local function evalRuntime(requestData)
+	local code = requestData.code
+	if not (code ~= "" and code) or code == "" then
+		return {
+			error = "Code is required",
+		}
+	end
+	local config = getBridgeConfig()
+	if config.error ~= nil then
+		return {
+			bridge = "missing",
+			error = config.error,
+		}
+	end
+	local bridge = config.service:FindFirstChild(config.bridgeName)
+	if not bridge or not bridge:IsA("BindableFunction") then
+		return {
+			bridge = "missing",
+			error = config.missingError,
+		}
+	end
+	local m = Instance.new("ModuleScript")
+	m.Name = PAYLOAD_INSTANCE_NAME
+	local userLines = LuauExec.countLines(code)
+	local wrapped = LuauExec.buildWrapper(code, PAYLOAD_INSTANCE_NAME)
+	local okSet, setErr = pcall(function()
+		m.Source = wrapped
+	end)
+	if not okSet then
+		m:Destroy()
+		return {
+			bridge = "ok",
+			ok = false,
+			error = `ModuleScript Source set failed: {tostring(setErr)}`,
+		}
+	end
+	m.Parent = game:GetService("Workspace")
+	local historyStart = #LogService:GetLogHistory()
+	local invokeOk, invokeResult = pcall(function()
+		return bridge:Invoke(m)
+	end)
+	m:Destroy()
+	if not invokeOk then
+		return {
+			bridge = "ok",
+			ok = false,
+			error = tostring(invokeResult),
+		}
+	end
+	if not (type(invokeResult) == "table") then
+		return {
+			bridge = "ok",
+			ok = false,
+			error = `Eval bridge returned invalid result: {tostring(invokeResult)}`,
+		}
+	end
+	local bridgeResult = invokeResult
+	if bridgeResult.ok ~= true then
+		return {
+			bridge = "ok",
+			ok = false,
+			error = LuauExec.recoverPayloadRequireError(bridgeResult.value, userLines, PAYLOAD_INSTANCE_NAME, historyStart),
+		}
+	end
+	local inner = bridgeResult.value
+	if not (type(inner) == "table") then
+		return {
+			bridge = "ok",
+			ok = true,
+			result = if inner == nil then nil else LuauExec.formatReturnValue(inner),
+		}
+	end
+	local r = inner
+	local ok = r.ok == true
+	local _object = {
+		bridge = "ok",
+		ok = ok,
+		result = if ok and r.value ~= nil then LuauExec.formatReturnValue(r.value) else nil,
+		error = if not ok then tostring(r.value) else nil,
+	}
+	local _left = "output"
+	local _condition = r.output
+	if _condition == nil then
+		_condition = {}
+	end
+	_object[_left] = _condition
+	return _object
+end
+return {
+	evalRuntime = evalRuntime,
+}
+]]></string>
+          </Properties>
+        </Item>
+        <Item class="ModuleScript" referent="10">
           <Properties>
             <string name="Name">InputHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -2641,7 +2800,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="10">
+        <Item class="ModuleScript" referent="11">
           <Properties>
             <string name="Name">InstanceHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -3148,7 +3307,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="11">
+        <Item class="ModuleScript" referent="12">
           <Properties>
             <string name="Name">LogHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -3158,16 +3317,15 @@ local function getRuntimeLogs(requestData)
 	local since = requestData.since
 	local tail = requestData.tail
 	local filter = requestData.filter
-	-- Plugin-side peer tag is generic ("edit"|"server"|"client"). The MCP-side
-	-- aggregator overrides it with the specific instance role (e.g. "client-1")
-	-- during fan-out for target=all, so this value is only authoritative for
-	-- the single-peer query path.
-	local peer = RuntimeLogBuffer.detectPeer()
+	-- This is the buffer that captured the LogService event, not necessarily
+	-- the script-origin peer. Ordinary playtests share/reflect logs across
+	-- edit/server/client LogService buffers.
+	local capturedBy = RuntimeLogBuffer.detectPeer()
 	return RuntimeLogBuffer.query({
 		since = since,
 		tail = tail,
 		filter = filter,
-	}, peer)
+	}, capturedBy)
 end
 return {
 	getRuntimeLogs = getRuntimeLogs,
@@ -3175,7 +3333,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="12">
+        <Item class="ModuleScript" referent="13">
           <Properties>
             <string name="Name">MemoryHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -3243,7 +3401,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="13">
+        <Item class="ModuleScript" referent="14">
           <Properties>
             <string name="Name">MetadataHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -3782,7 +3940,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="14">
+        <Item class="ModuleScript" referent="15">
           <Properties>
             <string name="Name">PropertyHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -4034,7 +4192,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="15">
+        <Item class="ModuleScript" referent="16">
           <Properties>
             <string name="Name">QueryHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -5076,7 +5234,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="16">
+        <Item class="ModuleScript" referent="17">
           <Properties>
             <string name="Name">SceneAnalysisHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -5325,7 +5483,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="17">
+        <Item class="ModuleScript" referent="18">
           <Properties>
             <string name="Name">ScriptHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -6021,7 +6179,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="18">
+        <Item class="ModuleScript" referent="19">
           <Properties>
             <string name="Name">SerializationHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -6207,7 +6365,7 @@ return {
 ]]></string>
           </Properties>
         </Item>
-        <Item class="ModuleScript" referent="19">
+        <Item class="ModuleScript" referent="20">
           <Properties>
             <string name="Name">TestHandlers</string>
             <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -6840,7 +6998,7 @@ return {
           </Properties>
         </Item>
       </Item>
-      <Item class="ModuleScript" referent="20">
+      <Item class="ModuleScript" referent="21">
         <Properties>
           <string name="Name">LuauExec</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -6849,10 +7007,12 @@ return {
 -- and the play-client peer (ClientBroker.handleExecuteLuau). Three things this
 -- module owns:
 --
---   1. The IIFE wrapper that captures print/warn, runs user code in xpcall,
---      and always returns { ok, value, output } so the ModuleScript itself
---      always returns exactly one value (otherwise `print("hi")` with no
---      return would fail with "Module code did not return exactly one value").
+--   1. The IIFE wrapper that captures print/warn, wraps require() so nested
+--      ModuleScript load failures can recover the real LogService diagnostic,
+--      runs user code in xpcall, and always returns { ok, value, output } so
+--      the ModuleScript itself always returns exactly one value (otherwise
+--      `print("hi")` with no return would fail with "Module code did not
+--      return exactly one value").
 --
 --   2. The loadstring-then-ModuleScript-require fallback, with the parse-error
 --      recovery hack that pulls the real diagnostic from LogService.
@@ -6870,14 +7030,14 @@ return {
 local HttpService = game:GetService("HttpService")
 local LogService = game:GetService("LogService")
 local PAYLOAD_INSTANCE_NAME = "__MCPExecLuauPayload"
-local PAYLOAD_PATH_PREFIX = `Workspace.{PAYLOAD_INSTANCE_NAME}:`
+local REQUIRE_GENERIC_ERROR = "Requested module experienced an error while loading"
 -- Number of lines the wrapper emits BEFORE the first line of user code.
 -- Used both inside the wrapper (Luau __mcp_LINE_OFFSET) and on the TS side
 -- (remapPayloadLines, for compile errors recovered from LogService) so user
 -- code errors report user-relative line numbers instead of the inflated
--- "line 23" the wrapper would otherwise expose. If you reorder buildWrapper's
--- prefix lines, update this constant — there's a self-check below.
-local WRAPPER_LINE_OFFSET = 23
+-- "line 49" the wrapper would otherwise expose. If you reorder buildWrapper's
+-- prefix lines, update this constant.
+local WRAPPER_LINE_OFFSET = 84
 -- Count source lines so the wrapper can filter traceback frames that fall
 -- outside the user code range (the wrapper's own preamble/postamble lines).
 local function countLines(s)
@@ -6902,20 +7062,31 @@ local function countLines(s)
 	end
 	return n
 end
-local function buildWrapper(code)
+local function luaPatternEscape(s)
+	local escaped = string.gsub(s, "([^%w])", "%%%1")
+	return escaped
+end
+local function buildWrapper(code, payloadInstanceName)
+	if payloadInstanceName == nil then
+		payloadInstanceName = PAYLOAD_INSTANCE_NAME
+	end
 	-- If you reorder the prefix lines below, update WRAPPER_LINE_OFFSET to
 	-- match the number of lines emitted BEFORE the ${code} substitution.
 	-- The constant is mirrored inside the wrapper (__mcp_LINE_OFFSET) and
 	-- used by remapPayloadLines on the TS side.
 	local userLines = countLines(code)
+	local payloadPattern = luaPatternEscape(payloadInstanceName)
 	return `return ((function()\
 \tlocal __mcp_traceback\
 \tlocal __mcp_remap\
 \tlocal __mcp_LINE_OFFSET = {WRAPPER_LINE_OFFSET}\
 \tlocal __mcp_USER_LINES = {userLines}\
+\tlocal __mcp_LogService = game:GetService("LogService")\
+\tlocal __mcp_REQUIRE_GENERIC = "{REQUIRE_GENERIC_ERROR}"\
 \tlocal __mcp_output = \{\}\
 \tlocal __mcp_real_print = print\
 \tlocal __mcp_real_warn = warn\
+\tlocal __mcp_real_require = require\
 \tlocal print = function(...)\
 \t\t__mcp_real_print(...)\
 \t\tlocal args = \{...\}\
@@ -6930,6 +7101,64 @@ local function buildWrapper(code)
 \t\tfor i, a in ipairs(args) do parts[i] = tostring(a) end\
 \t\ttable.insert(__mcp_output, "[warn] " .. table.concat(parts, "\\t"))\
 \tend\
+\tlocal function __mcp_is_stack_noise(msg)\
+\t\treturn msg == "Stack Begin" or msg == "Stack End" or string.sub(msg, 1, 8) == "Script '"\
+\tend\
+\tlocal function __mcp_is_actionable_require_log(entry)\
+\t\tif not entry or entry.messageType ~= Enum.MessageType.MessageError then return false end\
+\t\tlocal msg = tostring(entry.message)\
+\t\treturn msg ~= __mcp_REQUIRE_GENERIC and not __mcp_is_stack_noise(msg)\
+\tend\
+\tlocal function __mcp_entry_mentions_module(entry, module_path)\
+\t\tif not entry or not module_path or module_path == "" then return false end\
+\t\treturn string.find(tostring(entry.message), module_path, 1, true) ~= nil\
+\tend\
+\tlocal function __mcp_prior_module_error(hist, module_path)\
+\t\tif not module_path or module_path == "" then return nil end\
+\t\tfor i = #hist, 1, -1 do\
+\t\t\tlocal entry = hist[i]\
+\t\t\tif __mcp_entry_mentions_module(entry, module_path) then\
+\t\t\t\tif __mcp_is_actionable_require_log(entry) then\
+\t\t\t\t\treturn tostring(entry.message)\
+\t\t\t\tend\
+\t\t\t\tfor j = i - 1, math.max(1, i - 6), -1 do\
+\t\t\t\t\tlocal previous = hist[j]\
+\t\t\t\t\tif __mcp_is_actionable_require_log(previous) then\
+\t\t\t\t\t\treturn tostring(previous.message)\
+\t\t\t\t\tend\
+\t\t\t\tend\
+\t\t\tend\
+\t\tend\
+\t\treturn nil\
+\tend\
+\tlocal function __mcp_recover_require_error(err, history_start, module)\
+\t\tlocal err_msg = tostring(err)\
+\t\tif err_msg ~= __mcp_REQUIRE_GENERIC then return err_msg end\
+\t\tlocal module_path\
+\t\tif typeof(module) == "Instance" then\
+\t\t\tlocal ok_path, path = pcall(function()\
+\t\t\t\treturn module:GetFullName()\
+\t\t\tend)\
+\t\t\tif ok_path then module_path = path end\
+\t\tend\
+\t\ttask.wait(0.05)\
+\t\tlocal hist = __mcp_LogService:GetLogHistory()\
+\t\tfor i = #hist, history_start + 1, -1 do\
+\t\t\tlocal entry = hist[i]\
+\t\t\tif __mcp_is_actionable_require_log(entry) then\
+\t\t\t\treturn tostring(entry.message)\
+\t\t\tend\
+\t\tend\
+\t\tlocal prior = __mcp_prior_module_error(hist, module_path)\
+\t\tif prior then return prior end\
+\t\treturn err_msg\
+\tend\
+\tlocal function require(module)\
+\t\tlocal history_start = #__mcp_LogService:GetLogHistory()\
+\t\tlocal ok, value = pcall(__mcp_real_require, module)\
+\t\tif ok then return value end\
+\t\terror(__mcp_recover_require_error(value, history_start, module), 0)\
+\tend\
 \tlocal function __mcp_run()\
 {code}\
 \tend\
@@ -6940,15 +7169,20 @@ local function buildWrapper(code)
 \t\t-- Subtract LINE_OFFSET to get the user-relative number, then clamp.\
 \t\t-- Clamping matters for unclosed constructs ("local x = (") where the\
 \t\t-- parser keeps reading into wrapper postamble and reports a payload\
-\t\t-- line past user EOF. Without clamping the message says "user_code:49"\
-\t\t-- for one-line input, framing the wrapper as user code.\
+\t\t-- line past user EOF. Without clamping, that frames wrapper postamble\
+\t\t-- as user code.\
 \t\tlocal function __mcp_user_line(payload_n)\
 \t\t\tlocal user_n = payload_n - __mcp_LINE_OFFSET\
 \t\t\tif user_n < 1 then return "1" end\
 \t\t\tif user_n > __mcp_USER_LINES then return tostring(__mcp_USER_LINES) .. " (at end of input)" end\
 \t\t\treturn tostring(user_n)\
 \t\tend\
-\t\ts = string.gsub(s, "__MCPExecLuauPayload:(%d+)", function(num)\
+\t\ts = string.gsub(s, "Workspace%.{payloadPattern}:(%d+)", function(num)\
+\t\t\tlocal n = tonumber(num)\
+\t\t\tif n then return "user_code:" .. __mcp_user_line(n) end\
+\t\t\treturn "user_code:" .. num\
+\t\tend)\
+\t\ts = string.gsub(s, "{payloadPattern}:(%d+)", function(num)\
 \t\t\tlocal n = tonumber(num)\
 \t\t\tif n then return "user_code:" .. __mcp_user_line(n) end\
 \t\t\treturn "user_code:" .. num\
@@ -6998,7 +7232,10 @@ end
 -- pulling the real compile-error diagnostic out of LogService — that error
 -- references the payload module's line number directly, and never passes
 -- through the IIFE's runtime wrapper.
-local function remapPayloadLines(s, userLines)
+local function remapPayloadLines(s, userLines, payloadInstanceName)
+	if payloadInstanceName == nil then
+		payloadInstanceName = PAYLOAD_INSTANCE_NAME
+	end
 	-- Mirror of the Lua __mcp_remap inside the wrapper, for paths that
 	-- don't pass through the IIFE (compile errors recovered from
 	-- LogService, the immediate loadstring compileError surface). Same
@@ -7016,8 +7253,9 @@ local function remapPayloadLines(s, userLines)
 		end
 		return tostring(u)
 	end
+	local payloadPattern = luaPatternEscape(payloadInstanceName)
 	local out = s
-	local a = string.gsub(out, "__MCPExecLuauPayload:(%d+)", function(num)
+	local a = string.gsub(out, `Workspace%.{payloadPattern}:(%d+)`, function(num)
 		local n = tonumber(num)
 		if n ~= nil then
 			return `user_code:{userLine(n)}`
@@ -7025,7 +7263,7 @@ local function remapPayloadLines(s, userLines)
 		return `user_code:{num}`
 	end)
 	out = a
-	local b = string.gsub(out, '%[string "[^"]+"%]:(%d+)', function(num)
+	local b = string.gsub(out, `{payloadPattern}:(%d+)`, function(num)
 		local n = tonumber(num)
 		if n ~= nil then
 			return `user_code:{userLine(n)}`
@@ -7033,8 +7271,16 @@ local function remapPayloadLines(s, userLines)
 		return `user_code:{num}`
 	end)
 	out = b
-	return out
+	local c = string.gsub(out, '%[string "[^"]+"%]:(%d+)', function(num)
+		local n = tonumber(num)
+		if n ~= nil then
+			return `user_code:{userLine(n)}`
+		end
+		return `user_code:{num}`
+	end)
+	return c
 end
+local recoverPayloadRequireError
 local function runViaModuleScript(wrapped, userLines)
 	local m = Instance.new("ModuleScript")
 	m.Name = PAYLOAD_INSTANCE_NAME
@@ -7054,26 +7300,11 @@ local function runViaModuleScript(wrapped, userLines)
 	end)
 	m:Destroy()
 	if not okReq then
-		local errMsg = tostring(reqResult)
-		-- pcall(require, m) collapses parse/compile failures into the canned
-		-- engine string. The real diagnostic was emitted to LogService on the
-		-- next engine frame — give it ~50ms to land then scan backward.
-		if errMsg == "Requested module experienced an error while loading" then
-			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, #PAYLOAD_PATH_PREFIX) == PAYLOAD_PATH_PREFIX then
-					errMsg = e.message
-					break
-				end
-			end
-		end
 		-- Compile errors reference the payload module's line number directly
 		-- — remap + clamp to user-relative line numbers so `local x = 1 +`
 		-- reports :1: instead of :23:, and reports the clamp annotation
 		-- when the parser ran off the end of user code into wrapper code.
-		error(remapPayloadLines(errMsg, userLines), 0)
+		error(recoverPayloadRequireError(reqResult, userLines, PAYLOAD_INSTANCE_NAME), 0)
 	end
 	return reqResult
 end
@@ -7100,6 +7331,44 @@ local function formatReturnValue(value)
 	end
 	return tostring(value)
 end
+function recoverPayloadRequireError(err, userLines, payloadInstanceName, historyStart)
+	if payloadInstanceName == nil then
+		payloadInstanceName = PAYLOAD_INSTANCE_NAME
+	end
+	if historyStart == nil then
+		historyStart = 0
+	end
+	local errMsg = tostring(err)
+	-- pcall(require, m) collapses parse/compile failures into the canned
+	-- engine string. The real diagnostic is emitted to LogService on the
+	-- next engine frame — give it ~50ms to land then scan backward.
+	if errMsg == REQUIRE_GENERIC_ERROR then
+		task.wait(0.05)
+		local payloadPathPrefix = `Workspace.{payloadInstanceName}:`
+		local hist = LogService:GetLogHistory()
+		local start = math.max(0, historyStart)
+		do
+			local i = #hist - 1
+			local _shouldIncrement = false
+			while true do
+				if _shouldIncrement then
+					i -= 1
+				else
+					_shouldIncrement = true
+				end
+				if not (i >= start) then
+					break
+				end
+				local e = hist[i + 1]
+				if e.messageType == Enum.MessageType.MessageError and string.sub(e.message, 1, #payloadPathPrefix) == payloadPathPrefix then
+					errMsg = e.message
+					break
+				end
+			end
+		end
+	end
+	return remapPayloadLines(errMsg, userLines, payloadInstanceName)
+end
 local function execute(code)
 	if not (code ~= "" and code) or code == "" then
 		return {
@@ -7153,12 +7422,17 @@ local function execute(code)
 	}
 end
 return {
+	buildWrapper = buildWrapper,
+	countLines = countLines,
 	execute = execute,
+	formatReturnValue = formatReturnValue,
+	recoverPayloadRequireError = recoverPayloadRequireError,
+	remapPayloadLines = remapPayloadLines,
 }
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="21">
+      <Item class="ModuleScript" referent="22">
         <Properties>
           <string name="Name">Recording</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -7188,7 +7462,7 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="22">
+      <Item class="ModuleScript" referent="23">
         <Properties>
           <string name="Name">RenderMonitor</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -7256,12 +7530,12 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="23">
+      <Item class="ModuleScript" referent="24">
         <Properties>
           <string name="Name">RuntimeLogBuffer</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
 local TS = require(script.Parent.Parent.include.RuntimeLib)
--- Per-peer in-memory ring buffer for LogService.MessageOut events.
+-- Per-capture in-memory ring buffer for LogService.MessageOut events.
 -- Powers the get_runtime_logs MCP tool. Replaces the out-of-tree LogBuffer
 -- primitives + StringValue approach from chrrxs/roblox-mcp-primitives.
 --
@@ -7271,12 +7545,12 @@ local TS = require(script.Parent.Parent.include.RuntimeLib)
 -- DataModel. The buffer is bounded by a message-byte budget; oldest entries
 -- drop when over budget.
 --
--- Peer-tag caveat: returned entries reflect which peer's plugin CAPTURED the
+-- Capture caveat: returned entries reflect which plugin buffer CAPTURED the
 -- entry, NOT which peer's script originated the print. LogService reflects
--- prints across peers in Studio Play (a server print ends up in both the
--- server and client LogService:GetLogHistory()) and origin is empirically
--- undetectable from inside MessageOut. The MCP-side aggregator handles
--- cross-peer dedup via a 2s timestamp window.
+-- prints across peers in ordinary Studio Play (a server print can appear in
+-- server and client LogService:GetLogHistory()). The MCP-side aggregator
+-- exposes that as capturedBy, and only promotes it to origin peer in
+-- StudioTestService multiplayer sessions where peer attribution is reliable.
 local _services = TS.import(script, script.Parent.Parent, "node_modules", "@rbxts", "services")
 local LogService = _services.LogService
 local RunService = _services.RunService
@@ -7340,7 +7614,7 @@ local function detectPeer()
 	end
 	return "client"
 end
-local function query(opts, peer)
+local function query(opts, capturedBy)
 	local _result
 	if opts.since ~= nil then
 		-- ▼ ReadonlyArray.filter ▼
@@ -7411,7 +7685,7 @@ local function query(opts, peer)
 	end
 	local last = if #entries > 0 then entries[#entries] else nil
 	local _object = {
-		peer = peer,
+		capturedBy = capturedBy,
 		entries = result,
 		totalDropped = totalDropped,
 	}
@@ -7437,11 +7711,12 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="24">
+      <Item class="ModuleScript" referent="25">
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.15.0"
+local CURRENT_VERSION = "2.15.2"
+local PLUGIN_VARIANT = "inspector"
 local MAX_CONNECTIONS = 5
 local BASE_PORT = 58741
 local activeTabIndex = 0
@@ -7519,6 +7794,7 @@ local function getConnections()
 end
 return {
 	CURRENT_VERSION = CURRENT_VERSION,
+	PLUGIN_VARIANT = PLUGIN_VARIANT,
 	MAX_CONNECTIONS = MAX_CONNECTIONS,
 	BASE_PORT = BASE_PORT,
 	connections = connections,
@@ -7533,7 +7809,7 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="25">
+      <Item class="ModuleScript" referent="26">
         <Properties>
           <string name="Name">StopPlayMonitor</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -7678,7 +7954,7 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="26">
+      <Item class="ModuleScript" referent="27">
         <Properties>
           <string name="Name">UI</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -7691,6 +7967,7 @@ local buttonHover = false
 local toolbarButton
 local toolbarIcons
 local lastToolbarIcon
+local activeBannerKind
 local updateToolbarIcon
 local function setToolbarButton(btn, icons)
 	toolbarButton = btn
@@ -7721,6 +7998,23 @@ local TWEEN_QUICK = TweenInfo.new(0.15, Enum.EasingStyle.Quad, Enum.EasingDirect
 local function tweenProp(instance, props)
 	TweenService:Create(instance, TWEEN_QUICK, props):Play()
 end
+local function showBanner(kind, text)
+	activeBannerKind = kind
+	elements.updateBannerText.Text = text
+	elements.updateBanner.Visible = true
+	elements.contentFrame.Position = UDim2.new(0, 8, 0, 92)
+	elements.contentFrame.Size = UDim2.new(1, -16, 1, -100)
+end
+local function hideBanner(kind)
+	if kind ~= nil and activeBannerKind ~= kind then
+		return nil
+	end
+	activeBannerKind = nil
+	elements.updateBanner.Visible = false
+	elements.updateBannerText.Text = ""
+	elements.contentFrame.Position = UDim2.new(0, 8, 0, 66)
+	elements.contentFrame.Size = UDim2.new(1, -16, 1, -74)
+end
 local C = {
 	bg = Color3.fromRGB(14, 14, 14),
 	card = Color3.fromRGB(22, 22, 22),
@@ -8422,6 +8716,8 @@ return {
 	startPulseAnimation = startPulseAnimation,
 	setToolbarButton = setToolbarButton,
 	updateToolbarIcon = updateToolbarIcon,
+	showBanner = showBanner,
+	hideBanner = hideBanner,
 	getElements = function()
 		return elements
 	end,
@@ -8429,7 +8725,7 @@ return {
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="27">
+      <Item class="ModuleScript" referent="28">
         <Properties>
           <string name="Name">Utils</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
@@ -8959,11 +9255,11 @@ return {
         </Properties>
       </Item>
     </Item>
-    <Item class="Folder" referent="31">
+    <Item class="Folder" referent="32">
       <Properties>
         <string name="Name">include</string>
       </Properties>
-      <Item class="ModuleScript" referent="28">
+      <Item class="ModuleScript" referent="29">
         <Properties>
           <string name="Name">Promise</string>
           <string name="Source"><![CDATA[--[[
@@ -11037,7 +11333,7 @@ return Promise
 ]]></string>
         </Properties>
       </Item>
-      <Item class="ModuleScript" referent="29">
+      <Item class="ModuleScript" referent="30">
         <Properties>
           <string name="Name">RuntimeLib</string>
           <string name="Source"><![CDATA[local Promise = require(script.Parent.Promise)
@@ -11304,15 +11600,15 @@ return TS
         </Properties>
       </Item>
     </Item>
-    <Item class="Folder" referent="32">
+    <Item class="Folder" referent="33">
       <Properties>
         <string name="Name">node_modules</string>
       </Properties>
-      <Item class="Folder" referent="33">
+      <Item class="Folder" referent="34">
         <Properties>
           <string name="Name">@rbxts</string>
         </Properties>
-      <Item class="ModuleScript" referent="30">
+      <Item class="ModuleScript" referent="31">
         <Properties>
           <string name="Name">services</string>
           <string name="Source"><![CDATA[return setmetatable({}, {