goldsync 0.1.14 → 0.1.16

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.16"
 local REQUEST_INTERVAL_SECONDS = 0.15
 local REQUEST_RETRY_DELAYS = { 1, 2, 4 }
 
@@ -185,13 +185,13 @@ local searchBox = Instance.new("TextBox")
 searchBox.BackgroundColor3 = Color3.fromRGB(40, 40, 46)
 searchBox.BorderSizePixel = 0
 searchBox.ClearTextOnFocus = false
-searchBox.Font = Enum.Font.BuilderSans
-searchBox.PlaceholderColor3 = Color3.fromRGB(130, 130, 140)
+searchBox.FontFace = Font.new("rbxasset://fonts/families/BuilderSans.json", Enum.FontWeight.Medium)
+searchBox.PlaceholderColor3 = Color3.fromRGB(195, 195, 205)
 searchBox.PlaceholderText = "Search files and instances..."
 searchBox.Size = UDim2.new(1, -132, 1, 0)
 searchBox.Text = ""
 searchBox.TextColor3 = Color3.fromRGB(235, 235, 240)
-searchBox.TextSize = 12
+searchBox.TextSize = 13
 searchBox.TextXAlignment = Enum.TextXAlignment.Left
 searchBox.Parent = controls
 Instance.new("UICorner", searchBox).CornerRadius = UDim.new(0, 4)
@@ -202,12 +202,12 @@ searchPadding.PaddingRight = UDim.new(0, 8)
 local unsyncedButton = Instance.new("TextButton")
 unsyncedButton.BackgroundColor3 = Color3.fromRGB(40, 40, 46)
 unsyncedButton.BorderSizePixel = 0
-unsyncedButton.Font = Enum.Font.BuilderSansBold
+unsyncedButton.FontFace = Font.new("rbxasset://fonts/families/BuilderSans.json", Enum.FontWeight.Medium)
 unsyncedButton.Position = UDim2.new(1, -126, 0, 0)
 unsyncedButton.Size = UDim2.fromOffset(92, 28)
 unsyncedButton.Text = "Unsynced · 0"
 unsyncedButton.TextColor3 = Color3.fromRGB(190, 190, 198)
-unsyncedButton.TextSize = 11
+unsyncedButton.TextSize = 13
 unsyncedButton.Parent = controls
 Instance.new("UICorner", unsyncedButton).CornerRadius = UDim.new(0, 4)
 
@@ -292,12 +292,12 @@ local statusBanner = Instance.new("TextLabel")
 statusBanner.AutomaticSize = Enum.AutomaticSize.Y
 statusBanner.BackgroundColor3 = Color3.fromRGB(66, 38, 40)
 statusBanner.BorderSizePixel = 0
-statusBanner.Font = Enum.Font.BuilderSans
+statusBanner.FontFace = Font.new("rbxasset://fonts/families/BuilderSans.json", Enum.FontWeight.Medium)
 statusBanner.LayoutOrder = 4
 statusBanner.Size = UDim2.new(1, 0, 0, 0)
 statusBanner.Text = ""
 statusBanner.TextColor3 = Color3.fromRGB(255, 205, 205)
-statusBanner.TextSize = 11
+statusBanner.TextSize = 14
 statusBanner.TextWrapped = true
 statusBanner.TextXAlignment = Enum.TextXAlignment.Left
 statusBanner.Visible = false
@@ -309,13 +309,66 @@ statusBannerPadding.PaddingLeft = UDim.new(0, 8)
 statusBannerPadding.PaddingRight = UDim.new(0, 8)
 statusBannerPadding.PaddingTop = UDim.new(0, 7)
 
