rbxstudio-mcp 2.3.2 → 2.4.0

package/studio-plugin/MCPPlugin.rbxmx

@@ -748,13 +748,6 @@ processRequest = function(request)
 		return handlers.getScriptSource(data)
 	elseif endpoint == "/api/set-script-source" then
 		return handlers.setScriptSource(data)
-		-- Partial script editing endpoints
-	elseif endpoint == "/api/edit-script-lines" then
-		return handlers.editScriptLines(data)
-	elseif endpoint == "/api/insert-script-lines" then
-		return handlers.insertScriptLines(data)
-	elseif endpoint == "/api/delete-script-lines" then
-		return handlers.deleteScriptLines(data)
 		-- Attribute endpoints
 	elseif endpoint == "/api/get-attribute" then
 		return handlers.getAttribute(data)
@@ -2498,237 +2491,9 @@ handlers.setScriptSource = function(requestData)
 	end
 end
 
--- Partial Script Editing: Edit specific lines
-handlers.editScriptLines = function(requestData)
-	local instancePath = requestData.instancePath
-	local startLine = requestData.startLine
-	local endLine = requestData.endLine
-	local newContent = requestData.newContent
-
-	if not instancePath or not startLine or not endLine or not newContent then
-		return { error = "Instance path, startLine, endLine, and newContent are required" }
-	end
-
-	-- Normalize escape sequences that may have been double-escaped
-	newContent = newContent:gsub("\\n", "\n")
-	newContent = newContent:gsub("\\t", "\t")
-	newContent = newContent:gsub("\\r", "\r")
-	newContent = newContent:gsub("\\\\", "\\")
-
-	local instance = getInstanceByPath(instancePath)
-	if not instance then
-		return { error = "Instance not found: " .. instancePath }
-	end
-
-	if not instance:IsA("LuaSourceContainer") then
-		return { error = "Instance is not a script-like object: " .. instance.ClassName }
-	end
-
-	local success, result = pcall(function()
-		local lines, hadTrailingNewline = splitLines(instance.Source)
-		local totalLines = #lines
-
-		if startLine < 1 or startLine > totalLines then
-			error("startLine out of range (1-" .. totalLines .. ")")
-		end
-		if endLine < startLine or endLine > totalLines then
-			error("endLine out of range (" .. startLine .. "-" .. totalLines .. ")")
-		end
-
-		-- Split new content into lines
-		local newLines = select(1, splitLines(newContent))
-
-		-- Build new source: lines before + new content + lines after
-		local resultLines = {}
-
-		-- Lines before the edit
-		for i = 1, startLine - 1 do
-			table.insert(resultLines, lines[i])
-		end
-
-		-- New content lines
-		for _, line in ipairs(newLines) do
-			table.insert(resultLines, line)
-		end
-
-		-- Lines after the edit
-		for i = endLine + 1, totalLines do
-			table.insert(resultLines, lines[i])
-		end
-
-		local newSource = joinLines(resultLines, hadTrailingNewline)
-
-		-- Use UpdateSourceAsync for editor compatibility
-		ScriptEditorService:UpdateSourceAsync(instance, function(oldContent)
-			return newSource
-		end)
-
-		ChangeHistoryService:SetWaypoint("Edit script lines " .. startLine .. "-" .. endLine .. ": " .. instance.Name)
-
-		return {
-			success = true,
-			instancePath = instancePath,
-			editedLines = { startLine = startLine, endLine = endLine },
-			linesRemoved = endLine - startLine + 1,
-			linesAdded = #newLines,
-			newLineCount = #resultLines,
-			message = "Script lines edited successfully"
-		}
-	end)
-
-	if success then
-		return result
-	else
-		return { error = "Failed to edit script lines: " .. tostring(result) }
-	end
-end
-
--- Partial Script Editing: Insert lines after a specific line
-handlers.insertScriptLines = function(requestData)
-	local instancePath = requestData.instancePath
-	local afterLine = requestData.afterLine or 0 -- 0 means insert at beginning
-	local newContent = requestData.newContent
-
-	if not instancePath or not newContent then
-		return { error = "Instance path and newContent are required" }
-	end
-
-	-- Normalize escape sequences that may have been double-escaped
-	newContent = newContent:gsub("\\n", "\n")
-	newContent = newContent:gsub("\\t", "\t")
-	newContent = newContent:gsub("\\r", "\r")
-	newContent = newContent:gsub("\\\\", "\\")
-
-	local instance = getInstanceByPath(instancePath)
-	if not instance then
-		return { error = "Instance not found: " .. instancePath }
-	end
-
-	if not instance:IsA("LuaSourceContainer") then
-		return { error = "Instance is not a script-like object: " .. instance.ClassName }
-	end
-
-	local success, result = pcall(function()
-		local lines, hadTrailingNewline = splitLines(instance.Source)
-		local totalLines = #lines
-
-		if afterLine < 0 or afterLine > totalLines then
-			error("afterLine out of range (0-" .. totalLines .. ")")
-		end
-
-		-- Split new content into lines
-		local newLines = select(1, splitLines(newContent))
-
-		-- Build new source
-		local resultLines = {}
-
-		-- Lines before insertion point
-		for i = 1, afterLine do
-			table.insert(resultLines, lines[i])
-		end
-
-		-- New content lines
-		for _, line in ipairs(newLines) do
-			table.insert(resultLines, line)
-		end
-
-		-- Lines after insertion point
-		for i = afterLine + 1, totalLines do
-			table.insert(resultLines, lines[i])
-		end
-
-		local newSource = joinLines(resultLines, hadTrailingNewline)
-
-		-- Use UpdateSourceAsync for editor compatibility
-		ScriptEditorService:UpdateSourceAsync(instance, function(oldContent)
-			return newSource
-		end)
-
-		ChangeHistoryService:SetWaypoint("Insert script lines after line " .. afterLine .. ": " .. instance.Name)
-
-		return {
-			success = true,
-			instancePath = instancePath,
-			insertedAfterLine = afterLine,
-			linesInserted = #newLines,
-			newLineCount = #resultLines,
-			message = "Script lines inserted successfully"
-		}
-	end)
-
-	if success then
-		return result
-	else
-		return { error = "Failed to insert script lines: " .. tostring(result) }
-	end
-end
-
--- Partial Script Editing: Delete specific lines
-handlers.deleteScriptLines = function(requestData)
-	local instancePath = requestData.instancePath
-	local startLine = requestData.startLine
-	local endLine = requestData.endLine
-
-	if not instancePath or not startLine or not endLine then
-		return { error = "Instance path, startLine, and endLine are required" }
-	end
-
-	local instance = getInstanceByPath(instancePath)
-	if not instance then
-		return { error = "Instance not found: " .. instancePath }
-	end
-
-	if not instance:IsA("LuaSourceContainer") then
-		return { error = "Instance is not a script-like object: " .. instance.ClassName }
-	end
-
-	local success, result = pcall(function()
-		local lines, hadTrailingNewline = splitLines(instance.Source)
-		local totalLines = #lines
-
-		if startLine < 1 or startLine > totalLines then
-			error("startLine out of range (1-" .. totalLines .. ")")
-		end
-		if endLine < startLine or endLine > totalLines then
-			error("endLine out of range (" .. startLine .. "-" .. totalLines .. ")")
-		end
-
-		-- Build new source without the deleted lines
-		local resultLines = {}
-
-		for i = 1, startLine - 1 do
-			table.insert(resultLines, lines[i])
-		end
-
-		for i = endLine + 1, totalLines do
-			table.insert(resultLines, lines[i])
-		end
-
-		local newSource = joinLines(resultLines, hadTrailingNewline)
-
-		-- Use UpdateSourceAsync for editor compatibility
-		ScriptEditorService:UpdateSourceAsync(instance, function(oldContent)
-			return newSource
-		end)
-
-		ChangeHistoryService:SetWaypoint("Delete script lines " .. startLine .. "-" .. endLine .. ": " .. instance.Name)
-
-		return {
-			success = true,
-			instancePath = instancePath,
-			deletedLines = { startLine = startLine, endLine = endLine },
-			linesDeleted = endLine - startLine + 1,
-			newLineCount = #resultLines,
-			message = "Script lines deleted successfully"
-		}
-	end)
-
-	if success then
-		return result
-	else
-		return { error = "Failed to delete script lines: " .. tostring(result) }
-	end
-end
+-- Legacy line-based partial script editors removed — superseded by the
+-- string-based editScript handler below. Line numbers shift after each
+-- edit, which makes line-based editing unreliable for AI workflows.
 
 -- Attribute Tools: Get a single attribute
 handlers.getAttribute = function(requestData)