goldsync 0.1.14 → 0.1.15

package/README.md

@@ -73,6 +73,10 @@ Place `goldsync.project.json` at the game repository root:
 
 A `file` root becomes one RBXM. A `directory` root creates separate files for descendants at `splitDepth`. Split at complete models, effects, or UI components—not individual parts or attachments. Never map the same hierarchy in Rojo, and do not include scripts inside a GoldSync root.
 
+Directory roots can contain more specific roots. Discovery skips branches owned by those roots, including existing files on disk. For example, split `ReplicatedStorage.Assets` at depth 1 and add another depth-1 directory root for `ReplicatedStorage.Assets.Effects`. New asset categories and individual effects are discovered automatically, without creating an overlapping `Effects.rbxm`. Existing explicit file roots can stay in place to preserve their sync history. File roots cannot contain other roots because their snapshots include the entire subtree.
+
+When upgrading an existing project, put the new parent directory roots in the optional top-level `discoveryRoots` array, using the same fields as directory roots. Older GoldSync versions ignore that array, so they cannot create overlapping snapshots before the service is restarted with nested-root support. Existing roots and files remain unchanged.
+
 GoldSync reads the Rojo project name and fallback `servePort` from `default.project.json`. It automatically detects `rojo serve --port` overrides and verifies the listener's project name before connecting. That active Rojo session selects the GoldSync workspace, so Studio place IDs and names do not need configuration. Set `rojo.projectFile` only when the repository uses another project file.
 
 Map GoldSync's generated live-session marker in the Rojo project tree:

package/assets/GoldSync.rbxmx

@@ -22,7 +22,7 @@ local CLIENT_HEADERS = {
 }
 local SETTINGS_PREFIX = "GoldSync:v2:"
 local UI_SETTINGS_PREFIX = "GoldSync:ui:v1:"
-local VERSION = "0.1.14"
+local VERSION = "0.1.15"
 local REQUEST_INTERVAL_SECONDS = 0.15
 local REQUEST_RETRY_DELAYS = { 1, 2, 4 }
 
@@ -527,6 +527,24 @@ local function discoverSplitRoot(splitRoot, root)
 	end
 	local missingPaths = {}
 	for _, relativePath in collectSplitPaths(root, splitRoot.splitDepth) do
+		local excluded = false
+		for _, excludedPath in splitRoot.excludedPaths or {} do
+			local matches = true
+			for index, segment in excludedPath do
+				if relativePath[index] ~= segment then
+					matches = false
+					break
+				end
+			end
+			if matches then
+				excluded = true
+				break
+			end
+		end
+		if excluded then
+			continue
+		end
+
 		local fullPath = table.clone(splitRoot.studioPath)
 		for _, segment in relativePath do
 			table.insert(fullPath, segment)
@@ -1553,14 +1571,6 @@ local function createGroup(path, order)
 	section.LayoutOrder = order
 	section.Size = UDim2.new(1, 0, 0, 0)
 	section.Parent = if parentGroup then parentGroup.list else rootsContainer
-	if parentGroup then
-		local connector = Instance.new("Frame")
-		connector.BackgroundColor3 = Color3.fromRGB(72, 72, 82)
-		connector.BorderSizePixel = 0
-		connector.Position = UDim2.new(0, -8, 0, 11)
-		connector.Size = UDim2.fromOffset(35, 1)
-		connector.Parent = section
-	end
 
 	local sectionLayout = Instance.new("UIListLayout")
 	sectionLayout.Padding = UDim.new(0, 2)
@@ -1573,6 +1583,14 @@ local function createGroup(path, order)
 	header.LayoutOrder = 1
 	header.Size = UDim2.new(1, 0, 0, 24)
 	header.Parent = section
+	if parentGroup then
+		local connector = Instance.new("Frame")
+		connector.BackgroundColor3 = Color3.fromRGB(72, 72, 82)
+		connector.BorderSizePixel = 0
+		connector.Position = UDim2.new(0, -8, 0, 11)
+		connector.Size = UDim2.fromOffset(35, 1)
+		connector.Parent = header
+	end
 
 	local disclosure = Instance.new("TextButton")
 	disclosure.BackgroundTransparency = 1
@@ -1865,7 +1883,6 @@ local function createInstanceNode(instance, parent, depth, order)
 	connector.BorderSizePixel = 0
 	connector.Position = UDim2.new(0, -8, 0, 11)
 	connector.Size = UDim2.fromOffset(35, 1)
-	connector.Parent = section
 
 	local header = Instance.new("Frame")
 	header.BackgroundColor3 = Color3.fromRGB(40, 40, 46)
@@ -1873,6 +1890,7 @@ local function createInstanceNode(instance, parent, depth, order)
 	header.LayoutOrder = 1
 	header.Size = UDim2.new(1, 0, 0, 22)
 	header.Parent = section
+	connector.Parent = header
 	header.MouseEnter:Connect(function()
 		header.BackgroundTransparency = 0
 	end)