+local attentionBox = Instance.new("Frame")
+attentionBox.Name = "AttentionBox"
+attentionBox.AutomaticSize = Enum.AutomaticSize.Y
+attentionBox.BackgroundColor3 = Color3.fromRGB(43, 42, 38)
+attentionBox.BorderSizePixel = 0
+attentionBox.LayoutOrder = 5
+attentionBox.Size = UDim2.fromScale(1, 0)
+attentionBox.Visible = false
+attentionBox.Parent = content
+Instance.new("UICorner", attentionBox).CornerRadius = UDim.new(0.04, 0)
+local attentionBoxLayout = Instance.new("UIListLayout")
+attentionBoxLayout.SortOrder = Enum.SortOrder.LayoutOrder
+attentionBoxLayout.Parent = attentionBox
+
+local attentionActions = Instance.new("Frame")
+attentionActions.BackgroundTransparency = 1
+attentionActions.LayoutOrder = 1
+attentionActions.Size = UDim2.new(1, 0, 0, 30)
+attentionActions.Parent = attentionBox
+local attentionBulkButtons = {}
+for index, text in { "Push All →", "← Pull All" } do
+	local button = Instance.new("ImageButton")
+	button.AnchorPoint = Vector2.new(0.5, 0.5)
+	button.BackgroundColor3 = Color3.fromRGB(57, 57, 67)
+	button.BorderSizePixel = 0
+	button.Position = UDim2.fromScale(0.25 + (index - 1) * 0.5, 0.5)
+	button.Size = UDim2.fromScale(0.47, 0.8)
+	button.Parent = attentionActions
+	Instance.new("UICorner", button).CornerRadius = UDim.new(0.22, 0)
+	local label = Instance.new("TextLabel")
+	label.BackgroundTransparency = 1
+	label.Position = UDim2.fromScale(0.12, 0.18)
+	label.Size = UDim2.fromScale(0.76, 0.64)
+	label.Font = Enum.Font.BuilderSansBold
+	label.Text = text
+	label.TextColor3 = Color3.fromRGB(215, 225, 240)
+	label.TextScaled = true
+	label.Parent = button
+	attentionBulkButtons[index] = button
+end
+
+local attentionList = Instance.new("Frame")
+attentionList.Name = "AttentionList"
+attentionList.AutomaticSize = Enum.AutomaticSize.Y
+attentionList.BackgroundTransparency = 1
+attentionList.LayoutOrder = 5
+attentionList.Size = UDim2.fromScale(1, 0)
+attentionList.Visible = false
+attentionList.Parent = attentionBox
+local attentionLayout = Instance.new("UIListLayout")
+attentionLayout.SortOrder = Enum.SortOrder.LayoutOrder
+attentionLayout.Parent = attentionList
+
 local directionLegend = Instance.new("TextLabel")
 directionLegend.BackgroundTransparency = 1
-directionLegend.Font = Enum.Font.BuilderSans
-directionLegend.LayoutOrder = 5
+directionLegend.FontFace = Font.new("rbxasset://fonts/families/BuilderSans.json", Enum.FontWeight.Medium)
+directionLegend.LayoutOrder = 6
 directionLegend.Size = UDim2.new(1, 0, 0, 18)
 directionLegend.Text = "→  Studio to file      ←  File to Studio"
-directionLegend.TextColor3 = Color3.fromRGB(145, 145, 155)
+directionLegend.TextColor3 = Color3.fromRGB(195, 195, 205)
 directionLegend.TextSize = 11
 directionLegend.TextXAlignment = Enum.TextXAlignment.Right
 directionLegend.Parent = content
@@ -323,7 +376,7 @@ directionLegend.Parent = content
 local rootsContainer = Instance.new("Frame")
 rootsContainer.AutomaticSize = Enum.AutomaticSize.Y
 rootsContainer.BackgroundTransparency = 1
-rootsContainer.LayoutOrder = 6
+rootsContainer.LayoutOrder = 7
 rootsContainer.Size = UDim2.new(1, 0, 0, 0)
 rootsContainer.Parent = content
 
@@ -354,6 +407,34 @@ local resolveGitConflict
 local applyTreeFilter
 local resetInstanceTrees
 
+for index, button in attentionBulkButtons do
+	button.Activated:Connect(function()
+		if bulkRunning or not serverConfig or not serverConfig.rojoConnected then
+			return
+		end
+		local affected = {}
+		for _, state in states do
+			if state.statusKind == "issue" and not state.manifest.gitConflict then
+				table.insert(affected, state)
+			end
+		end
+		bulkRunning = true
+		refreshStatusSummary()
+		for _, state in affected do
+			if states[state.manifest.id] ~= state or state.manifest.gitConflict then
+				continue
+			end
+			if index == 1 then
+				upload(state, true)
+			else
+				pull(state)
+			end
+		end
+		bulkRunning = false
+		refreshStatusSummary()
+	end)
+end
+
 local function collectSplitPaths(root, depth)
 	local paths = {}
 	local function visit(instance, remainingDepth, relativePath)
@@ -527,6 +608,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 +1652,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 +1664,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
@@ -1735,6 +1834,7 @@ updateGroupSummaries = function()
 		local total = 0
 		local synced = 0
 		local issues = 0
+		local issueColor = Color3.fromRGB(244, 194, 92)
 		local pending = 0
 		local paused = 0
 		for _, state in states do
@@ -1744,6 +1844,9 @@ updateGroupSummaries = function()
 					synced += 1
 				elseif state.statusKind == "issue" then
 					issues += 1
