@adiholick/nvstudio-mcp 2.1.8 → 2.2.0

package/studio-plugin/Main.server.lua

@@ -31,6 +31,7 @@ local ctx = {
 	recentLogs    = {},
 	HttpService   = HttpService,
 	studioId      = STUDIO_ID,
+	getInstanceFromPath = nil -- akan di-bind di bawah
 }
 local MAX_LOGS           = 50
 local isRunning          = false  -- apakah loop aktif
@@ -43,6 +44,7 @@ local serverReachable    = false  -- apakah server Node.js reachable
 local RECONNECT_DELAY_INITIAL = 1.5
 local RECONNECT_DELAY_MAX     = 1.5
 local reconnectDelay          = RECONNECT_DELAY_INITIAL
+local currentLoopId           = nil
 
 -- ── Skills (ModuleScript) ─────────────────────────────────────────────────────
 local skills = {}
@@ -122,21 +124,41 @@ LogService.MessageOut:Connect(addSystemLog)
 
 -- ── Helper: Resolve path ke Instance ─────────────────────────────────────────
 local function getInstanceFromPath(pathString)
+	if type(pathString) ~= "string" or pathString == "" then return nil end
 	local parts   = string.split(pathString, ".")
+	if #parts == 0 then return nil end
 	local current = game
 	if parts[1] == "game" then table.remove(parts, 1) end
 	pcall(function()
 		local svc = game:GetService(parts[1])
 		if svc then current = svc; table.remove(parts, 1) end
 	end)
-	for _, part in ipairs(parts) do
+	local i = 1
+	while i <= #parts do
+		local part = parts[i]
 		local found = current:FindFirstChild(part)
+		
+		-- Jika tidak ketemu, coba gabungkan dengan part berikutnya (untuk nama yang mengandung titik)
+		local combinedPart = part
+		local lookAhead = i + 1
+		while not found and lookAhead <= #parts do
+			combinedPart = combinedPart .. "." .. parts[lookAhead]
+			found = current:FindFirstChild(combinedPart)
+			if found then
+				i = lookAhead
+			end
+			lookAhead = lookAhead + 1
+		end
+		
 		if not found then return nil end
 		current = found
+		i = i + 1
 	end
 	return current
 end
 
+ctx.getInstanceFromPath = getInstanceFromPath
+
 -- ── Proses Task dari Server ───────────────────────────────────────────────────
 local function processTask(taskData)
 	if not taskData or not taskData.id or not taskData.command then return nil end
@@ -147,7 +169,11 @@ local function processTask(taskData)
 
 	local targetInstance = getInstanceFromPath(taskData.target)
 	if not targetInstance then
-		return { id = taskData.id, status = "error", error = "Target path tidak ditemukan: " .. tostring(taskData.target) }
+		if taskData.target == "" or taskData.target == "nil" then
+			targetInstance = game
+		else
+			return { id = taskData.id, status = "error", error = "Target path tidak ditemukan: " .. tostring(taskData.target) }
+		end
 	end
 
 	local skillFunc = skills[taskData.command]
@@ -155,8 +181,16 @@ local function processTask(taskData)
 		return { id = taskData.id, status = "error", error = "Command tidak dikenali: " .. taskData.command }
 	end
 
-	local result = skillFunc(targetInstance, taskData.data, ctx, taskData.target)
-	result.id    = taskData.id
+	local ok, result = pcall(function()
+		return skillFunc(targetInstance, taskData.data, ctx, taskData.target)
+	end)
+	
+	if not ok then
+		result = { status = "error", error = "Internal Plugin Error: " .. tostring(result) }
+	end
+	
+	result = result or { status = "success", result = "Eksekusi selesai (Tidak ada return value)" }
+	result.id = taskData.id
 
 	if ui then
 		local color = result.status == "success" and Color3.fromRGB(100, 255, 100) or Color3.fromRGB(255, 100, 100)
@@ -188,24 +222,11 @@ local function sendResponse(taskId, resultData)
 	end)
 end
 
