fix(repo): nx dependency management and deno bundling

Author: mandarini

package-lock.json

@@ -84,10 +84,10 @@
     "@supabase/functions-js": "*",
     "@supabase/postgrest-js": "*",
     "@supabase/realtime-js": "*",
-    "@supabase/storage-js": "*",
-    "@supabase/tracing": "*"
+    "@supabase/storage-js": "*"
   },
   "devDependencies": {
+    "@supabase/tracing": "*",
     "jsr": "^0.13.5",
     "puppeteer": "^24.9.0"
   },

packages/core/supabase-js/package.json

@@ -44,7 +44,7 @@ export const fetchWithAuth = (
     }
 
     // Inject trace headers
-    const traceHeaders = getTraceHeaders(input, supabaseUrl, tracePropagationOptions)
+    const traceHeaders = await getTraceHeaders(input, supabaseUrl, tracePropagationOptions)
 
     if (traceHeaders) {
       if (traceHeaders.traceparent && !headers.has('traceparent')) {
@@ -70,11 +70,11 @@ export const fetchWithAuth = (
  * @param options - Trace propagation options
  * @returns Trace context headers, or null if trace propagation is disabled or unavailable
  */
-function getTraceHeaders(
+async function getTraceHeaders(
   input: RequestInfo | URL,
   supabaseUrl: string,
   options?: TracePropagationOptions
-): TraceContext | null {
+): Promise<TraceContext | null> {
   // Check if trace propagation is enabled
   if (options?.enabled === false) {
     return null
@@ -93,7 +93,7 @@ function getTraceHeaders(
   }
 
   // Extract trace context
-  const traceContext = extractTraceContext()
+  const traceContext = await extractTraceContext()
 
   if (!traceContext || !traceContext.traceparent) {
     return null

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

@@ -1,5 +1,15 @@
 import { resolveFetch, resolveHeadersConstructor, fetchWithAuth } from '../../src/lib/fetch'
 
+jest.mock('@supabase/tracing', () => {
+  const actual = jest.requireActual('@supabase/tracing')
+  return {
+    ...actual,
+    extractTraceContext: jest.fn().mockResolvedValue({
+      traceparent: '00-0af7651916cd43dd8448eb211c80319c-b7ad6b7169203331-00',
+    }),
+  }
+})
+
 // Mock fetch for testing
 const mockFetch = jest.fn()
 const mockHeaders = jest.fn()

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

@@ -14,13 +14,15 @@ export default defineConfig([
     clean: true,
     outDir: 'dist',
     // CRITICAL: Keep @supabase/* packages external - not bundled
+    // NOTE: @supabase/tracing intentionally omitted — it gets bundled via noExternal
     external: [
       '@supabase/auth-js',
       '@supabase/functions-js',
       '@supabase/postgrest-js',
       '@supabase/realtime-js',
       '@supabase/storage-js',
     ],
+    noExternal: ['@supabase/tracing'],
     fixedExtension: true,
     hash: false,
     target: 'es2017',

packages/core/supabase-js/tsdown.config.ts

@@ -5,6 +5,13 @@
   "main": "dist/main/index.js",
   "module": "dist/module/index.js",
   "types": "dist/module/index.d.ts",
+  "exports": {
+    ".": {
+      "import": "./dist/module/index.js",
+      "require": "./dist/main/index.js",
+      "types": "./dist/module/index.d.ts"
+    }
+  },
   "sideEffects": false,
   "scripts": {
     "build": "npm run build:main && npm run build:module",

packages/shared/tracing/package.json

@@ -14,12 +14,14 @@ import type { TraceContext } from './types'
  * const context = extractTraceContext()
  * ```
  */
-export function extractTraceContext(): TraceContext | null {
-  // Try OpenTelemetry API (dynamic require, fails gracefully if not available)
+export async function extractTraceContext(): Promise<TraceContext | null> {
+  // Try OpenTelemetry API (dynamic import, fails gracefully if not available)
   try {
-    // Use dynamic require to avoid bundling @opentelemetry/api
-    // This will only work if the user has installed it
-    const otel = require('@opentelemetry/api')
+    // Use dynamic import to avoid bundling @opentelemetry/api and to prevent
+    // bundlers from emitting a top-level createRequire(import.meta.url) shim,
+    // which breaks in Deno when the module is loaded via an https:// URL.
+    // @ts-ignore - @opentelemetry/api is an optional peer dependency
+    const otel = await import('@opentelemetry/api')
 
     if (!otel || !otel.propagation || !otel.context) {
       return null

packages/shared/tracing/src/extract.ts

@@ -2,9 +2,9 @@ import { extractTraceContext } from '../src/extract'
 
 describe('extractTraceContext', () => {
   describe('OpenTelemetry API', () => {
-    it('should return null when OTel API is not available', () => {
+    it('should return null when OTel API is not available', async () => {
       // Without OpenTelemetry API installed, should return null
-      const result = extractTraceContext()
+      const result = await extractTraceContext()
 
       // Since @opentelemetry/api is not installed in this package, this should return null
       expect(result).toBeNull()