+					if state.status.TextColor3 == Color3.fromRGB(245, 106, 106) then
+						issueColor = state.status.TextColor3
+					end
 				elseif state.statusKind == "paused" then
 					paused += 1
 				else
@@ -1766,7 +1869,7 @@ updateGroupSummaries = function()
 			group.bulkPush.Visible = true
 			group.bulkPull.Visible = true
 			group.statusDot.TextColor3 = if issues > 0
-				then Color3.fromRGB(245, 106, 106)
+				then issueColor
 				elseif synced == total then Color3.fromRGB(102, 214, 139)
 				elseif pending > 0 or paused > 0 then Color3.fromRGB(244, 194, 92)
 				else Color3.fromRGB(180, 180, 190)
@@ -1802,7 +1905,7 @@ refreshUnsyncedButton = function()
 		then Color3.fromRGB(40, 40, 46)
 		else Color3.fromRGB(78, 66, 38)
 	unsyncedButton.TextColor3 = if showSynced
-		then Color3.fromRGB(190, 190, 198)
+		then Color3.fromRGB(220, 220, 230)
 		else Color3.fromRGB(255, 218, 126)
 end
 
@@ -1813,13 +1916,29 @@ refreshStatusSummary = function()
 		local entry = {
 			path = table.concat(state.manifest.studioPath, "."),
 			status = state.status.Text,
+			state = state,
 		}
 		if state.statusKind == "issue" then
 			table.insert(issues, entry)
-		elseif state.statusKind == "working" then
-			table.insert(working, entry)
+		else
+			if state.row.Parent == attentionList then
+				state.row.Parent = state.group.list
+				state.connector.Visible = true
+				state.row.LayoutOrder = state.treeOrder
+				state.title.Text = state.manifest.studioPath[#state.manifest.studioPath]
+				setRootExpanded(state, state.expandedBeforeAttention)
+			end
+			if state.statusKind == "working" then
+				table.insert(working, entry)
+			end
 		end
 	end
+	attentionList.Visible = #issues > 0
+	attentionBox.Visible = #issues > 0
+	for _, button in attentionBulkButtons do
+		button.Active = serverConfig.rojoConnected and not bulkRunning
+		button.AutoButtonColor = button.Active
+	end
 	local entries = if #issues > 0 then issues else working
 	if #entries == 0 then
 		statusBanner.Visible = false
@@ -1833,19 +1952,25 @@ refreshStatusSummary = function()
 			then string.format("%d need attention", #issues)
 			else string.format("%d still syncing", #working),
 	}
-	for index = 1, math.min(#entries, 3) do
-		local entry = entries[index]
-		table.insert(lines, entry.path .. " — " .. entry.status)
-	end
-	if #entries > 3 then
-		table.insert(lines, string.format("+%d more", #entries - 3))
+	for index, entry in entries do
+		if #issues > 0 then
+			local state = entry.state
+			if state.row.Parent ~= attentionList then
+				state.expandedBeforeAttention = state.expanded
+				state.row.Parent = attentionList
+				state.connector.Visible = false
+				setRootExpanded(state, false)
+			end
+			state.row.LayoutOrder = index
+			state.title.Text = entry.path
+		else
+			table.insert(lines, entry.path .. " — " .. entry.status)
+		end
 	end
 	statusBanner.BackgroundColor3 = if #issues > 0
-		then Color3.fromRGB(66, 38, 40)
-		else Color3.fromRGB(66, 56, 34)
-	statusBanner.TextColor3 = if #issues > 0
-		then Color3.fromRGB(255, 205, 205)
-		else Color3.fromRGB(255, 224, 145)
+		then Color3.fromRGB(49, 46, 38)
+		else Color3.fromRGB(43, 42, 38)
+	statusBanner.TextColor3 = Color3.fromRGB(230, 222, 200)
 	statusBanner.Text = table.concat(lines, "\n")
 	statusBanner.Visible = true
 end
@@ -1865,7 +1990,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 +1997,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 +2158,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 +2165,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 +2179,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
@@ -2256,6 +2379,8 @@ local function createRootRow(manifest, order)
 		manifest = manifest,
 		group = group,
 		row = row,
+		treeOrder = order,
+		connector = connector,
 		title = title,
 		icon = icon,
 		iconClass = if rootInstance then rootInstance.ClassName else "Model",
@@ -2566,6 +2691,7 @@ refreshConfig = function()
 		for _, state in states do
 			disconnectRoot(state)
 			destroyConflictWorkspace(state)
+			state.row:Destroy()
 		end
 		disconnectSplitRoots()
 		table.clear(states)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "goldsync",
-  "version": "0.1.14",
+  "version": "0.1.16",
   "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();