windmill-components 1.655.2 → 1.665.1

package/package/components/flows/map/FlowStickyNode.svelte.d.ts

@@ -10,6 +10,7 @@ interface Props {
     toggleNoteMode?: () => void;
     disableAi?: boolean;
     diffManager?: FlowDiffManager;
+    compact?: boolean;
 }
 interface $$__sveltets_2_IsomorphicComponent<Props extends Record<string, any> = any, Events extends Record<string, any> = any, Slots extends Record<string, any> = any, Exports = {}, Bindings = string> {
     new (options: import('svelte').ComponentConstructorOptions<Props>): import('svelte').SvelteComponent<Props, Events, Slots> & {

package/package/components/flows/map/VirtualItemWrapper.svelte

@@ -12,7 +12,7 @@ let hover = $state(false);
 	<!-- svelte-ignore a11y_no_static_element_interactions -->
 	<div
 		class={classNames(
-			'w-full flex relative rounded-md drop-shadow-base',
+			'w-full flex relative rounded-md drop-shadow-sm',
 			colorClasses.bg,
 			onTop ? 'z-[901]' : '',
 			className

package/package/components/flows/pickers/PickHubScriptQuick.svelte.d.ts

@@ -35,6 +35,6 @@ declare const PickHubScriptQuick: $$__sveltets_2_IsomorphicComponent<Props, {
     pickScript: CustomEvent<any>;
 } & {
     [evt: string]: CustomEvent<any>;
-}, {}, {}, "filter" | "loading" | "items" | "apps">;
+}, {}, {}, "apps" | "filter" | "loading" | "items">;
 type PickHubScriptQuick = InstanceType<typeof PickHubScriptQuick>;
 export default PickHubScriptQuick;

package/package/components/flows/propPicker/OutputPickerInner.svelte

@@ -130,7 +130,9 @@ function togglePin() {
     }
 }
 function updateLastJob() {
-    if (testJob) {
+    // Prefer testJob when actively running/streaming (individual step test in progress)
+    // or when there's no moduleId (result node — no flowStateStore entry exists)
+    if (testJob && (testJob.result_stream || testJob.type === 'QueuedJob' || !moduleId)) {
         return testJob;
     }
     if (!flowStateStore ||
@@ -138,6 +140,8 @@ function updateLastJob() {
         flowStateStore.val[moduleId]?.previewResult === 'never tested this far') {
         return;
     }
+    // Use flowStateStore as source of truth — it's updated by both individual step tests
+    // (ModuleTest.jobDone) and flow tests (FlowStatusViewerInner.onJobsLoadedInner)
     return {
         id: flowStateStore.val[moduleId]?.previewJobId ?? '',
         result: flowStateStore.val[moduleId]?.previewResult,

package/package/components/flows/scheduleUtils.js

@@ -142,7 +142,9 @@ export async function saveScheduleFromCfg(scheduleCfg, edit, workspace) {
         tag: scheduleCfg.tag,
         paused_until: scheduleCfg.paused_until,
         cron_version: scheduleCfg.cron_version,
-        dynamic_skip: scheduleCfg.dynamic_skip
+        dynamic_skip: scheduleCfg.dynamic_skip,
+        permissioned_as: scheduleCfg.permissioned_as,
+        preserve_permissioned_as: scheduleCfg.preserve_permissioned_as
     };
     try {
         if (edit) {

package/package/components/flows/utils.svelte.js

@@ -50,11 +50,16 @@ export function evalValue(k, mod, pickableProperties, showError) {
     }
     return v;
 }
+/** Ensure modules comes first and groups last in the value object for readable YAML export. */
+function reorderFlowValue(value) {
+    const { modules, groups, ...rest } = value;
+    return { modules, ...rest, ...(groups != null ? { groups } : {}) };
+}
 export function filteredContentForExport(flow) {
     let o = {
         summary: flow.summary,
         description: flow.description,
-        value: flow.value,
+        value: reorderFlowValue(flow.value),
         schema: flow.schema
     };
     if (flow.dedicated_worker) {

package/package/components/git_sync/GitSyncContext.svelte.js

@@ -1,4 +1,7 @@
 import { getContext, setContext } from 'svelte';
+import { enterpriseLicense } from '../../stores';
+import { get } from 'svelte/store';
+import { sendUserToast } from '../../toast';
 import { JobService, WorkspaceService, ResourceService } from '../../gen';
 import { jobManager } from '../../services/JobManager';
 import hubPaths from '../../hubPaths.json';
@@ -417,8 +420,8 @@ export function createGitSyncContext(workspace) {
             // Use JobManager for polling
             await jobManager.runWithProgress(() => Promise.resolve(jobId), {
                 workspace,
-                timeout: 5000,
-                timeoutMessage: 'Git sync test job timed out after 5s',
+                timeout: 10000,
+                timeoutMessage: 'Git sync test job timed out after 10s',
                 onProgress: (status) => {
                     gitSyncTestJobs[idx].status =
                         status.status === 'success'
@@ -496,6 +499,10 @@ export function createGitSyncContext(workspace) {
         }
     }
     function addSyncRepository() {
+        if (!get(enterpriseLicense) && repositories && repositories.length >= 1) {
+            sendUserToast('Multiple repositories requires Enterprise Edition', true);
+            return;
+        }
         repositories.push({
             git_repo_resource_path: '',
             script_path: undefined,
@@ -518,6 +525,10 @@ export function createGitSyncContext(workspace) {
         });
     }
     function addPromotionRepository() {
+        if (!get(enterpriseLicense)) {
+            sendUserToast('Promotion mode requires Enterprise Edition', true);
+            return;
+        }
         repositories.push({
             git_repo_resource_path: '',
             script_path: undefined,

package/package/components/git_sync/GitSyncSection.svelte

@@ -5,10 +5,33 @@ import { setGitSyncContext } from './GitSyncContext.svelte';
 import GitSyncRepositoryCard from './GitSyncRepositoryCard.svelte';
 import GitSyncModalManager from './GitSyncModalManager.svelte';
 import { enterpriseLicense, workspaceStore } from '../../stores';
+import { WorkspaceService } from '../../gen';
 import { sendUserToast } from '../../toast';
 import { untrack } from 'svelte';
 // Create context reactively based on workspaceStore
 const gitSyncContext = $derived($workspaceStore ? setGitSyncContext($workspaceStore) : null);
+// Fetch git sync eligibility
+let gitSyncStatus = $state({ enabled: false, reason: null, max_repos: null, user_count: null, max_users: null });
+$effect(() => {
+    if ($workspaceStore) {
+        WorkspaceService.getGitSyncEnabled({ workspace: $workspaceStore })
+            .then((status) => {
+            gitSyncStatus = status;
+        })
+            .catch(() => {
+            gitSyncStatus = {
+                enabled: false,
+                reason: null,
+                max_repos: null,
+                user_count: null,
+                max_users: null
+            };
+        });
+    }
+});
+const gitSyncAllowed = $derived(gitSyncStatus.enabled);
+const isFreeTier = $derived(gitSyncAllowed && !$enterpriseLicense);
+const hasConfiguredRepos = $derived(gitSyncContext?.repositories?.some((r) => r.git_repo_resource_path) ?? false);
 // Load settings when workspace context changes
 $effect(() => {
     if (gitSyncContext) {
@@ -51,7 +74,7 @@ const hasUnsavedSecondaryPromotion = $derived(secondaryPromotion.some((s) => s.r
 		link="https://www.windmill.dev/docs/advanced/git_sync"
 	>
 		{#snippet actions()}
-			{#if $enterpriseLicense && gitSyncContext.repositories != undefined}
+			{#if (gitSyncAllowed || gitSyncStatus.user_count != null) && gitSyncContext?.repositories != undefined}
 				<Button
 					variant="accent"
 					target="_blank"
@@ -67,15 +90,26 @@ const hasUnsavedSecondaryPromotion = $derived(secondaryPromotion.some((s) => s.r
 		Only new changes matching the filters will trigger a git sync. You still need to initialize the
 		repo to the desired state first.
 	</Alert>
-	{#if !$enterpriseLicense}
+	{#if !gitSyncAllowed}
+		<div class="mb-2"></div>
+
+		<Alert type={hasConfiguredRepos ? 'error' : 'warning'} title="Git sync disabled">
+			Git sync is an EE feature provided in CE for testing and hobbyist use when workspace members
+			&le; {gitSyncStatus.max_users}. Your workspace has {gitSyncStatus.user_count} members. Settings
+			below are preserved but sync is inactive until membership is reduced or you upgrade to EE.
+		</Alert>
+		<div class="mb-2"></div>
+	{:else if isFreeTier}
 		<div class="mb-2"></div>
 
-		<Alert type="warning" title="Syncing workspace to Git is an EE feature">
-			Automatically saving scripts to a Git repository on each deploy is a Windmill EE feature.
+		<Alert type="warning" title="CE Limited Feature">
+			Git sync is an EE feature provided in CE for testing and hobbyist use when workspace members
+			&le; {gitSyncStatus.max_users}. Limited to a single repository. Upgrade to EE for multiple
+			repositories, promotion mode, and GitHub App authentication.
 		</Alert>
 		<div class="mb-2"></div>
 	{/if}
-	{#if $enterpriseLicense && gitSyncContext.repositories != undefined}
+	{#if (gitSyncAllowed || gitSyncStatus.user_count != null) && gitSyncContext?.repositories != undefined}
 		<!-- Primary Sync Repository -->
 		<div class="space-y-6 pt-6">
 			<GitSyncRepositoryCard
@@ -88,120 +122,46 @@ const hasUnsavedSecondaryPromotion = $derived(secondaryPromotion.some((s) => s.r
 				showEmptyState={primarySync?.repo === null}
 			/>
 
-			<!-- Secondary Sync Repositories -->
-			{#if primarySync && !primarySync.repo?.isUnsavedConnection}
-				{#if secondarySync.length > 0 || secondarySyncExpanded}
-					<div class="mt-4">
-						<button
-							class="flex items-center gap-2 text-sm text-secondary hover:text-primary transition-colors"
-							onclick={() => (secondarySyncExpanded = !secondarySyncExpanded)}
-						>
-							{#if secondarySyncExpanded}
-								<ChevronDown size={16} />
-							{:else}
-								<ChevronRight size={16} />
-							{/if}
-							Secondary sync repositories ({secondarySync.length})
-						</button>
-
-						{#if secondarySyncExpanded}
-							<div class="mt-3 space-y-3">
-								{#if secondarySync.length === 0}
-									<div class="text-sm text-secondary italic">
-										No secondary sync repositories configured
-									</div>
-								{:else}
-									{#each secondarySync as { repo, idx } (repo.git_repo_resource_path)}
-										<div class="pl-4">
-											<GitSyncRepositoryCard variant="secondary" {idx} isSecondary={true} />
-										</div>
-									{/each}
-								{/if}
-
-								{#if !hasUnsavedSecondary}
-									<div class="pl-4">
-										<Button
-											size="xs"
-											variant="default"
-											startIcon={{ icon: Plus }}
-											onclick={() => gitSyncContext.addSyncRepository()}
-										>
-											Add secondary sync
-										</Button>
-									</div>
-								{/if}
-							</div>
-						{/if}
-					</div>
-				{:else}
-					<!-- Collapsed state when no secondary repos exist -->
-					{#if !hasUnsavedSecondary}
-						<div class="mt-2">
-							<button
-								class="text-xs text-primary hover:text-secondary transition-colors"
-								onclick={() => {
-									secondarySyncExpanded = true
-									gitSyncContext.addSyncRepository()
-								}}
-							>
-								+ Add secondary sync repository
-							</button>
-						</div>
-					{/if}
-				{/if}
-			{/if}
-
-			<!-- Primary Promotion Repository -->
-			<div class="mt-6">
-				<GitSyncRepositoryCard
-					variant="primary-promotion"
-					mode="promotion"
-					idx={primaryPromotion?.idx ?? null}
-					repository={primaryPromotion?.repo ?? null}
-					onAdd={() => gitSyncContext.addPromotionRepository()}
-					isCollapsible={false}
-					showEmptyState={primaryPromotion?.repo === null}
-				/>
-
-				<!-- Secondary Promotion Repositories -->
-				{#if primaryPromotion && !primaryPromotion.repo?.isUnsavedConnection}
-					{#if secondaryPromotion.length > 0 || secondaryPromotionExpanded}
+			{#if $enterpriseLicense}
+				<!-- Secondary Sync Repositories (EE only) -->
+				{#if primarySync && !primarySync.repo?.isUnsavedConnection}
+					{#if secondarySync.length > 0 || secondarySyncExpanded}
 						<div class="mt-4">
 							<button
 								class="flex items-center gap-2 text-sm text-secondary hover:text-primary transition-colors"
-								onclick={() => (secondaryPromotionExpanded = !secondaryPromotionExpanded)}
+								onclick={() => (secondarySyncExpanded = !secondarySyncExpanded)}
 							>
-								{#if secondaryPromotionExpanded}
+								{#if secondarySyncExpanded}
 									<ChevronDown size={16} />
 								{:else}
 									<ChevronRight size={16} />
 								{/if}
-								Secondary promotion repositories ({secondaryPromotion.length})
+								Secondary sync repositories ({secondarySync.length})
 							</button>
 
-							{#if secondaryPromotionExpanded}
+							{#if secondarySyncExpanded}
 								<div class="mt-3 space-y-3">
-									{#if secondaryPromotion.length === 0}
+									{#if secondarySync.length === 0}
 										<div class="text-sm text-secondary italic">
-											No secondary promotion repositories configured
+											No secondary sync repositories configured
 										</div>
 									{:else}
-										{#each secondaryPromotion as { repo, idx } (repo.git_repo_resource_path)}
+										{#each secondarySync as { repo, idx } (repo.git_repo_resource_path)}
 											<div class="pl-4">
 												<GitSyncRepositoryCard variant="secondary" {idx} isSecondary={true} />
 											</div>
 										{/each}
 									{/if}
 
-									{#if !hasUnsavedSecondaryPromotion}
+									{#if !hasUnsavedSecondary}
 										<div class="pl-4">
 											<Button
 												size="xs"
 												variant="default"
 												startIcon={{ icon: Plus }}
-												onclick={() => gitSyncContext.addPromotionRepository()}
+												onclick={() => gitSyncContext.addSyncRepository()}
 											>
-												Add secondary promotion
+												Add secondary sync
 											</Button>
 										</div>
 									{/if}
@@ -209,23 +169,99 @@ const hasUnsavedSecondaryPromotion = $derived(secondaryPromotion.some((s) => s.r
 							{/if}
 						</div>
 					{:else}
-						<!-- Collapsed state when no secondary promotion repos exist -->
-						{#if !hasUnsavedSecondaryPromotion}
+						<!-- Collapsed state when no secondary repos exist -->
+						{#if !hasUnsavedSecondary}
 							<div class="mt-2">
 								<button
 									class="text-xs text-primary hover:text-secondary transition-colors"
 									onclick={() => {
-										secondaryPromotionExpanded = true
-										gitSyncContext.addPromotionRepository()
+										secondarySyncExpanded = true
+										gitSyncContext.addSyncRepository()
 									}}
 								>
-									+ Add secondary promotion repository
+									+ Add secondary sync repository
 								</button>
 							</div>
 						{/if}
 					{/if}
 				{/if}
-			</div>
+
+				<!-- Primary Promotion Repository (EE only) -->
+				<div class="mt-6">
+					<GitSyncRepositoryCard
+						variant="primary-promotion"
+						mode="promotion"
+						idx={primaryPromotion?.idx ?? null}
+						repository={primaryPromotion?.repo ?? null}
+						onAdd={() => gitSyncContext.addPromotionRepository()}
+						isCollapsible={false}
+						showEmptyState={primaryPromotion?.repo === null}
+					/>
+
+					<!-- Secondary Promotion Repositories -->
+					{#if primaryPromotion && !primaryPromotion.repo?.isUnsavedConnection}
+						{#if secondaryPromotion.length > 0 || secondaryPromotionExpanded}
+							<div class="mt-4">
+								<button
+									class="flex items-center gap-2 text-sm text-secondary hover:text-primary transition-colors"
+									onclick={() => (secondaryPromotionExpanded = !secondaryPromotionExpanded)}
+								>
+									{#if secondaryPromotionExpanded}
+										<ChevronDown size={16} />
+									{:else}
+										<ChevronRight size={16} />
+									{/if}
+									Secondary promotion repositories ({secondaryPromotion.length})
+								</button>
+
+								{#if secondaryPromotionExpanded}
+									<div class="mt-3 space-y-3">
+										{#if secondaryPromotion.length === 0}
+											<div class="text-sm text-secondary italic">
+												No secondary promotion repositories configured
+											</div>
+										{:else}
+											{#each secondaryPromotion as { repo, idx } (repo.git_repo_resource_path)}
+												<div class="pl-4">
+													<GitSyncRepositoryCard variant="secondary" {idx} isSecondary={true} />
+												</div>
+											{/each}
+										{/if}
+
+										{#if !hasUnsavedSecondaryPromotion}
+											<div class="pl-4">
+												<Button
+													size="xs"
+													variant="default"
+													startIcon={{ icon: Plus }}
+													onclick={() => gitSyncContext.addPromotionRepository()}
+												>
+													Add secondary promotion
+												</Button>
+											</div>
+										{/if}
+									</div>
+								{/if}
+							</div>
+						{:else}
+							<!-- Collapsed state when no secondary promotion repos exist -->
+							{#if !hasUnsavedSecondaryPromotion}
+								<div class="mt-2">
+									<button
+										class="text-xs text-primary hover:text-secondary transition-colors"
+										onclick={() => {
+											secondaryPromotionExpanded = true
+											gitSyncContext.addPromotionRepository()
+										}}
+									>
+										+ Add secondary promotion repository
+									</button>
+								</div>
+							{/if}
+						{/if}
+					{/if}
+				</div>
+			{/if}
 		</div>
 
 		<!-- Modals -->

package/package/components/graph/DragGhost.svelte

@@ -93,7 +93,15 @@ function computeGhost(moduleId, draggedNodeIds, allNodes, allEdges) {
         y: (PADDING - minY) * scale,
         zoom: scale
     };
-    return { containerWidth, containerHeight, ghostNodes, ghostEdges, offsetX, offsetY, initialViewport };
+    return {
+        containerWidth,
+        containerHeight,
+        ghostNodes,
+        ghostEdges,
+        offsetX,
+        offsetY,
+        initialViewport
+    };
 }
 let isNearDrop = $derived(moveManager.nearestDropZone != null);
 let ghost = $derived.by(() => {
@@ -109,7 +117,8 @@ let ghost = $derived.by(() => {
 		class="fixed pointer-events-none z-[10001] flex items-center justify-center w-5 h-5 rounded-full shadow border border-border transition-colors duration-150 {isNearDrop
 			? 'bg-surface-accent-primary text-white'
 			: 'bg-surface text-secondary'}"
-		style="left: {moveManager.ghostScreenX + CURSOR_INDICATOR_OFFSET}px; top: {moveManager.ghostScreenY + CURSOR_INDICATOR_OFFSET}px;"
+		style="left: {moveManager.ghostScreenX +
+			CURSOR_INDICATOR_OFFSET}px; top: {moveManager.ghostScreenY + CURSOR_INDICATOR_OFFSET}px;"
 	>
 		<Move size={12} />
 	</div>