remove camelCase param normalization from custom integrations, use snake_case only

Author: natansil

src/modules/custom-integrations.ts

@@ -5,44 +5,6 @@ import {
   CustomIntegrationCallResponse,
 } from "./custom-integrations.types.js";
 
-/**
- * Normalizes parameters to snake_case for the API.
- *
- * Supports both camelCase (pathParams) and snake_case (path_params) input,
- * always outputting snake_case for the backend.
- */
-function normalizeParams(params?: CustomIntegrationCallParams): Record<string, any> {
-  if (!params) {
-    return {};
-  }
-
-  const normalized: Record<string, any> = {};
-
-  // Handle payload
-  if (params.payload !== undefined) {
-    normalized.payload = params.payload;
-  }
-
-  // Handle path_params (support both camelCase and snake_case)
-  const pathParams = params.pathParams ?? params.path_params;
-  if (pathParams !== undefined) {
-    normalized.path_params = pathParams;
-  }
-
-  // Handle query_params (support both camelCase and snake_case)
-  const queryParams = params.queryParams ?? params.query_params;
-  if (queryParams !== undefined) {
-    normalized.query_params = queryParams;
-  }
-
-  // Handle headers
-  if (params.headers !== undefined) {
-    normalized.headers = params.headers;
-  }
-
-  return normalized;
-}
-
 /**
  * Creates the custom integrations module for the Base44 SDK.
  *
@@ -69,13 +31,10 @@ export function createCustomIntegrationsModule(
         throw new Error("Operation ID is required");
       }
 
-      // Normalize parameters to snake_case
-      const normalizedParams = normalizeParams(params);
-
       // Make the API call
       const response = await axios.post<CustomIntegrationCallResponse>(
         `/apps/${appId}/integrations/custom/${slug}/${operationId}`,
-        normalizedParams
+        params ?? {}
       );
 
       // Return the response data
@@ -84,4 +43,3 @@ export function createCustomIntegrationsModule(
     },
   };
 }
-

src/modules/custom-integrations.types.ts

@@ -1,8 +1,5 @@
 /**
  * Parameters for calling a custom integration endpoint.
- *
- * Supports both camelCase and snake_case parameter names for developer convenience.
- * The SDK will normalize to snake_case before sending to the API.
  */
 export interface CustomIntegrationCallParams {
   /**
@@ -12,25 +9,11 @@ export interface CustomIntegrationCallParams {
 
   /**
    * Path parameters to substitute in the URL (e.g., `{ owner: "user", repo: "repo" }`).
-   * Can use either `pathParams` (camelCase) or `path_params` (snake_case).
-   */
-  pathParams?: Record<string, string>;
-
-  /**
-   * Path parameters to substitute in the URL (snake_case variant).
-   * @see {@link pathParams}
    */
   path_params?: Record<string, string>;
 
   /**
    * Query string parameters to append to the URL.
-   * Can use either `queryParams` (camelCase) or `query_params` (snake_case).
-   */
-  queryParams?: Record<string, any>;
-
-  /**
-   * Query string parameters (snake_case variant).
-   * @see {@link queryParams}
    */
   query_params?: Record<string, any>;
 
@@ -81,8 +64,8 @@ export interface CustomIntegrationCallResponse {
  *   "github",        // integration slug (defined by workspace admin)
  *   "listIssues",    // operation ID from the OpenAPI spec
  *   {
- *     pathParams: { owner: "myorg", repo: "myrepo" },
- *     queryParams: { state: "open", per_page: 100 }
+ *     path_params: { owner: "myorg", repo: "myrepo" },
+ *     query_params: { state: "open", per_page: 100 }
  *   }
  * );
  *
@@ -100,7 +83,7 @@ export interface CustomIntegrationCallResponse {
  *   "github",
  *   "createIssue",
  *   {
- *     pathParams: { owner: "myorg", repo: "myrepo" },
+ *     path_params: { owner: "myorg", repo: "myrepo" },
  *     payload: {
  *       title: "Bug report",
  *       body: "Something is broken",
@@ -131,4 +114,3 @@ export interface CustomIntegrationsModule {
     params?: CustomIntegrationCallParams
   ): Promise<CustomIntegrationCallResponse>;
 }
-

tests/unit/custom-integrations.test.ts

@@ -56,42 +56,6 @@ describe('Custom Integrations Module', () => {
     expect(scope.isDone()).toBe(true);
   });
 
-  test('custom.call() should support camelCase parameter names', async () => {
-    const slug = 'github';
-    const operationId = 'getRepo';
-    const params = {
-      pathParams: { owner: 'testuser', repo: 'testrepo' },
-      queryParams: { include: 'stats' },
-    };
-
-    // Expected body should have snake_case keys
-    const expectedBody = {
-      path_params: { owner: 'testuser', repo: 'testrepo' },
-      query_params: { include: 'stats' },
-    };
-
-    const mockResponse = {
-      success: true,
-      status_code: 200,
-      data: { name: 'testrepo', stars: 100 },
-    };
-
-    // Mock the API response - expecting snake_case in the body
-    scope
-      .post(`/api/apps/${appId}/integrations/custom/${slug}/${operationId}`, expectedBody)
-      .reply(200, mockResponse);
-
-    // Call with camelCase params
-    const result = await base44.integrations.custom.call(slug, operationId, params);
-
-    // Verify the response
-    expect(result.success).toBe(true);
-    expect(result.data.name).toBe('testrepo');
-
-    // Verify all mocks were called
-    expect(scope.isDone()).toBe(true);
-  });
-
   test('custom.call() should work with empty params', async () => {
     const slug = 'github';
     const operationId = 'getAuthenticatedUser';
@@ -255,4 +219,3 @@ describe('Custom Integrations Module', () => {
     expect(scope.isDone()).toBe(true);
   });
 });
-