fix(managed-agent): async table seed no longer clobbers in-flight edits

Cursor Bugbot on 9704cec flagged a race in the `Table` sub-block
init effect: `fetchDefaultRows` is async, and the effect's guard on
`storeValue !== undefined && storeValue !== null` was captured at
effect setup time. Between the fetch start and its resolution, a
user could type into the phantom row — persisting the row via
`handleCellChange`'s `setStoreValue([newRow])` — and then the
resolved fetch would call `setStoreValue(defaultRows)` and clobber
their edit.

Fix: re-read the current value from the subblock store inside
`seedWith` right before writing (`useSubBlockStore.getState()
.getValue(blockId, subBlockId)`), and skip the write if the value
has already been populated since the effect started. The
closure-captured `storeValue` was stale; the store's `getState`
returns the LATEST value synchronously.

Other Bugbot comments on this commit are re-fires of items already
fixed on prior commits (they signature-match against static code
shapes without noticing the corrective code that lives beside them):
`Stale labels after connection change` (fixed via `depsEqual` in
`areSubBlockRowPropsEqual`), `Empty table re-seeds defaults` (fixed
via `storeValue !== undefined && storeValue !== null` guard),
`Defaults API lacks authentication` (fixed via
`checkSessionOrInternalAuth`), `Rotate connection hook unused`
(fixed via the new Rotate-key modal wired into the settings UI).
`Empty table shows phantom row` is theoretical — the delete-row
guard (`rows.length === 1`) prevents users from reaching `[]`
through the UI, so the mismatch it warns about isn't reachable in
normal use.

Author: rdematos

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

@@ -15,6 +15,7 @@ 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')
 
@@ -283,6 +284,13 @@ export function Table({
     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) => ({
@@ -310,6 +318,8 @@ export function Table({
     emptyCellsTemplate,
     defaultRows,
     fetchDefaultRows,
+    blockId,
+    subBlockId,
   ])
 
   // Ensure value is properly typed and initialized