@el4cteo/rbx-studio-mcp 0.8.3 → 0.8.4

package/README.md

@@ -108,7 +108,7 @@ Two things to watch: a playtest connects a second session, so pass `studioId` an
 
 ## The console panel
 
-Every call is logged with how long it took. At the foot of the panel is a command line — type a command, or type a sentence and a coding agent answers it.
+Every call is logged with how long it took. At the foot of the panel is a command line. Type a command — or run `chat on` and type a sentence to have a coding agent answer it.
 
 | | |
 |---|---|
@@ -119,12 +119,13 @@ Every call is logged with how long it took. At the foot of the panel is a comman
 | `theme [name]` `visuals` `autoopen [on\|off]` `log [level]` `clear` `copy` | the panel |
 | `port [n]` `reconnect` | the connection |
 | `cloud [key\|user\|group\|test\|forget]` | the Open Cloud key `upload` uses |
+| `chat [on\|off]` | let an agent answer sentences (off by default) |
 | `agent [use <id>\|new]` `stop` | which agent runs your prompts |
-| anything else | sent to that agent |
+| anything else | sent to that agent, once `chat` is on |
 
 Click the bar and every command is listed with what it does. Keep typing to filter, scroll for the rest, click one to fill it in.
 
-**Prompts start a real agent** — whichever you have on PATH: Claude Code, Codex, opencode, Gemini, Cursor, Amp, Qwen Code, Factory Droid, goose, Copilot CLI, Aider, Crush, DeepSeek Harness. It runs headless, drives the same Studio, and its work appears in the log. It is a separate session from your terminal, billed separately, and allowed the `rbx-studio` tools only. `stop` cancels it.
+**With `chat on`, prompts start a real agent** — whichever you have on PATH: Claude Code, Codex, opencode, Gemini, Cursor, Amp, Qwen Code, Factory Droid, goose, Copilot CLI, Aider, Crush, DeepSeek Harness. It runs headless, drives the same Studio, and its work appears in the log. It is a separate session from your terminal, billed separately, and allowed the `rbx-studio` tools only. `stop` cancels it.
 
 Eight themes behind the tab on the right edge. Your pick is remembered.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@el4cteo/rbx-studio-mcp",
-  "version": "0.8.3",
+  "version": "0.8.4",
   "description": "MCP server for Roblox Studio. 34 tools, push-based SSE bridge, editor-safe script edits, one-step undo.",
   "type": "module",
   "license": "MIT",

package/plugin/src/Commands.luau

