fix(sdk): apply PR review feedback for v0.2.0

- Add headers to RequestOptions for type-safe header passing
- Make accessToken/token optional in AuthResponse
- Remove (options as any).headers cast in client.ts
- Add public getBaseUrl()/getApiKey() getters to UrBackendClient
- Fix refreshToken() to use RequestOptions instead of any
- Fix socialStart() to use public getters instead of bracket access
- Fix socialExchange() return type to SocialExchangeResponse
- Fix getSchema() to extract response.collection
- Fix delete() to validate response keys properly
- Update schema/database test mocks to match real API shapes
- Make beforeEach in auth test async and await logout()

Agent-Logs-Url: https://github.com/geturbackend/urBackend/sessions/2c6f46de-4645-4d3e-8ae1-10e938a9e258

Co-authored-by: yash-pouranik <172860064+yash-pouranik@users.noreply.github.com>

Author: Copilot

sdks/urbackend-sdk/src/client.ts

@@ -63,6 +63,14 @@ export class UrBackendClient {
     return this._mail;
   }
 
+  public getBaseUrl(): string {
+    return this.baseUrl;
+  }
+
+  public getApiKey(): string {
+    return this.apiKey;
+  }
+
   /**
    * Internal request handler
    */
@@ -83,8 +91,8 @@ export class UrBackendClient {
     }
 
     // Merge custom headers from options if provided
-    if ((options as any).headers) {
-      Object.assign(headers, (options as any).headers);
+    if (options.headers) {
+      Object.assign(headers, options.headers);
     }
 
     let requestBody: BodyInit | undefined;

sdks/urbackend-sdk/src/modules/auth.ts

@@ -12,7 +12,7 @@ import {
   ResetPasswordPayload,
   SocialExchangePayload,
   SocialExchangeResponse,
-  ApiResponse,
+  RequestOptions,
 } from '../types';
 import { AuthError } from '../errors';
 
@@ -140,7 +140,7 @@ export class AuthModule {
    * @param refreshToken Optional refresh token for header mode. If omitted, uses cookie mode.
    */
   public async refreshToken(refreshToken?: string): Promise<AuthResponse> {
-    const options: any = {};
+    const options: RequestOptions = {};
     if (refreshToken) {
       options.headers = { 'x-refresh-token': refreshToken, 'x-refresh-token-mode': 'header' };
     } else {
@@ -157,17 +157,16 @@ export class AuthModule {
    * Redirect the user's browser to this URL to begin the flow.
    */
   public socialStart(provider: 'github' | 'google'): string {
-    return `${this.client['baseUrl']}/api/userAuth/social/${provider}/start?key=${this.client['apiKey']}`;
+    return `${this.client.getBaseUrl()}/api/userAuth/social/${provider}/start?key=${this.client.getApiKey()}`;
   }
 
   /**
    * Exchange social auth rtCode for a refresh token
    */
-  public async socialExchange(payload: SocialExchangePayload): Promise<ApiResponse<SocialExchangeResponse>> {
-    const response = await this.client.request<ApiResponse<SocialExchangeResponse>>('POST', '/api/userAuth/social/exchange', {
+  public async socialExchange(payload: SocialExchangePayload): Promise<SocialExchangeResponse> {
+    return this.client.request<SocialExchangeResponse>('POST', '/api/userAuth/social/exchange', {
       body: payload,
     });
-    return response;
   }
 
   /**

sdks/urbackend-sdk/src/modules/database.ts

@@ -82,11 +82,18 @@ export class DatabaseModule {
    * Delete a document by its ID
    */
   public async delete(collection: string, id: string, token?: string): Promise<{ deleted: boolean }> {
-    const result = await this.client.request<any>('DELETE', `/api/data/${collection}/${id}`, {
-      token
-    });
-    // The API might return { message: "Document deleted", id: "..." }
-    return { deleted: !!result };
+    const result = await this.client.request<{ message?: string; id?: string } | null>(
+      'DELETE',
+      `/api/data/${collection}/${id}`,
+      { token },
+    );
+
+    const deleted =
+      typeof result === 'object' &&
+      result !== null &&
+      (result.id === id || result.message === 'Document deleted');
+
+    return { deleted };
   }
 
   /**

sdks/urbackend-sdk/src/modules/schema.ts

@@ -8,6 +8,11 @@ export class SchemaModule {
    * Fetch the schema definition for a collection
    */
   public async getSchema(collection: string): Promise<CollectionSchema> {
-    return this.client.request<CollectionSchema>('GET', `/api/schemas/${collection}`);
+    const response = await this.client.request<{
+      message: string;
+      collection: CollectionSchema;
+    }>('GET', `/api/schemas/${collection}`);
+
+    return response.collection;
   }
 }

sdks/urbackend-sdk/src/types/index.ts

@@ -9,6 +9,7 @@ export interface RequestOptions {
   token?: string;
   isMultipart?: boolean;
   credentials?: RequestCredentials;
+  headers?: Record<string, string>;
 }
 
 export interface QueryParams {
@@ -82,9 +83,9 @@ export interface AuthUser {
 }
 
 export interface AuthResponse {
-  accessToken: string;
+  accessToken?: string;
   /** @deprecated use accessToken instead */
-  token: string;
+  token?: string;
   expiresIn?: string;
   userId?: string;
   user?: AuthUser;

sdks/urbackend-sdk/tests/auth.test.ts

@@ -4,9 +4,9 @@ import urBackend from '../src/index';
 const mockApiKey = 'pk_live_test';
 const client = urBackend({ apiKey: mockApiKey });
 
-beforeEach(() => {
+beforeEach(async () => {
   vi.resetAllMocks();
-  client.auth.logout();
+  await client.auth.logout();
 });
 
 test('signUp returns user object on success', async () => {

sdks/urbackend-sdk/tests/database.test.ts

@@ -115,7 +115,7 @@ test('delete returns { deleted: true } and handles token', async () => {
   const fetchMock = vi.fn().mockResolvedValue({
     ok: true,
     headers: new Headers({ 'content-type': 'application/json' }),
-    json: () => Promise.resolve({ success: true, data: { message: 'Deleted' } }),
+    json: () => Promise.resolve({ success: true, data: { message: 'Document deleted', id: 'id-1' } }),
   });
   vi.stubGlobal('fetch', fetchMock);
 

sdks/urbackend-sdk/tests/schema.test.ts

@@ -22,7 +22,7 @@ test('getSchema returns collection schema', async () => {
     vi.fn().mockResolvedValue({
       ok: true,
       headers: new Headers({ 'content-type': 'application/json' }),
-      json: () => Promise.resolve({ success: true, data: mockSchema }),
+      json: () => Promise.resolve({ message: 'Schema exists', collection: mockSchema }),
     }),
   );