@chrrxs/robloxstudio-mcp-inspector 2.21.0 → 2.22.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrrxs/robloxstudio-mcp-inspector",
-  "version": "2.21.0",
+  "version": "2.22.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/MCPInspectorPlugin.rbxmx

@@ -712,11 +712,11 @@ local function computeInstanceId()
 	return `anon:{fresh}`
 end
 local assignedRole
-local duplicateInstanceRole = false
 local hasVersionMismatch = false
 local lastVersionMismatchWarningKey
 local lastReadyInstanceId
 local readyFailureLogKeys = {}
+local retryingDuplicateReady = false
 -- Cache the published place name from MarketplaceService:GetProductInfo so
 -- /ready can carry a friendly identifier (e.g. "Natural Disasters") distinct
 -- from game.Name (the DataModel name, often "Place1" in edit). We only fetch
@@ -906,11 +906,12 @@ local function ensureIdentityWatcher(conn)
 	end
 end
 function sendReady(conn)
-	if duplicateInstanceRole then
-		return nil
-	end
 	local now = tick()
-	if now - lastReadyPostAt < 2 then
+	-- Normal identity refreshes stay conservatively throttled. Once a stale
+	-- predecessor causes 409, retry once per second so takeover follows the
+	-- server's short inactivity lease without another two seconds of jitter.
+	local readyInterval = if retryingDuplicateReady then 1 else 2
+	if now - lastReadyPostAt < readyInterval then
 		return nil
 	end
 	lastReadyPostAt = now
@@ -942,28 +943,39 @@ function sendReady(conn)
 		local readyRole = detectRole()
 		local readyLogKey = `{conn.serverUrl}|{instanceId}|{readyRole}`
 		if not readyOk then
+			local shouldLog = not (readyFailureLogKeys[readyLogKey] ~= nil)
 			readyFailureLogKeys[readyLogKey] = true
-			warn(`[robloxstudio-mcp] /ready failed for {instanceId}/{readyRole}: {HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`)
+			if shouldLog then
+				warn(`[robloxstudio-mcp] /ready failed for {instanceId}/{readyRole}: {HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`)
+			end
 			return nil
 		end
 		if not readyResult.Success then
 			local reason = HttpDiagnostics.formatRequestFailure(readyUrl, true, readyResult)
+			local shouldLog = not (readyFailureLogKeys[readyLogKey] ~= nil)
 			readyFailureLogKeys[readyLogKey] = true
-			-- 409 = duplicate_instance_role. Surface in UI and stop polling.
+			-- A predecessor can remain registered briefly when Studio exits before
+			-- its asynchronous /disconnect completes. Keep polling and retrying
+			-- /ready: the server will take us over once the predecessor has stopped
+			-- polling, while a genuinely active duplicate continues to hold routing.
 			if readyResult.StatusCode == 409 then
-				duplicateInstanceRole = true
-				conn.isActive = false
+				retryingDuplicateReady = true
 				local ui = UI.getElements()
-				ui.statusLabel.Text = "Duplicate instance"
-				ui.statusLabel.TextColor3 = Color3.fromRGB(239, 68, 68)
+				ui.statusLabel.Text = "Waiting for previous instance"
+				ui.statusLabel.TextColor3 = Color3.fromRGB(245, 158, 11)
 				ui.detailStatusLabel.Text = reason
-				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(239, 68, 68)
-				warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(245, 158, 11)
+				if shouldLog then
+					warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+				end
 				return nil
 			end
-			warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+			if shouldLog then
+				warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+			end
 			return nil
 		end
+		retryingDuplicateReady = false
 		local parseOk, readyData = pcall(function()
 			return HttpService:JSONDecode(readyResult.Body)
 		end)
@@ -1413,9 +1425,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.21.0" is replaced with the package version at package time
+	-- "2.22.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.21.0`
+	return `{tostring(h)}-2.22.1`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10449,24 +10461,30 @@ local function pushEntry(msg, t, ts)
 	totalBytes += bytes
 end
 local function seedRuntimeHistory()
-	if not RunService:IsRunning() then
-		return nil
-	end
 	local ok, history = pcall(function()
 		return LogService:GetLogHistory()
 	end)
 	if not ok then
 		return nil
 	end
+	local isEdit = not RunService:IsRunning()
+	-- GetLogHistory timestamps and DateTime.now() share Unix time, while
+	-- os.clock() is elapsed time for this Studio process. Their difference is
+	-- therefore the process launch boundary. Edit-mode history is filtered to
+	-- that boundary so startup errors from this launch are recovered without
+	-- importing history left by an earlier Studio process.
+	local processStartedAt = nowSec() - os.clock()
 	for _, entry in history do
 		local _message = entry.message
 		if not (type(_message) == "string") then
 			continue
 		end
