fix(workspace-forking): stop a required file input deadlocking Sync

Both PR bots flagged this and they were right. A repointed custom block's
`file[]` input renders as a disabled control — it is an upload on the canvas,
and there is nothing to type here — but the Sync gate still demanded a
non-empty value for every REQUIRED dependent. So a custom block with a required
file input turned Sync off permanently, while the field's own hint told the
user to go set it in a workflow they could only reach BY syncing.

`isForkSyncConfigurableField` is the one predicate for "can the modal put a
value in this field", used by the gate and by the per-kind status badge so the
two cannot disagree. Skipping the gate is only safe because the sync no longer
clears the field: the target keeps what it has, and a genuinely missing value is
still caught by the block's own required-field validation at run/deploy time —
the same fallback every other unconfigured required field already relies on.

Also gives the disabled control an `aria-label` (the row's visible label is a
sibling, not associated), closing the second review note.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: Vikhyath Mondreti

apps/sim/ee/workspace-forking/components/fork-sync/custom-block-input-control.test.ts

@@ -9,6 +9,7 @@ import {
   CUSTOM_BLOCK_BOOLEAN_UNSET,
   customBlockBooleanOptions,
   customBlockInputControl,
+  isForkSyncConfigurableField,
 } from '@/ee/workspace-forking/components/fork-sync/custom-block-input-control'
 
 describe('customBlockInputControl', () => {
@@ -50,6 +51,31 @@ describe('customBlockInputControl', () => {
   })
 })
 
+describe('isForkSyncConfigurableField', () => {
+  it('excludes a custom block’s file input from the Sync gate', () => {
+    // The modal renders it disabled, so a REQUIRED one could never be satisfied: Sync would
+    // stay off forever while the hint told the user to set it in a workflow they can only
+    // reach by syncing. The sync no longer clears the field, so skipping the gate is safe.
+    expect(isForkSyncConfigurableField({ parentKind: 'custom-block', fieldType: 'file[]' })).toBe(
+      false
+    )
+  })
+
+  it('still gates every custom-block input that HAS a control', () => {
+    for (const fieldType of ['string', 'number', 'boolean', 'object', 'array']) {
+      expect(isForkSyncConfigurableField({ parentKind: 'custom-block', fieldType })).toBe(true)
+    }
+  })
+
+  it('leaves every other parent kind gated', () => {
+    // Only custom-block inputs classify their own control here; a selector-backed dependent
+    // is always configurable.
+    for (const parentKind of ['credential', 'knowledge-base', 'table'] as const) {
+      expect(isForkSyncConfigurableField({ parentKind, fieldType: 'file[]' })).toBe(true)
+    }
+  })
+})
+
 describe('customBlockBooleanOptions', () => {
   it('lets an OPTIONAL flag return to the workflow default', () => {
     // Without this a single click permanently pins the flag: a two-segment switch has no

apps/sim/ee/workspace-forking/components/fork-sync/custom-block-input-control.ts

@@ -1,3 +1,4 @@
+import type { ForkDependentReconfig } from '@/lib/api/contracts/workspace-fork'
 import { subBlockTypeForField } from '@/blocks/custom/build-config'
 
 /**
@@ -74,3 +75,23 @@ export function customBlockBooleanOptions(required: boolean) {
  * untouched (see `replaceCustomBlockInputs`), so the field is safe to leave alone.
  */
 export const CUSTOM_BLOCK_UNSUPPORTED_HINT = 'Uploaded in the workflow — kept as configured there'
+
+/**
+ * Whether the sync modal can actually put a value in this field.
+ *
+ * The Sync gate demands a value for every REQUIRED dependent, which is right for a field the
+ * user can fill and a deadlock for one they cannot: a required `file[]` renders as a disabled
+ * control, so the gate could never be satisfied and Sync stayed off forever — with the hint
+ * telling the user to go set it in a workflow they could not reach.
+ *
+ * Excluding it is safe because the sync no longer clears it. The target keeps whatever it has
+ * (see `replaceCustomBlockInputs`), and a genuinely missing value is still caught by the
+ * block's own required-field validation at run/deploy time — the same fallback every other
+ * unconfigured required field relies on.
+ */
+export function isForkSyncConfigurableField(
+  field: Pick<ForkDependentReconfig, 'parentKind' | 'fieldType'>
+): boolean {
+  if (field.parentKind !== 'custom-block') return true
+  return customBlockInputControl(field.fieldType) !== 'unsupported'
+}

apps/sim/ee/workspace-forking/components/fork-sync/fork-sync-view.tsx

@@ -260,6 +260,7 @@ function DependentSelector({
             onChange={() => {}}
             disabled
             placeholder={CUSTOM_BLOCK_UNSUPPORTED_HINT}
+            aria-label={field.title}
           />
         )
       default:

apps/sim/ee/workspace-forking/components/fork-sync/use-fork-sync.ts

@@ -30,6 +30,7 @@ import {
   forkVisibleCopyables,
   isForkRequiredComplete,
 } from '@/ee/workspace-forking/components/fork-sync/copy-reconciliation'
+import { isForkSyncConfigurableField } from '@/ee/workspace-forking/components/fork-sync/custom-block-input-control'
 import {
   type DependentReconfigState,
   dependentKey,
@@ -561,6 +562,9 @@ export function useForkSync(params: {
   // instead, so it's skipped here.
   const reconfigComplete = dependentReconfigs.every((field) => {
     if (!field.required) return true
+    // A field the modal renders no control for can never satisfy this gate — see
+    // `isForkSyncConfigurableField`.
+    if (!isForkSyncConfigurableField(field)) return true
     const parent = entryForDependent(field)
     if (!parent) return true
     const resolution = resolutionFor(parent)
@@ -574,6 +578,9 @@ export function useForkSync(params: {
   const reconfigPendingByKind = new Set<MappableMappingKind>()
   for (const field of dependentReconfigs) {
     if (!field.required) continue
+    // Mirrors the Sync gate above: a field it cannot block on must not make the kind's badge
+    // read "Needs setup" forever either.
+    if (!isForkSyncConfigurableField(field)) continue
     const parent = entryForDependent(field)
     if (!parent) continue
     const resolution = resolutionFor(parent)