fix: Use optional chaining and remove unused imports

Address lint failures: use `res?.error` instead of `res && res.error`,
and remove unused imports in the test file.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: antonis

packages/core/src/integrations/supabase.ts

@@ -403,7 +403,7 @@ function instrumentPostgRESTFilterBuilder(PostgRESTFilterBuilder: PostgRESTFilte
                     span.end();
                   }
 
-                  if (res && res.error) {
+                  if (res?.error) {
                     const err = new Error(res.error.message) as SupabaseError;
                     if (res.error.code) {
                       err.code = res.error.code;

packages/core/test/lib/integrations/supabase.test.ts

@@ -2,13 +2,11 @@ import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import * as breadcrumbModule from '../../../src/breadcrumbs';
 import * as exportsModule from '../../../src/exports';
 import {
-  DB_OPERATIONS_TO_INSTRUMENT,
   extractOperation,
   instrumentSupabaseClient,
   translateFiltersIntoMethods,
 } from '../../../src/integrations/supabase';
 import type {
-  PostgRESTFilterBuilder,
   PostgRESTQueryBuilder,
   SupabaseClientInstance,
 } from '../../../src/integrations/supabase';