dominus-cli 0.5.1 → 0.5.3

package/plugin/src/Properties.lua

@@ -14,6 +14,70 @@ local IGNORED_PROPERTIES = {
 	Archivable = true,
 }
 
+-- Properties that are read-only / computed / redundant — strip in compact mode
+local COMPACT_SKIP = {
+	-- Read-only computed
+	AbsolutePosition = true,
+	AbsoluteSize = true,
+	AbsoluteRotation = true,
+	AbsoluteCanvasSize = true,
+	TextBounds = true,
+	TextFits = true,
+	IsLoaded = true,
+	ContentText = true,
+	LocalizedText = true,
+	GuiState = true,
+	-- Deprecated / redundant (BrickColor duplicates Color3, Font duplicates FontFace)
+	BackgroundColor = true,
+	BorderColor = true,
+	TextColor = true,
+	Font = true,
+	FontSize = true,
+	-- Internal / security
+	Capabilities = true,
+	Sandboxed = true,
+	archivable = true,
+	className = true,
+	Localize = true,
+	-- Selection navigation (almost always nil/default)
+	NextSelectionDown = true,
+	NextSelectionLeft = true,
+	NextSelectionRight = true,
+	NextSelectionUp = true,
+	SelectionBehaviorDown = true,
+	SelectionBehaviorLeft = true,
+	SelectionBehaviorRight = true,
+	SelectionBehaviorUp = true,
+	SelectionGroup = true,
+	SelectionOrder = true,
+	SelectionImageObject = true,
+	-- Rarely useful defaults
+	Draggable = true,
+	InputSink = true,
+	RootLocalizationTable = true,
+	AutoLocalize = true,
+	OpenTypeFeatures = true,
+	OpenTypeFeaturesError = true,
+	Interactable = true,
+	Style = true,
+	BorderMode = true,
+	SizeConstraint = true,
+}
+
+-- Default values that are not worth reporting in compact mode
+local COMPACT_DEFAULT_VALUES = {
+	["0"] = true,
+	["1"] = true,
+	["false"] = true,
+	["nil"] = true,
+	[""] = true,
+	["Enum.AutomaticSize.None"] = true,
+	["Enum.TextTruncate.None"] = true,
+	["Enum.TextDirection.Auto"] = true,
+	["Enum.ResamplerMode.Default"] = true,
+	["Enum.ScaleType.Stretch"] = true,
+}
+
 -- Cache property type maps per class to avoid repeated reflection calls
 local typeCache = {}
 
@@ -88,12 +152,54 @@ function Properties.get(path)
 	return { properties = properties }
 end
 
-function Properties.getDescendants(path)
+function Properties.getDescendants(path, compact)
 	local rootInstance = Explorer.resolveInstance(path)
 	if not rootInstance then
 		return { success = false, error = "Instance not found: " .. path }
 	end
 
+	-- In compact mode, skip noisy/default properties to reduce payload size
+	local shouldSkip = compact
+			and function(propName, valueStr)
+				if COMPACT_SKIP[propName] then
+					return true
+				end
+				-- Skip nil-valued properties
+				if valueStr == "nil" then
+					return true
+				end
+				return false
+			end
+		or function()
+			return false
+		end
+
+	-- Extra compact filter: skip properties that are clearly defaults and non-informative
+	local isBoringDefault = compact
+			and function(propName, valueStr)
+				-- These specific properties at default values are noise
+				local boringAtDefault = {
+					Active = "false",
+					Selectable = "false",
+					ClipsDescendants = "false",
+					TextWrap = "false",
+					TextWrapped = "false",
+					RichText = "false",
+					TextScaled = "false",
+					Rotation = "0",
+					LayoutOrder = "0",
+					MaxVisibleGraphemes = "-1",
+					SliceScale = "1",
+					LineHeight = "1",
+					BorderSizePixel = "1",
+					Visible = "true",
+				}
+				return boringAtDefault[propName] == valueStr
+			end
+		or function()
+			return false
+		end
+
 	local function getProps(instance)
 		local properties = {}
 
@@ -101,17 +207,20 @@ function Properties.getDescendants(path)
 			local info = {}
 			local propData = ReflectionService:GetPropertiesOfClass(instance.ClassName)
 			for _, prop in propData do
-				if not IGNORED_PROPERTIES[prop.Name] then
+				if not IGNORED_PROPERTIES[prop.Name] and not shouldSkip(prop.Name, "") then
 					local valueOk, value = pcall(function()
 						return instance[prop.Name]
 					end)
 					if valueOk then
-						table.insert(info, {
-							name = prop.Name,
-							value = tostring(value),
-							type = prop.ValueType and tostring(prop.ValueType) or typeof(value),
-							category = prop.Category or "Other",
-						})
+						local valueStr = tostring(value)
+						if not shouldSkip(prop.Name, valueStr) and not isBoringDefault(prop.Name, valueStr) then
+							table.insert(info, {
+								name = prop.Name,
+								value = valueStr,
+								type = prop.ValueType and tostring(prop.ValueType) or typeof(value),
+								category = prop.Category or "Other",
+							})
+						end
 					end
 				end
 			end

package/plugin/src/UIBuilder.lua

@@ -434,6 +434,19 @@ local function applyProperty(instance, key, value)
 	end)
 end
 
+local RESERVED_KEYS = {
+	ClassName = true,
+	className = true,
+	Name = true,
+	name = true,
+	Children = true,
+	children = true,
+	properties = true,
+	Properties = true,
+	props = true,
+	Props = true,
+}
+
 local function buildNode(spec, parent, animate)
 	local className = spec.ClassName or spec.className or "Frame"
 	local name = spec.Name or spec.name or className
@@ -441,20 +454,25 @@ local function buildNode(spec, parent, animate)
 	local instance = Instance.new(className)
 	instance.Name = name
 
-	-- Apply all properties
+	-- Apply flattened top-level properties
 	for key, value in spec do
-		if
-			key ~= "ClassName"
-			and key ~= "className"
-			and key ~= "Name"
-			and key ~= "name"
-			and key ~= "Children"
-			and key ~= "children"
-		then
+		if not RESERVED_KEYS[key] then
 			applyProperty(instance, key, value)
 		end
 	end
 
+	-- Also accept a nested "properties" / "Properties" / "props" bag for convenience.
+	-- This lets callers group properties explicitly without the key being misread
+	-- as an instance property name.
+	local propBag = spec.properties or spec.Properties or spec.props or spec.Props
+	if type(propBag) == "table" then
+		for key, value in propBag do
+			if not RESERVED_KEYS[key] then
+				applyProperty(instance, key, value)
+			end
+		end
+	end
+
 	-- Set parent after properties to avoid unnecessary layout computations
 	instance.Parent = parent
 

package/plugin/src/init.server.lua

@@ -82,7 +82,11 @@ local function handleMessage(rawMessage)
 			ts = os.clock(),
 		}))
 	elseif msgType == "cli:get:descendants:properties" then
-		local result = Properties.getDescendants(payload.path)
+		local compact = payload.compact
+		if compact == nil then
+			compact = true
+		end -- default to compact
+		local result = Properties.getDescendants(payload.path, compact)
 		WsClient.send(Protocol.encode({
 			id = msgId,
 			type = "studio:response",