refactor(managed-agent): remove self-hosted default-row pre-seeding

Pre-seeding the self-hosted block's Session parameters table from the
server-only MANAGED_AGENT_SELF_HOSTED_DEFAULTS env var is better served
by Sim's workflow blocks. Remove the capability end-to-end:

- Delete GET /api/managed-agent-defaults route + test
- Drop the MANAGED_AGENT_SELF_HOSTED_DEFAULTS env var
- Remove getManagedAgentDefaultsContract + its schemas/types
- Remove the fetchManagedAgentSelfHostedDefaults client fetcher
- Unwire fetchDefaultRows from the self-hosted block and fix the two
  stale comments that referenced the removed env var

Revert the shared table subblock seeding machinery (table.tsx,
sub-block.tsx, blocks/types.ts) to its pre-feature behavior so no other
block is affected — table.tsx is now byte-identical to main.

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

Author: rdematos

apps/sim/app/api/managed-agent-defaults/route.test.ts

@@ -15,7 +15,6 @@ import { useSubBlockInput } from '@/app/workspace/[workspaceId]/w/[workflowId]/c
 import { useSubBlockValue } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/components/sub-block/hooks/use-sub-block-value'
 import { useActiveSearchTarget } from '@/app/workspace/[workspaceId]/w/[workflowId]/components/panel/components/editor/providers/active-search-target-provider'
 import { useAccessibleReferencePrefixes } from '@/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-accessible-reference-prefixes'
-import { useSubBlockStore } from '@/stores/workflows/subblock/store'
 
 const logger = createLogger('Table')
 
