test(repo): test module resolution

Author: mandarini

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

@@ -110,6 +110,42 @@ jobs:
           parallel-finished: true
           fail-on-error: false
         continue-on-error: true
+
+  module-resolution:
+    name: Module Resolution Tests
+    runs-on: ubuntu-latest
+    needs: build-package
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+        with:
+          filter: tree:0
+          fetch-depth: 0
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: ${{ env.NODE_VERSION }}
+          cache: 'npm'
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Download build artifacts
+        uses: actions/download-artifact@v4
+        with:
+          name: packages-dist
+          path: ./packages/core
+
+      - name: Validate package exports (attw)
+        run: npx nx test:exports supabase-js
+
+      - name: Test ESM imports
+        run: npx nx test:esm supabase-js
+
+      - name: Test CJS requires
+        run: npx nx test:cjs supabase-js
+
   deno-tests:
     name: Deno Tests
     runs-on: ubuntu-latest

package-lock.json

@@ -19,6 +19,7 @@
   },
   "private": true,
   "devDependencies": {
+    "@arethetypeswrong/cli": "^0.18.2",
     "@commitlint/cli": "^19.8.1",
     "@commitlint/config-conventional": "^19.8.1",
     "@commitlint/cz-commitlint": "^19.8.1",

package.json

@@ -46,7 +46,6 @@
     "ws": "^8.18.2"
   },
   "devDependencies": {
-    "@arethetypeswrong/cli": "^0.16.4",
     "@vitest/coverage-v8": "^3.1.4",
     "esm": "^3.2.25",
     "jsdom": "^16.7.0",

packages/core/realtime-js/example/package-lock.json

@@ -56,6 +56,10 @@
     "test:expo": "cd test/integration/expo && npm test",
     "test:next": "cd test/integration/next && npm test",
     "test:types": "tsd --files test/types/*.test-d.ts && jsr publish --dry-run --allow-dirty",
+    "test:exports": "attw --pack .",
+    "test:esm": "node test/module-resolution.test.mjs",
+    "test:cjs": "node test/module-resolution.test.cjs",
+    "test:module-resolution": "npm run test:exports && npm run test:esm && npm run test:cjs",
     "docs": "typedoc --entryPoints src/index.ts --out docs/v2",
     "docs:json": "typedoc --entryPoints src/index.ts --json docs/v2/spec.json --excludeExternals",
     "serve:coverage": "npx nx test:coverage supabase-js && serve test/coverage",

packages/core/realtime-js/package.json

@@ -81,6 +81,19 @@
           return originalSetInterval.apply(this, [fn, delay, ...args])
         }
 
