[ChuTingzj]

Relates to #11 — #11

Rebased onto main after #12 squash-merge (c09de54). Tests only; no runner image or npm publish.

Why

#12 landed the runner fix (streamPgQuery + unit tests on event objects). This PR locks the host-visible NDJSON contract that issue #11 actually broke: consumeNdjsonResult getting columns: [] because pg 8.x never emits fields.

Asserted

Against the Postgres execute path (streamPgQuery) with a mocked pg 8.x Result (FieldDef[] on result.fields):

1. Query never emits fields — Pg8Query.emit('fields') throws; fieldsEmitted === 0.
2. NDJSON meta — first line is {"type":"meta","columns":["order_date","daily_total"]} from result.fields.
3. Host consumer — consumeNdjsonResult returns those column names (not []).
4. Zero-row schema — end with result.fields still writes meta then end.
5. execute.js wiring — streamPg delegates to streamPgQuery and does not query.on('fields').

These tests fail on fields-event-only code (rows/end, no meta, columns: []) and pass on current main + this commit.

How to run

pnpm test
pnpm typecheck
pnpm build

Local: 63 tests, 0 failures. Do not merge. Do not publish npm.

  

[ChuTingzj]

Pre-scan (533f9d4)

Blocker for merge: base is still cursor/postgres-meta-columns-6f52 (4f5752a), not main (c09de54 where #12 landed). Rebase onto main first, then re-request review.

Content (looks good once rebased): locks the real #11 failure mode — host consumeNdjsonResult gets non-empty columns from NDJSON meta, with a Query that throws if fields is emitted. Zero-row schema + execute.js / pg-query-stream.js source guards are the right layer above #12’s unit tests.

No code must-fix in the test file itself. Hold merge until rebase.

[cursor]

Approved: Cursor Bugbot completed successfully with no findings that need human review. No approval-policy files apply, and this automation has not assigned reviewers.

  

_{Sent by Cursor Approval Agent: Pull Request Router and Approver}

[ChuTingzj]

Re-scan (6875bdf)

Rebase blocker cleared — base is main (c09de54). Locks #11 host contract: NDJSON meta from result.fields, no fields event, consumeNdjsonResult.columns non-empty, zero-row schema, execute.js wiring. No must-fix.

OK to merge (human). Digest/0.1.4 stays on the other line.