@@ -26,21 +25,6 @@ interface TableProps {
   isPreview?: boolean
   previewValue?: WorkflowTableRow[] | null
   disabled?: boolean
-  /**
-   * Optional seed rows applied when a fresh block first mounts (store value
-   * is missing/empty). Each entry's `cells` keys must match `columns` — any
-   * missing columns fall back to `""`. Existing values are never overwritten.
-   */
-  defaultRows?: Array<{ cells: Record<string, string> }>
-  /**
-   * Optional async fetcher for seed rows — used when the block's default
-   * rows come from a server-side source (e.g. deployer-configured env vars
-   * read via an API route). Called once on first mount when the store is
-   * empty. If the fetcher rejects or returns an empty array, falls back to
-   * `defaultRows`, then to a single empty row. Same overwrite rules as
-   * `defaultRows`: existing store values are never touched.
-   */
-  fetchDefaultRows?: () => Promise<Array<{ cells: Record<string, string> }>>
 }
 
 interface WorkflowTableRow {
@@ -230,8 +214,6 @@ export function Table({
   isPreview = false,
   previewValue,
   disabled = false,
-  defaultRows,
-  fetchDefaultRows,
 }: TableProps) {
   const activeSearchTarget = useActiveSearchTarget()
   const params = useParams()
@@ -266,61 +248,18 @@ export function Table({
   )
 
   /**
-   * Initialize the table when the component mounts and the store value has
-   * never been set. Precedence:
-   *   1. `fetchDefaultRows` (async) — used when defaults live server-side
-   *      (e.g. deployer env-var read via an API route).
-   *   2. `defaultRows` (sync) — static seeds declared on the block config.
-   *   3. Single empty row — the pre-existing behavior.
-   *
-   * We only seed when `storeValue` is `null` / `undefined` — a stored
-   * `[]` means the workflow author intentionally cleared every row and
-   * must NOT be treated as "unseeded" (would re-seed defaults on every
-   * remount and clobber the empty state they chose).
+   * Initialize the table with a default empty row when the component mounts
+   * and when the current store value is missing or empty.
    */
   useEffect(() => {
-    if (isPreview || disabled) return
-    if (storeValue !== undefined && storeValue !== null) return
-    let cancelled = false
-    const seedWith = (rows: Array<{ cells: Record<string, string> }> | undefined) => {
-      if (cancelled) return
-      // Re-read the LATEST value from the subblock store, not the
-      // closure-captured `storeValue` from render time. Otherwise an
-      // async `fetchDefaultRows` that resolves after the user has
-      // already started editing the table would overwrite their
-      // in-progress rows with the deployer defaults.
-      const currentValue = useSubBlockStore.getState().getValue(blockId, subBlockId)
-      if (currentValue !== undefined && currentValue !== null) return
-      const seedRows: WorkflowTableRow[] =
-        Array.isArray(rows) && rows.length > 0
-          ? rows.map((row) => ({
-              id: generateId(),
-              cells: { ...emptyCellsTemplate, ...(row.cells ?? {}) },
-            }))
-          : [{ id: generateId(), cells: { ...emptyCellsTemplate } }]
-      setStoreValue(seedRows)
-    }
-    if (fetchDefaultRows) {
-      fetchDefaultRows()
-        .then((rows) => seedWith(rows.length > 0 ? rows : defaultRows))
-        .catch(() => seedWith(defaultRows))
-    } else {
-      seedWith(defaultRows)
-    }
-    return () => {
-      cancelled = true
+    if (!isPreview && !disabled && (!Array.isArray(storeValue) || storeValue.length === 0)) {
+      const initialRow: WorkflowTableRow = {
+        id: generateId(),
+        cells: { ...emptyCellsTemplate },
+      }
+      setStoreValue([initialRow])
     }
-  }, [
-    isPreview,
-    disabled,
-    storeValue,
-    setStoreValue,
-    emptyCellsTemplate,
-    defaultRows,
-    fetchDefaultRows,
-    blockId,
-    subBlockId,
-  ])
+  }, [isPreview, disabled, storeValue, setStoreValue, emptyCellsTemplate])
 
   // Ensure value is properly typed and initialized
   const rows = useMemo(() => {

apps/sim/app/api/managed-agent-defaults/route.ts

@@ -749,12 +749,6 @@ function SubBlockComponent({
             isPreview={isPreview}
             previewValue={previewValue as any}
             disabled={isDisabled}
-            defaultRows={
-              Array.isArray(config.defaultValue)
-                ? (config.defaultValue as Array<{ cells: Record<string, string> }>)
-                : undefined
-            }
-            fetchDefaultRows={config.fetchDefaultRows}
           />
         )
 

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

@@ -19,7 +19,6 @@ vi.mock('@/lib/managed-agents/subblock-options', () => ({
   fetchManagedAgentAgentOptions: vi.fn(),
   fetchManagedAgentConnectionOptions: vi.fn(),
   fetchManagedAgentMemoryStoreOptions: vi.fn(),
-  fetchManagedAgentSelfHostedDefaults: vi.fn(),
   fetchManagedAgentSelfHostedEnvironmentOptions: vi.fn(),
   fetchManagedAgentVaultOptions: vi.fn(),
 }))

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

@@ -4,7 +4,6 @@ import {
   fetchManagedAgentAgentOptions,
   fetchManagedAgentConnectionOptions,
   fetchManagedAgentMemoryStoreOptions,
-  fetchManagedAgentSelfHostedDefaults,
   fetchManagedAgentSelfHostedEnvironmentOptions,
   fetchManagedAgentVaultOptions,
 } from '@/lib/managed-agents/subblock-options'
@@ -65,8 +64,7 @@ const memorySubBlocks: SubBlockConfig[] = isSelfHostedMemoryEnabled()
  * `resources` array. Session metadata (top-level `metadata` field) is
  * exposed to the self-hosted agent sandbox as env vars, so this
  * block's Session parameters table is where you set the keys your
- * deployment reads. The set of supported keys is deployment-specific
- * — seed defaults with `NEXT_PUBLIC_MANAGED_AGENT_SELF_HOSTED_DEFAULTS`.
+ * deployment reads. The set of supported keys is deployment-specific.
  *
  * The environment picker is filtered to `config.type === 'self_hosted'`.
  * Icon renders black-on-white to visually differentiate from the cloud
@@ -161,17 +159,12 @@ export const ManagedAgentSelfHostedBlock: BlockConfig = {
     {
       // Session metadata forwarded to the self-hosted agent sandbox as
       // env vars. The set of supported keys is deployment-specific and
-      // lives with the deployer, not in this repo. Seed rows come from
-      // the server-only `MANAGED_AGENT_SELF_HOSTED_DEFAULTS` env var,
-      // fetched via `/api/managed-agent-defaults` — the values never
-      // enter the client bundle, so deployers can safely include
-      // anything their sandbox reads.
+      // lives with the deployer, not in this repo.
       id: 'sessionParameters',
       title: 'Session parameters',
       type: 'table',
       required: false,
       columns: ['Key', 'Value'],
-      fetchDefaultRows: fetchManagedAgentSelfHostedDefaults,
       description:
         'Key/value pairs forwarded to the self-hosted agent sandbox as environment variables. Supported keys depend on your deployment — consult your deployment docs. Value cells support <block.output> / <var.name> references.',
     },

apps/sim/blocks/blocks/managed_agent_self_hosted.test.ts

@@ -285,13 +285,6 @@ export interface SubBlockConfig {
         }
       })
   defaultValue?: string | number | boolean | Record<string, unknown> | Array<unknown>
-  /**
-   * Optional async fetcher for a `table` subblock's initial rows — used
-   * when defaults come from a server-side source (e.g. deployer-configured
-   * env vars) and must not be inlined into the client bundle. Called once
-   * on first mount when the store is empty. See `Table.fetchDefaultRows`.
-   */
-  fetchDefaultRows?: () => Promise<Array<{ cells: Record<string, string> }>>
   options?:
     | {
         label: string

apps/sim/blocks/blocks/managed_agent_self_hosted.ts

@@ -204,28 +204,3 @@ export const listManagedAgentMemoryStoresContract = defineRouteContract({
     }),
   },
 })
-
-/**
- * Server-side default rows the Claude Managed Agents (self-hosted) block
- * seeds into its Session parameters table. Deployers configure this via
- * the server-only `MANAGED_AGENT_SELF_HOSTED_DEFAULTS` env var (a JSON
- * object) so seeded values — which may reference internal keys — never
- * touch the client bundle.
- */
-export const managedAgentSelfHostedDefaultRowSchema = z.object({
-  cells: z.record(z.string(), z.string()),
-})
-export type ManagedAgentSelfHostedDefaultRow = z.output<
-  typeof managedAgentSelfHostedDefaultRowSchema
->
-
-export const getManagedAgentDefaultsContract = defineRouteContract({
-  method: 'GET',
-  path: '/api/managed-agent-defaults',
-  response: {
-    mode: 'json',
-    schema: z.object({
-      selfHosted: z.array(managedAgentSelfHostedDefaultRowSchema),
-    }),
-  },
-})

apps/sim/blocks/types.ts

@@ -149,7 +149,6 @@ export const env = createEnv({
     OPENAI_API_KEY_3:                      z.string().min(1).optional(),           // Additional OpenAI API key for load balancing
     MISTRAL_API_KEY:                       z.string().min(1).optional(),           // Mistral AI API key
     MANAGED_AGENT_DEBUG_PAYLOAD:           z.string().optional(),                  // When "1" | "true" | "yes", the Managed Agent workflow-block tool logs the full session-create payload (never includes the api key) so you can inspect memory / vault / metadata routing
-    MANAGED_AGENT_SELF_HOSTED_DEFAULTS:    z.string().optional(),                  // Server-only JSON object of default session-metadata key/value pairs the Claude Managed Agents (self-hosted) block seeds into fresh workflows. Example: '{"SOURCE_TYPE":"git","SOURCE_REF":"main"}'. Never exposed to the client bundle — read via GET /api/managed-agent-defaults.
     ANTHROPIC_API_KEY_1:                   z.string().min(1).optional(),           // Primary Anthropic Claude API key
     ANTHROPIC_API_KEY_2:                   z.string().min(1).optional(),           // Additional Anthropic API key for load balancing
     ANTHROPIC_API_KEY_3:                   z.string().min(1).optional(),           // Additional Anthropic API key for load balancing