fix(supabase): use direct GoTrueClientOptions import

Author: mandarini

packages/core/auth-js/src/GoTrueClient.ts

@@ -381,6 +381,13 @@ export default class GoTrueClient {
     this.initialize()
   }
 
+  /**
+   * Returns whether error throwing mode is enabled for this client.
+   */
+  public isThrowOnErrorEnabled(): boolean {
+    return this.throwOnError
+  }
+
   /**
    * Centralizes return handling with optional error throwing. When `throwOnError` is enabled
    * and the provided result contains a non-nullish error, the error is thrown instead of

packages/core/supabase-js/src/SupabaseClient.ts

@@ -340,6 +340,7 @@ export default class SupabaseClient<
       flowType,
       lock,
       debug,
+      throwOnError,
     }: SupabaseAuthClientOptions,
     headers?: Record<string, string>,
     fetch?: Fetch
@@ -360,6 +361,7 @@ export default class SupabaseClient<
       flowType,
       lock,
       debug,
+      throwOnError,
       fetch,
       // auth checks if there is a custom authorizaiton header using this flag
       // so it knows whether to return an error when getUser is called with no session

packages/core/supabase-js/src/lib/types.ts

@@ -1,4 +1,4 @@
-import { AuthClient } from '@supabase/auth-js'
+import { GoTrueClientOptions } from '@supabase/auth-js'
 import { RealtimeClientOptions } from '@supabase/realtime-js'
 import { PostgrestError } from '@supabase/postgrest-js'
 import type { StorageClientOptions } from '@supabase/storage-js'
@@ -21,9 +21,7 @@ export type {
   GenericFunction,
 }
 
-type AuthClientOptions = ConstructorParameters<typeof AuthClient>[0]
-
-export interface SupabaseAuthClientOptions extends AuthClientOptions {}
+export interface SupabaseAuthClientOptions extends GoTrueClientOptions {}
 
 export type Fetch = typeof fetch
 
@@ -78,6 +76,11 @@ export type SupabaseClientOptions<SchemaName> = {
      * @experimental
      */
     lock?: SupabaseAuthClientOptions['lock']
+    /**
+     * If there is an error with the query, throwOnError will reject the promise by
+     * throwing the error instead of returning it as part of a successful response.
+     */
+    throwOnError?: SupabaseAuthClientOptions['throwOnError']
   }
   /**
    * Options passed to the realtime-js instance

packages/core/supabase-js/test/unit/SupabaseAuthClient.test.ts

@@ -33,3 +33,21 @@ test('_initSupabaseAuthClient should overwrite authHeaders if headers are provid
   expect(authClient['headers']['Authorization']).toBe('Bearer custom-auth-header')
   expect(authClient['headers']['apikey']).toBe('supabaseKey')
 })
+
+test('_initSupabaseAuthClient should pass through throwOnError option', () => {
+  const client = new SupabaseClient('https://example.supabase.com', 'supabaseKey')
+  const authClient = client['_initSupabaseAuthClient'](
+    { ...authSettings, throwOnError: true },
+    undefined,
+    undefined
+  )
+
+  expect((authClient as any).isThrowOnErrorEnabled()).toBe(true)
+})
+
+test('createClient should accept auth.throwOnError and wire it to auth client', () => {
+  const supa = new SupabaseClient('https://example.supabase.com', 'supabaseKey', {
+    auth: { throwOnError: true },
+  })
+  expect((supa.auth as any).isThrowOnErrorEnabled()).toBe(true)
+})