fix(oci-secrets): normalize native block optional parameters

Author: Bill Leoutsakos

apps/sim/blocks/blocks/oci_secrets.ts

@@ -896,7 +896,10 @@ export const OciSecretsBlock: BlockConfig = {
         compartmentId: params.compartmentId,
         secretId: params.secretId,
         secretName: params.secretName,
-        vaultId: params.vaultId,
+        vaultId:
+          params.operation === 'list_secrets' && (params.vaultId === null || params.vaultId === '')
+            ? undefined
+            : params.vaultId,
         keyId: params.keyId,
         name: params.name || undefined,
         lifecycleState: params.lifecycleState || undefined,

apps/sim/lib/internal/oci-secrets/execute-tool.test.ts

@@ -21,6 +21,8 @@ vi.mock('@/lib/internal/oci-secrets/operations', () => ({
 import { OciClientError } from '@/lib/internal/oci/errors'
 import { executeOciSecretsTool } from '@/lib/internal/oci-secrets/execute-tool'
 import type { InternalToolOperationCall } from '@/lib/internal/tool-operations/types'
+import { OciSecretsBlock } from '@/blocks/blocks/oci_secrets'
+import { ociSecretsListSecretsTool } from '@/tools/oci_secrets/list_secrets'
 
 const client = { request: vi.fn() }
 const access = {
@@ -91,6 +93,35 @@ describe('executeOciSecretsTool', () => {
     })
   })
 
+  it.each([null, '', undefined, 'vault-1'])(
+    'normalizes the optional native vault filter (%s)',
+    async (vaultId) => {
+      const raw = {
+        operation: 'list_secrets',
+        oauthCredential: 'selected-credential',
+        compartmentId: 'compartment-1',
+        vaultId,
+      }
+      const params = { ...raw, ...OciSecretsBlock.tools.config?.params?.(raw) }
+      const response = await executeOciSecretsTool(
+        request({
+          toolId: ociSecretsListSecretsTool.id,
+          input: ociSecretsListSecretsTool.operation.input(params),
+        })
+      )
+      expect(response.status).toBe(200)
+      expect(mocks.execute).toHaveBeenCalledWith(
+        expect.anything(),
+        expect.objectContaining({
+          operation: 'list_secrets',
+          vaultId: vaultId || undefined,
+          compartmentId: 'compartment-1',
+        }),
+        undefined
+      )
+    }
+  )
+
   it.each(operations)(
     'dispatches %s using the registered ID and trusted authority',
     async (operation) => {