Merge pull request #3 from Tam2/fix/patch-put-bare-body-fallback

fix(runtime): forward a bare body on PATCH/PUT like POST

Author: Tam2

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sveltekit-openapi-remote",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Generate SvelteKit remote functions directly from OpenAPI/Swagger specs.",
   "type": "module",
   "license": "MIT",

src/runtime/index.ts

@@ -43,19 +43,23 @@ export function createRemoteHandlers(client: OpenapiClient) {
   async function handlePatchCommand(path: string, input: any) {
     const hasPath = input && typeof input === 'object' && 'path' in input;
     const hasBody = input && typeof input === 'object' && 'body' in input;
+    // For a no-path endpoint the generated wrapper passes the body directly (no `body` key), so fall
+    // back to the whole input as the body — matching handlePostCommand. Without this, a bare body was
+    // silently dropped, sending an empty PATCH.
     const { data, error, response } = await client.PATCH(path, {
       ...(hasPath && { params: { path: input.path } }),
-      ...(hasBody && { body: input.body }),
+      body: hasBody ? input.body : input,
     });
     return handleResponse(response, error, data);
   }
 
   async function handlePutCommand(path: string, input: any) {
     const hasPath = input && typeof input === 'object' && 'path' in input;
     const hasBody = input && typeof input === 'object' && 'body' in input;
+    // Same bare-body fallback as handlePostCommand/handlePatchCommand.
     const { data, error, response } = await client.PUT(path, {
       ...(hasPath && { params: { path: input.path } }),
-      ...(hasBody && { body: input.body }),
+      body: hasBody ? input.body : input,
     });
     return handleResponse(response, error, data);
   }

test/runtime.test.ts

@@ -117,6 +117,23 @@ describe('createRemoteHandlers', () => {
       });
       expect(result).toEqual({ updated: true });
     });
+
+    it('treats a bare body (no path/body key) as the request body, like POST', async () => {
+      // A no-path endpoint's generated wrapper passes the body directly (e.g. patchUsersCommand({...})).
+      // Regression: handlePatchCommand used to only forward input.body, silently dropping a bare body.
+      const client = createMockClient();
+      client.PATCH.mockResolvedValue({
+        data: { updated: true },
+        error: undefined,
+        response: { ok: true, status: 200 },
+      });
+      const { handlePatchCommand } = createRemoteHandlers(client as any);
+      const result = await handlePatchCommand('/users', { firstName: 'Tam', lastName: 'M' });
+      expect(client.PATCH).toHaveBeenCalledWith('/users', {
+        body: { firstName: 'Tam', lastName: 'M' },
+      });
+      expect(result).toEqual({ updated: true });
+    });
   });
 
   describe('handlePutCommand', () => {
@@ -138,6 +155,21 @@ describe('createRemoteHandlers', () => {
       });
       expect(result).toEqual({ id: 1, name: 'Replaced' });
     });
+
+    it('treats a bare body (no path/body key) as the request body, like POST', async () => {
+      const client = createMockClient();
+      client.PUT.mockResolvedValue({
+        data: { replaced: true },
+        error: undefined,
+        response: { ok: true, status: 200 },
+      });
+      const { handlePutCommand } = createRemoteHandlers(client as any);
+      const result = await handlePutCommand('/settings', { theme: 'dark' });
+      expect(client.PUT).toHaveBeenCalledWith('/settings', {
+        body: { theme: 'dark' },
+      });
+      expect(result).toEqual({ replaced: true });
+    });
   });
 
   describe('handleDeleteCommand', () => {