fix(misc): add tests

Author: mandarini

.github/workflows/ci-supabase-js.yml

@@ -129,3 +129,6 @@ jobs:
 
       - name: Test CJS requires
         run: pnpm nx test:cjs supabase-js
+
+      - name: Test Hermes (React Native) compatibility
+        run: pnpm nx test:hermes-compat supabase-js

packages/core/supabase-js/package.json

@@ -69,7 +69,8 @@
     "test:exports": "attw --pack . --ignore-rules no-resolution",
     "test:esm": "node test/module-resolution.test.mjs && node test/module-resolution-cors.test.mjs",
     "test:cjs": "node test/module-resolution.test.cjs && node test/module-resolution-cors.test.cjs",
-    "test:module-resolution": "npm run test:exports && npm run test:esm && npm run test:cjs",
+    "test:hermes-compat": "node test/bundle-hermes-compat.test.cjs",
+    "test:module-resolution": "npm run test:exports && npm run test:esm && npm run test:cjs && npm run test:hermes-compat",
     "docs": "typedoc --entryPoints src/index.ts --entryPoints src/cors.ts --out docs/v2",
     "docs:json": "typedoc --entryPoints src/index.ts --entryPoints src/cors.ts --json docs/v2/spec.json --excludeExternals",
     "serve:coverage": "pnpm nx test:coverage supabase-js && pnpm dlx serve test/coverage",

packages/core/supabase-js/test/bundle-hermes-compat.test.cjs

@@ -0,0 +1,44 @@
+/**
+ * Hermes Compatibility Test
+ *
+ * Asserts dist/index.cjs contains no bare import() expressions.
+ * hermesc (React Native bytecode compiler) rejects import() at parse time —
+ * before dead-code elimination — so the syntax must be physically absent.
+ * The only allowed occurrence is inside a new Function() string body, which
+ * is opaque to all static parsers including hermesc.
+ *
+ * Run with: node test/bundle-hermes-compat.test.cjs
+ */
+
+const assert = require('assert')
+const fs = require('fs')
+const path = require('path')
+
+console.log('Testing Hermes (React Native) compatibility...\n')
+
+const cjsPath = path.join(__dirname, '../dist/index.cjs')
+const cjs = fs.readFileSync(cjsPath, 'utf8')
+const lines = cjs.split('\n')
+
+let bareImportCount = 0
+for (let i = 0; i < lines.length; i++) {
+  const line = lines[i]
+  if (!line.includes('import(')) continue
+  if (!line.includes('new Function(')) {
+    bareImportCount++
+    console.error(
+      `❌ Bare import() at line ${i + 1}:\n   ${line.trim()}\n` +
+        `   This breaks hermesc (Hermes bytecode compiler for React Native).`
+    )
+  }
+}
+
+assert.strictEqual(
+  bareImportCount,
+  0,
+  `${bareImportCount} bare import() expression(s) in dist/index.cjs — breaks React Native Hermes builds.`
+)
+
+console.log('1. No bare import() expressions in dist/index.cjs')
+console.log('   ✅ Hermes-safe (React Native compatible)\n')
+console.log('🎉 Hermes compatibility test passed!')

packages/shared/tracing/src/extract.ts

@@ -21,9 +21,7 @@ const OTEL_PKG = '@opentelemetry/api'
 // absent from the bundle. new Function() bodies are opaque strings to all
 // static parsers, including hermesc.
 // eslint-disable-next-line no-new-func
-const _dynamicImport = new Function('p', 'return import(p)') as (
-  pkg: string
-) => Promise<any>
+const _dynamicImport = new Function('p', 'return import(p)') as (pkg: string) => Promise<any>
 
 function loadOtel(): Promise<any | null> {
   if (otelModulePromise === null) {