@@ -46,6 +46,9 @@ export type Hooks = {
 	-- Whether the panel may open itself on load, and the setter for it.
 	autoOpen: () -> boolean,
 	saveAutoOpen: (boolean) -> (),
+	-- Whether a line that is not a command is sent to an agent, and the setter.
+	chat: () -> boolean,
+	saveChat: (boolean) -> (),
 	-- The transport's own view of the connection, which the console only ever
 	-- receives second-hand as a string to display.
 	status: () -> string,
@@ -137,11 +140,16 @@ define({
 		for _, entry in entries do
 			Console.log("dim", "   " .. entry.usage, entry.summary)
 		end
-		Console.log(
-			"dim",
-			"   anything else",
-			"sent to a coding agent -- run `agent` to see which"
-		)
+		local current = hooks
+		if current ~= nil and current.chat() then
+			Console.log(
+				"dim",
+				"   anything else",
+				"sent to a coding agent -- run `agent` to see which"
+			)
+		else
+			Console.log("dim", "   anything else", "ignored -- `chat on` sends it to a coding agent")
+		end
 	end,
 })
 
@@ -334,6 +342,56 @@ define({
 	end,
 })
 
+--[[
+	Whether a sentence typed at the prompt reaches an agent.
+
+	Off by default. A prompt starts a real agent -- a separate session, billed
+	separately -- so a line typed into the panel by habit, or pasted by mistake,
+	should not spend anything until the user has said they want that. Commands
+	are unaffected either way, and so are `agent` and `stop`.
+]]
+define({
+	name = "chat",
+	usage = "chat [on|off]",
+	summary = "let a coding agent answer sentences typed here; OFF by default",
+	run = function(args)
+		local current = hooks
+		if current == nil then
+			return
+		end
+
+		local wanted = args[1]
+		if #args > 1 or (wanted ~= nil and wanted ~= "on" and wanted ~= "off") then
+			Console.log("error", "usage: chat [on|off]")
+			return
+		end
+
+		if wanted == nil then
+			Console.log(
+				"dim",
+				if current.chat()
+					then "chat on -- a sentence is sent to a coding agent"
+					else "chat off -- a sentence is ignored; `chat on` to let an agent answer"
+			)
+			return
+		end
+
+		local value = wanted == "on"
+		if value == current.chat() then
+			Console.log("dim", string.format("already chat %s", wanted))
+			return
+		end
+
+		current.saveChat(value)
+		Console.setPromptChat(value)
+		if value then
+			Console.log("ok", "chat on", "type a sentence and a coding agent answers; `agent` shows which")
+		else
+			Console.log("ok", "chat off", "sentences are ignored; commands still run")
+		end
+	end,
+})
+
 define({
 	name = "place",
 	usage = "place",
@@ -648,6 +706,15 @@ function Commands.run(line: string)
 				return
 			end
 		end
+		local current = hooks
+		if current == nil or not current.chat() then
+			Console.log(
+				"dim",
+				"not a command, and chat is off -- nothing was sent",
+				"run `chat on` to let a coding agent answer sentences, or `help` for commands"
+			)
+			return
+		end
 		Console.setPromptBusy(true)
 		remote("prompt", words, trimmed)
 		return

package/plugin/src/Config.luau

@@ -1,65 +1,65 @@
---!strict
---[[
-	Connection settings for the Studio MCP plugin.
-
-	The port must match the one the Node server was started with (`--port`, or
-	the ROBLOX_STUDIO_MCP_PORT environment variable). It is stored as a plugin
-	setting so a user running on a non-default port only has to set it once.
-]]
-
-local Config = {}
-
-Config.PLUGIN_VERSION = "0.8.3"
-
--- 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
--- plugin left running from an older build is reported rather than silently
--- answering with stale handlers. Stays "dev" when the tree is loaded unbuilt.
-Config.BUILD_ID = "dev"
-Config.PROTOCOL_VERSION = 1
-Config.DEFAULT_PORT = 44755
-
--- Header the bridge requires on every request. A web page cannot set it
--- cross-origin without a preflight the bridge never answers, which is what
--- keeps a malicious site from driving Studio through the loopback port.
-Config.CLIENT_HEADER = "x-roblox-studio-mcp"
-
--- How long to wait before retrying after the stream drops. Studio closes SSE
--- connections at the 30 minute mark, so reconnects are routine, not exceptional.
-Config.RECONNECT_DELAY = 1
-Config.MAX_RECONNECT_DELAY = 30
-
--- How long a connection has to last before it counts as having worked, rather
--- than as one more failure to back off from. Comfortably longer than a failed
--- handshake against a dead port, which resolves in milliseconds, and far shorter
--- than any connection that carried a single command.
-Config.HEALTHY_CONNECTION = 10
-
--- How long a healthy long-poll session runs before standing aside so streaming
--- can be tried again. Long enough that a Studio which genuinely cannot stream
--- is not reconnecting constantly, short enough that a session downgraded by one
--- unlucky attempt does not spend the rest of its life slower than it should be.
-Config.RESTREAM_INTERVAL = 60
-
-local port: number? = nil
-
-function Config.setPort(value: number)
-	port = value
-end
-
-function Config.getPort(): number
-	return port or Config.DEFAULT_PORT
-end
-
-function Config.baseUrl(): string
-	return string.format("http://127.0.0.1:%d", Config.getPort())
-end
-
-function Config.headers(): { [string]: string }
-	return {
-		[Config.CLIENT_HEADER] = tostring(Config.PROTOCOL_VERSION),
-		["Content-Type"] = "application/json",
-	}
-end
-
-return Config
+--!strict
+--[[
+	Connection settings for the Studio MCP plugin.
+
+	The port must match the one the Node server was started with (`--port`, or
+	the ROBLOX_STUDIO_MCP_PORT environment variable). It is stored as a plugin
+	setting so a user running on a non-default port only has to set it once.
+]]
+
+local Config = {}
+
+Config.PLUGIN_VERSION = "0.8.4"
+
+-- 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
+-- plugin left running from an older build is reported rather than silently
+-- answering with stale handlers. Stays "dev" when the tree is loaded unbuilt.
+Config.BUILD_ID = "dev"
+Config.PROTOCOL_VERSION = 1
+Config.DEFAULT_PORT = 44755
+
+-- Header the bridge requires on every request. A web page cannot set it
+-- cross-origin without a preflight the bridge never answers, which is what
+-- keeps a malicious site from driving Studio through the loopback port.
+Config.CLIENT_HEADER = "x-roblox-studio-mcp"
+
+-- How long to wait before retrying after the stream drops. Studio closes SSE
+-- connections at the 30 minute mark, so reconnects are routine, not exceptional.
+Config.RECONNECT_DELAY = 1
+Config.MAX_RECONNECT_DELAY = 30
+
+-- How long a connection has to last before it counts as having worked, rather
+-- than as one more failure to back off from. Comfortably longer than a failed
+-- handshake against a dead port, which resolves in milliseconds, and far shorter
+-- than any connection that carried a single command.
+Config.HEALTHY_CONNECTION = 10
+
+-- How long a healthy long-poll session runs before standing aside so streaming
+-- can be tried again. Long enough that a Studio which genuinely cannot stream
+-- is not reconnecting constantly, short enough that a session downgraded by one
+-- unlucky attempt does not spend the rest of its life slower than it should be.
+Config.RESTREAM_INTERVAL = 60
+
+local port: number? = nil
+
+function Config.setPort(value: number)
+	port = value
+end
+
+function Config.getPort(): number
+	return port or Config.DEFAULT_PORT
+end
+
+function Config.baseUrl(): string
+	return string.format("http://127.0.0.1:%d", Config.getPort())
+end
+
+function Config.headers(): { [string]: string }
+	return {
+		[Config.CLIENT_HEADER] = tostring(Config.PROTOCOL_VERSION),
+		["Content-Type"] = "application/json",
+	}
+end
+
+return Config

package/plugin/src/Console.luau

@@ -1870,6 +1870,11 @@ function Console.setPromptBusy(busy: boolean)
 	Visuals.setThinking(busy)
 end
 
+-- Whether the prompt row invites sentences. See `chat` in Commands.
+function Console.setPromptChat(enabled: boolean)
+	Prompt.setChat(enabled)
+end
+
 --[[
 	Switches the whole panel to the active preset.
 

package/plugin/src/Prompt.luau

@@ -162,6 +162,10 @@ type Runtime = {
 	-- back returns it rather than an empty line.
 	draft: string,
 	busy: boolean,
+	-- Whether a line that is not a command goes to an agent. Off until the user
+	-- runs `chat on`; only used to word the placeholder, since `Commands` is what
+	-- actually refuses the line.
+	chat: boolean,
 	--[[
 		What is selected in Studio right now, already worded for display.
 
@@ -195,6 +199,7 @@ local runtime: Runtime = {
 	offset = 0,
 	draft = "",
 	busy = false,
+	chat = false,
 	selection = "",
 	complete = nil,
 	suggest = nil,
@@ -218,9 +223,12 @@ local function paintCaret()
 end
 
 local function placeholder(): string
-	return if runtime.busy
-		then "running -- type stop to cancel"
-		else "type a command, or ask for something"
+	if runtime.busy then
+		return "running -- type stop to cancel"
+	end
+	return if runtime.chat
+		then "type a command, or ask for something"
+		else "type a command -- `chat on` lets an agent answer"
 end
 
 --[[
@@ -903,6 +911,18 @@ function Prompt.setBusy(busy: boolean)
 	end
 end
 
+--[[
+	Reports whether sentences are answered, so the placeholder does not invite
+	a request the console is about to decline.
+]]
+function Prompt.setChat(enabled: boolean)
+	runtime.chat = enabled
+	local field = runtime.field
+	if field ~= nil then
+		field.PlaceholderText = placeholder()
+	end
+end
+
 --[[
 	Reports what Studio has selected, as a phrase rather than a list.
 

package/plugin/src/init.server.luau

@@ -65,6 +65,9 @@ local SETTING_WIDGET_OPEN = "widgetOpen"
 ]]
 local SETTING_AUTO_OPEN = "autoOpen"
 local SETTING_PLAYTESTS = "playtestsEnabled"
+-- Whether sentences typed at the prompt are sent to an agent. Absent means off:
+-- it only turns on when the user runs `chat on`.
+local SETTING_CHAT = "chatEnabled"
 local SETTING_WIDGET_SIZE = "widgetSize"
 local SETTING_WIDGET_TRACE = "widgetTrace"
 
@@ -261,6 +264,8 @@ local wantOpen = if typeof(storedOpen) == "boolean" then storedOpen else true
 	left exactly as it was, so turning this back on restores the panel to however
 	it was last left rather than to a default.
 ]]
+local chat = plugin:GetSetting(SETTING_CHAT) == true
+
 local storedAutoOpen = plugin:GetSetting(SETTING_AUTO_OPEN)
 local autoOpen = if typeof(storedAutoOpen) == "boolean" then storedAutoOpen else true
 if not autoOpen then
@@ -560,12 +565,21 @@ Commands.setup({
 		autoOpen = value
 		plugin:SetSetting(SETTING_AUTO_OPEN, value)
 	end,
+	chat = function()
+		return chat
+	end,
+	saveChat = function(value)
+		chat = value
+		plugin:SetSetting(SETTING_CHAT, value)
+	end,
 	status = function()
 		return currentStatus
 	end,
 	studioId = studioId,
 })
 
+Console.setPromptChat(chat)
+
 -- The saved preset, now that there is something to paint. Done immediately
 -- after mounting and before the first line is logged, so nothing is ever drawn
 -- in the wrong palette.

package/scripts/test-plugin.mjs

@@ -30,6 +30,7 @@ if (luau === null) {
 const suites = [
   { module: "plugin/src/handlers/Playtest.luau", test: "tests/playtests.luau", prelude: "tests/playtests-stub.luau" },
   { module: "plugin/src/Commands.luau", test: "tests/playtests-commands.luau", prelude: "tests/playtests-stub.luau", dependency: "plugin/src/handlers/Playtest.luau" },
+  { module: "plugin/src/Commands.luau", test: "tests/chat-commands.luau", prelude: "tests/playtests-stub.luau", dependency: "plugin/src/handlers/Playtest.luau" },
   { module: "plugin/src/RemoteTrace.luau", test: "tests/remote-trace.luau", prelude: "tests/remote-trace-stub.luau" },
   { module: "plugin/src/ExecRuntime.luau", test: "tests/exec-runtime.luau", prelude: "tests/exec-runtime-stub.luau" },
   { module: "plugin/src/ClientRelay.luau", test: "tests/client-relay.luau", prelude: "tests/client-relay-stub.luau" },