@el4cteo/rbx-studio-mcp 0.7.7 → 0.8.0

package/package.json

@@ -1,75 +1,75 @@
-{
-  "name": "@el4cteo/rbx-studio-mcp",
-  "version": "0.7.7",
-  "description": "MCP server for Roblox Studio. 35 tools, push-based SSE bridge, editor-safe script edits, one-step undo.",
-  "type": "module",
-  "license": "MIT",
-  "bin": {
-    "rbx-studio-mcp": "dist/index.js"
-  },
-  "files": [
-    "dist",
-    "plugin",
-    "scripts",
-    "config/dsh.cordis.yml",
-    "README.md",
-    "LICENSE"
-  ],
-  "dsh": {
-    "bundle": {
-      "patch": "./config/dsh.cordis.yml"
-    }
-  },
-  "engines": {
-    "node": ">=22.15"
-  },
-  "scripts": {
-    "build": "tsc",
-    "watch": "tsc --watch",
-    "typecheck": "tsc --noEmit",
-    "start": "node dist/index.js",
-    "build:plugin": "node scripts/check-plugin.mjs && node scripts/build-plugin.mjs",
-    "sourcemap": "node scripts/sourcemap.mjs",
-    "test:live": "node scripts/test-live.mjs",
-    "test": "node scripts/test-plugin.mjs && node scripts/test-transport.mjs && node scripts/test-server.mjs && node scripts/test-bridge.mjs && node scripts/test-console.mjs && node scripts/test-failover.mjs && node scripts/test-tools.mjs",
-    "build:all": "npm run build && npm run build:plugin",
-    "install:plugin": "node scripts/install-plugin.mjs",
-    "check:plugin": "node scripts/check-plugin.mjs",
-    "doctor": "node dist/index.js doctor",
-    "prepack": "npm run build"
-  },
-  "keywords": [
-    "mcp",
-    "mcp-server",
-    "model-context-protocol",
-    "roblox",
-    "roblox-studio",
-    "roblox-development",
-    "luau",
-    "claude",
-    "claude-code",
-    "cursor",
-    "ai",
-    "ai-agent"
-  ],
-  "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.30.0",
-    "zod": "^4.6.5"
-  },
-  "devDependencies": {
-    "@types/node": "^26.6.2",
-    "typescript": "^7.0.2"
-  },
-  "author": "EL4CTEO",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/EL4CTEO/rbx-studio-mcp.git"
-  },
-  "homepage": "https://github.com/EL4CTEO/rbx-studio-mcp#readme",
-  "bugs": {
-    "url": "https://github.com/EL4CTEO/rbx-studio-mcp/issues"
-  },
-  "publishConfig": {
-    "access": "public"
-  }
-}
+{
+  "name": "@el4cteo/rbx-studio-mcp",
+  "version": "0.8.0",
+  "description": "MCP server for Roblox Studio. 34 tools, push-based SSE bridge, editor-safe script edits, one-step undo.",
+  "type": "module",
+  "license": "MIT",
+  "bin": {
+    "rbx-studio-mcp": "dist/index.js"
+  },
+  "files": [
+    "dist",
+    "plugin",
+    "scripts",
+    "config/dsh.cordis.yml",
+    "README.md",
+    "LICENSE"
+  ],
+  "dsh": {
+    "bundle": {
+      "patch": "./config/dsh.cordis.yml"
+    }
+  },
+  "engines": {
+    "node": ">=22.15"
+  },
+  "scripts": {
+    "build": "tsc",
+    "watch": "tsc --watch",
+    "typecheck": "tsc --noEmit",
+    "start": "node dist/index.js",
+    "build:plugin": "node scripts/check-plugin.mjs && node scripts/build-plugin.mjs",
+    "sourcemap": "node scripts/sourcemap.mjs",
+    "test:live": "node scripts/test-live.mjs",
+    "test": "node scripts/test-plugin.mjs && node scripts/test-transport.mjs && node scripts/test-server.mjs && node scripts/test-bridge.mjs && node scripts/test-console.mjs && node scripts/test-failover.mjs && node scripts/test-tools.mjs",
+    "build:all": "npm run build && npm run build:plugin",
+    "install:plugin": "node scripts/install-plugin.mjs",
+    "check:plugin": "node scripts/check-plugin.mjs",
+    "doctor": "node dist/index.js doctor",
+    "prepack": "npm run build"
+  },
+  "keywords": [
+    "mcp",
+    "mcp-server",
+    "model-context-protocol",
+    "roblox",
+    "roblox-studio",
+    "roblox-development",
+    "luau",
+    "claude",
+    "claude-code",
+    "cursor",
+    "ai",
+    "ai-agent"
+  ],
+  "dependencies": {
+    "@modelcontextprotocol/sdk": "^1.30.1",
+    "zod": "^4.6.5"
+  },
+  "devDependencies": {
+    "@types/node": "^26.6.2",
+    "typescript": "^7.0.2"
+  },
+  "author": "EL4CTEO",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/EL4CTEO/rbx-studio-mcp.git"
+  },
+  "homepage": "https://github.com/EL4CTEO/rbx-studio-mcp#readme",
+  "bugs": {
+    "url": "https://github.com/EL4CTEO/rbx-studio-mcp/issues"
+  },
+  "publishConfig": {
+    "access": "public"
+  }
+}

