refactor(sub-blocks): delete fetchOptions — a sub-block's options are a selector or derived, never both

Completes the migration. `fetchOptions`/`fetchOptionById` are off `SubBlockConfig`,
off both controls, and out of `useFetchedOptions`, leaving exactly two ways a
sub-block gets its options:

  selectorKey  — a registered selector. The ONLY way to load a remote list.
                 Parameterized by an explicit SelectorContext, so it works on the
                 canvas, in the fork sync modal, and anywhere else.
  options      — a static array, or a pure function of the block's own values.
                 No I/O.

Reading the remaining callsites showed most of the "derived" ones were nothing of
the kind — they were workspace-scoped remote fetches wearing a local-looking
signature. Those became seven `workspace.*` selectors (credential providers,
credential groups + their per-group providers, secret names, raw secret names,
sandboxes, trigger types) plus `providers.openrouterEmbeddingModels`. Only the
agent block's three capability dropdowns were genuinely derived; `options` now
takes the block's values so they can say so directly. The parameter is optional,
so every existing zero-argument options function is untouched.

`imap.mailboxes` is the one selector whose account is typed rather than stored.
Its password is deliberately absent from the query key: a query key identifies a
resource, a credential authorizes access to it. `oauthCredential` is safe there
because it is only an id — a typed password is a secret, and keys are cached and
surfaced by devtools. Host, port, TLS and username already identify the mailbox
list uniquely; the password rides the body exactly as before.

`selectorExcludeSelf` replaces the one thing a shared `sim.workflows` selector
could not express. It is a declared flag rather than a blanket rule because the
answer differs per field: the Sim trigger never receives events about its own
workflow, while the Logs block legitimately reads the logs of the workflow it
runs in.

Deletes `lib/workflows/subblocks/options.ts` and `triggers/editor-state.ts`
entirely — every caller was a `fetchOptions` resolver. The live-registry test for
the trigger vocabulary moves to the selector that now owns it, keeping its
lazy-import cycle guarantee under test.

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

Author: icecrasher321

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

@@ -71,13 +71,8 @@ interface ComboBoxProps {
   config: SubBlockConfig
   /** Registered selector supplying the options. The canonical source for a remote list. */
   selectorKey?: SelectorKey
-  /** Async function to fetch options dynamically */
-  fetchOptions?: (blockId: string) => Promise<Array<{ label: string; id: string }>>
-  /** Async function to fetch a single option's label by ID (for hydration) */
-  fetchOptionById?: (
-    blockId: string,
-    optionId: string
-  ) => Promise<{ label: string; id: string } | null>
+  /** Drop the hosting workflow from a `sim.workflows` list. */
+  selectorExcludeSelf?: boolean
   /** Field dependencies that trigger option refetch when changed */
   dependsOn?: SubBlockConfig['dependsOn']
 }
