dominus-cli 0.5.4 → 0.5.6

package/dist/mcp.js

@@ -703,10 +703,19 @@ async function startMcpServer() {
   }
   const mcp = new McpServer({
     name: "dominus",
-    version: "0.2.1"
+    version: "0.5.5"
   }, {
     instructions: INTERNAL_MEMORY
   });
+  function luaStr(s) {
+    return `"${s.replace(/\\/g, "\\\\").replace(/"/g, '\\"').replace(/\n/g, "\\n").replace(/\r/g, "\\r")}"`;
+  }
+  function resolvePath(dotPath) {
+    const parts = dotPath.split(".");
+    let r = "game";
+    for (const p of parts) r += `[${luaStr(p)}]`;
+    return r;
+  }
   mcp.tool(
     "read_script",
     "Read the source code of a script in Roblox Studio",
@@ -758,11 +767,14 @@ async function startMcpServer() {
   );
   mcp.tool(
     "get_properties",
-    "Get all properties of an instance in Roblox Studio",
-    { path: z.string().describe("Full instance path") },
-    async ({ path: path2 }) => {
+    "Get all properties of a single instance in Roblox Studio. Returns property name, value, type, and category. For inspecting UI trees, prefer serialize_ui (structured tree) or get_descendants_properties (flat list with compact filtering).",
+    {
+      path: z.string().describe('Full instance path (e.g. "Workspace.Part", "StarterGui.MyUI.Frame")'),
+      compact: z.boolean().optional().default(true).describe("Strip read-only, nil, deprecated, and default-valued properties (default true). Set false for full dump.")
+    },
+    async ({ path: path2, compact }) => {
       assertConnected();
-      const res = await bridge.sendRequest(CliMsg.GET_PROPERTIES, { path: path2 });
+      const res = await bridge.sendRequest(CliMsg.GET_PROPERTIES, { path: path2, compact });
       return { content: [{ type: "text", text: JSON.stringify(res.payload, null, 2) }] };
     }
   );
@@ -978,16 +990,10 @@ async function startMcpServer() {
     },
     async (params) => {
       assertConnected();
-      const parts = params.path.split(".");
-      let resolve = "game";
-      for (const p of parts) resolve += `["${p}"]`;
-      const lines = [`local orig = ${resolve}`, `local clone = orig:Clone()`];
-      if (params.newName) lines.push(`clone.Name = "${params.newName}"`);
+      const lines = [`local orig = ${resolvePath(params.path)}`, `local clone = orig:Clone()`];
+      if (params.newName) lines.push(`clone.Name = ${luaStr(params.newName)}`);
       if (params.newParent) {
-        const pp = params.newParent.split(".");
-        let pr = "game";
-        for (const p of pp) pr += `["${p}"]`;
-        lines.push(`clone.Parent = ${pr}`);
+        lines.push(`clone.Parent = ${resolvePath(params.newParent)}`);
       } else {
         lines.push(`clone.Parent = orig.Parent`);
       }
@@ -1003,7 +1009,7 @@ async function startMcpServer() {
   );
   mcp.tool(
     "group_instances",
-    "Group multiple instances into a Model or Folder",
+    "Group multiple instances into a Model or Folder. All listed instances become children of the new group.",
     {
       paths: z.array(z.string()).describe("Array of instance paths to group"),
       groupName: z.string().describe("Name for the group"),
@@ -1014,20 +1020,11 @@ async function startMcpServer() {
       assertConnected();
       const gc = params.groupClass ?? "Model";
       const parent = params.parent ?? "Workspace";
-      const parentParts = parent.split(".");
-      let parentResolve = "game";
-      for (const p of parentParts) parentResolve += `["${p}"]`;
-      const resolves = params.paths.map((path2) => {
-        const parts = path2.split(".");
-        let r = "game";
-        for (const part of parts) r += `["${part}"]`;
-        return r;
-      });
       const code = [
-        `local group = Instance.new("${gc}")`,
-        `group.Name = "${params.groupName}"`,
-        `group.Parent = ${parentResolve}`,
-        ...resolves.map((r) => `${r}.Parent = group`),
+        `local group = Instance.new(${luaStr(gc)})`,
+        `group.Name = ${luaStr(params.groupName)}`,
+        `group.Parent = ${resolvePath(parent)}`,
+        ...params.paths.map((p) => `${resolvePath(p)}.Parent = group`),
         `print("Grouped ${params.paths.length} instances into: " .. group:GetFullName())`
       ].join("\n");
       const res = await bridge.sendRequest(CliMsg.RUN_CODE, { code });
@@ -1036,7 +1033,7 @@ async function startMcpServer() {
   );
   mcp.tool(
     "build_multiple",
-    "Create many parts/instances in one batch. Send an array of specs \u2014 all created in a single round-trip. Ideal for building trees, houses, vehicles.",
+    "Create many parts/instances in one batch. Send an array of specs \u2014 all created in a single round-trip. Ideal for building trees, houses, vehicles. Supports hex colors, rgb() colors, and BrickColor names.",
     {
       parts: z.array(z.object({
         name: z.string(),
@@ -1044,10 +1041,11 @@ async function startMcpServer() {
         parent: z.string().optional(),
         position: z.object({ x: z.number(), y: z.number(), z: z.number() }).optional(),
         size: z.object({ x: z.number(), y: z.number(), z: z.number() }).optional(),
-        color: z.string().optional().describe('Hex "#RRGGBB" or BrickColor name'),
-        material: z.string().optional(),
+        color: z.string().optional().describe('Hex "#RRGGBB", "rgb(r,g,b)", or BrickColor name'),
+        material: z.string().optional().describe("Enum.Material name: Plastic, Wood, Neon, Glass, etc."),
         transparency: z.number().optional(),
         anchored: z.boolean().optional(),
+        canCollide: z.boolean().optional(),
         shape: z.enum(["Block", "Ball", "Cylinder", "Wedge"]).optional()
       })).describe("Array of part specifications"),
       groupName: z.string().optional().describe("Group all parts into a Model with this name"),
@@ -1058,35 +1056,36 @@ async function startMcpServer() {
       const parts = params.parts;
       const groupName = params.groupName;
       const groupParent = params.groupParent ?? "Workspace";
-      function resolveP(path2) {
-        const segs = path2.split(".");
-        let r = "game";
-        for (const s of segs) r += `["${s}"]`;
-        return r;
-      }
       const lines = [];
       if (groupName) {
         lines.push(`local _group = Instance.new("Model")`);
-        lines.push(`_group.Name = "${groupName}"`);
-        lines.push(`_group.Parent = ${resolveP(groupParent)}`);
+        lines.push(`_group.Name = ${luaStr(groupName)}`);
+        lines.push(`_group.Parent = ${resolvePath(groupParent)}`);
       }
       for (let i = 0; i < parts.length; i++) {
         const p = parts[i];
         const v = `_p${i}`;
         const cn = p.className ?? (p.shape === "Wedge" ? "WedgePart" : "Part");
-        lines.push(`local ${v} = Instance.new("${cn}")`);
-        lines.push(`${v}.Name = "${p.name}"`);
+        lines.push(`local ${v} = Instance.new(${luaStr(cn)})`);
+        lines.push(`${v}.Name = ${luaStr(p.name)}`);
         if (p.position) lines.push(`${v}.Position = Vector3.new(${p.position.x}, ${p.position.y}, ${p.position.z})`);
         if (p.size) lines.push(`${v}.Size = Vector3.new(${p.size.x}, ${p.size.y}, ${p.size.z})`);
         if (p.color) {
-          if (p.color.startsWith("#")) lines.push(`${v}.Color = Color3.fromHex("${p.color}")`);
-          else lines.push(`${v}.BrickColor = BrickColor.new("${p.color}")`);
+          if (p.color.startsWith("#")) {
+            lines.push(`${v}.Color = Color3.fromHex(${luaStr(p.color)})`);
+          } else if (p.color.startsWith("rgb(")) {
+            const m = p.color.match(/rgb\(\s*(\d+)\s*,\s*(\d+)\s*,\s*(\d+)\s*\)/);
+            if (m) lines.push(`${v}.Color = Color3.fromRGB(${m[1]}, ${m[2]}, ${m[3]})`);
+          } else {
+            lines.push(`${v}.BrickColor = BrickColor.new(${luaStr(p.color)})`);
+          }
         }
         if (p.material) lines.push(`${v}.Material = Enum.Material.${p.material}`);
         if (p.transparency !== void 0) lines.push(`${v}.Transparency = ${p.transparency}`);
         lines.push(`${v}.Anchored = ${p.anchored ?? true}`);
+        if (p.canCollide !== void 0) lines.push(`${v}.CanCollide = ${p.canCollide}`);
         if (p.shape && p.shape !== "Block" && cn === "Part") lines.push(`${v}.Shape = Enum.PartType.${p.shape}`);
-        lines.push(`${v}.Parent = ${groupName ? "_group" : resolveP(p.parent ?? "Workspace")}`);
+        lines.push(`${v}.Parent = ${groupName ? "_group" : resolvePath(p.parent ?? "Workspace")}`);
       }
       lines.push(`print("Built ${parts.length} parts${groupName ? ` in ${groupName}` : ""}")`);
       const res = await bridge.sendRequest(CliMsg.RUN_CODE, { code: lines.join("\n") });