chore: bump @e4a/pg-js to 1.10.0#107
Conversation
There was a problem hiding this comment.
Self-authored, so cannot --approve. Verdict: LGTM, ready to merge from this reviewer's side.
Verified locally:
npm ciclean, 0 vulnerabilitiesnpm run docs:buildsucceeds in ~7s- CI build job passes
Heads-up (non-blocking): @e4a/pg-js@1.10.0 declares engines.node: ">=22", but Dockerfile still pins node:20-alpine. npm doesn't enforce engines by default, so the Docker build still passes — but a future tightening (e.g. engine-strict=true or pg-js dropping Node 20 syntactically) would break the prod image. Worth a follow-up bump to node:22-alpine at some point.
@e4a/pg-js@1.10.0 declares engines.node >=22; align the Docker build image and README prerequisite accordingly.
There was a problem hiding this comment.
Addressed the Node engine heads-up in 9e749c2:
- Dockerfile:
node:20-alpine→node:22-alpine(matches@e4a/pg-js@1.10.0enginesnode >=22) - README prerequisite:
Node.js 18+→Node.js 22+
Verified npm ci + npm run docs:build still pass locally.
There was a problem hiding this comment.
Self-authored, so cannot --approve. Verdict: LGTM, ready to merge from this reviewer's side (review cycle 2/3).
Verified locally on cycle 2:
npm ciclean, 0 vulnerabilitiesnpm run docs:buildsucceeds in ~8s on Node 22- CI: build + Conventional Commit checks both pass
The engines heads-up from cycle 1 is addressed in 9e749c2: Dockerfile is now node:22-alpine and README prerequisite is Node.js 22+, matching @e4a/pg-js@1.10.0 (engines.node: >=22). Diff is tight (4 files, all expected for a pg-js minor bump + Node alignment). Nothing else to flag.
Bumps
@e4a/pg-jsfrom 1.8.0 to 1.10.0 (single minor jump; 1.9.0 → 1.10.0 chained through one npm install).Verified
npm run docs:buildsucceeds in 8.4s. No other test suite in this repo.Closes #106