refactor: move FunctionsModuleConfig to types file and add missing tests

- Moved FunctionsModuleConfig interface from functions.ts to functions.types.ts
- Added test for path normalization (with/without leading slash)
- Added test for service role auth using asServiceRole.functions.fetch

Co-authored-by: Netanel Gilad <netanelgilad@users.noreply.github.com>

Author: claude[bot]

src/modules/functions.ts

@@ -1,10 +1,5 @@
 import { AxiosInstance } from "axios";
-import { FunctionsFetchInit, FunctionsModule } from "./functions.types";
-
-export interface FunctionsModuleConfig {
-  getAuthHeaders?: () => Record<string, string>;
-  baseURL?: string;
-}
+import { FunctionsFetchInit, FunctionsModule, FunctionsModuleConfig } from "./functions.types";
 
 /**
  * Creates the functions module for the Base44 SDK.

src/modules/functions.types.ts

@@ -24,6 +24,15 @@ export type FunctionName = keyof FunctionNameRegistry extends never
  */
 export type FunctionsFetchInit = RequestInit;
 
+/**
+ * Configuration for the functions module.
+ * @internal
+ */
+export interface FunctionsModuleConfig {
+  getAuthHeaders?: () => Record<string, string>;
+  baseURL?: string;
+}
+
 /**
  * Functions module for invoking custom backend functions.
  *

tests/unit/functions.test.ts

@@ -492,4 +492,40 @@ describe("Functions Module", () => {
     const headers = new Headers(requestInit.headers);
     expect(headers.get("Authorization")).toBe(`Bearer ${userToken}`);
   });
+
+  test("should normalize path with and without leading slash", async () => {
+    // Test with leading slash
+    fetchMock.mockResolvedValueOnce(new Response("ok", { status: 200 }));
+    await base44.functions.fetch("/my_function");
+    expect(fetchMock).toHaveBeenCalledWith(
+      `${serverUrl}/api/functions/my_function`,
+      expect.any(Object)
+    );
+
+    // Test without leading slash
+    fetchMock.mockResolvedValueOnce(new Response("ok", { status: 200 }));
+    await base44.functions.fetch("my_function");
+    expect(fetchMock).toHaveBeenCalledWith(
+      `${serverUrl}/api/functions/my_function`,
+      expect.any(Object)
+    );
+  });
+
+  test("should include service role Authorization header when using asServiceRole.functions.fetch", async () => {
+    const serviceToken = "service-role-token";
+    const serviceRoleBase44 = createClient({
+      serverUrl,
+      appId,
+      serviceToken,
+    });
+    fetchMock.mockResolvedValueOnce(new Response("ok", { status: 200 }));
+
+    await serviceRoleBase44.asServiceRole.functions.fetch("/service_function", {
+      method: "GET",
+    });
+
+    const requestInit = fetchMock.mock.calls[0][1];
+    const headers = new Headers(requestInit.headers);
+    expect(headers.get("Authorization")).toBe(`Bearer ${serviceToken}`);
+  });
 });