package/plugin/src/ClientRelay.luau

@@ -1,9 +1,136 @@
 --!strict
--- One temporary client relay lifecycle for input and execution.
+-- Shared client relay lifecycles for temporary calls and persistent diagnostics.
 local Players = game:GetService("Players")
 local RunService = game:GetService("RunService")
 local Dispatch = require(script.Parent.Dispatch)
+local LogBuffer = require(script.Parent.LogBuffer)
 local ClientRelay = {}
+
+-- Long-lived diagnostics use the same LocalScript/RemoteEvent replication path
+-- as temporary client calls. The server owns the buffers and validates the
+-- sender; the client never needs HTTP access.
+local DIAGNOSTIC_SOURCE = [==[
+local LogService = game:GetService("LogService")
+local ScriptContext = game:GetService("ScriptContext")
+local report = script:WaitForChild("Report", 10)
+if not report then return end
+local open = true
+local connections = {}
+local levels = {
+	[Enum.MessageType.MessageOutput] = "print",
+	[Enum.MessageType.MessageInfo] = "info",
+	[Enum.MessageType.MessageWarning] = "warning",
+	[Enum.MessageType.MessageError] = "error",
+}
+local function send(payload)
+	if open and report.Parent then report:FireServer(payload) end
+end
+table.insert(connections, LogService.MessageOut:Connect(function(message, kind)
+	send({level = levels[kind] or "print", message = string.sub(message, 1, 2000), timestamp = os.time()})
+end))
+pcall(function()
+	table.insert(connections, ScriptContext.Error:Connect(function(message, stack, source)
+		local path
+		if typeof(source) == "Instance" then
+			local ok, full = pcall(function() return source:GetFullName() end)
+			path = if ok then full else source.Name
+		end
+		send({kind = "trace", message = string.sub(message, 1, 2000), stack = string.sub(stack, 1, 4000), source = path})
+	end))
+end)
+script.Destroying:Connect(function()
+	open = false
+	for _, connection in connections do connection:Disconnect() end
+end)
+send({kind = "ready"})
+]==]
+
+local diagnosticsStarted = false
+local diagnosticConnections: { [Player]: RBXScriptConnection } = {}
+local diagnosticRelays: { [Player]: LocalScript } = {}
+local diagnosticInstalling: { [Player]: boolean } = {}
+local characterConnections: { [Player]: RBXScriptConnection } = {}
+local playerAddedConnection: RBXScriptConnection? = nil
+local playerRemovingConnection: RBXScriptConnection? = nil
+local function installDiagnostics(player: Player)
+	if diagnosticInstalling[player] or diagnosticConnections[player] then return end
+	diagnosticInstalling[player] = true
+	task.spawn(function()
+		-- PlayerGui is server-visible (PlayerScripts is created on the client).
+		local gui = player:FindFirstChildOfClass("PlayerGui") or player:WaitForChild("PlayerGui", 15)
+		if not diagnosticsStarted or not gui or player.Parent ~= Players then diagnosticInstalling[player] = nil; return end
+		local old = gui:FindFirstChild("MCPDiagnosticsRelay")
+		if old then old:Destroy() end
+		LogBuffer.startClient(player)
+		local relay = Instance.new("LocalScript")
+		local remote = Instance.new("RemoteEvent")
+		local ok = pcall(function()
+			relay.Name = "MCPDiagnosticsRelay"
+			relay:SetAttribute("MCPClientRelay", true)
+			relay.Source = DIAGNOSTIC_SOURCE
+			remote.Name = "Report"
+			remote.Parent = relay
+			diagnosticConnections[player] = remote.OnServerEvent:Connect(function(from, payload)
+				if from == player then LogBuffer.pushClient(player, payload) end
+			end)
+			relay.Parent = gui
+			diagnosticRelays[player] = relay
+		end)
+		if not ok then
+			if diagnosticConnections[player] then diagnosticConnections[player]:Disconnect() end
+			diagnosticConnections[player] = nil
+			relay:Destroy()
+			LogBuffer.stopClient(player)
+		end
+		diagnosticInstalling[player] = nil
+		if not characterConnections[player] then
+			characterConnections[player] = player.CharacterAdded:Connect(function()
+				task.delay(0.5, function()
+					if not diagnosticsStarted or player.Parent ~= Players then return end
+					local currentGui = player:FindFirstChildOfClass("PlayerGui")
+					local current = diagnosticRelays[player]
+					if current and current.Parent == currentGui then return end
+					if diagnosticConnections[player] then diagnosticConnections[player]:Disconnect() end
+					diagnosticConnections[player], diagnosticRelays[player] = nil, nil
+					installDiagnostics(player)
+				end)
+			end)
+		end
+	end)
+end
+
+function ClientRelay.startDiagnostics()
+	if diagnosticsStarted or not RunService:IsRunning() or not RunService:IsServer() then return end
+	diagnosticsStarted = true
+	playerAddedConnection = Players.PlayerAdded:Connect(installDiagnostics)
+	playerRemovingConnection = Players.PlayerRemoving:Connect(function(player)
+		if diagnosticConnections[player] then diagnosticConnections[player]:Disconnect() end
+		diagnosticConnections[player] = nil
+		if diagnosticRelays[player] then diagnosticRelays[player]:Destroy() end
+		diagnosticRelays[player] = nil
+		if characterConnections[player] then characterConnections[player]:Disconnect() end
+		characterConnections[player] = nil
+		LogBuffer.stopClient(player)
+	end)
+	for _, player in Players:GetPlayers() do installDiagnostics(player) end
+end
+
+function ClientRelay.stopDiagnostics()
+	if not diagnosticsStarted then return end
+	diagnosticsStarted = false
+	if playerAddedConnection then playerAddedConnection:Disconnect() end
+	if playerRemovingConnection then playerRemovingConnection:Disconnect() end
+	playerAddedConnection, playerRemovingConnection = nil, nil
+	for player, connection in diagnosticConnections do
+		connection:Disconnect()
+		if diagnosticRelays[player] then diagnosticRelays[player]:Destroy() end
+		LogBuffer.stopClient(player)
+	end
+	for _, connection in characterConnections do connection:Disconnect() end
+	table.clear(diagnosticConnections)
+	table.clear(diagnosticRelays)
+	table.clear(characterConnections)
+end
 function ClientRelay.playerFor(name: string?): Player
 	if not RunService:IsRunning() then
 		Dispatch.fail("NOT_RUNNING", "Client access needs a running playtest; this is an edit session.")

package/plugin/src/Config.luau

@@ -9,7 +9,7 @@
 
 local Config = {}
 
-Config.PLUGIN_VERSION = "0.7.7"
+Config.PLUGIN_VERSION = "0.8.0"
 
 -- Fingerprint of plugin/src, stamped in by scripts/build-plugin.mjs. The server
 -- computes the same hash from its own copy of the sources and compares, so a