From fc49a8fed655a66b89e2324f49121402cdffd507 Mon Sep 17 00:00:00 2001 From: Ruben Fiszel Date: Thu, 16 Apr 2026 07:29:38 -0700 Subject: [PATCH] fix: include workspacedependencies in default git sync include_type (#8852) * fix: include workspacedependencies in default git sync include_type Workspace dependencies were not synced by default because the default include_type arrays did not contain 'workspacedependencies'. This meant users had to manually add the type to their git sync config. Co-Authored-By: Claude Opus 4.6 (1M context) * fix: add missing folder to new-format fallback and align all defaults - Add 'folder' to the new-format repo fallback default include_type (was missing unlike the other 4 locations) - Add 'workspacedependencies' to legacy defaultTypes fallback - Align GitSyncFilterSettings prop default with context defaults Co-Authored-By: Claude Opus 4.6 (1M context) --------- Co-authored-by: Claude Opus 4.6 (1M context) --- .../components/git_sync/GitSyncContext.svelte.ts | 16 +++++++++++----- .../GitSyncFilterSettings.svelte | 8 +++++++- 2 files changed, 18 insertions(+), 6 deletions(-) diff --git a/frontend/src/lib/components/git_sync/GitSyncContext.svelte.ts b/frontend/src/lib/components/git_sync/GitSyncContext.svelte.ts index ba919fa16b..3bd2ce477d 100644 --- a/frontend/src/lib/components/git_sync/GitSyncContext.svelte.ts +++ b/frontend/src/lib/components/git_sync/GitSyncContext.svelte.ts @@ -194,7 +194,7 @@ export function createGitSyncContext(workspace: string) { include_path: ['f/**'], exclude_path: [], extra_include_path: [], - include_type: ['script', 'flow', 'app', 'folder'] + include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies'] }, exclude_types_override: [], legacyImported: false, @@ -408,7 +408,7 @@ export function createGitSyncContext(workspace: string) { const defaultTypes: GitSyncObjectType[] = workspaceLegacyIncludeType.length > 0 ? [...workspaceLegacyIncludeType] - : ['script', 'flow', 'app', 'folder'] + : ['script', 'flow', 'app', 'folder', 'workspacedependencies'] let repoSettings: SettingsObject if (isRepoLegacy) { @@ -433,7 +433,13 @@ export function createGitSyncContext(workspace: string) { include_path: repo.settings?.include_path ?? ['f/**'], exclude_path: repo.settings?.exclude_path ?? [], extra_include_path: repo.settings?.extra_include_path ?? [], - include_type: repo.settings?.include_type ?? ['script', 'flow', 'app'] + include_type: repo.settings?.include_type ?? [ + 'script', + 'flow', + 'app', + 'folder', + 'workspacedependencies' + ] } } @@ -662,7 +668,7 @@ export function createGitSyncContext(workspace: string) { include_path: ['f/**'], exclude_path: [], extra_include_path: [], - include_type: ['script', 'flow', 'app', 'folder'] + include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies'] }, exclude_types_override: [], legacyImported: false, @@ -689,7 +695,7 @@ export function createGitSyncContext(workspace: string) { include_path: ['f/**'], exclude_path: [], extra_include_path: [], - include_type: ['script', 'flow', 'app', 'folder'] + include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies'] }, exclude_types_override: [], legacyImported: false, diff --git a/frontend/src/lib/components/workspaceSettings/GitSyncFilterSettings.svelte b/frontend/src/lib/components/workspaceSettings/GitSyncFilterSettings.svelte index 1e2a0aaa75..cbf12c6314 100644 --- a/frontend/src/lib/components/workspaceSettings/GitSyncFilterSettings.svelte +++ b/frontend/src/lib/components/workspaceSettings/GitSyncFilterSettings.svelte @@ -30,7 +30,13 @@ let { git_repo_resource_path = $bindable(''), include_path = $bindable(['f/**']), - include_type = $bindable(['script', 'flow', 'app', 'folder'] as GitSyncObjectType[]), + include_type = $bindable([ + 'script', + 'flow', + 'app', + 'folder', + 'workspacedependencies' + ] as GitSyncObjectType[]), exclude_types_override = $bindable([] as GitSyncObjectType[]), isLegacyRepo = false, excludes = $bindable([] as string[]),