fix: delete requests drop query params passed via {query: ...} (#1668)

Co-authored-by: Claude <noreply@anthropic.com>

Author: claude[bot]

src/pipes/pipes.spec.ts

@@ -308,6 +308,10 @@ describe('Pipes', () => {
       expect(new URL(String(fetchURL())).pathname).toBe(
         '/user_management/users/test_userId/connected_accounts/test_slug',
       );
+      expect(fetchSearchParams()).toEqual({
+        organization_id: 'org_01EHZNVPK3SFK441A1RGBFSHRT',
+      });
+      expect(fetchSearchParams()).not.toHaveProperty('query');
     });
   });
 

src/pipes/pipes.ts

@@ -477,11 +477,9 @@ export class Pipes {
     await this.workos.delete(
       `/user_management/users/${encodeURIComponent(userId)}/connected_accounts/${encodeURIComponent(slug)}`,
       {
-        query: {
-          ...(options.organizationId !== undefined && {
-            organization_id: options.organizationId,
-          }),
-        },
+        ...(options.organizationId !== undefined && {
+          organization_id: options.organizationId,
+        }),
       },
     );
   }

src/vault/vault.spec.ts

@@ -201,6 +201,10 @@ describe('Vault', () => {
 
       expect(fetchMethod()).toBe('DELETE');
       expect(new URL(String(fetchURL())).pathname).toBe('/vault/v1/kv/test_id');
+      expect(fetchSearchParams()).toEqual({
+        version_check: 'test_version_check',
+      });
+      expect(fetchSearchParams()).not.toHaveProperty('query');
     });
   });
 

src/vault/vault.ts

@@ -305,11 +305,9 @@ export class Vault {
   async deleteObject(options: DeleteVaultObjectOptions): Promise<void> {
     const { id } = options;
     await this.workos.delete(`/vault/v1/kv/${encodeURIComponent(id)}`, {
-      query: {
-        ...(options.versionCheck !== undefined && {
-          version_check: options.versionCheck,
-        }),
-      },
+      ...(options.versionCheck !== undefined && {
+        version_check: options.versionCheck,
+      }),
     });
   }
 

src/workos.ts

@@ -358,7 +358,10 @@ export class WorkOS {
     return { data: await res.toJSON() };
   }
 
-  async delete(path: string, query?: any): Promise<void> {
+  async delete(
+    path: string,
+    query?: Record<string, string | number | boolean | undefined>,
+  ): Promise<void> {
     this.requireApiKey(path);
 
     try {