fix(transport): refs PerryTS/perry#536 — call net.createConnection(port, host) on Perry, not (host, port)

The perry path of `openSocket` swapped Node's positional argument order
based on a mistaken assertion in the file header. Perry's
`net.createConnection` has always matched Node's documented
`(port[, host][, connectListener])` — see
`crates/perry-codegen/src/lower_call.rs` (NA_F64 first, NA_STR
second) in the perry tree. Under the wrong order Perry coerced the
host string to a port (NaN-coerced) and the port number to a host
pointer, returning an invalid socket handle that silently no-op'd
through every `sock.on(...)` registration. `await connect(...)` then
never resolved against any MySQL server, surfacing as an immediate
exit with no rows on the bare `await pool.query("SELECT 1 AS n")`
shape from the issue's repro.

Header comment rewritten to reflect Perry's actual `(port, host)`
contract. The Node.js code path is unchanged — Node accepts the
object form `({ host, port })` which the driver already used.

Bumps to 0.1.3 and adds a CHANGELOG entry. After running with
perry >= v0.5.652 (which includes #536's runtime-side fixes for
[].pop()/shift() returning undefined and ext-net active-handles
wiring), the issue's repro now returns the SELECT 1 row end-to-end.

Author: proggeramlug

CHANGELOG.md

@@ -1,5 +1,22 @@
 # Changelog
 
+## v0.1.3
+
+- Fix the Perry path of `transport/net-socket.ts::openSocket` to call
+  `net.createConnection(port, host)` matching Node's documented
+  positional signature. The pre-fix call passed `(host, port)` based
+  on a mistaken assertion in the file's own header comment that Perry
+  used a different order — Perry has always matched Node's
+  `(port, host)` (see `crates/perry-codegen/src/lower_call.rs` in the
+  perry tree). Under the wrong order Perry coerced the host string to
+  a port (NaN) and the port number to a host pointer, returning an
+  invalid socket handle that silently no-op'd through every subsequent
+  `sock.on(...)` registration — `await connect(...)` then never
+  resolved against any MySQL server. Header comment rewritten to
+  reflect Perry's actual `(port, host)` contract. PerryTS/perry#536.
+- The Node.js code path is unchanged — Node accepts the object form
+  `({ host, port })` which the driver already used.
+
 ## v0.1.2
 
 - Fix binary decoders for `DATE`, `DATETIME`/`TIMESTAMP`, and `TIME`

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@perryts/mysql",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Pure-TypeScript MySQL/MariaDB wire-protocol driver. Runs on Node.js and Bun, and compiles to a native binary via Perry (LLVM). Zero native dependencies.",
   "keywords": [
     "mysql",

src/transport/net-socket.ts

@@ -1,9 +1,8 @@
 // Cross-environment socket adapter. Perry and Node/Bun both ship a
-// Node-compatible `net` module, but `createConnection` has slightly
-// different signatures:
-//
-//   - Perry (perry-stdlib):  net.createConnection(host, port)
-//   - Node (node:net):       net.createConnection({ host, port })
+// Node-compatible `net` module. Both accept the positional
+// `(port, host)` form (matching Node's documented signature
+// `net.createConnection(port[, host][, connectListener])`); Node
+// additionally accepts the object form `({ host, port })`.
 //
 // This file is the only place in the driver that cares. Everything else
 // consumes the returned `Socket` interface.
@@ -52,5 +51,11 @@ export function openSocket(host: string, port: number): Socket {
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         return (net as any).createConnection({ host: host, port: port }) as Socket;
     }
-    return net.createConnection(host as never, port as never) as unknown as Socket;
+    // Perry positional signature is `(port, host)` per Node's documented
+    // `net.createConnection(port[, host])`. Pre-fix this called
+    // `(host, port)` — perry coerced the host string to a port (NaN)
+    // and the port number to a host pointer, returning an invalid
+    // socket handle that silently no-op'd the rest of the connection
+    // lifecycle. PerryTS/perry#536.
+    return net.createConnection(port as never, host as never) as unknown as Socket;
 }