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

Author: CHC383

packages/pg-cloudflare/README.md

@@ -10,6 +10,28 @@
 npm i --save-dev pg-cloudflare
 ```
 
+The package uses conditional exports to support bundlers that don't know about
+`cloudflare:sockets`, so the consumer code by default imports an empty file. To
+enable the package, resolve to the `cloudflare` condition in your bundler's
+config. For example:
+
+- `webpack.config.js`
+  ```js
+  resolve: { conditionNames: [..., "cloudflare"] },
+  plugins: [
+    // ignore cloudflare:sockets imports
+    new webpack.IgnorePlugin({
+      resourceRegExp: /^cloudflare:sockets$/,
+    }),
+  ],
+  ```
+- `vite.config.js`
+  ```js
+  resolve: {
+    conditions: [..., "cloudflare"],
+  },
+  ```
+
 ## How to use conditionally, in non-Node.js environments
 
 As implemented in `pg` [here](https://github.com/brianc/node-postgres/commit/07553428e9c0eacf761a5d4541a3300ff7859578#diff-34588ad868ebcb232660aba7ee6a99d1e02f4bc93f73497d2688c3f074e60533R5-R13), a typical use case might look as follows, where in a Node.js environment the `net` module is used, while in a non-Node.js environment, where `net` is unavailable, `pg-cloudflare` is used instead, providing an equivalent interface:
@@ -21,15 +43,14 @@ module.exports.getStream = function getStream(ssl = false) {
     return net.Socket()
   }
   const { CloudflareSocket } = require('pg-cloudflare')
-  return new CloudflareSocket(ssl);
+  return new CloudflareSocket(ssl)
 }
 ```
 
 ## Node.js implementation of the Socket API proposal
 
 If you're looking for a way to rely on `connect()` as the interface you use to interact with raw sockets, but need this interface to be available in a Node.js environment, [`@arrowood.dev/socket`](https://github.com/Ethan-Arrowood/socket) provides a Node.js implementation of the Socket API.
 
-
 ### license
 
 The MIT License (MIT)

packages/pg-cloudflare/package.json

@@ -11,9 +11,11 @@
   },
   "exports": {
     ".": {
-      "import": "./esm/index.mjs",
-      "require": "./dist/index.js",
-      "default": "./dist/index.js"
+      "cloudflare": {
+        "import": "./esm/index.mjs",
+        "require": "./dist/index.js"
+      },
+      "default": "./dist/empty.js"
     }
   },
   "scripts": {

packages/pg-esm-test/package.json

@@ -5,7 +5,7 @@
   "main": "index.js",
   "type": "module",
   "scripts": {
-    "test": "node --test"
+    "test": "node --test --conditions=cloudflare"
   },
   "keywords": [
     "postgres",

packages/pg-webpack-test/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "webpack-test",
+  "version": "0.0.0",
+  "description": "Test webpack bundle with pg-cloudflare, https://github.com/brianc/node-postgres/issues/3452",
+  "private": true,
+  "license": "MIT",
+  "devDependencies": {
+    "webpack": "^5.99.9",
+    "webpack-cli": "^6.0.1",
+    "pg-cloudflare": "^1.2.5"
+  },
+  "scripts": {
+    "test": "webpack --config webpack-empty.config.js && webpack --config webpack-cloudflare.config.js"
+  }
+}

packages/pg-webpack-test/src/index.js

@@ -0,0 +1 @@
+const _ = require('pg-cloudflare')

packages/pg-webpack-test/webpack-cloudflare.config.js

@@ -0,0 +1,14 @@
+import webpack from 'webpack'
+
+const config = {
+  mode: 'production',
+  resolve: { conditionNames: ['import', 'cloudflare'] },
+  plugins: [
+    // ignore cloudflare:sockets imports
+    new webpack.IgnorePlugin({
+      resourceRegExp: /^cloudflare:sockets$/,
+    }),
+  ],
+}
+
+export default config

packages/pg-webpack-test/webpack-empty.config.js

@@ -0,0 +1,5 @@
+const config = {
+  mode: 'production',
+}
+
+export default config