@@ -94,8 +89,7 @@ export const ComboBox = memo(function ComboBox({
   placeholder = 'Type or select an option...',
   config,
   selectorKey,
-  fetchOptions,
-  fetchOptionById,
+  selectorExcludeSelf,
   dependsOn,
 }: ComboBoxProps) {
   const activeSearchTarget = useActiveSearchTarget()
@@ -136,8 +130,7 @@ export const ComboBox = memo(function ComboBox({
     blockId,
     dependsOnFields,
     selectorKey,
-    fetchOptions,
-    fetchOptionById,
+    selectorExcludeSelf,
     isPreview: Boolean(isPreview),
     disabled: Boolean(disabled),
     valueToHydrate: value as string | null | undefined,
@@ -202,7 +195,7 @@ export const ComboBox = memo(function ComboBox({
     let opts: ComboBoxOption[] =
       isDynamic && normalizedFetchedOptions.length > 0 ? normalizedFetchedOptions : staticOptions
 
-    if (subBlockId === 'model' && fetchOptions && normalizedFetchedOptions.length > 0) {
+    if (subBlockId === 'model' && isDynamic && normalizedFetchedOptions.length > 0) {
       opts = opts.filter((opt) => isModelUsable(typeof opt === 'string' ? opt : opt.id))
     }
 
@@ -230,7 +223,7 @@ export const ComboBox = memo(function ComboBox({
 
     return opts
   }, [
-    fetchOptions,
+    isDynamic,
     normalizedFetchedOptions,
     staticOptions,
     hydratedOption,

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

@@ -17,6 +17,8 @@ import type { SubBlockConfig } from '@/blocks/types'
 import { ResponseBlockHandler } from '@/executor/handlers/response/response-handler'
 import type { SelectorKey } from '@/hooks/selectors/types'
 import { useOperationAccess } from '@/hooks/use-operation-access'
+import { useWorkflowRegistry } from '@/stores/workflows/registry/store'
+import { useSubBlockStore } from '@/stores/workflows/subblock/store'
 import { useWorkflowStore } from '@/stores/workflows/workflow/store'
 
 /** Selected-value badges shown before folding the rest into a "+N" badge. */
@@ -41,7 +43,7 @@ type DropdownOption =
  */
 interface DropdownProps {
   /** Static options array or function that returns options */
-  options: DropdownOption[] | (() => DropdownOption[])
+  options: DropdownOption[] | ((params?: { values: Record<string, unknown> }) => DropdownOption[])
   /** Default value to select when no value is set */
   defaultValue?: string
   /** Unique identifier for the block */
@@ -62,13 +64,8 @@ interface DropdownProps {
   multiSelect?: boolean
   /** Registered selector supplying the options. The canonical source for a remote list. */
   selectorKey?: SelectorKey
-  /** Async function to fetch options dynamically */
-  fetchOptions?: (blockId: string) => Promise<Array<{ label: string; id: string }>>
-  /** Async function to fetch a single option's label by ID (for hydration) */
-  fetchOptionById?: (
-    blockId: string,
-    optionId: string
-  ) => Promise<{ label: string; id: string } | null>
+  /** Drop the hosting workflow from a `sim.workflows` list. */
+  selectorExcludeSelf?: boolean
   /** Field dependencies that trigger option refetch when changed */
   dependsOn?: SubBlockConfig['dependsOn']
   /** Enable search input in dropdown */
@@ -98,8 +95,7 @@ export const Dropdown = memo(function Dropdown({
   placeholder = 'Select an option...',
   multiSelect = false,
   selectorKey,
-  fetchOptions,
-  fetchOptionById,
+  selectorExcludeSelf,
   dependsOn,
   searchable = false,
   preserveLabelCase = false,
@@ -140,9 +136,15 @@ export const Dropdown = memo(function Dropdown({
         : []
     : null
 
+  // Derived option lists read the block's own values (a model's valid reasoning efforts);
+  // `dependsOn` already re-renders this control when one of those siblings changes.
+  const activeWorkflowId = useWorkflowRegistry((state) => state.activeWorkflowId)
+  const blockValues = useSubBlockStore((state) =>
+    activeWorkflowId ? state.workflowValues[activeWorkflowId]?.[blockId] : undefined
+  )
   const evaluatedOptions = useMemo(() => {
-    return typeof options === 'function' ? options() : options
-  }, [options])
+    return typeof options === 'function' ? options({ values: blockValues ?? {} }) : options
+  }, [options, blockValues])
 
   const {
     fetchedOptions,
@@ -155,8 +157,7 @@ export const Dropdown = memo(function Dropdown({
     blockId,
     dependsOnFields,
     selectorKey,
-    fetchOptions,
-    fetchOptionById,
+    selectorExcludeSelf,
     isPreview: Boolean(isPreview),
     disabled: Boolean(disabled),
     valueToHydrate: singleValue,

apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/hooks/use-fetched-options.ts

@@ -33,8 +33,8 @@ interface UseFetchedOptionsProps {
    * work on the canvas, and was in every case a duplicate of a selector that already existed.
    */
   selectorKey?: SelectorKey
-  fetchOptions?: (blockId: string) => Promise<FetchedOption[]>
-  fetchOptionById?: (blockId: string, optionId: string) => Promise<FetchedOption | null>
+  /** Drop the hosting workflow from the list — see `SubBlockConfig.selectorExcludeSelf`. */
+  selectorExcludeSelf?: boolean
   isPreview: boolean
   disabled: boolean
   /**
@@ -82,8 +82,7 @@ export function useFetchedOptions({
   blockId,
   dependsOnFields,
   selectorKey,
-  fetchOptions: fetchOptionsProp,
-  fetchOptionById: fetchOptionByIdProp,
+  selectorExcludeSelf,
   isPreview,
   disabled,
   valueToHydrate,
@@ -128,12 +127,14 @@ export function useFetchedOptions({
       : {}
     const merged: Record<string, { value?: unknown }> = { ...(block.subBlocks ?? {}) }
     for (const [id, value] of Object.entries(live)) merged[id] = { ...merged[id], value }
-    return buildSelectorContextFromBlock(block.type, merged, {
+    const context = buildSelectorContextFromBlock(block.type, merged, {
       workflowId: activeWorkflowId ?? undefined,
       workspaceId: workspaceId ?? undefined,
       canonicalModes: block.data?.canonicalModes,
     })
-  }, [blockId, activeWorkflowId, workspaceId])
+    if (selectorExcludeSelf && activeWorkflowId) context.excludeWorkflowId = activeWorkflowId
+    return context
+  }, [blockId, activeWorkflowId, workspaceId, selectorExcludeSelf])
 
   const selectorDefinition = selectorKey ? getSelectorDefinition(selectorKey) : undefined
 
@@ -146,7 +147,6 @@ export function useFetchedOptions({
    * new identity and the scope reset below refetches, exactly as it does for a prop fetcher.
    */
   const fetchOptions = useMemo(() => {
-    if (fetchOptionsProp) return fetchOptionsProp
     if (!selectorDefinition) return undefined
     const definition = selectorDefinition
     return async (): Promise<FetchedOption[]> => {
@@ -162,11 +162,10 @@ export function useFetchedOptions({
       return options.map((option) => ({ id: option.id, label: option.label }))
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps -- dependencyValues is the refetch scope
-  }, [fetchOptionsProp, selectorDefinition, readSelectorContext, dependencyValues])
+  }, [selectorDefinition, readSelectorContext, dependencyValues])
 
   /** Label hydration for a stored id, from the same definition. */
   const fetchOptionById = useMemo(() => {
-    if (fetchOptionByIdProp) return fetchOptionByIdProp
     const definition = selectorDefinition
     const fetchById = definition?.fetchById
     if (!definition || !fetchById) return undefined
@@ -176,7 +175,7 @@ export function useFetchedOptions({
       const option = await fetchById({ key: definition.key, context, detailId: optionId, signal })
       return option ? { id: option.id, label: option.label } : null
     }
-  }, [fetchOptionByIdProp, selectorDefinition, readSelectorContext])
+  }, [selectorDefinition, readSelectorContext])
 
   const [fetchedOptions, setFetchedOptions] = useState<FetchedOption[]>([])
   const [isLoadingOptions, setIsLoadingOptions] = useState(false)
@@ -204,7 +203,7 @@ export function useFetchedOptions({
     setIsLoadingOptions(true)
     setFetchError(null)
     try {
-      const options = await fetchOptions(blockId)
+      const options = await fetchOptions()
       if (requestId !== fetchRequestIdRef.current) return
       setFetchedOptions(options)
     } catch (error) {

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

@@ -681,8 +681,7 @@ function SubBlockComponent({
               disabled={isDisabled}
               multiSelect={config.multiSelect}
               selectorKey={config.selectorKey}
-              fetchOptions={config.fetchOptions}
-              fetchOptionById={config.fetchOptionById}
+              selectorExcludeSelf={config.selectorExcludeSelf}
               dependsOn={config.dependsOn}
               searchable={config.searchable}
               preserveLabelCase={config.preserveLabelCase}
@@ -717,8 +716,7 @@ function SubBlockComponent({
               disabled={isDisabled}
               config={config}
               selectorKey={config.selectorKey}
-              fetchOptions={config.fetchOptions}
-              fetchOptionById={config.fetchOptionById}
+              selectorExcludeSelf={config.selectorExcludeSelf}
               dependsOn={config.dependsOn}
             />
           </div>

apps/sim/blocks/blocks.test.ts

@@ -904,7 +904,7 @@ describe.concurrent('Blocks Module', () => {
           (sb) => sb.id === 'model' && sb.condition?.value === provider
         )
         if (provider === 'openrouter') {
-          expect(modelSubBlock?.fetchOptions).toBeTypeOf('function')
+          expect(modelSubBlock?.selectorKey).toBeTypeOf('string')
         } else {
           expect(
             Array.isArray(modelSubBlock?.options) ? modelSubBlock.options.length : 0