@chrrxs/robloxstudio-mcp 2.9.1 → 2.10.1

package/studio-plugin/src/modules/handlers/MetadataHandlers.ts

@@ -294,8 +294,19 @@ function executeLuau(requestData: Record<string, unknown>) {
 	const runViaModuleScript = () => {
 		const m = new Instance("ModuleScript");
 		m.Name = "__MCPExecLuauPayload";
+		// Wrap user code in an IIFE so require() always gets exactly one
+		// return value. Without this, code like `print("x")` errors with
+		// "Module code did not return exactly one value" because top-level
+		// ModuleScripts must return exactly one value.
+		//
+		// The DOUBLE parens around the call are load-bearing: in Luau,
+		// `return f()` propagates whatever multi-value tuple f returns,
+		// including zero values. Outer parens adjust the call to exactly
+		// one value (the first, or nil). So `return ((f)())` always
+		// returns exactly one value, regardless of what f does.
+		const wrapped = `return ((function()\n${code}\nend)())`;
 		const [okSet, setErr] = pcall(() => {
-			(m as unknown as { Source: string }).Source = code;
+			(m as unknown as { Source: string }).Source = wrapped;
 		});
 		if (!okSet) {
 			m.Destroy();

package/studio-plugin/src/modules/handlers/QueryHandlers.ts

@@ -96,8 +96,23 @@ function searchFiles(requestData: Record<string, unknown>) {
 }
 
 function getPlaceInfo(_requestData: Record<string, unknown>) {
+	const dataModelName = game.Name;
+	let placeName = dataModelName;
+
+	if (game.PlaceId > 0) {
+		const MarketplaceService = game.GetService("MarketplaceService");
+		const [ok, info] = pcall(() => MarketplaceService.GetProductInfo(game.PlaceId));
+		if (ok && info !== undefined) {
+			const name = (info as { Name?: string }).Name;
+			if (typeIs(name, "string") && name !== "") {
+				placeName = name;
+			}
+		}
+	}
+
 	return {
-		placeName: game.Name,
+		placeName,
+		dataModelName,
 		placeId: game.PlaceId,
 		gameId: game.GameId,
 		jobId: game.JobId,

package/studio-plugin/src/modules/handlers/TestHandlers.ts

@@ -1,5 +1,5 @@
 import { HttpService, LogService } from "@rbxts/services";
-import { installBridges, cleanupBridges, loadStringEnabled } from "../EvalBridges";
+import { installBridges, cleanupBridges } from "../EvalBridges";
 
 const StudioTestService = game.GetService("StudioTestService");
 const ServerScriptService = game.GetService("ServerScriptService");
@@ -159,7 +159,6 @@ function startPlaytest(requestData: Record<string, unknown>) {
 	// so eval_server_runtime / eval_client_runtime work without manual setup.
 	// Bridges are cleaned up from the edit DM after the play DMs tear down.
 	const bridgeInstall = installBridges();
-	const hasLoadString = loadStringEnabled();
 	if (!bridgeInstall.installed) {
 		warn(`[MCP] Eval bridge install failed: ${bridgeInstall.error}`);
 	}
@@ -203,17 +202,6 @@ function startPlaytest(requestData: Record<string, unknown>) {
 		evalBridges: bridgeInstall.installed ? "installed" : `failed: ${bridgeInstall.error}`,
 	};
 
-	// Surface loadstring availability up-front so callers know whether
-	// eval_server_runtime will work before they try it. eval_client_runtime
-	// doesn't need loadstring (it uses ModuleScript+require), so this only
-	// affects the server bridge.
-	if (!hasLoadString) {
-		response.serverEvalNote =
-			"ServerScriptService.LoadStringEnabled is false. eval_server_runtime will not work " +
-			"until you enable it (ServerScriptService > Properties > LoadStringEnabled = true) " +
-			"and restart the playtest. eval_client_runtime is unaffected.";
-	}
-
 	return response;
 }
 

package/studio-plugin/src/server/index.server.ts

@@ -2,14 +2,24 @@ import State from "../modules/State";
 import UI from "../modules/UI";
 import Communication from "../modules/Communication";
 import ClientBroker from "../modules/ClientBroker";
+import RuntimeLogBuffer from "../modules/RuntimeLogBuffer";
 
+// Attach the per-peer LogService.MessageOut listener as early as possible so
+// boot-time prints from the user's place scripts are captured. Powers the
+// get_runtime_logs MCP tool. Idempotent; safe to call before UI.init().
+RuntimeLogBuffer.install();
 
 UI.init(plugin);
 const elements = UI.getElements();
 
 
+const ICON_DISCONNECTED = "rbxassetid://__BUTTON_ICON_DISCONNECTED__";
+const ICON_CONNECTING = "rbxassetid://__BUTTON_ICON_CONNECTING__";
+const ICON_CONNECTED = "rbxassetid://__BUTTON_ICON_CONNECTED__";
+
 const toolbar = plugin.CreateToolbar("__TOOLBAR_NAME__");
-const button = toolbar.CreateButton("__BUTTON_TITLE__", "__BUTTON_TOOLTIP__", "rbxassetid://__BUTTON_ICON_ID__");
+const button = toolbar.CreateButton("__BUTTON_TITLE__", "__BUTTON_TOOLTIP__", ICON_DISCONNECTED);
+UI.setToolbarButton(button, { disconnected: ICON_DISCONNECTED, connecting: ICON_CONNECTING, connected: ICON_CONNECTED });
 
 
 elements.connectButton.Activated.Connect(() => {

package/studio-plugin/src/types/index.d.ts

@@ -32,6 +32,10 @@ export interface PollResponse {
 	mcpConnected: boolean;
 	request?: RequestPayload;
 	requestId?: string;
+	// Server signals knownInstance=false when its in-memory instances map
+	// doesn't contain our instanceId (typically after an MCP process restart).
+	// The plugin re-issues /ready when it sees this.
+	knownInstance?: boolean;
 }
 
 export interface ReadyResponse {