wip: exclude __InternalSupabase

Author: avallete

src/SupabaseClient.ts

@@ -34,10 +34,10 @@ export default class SupabaseClient<
   // support both cases.
   // TODO: Allow setting db_schema from ClientOptions.
   SchemaNameOrClientOptions extends
-    | (string & keyof Database)
-    | { PostgrestVersion: string } = 'public' extends keyof Database
+    | (string & keyof Omit<Database, '__InternalSupabase'>)
+    | { PostgrestVersion: string } = 'public' extends keyof Omit<Database, '__InternalSupabase'>
     ? 'public'
-    : string & keyof Database,
+    : string & keyof Omit<Database, '__InternalSupabase'>,
   SchemaName extends string &
     keyof Omit<Database, '__InternalSupabase'> = SchemaNameOrClientOptions extends string &
     keyof Omit<Database, '__InternalSupabase'>

src/index.ts

@@ -27,10 +27,10 @@ export type { SupabaseClientOptions, QueryResult, QueryData, QueryError } from '
 export const createClient = <
   Database = any,
   SchemaNameOrClientOptions extends
-    | (string & keyof Database)
-    | { PostgrestVersion: string } = 'public' extends keyof Database
+    | (string & keyof Omit<Database, '__InternalSupabase'>)
+    | { PostgrestVersion: string } = 'public' extends keyof Omit<Database, '__InternalSupabase'>
     ? 'public'
-    : string & keyof Database,
+    : string & keyof Omit<Database, '__InternalSupabase'>,
   SchemaName extends string &
     keyof Omit<Database, '__InternalSupabase'> = SchemaNameOrClientOptions extends string &
     keyof Omit<Database, '__InternalSupabase'>

test/types/index.test-d.ts

@@ -188,6 +188,10 @@ const supabase = createClient<Database>(URL, KEY)
   // Note: The template argument properties (PostgrestVersion) will not be autocompleted
   // due to a Typescript bug tracked here: https://github.com/microsoft/TypeScript/issues/56299
   const pg13Client = createClient<DatabaseWithInternals>('HTTP://localhost:3000', KEY)
+  // @ts-expect-error should raise error if providing __InternalSupabase as schema name
+  createClient<DatabaseWithInternals, '__InternalSupabase'>('HTTP://localhost:3000', KEY)
+  // @ts-expect-error should raise error if providing __InternalSupabase as schema name
+  new SupabaseClient<DatabaseWithInternals, '__InternalSupabase'>('HTTP://localhost:3000', KEY)
   const pg12Client = createClient<Database>('HTTP://localhost:3000', KEY)
   const res13 = await pg13Client.from('shops').update({ id: 21 }).maxAffected(1)
   const res12 = await pg12Client.from('shops').update({ id: 21 }).maxAffected(1)