-		local _exp = entry.message
-		local _exp_1 = entry.messageType
 		local _timestamp = entry.timestamp
-		pushEntry(_exp, _exp_1, if type(_timestamp) == "number" then entry.timestamp else nil)
+		local timestamp = if type(_timestamp) == "number" then entry.timestamp else nil
+		if isEdit and (timestamp == nil or timestamp < processStartedAt - 1) then
+			continue
+		end
+		pushEntry(entry.message, entry.messageType, timestamp)
 	end
 end
 local function install()
@@ -10477,9 +10495,9 @@ local function install()
 		return nil
 	end
 	installed = true
-	-- Play peers can emit startup logs before the plugin finishes loading.
-	-- Seed from per-DataModel LogHistory so get_runtime_logs can still see
-	-- those early messages; skip edit mode to avoid stale prior-session logs.
+	-- Every peer can emit startup logs before the plugin finishes loading.
+	-- Seed from per-DataModel LogHistory so get_runtime_logs can still see them;
+	-- edit history is bounded to the current Studio process above.
 	seedRuntimeHistory()
 	LogService.MessageOut:Connect(function(msg, t)
 		pushEntry(msg, t)
@@ -10745,7 +10763,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.21.0"
+local CURRENT_VERSION = "2.22.1"
 local PLUGIN_VARIANT = "inspector"
 local BASE_PORT = 58741
 local function createConnection(port)

package/studio-plugin/MCPPlugin.rbxmx

@@ -712,11 +712,11 @@ local function computeInstanceId()
 	return `anon:{fresh}`
 end
 local assignedRole
-local duplicateInstanceRole = false
 local hasVersionMismatch = false
 local lastVersionMismatchWarningKey
 local lastReadyInstanceId
 local readyFailureLogKeys = {}
+local retryingDuplicateReady = false
 -- Cache the published place name from MarketplaceService:GetProductInfo so
 -- /ready can carry a friendly identifier (e.g. "Natural Disasters") distinct
 -- from game.Name (the DataModel name, often "Place1" in edit). We only fetch
@@ -906,11 +906,12 @@ local function ensureIdentityWatcher(conn)
 	end
 end
 function sendReady(conn)
-	if duplicateInstanceRole then
-		return nil
-	end
 	local now = tick()
-	if now - lastReadyPostAt < 2 then
+	-- Normal identity refreshes stay conservatively throttled. Once a stale
+	-- predecessor causes 409, retry once per second so takeover follows the
+	-- server's short inactivity lease without another two seconds of jitter.
+	local readyInterval = if retryingDuplicateReady then 1 else 2
+	if now - lastReadyPostAt < readyInterval then
 		return nil
 	end
 	lastReadyPostAt = now
@@ -942,28 +943,39 @@ function sendReady(conn)
 		local readyRole = detectRole()
 		local readyLogKey = `{conn.serverUrl}|{instanceId}|{readyRole}`
 		if not readyOk then
+			local shouldLog = not (readyFailureLogKeys[readyLogKey] ~= nil)
 			readyFailureLogKeys[readyLogKey] = true
-			warn(`[robloxstudio-mcp] /ready failed for {instanceId}/{readyRole}: {HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`)
+			if shouldLog then
+				warn(`[robloxstudio-mcp] /ready failed for {instanceId}/{readyRole}: {HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`)
+			end
 			return nil
 		end
 		if not readyResult.Success then
 			local reason = HttpDiagnostics.formatRequestFailure(readyUrl, true, readyResult)
+			local shouldLog = not (readyFailureLogKeys[readyLogKey] ~= nil)
 			readyFailureLogKeys[readyLogKey] = true
-			-- 409 = duplicate_instance_role. Surface in UI and stop polling.
+			-- A predecessor can remain registered briefly when Studio exits before
+			-- its asynchronous /disconnect completes. Keep polling and retrying
+			-- /ready: the server will take us over once the predecessor has stopped
+			-- polling, while a genuinely active duplicate continues to hold routing.
 			if readyResult.StatusCode == 409 then
-				duplicateInstanceRole = true
-				conn.isActive = false
+				retryingDuplicateReady = true
 				local ui = UI.getElements()
-				ui.statusLabel.Text = "Duplicate instance"
-				ui.statusLabel.TextColor3 = Color3.fromRGB(239, 68, 68)
+				ui.statusLabel.Text = "Waiting for previous instance"
+				ui.statusLabel.TextColor3 = Color3.fromRGB(245, 158, 11)
 				ui.detailStatusLabel.Text = reason
-				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(239, 68, 68)
-				warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(245, 158, 11)
+				if shouldLog then
+					warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+				end
 				return nil
 			end
-			warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+			if shouldLog then
+				warn(`[robloxstudio-mcp] /ready rejected for {instanceId}/{readyRole}: {reason}`)
+			end
 			return nil
 		end
+		retryingDuplicateReady = false
 		local parseOk, readyData = pcall(function()
 			return HttpService:JSONDecode(readyResult.Body)
 		end)
@@ -1413,9 +1425,9 @@ local function computeBridgeStamp()
 	for i = 1, #combined do
 		h = (h * 33 + (string.byte(combined, i))) % 2147483647
 	end
-	-- "2.21.0" is replaced with the package version at package time
+	-- "2.22.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.21.0`
+	return `{tostring(h)}-2.22.1`
 end
 local BRIDGE_STAMP = computeBridgeStamp()
 local function setSource(scriptInst, source)
@@ -10449,24 +10461,30 @@ local function pushEntry(msg, t, ts)
 	totalBytes += bytes
 end
 local function seedRuntimeHistory()
-	if not RunService:IsRunning() then
-		return nil
-	end
 	local ok, history = pcall(function()
 		return LogService:GetLogHistory()
 	end)
 	if not ok then
 		return nil
 	end
+	local isEdit = not RunService:IsRunning()
+	-- GetLogHistory timestamps and DateTime.now() share Unix time, while
+	-- os.clock() is elapsed time for this Studio process. Their difference is
+	-- therefore the process launch boundary. Edit-mode history is filtered to
+	-- that boundary so startup errors from this launch are recovered without
+	-- importing history left by an earlier Studio process.
+	local processStartedAt = nowSec() - os.clock()
 	for _, entry in history do
 		local _message = entry.message
 		if not (type(_message) == "string") then
 			continue
 		end
-		local _exp = entry.message
-		local _exp_1 = entry.messageType
 		local _timestamp = entry.timestamp
-		pushEntry(_exp, _exp_1, if type(_timestamp) == "number" then entry.timestamp else nil)
+		local timestamp = if type(_timestamp) == "number" then entry.timestamp else nil
+		if isEdit and (timestamp == nil or timestamp < processStartedAt - 1) then
+			continue
+		end
+		pushEntry(entry.message, entry.messageType, timestamp)
 	end
 end
 local function install()
@@ -10477,9 +10495,9 @@ local function install()
 		return nil
 	end
 	installed = true
-	-- Play peers can emit startup logs before the plugin finishes loading.
-	-- Seed from per-DataModel LogHistory so get_runtime_logs can still see
-	-- those early messages; skip edit mode to avoid stale prior-session logs.
+	-- Every peer can emit startup logs before the plugin finishes loading.
+	-- Seed from per-DataModel LogHistory so get_runtime_logs can still see them;
+	-- edit history is bounded to the current Studio process above.
 	seedRuntimeHistory()
 	LogService.MessageOut:Connect(function(msg, t)
 		pushEntry(msg, t)
@@ -10745,7 +10763,7 @@ return {
         <Properties>
           <string name="Name">State</string>
           <string name="Source"><![CDATA[-- Compiled with roblox-ts v3.0.0
-local CURRENT_VERSION = "2.21.0"
+local CURRENT_VERSION = "2.22.1"
 local PLUGIN_VARIANT = "main"
 local BASE_PORT = 58741
 local function createConnection(port)

package/studio-plugin/src/modules/Communication.ts

@@ -53,11 +53,11 @@ function computeInstanceId(): string {
 }
 
 let assignedRole: string | undefined;
-let duplicateInstanceRole = false;
 let hasVersionMismatch = false;
 let lastVersionMismatchWarningKey: string | undefined;
 let lastReadyInstanceId: string | undefined;
 const readyFailureLogKeys = new Set<string>();
+let retryingDuplicateReady = false;
 
 // Cache the published place name from MarketplaceService:GetProductInfo so
 // /ready can carry a friendly identifier (e.g. "Natural Disasters") distinct
@@ -248,9 +248,12 @@ function ensureIdentityWatcher(conn: Connection): void {
 }
 
 function sendReady(conn: Connection): void {
-	if (duplicateInstanceRole) return; // stop retrying once the server has rejected us
 	const now = tick();
-	if (now - lastReadyPostAt < 2) return; // throttle to ≤1 /ready every 2s
+	// Normal identity refreshes stay conservatively throttled. Once a stale
+	// predecessor causes 409, retry once per second so takeover follows the
+	// server's short inactivity lease without another two seconds of jitter.
+	const readyInterval = retryingDuplicateReady ? 1 : 2;
+	if (now - lastReadyPostAt < readyInterval) return;
 	lastReadyPostAt = now;
 	const instanceId = computeInstanceId();
 	task.spawn(() => {
@@ -278,28 +281,39 @@ function sendReady(conn: Connection): void {
 		const readyRole = detectRole();
 		const readyLogKey = `${conn.serverUrl}|${instanceId}|${readyRole}`;
 		if (!readyOk) {
+			const shouldLog = !readyFailureLogKeys.has(readyLogKey);
 			readyFailureLogKeys.add(readyLogKey);
-			warn(`[robloxstudio-mcp] /ready failed for ${instanceId}/${readyRole}: ${HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`);
+			if (shouldLog) {
+				warn(`[robloxstudio-mcp] /ready failed for ${instanceId}/${readyRole}: ${HttpDiagnostics.formatRequestFailure(readyUrl, readyOk, readyResult)}`);
+			}
 			return;
 		}
 		if (!readyResult.Success) {
 			const reason = HttpDiagnostics.formatRequestFailure(readyUrl, true, readyResult);
+			const shouldLog = !readyFailureLogKeys.has(readyLogKey);
 			readyFailureLogKeys.add(readyLogKey);
-			// 409 = duplicate_instance_role. Surface in UI and stop polling.
+			// A predecessor can remain registered briefly when Studio exits before
+			// its asynchronous /disconnect completes. Keep polling and retrying
+			// /ready: the server will take us over once the predecessor has stopped
+			// polling, while a genuinely active duplicate continues to hold routing.
 			if (readyResult.StatusCode === 409) {
-				duplicateInstanceRole = true;
-				conn.isActive = false;
+				retryingDuplicateReady = true;
 				const ui = UI.getElements();
-				ui.statusLabel.Text = "Duplicate instance";
-				ui.statusLabel.TextColor3 = Color3.fromRGB(239, 68, 68);
+				ui.statusLabel.Text = "Waiting for previous instance";
+				ui.statusLabel.TextColor3 = Color3.fromRGB(245, 158, 11);
 				ui.detailStatusLabel.Text = reason;
-				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(239, 68, 68);
-				warn(`[robloxstudio-mcp] /ready rejected for ${instanceId}/${readyRole}: ${reason}`);
+				ui.detailStatusLabel.TextColor3 = Color3.fromRGB(245, 158, 11);
+				if (shouldLog) {
+					warn(`[robloxstudio-mcp] /ready rejected for ${instanceId}/${readyRole}: ${reason}`);
+				}
 				return;
 			}
-			warn(`[robloxstudio-mcp] /ready rejected for ${instanceId}/${readyRole}: ${reason}`);
+			if (shouldLog) {
+				warn(`[robloxstudio-mcp] /ready rejected for ${instanceId}/${readyRole}: ${reason}`);
+			}
 			return;
 		}
+		retryingDuplicateReady = false;
 		const [parseOk, readyData] = pcall(
 			() => HttpService.JSONDecode(readyResult.Body) as ReadyResponse,
 		);

package/studio-plugin/src/modules/RuntimeLogBuffer.ts

@@ -77,14 +77,21 @@ interface LogHistoryEntry {
 }
 
 function seedRuntimeHistory(): void {
-	if (!RunService.IsRunning()) return;
-
 	const [ok, history] = pcall(() => LogService.GetLogHistory() as LogHistoryEntry[]);
 	if (!ok) return;
+	const isEdit = !RunService.IsRunning();
+	// GetLogHistory timestamps and DateTime.now() share Unix time, while
+	// os.clock() is elapsed time for this Studio process. Their difference is
+	// therefore the process launch boundary. Edit-mode history is filtered to
+	// that boundary so startup errors from this launch are recovered without
+	// importing history left by an earlier Studio process.
+	const processStartedAt = nowSec() - os.clock();
 
 	for (const entry of history) {
 		if (!typeIs(entry.message, "string")) continue;
-		pushEntry(entry.message, entry.messageType, typeIs(entry.timestamp, "number") ? entry.timestamp : undefined);
+		const timestamp = typeIs(entry.timestamp, "number") ? entry.timestamp : undefined;
+		if (isEdit && (timestamp === undefined || timestamp < processStartedAt - 1)) continue;
+		pushEntry(entry.message, entry.messageType, timestamp);
 	}
 }
 
@@ -92,9 +99,9 @@ function install(): void {
 	if (installed) return;
 	if (!RunService.IsStudio()) return;
 	installed = true;
-	// Play peers can emit startup logs before the plugin finishes loading.
-	// Seed from per-DataModel LogHistory so get_runtime_logs can still see
-	// those early messages; skip edit mode to avoid stale prior-session logs.
+	// Every peer can emit startup logs before the plugin finishes loading.
+	// Seed from per-DataModel LogHistory so get_runtime_logs can still see them;
+	// edit history is bounded to the current Studio process above.
 	seedRuntimeHistory();
 	LogService.MessageOut.Connect((msg, t) => {
 		pushEntry(msg, t);