**13 issues found** across 73 files

Prompt for AI agents (unresolved issues)

```text Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately. P2: Wrap `client.connect()` in the same cleanup path as the rest of the test. A rejected handshake/TLS connect currently skips `client.close()` and can leak the underlying socket. P2: This assertion does not verify input ordering; it only checks that the two message IDs differ. P2: This drops the previously exported `RPCError` symbol from the package root, which is a breaking change for existing imports. Keep a compatibility alias when introducing `ProtocolError`. P2: Await `client.close()` in these tests; otherwise they can pass without actually verifying that the async close operation succeeds. P2: Close the readiness probe in a `finally` block so failed attempts do not leak client sockets. P2: Handle `adminClient.connect()` failures by cleaning up the spawned server and temp directory before rethrowing. P2: Infinite loop in `grow()` when buffer length is 0. If `initialCapacity` is 0 (or the buffer somehow becomes length 0), `newSize` starts at 0 and `0 * 2` stays 0 forever. Guard the doubling with a minimum size. P2: `close()` sets `closed` before sending `OP_DISCONNECT`, so the graceful disconnect frame is never transmitted. P2: Bitwise wrapping turns request IDs negative after `2^31 - 1` requests, which breaks frame encoding. P2: `parseAddr()` breaks IPv6 addresses by treating the last colon as the host/port separator. P1: Leader-redirected consumers outlive `client.close()` because the extra `leaderConn` is never tracked or closed by the client. P2: The redirected consume path should validate `OP_CONSUME_OK` before entering the delivery loop. P2: Validate batch sizes before storing them; `0`, negative, or `NaN` values make `flushAll()` spin forever. ```

_{Reply with feedback, questions, or to request a fix. Tag `@cubic-dev-ai` to re-run a review.}