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

Author: Bill Leoutsakos

apps/sim/blocks/blocks/oci_compute.ts

@@ -1686,7 +1686,7 @@ export const OciComputeBlock: BlockConfig<OciComputeResponse> = {
               integer: true,
               min: field === 'size' ? 0 : 1,
             })
-          } else delete result[field]
+          } else result[field] = undefined
         }
         const booleanOperations: Record<string, readonly string[]> = {
           allowDenseRebootMigration: ALLOW_DENSE_REBOOT_MIGRATION_OPERATIONS,
@@ -1699,7 +1699,7 @@ export const OciComputeBlock: BlockConfig<OciComputeResponse> = {
         }
         for (const [field, operations] of Object.entries(booleanOperations)) {
           if (operations.includes(params.operation)) result[field] = optionalBoolean(params[field])
-          else delete result[field]
+          else result[field] = undefined
         }
         if (params.operation === 'oci_compute_create_instance_pool') {
           if (result.instanceDisplayNameFormatter === '')
@@ -1708,7 +1708,7 @@ export const OciComputeBlock: BlockConfig<OciComputeResponse> = {
         }
         if (
           params.operation === 'oci_compute_list_instances' &&
-          result.capacityReservationId === ''
+          (result.capacityReservationId === '' || result.capacityReservationId === null)
         ) {
           result.capacityReservationId = undefined
         }
@@ -1760,7 +1760,11 @@ export const OciComputeBlock: BlockConfig<OciComputeResponse> = {
           'resourceId',
           'workRequestId',
         ]) {
-          if (result[field] === '') delete result[field]
+          if (
+            result[field] === '' ||
+            (result[field] === null && params.operation.startsWith('oci_compute_list_'))
+          )
+            result[field] = undefined
         }
         if (params.operation === 'oci_compute_launch_instance') {
           let vnic = result.createVnicDetails

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

@@ -18,6 +18,13 @@ vi.mock('@/lib/internal/oci-compute/operations', () => ({
 }))
 
 import { executeOciComputeTool } from '@/lib/internal/oci-compute/execute-tool'
+import { OciComputeBlock } from '@/blocks/blocks/oci_compute'
+import { ociComputeListImagesTool } from '@/tools/oci_compute/list_images'
+import { ociComputeListInstanceConfigurationsTool } from '@/tools/oci_compute/list_instance_configurations'
+import { ociComputeListInstancePoolsTool } from '@/tools/oci_compute/list_instance_pools'
+import { ociComputeListInstancesTool } from '@/tools/oci_compute/list_instances'
+import { ociComputeListShapesTool } from '@/tools/oci_compute/list_shapes'
+import { ociComputeListSubnetsTool } from '@/tools/oci_compute/list_subnets'
 
 function call(overrides: Partial<InternalToolOperationCall> = {}): InternalToolOperationCall {
   return {
@@ -42,6 +49,85 @@ beforeEach(() => {
 })
 
 describe('OCI Compute trusted execution wiring', () => {
+  it.each([
+    ociComputeListInstancesTool,
+    ociComputeListImagesTool,
+    ociComputeListShapesTool,
+    ociComputeListSubnetsTool,
+    ociComputeListInstanceConfigurationsTool,
+    ociComputeListInstancePoolsTool,
+  ])('normalizes native blank discovery filters for $id', async (tool) => {
+    for (const blank of [null, '', undefined]) {
+      const raw = {
+        operation: tool.id,
+        oauthCredential: 'submitted',
+        region: 'us-ashburn-1',
+        compartmentId: 'compartment',
+        limit: '10',
+        page: blank,
+        sortBy: blank,
+        sortOrder: blank,
+        displayName: blank,
+        availabilityDomain: blank,
+        lifecycleState: blank,
+        capacityReservationId: blank,
+        operatingSystemVersion: blank,
+        shape: blank,
+        imageId: blank,
+        vcnId: blank,
+      }
+      const params = { ...raw, ...OciComputeBlock.tools.config?.params?.(raw) }
+      const response = await executeOciComputeTool(
+        call({ toolId: tool.id, input: tool.operation.input(params) })
+      )
+      expect(response.status).toBe(200)
+      expect(mocks.execute).toHaveBeenLastCalledWith(
+        expect.anything(),
+        tool.id.replace('oci_compute_', ''),
+        expect.objectContaining({ compartmentId: 'compartment', limit: 10 }),
+        undefined
+      )
+      expect(mocks.execute.mock.lastCall?.[2].page).toBeUndefined()
+    }
+  })
+
+  it('keeps meaningful zero, false and empty mutation values while rejecting invalid list input', async () => {
+    const raw = {
+      operation: 'oci_compute_update_instance',
+      capacityReservationId: '',
+      preserveBootVolume: false,
+    }
+    const params = { ...raw, ...OciComputeBlock.tools.config?.params?.(raw) }
+    expect(params.capacityReservationId).toBe('')
+    const resized = OciComputeBlock.tools.config?.params?.({
+      operation: 'oci_compute_update_instance_pool',
+      size: 0,
+    })
+    expect(resized?.size).toBe(0)
+    const terminated = OciComputeBlock.tools.config?.params?.({
+      operation: 'oci_compute_terminate_instance',
+      preserveBootVolume: false,
+    })
+    expect(terminated?.preserveBootVolume).toBe(false)
+    const invalid = {
+      operation: ociComputeListInstancesTool.id,
+      oauthCredential: 'submitted',
+      region: 'us-ashburn-1',
+      compartmentId: 'compartment',
+      page: 12,
+    }
+    const response = await executeOciComputeTool(
+      call({
+        toolId: ociComputeListInstancesTool.id,
+        input: ociComputeListInstancesTool.operation.input({
+          ...invalid,
+          ...OciComputeBlock.tools.config?.params?.(invalid),
+        }),
+      })
+    )
+    expect(response.status).toBe(400)
+    expect(mocks.execute).not.toHaveBeenCalled()
+  })
   it('authorizes submitted identity and binds only the resolved credential and trusted scope', async () => {
     const signal = new AbortController().signal
     expect((await executeOciComputeTool(call({ signal }))).status).toBe(200)

apps/sim/lib/internal/oci-compute/projection-contract.test.ts

@@ -102,7 +102,7 @@ describe('OCI Compute input and resource projections', () => {
       size: 0,
       instanceDisplayNameFormatter: '',
     })
-    expect(values).not.toHaveProperty('isAutoTerminate')
+    expect(values.isAutoTerminate).toBeUndefined()
     expect(
       normalize({
         operation: 'oci_compute_detach_instance_pool_instance',