Skip to content

Commit 940479b

Browse files
briancmesqueeb
brianc
and
mesqueeb
authored
Add esm exports (#3423)
* build: add esm exports * fix: add defaults as per arethetypeswrong report * fix: add missing types * lint * Fix broken tests * Add (failing) test for esm compat * Begin moving files to proper extension and adding tests * Add tests for connection-string and fix cloudflare module type and esm compat * Add query-stream and cursor as esm exports * Update PR copilot review * Publish - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] * More cf compat work * Publish - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] - [email protected] * Add more cf compat and update tests * Make tests pass - update exports for esm * Use env vars for test connection in cf tests * Fix lint * Fit vitest into existing legacy framework * Skip worker tests on node below 18 * Revert doc changes for now * Remove legacy worker test in favor of vitest --------- Co-authored-by: Luca Ban <[email protected]>
1 parent 81ec063 commit 940479b

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+1684
-463
lines changed

‎package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"packages/*"
1111
],
1212
"scripts": {
13-
"test": "yarn lerna exec yarn test",
13+
"test": "yarn lerna exec --concurrency 1 yarn test",
1414
"build": "tsc --build",
1515
"build:watch": "tsc --build --watch",
1616
"docs:build": "cd docs && yarn build",

‎packages/pg-cloudflare/esm/index.mjs‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import cf from '../dist/index.js'
2+
3+
export const CloudflareSocket = cf.CloudflareSocket

‎packages/pg-cloudflare/package.json‎

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,20 @@
11
{
22
"name": "pg-cloudflare",
3-
"version": "1.1.1",
3+
"version": "1.1.2-alpha.1",
44
"description": "A socket implementation that can run on Cloudflare Workers using native TCP connections.",
5-
"main": "dist/empty.js",
5+
"main": "dist/index.js",
66
"types": "dist/index.d.ts",
77
"license": "MIT",
88
"devDependencies": {
99
"ts-node": "^8.5.4",
1010
"typescript": "^4.0.3"
1111
},
1212
"exports": {
13-
"workerd": "./dist/index.js",
14-
"default": "./dist/empty.js"
13+
".": {
14+
"import": "./esm/index.mjs",
15+
"require": "./dist/index.js",
16+
"default": "./dist/index.js"
17+
}
1518
},
1619
"scripts": {
1720
"build": "tsc",
@@ -26,6 +29,7 @@
2629
},
2730
"files": [
2831
"/dist/*{js,ts,map}",
29-
"/src"
32+
"/src",
33+
"/esm"
3034
]
3135
}

‎packages/pg-cloudflare/src/index.ts‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,8 @@ export class CloudflareSocket extends EventEmitter {
3737
if (connectListener) this.once('connect', connectListener)
3838

3939
const options: SocketOptions = this.ssl ? { secureTransport: 'starttls' } : {}
40-
const { connect } = await import('cloudflare:sockets')
40+
const mod = await import('cloudflare:sockets')
41+
const connect = mod.connect
4142
this._cfSocket = connect(`${host}:${port}`, options)
4243
this._cfWriter = this._cfSocket.writable.getWriter()
4344
this._addClosedHandler()

‎packages/pg-cloudflare/tsconfig.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
{
22
"compilerOptions": {
3-
"module": "ES2020",
3+
"module": "node16",
44
"esModuleInterop": true,
55
"allowSyntheticDefaultImports": true,
66
"strict": true,
7-
"target": "ES2020",
7+
"target": "es2020",
88
"noImplicitAny": true,
9-
"moduleResolution": "node",
9+
"moduleResolution": "node16",
1010
"sourceMap": true,
1111
"outDir": "dist",
1212
"incremental": true,

‎packages/pg-connection-string/esm/index.mjs‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
// ESM wrapper for pg-connection-string
2+
import connectionString from '../index.js'
3+
4+
// Re-export the parse function
5+
export const parse = connectionString.parse
6+
export const toClientConfig = connectionString.toClientConfig
7+
export const parseIntoClientConfig = connectionString.parseIntoClientConfig

‎packages/pg-connection-string/package.json‎

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,17 @@
11
{
22
"name": "pg-connection-string",
3-
"version": "2.7.0",
3+
"version": "2.7.1-alpha.0",
44
"description": "Functions for dealing with a PostgresSQL connection string",
55
"main": "./index.js",
66
"types": "./index.d.ts",
7+
"exports": {
8+
".": {
9+
"types": "./index.d.ts",
10+
"import": "./esm/index.mjs",
11+
"require": "./index.js",
12+
"default": "./index.js"
13+
}
14+
},
715
"scripts": {
816
"test": "istanbul cover _mocha && npm run check-coverage",
917
"check-coverage": "istanbul check-coverage --statements 100 --branches 100 --lines 100 --functions 100",
@@ -34,6 +42,7 @@
3442
},
3543
"files": [
3644
"index.js",
37-
"index.d.ts"
45+
"index.d.ts",
46+
"esm"
3847
]
3948
}

‎packages/pg-cursor/esm/index.mjs‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
// ESM wrapper for pg-cursor
2+
import Cursor from '../index.js'
3+
4+
// Export as default only to match CJS module
5+
export default Cursor

‎packages/pg-cursor/index.js‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
2-
const Result = require('pg/lib/result.js')
3-
const prepare = require('pg/lib/utils.js').prepareValue
2+
const pg = require('pg')
3+
const { Result, utils } = pg
4+
const prepare = utils.prepareValue
45
const EventEmitter = require('events').EventEmitter
56
const util = require('util')
67

‎packages/pg-cursor/package.json‎

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,15 @@
11
{
22
"name": "pg-cursor",
3-
"version": "2.13.1",
3+
"version": "2.13.2-alpha.1",
44
"description": "Query cursor extension for node-postgres",
55
"main": "index.js",
6+
"exports": {
7+
".": {
8+
"import": "./esm/index.mjs",
9+
"require": "./index.js",
10+
"default": "./index.js"
11+
}
12+
},
613
"directories": {
714
"test": "test"
815
},
@@ -18,9 +25,13 @@
1825
"license": "MIT",
1926
"devDependencies": {
2027
"mocha": "^10.5.2",
21-
"pg": "^8.14.1"
28+
"pg": "^8.14.2-alpha.1"
2229
},
2330
"peerDependencies": {
2431
"pg": "^8"
25-
}
32+
},
33+
"files": [
34+
"index.js",
35+
"esm"
36+
]
2637
}

0 commit comments

Comments
 (0)