fix(pg-cloudflare): use conditional export to support bundlers that don't know about cloudflare:sockets (#3482)

Author: CHC383

packages/pg-bundler-test/esbuild-cloudflare.config.mjs

@@ -0,0 +1,8 @@
+import * as esbuild from 'esbuild'
+
+await esbuild.build({
+ entryPoints: ['./src/index.mjs'],
+ bundle: true,
+ outfile: './dist/esbuild-cloudflare.js',
+ conditions: ['import', 'cloudflare'],
+})

packages/pg-bundler-test/esbuild-empty.config.mjs

@@ -0,0 +1,7 @@
+import * as esbuild from 'esbuild'
+
+await esbuild.build({
+ entryPoints: ['./src/index.mjs'],
+ bundle: true,
+ outfile: './dist/esbuild-empty.js',
+})

packages/pg-bundler-test/package.json

@@ -0,0 +1,25 @@
+{
+ "name": "pg-bundler-test",
+ "version": "0.0.0",
+ "description": "Test bundlers with pg-cloudflare, https://github.com/brianc/node-postgres/issues/3452",
+ "license": "MIT",
+ "private": true,
+ "type": "module",
+ "devDependencies": {
+ "pg-cloudflare": "^1.2.5",
+ "webpack": "^5.99.9",
+ "webpack-cli": "^6.0.1",
+ "rollup": "^4.41.1",
+ "@rollup/plugin-node-resolve": "^16.0.1",
+ "@rollup/plugin-commonjs": "^28.0.3",
+ "vite": "^6.3.5",
+ "esbuild": "^0.25.5"
+ },
+ "scripts": {
+ "test": "yarn webpack && yarn rollup && yarn vite && yarn esbuild",
+ "webpack": "webpack --config webpack-empty.config.mjs && webpack --config webpack-cloudflare.config.mjs",
+ "rollup": "rollup --config rollup-empty.config.mjs --failAfterWarnings && rollup --config rollup-cloudflare.config.mjs --failAfterWarnings",
+ "vite": "[ $(node --version | sed 's/v//' | cut -d'.' -f1) -ge 18 ] && vite build --config vite-empty.config.mjs && vite build --config vite-cloudflare.config.mjs || echo 'Skip Vite test'",
+ "esbuild": "node esbuild-empty.config.mjs && node esbuild-cloudflare.config.mjs"
+ }
+}

packages/pg-bundler-test/rollup-cloudflare.config.mjs

@@ -0,0 +1,13 @@
+import { defineConfig } from 'rollup'
+import { nodeResolve } from '@rollup/plugin-node-resolve'
+import commonjs from '@rollup/plugin-commonjs'
+
+export default defineConfig({
+ input: './src/index.mjs',
+ output: {
+ file: 'dist/rollup-cloudflare.js',
+ format: 'es',
+ },
+ plugins: [nodeResolve({ exportConditions: ['import', 'cloudflare'], preferBuiltins: true }), commonjs()],
+ external: ['cloudflare:sockets'],
+})

packages/pg-bundler-test/rollup-empty.config.mjs

@@ -0,0 +1,12 @@
+import { defineConfig } from 'rollup'
+import { nodeResolve } from '@rollup/plugin-node-resolve'
+import commonjs from '@rollup/plugin-commonjs'
+
+export default defineConfig({
+ input: './src/index.mjs',
+ output: {
+ file: 'dist/rollup-empty.js',
+ format: 'es',
+ },
+ plugins: [nodeResolve(), commonjs()],
+})

packages/pg-bundler-test/src/index.mjs

@@ -0,0 +1 @@
+import 'pg-cloudflare'

packages/pg-bundler-test/vite-cloudflare.config.mjs

@@ -0,0 +1,20 @@
+import { defineConfig } from 'vite'
+import commonjs from '@rollup/plugin-commonjs'
+
+export default defineConfig({
+ build: {
+ emptyOutDir: false,
+ lib: {
+ entry: './src/index.mjs',
+ fileName: 'vite-cloudflare',
+ formats: ['es'],
+ },
+ rollupOptions: {
+ external: ['cloudflare:sockets'],
+ },
+ },
+ resolve: {
+ conditions: ['import', 'cloudflare'],
+ },
+ plugins: [commonjs()],
+})

packages/pg-bundler-test/vite-empty.config.mjs

@@ -0,0 +1,12 @@
+import { defineConfig } from 'vite'
+
+export default defineConfig({
+ build: {
+ emptyOutDir: false,
+ lib: {
+ entry: './src/index.mjs',
+ fileName: 'vite-empty',
+ formats: ['es'],
+ },
+ },
+})

packages/pg-bundler-test/webpack-cloudflare.config.mjs

@@ -0,0 +1,16 @@
+import webpack from 'webpack'
+
+export default {
+ mode: 'production',
+ entry: './src/index.mjs',
+ output: {
+ filename: 'webpack-cloudflare.js',
+ },
+ resolve: { conditionNames: ['import', 'cloudflare'] },
+ plugins: [
+ // ignore cloudflare:sockets imports
+ new webpack.IgnorePlugin({
+ resourceRegExp: /^cloudflare:sockets$/,
+ }),
+ ],
+}

packages/pg-bundler-test/webpack-empty.config.mjs

@@ -0,0 +1,7 @@
+export default {
+ mode: 'production',
+ entry: './src/index.mjs',
+ output: {
+ filename: 'webpack-empty.js',
+ },
+}