From d032d047c66ac6f7b640c6e3c078692a3c1417d0 Mon Sep 17 00:00:00 2001
From: Aiden Cline <aidenpcline@gmail.com>
Date: Wed, 22 Jul 2026 21:05:35 +0000
Subject: [PATCH] test(cli): stabilize service registration test

---
 packages/cli/test/service.test.ts | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/packages/cli/test/service.test.ts b/packages/cli/test/service.test.ts
index 3e29d6e443..cd47de29b4 100644
--- a/packages/cli/test/service.test.ts
+++ b/packages/cli/test/service.test.ts
@@ -430,17 +430,15 @@ test("stale dead registration is replaced after binding the selected port", asyn
     stdout: "ignore",
   })
   try {
-    const info = await waitForInfo(registration, (value) => value.id !== "dead")
+    const info = await waitForInfo(registration, (value) => value.id !== "dead", 60_000)
     expect(new URL(info.url).port).toBe(String(port))
     expect(info.pid).toBe(owner.pid)
-    await Effect.runPromise(Service.stop({ file: registration }).pipe(Effect.provide(NodeFileSystem.layer)))
-    await owner.exited
   } finally {
     owner.kill("SIGTERM")
     await owner.exited
     await fs.rm(root, { recursive: true, force: true })
   }
-}, 30_000)
+}, 75_000)
 
 test("a failed service stays registered and owns the selected port until stopped", async () => {
   const root = await fs.mkdtemp(path.join(os.tmpdir(), "opencode-service-failed-"))
@@ -481,8 +479,9 @@ test("a failed service stays registered and owns the selected port until stopped
   }
 }, 30_000)
 
-async function waitForInfo(file: string, accept: (info: Info) => boolean = () => true) {
-  for (let attempt = 0; attempt < 400; attempt++) {
+async function waitForInfo(file: string, accept: (info: Info) => boolean = () => true, timeout = 20_000) {
+  const deadline = Date.now() + timeout
+  while (Date.now() < deadline) {
     const value = await Bun.file(file)
       .json()
       .catch(() => undefined)