fix(buffer): normalize cleared inputs and complete test scope

Author: waleedlatif1

apps/sim/blocks/blocks/buffer.ts

@@ -666,7 +666,11 @@ export const BufferBlock: BlockConfig = {
             key === 'operation'
           )
             continue
-          if (value === undefined || value === null || value === '') continue
+          if (value === undefined || value === null) continue
+          if (value === '') {
+            result[key] = undefined
+            continue
+          }
           if ((key === 'mode' || key === 'schedulingType') && value === 'default') {
             result[key] =
               params.operation === 'create_post'

apps/sim/executor/handlers/generic/generic-handler.test.ts

@@ -80,7 +80,13 @@ describe('GenericBlockHandler', () => {
     mockExecuteTool.mockResolvedValue({ success: true, output: { customResult: 'OK' } })
   })
 
-  it('preserves the current Buffer schedule and approval state when editing only text', async () => {
+  it.each([
+    {
+      selection: 'keep-current scheduling',
+      optional: { mode: 'default', schedulingType: 'default', approvalChange: 'default' },
+    },
+    { selection: 'cleared structured editors', optional: { assets: '', metadata: '' } },
+  ])('preserves existing Buffer state for a caption edit with $selection', async ({ optional }) => {
     mockGetBlock.mockReturnValue(BufferBlock)
     mockGetTool.mockReturnValue(bufferEditPostTool)
     const providerInputs: Record<string, unknown>[] = []
@@ -115,9 +121,7 @@ describe('GenericBlockHandler', () => {
         apiKey: 'buffer-key',
         postId: 'post-1',
         text: 'Updated caption',
-        mode: 'default',
-        schedulingType: 'default',
-        approvalChange: 'default',
+        ...optional,
       }
     )
 

apps/sim/scripts/test-buffer-e2e.ts

@@ -53,7 +53,7 @@ async function tool(id: string, params: Record<string, unknown> = {}) {
     { apiKey, ...params },
     {
       signal: AbortSignal.timeout(60_000),
-      operationContext: { userId: fixtureId, workflowId: fixtureId },
+      operationContext: { userId: fixtureId, workflowId: fixtureId, workspaceId: fixtureId },
     }
   )
   assert(result.success, result.error ?? `${id} failed`)