@@ -2033,7 +2051,6 @@ local function createRootRow(manifest, order)
 	connector.BorderSizePixel = 0
 	connector.Position = UDim2.new(0, -8, 0, 11)
 	connector.Size = UDim2.fromOffset(35, 1)
-	connector.Parent = row
 
 	local corner = Instance.new("UICorner")
 	corner.CornerRadius = UDim.new(0, 6)
@@ -2041,8 +2058,6 @@ local function createRootRow(manifest, order)
 
 	local rowPadding = Instance.new("UIPadding")
 	rowPadding.PaddingBottom = UDim.new(0, 5)
-	rowPadding.PaddingLeft = UDim.new(0, 6)
-	rowPadding.PaddingRight = UDim.new(0, 6)
 	rowPadding.PaddingTop = UDim.new(0, 5)
 	rowPadding.Parent = row
 
@@ -2057,6 +2072,7 @@ local function createRootRow(manifest, order)
 	titleLine.LayoutOrder = 1
 	titleLine.Size = UDim2.new(1, 0, 0, 22)
 	titleLine.Parent = row
+	connector.Parent = titleLine
 
 	local contentDisclosure = Instance.new("TextButton")
 	contentDisclosure.BackgroundTransparency = 1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "goldsync",
-  "version": "0.1.14",
+  "version": "0.1.15",
   "description": "Automatic native Roblox asset synchronization for Rojo projects.",
   "author": "tay",
   "license": "SEE LICENSE IN LICENSE.txt",

package/src/config.mjs

@@ -86,6 +86,10 @@ export async function loadConfig(configPath) {
   if (!Array.isArray(raw.roots) || raw.roots.length === 0) {
     throw new Error("roots must contain at least one sync root");
   }
+  if (raw.discoveryRoots !== undefined && (!Array.isArray(raw.discoveryRoots)
+    || raw.discoveryRoots.some((root) => !root || root.splitDepth === undefined))) {
+    throw new Error("discoveryRoots must contain directory roots with splitDepth");
+  }
   const maxPathDepth = raw.maxPathDepth ?? 3;
   if (!Number.isInteger(maxPathDepth) || maxPathDepth < 2 || maxPathDepth > 10) {
     throw new Error("maxPathDepth must be an integer between 2 and 10");
@@ -95,8 +99,8 @@ export async function loadConfig(configPath) {
   const studioPaths = new Set();
   const files = new Set();
   const splitRoots = [];
-  const roots = raw.roots.flatMap((root, index) => {
-    const label = `roots[${index}]`;
+  const roots = [...raw.roots, ...(raw.discoveryRoots ?? [])].flatMap((root, index) => {
+    const label = index < raw.roots.length ? `roots[${index}]` : `discoveryRoots[${index - raw.roots.length}]`;
     const id = requireString(root.id, `${label}.id`);
     if (!ID_PATTERN.test(id)) {
       throw new Error(`${label}.id must contain lowercase letters, numbers, or hyphens`);
@@ -145,6 +149,22 @@ export async function loadConfig(configPath) {
     }];
   });
 
+  const configuredRoots = [...roots, ...splitRoots];
+  for (const root of roots) {
+    if (configuredRoots.some((other) => other !== root
+      && other.studioPath.length > root.studioPath.length
+      && root.studioPath.every((segment, index) => other.studioPath[index] === segment))) {
+      throw new Error(`file root ${root.id} contains another sync root; use a directory root instead`);
+    }
+  }
+  for (const root of splitRoots) {
+    root.excludedPaths = configuredRoots
+      .filter((other) => other !== root
+        && other.studioPath.length > root.studioPath.length
+        && root.studioPath.every((segment, index) => other.studioPath[index] === segment))
+      .map((other) => other.studioPath.slice(root.studioPath.length, root.studioPath.length + root.splitDepth));
+  }
+
   return {
     version: 1,
     name: requireString(raw.name, "name"),

package/src/server.mjs

@@ -161,6 +161,7 @@ export class GoldSyncServer {
           id: root.id,
           studioPath: root.studioPath,
           splitDepth: root.splitDepth,
+          excludedPaths: root.excludedPaths ?? [],
         })),
         roots: this.store.getManifests().map((manifest) => {
           const conflict = conflicts.get(manifest.id);

package/src/store.mjs

@@ -140,7 +140,12 @@ export class SnapshotStore {
     }
     let changed = false;
     for (const relativePath of relativePaths) {
-      changed = (await this.addRoot(this.makeSplitRoot(splitRoot, relativePath))) || changed;
+      const root = this.makeSplitRoot(splitRoot, relativePath);
+      if ((splitRoot.excludedPaths ?? []).some((excluded) =>
+        excluded.every((segment, index) => relativePath[index] === segment))) {
+        continue;
+      }
+      changed = (await this.addRoot(root)) || changed;
     }
     if (changed) {
       await this.persistState();