@el4cteo/rbx-studio-mcp 0.4.1 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@el4cteo/rbx-studio-mcp",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "MCP server for Roblox Studio. 29 tools, push-based SSE bridge, editor-safe script edits, one-step undo.",
   "type": "module",
   "license": "MIT",
@@ -45,11 +45,11 @@
     "ai-agent"
   ],
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.22.0",
-    "zod": "^4.4.3"
+    "@modelcontextprotocol/sdk": "^1.30.0",
+    "zod": "^4.5.4"
   },
   "devDependencies": {
-    "@types/node": "^26.2.0",
+    "@types/node": "^26.4.1",
     "typescript": "^7.0.2"
   },
   "author": "EL4CTEO",

package/plugin/src/Config.luau

@@ -9,7 +9,7 @@
 
 local Config = {}
 
-Config.PLUGIN_VERSION = "0.4.1"
+Config.PLUGIN_VERSION = "0.4.2"
 
 -- 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

package/plugin/src/init.server.luau

@@ -37,6 +37,14 @@ local Api = require(script.handlers.Api)
 local Scripts = require(script.handlers.Scripts)
 local Session = require(script.handlers.Session)
 
+--[[
+	How long a closed panel has to stay closed before that is written down.
+
+	Long enough to outlast Studio tearing a place down, short enough that a user
+	who closes the panel and quits immediately still has it remembered.
+]]
+local WIDGET_CLOSE_GRACE = 1
+
 local SETTING_PORT = "port"
 local SETTING_AUTOCONNECT = "autoConnect"
 local SETTING_FORCE_POLL = "forcePoll"
@@ -243,10 +251,40 @@ if RunService:IsEdit() then
 		unloading = true
 	end)
 
+	--[[
+		Only a close the user actually asked for is remembered.
+
+		Opening is recorded at once -- there is no teardown that opens a panel,
+		so a `true` is always the user. A `false` is not: Studio disables the
+		widget on its way out of a place, and `plugin.Unloading` is not
+		guaranteed to have fired by then, so the `unloading` flag alone lost the
+		race often enough that people found the panel closed on their next
+		launch with no memory of closing it. That is the "it does not open any
+		more" report, and it was self-inflicted and permanent -- once `false` is
+		stored, nothing writes `true` again until the panel is opened by hand.
+
+		So a close has to survive a moment before it counts. A teardown never
+		does: the DataModel is gone long before the delay is up, taking the
+		pending write with it. A user clicking the X does, and a user who
+		re-opens within the delay cancels it, which is also correct.
+	]]
+	local closeGeneration = 0
 	widget:GetPropertyChangedSignal("Enabled"):Connect(function()
-		if not unloading then
-			plugin:SetSetting(SETTING_WIDGET_OPEN, widget.Enabled)
+		if unloading then
+			return
+		end
+		closeGeneration += 1
+		if widget.Enabled then
+			plugin:SetSetting(SETTING_WIDGET_OPEN, true)
+			return
 		end
+		local generation = closeGeneration
+		task.delay(WIDGET_CLOSE_GRACE, function()
+			if unloading or generation ~= closeGeneration or widget.Enabled then
+				return
+			end
+			plugin:SetSetting(SETTING_WIDGET_OPEN, false)
+		end)
 	end)
 
 	-- Only a real, sane size, and only while the panel is up: a hidden or

package/scripts/install-plugin.mjs

@@ -7,7 +7,7 @@
  * Usage: node scripts/install-plugin.mjs
  */
 import { execFileSync } from "node:child_process";
-import { copyFileSync, existsSync, mkdirSync } from "node:fs";
+import { copyFileSync, existsSync, mkdirSync, renameSync, rmSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
@@ -36,4 +36,27 @@ execFileSync(process.execPath, [join(root, "scripts", "build-plugin.mjs")], { st
 
 const target = pluginsDir();
 if (!existsSync(target)) mkdirSync(target, { recursive: true });
-copyFileSync(built, join(target, "StudioMCP.rbxmx"));
+
+/*
+ * Copied beside the target and renamed over it, never written in place.
+ *
+ * Studio watches this directory and reloads the moment the file changes, so an
+ * in-place copy hands it whatever is on disk at that instant -- which, for a
+ * file of this size, is regularly half of one build and none of the next. The
+ * plugin then fails to parse and Studio drops it silently: no toolbar button,
+ * nothing in the plugin list, and no error saying why. It reads as the plugin
+ * randomly not being installed, and re-running the install "fixes" it, which is
+ * exactly what you would expect from a race.
+ *
+ * Rename within one directory is atomic, so Studio sees the old file or the new
+ * one and never the seam between them.
+ */
+const destination = join(target, "StudioMCP.rbxmx");
+const staged = join(target, "StudioMCP.rbxmx.incoming");
+try {
+  copyFileSync(built, staged);
+  renameSync(staged, destination);
+} catch (cause) {
+  rmSync(staged, { force: true });
+  throw cause;
+}