dominus-cli 0.5.2 → 0.5.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dominus-cli",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "AI-powered autonomous agent for Roblox Studio development. Real-time WebSocket bridge, persistent memory, and intelligent tool use.",
   "type": "module",
   "bin": {
@@ -8,6 +8,21 @@
     "dominus-mcp": "./dist/mcp.js"
   },
   "main": "./dist/index.js",
+  "scripts": {
+    "dev": "tsx src/index.tsx",
+    "dev:mcp": "nodemon --watch src -e ts,tsx --exec \"pnpm run build && node dist/mcp.js\"",
+    "build": "tsup",
+    "build:plugin": "rojo build plugin/default.project.json -o plugin/Dominus.rbxm",
+    "start": "node dist/index.js",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
+    "lint": "prettier --check .",
+    "format": "prettier --write .",
+    "typecheck": "tsc --noEmit",
+    "prepublishOnly": "pnpm build",
+    "install-plugin": "tsx src/install-plugin.ts"
+  },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.27.0",
     "chalk": "^5.4.1",
@@ -61,19 +76,5 @@
   "repository": {
     "type": "git",
     "url": "https://github.com/eli/dominus"
-  },
-  "scripts": {
-    "dev": "tsx src/index.tsx",
-    "dev:mcp": "nodemon --watch src -e ts,tsx --exec \"pnpm run build && node dist/mcp.js\"",
-    "build": "tsup",
-    "build:plugin": "rojo build plugin/default.project.json -o plugin/Dominus.rbxm",
-    "start": "node dist/index.js",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
-    "lint": "prettier --check .",
-    "format": "prettier --write .",
-    "typecheck": "tsc --noEmit",
-    "install-plugin": "tsx src/install-plugin.ts"
   }
-}
+}

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/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",