mysql: keep fractional seconds when decoding binary TIME values by alii · Pull Request #31738 · oven-sh/bun

alii · 2026-06-02T22:32:27Z

Bun.sql (MySQL) silently dropped the fractional-seconds part of a binary TIME value: TIME(6) 02:03:04.5 decoded to "02:03:04". The binary decoder parsed the microseconds field but the format string never wrote it.

Route the binary TIME arm through the shared SQL time formatter (Postgres__formatTime), which already emits this shape: zero-padded components with the fractional part zero-padded to 6 digits and trailing zeros stripped (so 02:03:04.5 → .5, .123456 → .123456, .000500 → .0005). Only the sign stays MySQL-specific (the wire form carries absolute components plus an is_negative byte). The no-fractional / 8-byte form stays "HH:MM:SS", and the µs total is computed in u64 clamped to the ±838:59:59.999999 cap — which also removes a u32 overflow the old hours math had for malformed day counts. The text protocol was already correct (it passes the server's string through verbatim) — only the binary path was affected.

Verified against a live MariaDB: post-fix output is byte-identical to node mysql2 across 02:03:04 / .5 / .123456 / -… / 838:59:58.999999 on both protocols. (For sub-0.1s fractions Bun emits the numerically-correct value; mysql2 happens to mis-format those, so Bun is strictly more correct there.) Not a port regression — the Zig decoder has the same omission. Adds a real-MySQL TIME(6) regression test, plus a no-docker mock-server case (the existing NEWDECIMAL mock suite, parameterized) that drives the binary decoder over the 8-byte and 12-byte wire forms so the fix is exercised where no docker daemon is available.

alii · 2026-06-02T22:32:28Z

@robobun adopt

coderabbitai · 2026-06-02T22:32:33Z

Warning

Review limit reached

@robobun, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 46 minutes and 14 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 36ac8cda-2942-4554-a158-196725221522

📥 Commits

Reviewing files that changed from the base of the PR and between c6ac56d and 8e32b7d.

📒 Files selected for processing (2)

test/js/sql/sql-mysql-time-fraction.test.ts
test/js/sql/sql-mysql.test.ts

Walkthrough

This PR refactors MySQL's TIME binary decoding to reuse PostgreSQL's existing time formatter. The PostgreSQL formatter is exposed as pub(crate), documented to clarify fractional-second and sign handling across both drivers, and MySQL's MYSQL_TYPE_TIME path is refactored to compute microseconds and delegate to the shared formatter. Comprehensive test coverage validates fractional-second preservation in binary protocol against text-protocol behavior.

Changes

Shared time formatter across MySQL and PostgreSQL drivers

Layer / File(s)	Summary
Expose and document shared time formatter `src/sql_jsc/postgres/DataCell.rs`, `src/jsc/bindings/SQLClient.cpp`	`Postgres__formatTime` FFI declaration becomes `pub(crate)` visible with expanded documentation on buffer guarantees and cross-driver sharing. Comment header is updated to clarify how PostgreSQL and MySQL TIME formatting and sign handling differ, especially for fractional seconds.
MySQL TIME binary decoding refactor `src/sql_jsc/mysql/protocol/DecodeBinaryValue.rs`	MySQL imports `Postgres__formatTime` and refactors `MYSQL_TYPE_TIME` decoding to compute total microseconds, clamp to MySQL's range, handle negative sign, and format via the shared helper using a fixed-size buffer instead of manual formatting.
Test coverage for MySQL TIME fractional seconds `test/js/sql/sql-mysql.test.ts`	Docker-backed test validates TIME(6) binary and text protocol output. Mock-server utilities are extracted and reused across test suites. New mock-server suite verifies binary TIME decoding preserves microsecond fractional seconds (including negative and days forms) while text protocol passes through server-provided strings verbatim.

Suggested reviewers

cirospaciari

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: fixing fractional seconds preservation in MySQL binary TIME decoding.
Description check	✅ Passed	The description provides comprehensive details about what the PR does and how it was verified, covering both aspects of the required template.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

robobun · 2026-06-02T22:32:37Z

^{Updated 12:10 AM PT - Jun 3rd, 2026}

✅ @robobun, your commit 8e32b7d08bf94018df021cd804a520970bf06cda passed in Build #60096! 🎉

🧪 To try this PR locally:

bunx bun-pr 31738

That installs a local version of the PR into your bun-31738 executable, so you can run:

bun-31738 --bun

robobun · 2026-06-02T22:39:43Z

✅ Validated and extended. The decoder change checks out, and per review feedback it now reuses the existing shared time formatter (Postgres__formatTime) instead of hand-rolled digit emission (c6ac56d) — same output, less duplication, and it removes a u32 overflow the old hours math had for malformed packets.

The regression is covered twice: the docker-backed TIME(6) test in sql-mysql.test.ts (runs in CI against real MySQL), and a standalone no-docker mock-server test (sql-mysql-time-fraction.test.ts, 8e32b7d) driving the 8-byte and 12-byte wire forms, trailing-zero stripping (.5, .0005), sign, >99h values, and zero microseconds. Confirmed locally: the mock test fails against the unfixed decoder and passes with the fix.

CI on the head commit (#60096): 73+ checks green — including all musl lanes (the intermittent musl-LTO failures from #60037/#60094 did not recur) and the ASAN lane. The only red is darwin-14-aarch64-test-bun marked Expired — no agent picked the job up, so it never ran; it just needs a retry from the Buildkite UI (or can be disregarded). Earlier builds' reds were likewise single-lane flakes (bun-install-security-provider IPC timeout on Windows aarch64, serve-body-leak RSS ceiling on ASAN). Already used my one re-roll — ready for maintainer review/merge.

claude

LGTM — small, well-contained fix that appends the already-parsed microseconds to the binary TIME string, with a real-server regression test.

Extended reasoning...

Overview

This PR fixes a data-loss bug in the MySQL binary protocol decoder: the Time struct already parsed microseconds from the 12-byte wire form, but the formatter at src/sql_jsc/mysql/protocol/DecodeBinaryValue.rs never emitted them. The fix adds a ~20-line block that, when time.microseconds > 0, writes . followed by the value zero-padded to 6 digits with trailing zeros stripped. A new TIME(6) test in test/js/sql/sql-mysql.test.ts covers zero-fraction, partial-fraction, full 6-digit, negative, and near-max (838:59:58.999999) values on both the binary and text protocols.

Security risks

None. The change is pure output formatting into a fixed 32-byte stack buffer. The fractional digits are written into a fixed [u8; 6] via % 10 arithmetic, so even a malformed u32 microseconds value cannot overflow either buffer. Worst-case output (-838:59:59.999999, 17 bytes) fits comfortably in 32. No untrusted input reaches new control flow; write_all errors map to the existing InvalidBinaryValue path.

Level of scrutiny

Low–medium. This is a narrow correctness fix to a single match arm in a value decoder, mirroring established patterns already in the file (manual digit emission, w.write_all(...).is_err() error handling, buffer[..32 - remaining] slicing). The 8-byte / zero-microsecond path is unchanged, so existing TIME behavior is preserved — confirmed by the pre-existing time test still expecting HH:MM:SS strings.

Other factors

The Time::from_data parser (MySQLValue.rs:829-832) already populated microseconds for the 12-byte form, so this is purely a formatting omission. The new test sits adjacent to and follows the exact structure of the existing time test. No CODEOWNERS entries cover these files, no outstanding reviewer comments, and the bug-hunting system found nothing.

…k server The docker-backed TIME(6) test only runs where a docker daemon is available. Parameterize the existing no-docker mock MySQL server by a canned column and rows, and add a binary TIME case covering the 8-byte and 12-byte wire forms (trailing-zero stripping, sign, >99h, zero microseconds) so the decoder change is exercised everywhere.

claude

LGTM — focused decoder fix with both real-server and offline mock-server coverage.

Extended reasoning...

Overview

This PR fixes a bug in the MySQL binary protocol TIME decoder (src/sql_jsc/mysql/protocol/DecodeBinaryValue.rs): the 12-byte wire form's microseconds field was parsed but never written to the output string, so TIME(6) values like 02:03:04.5 decoded as 02:03:04. The fix appends a . plus the microseconds zero-padded to 6 digits with trailing zeros stripped, only when microseconds > 0. The test file adds a docker-backed TIME(6) regression test and an offline mock-server suite; to support the latter, the existing NEWDECIMAL mock-server helpers are hoisted to module scope and generalized into a parameterized startMockServer({column, binaryRows, textRows}) / columnDef(...) pair.

Security risks

None. The change is output-formatting only — it writes ASCII digits derived from an already-parsed u32 into a fixed 32-byte stack buffer via write_all on a &mut [u8] cursor, which returns Err (mapped to InvalidBinaryValue) on overflow rather than writing past the end. Worst-case output (-838:59:58.999999, 17 bytes) fits comfortably; even pathological days values from a hostile server can't exceed the buffer because the slice-writer bounds-checks. No new parsing of untrusted input is introduced.

Level of scrutiny

Low-to-medium. The Rust change is ~20 lines of self-contained digit formatting with no allocation, no unsafe, and no change to cursor advancement or wire parsing. The 8-byte / zero-microseconds path is unchanged (Time::from_data leaves microseconds at 0, so the new block is skipped). The test-file changes are larger by line count but are a mechanical refactor (lifting helpers out of one describe to share between two) plus additive test cases; the pre-existing NEWDECIMAL mock test's behavior is preserved.

Other factors

CODEOWNERS does not cover either touched path. The bug-hunting system found no issues. The PR includes verification against a live MariaDB and node mysql2, and robobun confirmed the mock test fails pre-fix and passes post-fix. The trailing-zero strip loop's end > 1 guard correctly keeps at least one fractional digit, and the us % 10 → u8 cast is safe (max 9). I'm comfortable approving without human review.

claude

LGTM — focused fix that appends the previously-dropped fractional seconds to binary TIME decoding, with solid docker-backed and mock-server test coverage.

Extended reasoning...

Overview

This PR fixes a bug in Bun.sql's MySQL binary protocol decoder where TIME(n) values lost their fractional-seconds component. The 21-line Rust change in DecodeBinaryValue.rs appends .uuuuuu (zero-padded, trailing-zeros-stripped) to the formatted string when time.microseconds > 0. The test file adds a docker-backed TIME(6) regression test and refactors the existing NEWDECIMAL mock-server harness into reusable helpers (columnDef, startMockServer) so a second no-docker suite can drive the binary TIME decoder over canned 8- and 12-byte wire frames.

Security risks

None. This is pure output formatting of an already-parsed u32 microseconds field into a fixed 6-byte stack array. The digit conversion (b'0' + (us % 10) as u8) and the trailing-zero strip loop (end > 1) are bounds-safe by construction. Worst-case output (-HHHHHHHHHH:MM:SS.uuuuuu, ~24 bytes even with a hostile days value) still fits the existing 32-byte buffer, and write_all would error rather than overflow anyway. No untrusted input reaches new control flow.

Level of scrutiny

Low-to-moderate. The production change is small, additive, and confined to one match arm of one decoder function; it does not alter cursor advancement, the 8-byte / zero-microseconds path, or any other type's decoding. The test refactor is mechanical (hoisting helpers to module scope, parameterizing the result-set builder) and the pre-existing NEWDECIMAL assertions are preserved unchanged.

Other factors

Time::from_binary (verified in MySQLValue.rs) leaves microseconds at 0 for the 8-byte form, so the new branch is correctly skipped there.
The output format matches node mysql2 and MySQL's documented TIME literal format, per the PR description's MariaDB cross-check.
Test coverage is thorough: 8-byte form, 12-byte with trailing zeros (.5, .0005), full precision, negative sign, >99h, and explicit-zero microseconds — on both binary and text protocols.
No CODEOWNERS cover these paths. The single CI failure (bun-install-security-provider.test.ts on Windows aarch64) is unrelated to this change.
The bug-hunting system found no issues.

Jarred-Sumner

There should be an existing function for some of this? Right?

The fractional-seconds emission duplicated what Postgres__formatTime already does: zero-padded components with the fraction trimmed of trailing zeros. Route the whole TIME arm through that formatter — only the sign stays MySQL-specific since the wire form carries absolute components plus an is_negative byte — and compute the microsecond total in u64 clamped to MySQL's ±838:59:59.999999 cap, which also removes the u32 overflow a malformed day count could trigger in the old hours math.

robobun · 2026-06-03T02:18:25Z

@Jarred-Sumner yes — Postgres__formatTime in SQLClient.cpp already emits exactly this shape (zero-padded components, fractional seconds zero-padded to 6 digits with trailing zeros trimmed). Reworked in c6ac56d: the whole MySQL TIME arm now routes through it instead of hand-rolling the digits — only the sign stays on the Rust side since MySQL TIME is signed (absolute components + an is_negative byte on the wire). The µs total is computed in u64 and clamped to the ±838:59:59.999999 cap, which also removes a u32 overflow the old days * 24 hours math had for malformed packets. Mock + docker tests unchanged and passing.

(The other candidates didn't fit: bun_core::fmt::trimmed_precision is float-based and can't zero-pad the seconds field; shared/datetime_text.rs is parse-only.)

coderabbitai

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

test/js/sql/sql-mysql.test.ts (1)
1237-1521: 🧹 Nitpick | 🔵 Trivial | 🏗️ Heavy lift

Move the offline mock MySQL suites into a standalone sql-mysql-*.test.ts file.

These tests are useful, but sql-mysql.test.ts is the Docker-backed container suite in this repo. Keeping the mock-server coverage in its own non-Docker file preserves the existing execution split and avoids further mixing the two test modes.

Based on learnings: "keep Docker-gated MySQL container tests and non-Docker MySQL tests in separate files... Do not merge mock/non-Docker MySQL tests into sql-mysql.test.ts."
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test/js/sql/sql-mysql.test.ts` around lines 1237 - 1521, Extract the offline
mock-server test code into a new standalone test file (e.g.
sql-mysql-mock.test.ts): move the helper functions (u16le, u24le, u32le, packet,
lenenc, lenencStr, handshakeV10, okPacket, columnDef, startMockServer,
binaryTime and related constants) and the two describe blocks titled "NEWDECIMAL
decodes as a string (mock server, no docker)" and "binary TIME keeps fractional
seconds (mock server, no docker)" into that file, add any missing imports used
by those helpers/tests (net, once, using, SQL, expect/test harness utilities),
and remove these helpers and describe blocks from the original sql-mysql.test.ts
so the Docker-backed suite remains separate. Ensure the moved tests run
standalone by preserving their local references to startMockServer, column,
DECIMAL_VALUE, MYSQL_TYPE_* constants, and the binaryTime helper.

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Outside diff comments:
In `@test/js/sql/sql-mysql.test.ts`:
- Around line 1237-1521: Extract the offline mock-server test code into a new
standalone test file (e.g. sql-mysql-mock.test.ts): move the helper functions
(u16le, u24le, u32le, packet, lenenc, lenencStr, handshakeV10, okPacket,
columnDef, startMockServer, binaryTime and related constants) and the two
describe blocks titled "NEWDECIMAL decodes as a string (mock server, no docker)"
and "binary TIME keeps fractional seconds (mock server, no docker)" into that
file, add any missing imports used by those helpers/tests (net, once, using,
SQL, expect/test harness utilities), and remove these helpers and describe
blocks from the original sql-mysql.test.ts so the Docker-backed suite remains
separate. Ensure the moved tests run standalone by preserving their local
references to startMockServer, column, DECIMAL_VALUE, MYSQL_TYPE_* constants,
and the binaryTime helper.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 42dfe8d4-7ee5-4a80-98af-0e9a89c35052

📥 Commits

Reviewing files that changed from the base of the PR and between 31cae2e and c6ac56d.

📒 Files selected for processing (4)

src/jsc/bindings/SQLClient.cpp
src/sql_jsc/mysql/protocol/DecodeBinaryValue.rs
src/sql_jsc/postgres/DataCell.rs
test/js/sql/sql-mysql.test.ts

Non-docker mock-server MySQL tests live in their own sql-mysql-*.test.ts files in this suite; sql-mysql.test.ts keeps only the docker-backed TIME(6) test and returns to main's state otherwise.

robobun · 2026-06-03T02:28:10Z

Addressed the review note in 8e32b7d: the binary TIME mock coverage now lives in its own test/js/sql/sql-mysql-time-fraction.test.ts (matching the other standalone non-docker sql-mysql-*.test.ts files), and sql-mysql.test.ts is back to main's state plus only the docker-backed TIME(6) test. Verified the standalone test fails against the unfixed decoder and passes with the fix.

claude

I didn't find any issues — the formatter reuse, buffer sizing, and overflow clamping all check out — but the cross-driver coupling (MySQL importing Postgres__formatTime via pub(crate) FFI) and the unconfirmed CI status on the head commit make this worth a maintainer's sign-off rather than an auto-approve.

Extended reasoning...

Overview

Fixes a real bug in Bun.sql's MySQL binary-protocol TIME decoder: the microseconds field was parsed but never written to the output string. The fix routes the MYSQL_TYPE_TIME 8/12-byte arms through the existing C++ Postgres__formatTime helper (already used by the Postgres time/timetz decoders), prepending only the MySQL-specific sign byte on the Rust side. Touches DecodeBinaryValue.rs (the actual fix), DataCell.rs (pub(crate) visibility on the FFI declaration), SQLClient.cpp (comment only), plus a new standalone mock-server test and a docker-backed regression test.

Security risks

None identified. The input is server-supplied wire bytes already length-validated by the existing l @ (8 | 12) match and Time::from_data. The new code computes totals in u64 with saturating_mul/saturating_add and clamps to MySQL's documented ±838:59:59.999999 cap before casting to i64, so a malformed days field can no longer overflow (the old days * 24 was a wrapping u32 multiply). The 33-byte stack buffer is sliced to exactly &mut [u8; 32] for the FFI call; worst-case output (838:59:59.999999 = 16 chars + NUL) fits comfortably, and snprintf is bounded by bufferSize. I confirmed %02d in the C++ formatter correctly widens for hours > 99.

Level of scrutiny

Moderate. This is a focused, well-tested correctness fix in a database result decoder — not security-sensitive, not in a CODEOWNER path, and the blast radius is limited to how TIME(n) columns render. However, it is not a mechanical change: it introduces a deliberate cross-module dependency (MySQL → Postgres formatter), exposes an FFI symbol crate-wide, and does non-trivial buffer-offset arithmetic at an FFI boundary. That design choice was discussed in-thread with Jarred and looks reasonable, but it's the kind of structural coupling a maintainer (cirospaciari is suggested) should explicitly OK.

Other factors

Tests: strong coverage — a docker-backed TIME(6) round-trip plus a no-docker mock-server test exercising the 8-byte form, 12-byte form, trailing-zero stripping, sign, >99h, and zero-µs cases.
CI: the last CI status comment is for c6ac56d (build #60094) and reports musl build-bun failures; the head commit 8e32b7d (test-file-only move) doesn't have a posted result yet. I'd want green CI before merge.
Prior review: no prior claude[bot] review on this PR; the timeline shows active maintainer engagement and feedback already incorporated.

robobun · 2026-08-15T13:37:42Z

Confirming this is still reproducible on current main (Bun 1.4.0-canary eabb96de7, MariaDB 11.8.6). Table (t0 TIME, t3 TIME(3), t6 TIME(6)), same rows read both ways:

prepared: {"t0":"01:02:03","t3":"01:02:03",    "t6":"01:02:03"}
simple:   {"t0":"01:02:03","t3":"01:02:03.456","t6":"01:02:03.456789"}

One data point for the formatting choice. The server's text protocol (what .simple() returns) pads the fraction to the column's fsp rather than trimming it:

simple:   {"t3":"-01:02:03.400","t6":"-100:02:03.000500"}   (inserted '-01:02:03.4' / '-100:02:03.000500')
simple:   {"t3":"00:00:00.000", "t6":"00:00:00.000000"}     (stored zero; binary protocol sends the 0-byte form here)
simple:   {"t3":"838:59:59.999","t6":"838:59:59.999999"}

So with the trailing-zero trimming in this branch, .simple() and prepared queries still disagree for values like .400 or an all-zero TIME(6). The column definition packet carries the fsp (ColumnDefinition41.decimals, already parsed in src/sql/mysql/protocol/ColumnDefinition41.rs, currently pub(crate)); libmariadb's mariadb_time_to_string and go-sql-driver both pad to that value, which makes both protocols return identical strings. The 0-byte and 8-byte wire forms would need the padding too, since the server picks the wire length from the value, not from the column's fsp.

Unrelated to the choice above: #39013 adds a row to the MySQL type table in docs/runtime/sql.mdx saying prepared queries drop fractional seconds for TIME, so whichever of the two lands second needs to adjust that row.

…quests (#45304) ### What does this PR do? It folds 197 open pull requests on `Bun.sql` (PostgreSQL, MySQL, MariaDB, SQLite), `bun:sqlite` and `node:sqlite` into one change: 149 are replaced by it, 18 are closed without porting, 30 stay open. Most PRs were taken as a diagnosis; the files many of them patched were rewritten once. 257 files, +41,670 / -21,600: `src` +12,536 / -16,054, tests +26,903 / -5,100. CI ran the suite on Linux x64 / aarch64, musl, ASAN, macOS and Windows at this head (78e0830cdd): every job that ran passed (178; one macOS aarch64 job could not download the build, two macOS x64 jobs got no machine). Numbers marked (release) compare CI's Linux x64 release binaries of main (f841e3670) and of earlier commits of this PR, in user-space instructions unless a unit is given; the others are from a local debug + ASAN build. ### How did you verify your code works? 102 SQL and SQLite test files on a Linux x64 debug + ASAN build against PostgreSQL, MySQL and MariaDB containers: 5,918 tests pass; 8 time out on a machine at load 210 to 250, each of which passes alone or is known to depend on load. Nearly every new test was run on main first and fails there; the exceptions are guards that pin behaviour we keep. PostgreSQL parameters against main, cell by cell (value, type, mode): of 942, 726 are equal, 137 work that main rejects, 17 reject with another code, 44 are new errors, 18 stored another value (six cases: two are main's again since, four are listed). The option parser against main: 120,000 generated cases, every difference named; TLS is weaker than on main only on a unix socket (3,191), where `url` names another driver than a variable's name (14), or where the adapter's own URL rules and main read another adapter's (21); no file main opens read-only is writable. Option keys that an options object inherits are read where main reads them. On the wire against main, the startup packet and the number of round trips are the same (no parameter, one, jsonb + text + timestamptz, begin / commit; prepared and `prepare: false`); only a redundant Flush is gone. Client CPU (release, 43ad527e44): point select -48 %, 100-row select -30 %, insert -21 %. The pool, `cancel()`, MySQL pipelining and the `bun:sqlite` hooks were also run under randomized stress or differential scripts. ### Behaviour changes to look at first `main -> now`. The rule: a visible change stays only where main is broken (crash, hang, data lost or stored wrong, security hole, against its own docs); elsewhere this PR behaves like main, and the pull request that wanted the change stays open. Every bullet was run on main, on this branch and, where there is one, on the reference clients: postgres.js, pg, mysql2, mariadb, Node 26.3's `node:sqlite`, psql 15 (not better-sqlite3). `ERR_*_X` stands for the `POSTGRES`, `MYSQL`, `SQLITE` codes. **Waiting for the maintainer's word** - **Integers from 2^51 to 2^53 - 1** were `float8` (MySQL: DOUBLE) -> `int8` (BIGINT). PostgreSQL stored 2^52 + 1 as 4503599627370500 in `numeric` and `4.503599627370497e+15` in `text`; now exact. The price: `select ${2 ** 52} as x` returns a string and `/ 3` divides as integers, as from 2^31 to 2^51 already. On MySQL only VARCHAR was wrong; there `* 4096` is a new errno 1690, and mysql2 and mariadb behave like main (#43293). - **An array with an enumerable `raw` is refused.** `Object.assign([text], { raw: [text] })`, also what Babel's and swc's loose mode emit for a tagged template, ran `text` as SQL, as in postgres.js -> `TypeError`; tslib and the strict helpers (non-enumerable `raw`) still run. It closes `ids.raw = []` on a data array pasting `ids[0]` into the SQL, which only the application itself can set. The own-property half of the check closes more: on main a polluted `Array.prototype.raw = []` turns `sql([userData])` into SQL text. The price is every tagged call compiled in loose mode (#38219). - **An untyped `sql.array()` has no cast.** `sql.array(["a", "b"])` without a type was cast to `::JSON[]`, so both examples of main's docs fail -> no cast, the server decides (#41301, #41242). Where it can't tell, main and postgres.js work and this PR doesn't: `select ${sql.array([1, 2])}` is the string `{"1","2"}`, `unnest()` 42725, `array_length()` 42804. Pass the type. - **`prepare: false`:** an object with its own `toString()` bound to `json` / `jsonb` was that string (decimal.js: the number `1.5`) -> its JSON, as in a prepared query on main (#39452). - **bun:sqlite `db.transaction(fn)`, the minimal fix.** When `fn` returns a native promise that is already rejected (an `async` function that throws before its first `await`), main committed -> rolls back and returns the promise. All else is as on main: an `async` callback commits at its first `await`, a `throw` behind it keeps the rows, and the docs now warn. The alternative is to refuse async callbacks outright (#45132), which also refuses what works today: `async` without an `await`, an `await` that comes last, a returned fulfilled promise or thenable. - **bun:sqlite, not changed:** without `safeIntegers` a BigInt parameter above 64 bits keeps its low 64 bits (`2n ** 64n + 5n` binds 5), as on main; a `RangeError` for it (a third of #32841) was in and is reverted. Proposed on this PR: reject below -2^63 or above 2^64 - 1, which still lets `Bun.hash()` output bind but makes a released call throw. **Silent: the same code gives another result** - bun:sqlite `changes` also counted rows written by triggers, cascades, FTS5 / rtree shadow tables -> only the statement's own, like node:sqlite and SQLite's `changes()`. An insert through an INSTEAD OF trigger on a view counted what the trigger wrote -> 0. The SQLite adapter's `count` follows (#43306). - `affectedRows` was `null` on PostgreSQL and SQLite -> rows written, 0 for a `SELECT`, so `r.affectedRows ?? r.count` gives 0 after a `SELECT` (#40434, #40432). MySQL `count` of a write: 0 -> rows affected (#41886). SQLite adapter: `lastInsertRowid` of `INSERT ... RETURNING` `null` -> the rowid; `count` of `WITH ... INSERT` 0 -> 1; `command` is the first keyword: an upsert `UPDATE` -> `INSERT`, but `with ... insert` `INSERT` -> `WITH`. - `uuid[]`, `int4range[]`, `tsvector[]`, `reg*[]`, 35 built-in array types in all, read as the literal `"{...}"` -> arrays, like postgres.js (pg: `uuid[]` only) (#41042, #41039). Columns named `01` and `1` collapsed into one key -> each keeps its name; `row[1]` no longer finds `01` (#45096). - Into `text`: a `Date` was `Tue Jan 02 2024 03:04:05 GMT+0000 (...)` -> ISO 8601; a plain object was `[object Object]`, as in postgres.js -> JSON, as in pg (#41976, #29010). - `null` in `sql.array(values, "TEXT")`, `"INT"` and the rest was the string `"null"` or an error -> SQL NULL (#35111); in a `"JSON"` / `"JSONB"` array it stays the JSON value `null`, as on main. A `Buffer` element there was its hex string -> `JSON.stringify(buffer)`. - PostgreSQL, a late mode on a statement that is already prepared: `.execute().raw()` gives text for `numeric`, `time` and binary for `int2[]`, `float8[]`, the reverse of main; `.raw()` and a late `.values()` of `numeric`, `time` rejects with `ERR_POSTGRES_UNKNOWN_FORMAT_CODE`. All else of `.raw()` is as on main (658 of 666 cells). - `prepare: false` sends what a prepared query sends (one more round trip next to an array, an `Int32Array`, an object with its own `toString()`): `[1]` into `jsonb` was the scalar `1`, an `Int32Array` into `bytea` the bytes of the text `1,2`; an `Array` into `bytea` is now `ERR_INVALID_ARG_TYPE` (#39452). - bun:sqlite `get()` read every row, as documented, so an error in row 2 was thrown -> one step. `SELECT 1 a, 2 b, 3 a`: `{b, a}` -> `{a: 3, b: 2}` (key order). A column named like an index (`"2023"`) now comes first in `Object.keys(row)`, as JS orders keys (#37238). `values()` / `raw()` without columns: `null` -> `[]`. node:sqlite `run()` on a `SELECT` stepped through every row -> one step. All like Node. - SQLite's `MEMSTATUS` is off: `PRAGMA hard_heap_limit` / `soft_heap_limit` are accepted and no longer enforced, as in node:sqlite (#44197, #44187). - PostgreSQL: a Flush after every simple Query disarmed the server's idle timers; after an extended query they already worked -> `idle_in_transaction_session_timeout` / `idle_session_timeout` end Bun's sessions on both paths (25P03, 57P05), `listen()` included (docs: `connection: { idle_session_timeout: 0 }`) (#44784). - After `SET datestyle = 'SQL, DMY'` April 3 read as March 4, as in postgres.js -> any non-ISO style reads Invalid Date, also MDY, which main read right. - TCP keepalive (60 s idle, 10 probes 1 s apart) on every SQL socket, no option, as in postgres.js (pg, mariadb: none): a query silent for 60 s can reject after a 10 s outage, where main resumed (not run). #44983 asked for `listen()` only. - `then()`, `catch()`, `finally()`, `run()` started the query a microtask later -> in the call, like `execute()` (#44105). `begin()` / `reserve()` overtook older plain queries -> one queue, oldest first, like postgres.js (#45275). `sql.file(path)` ran at once -> a lazy `Query`, as typed and as in postgres.js. - A result returned from a `begin()` / `savepoint()` callback came back as a plain `Array` without `count`, `command`, `columns` -> the result itself, like postgres.js; `assert.deepStrictEqual` with an array literal then fails (#45387). - A statement on an ended `tx`: `ERR_*_CONNECTION_CLOSED` -> `ERR_*_INVALID_TRANSACTION_STATE`, because retry helpers rerun on "connection closed" (#43263). - With a URL, `host` / `user` / `pass` beat `hostname` / `username` / `password`; without one they lost -> the long names always win. `?path=A` beat `{ path: B }` (also `unix://A` and `{ socket: B }`) -> the option wins, like every other option. - MySQL / MariaDB ignored `?connect_timeout=`, `?idle_timeout=`, `?max_lifetime=`, `?max=` in the URL -> they set the options (see New). - MySQL / MariaDB: `close()` and the timers send COM_QUIT, like mysql2 and mariadb. "Connection closed" / "failed to bind and execute query" for most client-side codes -> one sentence per code (#38468). An ERR packet the server sends unasked reaches `onclose` as the server's error, not as `ERR_MYSQL_UNEXPECTED_PACKET` (#40913). - Driver errors: `stack` is the header line only (it had two `internal:sql/*` frames); a non-`Error` thrown while a parameter is read is no longer wrapped (#44118, #44204). PostgreSQL statement names in server logs: `P` + 40 bytes of SQL + `$` + id -> `P<id>` (#44042). **Loud: an error where main ran** - `PGUSERNAME` and `USERNAME` are read as the PostgreSQL user, as main's docs promise; main ignored both. `PGUSERNAME` beats `USER`. `USERNAME` counts where `USER` isn't set, as on Windows, where main logged in as `postgres` (not run there) (#26711, #26684). - `mariadb://` without a database asked for one named `mariadb`, which no server has, and couldn't connect (1049) -> selects none, like the mariadb connector; the first unqualified table is a 1046. MySQL keeps `mysql`, as main's docs promise; the types said the user name (#44367, #44373). - MySQL / MariaDB: `sql("db.table")` was one name `` `db.table` `` -> `` `db`.`table` ``, as documented, as on PostgreSQL and SQLite and in mysql2 (mariadb keeps one name). Helper keys (`sql({ "c.d": 7 })`) and aliases split too, so a name with a real dot can't be passed. A name taken from a user can now reach another schema (#24050). - A query created on `reserved` / `tx` / `sp` and first awaited after `release()`, COMMIT, ROLLBACK or RELEASE ran on the pooled connection, as in postgres.js: a `select` worked, an insert that landed in another caller's transaction resolved and kept 0 rows -> rejects unsent. Typical: ``return reserved`...` `` without `await` out of a `using` block (#43261, #43263). - `sp === tx` in `tx.savepoint(sp => ...)` -> `sp` is its own handle, like postgres.js, and ends with its savepoint; on main and in postgres.js it kept running in the transaction (#43263). - PostgreSQL: an array or object bound to `bool` / `int4` / `float8` / `timestamp` was coerced in the client (`[]`: `true`, 0, 0, 2000-01-01), so `delete ... where ${[]}` removed every row -> sent as text, which the server rejects (22P02, 22007) unless it is valid: `[5]` into `int4` is still 5, and `['2024-01-02']` into `timestamptz`, 2000-01-01 on main, is that date (#41976, #41970). - An Invalid Date was stored as 1970-01-01 on PostgreSQL (#41976) and MySQL -> rejected. MySQL: a function, or an object whose `toJSON()` returns undefined, was bound as `""` (MariaDB: errno 1210) -> `TypeError`, like mysql2. - bun:sqlite: `stmt.run(new Date())`, a Map, a Promise as the only argument bound NULL, as in node:sqlite; in any other position main throws already -> `TypeError` everywhere. An `ArrayBuffer` alone bound NULL -> BLOB. - `channelBinding: "require"` / `?channel_binding=require` on MySQL, MariaDB, SQLite was an unknown key and ignored -> `TypeError`. - A `path` that doesn't exist was dropped and TCP dialed -> `ERR_*_CONNECTION_REFUSED` (postgres.js, pg: ENOENT) (#41570). - `tls.checkServerIdentity` was never called -> called, and it raises an `sslmode` below `verify-ca` to `verify-full`, so a self-signed certificate without `ca` fails, as in pg and postgres.js (#42054, #41856). - SQLite adapter: what `onconnect` / `onclose` throw was swallowed -> uncaught exception, as on the other adapters. `?mode=RO`, `?mode=readonly`, `?mode=`, `{ readOnly: true }` opened read-write -> throw (#45153, #45079). - The guard against `BEGIN` on a pooled client also catches `START\tTRANSACTION` and `/* c */ BEGIN`, and no longer trips on `begin_work()` or copies the query (#41758, #41740). - A nested `sql.unsafe(text, values)` behind other parameters read the wrong parameter on PostgreSQL, as in postgres.js -> `SyntaxError` (#38220). - A simple `FETCH` from a binary PostgreSQL cursor decoded `numeric`, `time` and read `int2` / `oid` as 0, `int8` / `uuid` as garbage -> these reject with `ERR_POSTGRES_UNKNOWN_FORMAT_CODE`. - `COPY ... TO STDOUT` without a stream resolved `[]`, `COPY ... FROM STDIN` hung and wedged the connection -> both reject with `ERR_POSTGRES_COPY_NOT_SUPPORTED` (#45332). **Main was broken; code that leaned on it will notice** - A pooled connection that came back inside an open or failed transaction (a reservation after `BEGIN`, `sql.unsafe("select 1; begin")`) went to the next caller, who got 25P02 or lost its writes -> the drivers keep the server's transaction status and the pool closes it. For that status to be current, a failed PostgreSQL query rejects at its ReadyForQuery, like libpq, not at the ErrorResponse. Exempt: plain queries at `max: 1`, where the docs allow `BEGIN` by hand, and a prepared XA branch. - `idleTimeout` / `maxLifetime` closed connections under a running query, a reservation or a `begin()` transaction -> they never interrupt those, as in postgres.js; past `maxLifetime` a connection takes nothing new, finishes, closes (#30648, #30646, #25405). - What the caller's own `catch(f)`, `then(_, f)` or `finally(f)` handler throws was swallowed -> an unhandled rejection. MySQL / MariaDB: `begin()` never settled when a statement on `tx` was only started and failed -> settles. (94 cells against main: these 32 differ.) - Callers waiting for a connection were rejected with the close error of somebody else's connection -> the pool dials for them (#44938). A client-side auth or TLS error before the first connection was cached until restart -> every query dials. - A second `close()` resolved while a session was still open -> all calls wait for one close, the shortest timeout wins (#43959). `close({ timeout: 0 })` waited for running queries -> closes at once, as documented (#33740, #32038). - `tx.close()` without `await`: `begin()` resolved with nothing stored, or committed -> one ROLLBACK, no COMMIT, `begin()` rejects (#44799). - Two `reserved.begin()` at once shared one server transaction -> the second waits. SQLite: a second `sql.begin()` failed with "cannot start a transaction within a transaction" -> it waits. Side effect: the same call awaited inside its own callback never settles, like `max: 1` on a server; use `tx.savepoint()`. - `where id in ${sql([])}` was a syntax error on the servers -> `in (NULL)`, no row, like postgres.js. SQLite keeps `in ()` / `not in ()`. - Statement caches were unbounded -> a connection keeps the statements it used last, 1,024 on PostgreSQL and 256 on MySQL / MariaDB (there all connections share the server's `max_prepared_stmt_count`, 16,382 by default), and sends Close / COM_STMT_CLOSE for the rest (#28980, #33190). MySQL: a failed prepare was replayed, even after `CREATE TABLE` -> every use prepares again (#33189, #42720). - A SCRAM password that SASLprep changes costs a second login (see New), so a wrong one counts as two failed logins per query (main: `max`) and a right one as at most one per `SQL` instance: the rest of the pool waits for the first login, which fills the pool one login later for such a password only. A `password` function that returns a promise is not held: `max` each. Other passwords, md5 and cleartext roles: one login, as before (#41605). - `$PASSWORD` was sent as the database password -> not read. `postgres:///x` was a socket if `/x` existed -> always database `x` (#41570). - `sql("a\0b")` returned a rejected promise no `try` could catch -> throws at the call (#45160). A `resolve` / `reject` overwritten on a query with a function that throws hung `close()` -> uncaught exception; under an `uncaughtException` listener it still hangs. - bun:test: `expect(p).resolves` / `.rejects` / `expect(() => p).toThrow()` never called `then()`, so a lazy `Query` hung `bun test` forever -> they call it once while `p` is pending, like Jest (#19130). - MySQL / MariaDB: `lastInsertRowid` above 2^53 - 1 was rounded -> exact, a string, or a BigInt with `bigint: true` (#44646). Commands are pipelined (see New). A command of 16 MB or more failed in the client (`ERR_MYSQL_OVERFLOW`, session kept) -> it is sent; over `max_allowed_packet` the query gets the server's 1153 and the connection fails, because the server hangs up. A 0xFB reply (a request for a local file) hung the query -> fails the connection, no file is read. - MySQL / MariaDB: prepared `TIME(n)` lost its fraction -> `"12:34:56.789"`, padded to the column's precision like `.simple()` (mysql2 and mariadb don't pad) (#31738). `.raw()` on a prepared query gave a string for TIME, a `Date` for DATE, DATETIME, TIMESTAMP -> a `Buffer`, as documented. `caching_sha2_password` over a unix socket couldn't log in -> the password goes in clear, as the server expects there and as mysql2 and mariadb do (#37819). - bun:sqlite: `{ readonly: true, create: true }` opened read-write -> `TypeError` (#41815). `db.run("a; b; c")` / `exec()` swallowed a runtime error of any statement but the last -> throws there, the rest doesn't run (#37418, #37415, #41010). Running a `Statement` again inside its own `iterate()` loop restarted the loop forever -> the newest run wins, the older iterator throws "This iterator was invalidated" (#32917). Both like node:sqlite. - node:sqlite: `stmt.get()` on `INSERT ... RETURNING` returned a row although the commit failed (deferred foreign key) -> throws. Node 26.3 has the bug. Undo: 6 lines in `NodeSqlite.cpp`. **Costs** - The first `new Bun.SQL()` of a process: 23.3 M -> 25.1 M (+7.9 %); the first bun:sqlite database +4.6 % (release). - bun:sqlite `prepare` + `finalize`: 21,739 -> 23,284 (+7.1 %, release), because statements report their native memory to the GC: 20,000 dropped 200-parameter statements 465 -> 27 MB. - More distinct texts than the statement cache holds are prepared again and again. PostgreSQL, 24,000 queries over N texts at `max: 4` (release, limit still 256): N = 1,000: 2.0 -> 4.7 s, client CPU 470 -> 880 ms; N = 300: 1.4 -> 1.6 s. With 1,024 (debug builds) 1,000 texts cost what 300 cost; the step lies between 1,000 and 1,100. A full cache costs the server about 41 MB per connection (41 KB per statement with a join and ten columns); main has no bound. - A bulk insert with an `undefined` cell on PostgreSQL: +2.1 % (see Memory and performance). ### New **`query.cancel()` works.** It was typed and documented but did nothing. A query that isn't written yet leaves the queue, is never sent and rejects with `ERR_*_QUERY_CANCELLED`. On PostgreSQL a running query with nothing else written behind it is stopped by a CancelRequest on a second connection, outside `max` (same peer, same TLS settings and certificate check, at most 5 s) and rejects with the server's 57014. The session writes nothing until the server has closed that connection, so a late cancel can't hit the next caller's query. There `reserved.close()` / `tx.close()` also stop the running query (#33370, #44625). **COPY streams (PostgreSQL).** `query.readable()` is a `ReadableStream<Uint8Array>` for `COPY ... TO STDOUT`, `query.writable()` a `WritableStream` for `COPY ... FROM STDIN`. Bytes only (text, CSV, binary), no row encoder. Backpressure both ways. `close()` / `pipeTo()` resolve when the server has stored the rows and reject with its error; `abort()` sends CopyFail, and so does the end of the transaction, savepoint or reservation that a stream is open on: `begin()` then rejects with that error and rolls back. The query still resolves with `count`. They are web streams, not Node streams, and a `COPY` without parameters always goes out as a simple query (#23350, #15088). **`result.columns`, `result.statement`.** PostgreSQL: `[{ name, type, table, number }]` (OIDs, like postgres.js); MySQL / MariaDB: `[{ name, type, table, length, flags }]`. `statement` is `{ string, types, columns }`. Both are built on the first read, prepared or not. The runs of a prepared statement share one object, which isn't frozen. On SQLite both stay `null`: capturing them cost a 1-row query +17 % to +34 % (main's release build) (#30037, #26809, #18866). **Column name transforms (all adapters).** `new SQL({ transform: SQL.camel })` reads column `user_id` as `userId` and writes `user_id` for `sql({ userId })`, `sql("userId")`. Presets `SQL.camel`, `pascal`, `kebab`; one direction with `SQL.toCamel`, `SQL.fromCamel` and so on; or `{ column: fn | { from, to }, undefined: value }`. Names, directions and casing are postgres.js's. With `undefined: value` (`null` counts) a helper leaves no column out, like postgres.js. No `value` / `row` transform, so keys inside json aren't renamed (#36451, #15087). **`sql.json(value)` (all adapters)** stores any value as JSON: a number, a boolean, `null`, an object or array on SQLite, and a JSON *string* where Bun sends a string as it is (MySQL, MariaDB, SQLite, PostgreSQL with `prepare: false`). On PostgreSQL it returns a small wrapper, elsewhere `JSON.stringify(value)`. **SCRAM-SHA-256-PLUS, `channelBinding: "disable" | "prefer" | "require"`**, also `?channel_binding=` and `PGCHANNELBINDING`. `tls-server-end-point`, hash as in RFC 5929, RSASSA-PSS included, checked against libpq 15. `require` fails with `ERR_POSTGRES_CHANNEL_BINDING_REQUIRED` before anything derived from the password is sent. Off (`disable`) unless asked for, on the maintainer's decision: the login on the wire is main's. With `prefer`, psql's default, a login fails behind a proxy that terminates TLS but forwards the server's `-PLUS` offer (28000) and with an Ed25519 certificate (#41703). **SASLprep, by retry.** A non-ASCII SCRAM password typed in another Unicode form than it was set in (NFD, soft hyphen) got 28P01 while psql logged in -> the first login uses the password as typed, like main; on 28P01 from a SCRAM role, if SASLprep (two `replace` and `normalize("NFKC")`, no Unicode tables) changes the string, Bun logs in once more with that form. The pool remembers which form worked; the LISTEN connection does the same (#41605). **`tls.checkServerIdentity(hostname, cert)` (PostgreSQL, MySQL, MariaDB).** Called once per connection, after the chain verified and before anything is sent inside TLS; replaces the built-in name check. A returned object or a throw fails the connection with that value; a Promise is `ERR_INVALID_RETURN_VALUE`. Not called next to `rejectUnauthorized: false` (SQL half of #42054, #41856). **Re-prepare after DDL (PostgreSQL).** After `ALTER TABLE`, `DEALLOCATE ALL` or `DISCARD ALL` every run of a cached statement failed forever (0A000, 26000) -> it is prepared again under a new name and the old one closed. The query is re-sent only when nothing is written behind it (a re-send would reorder writes) and outside a transaction block; otherwise it rejects with the server's error and the next run works. 20 concurrent runs after `ADD COLUMN`: main rejects 20 and every later run; now 19 at `max: 1`, 15 at `max: 5`, then none (#35120, #29484). **`bun --hot` keeps database connections (PostgreSQL, MySQL).** Every save opened up to `max` new connections and never closed the old ones (2 4 6 8 10 12 sessions at `max: 2`) -> a `new SQL()` with equal settings takes over the previous load's idle connections when it would dial. Settings are compared by value; a function counts as changed. A pool of an earlier load closes only when nobody asked it for a connection during a whole load: a replaced pool lives one reload longer, a pool kept on `globalThis` and used is never touched. Session state (`SET`, temporary tables) survives, `onconnect` runs again for a reused connection. A module-level `sql.listen()` shares one LISTEN connection across loads (#25623, #23215). **MySQL / MariaDB pipelining.** `.simple()` queries and queries whose statement is already prepared go out without waiting for earlier replies; an error fails only its own query. Nothing is written behind a text with a `;` until it has ended, and a packet out of sequence fails the connection. Main's docs promise this; it sent one command at a time. Undo: `BUN_FEATURE_FLAG_DISABLE_SQL_AUTO_PIPELINING=1`. 4 executes through a proxy that holds each reply 30 ms took 124 ms on main, now 31 ms on both servers, 1 write instead of 4 (release) (#44022). **MySQL: `sha256_password`** was documented, but every login failed with errno 1045. **Arrays as PostgreSQL array parameters.** `where id = any(${[1, 2]})` (08P01) and `sql({ tags: ["a"] })` into a `text[]` failed -> a JS array or an `Int32Array` bound to a built-in array type goes as an array literal, with NULLs and nesting, also with `prepare: false`. The other typed arrays stay `bytea`; `sql.array(new Float32Array([1.5]))`, which main rejects, puts them into an array column. Into `text`, an untyped position or a user-defined type an array stays `String(value)` (`a,b`), as on main and in postgres.js (pg: `{"a","b"}`), so an array of an enum needs `sql.array(values, "mood")`. A plain array into `json[]` / `jsonb[]` follows `sql.array(values, "JSONB")`: nested arrays are dimensions, `null` is the JSON value, `undefined` is SQL NULL, `sql.json([1, 2])` is one element (that half of #33579, #29551). **MySQL / MariaDB `foundRows: true`**, also `?foundRows=true`: `CLIENT_FOUND_ROWS`, so `affectedRows` counts rows matched, not rows changed, the default of mysql2 and mariadb. Off by default, as on main (#30848, #30843). **libpq URL keys and variables.** `?connect_timeout=`, `?sslrootcert=` ... went to the server, which refused the connection, and `?password=` went out in the startup packet. Now read: `host`, `port`, `user`, `password`, `dbname`, `connect_timeout`, `idle_timeout`, `max_lifetime`, `max`, `prepare`, `channel_binding`, `sslrootcert`, `sslcert`, `sslkey`, `sslpassword`. Keys that only tune libpq (`keepalives*`, `target_session_attrs` ...) are dropped. Keys that restrict the connection (`require_auth`, `sslcrl` ...) still go to the server, which refuses them. New variables: `PGUSERNAME`, `USERNAME`, `PGAPPNAME`, `PGOPTIONS`, `PGCONNECT_TIMEOUT`, `PGCHANNELBINDING`. **bun:sqlite: `db.function()`, `db.aggregate()`**, better-sqlite3's signatures (options `varargs`, `deterministic`, `directOnly`, `safeIntegers`); `inverse` makes a window function. What a callback throws comes out of `get()` / `all()` / `run()` unchanged. A BigInt result outside 64 bits follows the function's `safeIntegers`, like a parameter (#8558, #25703, #1474, #5051). **bun:sqlite: `db.onInsert`, `db.onUpdate`, `db.onDelete`** `= function (database, tableName, rowid, schemaName)`; `null` removes it. What a hook throws is an uncaught exception at once, and the statement goes on. While a function, aggregate or hook runs, every other call on that `Database`, `close()` included, throws "This database connection is busy executing a query". No commit hook (#12631, #4175). **Small:** `db[Symbol.for("sqlite-type")]` (#22109), `stmt.reader`, `stmt.readonly` (#16469), `stmt.declaredTypes` before the first run. ### Fixed What is listed above isn't repeated. A line without `->` names a symptom that is gone. #### Pool and lifecycle - A pool that lost connections dialed nothing while one was ready (it stayed at 4 of 5 for ever and ran 3 parallel queries on 1 backend) -> it grows back, one dial per waiting caller that no dial in flight covers. - `execute()` / `begin()` inside `onclose` got the dead connection ("connection must be a PostgresSQLConnection") -> it has left the pool and the waiting callers are redialed for before `onclose` runs (#43958, #44938, #45039). - A connection the server killed in the read that finished the handshake got `onclose`, then `onconnect`, and stayed in the pool: every later query failed, `close()` hung -> `onconnect` never comes after `onclose`, both drivers (#40913, #45039, #42804). - `close({ timeout: 2 ** 31 })` overflowed `setTimeout` and killed the queries after 1 ms -> clamped (#32100, #32096). `close()`, then `close({ timeout: 0 })`, left a held reservation running (#43959). - A replaced `Promise.prototype.finally` could wedge the pool -> a private promise; the connection is released before the caller's reactions run (#44315). - `q.cancel(); await q` never settled -> rejects (#44799). - The connect-retry budget read `Date.now()`: endless under `setSystemTime()`, short on a clock step -> monotonic clock (#37960). - A `password` function that throws `undefined` left the query pending forever. `` sql`a; b`.values().simple() `` ignored `simple()`. `query.active` was always false. - PostgreSQL: a FATAL error for the running query left the connection in the pool until the FIN, and a session ended while idle reached `onclose` as `ERR_POSTGRES_EXPECTED_REQUEST` -> fails at once with the server's error, e.g. 57P01 (#40913). - `onconnect` / `onclose` never fired for the `sql.listen()` connection (#41051, #41050). - `bun --sql-preconnect` with a refused connection ran the script, then exited 1, against the docs -> exit 0. #### Handles, transactions, savepoints - `sql.begin(tx => Promise.all([...]))` with one failing task: the other tasks' later statements ran after ROLLBACK in autocommit, so rows of a rejected `begin()` or `savepoint()` stayed -> they reject, on all four databases (#43263). - `tx.unsafe()`, `tx.file()`, `reserved.unsafe()`, `reserved.file()`, `notify()`, `commitDistributed()` ignored the handle's state and ran after its end (#43249, #43261). `` tx`...`.values() `` on an ended handle threw `.values is not a function` -> a `Query` that rejects when run (#43257). - `tx.close({ timeout })` whose queries finished in time committed -> rolls back (#44799, #32148); `reserved.close({ timeout })` then lost the pool slot. `close({ timeout: -1 })` left the handle unusable (#32100). A handled failing query was reported again as unhandled. - `close()` or a lost connection started or rejected identifiers, fragments and unstarted queries, and a handle retained them all -> a query is tracked from start to settle (#44932). - `tx.close()` and the end of the callback could both end the transaction -> one end. Now that `cancel()` works, `close()` spares Bun's own BEGIN / COMMIT / SAVEPOINT (#44799, #44625). - Connection lost under a `begin()` whose callback doesn't return: later `begin()`, `reserve()`, `close()` hung -> the slot goes back at the loss (#43205). `reserved.unsafe()` then ran on the pool's next connection (#45039). - `reserved.release()` during a running `reserved.begin()`: the nested COMMIT committed the next caller's work -> the connection goes back after the transaction (#43255). - `tx.savepoint(sp => [q1, q2])` sent RELEASE before the queries (#43263). `tx.savepoint("audit", fn)` sent `SAVEPOINT s0` -> `s0_audit` (#45111). A failed ROLLBACK lost the callback's error -> it is the `cause` (#41817). - The `begin()` callback saw no `AsyncLocalStorage` store on a cold pool, another request's on a busy one -> its caller's (#43652). - A `reserve()` handle dropped without `release()` kept its connection forever -> released when collected (#35117). - `sql instanceof Bun.SQL` threw -> true for every handle; `class X extends Bun.SQL {}` threw at the definition -> `new X()` throws "Bun.SQL cannot be subclassed" (#42637). `new Bun.sql(url, { tls: true })` dropped the options and connected in plaintext (#45118). - SQLite: a failed COMMIT left the transaction open and every later `begin()` failed -> rolled back. After `INSERT OR ROLLBACK`, `RAISE(ROLLBACK)`, `SQLITE_FULL` the rest of the callback ran in autocommit -> nothing more runs (#45065). `close()` didn't reject a running `begin()` (#43116). #### Options, URLs, environment - `{ filename: "./dev.db" }` without `adapter` dialed PostgreSQL on host `.`; `SQLITE_URL=x.db` opened `postgres://x.db` (#44483). A server's URL in `filename`, or `hostname`, `port` ... next to it, still make it a connection string. `{ adapter: "sqlite", url }` opened `:memory:` (that half of #38427). - `{ adapter: "sqlite" }` next to a server `DATABASE_URL` used it as the file name -> `:memory:`. `{ adapter: "mysql" }` without a host next to `DATABASE_URL=postgres://...` dialed that host and timed out -> `MYSQL_URL`, else defaults, and then with the TLS level the passed-over URL asks for, as on main (`tls: false` or `ssl: "..."` replaces it); such a URL without a host fills in as on main (#45080). - `hostname`, `host`, `path`, `socket` or five explicit PostgreSQL fields next to `DATABASE_URL=:memory:` or another SQLite URL opened SQLite -> the server. `{ url: "postgres://..." }` next to `MYSQL_URL` ran the MySQL driver on it and timed out -> the scheme of `url` wins over the name of a variable (those parts of #41586). - SQLite: `?mode=rw` created the file; `?mode=rw&mode=ro`, a `mode=ro` next to a `#` (`?mode=ro#x`, `?a=b#mode=ro`), `?mode=ro` + `{ readonly: undefined }`, `{ readwrite: false }` opened read-write -> read-only (#38417, #45079, #45153). `bigint: true` did nothing -> `safeIntegers` (#41816). - `socket:` and `?socket=`, both documented, did nothing -> aliases of `path`. `mysql://user:pass@/db?socket=...` threw `Invalid URL` -> URLs without a host parse (#37807, #23954). - A host that starts with `/` (`hostname`, `PGHOST`, `?host=`, `%2F...`) went to DNS -> socket directory. `sslmode=require` on a PostgreSQL unix socket failed -> ignored there, like libpq. - PostgreSQL: a `path` that is a directory was dialed as it is -> `<directory>/.s.PGSQL.<port>`. The pathname of `unix:///run/a.sock` was also the database `run/a.sock` -> the default database. `SQLITE://` / `FILE:` in `filename` or in a variable were `Invalid URL` or PostgreSQL -> SQLite, as in the string form. A NUL in `path` was dropped and TCP dialed -> rejected. - `console.log(sql.options)` printed the password, `tls.key`, `tls.passphrase` -> `[REDACTED]` (#41725). `sql.options.sslMode = "require"` meant `disable` -> the next query rejects (#43818). #### Query building - ``where ${[sql`a`, sql`or b`]}`` bound one parameter and matched every row on PostgreSQL -> the fragments are joined. ``sql({ ts: sql`now()` })``, `sql({ n: sql.array(a, "INT4") })` failed -> work. - The MySQL update helper knew one spelling of `ON DUPLICATE KEY UPDATE` -> any (#32040, #32035). Keywords inside strings and comments confused helpers. Each helper copied and lower-cased the whole text so far (#43789). - `Buffer` / `Uint8Array` elements in `sql.array` stored garbage -> `\x` hex; a typed array stored `{NaN,NaN}` -> its numbers (#36241). - `sql({ "-1": 1 })` threw. An identifier used in another client's query was quoted for the wrong dialect (#24050). `sql.unsafe(text, values)` read the caller's array when the query ran. - `globalThis.Array = ...` before the first use of `Bun.SQL` threw "The superclass is not a constructor" (#32558). #### PostgreSQL protocol, types, errors - Every pattern of NULLs in a statement's parameters prepared another statement, and none was ever closed, so backends grew to several GB -> the cache key is the text plus the declared types, NULL fits any (#28980). - A statement name could end inside a multi-byte character, and two texts with the same first 40 bytes could share one (42P05) (#44042). A text that ends in NUL put the stream out of step; extended batches sent a Flush before Sync (#44784). - An `int4[]` / `float4[]` with a NULL or more than one dimension was rejected on a binary path (`NULLS_IN_ARRAY_NOT_SUPPORTED_YET`, `MULTIDIMENSIONAL_ARRAY_NOT_SUPPORTED_YET`) -> a plain `Array`, `null` for NULL, `float4` elements widened. The elements decide, not the header's has-nulls flag, which the server leaves set after the last NULL is gone (those parts of #45211, #45062). - A `numeric` of 131,069+ digits read `""` or `"-"` when sent in binary -> to decode, `numeric` and `time` are asked for in text and their binary decoders are deleted (#45014); `int2[]` and `float8[]` are asked for in binary. The values are the same. - Text arrays had a parser with its own number, JSON, date and bytea decoders -> a tokenizer that hands each element to the scalar decoder. Fixed by it: explicit bounds, `'[0:2]={1,2,3}'`, `indkey::int2[]` (#43335, #43332); `1e-07`, `1e+20` in float arrays, `12345678901`, `1e5`, `[null]` in `json[]`, `-0` (#43336, #45305); `json[]` nested over 100 deep. - BC dates were Invalid Date, dates past year 9999 were read in local time on the text paths -> one parser, same instant under every `TZ`. - 65,536 parameters, and on MySQL the second run of a failed prepare or a wrong parameter count, rejected with a plain object (no `stack`, not `instanceof Error`) (#44118, #43992, #43993). A failing query stayed pending forever when `Error.prepareStackTrace` threw -> `stack` is made on the first read (#44204). - `PostgresError.column` / `.line` held a position in Bun's own source when the server sent none -> absent (#45151, #45064). `detail`, `hint`, `constraint` ... were UTF-16 bytes read as text when non-ASCII. `severity` was translated on a server with another `lc_messages`. - The Bind format code came from the list of types Bun can decode, so `numeric`, `float4`, `time` were announced binary and sent as text (08P01) -> binary only where value and type match (`bool`, `int4`, `float8`, `bytea`, `timestamp(tz)`); the rest is text (#41976). - `Date` into `date` / `time` failed (#41976, #29010). `prepare: false` sent a `Date` as `toString()` and an object as `[object Object]` (#39452, #29010, #30221, #39450). decimal.js / Temporal values into `numeric`, `time` failed; `URL`, `DataView` threw; a bigint outside int64 failed -> `numeric`. - A CopyBothResponse hung the query -> "Streaming replication is not supported". - Ten codes that nothing can produce any more are gone: six `ERR_POSTGRES_*` (`*_NOT_SUPPORTED_YET`, `UNSUPPORTED_NUMERIC_FORMAT`, `INVALID_QUERY_BINDING`, `OVERFLOW`, `INVALID_BYTE_SEQUENCE_FOR_ENCODING`), four `ERR_MYSQL_*` (`INVALID_QUERY_BINDING`, `*LOCAL_INFILE*`, `PASSWORD_REQUIRED`). #### PostgreSQL authentication and TLS - `?channel_binding=` was forwarded as a startup parameter (42704). A server that offers no known mechanism got a SCRAM-SHA-256 answer anyway -> `ERR_POSTGRES_SASL_NO_KNOWN_MECHANISM`. A mandatory extension (`m=`) is rejected before the proof (#41703). - A SCRAM iteration count out of bounds failed with `ERR_POSTGRES_INVALID_MESSAGE` "Failed to read data" -> `ERR_POSTGRES_SASL_ITERATION_COUNT_OUT_OF_RANGE` with the count and the bounds (the error of #38133). A GSSAPI / SSPI request showed an unfilled format string -> "GSSAPI authentication is not supported". - Over TLS the StartupMessage (user, database) was queued when the handshake started -> written once chain and identity are accepted. - A TLS 1.2 server that renegotiates reports a second handshake: both drivers sent their login again, and the next query got the reply to it -> ignored after the first. #### MySQL / MariaDB - Two queries on one connection could resolve with each other's rows when a parameter's `toJSON` started a query (#32005). `BUN_FEATURE_FLAG_DISABLE_SQL_AUTO_PIPELINING=1` hung every prepared query (#44022). - One INSERT with K nullable columns was up to 2^K server statements, and statements piled up until the server ran out of memory (a production report is on #33190) (#33190, #28980). - `caching_sha2_password`: the scramble hashed 21 bytes (nonce + NUL), so fast authentication always failed: plain TCP needed `allowPublicKeyRetrieval: true` at every login, not just the first; a password of 20+ bytes failed with errno 1045 -> 20-byte nonce (#33504, #28161, #26195). - A length that points past its packet's end hung the query or decoded the next packet as data (a phantom row) -> `ERR_MYSQL_MALFORMED_PACKET` (#35118). - A `Date` before 1970 was rejected -> years 0 to 9999 (#31209). Prepared `TIMEDIFF()`, `SEC_TO_TIME(3661.25)` lost their fraction (#31738). MariaDB: every object or array parameter failed with errno 1210 (#37125). `DataView` was rejected. - An idle connection that got a query didn't ref the event loop: exit 0 with the query in flight, or a spinning core under top-level await (#38163, #27102). A short `maxLifetime` with a slow handshake rejected the first query. - Every `close()` counted as an aborted client on the server (`Aborted_clients` +3 after 3 closes -> +0). #### SQLite adapter - A valid `SELECT` resolved `[]` (`'%"%'` in the text, `/*c*/select`, `select*from`, `values (...)`), and `INSERT ... SELECT`, `WITH ... INSERT` reported `count` 0 -> the adapter asks SQLite whether the statement returns columns instead of tokenizing (#33583, #30811). - A text whose first statement returns rows (`sql.unsafe("SELECT 1; INSERT ...")`, a migration file that starts with a `PRAGMA`) ran only that statement -> every statement runs; the result is the first statement's rows. Only the first statement can have parameters: a later one that has any is refused, where main bound NULL or skipped it. - Named parameters on a nested `sql.unsafe()` were dropped and NULL was bound: `delete ... where name is not ${sql.unsafe("$o", { $o: "bob" })}` emptied the table -> `SyntaxError`. `sql.unsafe("... = ?", 0)` (also `""`, `false`, `0n`) bound NULL -> binds the value. PostgreSQL: a fragment with a second argument that is no array, in front of a parameter, sent `$NaN` -> works (those parts of #44745, #44867). - More than about 640,000 rows rejected with `RangeError: Maximum call stack size exceeded` (#38438). `sql.flush()` threw -> no-op (#41816). - Through the adapter too: multi-statement errors (#37418), `byteOffset` (#45229); a NUL in a file name throws from `new SQL()` (#37006). #### bun:sqlite - `{}`, `{ create: false }`, `{ readonly: false }`, `{ readwrite: false }` failed with `SQLITE_MISUSE`, `{ create: false, strict: true }` created the file -> the mode comes from `readonly`, `create`, `readwrite` only (#44908, #32422, #9410, #15876). - `SQLiteError.byteOffset` counted from the failing statement -> from the start of the text (#45229). - A kept statement returned rows with the old keys, or values under wrong keys, after `ALTER TABLE` (#32841, #1332); `stmt.columnNames` / `declaredTypes` still update at the next run, as in Node. `SELECT 1`: `row["1"]` was `undefined` (#37238, #18326). - `{ name: 1 }` for `$name` bound NULL (default), `{ $name: 1 }` threw (strict) -> both work (#13409, #18456). Strict `$1` read key `0` (#37240). - UTF-16 strings: a lone surrogate read back as three U+FFFD, a leading U+FEFF was dropped, a leading U+FFFE byte-swapped the rest (#32841). A whole number from 2^51 on was stored as REAL (#43281). `db.prepare(sql, "x")` ignored the binding (#25472, #45367); a lone value is still dropped unless the statement has one parameter. A bind that threw in `prepare()` kept the file open after `close()` until a GC. - `break` out of `iterate()` left the cursor open (`SQLITE_LOCKED`, "database is locked") -> reset (#32917). - A NUL byte in a file name, extension path or schema name cut the string short -> throws (#37006, part of #38514). `Database.setCustomSQLite()` returned `true` where it does nothing (#36371). - A bind failure was a plain `Error` -> `SQLiteError`. SQL text over 1e9 bytes failed as a whole -> the limit is per statement. System-library builds asked `dlopen` for `sqlite3` -> `libsqlite3.so.0` (#26629). #### node:sqlite - `db.close()` / `using db` kept the file and `-wal` / `-shm` open until every `StatementSync` was collected (`EBUSY` on Windows) -> `close()` finalizes the connection's statements and releases the file at once, like Node (#40005, #41890, #40001). - `stmt.get()` built a new Structure per call (#44103, #44084). A `Proxy` (Vue `reactive()`, MobX), a module namespace or a `String` object as named parameters bound NULL to everything. `applyChangeset()` errors had an `errstr` Node's don't have (#34453). #### Crashes and memory safety - PostgreSQL: a query started from a parameter's `toString()` / `toJSON()` on the same connection panicked (`@intCast`) with `prepare: false`, or gave 08P01 plus two queries that never settle -> parameters are converted first, writing Bind runs no JS, and the nested query runs behind the outer one (#43918). - `sql.options.idleTimeout = -1` (or `"abc"`, `{}`) panicked at the next connection, both drivers (#43818). `run()` on the private PostgreSQL query handle with a missing argument panicked (SQL half of #44897). - MySQL: a later parameter's `toJSON` that shrank an earlier resizable `ArrayBuffer` segfaulted -> bytes are read after all user code ran (MySQL half of #38289, #38277). - bun:sqlite: `db.fileControl(op, null)` segfaulted, `(op)` threw -> storage Bun owns (#44089, covers #36301). A `Database` could be closed under its own running `serialize()` (#41611). `RAISE(ABORT, '')` hit a debug assertion (part of #43797). - bun:sqlite, node:sqlite: a 2^30-character string aborted the process -> `RangeError` (parts of #42868). #### Memory and performance - GC cells per awaited `select 1` (release): PostgreSQL 84 -> 28, MySQL 76 -> 27, SQLite 52 -> 33 (whole PR; #44105, #44315). Per cached PostgreSQL query: 47,770 -> 23,291 instructions (-51 %). - A bulk insert of 1,000 x 10, end to end (release, 43ad527e44, the union of keys still in): PostgreSQL -38.0 %, MySQL -52.6 %; with one `undefined` cell +2.1 % and -15.7 %. - The PostgreSQL read buffer was never compacted while reads ended inside a message (rows larger than a socket read): a 1,000 MiB COPY stream grew RSS by 1,235 MB without the fix. Now (release): 255 MiB/s, peak RSS 59 MiB. - bun:sqlite (release): `all()` per row, `get` by key and an insert of 3 ints cost 4,613, 8,375 and 4,489 on main, now 3,225, 6,502 and 3,381; 29 of 31 cases are cheaper. Rows of 63+ columns made one Structure per row -> one per statement, 1,665 -> 310 per cell (#31669); an `exec()` of 80,000 statements 5.34 -> 0.11 s of CPU (#45207). - bun:sqlite: the `Database` is a GC cell that owns its connection (#41611), so dropped `deserialize()` databases and failed opens (#34898) no longer pile up: 40,000 of the first +3,016 -> +3.9 MB. #28911's `db.query()` loop 953 -> 92 MB; text over 64 KB isn't cached (#28912, #28911). Workers shared SQLite's malloc mutex -> configured once per process, `MEMSTATUS` off and `MULTITHREAD` (#44197). - node:sqlite (release): a 5-column `get()` 13,458 -> 4,077 (#44103); all 10 cases are cheaper. #### Tests and CI - **The PostgreSQL test containers crash-restarted under load** (57P03 "the database system is in recovery mode"). We infer: docker kills the `sh -c pg_isready` health check at its 5 s timeout, the postmaster (PID 1) adopts the orphan and takes its exit 2 for a crashed backend. `test/docker/docker-compose.yml` now has an exec-form check and `init: true`. - `sql-tls-ip-literal-host.test.ts` failed 16 of 108 runs on main (mock on 127.0.0.1, client dialed `::1`) -> 0. - `postgres_auth` had the md5 and SCRAM verifiers swapped and no test used it -> fixed; the login tests are in `test/js/sql/postgres-auth.test.ts`. - MariaDB runs the whole `sql-mysql.test.ts` matrix. `prestart-map.mjs` is complete (#35912, #44043), and `test/internal/prestart-map.test.ts` fails when a test's container is missing. The `isDockerEnabled()` wrappers are gone, so `BUN_TEST_SERVICE_*` overrides run the suites (#37123, SQL part of #39850). - Ported test-only PRs: #36948 #37114 #38221 #40193 #40194 #40537 #40633 #41104 #41176 #42587 #44035 #44850. #### Docs and types - `docs/runtime/sql.mdx` was rewritten. - Types: ``await sql`...` `` carries the result metadata (`SQL.ResultMetadata`) (#40434, #30037); `release()` is `Promise<void>` (#43255); `SavepointSQL` has `savepoint()`; bun:sqlite options accept `undefined` under `exactOptionalPropertyTypes` (#45079). - `onconnect` / `onclose` were documented as receiving a client -> an error or `null` (#38625, #38634). `timestamp` decodes as UTC (#39881, #39879). Multi-statement files are atomic on PostgreSQL only (#41744). Also #41813 (SQLite codes, placeholders, MySQL DDL), #44998 (what a savepoint holds), #44373 (defaults), #42206 (SQL half), #41094. - bun:sqlite: examples call `db.run()`, not the deprecated `db.exec()` (#24482); typings and docs for `safeIntegers()` (#26627), `isFinalized`, `toJSON()`, `clearQueryCache()`; `get()` / `all()` without arguments reuse the last bound values (#31747, #26622). ### Found and not fixed "Also on main" is marked (m). - **Worth a close read:** `signature_digest_nid` in `src/boringssl_sys/boringssl.rs` is new unsafe FFI over a certificate the peer chooses, and `on_handshake` now runs user code. - **node:sqlite, macOS:** #45171 (read-only open of a WAL database whose sidecars are gone) isn't fixed and gets easier to hit, because every `close()` is now a real close. - **Options, decided by the maintainer: as on main.** `NODE_TLS_REJECT_UNAUTHORIZED=0` also switches off `sslmode=verify-ca` / `verify-full` (#41589). `new SQL({ hostname })` takes the other fields of an environment URL, its user and password too (#41586). A `TLS_*` URL is taken as written: `?sslmode=disable` or `tls: false` make it plaintext. `PGDATABASE` / `MYSQL_DATABASE` / `MARIADB_DATABASE` beat the URL's database (#45276, #45287). The hardening was in this PR, is out again, and stays open in those pull requests. - **Pool (m):** no lazy or `min`-sized pool (#30631, #30632, #30636), no `AbortSignal` (#23175). Queries assigned to a connection but not sent yet die with it (#44924). At `max: 1` with a `BEGIN` by hand, `idleTimeout` / `maxLifetime` can close the connection between two statements (documented). Other callers' queries already sent behind a query that leaves a transaction open are rolled back with it. - **`--hot`:** a kept pool that nobody asks for a connection during a whole load has its idle connections closed and dials again on demand. An uncleared `setInterval` of an earlier load that queries makes its old pool fill again (m). `listen()` callbacks of earlier loads keep running (m). - **cancel():** a running PostgreSQL query with another one written behind it rejects at once but runs to its end on the server, as on main. MySQL / MariaDB (m): a query on the wire runs to its end. No `KILL QUERY` is sent: it names a session, and a late kill hit the session's next query in 34 % to 88 % of our tries, depending on server and set-up (#44625). - **MariaDB XA**, `max > 1` (m): the connection that prepared a branch answers other callers with 1399, and `commitDistributed()` elsewhere gets 1397. New: `reserved.beginDistributed()` directly followed by `reserved.release()` closes the connection. - **PostgreSQL (m):** `real`, `int4[]` and `float4[]` follow the wire format: `0.1` / `Array` in text, `0.10000000149011612` / a typed array in binary (#41888, #45211). A string bound to `json` / `jsonb` is stored as a JSON string when prepared, like postgres.js, and sent as it is with `prepare: false`, like pg; so `${JSON.stringify(obj)}` stores a string (#28819, #40944). Arrays of user-defined types read as the literal `"{a,b}"` (#41040), composites as `"(1,q)"` (#20379). A fractional number into `numeric` keeps 15 digits, a bare number into `json` / `money` fails (42804). `prepare: false`: a `Date` into `time`, `json`, `jsonb` is rejected. - **PostgreSQL auth:** `prefer` doesn't stop a server from asking for a cleartext or md5 password; `require` does. (m): the SCRAM iteration count must be within 4096..10,000,000, deliberate hardening; libpq, postgres.js and pg accept any count >= 1. Parity is the range check in the SASLContinue arm of `PostgresSQLConnection.rs` (#38133). - **MySQL:** `sha256_password` over a unix socket still needs `allowPublicKeyRetrieval: true`. (m): FLOAT reads `0.1` from `.simple()`, `0.10000000149011612` prepared, like mysql2 (#41888); BIGINT above 2^31 is a string; after `USE` a cached text still runs in the OLD database, silently; a multi-statement text that `KILL QUERY` hits between two statements never settles (`maxLifetime` bounds it). - **SQLite:** on every adapter `transform.undefined` skips `sql.unsafe()` values. `db.prepare("a; b").run()` runs only `a`, like node:sqlite (documented). - **Not reproduced on Linux, no claim:** #24130 #24845 #24434 #23305 #30494 #16844 #17173 #23654 #32099. ### Decisions reversed while writing this - `--hot` was first going to close the old load's connections; the maintainer overruled it. "Close when the settings changed" then closed pools the program still used; now use decides. - The MySQL statement cache was going to be keyed on the text alone, like mysql2 and mariadb. MySQL >= 8.0.22 settles a parameter's type at its first non-NULL value, so `WHERE name = ${"bob"}` after `${5}` returned wrong rows: one statement per text and set of non-NULL types. - Ed25519 under `prefer` was going to fall back to unbound SCRAM, as in #41703; psql 15 fails, and a fallback lets whoever picks the certificate switch the binding off. - `SQLITE_DEFAULT_WAL_SYNCHRONOUS=1` (#20662) was in and is reverted: an upgrade must not lower the durability of a COMMIT. - On the maintainer's decision: a bulk insert takes the first row's keys as its columns, as on main and in postgres.js, not the union of all rows' keys (half of #35111); `int4[]` / `float4[]` and `real` follow the wire format, not one form on every path (#45211, #41888); `console.log(rows)`, `Object.keys(rows)` and a spread list `count`, `command`, `lastInsertRowid`, `affectedRows`, as on main (#38443; `columns` and `statement`, new, are not listed); MySQL's default database stays `mysql`. - Five more are main's again, because main works there and nobody asked: a failing query that nobody handles no longer ends the process; the first query fills the pool to `max` (the lazy pool, #30636); a caller whose dial is refused waits for a connection that is reserved or in a transaction; PostgreSQL `.raw()` returns main's bytes, not text; in `sql.array(values, "JSON" | "JSONB")` `null` is the JSON value and a nested array a dimension. - Taken out before the merge, because main works there. Values: a string into `json` / `jsonb` sent as it is (#40944), a JS array into `text` as an array literal, `Infinity` untyped, typed arrays by the server's type, `foundRows: true` by default, `not in (NULL)` for an empty list on SQLite, the result mode fixed at `execute()`. Errors: for an `async` `db.transaction()` callback (#45132), `parts.map(sql.unsafe)`, `sql.unsafe(text, 0)` (#44745, #44867), `new Database(path, true)`, `{ readonly: true, create: true }` on the SQLite adapter, `prepare: false` on MariaDB. Options: `?` inside a plain SQLite file name (#38427), `new SQL("")` reading the environment. ### From outside contributors PRs by people, team members included. Every item is reimplemented from the idea unless it says otherwise. - #40913 @dylan-conway: his mechanism as it is (`onconnect` at the end of the read, only if still connected), ported by hand to both drivers. Also the error of his #38514. - #25623 @eriklangille: the diagnosis in #23215 and the goal, reuse across `--hot` reloads. Other mechanism: connections move between pools in `pool.ts`, no GC hook. - #23350 @gohryt: the request for COPY and the scenarios to test. Other API. - #36451 @pjb3: the `transform` option on all adapters, with postgres.js's directions, applied to the row shape. - #8558 @nikeee, #25703 @jdjkelly: user-defined functions; the name `function`, option names, arity rule, test ideas. - #12631 @jecquas: `sqlite3_update_hook`, in the shape its review suggested. - #31669 @alii: one cached Structure for rows wider than 62 columns. #31209, #31738 @alii: `Date` parameters before 1970, fractional seconds of binary TIME. - #43789 @ralyodio: the diagnosis that `detectCommand` copies and lower-cases the whole text per helper. #45151 @RubenPari: the cause (every JSC `Error` has its own `line` / `column`) and the `delete`. - #26711 @RajeshKumar11: `PGUSERNAME`. #45287 @wahajahmed010 (the URL's database above `PGDATABASE`) stays open: decided as on main. - #22109 @himself65: `Symbol.for("sqlite-type")`. #26627 @0xh3x: `safeIntegers()` typings. #31747 @Artoria2e5: the JSDoc correction. #24482 @jobansd: `exec` -> `run`, the same 15 edits. #41094 @deepshekhardas: trailing spaces. - Not taken, with thanks (reasons below): #26503 @S4N-T0S, #31751 @EffortlessSteven, #27442 @sosukesuzuki (cases of both are tests now), #22934 @nektro, #16884 @chrisbenincasa, #25836 @alimd, #26880 @theshadow27, #31078 @countradooku, #25403 @abelcha, #44854 and #44860 @deepshekhardas. - Not in this set: #41890 @steipete (only statements a `StatementSync` owns may be finalized), #28161 @kernoeb (found the 21-byte nonce first). Reports and measurements: @bompus (#44084, #44187), @gerardp (#35120, his burst script is a test), @Dex4Sure (#38163), @filipesarturi (#33190), @KilianB (#30037). ### Replaces #8558 #12631 #22109 #23350 #24482 #25623 #25703 #26627 #26711 #28912 #30037 #30648 #30848 #31209 #31669 #31738 #31747 #32040 #32100 #32422 #32558 #32841 #32917 #33189 #33190 #33370 #33504 #33579 #33583 #33740 #34453 #34898 #35111 #35117 #35118 #35120 #35912 #36241 #36301 #36371 #36451 #36948 #37006 #37114 #37123 #37125 #37238 #37240 #37418 #37807 #37819 #37960 #38163 #38219 #38220 #38221 #38417 #38438 #38468 #38625 #38634 #39452 #39881 #40005 #40193 #40194 #40434 #40537 #40633 #40913 #41042 #41051 #41094 #41104 #41176 #41301 #41570 #41605 #41611 #41703 #41725 #41744 #41758 #41813 #41815 #41816 #41817 #41886 #41970 #41976 #42587 #42637 #43116 #43205 #43249 #43255 #43257 #43261 #43263 #43281 #43293 #43306 #43335 #43336 #43652 #43789 #43818 #43918 #43958 #43959 #44022 #44035 #44042 #44043 #44089 #44103 #44105 #44118 #44197 #44204 #44315 #44373 #44483 #44625 #44646 #44784 #44799 #44850 #44908 #44932 #44938 #44983 #44998 #45014 #45065 #45079 #45080 #45096 #45111 #45118 #45151 #45153 #45160 #45207 #45229 #45305 #45332 #45367 #45387 ### Closed without porting - #16884: the compile option is already on main - #22934: those suppressions are already gone (#30875) - #25403: would make the private adapter interface public - #25836: part landed in #43649, two lines mistype - #26503: four claims already fixed on main; default `sslmode` stays `disable` - #26880: superseded by the author's own #26884 - #27442: already fixed on main (#35246) - #31078: nothing can call `interrupt()` while a statement runs - #31751: no observable bug on a real server - #34435: `numeric` is read as text; binary decoder deleted - #34673: its Structure slots are never freed on eviction - #35738: the mock repeats a real-server test - #35753: duplicate export already removed (#44614) - #37544: main already holds statements in `RefPtr` (#40516) - #37620: its leak was fixed on main (#40238) - #44040: superseded by #44105's synchronous start - #44854: makes the page's spelling less consistent - #44860: makes the guide's wording less consistent ### Left open - #26884: backup API is a product decision - #30636: main's docs promise the eager fill; a lazy pool is a product decision - #34281: `mkdir -p` on open is a product decision - #34863: `AsyncDatabase` is a product decision - #36712: the `better-sqlite3` alias is a product decision - #37013: SQL part obsolete since #44618; the rest is not SQL - #38133: the SCRAM floor is kept; only its better error is here - #38289: MySQL half is here; `Bun.Image` is not SQL - #38427: its `url` half is here; the `?` half changes what main opens - #38443: `console.log(rows)` keeps showing the four properties, as on main, decided - #38514: bun:sqlite part is here; the rest is not SQL - #39850: SQL gates are here; the rest is not SQL - #40250: the SQL code it repairs is deleted here; the rest is not SQL - #40944: main works, like postgres.js; the change breaks stored data - #41586: two parts are here; the rest is decided: as on main (Limits) - #41589: decided: as on main (Limits) - #41888: `real` follows the wire format, as on main, decided - #42054: SQL half is here; the Redis half is not - #42206: SQL half is here; the `nodejs-compat` half is not - #42868: SQLite parts are here; the rest is not SQL - #43797: SQLite half is here; the V8 addon half is not - #44745: its SQLite part is here; `parts.map(sql.unsafe)` works on main and in postgres.js - #44867: its SQLite part is here; on the servers `sql.unsafe(text, 0)` runs on main - #44897: SQL half is here; the node:http2 half is not - #44924: handing unsent requests back can half-commit at `max: 1`; needs a maintainer's yes - #44962: SQLite 3.54.0 should merge, and revert, alone - #45132: an `async` callback without `await` works on main - #45211: NULLs and dimensions on the binary paths are here; typed arrays stay, decided - #45287: decided: as on main (Limits) - #45437: a source break in the types, on its own ### Partly addressed - #39877: a bare key binds; an unknown key in default mode still binds NULL (`strict: true` throws). - #43113: not reproduced; the one route found to its errors (an open `iterate()` cursor) is closed. - #19130, #40077: `expect()` no longer hangs on a lazy `Query`; #19130's own script (Windows) didn't hang on Linux, #40077's other items are untouched. - #27102: the process no longer exits with a query in flight; its `CONNECTION_CLOSED` under load wasn't reproduced. #38277: MySQL half only. - #15088: its COPY item. #15087: `SQL.toCamel` and friends cover its SQL use. #18866: `columns` and `statement`, not `state`. #23175: `cancel()` works, no `signal`. #16665: `new Bun.sql(url, options)` applies the options. - #39450: Date and object are fixed; a Date into `time` / `json` with `prepare: false` still differs. - #28819, #40942: not changed, see Limits; `sql.json()` and the docs now say how to store each kind of value. - #26629: changed by reading, no such build was made. #16717: documented (macOS reports Apple's version). #26369: already fixed on main (#44618). - #44187: memory statistics and the connection mutex are gone; scaling like Node was measured on a loaded box only. - #20662: not changed; the docs give the `PRAGMA synchronous = NORMAL` tip. - #24662: async support isn't added; only a promise that is already rejected at the return rolls back, and the docs warn. - #45276: not changed, decided as on main (Limits). ### Issues fixed Fixes #1332. Fixes #1474. Fixes #4175. Fixes #5051. Fixes #9410. Fixes #13409. Fixes #15876. Fixes #16469. Fixes #18326. Fixes #18456. Fixes #23215. Fixes #23954. Fixes #24050. Fixes #25405. Fixes #25472. Fixes #26195. Fixes #26622. Fixes #26684. Fixes #26809. Fixes #28911. Fixes #28980. Fixes #29010. Fixes #29484. Fixes #29551. Fixes #30221. Fixes #30646. Fixes #30811. Fixes #30843. Fixes #32005. Fixes #32035. Fixes #32038. Fixes #32096. Fixes #32148. Fixes #37415. Fixes #39879. Fixes #40001. Fixes #40432. Fixes #41010. Fixes #41039. Fixes #41050. Fixes #41242. Fixes #41740. Fixes #41856. Fixes #42720. Fixes #42804. Fixes #44084. Fixes #44367. Fixes #45039. Fixes #45062. Fixes #45064. Fixes #45275.

Jarred-Sumner · 2026-10-17T07:07:10Z

(This is Claude, working with Jarred on consolidating the open SQL PRs.)

Thanks @alii for catching this: the binary TIME decoder parsed microseconds and never printed it, so a prepared query returned "02:03:04" for 02:03:04.5. Main still did that on MySQL 8.4 and MariaDB 11.8. Sorry for the wait since June.

#45304 fixes it with a different format. The last revision here goes through Postgres__formatTime, which trims trailing zeros. The server's text protocol pads to the column's precision, so a prepared query and .simple() would still disagree for .400, .000500 and a zero TIME(3) / TIME(6) (the August comment on this PR shows it). The decoder now takes the column definition and pads to ColumnDefinition41.decimals, so a prepared query, .simple() and the server's own text give the same string.

That is also the answer to Jarred's review question: no existing function prints MySQL's form, but the column's decimals byte was already parsed and never used.

What differs from this PR: 02:03:04.5 in a TIME(6) column comes back as "02:03:04.500000", and a zero TIME(3) as "00:00:00.000". Plain TIME is unchanged. A packet with 1,000,000 or more microseconds rejects with ERR_MYSQL_INVALID_BINARY_VALUE. .raw() on a prepared query now returns the value's bytes for TIME, like for other types.

The code is reimplemented in src/sql_jsc/mysql/protocol/DecodeBinaryValue.rs; your diff was read for the idea, and your 02:03:04.5 case is a row of the new table. Tests: "TIME(1)" to "TIME(6) decodes alike in a prepared and a simple query" and "expressions decode alike in a prepared and a simple query" (TIMEDIFF, SEC_TO_TIME, MAKETIME) in test/js/sql/sql-mysql-values.test.ts on both servers, and "a binary TIME no server sends" in test/js/sql/sql-mysql-packet-overrun.test.ts. The mock file sql-mysql-time-fraction.test.ts isn't kept. You're credited in the description of #45304.

mysql: keep fractional seconds when decoding binary TIME values

390e34f

claude Bot reviewed Jun 2, 2026

View reviewed changes

ci: retrigger

d456000

claude Bot reviewed Jun 2, 2026

View reviewed changes

Jarred-Sumner reviewed Jun 3, 2026

View reviewed changes

coderabbitai Bot reviewed Jun 3, 2026

View reviewed changes

test: move the binary TIME mock coverage to a standalone file

8e32b7d

Non-docker mock-server MySQL tests live in their own sql-mysql-*.test.ts files in this suite; sql-mysql.test.ts keeps only the docker-backed TIME(6) test and returns to main's state otherwise.

claude Bot reviewed Jun 3, 2026

View reviewed changes

robobun mentioned this pull request Aug 21, 2026

test: gate service-backed suites on isDockerServiceEnabled() so BUN_TEST_SERVICE overrides run them #39850

Open

Jarred-Sumner mentioned this pull request Oct 15, 2026

sql: consolidate the open Bun.sql, bun:sqlite and node:sqlite pull requests #45304

Merged

robobun mentioned this pull request Oct 15, 2026

sql(postgres): decode each unquoted array element by its column type #45305

Closed

Jarred-Sumner closed this Oct 17, 2026

Conversation

alii commented Jun 2, 2026 • edited by robobun Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alii commented Jun 2, 2026

Uh oh!

coderabbitai Bot commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Walkthrough

Changes

Suggested reviewers

Uh oh!

robobun commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

robobun commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Overview

Security risks

Level of scrutiny

Other factors

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Overview

Security risks

Level of scrutiny

Other factors

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Overview

Security risks

Level of scrutiny

Other factors

Uh oh!

Jarred-Sumner left a comment

Choose a reason for hiding this comment

Uh oh!

robobun commented Jun 3, 2026

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

robobun commented Jun 3, 2026

Uh oh!

claude Bot left a comment

Choose a reason for hiding this comment

Overview

Security risks

Level of scrutiny

Other factors

Uh oh!

robobun commented Aug 15, 2026

Uh oh!

Jarred-Sumner commented Oct 17, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

alii commented Jun 2, 2026 •

edited by robobun

Loading

coderabbitai Bot commented Jun 2, 2026 •

edited

Loading

robobun commented Jun 2, 2026 •

edited

Loading

robobun commented Jun 2, 2026 •

edited

Loading