@chrrxs/robloxstudio-mcp-inspector 2.23.0 → 2.23.1

package/dist/index.js

@@ -3856,8 +3856,8 @@ var init_http_server = __esm({
       solo_playtest: (tools, body) => tools.soloPlaytest(body.action, body.mode, body.timeout, body.instance_id),
       start_playtest: (tools, body) => tools.startPlaytest(body.mode, body.numPlayers, body.instance_id),
       stop_playtest: (tools, body) => tools.stopPlaytest(body.instance_id),
-      multiplayer_playtest: (tools, body) => tools.multiplayerPlaytest(body.action, body.numPlayers, body.target, body.testArgs, body.value, body.timeout, body.instance_id, body.force),
-      multiplayer_test_start: (tools, body) => tools.multiplayerTestStart(body.numPlayers, body.testArgs, body.timeout, body.instance_id, body.force),
+      multiplayer_playtest: (tools, body) => tools.multiplayerPlaytest(body.action, body.numPlayers, body.target, body.testArgs, body.value, body.timeout, body.instance_id),
+      multiplayer_test_start: (tools, body) => tools.multiplayerTestStart(body.numPlayers, body.testArgs, body.timeout, body.instance_id),
       multiplayer_test_state: (tools, body) => tools.multiplayerTestState(body.instance_id),
       multiplayer_test_add_players: (tools, body) => tools.multiplayerTestAddPlayers(body.numPlayers, body.timeout, body.instance_id),
       multiplayer_test_leave_client: (tools, body) => tools.multiplayerTestLeaveClient(body.target, body.timeout, body.instance_id),
@@ -7883,16 +7883,6 @@ var init_png_encoder = __esm({
 import * as fs3 from "fs";
 import * as os3 from "os";
 import * as path5 from "path";
-function multiplayerStopDisabledBody() {
-  return {
-    success: false,
-    error: "multiplayer_stop_disabled",
-    message: MULTIPLAYER_STOP_DISABLED_MESSAGE,
-    reason: "StudioTestService:EndTest does not reliably end StudioTestService multiplayer sessions from MCP right now.",
-    manualCleanupRequired: true,
-    recoveryHint: "Close the Roblox Studio multiplayer test windows manually."
-  };
-}
 function normalizeCreatorStoreSearch(assetType, query) {
   if (!CREATOR_STORE_SEARCH_TYPES.has(assetType)) {
     throw new Error(`search_assets assetType must be one of: ${Array.from(CREATOR_STORE_SEARCH_TYPES).join(", ")}`);
@@ -8548,7 +8538,7 @@ end
 error("Unsupported device simulator operation: " .. tostring(opts.operation), 0)
 `.trim();
 }
-var MAX_INLINE_IMAGE_BYTES, DEFAULT_ASSET_AUDIO_PREVIEWS, MAX_ASSET_AUDIO_PREVIEWS, MAX_INLINE_AUDIO_PREVIEW_BYTES, MAX_INLINE_AUDIO_PREVIEW_TOTAL_BYTES, DEFAULT_ASSET_PREVIEW_DEPTH, MAX_ASSET_PREVIEW_HIERARCHY_NODES, MAX_SEARCH_ASSET_DESCRIPTION_LENGTH, ROBLOX_CREATOR_USER_ID, MAX_DEVICE_MATRIX_ENTRIES, MAX_NETWORK_PACKET_LOSS_PERCENT, STUDIO_ASSISTANT_SOURCE_IMAGE_LABEL, CREATOR_STORE_SEARCH_TYPES, CREATOR_STORE_SORT_CATEGORIES, MULTIPLAYER_FORCE_REQUIRED_MESSAGE, MULTIPLAYER_STOP_DISABLED_MESSAGE, NETWORK_PROFILE_KEYS, NETWORK_PROFILES, ZERO_NETWORK_PROFILE, SIMULATION_PERSISTENCE_NOTES, RobloxStudioTools;
+var MAX_INLINE_IMAGE_BYTES, DEFAULT_ASSET_AUDIO_PREVIEWS, MAX_ASSET_AUDIO_PREVIEWS, MAX_INLINE_AUDIO_PREVIEW_BYTES, MAX_INLINE_AUDIO_PREVIEW_TOTAL_BYTES, DEFAULT_ASSET_PREVIEW_DEPTH, MAX_ASSET_PREVIEW_HIERARCHY_NODES, MAX_SEARCH_ASSET_DESCRIPTION_LENGTH, ROBLOX_CREATOR_USER_ID, MAX_DEVICE_MATRIX_ENTRIES, MAX_NETWORK_PACKET_LOSS_PERCENT, STUDIO_ASSISTANT_SOURCE_IMAGE_LABEL, CREATOR_STORE_SEARCH_TYPES, CREATOR_STORE_SORT_CATEGORIES, NETWORK_PROFILE_KEYS, NETWORK_PROFILES, ZERO_NETWORK_PROFILE, SIMULATION_PERSISTENCE_NOTES, RobloxStudioTools;
 var init_tools = __esm({
   "../core/dist/tools/index.js"() {
     "use strict";
@@ -8596,8 +8586,6 @@ var init_tools = __esm({
       "UpdatedTime",
       "Ratings"
     ]);
-    MULTIPLAYER_FORCE_REQUIRED_MESSAGE = "StudioTestService multiplayer stop is currently disabled because StudioTestService:EndTest is broken for this flow. Pass force=true only if you understand you must manually close the multiplayer test windows afterward.";
-    MULTIPLAYER_STOP_DISABLED_MESSAGE = "Multiplayer playtest stop/end is disabled because StudioTestService:EndTest is currently broken for this flow. Manually close the Studio multiplayer test windows instead.";
     NETWORK_PROFILE_KEYS = [
       "InboundNetworkMinDelayMs",
       "OutboundNetworkMinDelayMs",
@@ -10834,7 +10822,7 @@ ${code}`
         }
         return { ok: false, roles: this._rolesForInstance(instanceId), timedOut: true, phase: lastPhase };
       }
-      async multiplayerPlaytest(action, numPlayers, target, testArgs, value, timeout, instance_id, force) {
+      async multiplayerPlaytest(action, numPlayers, target, testArgs, value, timeout, instance_id) {
         if (action !== "start" && action !== "status" && action !== "add_players" && action !== "leave_client" && action !== "end") {
           throw new Error("multiplayer_playtest requires action=start|status|add_players|leave_client|end");
         }
@@ -10843,7 +10831,6 @@ ${code}`
           return {
             phase: state.phase,
             roles: Array.isArray(state.peers) ? state.peers.map((peer) => peer.role).filter((role) => typeof role === "string") : [],
-            clientRoles: Array.isArray(state.clientRoles) ? state.clientRoles : [],
             playerCount: typeof state.playerCount === "number" ? state.playerCount : void 0,
             error: typeof state.error === "string" ? state.error : void 0
           };
@@ -10856,102 +10843,87 @@ ${code}`
           });
         }
         if (action === "start") {
-          if (force !== true) {
-            return this._textResult({
-              success: false,
-              action,
-              error: "multiplayer_force_required",
-              message: MULTIPLAYER_FORCE_REQUIRED_MESSAGE,
-              requiresForce: true,
-              manualCleanupRequired: true
-            });
-          }
-          const body = this._parseTextResult(await this.multiplayerTestStart(numPlayers, testArgs, timeout, instance_id, force));
-          const state = body.state && typeof body.state === "object" ? body.state : {};
-          const launched = body.success === true && body.ready === true;
+          const body2 = this._parseTextResult(await this.multiplayerTestStart(numPlayers, testArgs, timeout, instance_id));
+          const state = body2.state && typeof body2.state === "object" ? body2.state : {};
+          const launched = body2.success === true && body2.ready === true;
           return this._textResult(launched ? {
             success: true,
             action,
-            message: "Multiplayer playtest started. Stop/end is disabled; close the multiplayer test windows manually when finished.",
-            ready: true,
-            manualCleanupRequired: true,
-            roles: Array.isArray(body.roles) ? body.roles : void 0,
-            clientRoles: Array.isArray(state.clientRoles) ? state.clientRoles : void 0,
+            message: "Multiplayer playtest started.",
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0,
             playerCount: typeof state.playerCount === "number" ? state.playerCount : void 0
           } : {
             success: false,
             action,
-            error: body.error ?? body.wait?.error ?? "multiplayer_start_not_detected",
-            message: body.success === true ? "Multiplayer playtest start was requested, but MCP did not detect the required server/client peers before timeout. You may need to close the test windows manually." : body.message ?? "Multiplayer playtest did not start.",
-            manualCleanupRequired: body.startRequested === true || body.launched === true ? true : void 0,
-            roles: Array.isArray(body.roles) ? body.roles : void 0
+            error: body2.error ?? body2.wait?.error ?? "multiplayer_start_not_detected",
+            message: body2.success === true ? "Multiplayer playtest start was requested, but MCP did not detect the required server/client peers before timeout." : body2.message ?? "Multiplayer playtest did not start.",
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0
           });
         }
         if (action === "add_players") {
-          const body = this._parseTextResult(await this.multiplayerTestAddPlayers(numPlayers, timeout, instance_id));
-          const state = body.state && typeof body.state === "object" ? body.state : {};
-          const success = body.success === true && body.ready === true;
+          const body2 = this._parseTextResult(await this.multiplayerTestAddPlayers(numPlayers, timeout, instance_id));
+          const state = body2.state && typeof body2.state === "object" ? body2.state : {};
+          const success = body2.success === true && body2.ready === true;
           return this._textResult(success ? {
             success: true,
             action,
             message: "Players added.",
-            roles: Array.isArray(body.roles) ? body.roles : void 0,
-            clientRoles: Array.isArray(state.clientRoles) ? state.clientRoles : void 0,
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0,
             playerCount: typeof state.playerCount === "number" ? state.playerCount : void 0
           } : {
             success: false,
             action,
-            error: body.error ?? "add_players_failed",
-            message: body.success === true ? "Players did not finish joining before timeout." : body.message ?? "Players were not added.",
-            roles: Array.isArray(body.roles) ? body.roles : void 0
+            error: body2.error ?? "add_players_failed",
+            message: body2.success === true ? "Players did not finish joining before timeout." : body2.message ?? "Players were not added.",
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0
           });
         }
         if (action === "leave_client") {
-          const body = this._parseTextResult(await this.multiplayerTestLeaveClient(target ?? "client-1", timeout, instance_id));
-          return this._textResult(body.success === true && body.left === true ? {
+          const body2 = this._parseTextResult(await this.multiplayerTestLeaveClient(target ?? "client-1", timeout, instance_id));
+          return this._textResult(body2.success === true && body2.left === true ? {
             success: true,
             action,
             message: "Client left.",
-            roles: Array.isArray(body.roles) ? body.roles : void 0
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0
           } : {
             success: false,
             action,
-            error: body.error ?? "leave_client_failed",
-            message: body.message ?? "Client did not leave.",
-            roles: Array.isArray(body.roles) ? body.roles : void 0
+            error: body2.error ?? "leave_client_failed",
+            message: body2.message ?? "Client did not leave.",
+            roles: Array.isArray(body2.roles) ? body2.roles : void 0
           });
         }
-        return this._textResult({
+        const body = this._parseTextResult(await this.multiplayerTestEnd(value, timeout, instance_id));
+        return this._textResult(body.success === true && body.ended === true ? {
+          success: true,
+          action,
+          message: body.alreadyEnded === true ? "Multiplayer playtest already ended." : body.teardownConfirmed === false ? 'Multiplayer playtest end requested; teardown still in progress. Use multiplayer_playtest action="status" to confirm.' : "Multiplayer playtest ended.",
+          teardownConfirmed: body.teardownConfirmed === true
+        } : {
+          success: false,
           action,
-          ...multiplayerStopDisabledBody()
+          error: body.error ?? "end_failed",
+          message: body.message ?? "Multiplayer playtest did not end.",
+          roles: Array.isArray(body.roles) ? body.roles : void 0,
+          editDone: body.editDone === false ? false : void 0
         });
       }
-      async multiplayerTestStart(numPlayers, testArgs, timeout, instance_id, force) {
+      async multiplayerTestStart(numPlayers, testArgs, timeout, instance_id) {
         if (!Number.isInteger(numPlayers) || numPlayers < 1 || numPlayers > 8) {
           throw new Error("numPlayers must be an integer from 1 to 8");
         }
         const editTarget = this._resolveSingleTarget("edit", instance_id);
-        if (force !== true) {
-          return this._textResult({
-            success: false,
-            error: "multiplayer_force_required",
-            message: MULTIPLAYER_FORCE_REQUIRED_MESSAGE,
-            requiresForce: true,
-            manualCleanupRequired: true
-          });
-        }
         const existingRuntime = this._runtimeTargetsForEquivalentInstances(editTarget.instanceId);
         if (existingRuntime.length > 0) {
           const roles = this._rolesForEquivalentInstances(editTarget.instanceId);
           return this._textResult({
             success: false,
             error: "Multiplayer playtest already running.",
-            message: "A Studio runtime is already connected for this place. Close the existing playtest windows manually before starting another multiplayer playtest.",
+            message: "A Studio runtime is already connected for this place. End the existing playtest before starting another multiplayer playtest.",
             ready: true,
             timedOut: false,
             roles,
-            runtimeRoles: existingRuntime.map((target) => target.role),
-            manualCleanupRequired: true
+            runtimeRoles: existingRuntime.map((target) => target.role)
           });
         }
         const startedAt = Date.now();
@@ -10977,8 +10949,7 @@ ${code}`
               roles: wait.roles,
               state,
               error: success ? void 0 : wait.error ?? "multiplayer_start_not_detected",
-              message: success ? "Multiplayer Studio test started and runtime peers detected." : "Multiplayer Studio test start was requested, but MCP did not detect the required server/client peers before timeout. Close the multiplayer test windows manually if Studio launched them.",
-              manualCleanupRequired: success || response.success === true ? true : void 0,
+              message: success ? "Multiplayer Studio test started and runtime peers detected." : "Multiplayer Studio test start was requested, but MCP did not detect the required server/client peers before timeout.",
               startedAt
             })
           }]
@@ -11040,10 +11011,39 @@ ${code}`
         };
       }
       async multiplayerTestEnd(value, timeout, instance_id) {
-        void value;
-        void timeout;
-        void instance_id;
-        return this._textResult(multiplayerStopDisabledBody());
+        let serverTarget;
+        try {
+          serverTarget = this._resolveSingleTarget("server", instance_id);
+        } catch (err2) {
+          const instanceId = this._resolveInstanceIdOnly(instance_id);
+          const hasRuntime = this._rolesForInstance(instanceId).some((role) => role === "server" || /^client-\d+$/.test(role));
+          if (!hasRuntime) {
+            return this._textResult({
+              success: true,
+              ended: true,
+              alreadyEnded: true,
+              teardownConfirmed: true,
+              message: "No active multiplayer test to end (already ended)."
+            });
+          }
+          throw err2;
+        }
+        const response = await this.client.request("/api/multiplayer-test-end", { value: value ?? "ended_by_mcp" }, serverTarget.instanceId, serverTarget.role);
+        if (response?.error) {
+          return this._textResult(response);
+        }
+        const editDone = await this._waitForMultiplayerEditDone(serverTarget.instanceId, timeout ?? 30);
+        const wait = await this._waitForRuntimeRoles(serverTarget.instanceId, { noRuntime: true }, timeout ?? 30);
+        const state = await this._buildMultiplayerState(serverTarget.instanceId);
+        return this._textResult({
+          ...response,
+          ended: response.success === true,
+          teardownConfirmed: wait.ok,
+          editDone,
+          timedOut: wait.timedOut,
+          roles: wait.roles,
+          state
+        });
       }
       async getConnectedInstances() {
         const instances = this.bridge.getPublicInstances();
@@ -14107,7 +14107,7 @@ var init_definitions = __esm({
       {
         name: "multiplayer_playtest",
         category: "write",
-        description: 'Start or inspect a StudioTestService multiplayer playtest. Use action="start" with numPlayers and force=true only when you accept that MCP cannot stop it and you must manually close the multiplayer test windows afterward. action="status" inspects state, action="add_players" adds players, and action="leave_client" removes one client. action="end" is disabled for now and returns the StudioTestService:EndTest broken-API reason. Returns brief lifecycle status only; read script output with get_runtime_logs.',
+        description: 'Start, inspect, add players to, remove a client from, or end a StudioTestService multiplayer playtest. Use action="start" with numPlayers, action="status", action="add_players" with numPlayers, action="leave_client" with target="client-N", or action="end". Returns brief lifecycle status only; read script output with get_runtime_logs.',
         inputSchema: {
           type: "object",
           properties: {
@@ -14128,15 +14128,11 @@ var init_definitions = __esm({
               description: 'For action="start": JSON-compatible table passed to StudioTestService:GetTestArgs() on server and clients.'
             },
             value: {
-              description: 'Ignored while action="end" is disabled.'
-            },
-            force: {
-              type: "boolean",
-              description: 'Required for action="start". Pass true only if you understand StudioTestService:EndTest is broken in this flow and you will manually close the multiplayer test windows.'
+              description: 'For action="end": JSON-compatible value returned to the edit-side ExecuteMultiplayerTestAsync call.'
             },
             timeout: {
               type: "number",
-              description: "Max seconds to wait for start peer detection or action completion. Defaults to 30."
+              description: "Max seconds to wait for action completion. Defaults to 30."
             },
             instance_id: {
               type: "string",
@@ -15479,7 +15475,7 @@ part(0,2,0,2,1,1,"b")`,
       {
         name: "multiplayer_test_start",
         category: "write",
-        description: 'Deprecated. Use multiplayer_playtest with action="start" instead. Starts a StudioTestService multiplayer test only when force=true acknowledges that MCP cannot stop it and the test windows must be closed manually.',
+        description: 'Deprecated. Use multiplayer_playtest with action="start" instead. Starts a StudioTestService multiplayer test.',
         inputSchema: {
           type: "object",
           properties: {
@@ -15490,10 +15486,6 @@ part(0,2,0,2,1,1,"b")`,
             testArgs: {
               description: "JSON-compatible table passed to StudioTestService:GetTestArgs() on server and clients."
             },
-            force: {
-              type: "boolean",
-              description: "Required. Pass true only if you understand StudioTestService:EndTest is broken in this flow and you will manually close the multiplayer test windows."
-            },
             timeout: {
               type: "number",
               description: "Max seconds to wait for server + clients to register (default 30)."
@@ -15503,7 +15495,7 @@ part(0,2,0,2,1,1,"b")`,
               description: "Which connected Studio place to target. Required when multiple places are connected; omit when one. Use get_connected_instances to list available IDs."
             }
           },
-          required: ["numPlayers", "force"]
+          required: ["numPlayers"]
         }
       },
       {
@@ -15568,16 +15560,16 @@ part(0,2,0,2,1,1,"b")`,
       {
         name: "multiplayer_test_end",
         category: "write",
-        description: "Deprecated. Multiplayer StudioTestService stop/end is disabled for now because StudioTestService:EndTest is broken in this flow. This tool returns a disabled error and does not call EndTest.",
+        description: 'Deprecated. Use multiplayer_playtest with action="end" instead. Ends a running StudioTestService multiplayer test.',
         inputSchema: {
           type: "object",
           properties: {
             value: {
-              description: "Ignored while multiplayer stop/end is disabled."
+              description: "JSON-compatible value returned to the edit-side ExecuteMultiplayerTestAsync call."
             },
             timeout: {
               type: "number",
-              description: "Ignored while multiplayer stop/end is disabled."
+              description: "Max seconds to wait for runtime peers to disconnect (default 30)."
             },
             instance_id: {
               type: "string",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp-inspector",
-  "version": "2.23.0",
+  "version": "2.23.1",
   "description": "Read-only MCP server for inspecting and debugging Roblox Studio from AI coding tools",
   "main": "dist/index.js",
   "type": "module",

package/studio-plugin/INSTALLATION.md

@@ -42,19 +42,19 @@ Complete your AI assistant integration with this easy-to-install Studio plugin.
 
 ## Setup & Configuration
 
-### 1. Enable HTTP Requests (Required)
-**Game Settings** > **Security** > **"Allow HTTP Requests"**
+### Optional: Allow third-party Creator Store assets
 
-To preview or insert public Creator Store assets that you do not own, also
-enable **"Allow Loading Third Party Assets"** on the same Security page.
+To preview or insert public Creator Store assets that you do not own, enable
+**"Allow Loading Third Party Assets"** under
+**Game Settings** > **Security**.
 Roblox disables this setting by default.
 
-### 2. Activate the Plugin
+### 1. Activate the Plugin
 **Plugins toolbar** > Click **"MCP Server"** button
 - **Green status** = Connected and ready
 - **Red status** = Disconnected (normal until MCP server runs)
 
-### 3. Install MCP Server
+### 2. Install MCP Server
 Choose your AI assistant:
 
 **For Claude Code:**
@@ -116,10 +116,6 @@ If you encounter issues, you may need to run it through `cmd`. Update your confi
 - Restart Roblox Studio completely
 - Check Output window for error messages
 
-### "HTTP 403 Forbidden" Errors
-- Enable "Allow HTTP Requests" in Game Settings > Security
-- Verify MCP server is running (status should show connected)
-
 ### Plugin Shows "Disconnected"
 - **Normal behavior** when MCP server isn't running
 - Click "MCP Server" button to activate
@@ -132,10 +128,18 @@ If you encounter issues, you may need to run it through `cmd`. Update your confi
 
 ## Security & Privacy
 
-- **Local-only**: All communication stays on your machine
-- **No external servers**: Plugin only talks to localhost
-- **Read-only access**: Plugin extracts data but never modifies your place
-- **No data collection**: Your projects remain private
+- **Local bridge by default**: The plugin connects to `http://localhost:58741`
+  unless you explicitly configure another server URL.
+- **Explicit tool access**: The full plugin can modify a place when a write tool
+  is called. Install the Inspector edition when you need read-only access.
+- **Provider boundary**: Tool results pass through your configured MCP client.
+  Your AI provider's data-handling policy applies to content sent by that
+  client.
+
+See the repository's
+[configuration guide](../docs/configuration.md) for HTTP bridge authentication
+and network settings. Report suspected vulnerabilities through the
+[security policy](../SECURITY.md), not a public issue.
 
 ## Advanced Usage
 

package/studio-plugin/MCPInspectorPlugin.rbxmx

@@ -1550,9 +1550,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.23.0" is replaced with the package version at package time
+	-- "2.23.1" is replaced with the package version at package time
 	-- (scripts/build-plugin.mjs injectVersion), so a release bump also restamps.
-	return `{tostring(h)}-2.23.0`
+	return `{tostring(h)}-2.23.1`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10036,13 +10036,25 @@ local function multiplayerTestLeaveClient(_requestData)
 		localPlayer = localPlayer,
 	}
 end
-local function multiplayerTestEnd(_requestData)
+local function multiplayerTestEnd(requestData)
+	if not RunService:IsRunning() or not RunService:IsServer() then
+		return {
+			error = "multiplayer_test_end must be called on the running server peer. Route with target=server.",
+		}
+	end
+	local value = if requestData.value ~= nil then requestData.value else "ended_by_mcp"
+	local ok, result = pcall(function()
+		return StudioTestService:EndTest(value)
+	end)
+	if not ok then
+		return {
+			error = tostring(result),
+		}
+	end
 	return {
-		success = false,
-		error = "multiplayer_stop_disabled",
-		message = "Multiplayer playtest stop/end is disabled because StudioTestService:EndTest is currently broken for this flow. Manually close the Studio multiplayer test windows instead.",
-		reason = "StudioTestService:EndTest does not reliably end StudioTestService multiplayer sessions from MCP right now.",
-		manualCleanupRequired = true,
+		success = true,
+		message = "Multiplayer Studio test end requested.",
+		value = value,
 	}
 end
 return {
@@ -11080,7 +11092,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.23.0"
+local CURRENT_VERSION = "2.23.1"
 local PLUGIN_VARIANT = "inspector"
 local BASE_PORT = 58741
 local function createConnection(port)

package/studio-plugin/MCPPlugin.rbxmx

@@ -1550,9 +1550,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.23.0" is replaced with the package version at package time
+	-- "2.23.1" is replaced with the package version at package time
 	-- (scripts/build-plugin.mjs injectVersion), so a release bump also restamps.
-	return `{tostring(h)}-2.23.0`
+	return `{tostring(h)}-2.23.1`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10036,13 +10036,25 @@ local function multiplayerTestLeaveClient(_requestData)
 		localPlayer = localPlayer,
 	}
 end
-local function multiplayerTestEnd(_requestData)
+local function multiplayerTestEnd(requestData)
+	if not RunService:IsRunning() or not RunService:IsServer() then
+		return {
+			error = "multiplayer_test_end must be called on the running server peer. Route with target=server.",
+		}
+	end
+	local value = if requestData.value ~= nil then requestData.value else "ended_by_mcp"
+	local ok, result = pcall(function()
+		return StudioTestService:EndTest(value)
+	end)
+	if not ok then
+		return {
+			error = tostring(result),
+		}
+	end
 	return {
-		success = false,
-		error = "multiplayer_stop_disabled",
-		message = "Multiplayer playtest stop/end is disabled because StudioTestService:EndTest is currently broken for this flow. Manually close the Studio multiplayer test windows instead.",
-		reason = "StudioTestService:EndTest does not reliably end StudioTestService multiplayer sessions from MCP right now.",
-		manualCleanupRequired = true,
+		success = true,
+		message = "Multiplayer Studio test end requested.",
+		value = value,
 	}
 end
 return {
@@ -11080,7 +11092,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.23.0"
+local CURRENT_VERSION = "2.23.1"
 local PLUGIN_VARIANT = "main"
 local BASE_PORT = 58741
 local function createConnection(port)

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

@@ -322,13 +322,20 @@ function multiplayerTestLeaveClient(_requestData: Record<string, unknown>) {
 	};
 }
 
-function multiplayerTestEnd(_requestData: Record<string, unknown>) {
+function multiplayerTestEnd(requestData: Record<string, unknown>) {
+	if (!RunService.IsRunning() || !RunService.IsServer()) {
+		return { error: "multiplayer_test_end must be called on the running server peer. Route with target=server." };
+	}
+
+	const value = requestData.value !== undefined ? requestData.value : "ended_by_mcp";
+	const [ok, result] = pcall(() => StudioTestService.EndTest(value));
+	if (!ok) {
+		return { error: tostring(result) };
+	}
 	return {
-		success: false,
-		error: "multiplayer_stop_disabled",
-		message: "Multiplayer playtest stop/end is disabled because StudioTestService:EndTest is currently broken for this flow. Manually close the Studio multiplayer test windows instead.",
-		reason: "StudioTestService:EndTest does not reliably end StudioTestService multiplayer sessions from MCP right now.",
-		manualCleanupRequired: true,
+		success: true,
+		message: "Multiplayer Studio test end requested.",
+		value,
 	};
 }