prevent unintended cascade by depends on during migration

Author: icecrasher321

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/components/credential-selector/credential-selector.tsx

@@ -24,6 +24,7 @@ import { useCredentialSets } from '@/hooks/queries/credential-sets'
 import { useOAuthCredentials } from '@/hooks/queries/oauth-credentials'
 import { useOrganizations } from '@/hooks/queries/organization'
 import { useSubscriptionData } from '@/hooks/queries/subscription'
+import { useCollaborativeWorkflow } from '@/hooks/use-collaborative-workflow'
 import { useCredentialRefreshTriggers } from '@/hooks/use-credential-refresh-triggers'
 import { getMissingRequiredScopes } from '@/hooks/use-oauth-scope-status'
 import { useWorkflowRegistry } from '@/stores/workflows/registry/store'
@@ -54,6 +55,7 @@ export function CredentialSelector({
   const [isEditing, setIsEditing] = useState(false)
   const { activeWorkflowId } = useWorkflowRegistry()
   const [storeValue, setStoreValue] = useSubBlockValue<string | null>(blockId, subBlock.id)
+  const { collaborativeSetSubblockValue } = useCollaborativeWorkflow()
 
   const requiredScopes = subBlock.requiredScopes || []
   const label = subBlock.placeholder || 'Select credential'
@@ -135,7 +137,9 @@ export function CredentialSelector({
         const data = await response.json()
         if (!cancelled && data.credential?.displayName) {
           if (data.credential.id !== selectedId) {
-            setStoreValue(data.credential.id)
+            collaborativeSetSubblockValue(blockId, subBlock.id, data.credential.id, {
+              skipDependsOn: true,
+            })
           }
           setInaccessibleCredentialName(data.credential.displayName)
         }

apps/sim/hooks/use-collaborative-workflow.ts

@@ -1232,7 +1232,12 @@ export function useCollaborativeWorkflow() {
   )
 
   const collaborativeSetSubblockValue = useCallback(
-    (blockId: string, subblockId: string, value: any, options?: { _visited?: Set<string> }) => {
+    (
+      blockId: string,
+      subblockId: string,
+      value: any,
+      options?: { _visited?: Set<string>; skipDependsOn?: boolean }
+    ) => {
       if (isApplyingRemoteChange.current) return
 
       if (isBaselineDiffView) {
@@ -1258,6 +1263,8 @@ export function useCollaborativeWorkflow() {
         })
       }
 
+      if (options?.skipDependsOn) return
+
       // Handle dependent subblock clearing (recursive calls)
       try {
         const visited = options?._visited || new Set<string>()