8.11.0 - Return outside of function

[bryantqo]

Starting in version 8 a line was added to which a return happens outside of a function. I think this was added in 5532ca5 located in packages/pg/lib/crypto/utils.js. This recently popped up in our jest tests with an error of

SyntaxError: /node_modules/pg/lib/crypto/utils.js: 'return' outside of function. (7:2)

Reverting to 8.10.0 solves the problem for now

[Jolg42]

I also noticed this today in a renovate PR upgrade of pg package.

[charmander]

This can be changed in pg for everyone’s convenience, but that sounds like a bug in whatever tool isn’t recognizing a legitimate (and not infrequently used) feature of CommonJS modules.

[darrinholst]

Just ran into this on an esm project that we were still using jest for. You have to jump through some hoops to get jest working with esm. Switching to vitest fixed it for us.

[lucgagan]

Confirmed that pinning version v8.10.0 fixes the issue.

For what it is worth, Next.js team is refusing to fix this (CC @shuding) calling this pg issue – making pg unusable with Next.js.

[gronka]

@lucgagan
How do you pin versions? I added this to package.json and ran rm -rf node_modules && npm install

	"overrides": {
		"node-postgres": "8.10.0"
	}

When I set "pg": "8.10.0", I get the same error

[orlein]

I just commented out the if clause using patch-package because I'm using node v18.12.0

1. comment out the problematic code in node_modules/pg/lib/crypto/utils.js
2. run npx patch-package pg
3. add "postinstall": "patch-package" inside package.json's scripts
4. run yarn add -D patch-package

and your project will automatically comment out the if clause whenever you and your colleague run yarn install.

check out for the further info: https://www.npmjs.com/package/patch-package

[marinamashina]

We ran into the same issue but on 8.5.1 while this was not happening before. Looking for any practical explanation and solution to this