+        // Verify all clients are bundled (catches AuthClient null bug from 2.86.1)
+        log('Verifying bundled clients...')
+        const testClient = createClient('https://example.supabase.co', 'test-key')
+        if (!testClient.auth) throw new Error('auth client missing (AuthClient null bug)')
+        if (typeof testClient.auth.signUp !== 'function') throw new Error('auth.signUp missing')
+        if (typeof testClient.auth.getSession !== 'function')
+          throw new Error('auth.getSession missing')
+        if (!testClient.storage) throw new Error('storage client missing')
+        if (!testClient.functions) throw new Error('functions client missing')
+        if (typeof testClient.from !== 'function') throw new Error('postgrest client missing')
+        if (typeof testClient.channel !== 'function') throw new Error('realtime client missing')
+        log('All clients bundled correctly')
+
         log('Creating Supabase client with WebSocket transport and vsn: ' + vsn + '...')
         const supabase = createClient(
           'http://127.0.0.1:54321',

packages/core/supabase-js/package.json

@@ -0,0 +1,70 @@
+/**
+ * Module Resolution Tests (CommonJS)
+ *
+ * Tests that the package works correctly when required via CommonJS.
+ * This catches issues like the wrapper.mjs bug that broke jsdelivr CDN imports.
+ *
+ * Run with: node test/module-resolution.test.cjs
+ */
+
+const assert = require('assert')
+
+console.log('Testing CommonJS require...\n')
+
+// Test 1: Main export works
+console.log('1. Testing main export...')
+const supabase = require('../dist/index.cjs')
+assert(supabase, 'Main export should exist')
+assert(typeof supabase.createClient === 'function', 'createClient should be a function')
+console.log('   ✅ Main export works\n')
+
+// Test 2: createClient works
+console.log('2. Testing createClient...')
+const { createClient } = supabase
+const client = createClient('https://example.supabase.co', 'test-key')
+assert(client, 'Client should be created')
+console.log('   ✅ createClient works\n')
+
+// Test 3: Auth client exists and has methods
+console.log('3. Testing Auth client...')
+assert(client.auth, 'client.auth should exist')
+assert(typeof client.auth.signUp === 'function', 'auth.signUp should be a function')
+assert(
+  typeof client.auth.signInWithPassword === 'function',
+  'auth.signInWithPassword should be a function'
+)
+assert(typeof client.auth.signOut === 'function', 'auth.signOut should be a function')
+assert(typeof client.auth.getSession === 'function', 'auth.getSession should be a function')
+assert(typeof client.auth.getUser === 'function', 'auth.getUser should be a function')
+assert(
+  typeof client.auth.onAuthStateChange === 'function',
+  'auth.onAuthStateChange should be a function'
+)
+console.log('   ✅ Auth client works\n')
+
+// Test 4: PostgREST client exists
+console.log('4. Testing PostgREST client...')
+assert(typeof client.from === 'function', 'client.from should be a function')
+const query = client.from('test')
+assert(query, 'from() should return a query builder')
+console.log('   ✅ PostgREST client works\n')
+
+// Test 5: Storage client exists
+console.log('5. Testing Storage client...')
+assert(client.storage, 'client.storage should exist')
+assert(typeof client.storage.from === 'function', 'storage.from should be a function')
+console.log('   ✅ Storage client works\n')
+
+// Test 6: Functions client exists
+console.log('6. Testing Functions client...')
+assert(client.functions, 'client.functions should exist')
+assert(typeof client.functions.invoke === 'function', 'functions.invoke should be a function')
+console.log('   ✅ Functions client works\n')
+
+// Test 7: Realtime client exists
+console.log('7. Testing Realtime client...')
+assert(typeof client.channel === 'function', 'client.channel should be a function')
+assert(typeof client.removeChannel === 'function', 'client.removeChannel should be a function')
+console.log('   ✅ Realtime client works\n')
+
+console.log('🎉 All CommonJS tests passed!')

packages/core/supabase-js/test/integration/node-browser/index.html

@@ -0,0 +1,76 @@
+/**
+ * Module Resolution Tests (ESM)
+ *
+ * Tests that the package works correctly when imported via ESM.
+ * This catches issues like the wrapper.mjs bug that broke jsdelivr CDN imports.
+ *
+ * Run with: node test/module-resolution.test.mjs
+ */
+
+import assert from 'assert'
+
+console.log('Testing ESM import...\n')
+
+// Test 1: Named exports work
+console.log('1. Testing named exports...')
+const { createClient } = await import('../dist/index.mjs')
+assert(typeof createClient === 'function', 'createClient should be a function')
+console.log('   ✅ Named exports work\n')
+
+// Test 2: createClient works
+console.log('2. Testing createClient...')
+const client = createClient('https://example.supabase.co', 'test-key')
+assert(client, 'Client should be created')
+console.log('   ✅ createClient works\n')
+
+// Test 3: Auth client exists and has methods (this is what broke in 2.86.1)
+console.log('3. Testing Auth client (AuthClient null bug check)...')
+assert(client.auth, 'client.auth should exist (was null in 2.86.1 bug)')
+assert(typeof client.auth.signUp === 'function', 'auth.signUp should be a function')
+assert(
+  typeof client.auth.signInWithPassword === 'function',
+  'auth.signInWithPassword should be a function'
+)
+assert(typeof client.auth.signOut === 'function', 'auth.signOut should be a function')
+assert(typeof client.auth.getSession === 'function', 'auth.getSession should be a function')
+assert(typeof client.auth.getUser === 'function', 'auth.getUser should be a function')
+assert(
+  typeof client.auth.onAuthStateChange === 'function',
+  'auth.onAuthStateChange should be a function'
+)
+console.log('   ✅ Auth client works (no AuthClient null bug)\n')
+
+// Test 4: PostgREST client exists
+console.log('4. Testing PostgREST client...')
+assert(typeof client.from === 'function', 'client.from should be a function')
+const query = client.from('test')
+assert(query, 'from() should return a query builder')
+console.log('   ✅ PostgREST client works\n')
+
+// Test 5: Storage client exists
+console.log('5. Testing Storage client...')
+assert(client.storage, 'client.storage should exist')
+assert(typeof client.storage.from === 'function', 'storage.from should be a function')
+console.log('   ✅ Storage client works\n')
+
+// Test 6: Functions client exists
+console.log('6. Testing Functions client...')
+assert(client.functions, 'client.functions should exist')
+assert(typeof client.functions.invoke === 'function', 'functions.invoke should be a function')
+console.log('   ✅ Functions client works\n')
+
+// Test 7: Realtime client exists
+console.log('7. Testing Realtime client...')
+assert(typeof client.channel === 'function', 'client.channel should be a function')
+assert(typeof client.removeChannel === 'function', 'client.removeChannel should be a function')
+console.log('   ✅ Realtime client works\n')
+
+// Test 8: Type exports (check they don't throw)
+console.log('8. Testing type re-exports...')
+const mod = await import('../dist/index.mjs')
+// These should all be importable without errors
+const exports = Object.keys(mod)
+assert(exports.includes('createClient'), 'Should export createClient')
+console.log(`   ✅ Module exports ${exports.length} items\n`)
+
+console.log('🎉 All ESM tests passed!')