From 21e46ec23f0ca223854e8da814c139861a5e0d52 Mon Sep 17 00:00:00 2001
From: Aaron Queen <bompus@users.noreply.github.com>
Date: Fri, 9 Oct 2026 21:15:59 -0600
Subject: [PATCH] test(register-hook): take ports from the kernel instead of
 picking one at random

Two tests picked a random port in 20000-40000 and started a server there, so a busy port failed the run with EADDRINUSE (seen on PR #128's first CI run). They now bind port 0; the test that starts its server late asks the kernel for a free port first.
---
 test/register-hook.test.js | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/test/register-hook.test.js b/test/register-hook.test.js
index 2f23c9b..29c9ff8 100644
--- a/test/register-hook.test.js
+++ b/test/register-hook.test.js
@@ -29,6 +29,14 @@ import { addT3V2Thread, createT3V2Tables } from "./fixtures/t3-v2-state.js";
 import { parseTag, sessionTag } from "../src/tag.ts";
 import { createServer } from "../src/server.ts";
 
+/** A port nothing listens on yet, for a server that starts later in the test. */
+function freePort() {
+  const probe = Bun.serve({ port: 0, fetch: () => new Response() });
+  const { port } = probe;
+  probe.stop(true);
+  return port;
+}
+
 /** A stand-in Swarmail MCP endpoint: `handler(tool, args)` returns the tool result and every call is recorded. */
 function fakeMail(handler) {
   const calls = [];
@@ -726,7 +734,7 @@ test("the hook registers under a fresh name when the server refuses the session'
 
 test("a failed registration tells Claude, then the next prompt retries against the server and reports the name", async () => {
   const dir = realpathSync.native(mkdtempSync(join(tmpdir(), "hook-retry-")));
-  const port = 20000 + Math.floor(Math.random() * 20000);
+  const port = freePort();
   let server;
   try {
     const repo = join(dir, "repo");
@@ -790,8 +798,8 @@ test("a failed registration tells Claude, then the next prompt retries against t
 
 test("a claude -p run registers on its first edit, not at start", async () => {
   const dir = realpathSync.native(mkdtempSync(join(tmpdir(), "hook-headless-")));
-  const port = 20000 + Math.floor(Math.random() * 20000);
-  const server = createServer(join(dir, "mail.sqlite3"), port);
+  const server = createServer(join(dir, "mail.sqlite3"), 0);
+  const port = server.server.port;
   try {
     const repo = join(dir, "repo");
     Bun.spawnSync(["git", "init", "-q", repo]);
@@ -843,8 +851,8 @@ test("a claude -p run registers on its first edit, not at start", async () => {
 
 test("a session registers at start, keeps that one name through edits, restarts and hand registration", async () => {
   const dir = realpathSync.native(mkdtempSync(join(tmpdir(), "hook-start-")));
-  const port = 20000 + Math.floor(Math.random() * 20000);
-  const server = createServer(join(dir, "mail.sqlite3"), port);
+  const server = createServer(join(dir, "mail.sqlite3"), 0);
+  const port = server.server.port;
   try {
     const repo = join(dir, "repo");
     Bun.spawnSync(["git", "init", "-q", repo]);