--- ── Update UI berdasarkan state ───────────────────────────────────────────────
-local function updateUIStatus()
-	if not ui then return end
-	if not serverReachable then
-		ui:setStatus("waiting")
-	elseif mcpAgentActive then
-		ui:setStatus("connected")
-	else
-		-- Server reachable tapi tidak ada AI agent
-		-- Tampilkan sebagai connected (karena server sse bridge aktif)
-		ui:setStatus("server_connected")
-	end
-end
-
 -- ── Proses satu event dari SSE stream ────────────────────────────────────────
 local function handleSSELine(line)
 	-- Format SSE: "data: {...json...}"
 	local payload = line
+	if line:sub(1, 6) == "event:" then return end
 	if line:sub(1, 5) == "data:" then
 		payload = line:sub(6):gsub("^%s+", ""):gsub("%s+$", "")
 	end
@@ -227,8 +248,12 @@ local function handleSSELine(line)
 		local changed = (mcpAgentActive ~= newActive)
 		mcpAgentActive = newActive
 		if ui then
-			-- Jaga status tetap Connected selama bridge server hidup
-			ui:setStatus("connected")
+			if mcpAgentActive then
+				ui:setStatus("connected")
+			else
+				ui:setStatus("server_connected")
+			end
+			
 			if newActive and changed then
 				ui:addLog("[System] ✅ IDE Agent (Antigravity) aktif.", Color3.fromRGB(100, 255, 100))
 			elseif not newActive and changed then
@@ -273,13 +298,16 @@ function startConnectionLoop()
 	isManualDisconnect = false
 	reconnectDelay = RECONNECT_DELAY_INITIAL
 
+	local loopId = HttpService:GenerateGUID(false)
+	currentLoopId = loopId
+
 	if ui then
 		ui:addLog("[System] NVStudio MCP v2.1.8 — Menghubungkan ke server...", Color3.fromRGB(200, 200, 200))
 	end
 
 	local consecutiveErrors = 0
 
