fix(oracle-epcm): address independent integration review

Author: Bill Leoutsakos

apps/docs/content/docs/integrations/oracle_epm_enterprise_profitability.mdx

@@ -512,7 +512,7 @@ Run an existing saved metadata import. ZIP entries must match configured dimensi
 | --------- | ---- | -------- | ----------- |
 | `oauthCredential` | string | Yes | Reusable Oracle EPM service-account credential for an EPCM tenant |
 | `applicationName` | string | Yes | Exact EPCM application name |
-| `jobName` | string | Yes | Exact saved exchange-job name; optional only with complete ad hoc data parameters |
+| `jobName` | string | Yes | Exact saved metadata import job name; an existing job is required |
 | `fileName` | string | No | Optional repository ZIP filename overriding the saved job's files |
 | `refreshCube` | boolean | No | Override the saved job's cube refresh option |
 
@@ -537,7 +537,7 @@ Run an existing saved metadata export to a repository ZIP file.
 | --------- | ---- | -------- | ----------- |
 | `oauthCredential` | string | Yes | Reusable Oracle EPM service-account credential for an EPCM tenant |
 | `applicationName` | string | Yes | Exact EPCM application name |
-| `jobName` | string | Yes | Exact saved exchange-job name; optional only with complete ad hoc data parameters |
+| `jobName` | string | Yes | Exact saved metadata export job name; an existing job is required |
 | `fileName` | string | No | Optional output ZIP filename; existing output may be replaced by Oracle |
 
 #### Output

apps/sim/blocks/blocks/oracle_epm_enterprise_profitability.ts

