refactor(functions): simplify fetch API per review comments

- Remove fallback path logic - only use /functions endpoint
- Remove X-App-Id header - not needed
- Remove automatic content-type handling - let fetch handle it natively
- Update tests to reflect the simplified behavior

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

Author: claude[bot]

src/modules/functions.ts

@@ -96,13 +96,9 @@ export function createFunctionsModule(
     async fetch(path: string, init: FunctionsFetchInit = {}) {
       const normalizedPath = path.startsWith("/") ? path : `/${path}`;
       const primaryPath = `/functions${normalizedPath}`;
-      const fallbackPath = `/apps/${appId}/functions${normalizedPath}`;
       const { data, ...fetchInit } = init;
 
       const headers = toHeaders(fetchInit.headers);
-      if (!headers.has("X-App-Id")) {
-        headers.set("X-App-Id", appId);
-      }
       let body: BodyInit | null | undefined = fetchInit.body;
 
       if (body === undefined && data !== undefined) {
@@ -115,9 +111,6 @@ export function createFunctionsModule(
           body = data as BodyInit;
         } else {
           body = JSON.stringify(data);
-          if (!headers.has("Content-Type")) {
-            headers.set("Content-Type", "application/json");
-          }
         }
       }
 
@@ -127,18 +120,11 @@ export function createFunctionsModule(
         body,
       };
 
-      let response = await fetch(
+      const response = await fetch(
         joinBaseUrl(axios.defaults.baseURL, primaryPath),
         requestInit
       );
 
-      if (response.status === 404) {
-        response = await fetch(
-          joinBaseUrl(axios.defaults.baseURL, fallbackPath),
-          requestInit
-        );
-      }
-
       return response;
     },
   };

tests/unit/functions.test.ts

@@ -474,29 +474,23 @@ describe("Functions Module", () => {
     );
 
     const requestInit = fetchMock.mock.calls[0][1];
-    const headers = new Headers(requestInit.headers);
-    expect(headers.get("Content-Type")).toBe("application/json");
-    expect(headers.get("X-App-Id")).toBe(appId);
     expect(requestInit.body).toBe(JSON.stringify({ mode: "sse" }));
   });
 
-  test("should fallback to app-scoped functions endpoint on 404", async () => {
+  test("should not fallback to app-scoped functions endpoint on 404", async () => {
     fetchMock
-      .mockResolvedValueOnce(new Response("Not found", { status: 404 }))
-      .mockResolvedValueOnce(new Response("ok", { status: 200 }));
+      .mockResolvedValueOnce(new Response("Not found", { status: 404 }));
 
-    await base44.functions.fetch("/streaming_demo/deep/path", {
+    const response = await base44.functions.fetch("/streaming_demo/deep/path", {
       method: "POST",
       data: { mode: "ndjson" },
     });
 
-    expect(fetchMock).toHaveBeenCalledTimes(2);
+    expect(fetchMock).toHaveBeenCalledTimes(1);
     expect(fetchMock.mock.calls[0][0]).toBe(
       `${serverUrl}/api/functions/streaming_demo/deep/path`
     );
-    expect(fetchMock.mock.calls[1][0]).toBe(
-      `${serverUrl}/api/apps/${appId}/functions/streaming_demo/deep/path`
-    );
+    expect(response.status).toBe(404);
   });
 
   test("should include Authorization header when using functions.fetch", async () => {