-	while isRunning and not isManualDisconnect do
+	while isRunning and not isManualDisconnect and currentLoopId == loopId do
 		-- Stream / polling snapshot dari server
 		local streamOk, streamRaw = pcall(function()
 			return HttpService:GetAsync(
@@ -295,7 +323,7 @@ function startConnectionLoop()
 			if not serverReachable then
 				serverReachable = true
 				if ui then
-					ui:setStatus("connected")
+					ui:setStatus(mcpAgentActive and "connected" or "server_connected")
 					ui:addLog("[System] ✅ Server MCP terhubung & siap.", Color3.fromRGB(100, 255, 150))
 				end
 			end

package/studio-plugin/create_instance.lua

@@ -7,18 +7,28 @@ return function(targetInstance, data, ctx, targetPath)
     local newSuccess, newInst = pcall(function()
         local inst = Instance.new(config.ClassName)
         inst.Parent = targetInstance
-        
-        if config.Properties then
-            for prop, val in pairs(config.Properties) do
-                pcall(function() inst[prop] = val end)
-            end
-        end
         return inst
     end)
     
-    if newSuccess then
-        return { status = "success", result = "Instance '"..config.ClassName.."' berhasil dibuat di " .. targetPath }
-    else
-        return { status = "error", error = "Gagal membuat Instance. Pastikan ClassName valid." }
+    if not newSuccess then
+        return { status = "error", error = "Gagal membuat Instance: " .. tostring(newInst) }
+    end
+
+    local propResultMsg = ""
+    if config.Properties and next(config.Properties) then
+        local setPropsFunc = require(script.Parent:FindFirstChild("set_properties"))
+        if setPropsFunc then
+            local propData = ctx.HttpService:JSONEncode(config.Properties)
+            local propRes = setPropsFunc(newInst, propData, ctx, "")
+            if propRes.status == "error" then
+                propResultMsg = " (Namun ada error properti: " .. propRes.result .. ")"
+            elseif propRes.result then
+                propResultMsg = " (" .. propRes.result .. ")"
+            end
+        else
+            propResultMsg = " (Gagal meload set_properties)"
+        end
     end
+    
+    return { status = "success", result = "Instance '"..config.ClassName.."' berhasil dibuat di " .. targetPath .. propResultMsg }
 end

package/studio-plugin/delete_instance.lua

@@ -1,4 +1,8 @@
 return function(targetInstance, data, ctx, targetPath)
+    if targetInstance == game or targetInstance.Parent == game then
+        return { status = "error", error = "Tidak dapat menghapus root DataModel atau Service utama!" }
+    end
+
     -- Guardrail: Cegah penghapusan Service utama
     local protectedServices = {
         "Workspace", "Players", "Lighting", "MaterialService", "NetworkClient", 
@@ -19,6 +23,11 @@ return function(targetInstance, data, ctx, targetPath)
     local className = targetInstance.ClassName
     local parentName = targetInstance.Parent and targetInstance.Parent.Name or "nil"
     
+    local childrenCount = #targetInstance:GetDescendants()
+    if childrenCount > 10 and data ~= "force" then
+        return { status = "error", error = "GUARDRAIL: Instance '" .. name .. "' memiliki " .. childrenCount .. " descendants. Jika Anda yakin ingin menghapus seluruhnya beserta isinya, sertakan data='force'." }
+    end
+    
     local success, err = pcall(function()
         targetInstance:Destroy()
     end)

package/studio-plugin/execute_luau.lua

@@ -10,7 +10,7 @@ return function(targetInstance, data, ctx, targetPath)
     local wrappedCode = "return function()\n" .. code .. "\nend"
     
     local moduleScript = Instance.new("ModuleScript")
-    moduleScript.Name = "AI_Execute_Temp"
+    moduleScript.Name = "AI_Execute_" .. HttpService:GenerateGUID(false):gsub("-", "")
     moduleScript.Source = wrappedCode
     
     -- Taruh di ReplicatedStorage sementara agar bisa di-require
@@ -21,10 +21,17 @@ return function(targetInstance, data, ctx, targetPath)
     local isFinished = false
     
     -- Eksekusi asinkron dengan batas waktu (timeout) 5 detik
-    task.spawn(function()
+    local logs = {}
+    
+    local execThread = task.spawn(function()
         local reqSuccess, func = pcall(function() return require(moduleScript) end)
         if reqSuccess and type(func) == "function" then
+            local LogService = game:GetService("LogService")
+            local connection = LogService.MessageOut:Connect(function(msg, msgType)
+                table.insert(logs, msg)
+            end)
             local execSuccess, res = pcall(func)
+            pcall(function() connection:Disconnect() end)
             if execSuccess then
                 resultData = res
             else
@@ -42,20 +49,31 @@ return function(targetInstance, data, ctx, targetPath)
         timeWaited = timeWaited + task.wait()
     end
     
+
+    if not isFinished then
+        pcall(function() task.cancel(execThread) end)
+    end
+    
     -- Pembersihan (Wajib dihancurkan)
     pcall(function() moduleScript:Destroy() end)
     
     if not isFinished then
-        return { status = "error", error = "Timeout 5 detik terlampaui. Kemungkinan ada infinite loop tanpa task.wait()." }
+        return { status = "error", error = "Timeout 5 detik terlampaui. Thread mungkin tetap berjalan di background jika terjebak dalam infinite loop tanpa task.wait(). PASTIKAN menambahkan task.wait() di dalam loop komputasi berat." }
     end
     
     if execError then
-        return { status = "error", error = execError }
+        return { status = "error", error = execError, logs = #logs > 0 and logs or nil }
     end
     
+    local returnPayload = { status = "success" }
+    if #logs > 0 then
+        returnPayload.logs = logs
+    end
+
     -- Cegah return nil error
     if resultData == nil then
-        return { status = "success", result = "Eksekusi berhasil (Tidak ada nilai kembalian)." }
+        returnPayload.result = "Eksekusi berhasil (Tidak ada nilai kembalian)."
+        return returnPayload
     end
     
     -- JSON Encode untuk mencegah raw Instance return crash
@@ -64,7 +82,8 @@ return function(targetInstance, data, ctx, targetPath)
     end)
     
     if serializeSuccess then
-        return { status = "success", result = HttpService:JSONDecode(jsonRes) } -- decode kembali agar format aslinya diteruskan oleh router
+        returnPayload.result = HttpService:JSONDecode(jsonRes) -- decode kembali agar format aslinya diteruskan oleh router
+        return returnPayload
     else
         return { status = "error", error = "Gagal menserialisasi hasil (Dilarang mengembalikan raw Roblox Instance): " .. tostring(jsonRes) }
     end

package/studio-plugin/generate_terrain.lua

@@ -6,7 +6,8 @@ return function(targetInstance, data, ctx, targetPath)
     
     local fillSuccess, err = pcall(function()
         local terrain = game.Workspace.Terrain
-        local materialEnum = Enum.Material[config.Material] or Enum.Material.Grass
+        local matOk, materialEnum = pcall(function() return Enum.Material[config.Material] end)
+        if not matOk then materialEnum = Enum.Material.Grass end
         
         local size = Vector3.new(config.Size[1], config.Size[2], config.Size[3])
         local position = Vector3.new(config.Position[1], config.Position[2], config.Position[3])

package/studio-plugin/get_children.lua

@@ -1,6 +1,12 @@
 return function(targetInstance, data, ctx, targetPath)
     local children = {}
-    for _, child in ipairs(targetInstance:GetChildren()) do
+    local allChildren = targetInstance:GetChildren()
+    local MAX_LIMIT = 200
+    for i, child in ipairs(allChildren) do
+        if i > MAX_LIMIT then
+            table.insert(children, "... (Terpotong, " .. tostring(#allChildren - MAX_LIMIT) .. " children lainnya tidak ditampilkan)")
+            break
+        end
         table.insert(children, child.Name .. " (" .. child.ClassName .. ")")
     end
     return { status = "success", result = children }

package/studio-plugin/get_full_tree.lua

@@ -0,0 +1,46 @@
+return function(targetInstance, data, ctx, targetPath)
+    local maxDepth = 2
+    if data and data ~= "" then
+        pcall(function() maxDepth = tonumber(data) or 2 end)
+    end
+    
+    local totalNodes = 0
+    local HARD_LIMIT = 2000 -- safeguard
+    
+    local hardLimitReached = false
+    
+    local function traverse(inst, currentDepth)
+        if hardLimitReached then return nil end
+        totalNodes = totalNodes + 1
+        local node = { name = inst.Name, className = inst.ClassName }
+        
+        if totalNodes >= HARD_LIMIT then
+            hardLimitReached = true
+            return { name = "... (terpotong, melebihi limit " .. HARD_LIMIT .. " nodes)", className = "Warning" }
+        end
+        
+        if currentDepth >= maxDepth then
+            local childCount = #inst:GetChildren()
+            if childCount > 0 then
+                node.children = "... (" .. childCount .. " children tersembunyi)"
+            end
+            return node
+        end
+        
+        local children = inst:GetChildren()
+        if #children > 0 then
+            node.children = {}
+            for _, child in ipairs(children) do
+                local childNode = traverse(child, currentDepth + 1)
+                if childNode then
+                    table.insert(node.children, childNode)
+                end
+            end
+        end
+        
+        return node
+    end
+    
+    local tree = traverse(targetInstance, 0)
+    return { status = "success", result = tree }
+end

package/studio-plugin/insert_asset.lua

@@ -1,6 +1,7 @@
 return function(targetInstance, data, ctx, targetPath)
     local InsertService = game:GetService("InsertService")
-    local assetId = tonumber(data)
+    local rawId = tostring(data or ""):gsub("rbxassetid://", ""):gsub("%s+", "")
+    local assetId = tonumber(rawId)
     if not assetId then
         return { status = "error", error = "Asset ID harus berupa angka yang valid." }
     end
@@ -10,8 +11,12 @@ return function(targetInstance, data, ctx, targetPath)
     end)
     
     if success and model then
-        model.Parent = game.Workspace
-        return { status = "success", result = "Aset berhasil dimasukkan ke Workspace." }
+        if targetInstance == game or targetInstance.ClassName == "DataModel" then
+            model.Parent = game.Workspace
+        else
+            model.Parent = targetInstance
+        end
+        return { status = "success", result = "Aset berhasil dimasukkan ke " .. model.Parent.Name .. "." }
     else
         return { status = "error", error = "Gagal memuat aset. Pastikan Asset ID benar dan akun/plugin Anda memiliki izin (Ownership/Public)." }
     end