Skip to content

Commit f081a5d

Browse files
author
Luca Forstner
committed
Remove unnecessary option
1 parent 5776eb7 commit f081a5d

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

packages/core/src/integrations/supabase.ts

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -67,10 +67,6 @@ export const FILTER_MAPPINGS = {
6767

6868
export const DB_OPERATIONS_TO_INSTRUMENT = ['select', 'insert', 'upsert', 'update', 'delete'];
6969

70-
// We may need options in the future, for now it's empty
71-
// eslint-disable-next-line @typescript-eslint/no-empty-interface
72-
interface SupabaseInstrumentationOptions {}
73-
7470
type AuthOperationFn = (...args: unknown[]) => Promise<unknown>;
7571
type AuthOperationName = (typeof AUTH_OPERATIONS_TO_INSTRUMENT)[number];
7672
type AuthAdminOperationName = (typeof AUTH_ADMIN_OPERATIONS_TO_INSTRUMENT)[number];
@@ -492,12 +488,7 @@ function instrumentPostgRESTQueryBuilder(PostgRESTQueryBuilder: new () => PostgR
492488
}
493489
}
494490

495-
export const instrumentSupabaseClient = (
496-
supabaseClient: unknown,
497-
// In future, we may need options. For now it's unused.
498-
// eslint-disable-next-line @typescript-eslint/no-unused-vars
499-
options: SupabaseInstrumentationOptions = {},
500-
): void => {
491+
export const instrumentSupabaseClient = (supabaseClient: unknown): void => {
501492
if (!supabaseClient) {
502493
DEBUG_BUILD && logger.warn('Supabase integration was not installed because no Supabase client was provided.');
503494
return;
@@ -511,17 +502,15 @@ export const instrumentSupabaseClient = (
511502

512503
const INTEGRATION_NAME = 'Supabase';
513504

514-
const _supabaseIntegration = ((supabaseClient: unknown, options: SupabaseInstrumentationOptions) => {
505+
const _supabaseIntegration = ((supabaseClient: unknown) => {
515506
return {
516507
setupOnce() {
517-
instrumentSupabaseClient(supabaseClient, options);
508+
instrumentSupabaseClient(supabaseClient);
518509
},
519510
name: INTEGRATION_NAME,
520511
};
521512
}) satisfies IntegrationFn;
522513

523-
export const supabaseIntegration = defineIntegration(
524-
(supabaseClient: unknown, options: SupabaseInstrumentationOptions) => {
525-
return _supabaseIntegration(supabaseClient, options);
526-
},
527-
) satisfies IntegrationFn;
514+
export const supabaseIntegration = defineIntegration((options: { supabaseClient: any }) => {
515+
return _supabaseIntegration(options.supabaseClient);
516+
}) satisfies IntegrationFn;

0 commit comments

Comments
 (0)