@@ -1315,6 +1315,7 @@ export const OracleEpcmBlock: BlockConfig<OracleEpcmResponse> = {
       id: 'offset',
       title: 'Offset',
       type: 'short-input',
+      mode: 'advanced',
       condition: {
         field: 'operation',
         value: [
@@ -1329,6 +1330,7 @@ export const OracleEpcmBlock: BlockConfig<OracleEpcmResponse> = {
       id: 'limit',
       title: 'Limit',
       type: 'short-input',
+      mode: 'advanced',
       condition: {
         field: 'operation',
         value: [
@@ -1631,6 +1633,11 @@ export const OracleEpcmBlock: BlockConfig<OracleEpcmResponse> = {
       id: 'rowMembers',
       title: 'Row Members',
       type: 'long-input',
+      wandConfig: {
+        enabled: true,
+        prompt: `Format the user's explicitly supplied EPCM export row selections as a comma-separated member list, for example Revenue,Expenses, or preserve a supplied expression such as ILvl0Descendants(&RowMembers). Preserve exact tenant names, expressions, and substitution variables. Do not invent selections. Return only the list or expression, without JSON quotes, markdown, or explanation.`,
+        placeholder: 'Provide the exact row members or member expression',
+      },
       condition: {
         field: 'operation',
         value: ['oracle_epm_enterprise_profitability_export_data'],
@@ -1641,6 +1648,11 @@ export const OracleEpcmBlock: BlockConfig<OracleEpcmResponse> = {
       id: 'columnMembers',
       title: 'Column Members',
       type: 'long-input',
+      wandConfig: {
+        enabled: true,
+        prompt: `Format the user's explicitly supplied EPCM export column selections as a comma-separated member list, for example Jan,Feb, or preserve a supplied expression such as ILvl0Descendants(&ColumnMembers). Preserve exact tenant names, expressions, and substitution variables. Do not invent selections. Return only the list or expression, without JSON quotes, markdown, or explanation.`,
+        placeholder: 'Provide the exact column members or member expression',
+      },
       condition: {
         field: 'operation',
         value: ['oracle_epm_enterprise_profitability_export_data'],
@@ -1649,8 +1661,13 @@ export const OracleEpcmBlock: BlockConfig<OracleEpcmResponse> = {
     },
     {
       id: 'povMembers',
-      title: 'Pov Members',
+      title: 'POV Members',
       type: 'long-input',
+      wandConfig: {
+        enabled: true,
+        prompt: `Format the user's explicitly supplied EPCM export POV selections as a comma-separated member list, for example Actual,FY26,Working,&Entity. Preserve exact tenant names, member expressions, substitution variables, and dimension order. Do not invent selections. Return only the list or expression, without JSON quotes, markdown, or explanation.`,
+        placeholder: 'Provide the exact POV members in cube dimension order',
+      },
       condition: {
         field: 'operation',
         value: ['oracle_epm_enterprise_profitability_export_data'],

apps/sim/blocks/registry-maps.ts

@@ -590,7 +590,6 @@ export const BLOCK_REGISTRY: Record<string, BlockConfig> = {
   mysql: MySQLBlock,
   neo4j: Neo4jBlock,
   netsuite: NetSuiteBlock,
-  oracle_epm_enterprise_profitability: OracleEpcmBlock,
   new_relic: NewRelicBlock,
   note: NoteBlock,
   notion: NotionBlock,
@@ -600,6 +599,7 @@ export const BLOCK_REGISTRY: Record<string, BlockConfig> = {
   onedrive: OneDriveBlock,
   onepassword: OnePasswordBlock,
   openai: OpenAIBlock,
+  oracle_epm_enterprise_profitability: OracleEpcmBlock,
   outlook: OutlookBlock,
   pagerduty: PagerDutyBlock,
   parallel_ai: ParallelBlock,
@@ -913,7 +913,6 @@ export const BLOCK_META_REGISTRY: Record<string, BlockMeta> = {
   mysql: MySQLBlockMeta,
   neo4j: Neo4jBlockMeta,
   netsuite: NetSuiteBlockMeta,
-  oracle_epm_enterprise_profitability: OracleEpcmBlockMeta,
   neverbounce: NeverBounceBlockMeta,
   new_relic: NewRelicBlockMeta,
   notion: NotionBlockMeta,
@@ -923,6 +922,7 @@ export const BLOCK_META_REGISTRY: Record<string, BlockMeta> = {
   onedrive: OneDriveBlockMeta,
   onepassword: OnePasswordBlockMeta,
   openai: OpenAIBlockMeta,
+  oracle_epm_enterprise_profitability: OracleEpcmBlockMeta,
   outlook: OutlookBlockMeta,
   pagerduty: PagerDutyBlockMeta,
   parallel_ai: ParallelBlockMeta,

apps/sim/lib/internal/tool-operations/registry.server.ts

@@ -1454,13 +1454,13 @@ registerFamily(handlerLoaders, MICROSOFT_AD_TOOL_IDS, async () => {
 registerFamily(handlerLoaders, NETSUITE_TOOL_IDS, async () => {
   return (await import('@/lib/internal/netsuite/execute-tool')).executeNetsuiteTool
 })
+registerFamily(handlerLoaders, OKTA_TOOL_IDS, async () => {
+  return (await import('@/lib/internal/okta/execute-tool')).executeOktaTool
+})
 registerFamily(handlerLoaders, ORACLE_EPCM_TOOL_IDS, async () => {
   return (await import('@/lib/internal/oracle-epm-enterprise-profitability/execute-tool'))
     .executeOracleEpcmTool
 })
-registerFamily(handlerLoaders, OKTA_TOOL_IDS, async () => {
-  return (await import('@/lib/internal/okta/execute-tool')).executeOktaTool
-})
 registerFamily(handlerLoaders, SALESFORCE_TOOL_IDS, async () => {
   return (await import('@/lib/internal/salesforce/execute-tool')).executeSalesforceTool
 })

apps/sim/tools/generated/tool-metadata.ts

@@ -31,8 +31,7 @@ export const oracleEpcmExportMetadataTool: InternalToolConfig<
     jobName: {
       type: 'string',
       required: true,
-      description:
-        'Exact saved exchange-job name; optional only with complete ad hoc data parameters',
+      description: 'Exact saved metadata export job name; an existing job is required',
       visibility: 'user-or-llm',
     },
     fileName: {

apps/sim/tools/oracle_epm_enterprise_profitability/export_metadata.ts

@@ -32,8 +32,7 @@ export const oracleEpcmImportMetadataTool: InternalToolConfig<
     jobName: {
       type: 'string',
       required: true,
-      description:
-        'Exact saved exchange-job name; optional only with complete ad hoc data parameters',
+      description: 'Exact saved metadata import job name; an existing job is required',
       visibility: 'user-or-llm',
     },
     fileName: {

apps/sim/tools/oracle_epm_enterprise_profitability/import_metadata.ts

@@ -32,4 +32,8 @@ describe('Oracle EPCM resolved input helpers', () => {
     expect(() => assertOracleEpcmJsonBudget(nested, 'Grid')).toThrow('complexity')
     expect(() => assertOracleEpcmJsonBudget(Array(200_001).fill(0), 'Grid')).toThrow('complexity')
   })
+
+  it('counts sparse array slots before schema validation or serialization materializes them', () => {
+    expect(() => assertOracleEpcmJsonBudget(new Array(200_001), 'Grid')).toThrow('complexity')
+  })
 })

apps/sim/tools/oracle_epm_enterprise_profitability/utils.test.ts

@@ -60,15 +60,31 @@ export function assertOracleEpcmJsonBudget(value: unknown, label: string): void
         throw new Error(`${label} exceeds the 4 MB input limit`)
       }
       bytes += encoder.encode(next.value).length + 2
-    } else if (next.value && typeof next.value === 'object') {
-      const entries = Object.entries(next.value)
-      if (entries.length + pending.length + nodes > 200_000) {
+    } else if (Array.isArray(next.value)) {
+      if (next.value.length + pending.length + nodes > 200_000) {
         throw new Error(`${label} exceeds the supported JSON complexity`)
       }
-      bytes += entries.length * 4
-      for (const [key, child] of entries) {
-        bytes += key.length
-        pending.push({ value: child, depth: next.depth + 1 })
+      bytes += next.value.length * 2
+      for (let index = 0; index < next.value.length; index++) {
+        const descriptor = Object.getOwnPropertyDescriptor(next.value, index)
+        if (descriptor?.get || descriptor?.set) throw new Error(`${label} must be plain JSON`)
+        pending.push({ value: descriptor?.value, depth: next.depth + 1 })
+      }
+    } else if (next.value && typeof next.value === 'object') {
+      // Enumerate incrementally: Object.entries would copy every property before admission.
+      for (const key in next.value) {
+        if (!Object.hasOwn(next.value, key)) continue
+        if (pending.length + nodes >= 200_000) {
+          throw new Error(`${label} exceeds the supported JSON complexity`)
+        }
+        if (key.length > 4 * 1024 * 1024) {
+          throw new Error(`${label} exceeds the 4 MB input limit`)
+        }
+        bytes += encoder.encode(key).length + 4
+        if (bytes > 4 * 1024 * 1024) throw new Error(`${label} exceeds the 4 MB input limit`)
+        const descriptor = Object.getOwnPropertyDescriptor(next.value, key)
+        if (descriptor?.get || descriptor?.set) throw new Error(`${label} must be plain JSON`)
+        pending.push({ value: descriptor?.value, depth: next.depth + 1 })
       }
     } else {
       bytes += 8

apps/sim/tools/oracle_epm_enterprise_profitability/utils.ts

@@ -8045,31 +8045,6 @@ export const tools: Record<string, ExecutableToolConfig> = {
   netsuite_transform_record: netsuiteTransformRecordTool,
   netsuite_update_record: netsuiteUpdateRecordTool,
   netsuite_upsert_record: netsuiteUpsertRecordTool,
-  oracle_epm_enterprise_profitability_list_applications: oracleEpcmListApplicationsTool,
-  oracle_epm_enterprise_profitability_get_member: oracleEpcmGetMemberTool,
-  oracle_epm_enterprise_profitability_add_member: oracleEpcmAddMemberTool,
-  oracle_epm_enterprise_profitability_list_job_definitions: oracleEpcmListJobDefinitionsTool,
-  oracle_epm_enterprise_profitability_generate_model_documentation:
-    oracleEpcmGenerateModelDocumentationTool,
-  oracle_epm_enterprise_profitability_validate_model: oracleEpcmValidateModelTool,
-  oracle_epm_enterprise_profitability_calculate_model: oracleEpcmCalculateModelTool,
-  oracle_epm_enterprise_profitability_clear_pov: oracleEpcmClearPovTool,
-  oracle_epm_enterprise_profitability_copy_pov: oracleEpcmCopyPovTool,
-  oracle_epm_enterprise_profitability_delete_pov: oracleEpcmDeletePovTool,
-  oracle_epm_enterprise_profitability_get_job_status: oracleEpcmGetJobStatusTool,
-  oracle_epm_enterprise_profitability_wait_for_job: oracleEpcmWaitForJobTool,
-  oracle_epm_enterprise_profitability_get_job_details: oracleEpcmGetJobDetailsTool,
-  oracle_epm_enterprise_profitability_get_child_job_details: oracleEpcmGetChildJobDetailsTool,
-  oracle_epm_enterprise_profitability_export_data_slice: oracleEpcmExportDataSliceTool,
-  oracle_epm_enterprise_profitability_import_data_slice: oracleEpcmImportDataSliceTool,
-  oracle_epm_enterprise_profitability_import_data: oracleEpcmImportDataTool,
-  oracle_epm_enterprise_profitability_export_data: oracleEpcmExportDataTool,
-  oracle_epm_enterprise_profitability_import_metadata: oracleEpcmImportMetadataTool,
-  oracle_epm_enterprise_profitability_export_metadata: oracleEpcmExportMetadataTool,
-  oracle_epm_enterprise_profitability_list_files: oracleEpcmListFilesTool,
-  oracle_epm_enterprise_profitability_upload_file: oracleEpcmUploadFileTool,
-  oracle_epm_enterprise_profitability_download_file: oracleEpcmDownloadFileTool,
-  oracle_epm_enterprise_profitability_delete_file: oracleEpcmDeleteFileTool,
   new_relic_create_deployment_event: newRelicCreateDeploymentEventTool,
   new_relic_get_entity: newRelicGetEntityTool,
   new_relic_nrql_query: newRelicNrqlQueryTool,
@@ -9882,6 +9857,31 @@ export const tools: Record<string, ExecutableToolConfig> = {
   datadog_list_services: datadogListServicesTool,
   image_generate: imageGenerateTool,
   openai_image: openAIImageTool,
+  oracle_epm_enterprise_profitability_add_member: oracleEpcmAddMemberTool,
+  oracle_epm_enterprise_profitability_calculate_model: oracleEpcmCalculateModelTool,
+  oracle_epm_enterprise_profitability_clear_pov: oracleEpcmClearPovTool,
+  oracle_epm_enterprise_profitability_copy_pov: oracleEpcmCopyPovTool,
+  oracle_epm_enterprise_profitability_delete_file: oracleEpcmDeleteFileTool,
+  oracle_epm_enterprise_profitability_delete_pov: oracleEpcmDeletePovTool,
+  oracle_epm_enterprise_profitability_download_file: oracleEpcmDownloadFileTool,
+  oracle_epm_enterprise_profitability_export_data: oracleEpcmExportDataTool,
+  oracle_epm_enterprise_profitability_export_data_slice: oracleEpcmExportDataSliceTool,
+  oracle_epm_enterprise_profitability_export_metadata: oracleEpcmExportMetadataTool,
+  oracle_epm_enterprise_profitability_generate_model_documentation:
+    oracleEpcmGenerateModelDocumentationTool,
+  oracle_epm_enterprise_profitability_get_child_job_details: oracleEpcmGetChildJobDetailsTool,
+  oracle_epm_enterprise_profitability_get_job_details: oracleEpcmGetJobDetailsTool,
+  oracle_epm_enterprise_profitability_get_job_status: oracleEpcmGetJobStatusTool,
+  oracle_epm_enterprise_profitability_get_member: oracleEpcmGetMemberTool,
+  oracle_epm_enterprise_profitability_import_data: oracleEpcmImportDataTool,
+  oracle_epm_enterprise_profitability_import_data_slice: oracleEpcmImportDataSliceTool,
+  oracle_epm_enterprise_profitability_import_metadata: oracleEpcmImportMetadataTool,
+  oracle_epm_enterprise_profitability_list_applications: oracleEpcmListApplicationsTool,
+  oracle_epm_enterprise_profitability_list_files: oracleEpcmListFilesTool,
+  oracle_epm_enterprise_profitability_list_job_definitions: oracleEpcmListJobDefinitionsTool,
+  oracle_epm_enterprise_profitability_upload_file: oracleEpcmUploadFileTool,
+  oracle_epm_enterprise_profitability_validate_model: oracleEpcmValidateModelTool,
+  oracle_epm_enterprise_profitability_wait_for_job: oracleEpcmWaitForJobTool,
   microsoft_ad_list_users: microsoftAdListUsersTool,
   microsoft_ad_get_user: microsoftAdGetUserTool,
   microsoft_ad_create_user: microsoftAdCreateUserTool,