From 6f0fa2d6c68561eff0826f5e9cf82ad13d3f2205 Mon Sep 17 00:00:00 2001 From: Kit Langton Date: Thu, 30 Jul 2026 20:46:46 -0400 Subject: [PATCH] core: remove unused ownerHeld bookkeeping in service restarter The ownerHeld flag was assigned but never read, making the service restart retry logic harder to follow. Removing it clarifies that the spawn delay only doubles when a previous contender finished cleanly. --- packages/client/src/promise/service.ts | 3 --- 1 file changed, 3 deletions(-) diff --git a/packages/client/src/promise/service.ts b/packages/client/src/promise/service.ts index 6d8aa5e327..ee9056e669 100644 --- a/packages/client/src/promise/service.ts +++ b/packages/client/src/promise/service.ts @@ -37,7 +37,6 @@ export async function ensure(options: EnsureOptions = {}): Promise { let announced = false let lastSpawn = 0 let spawnDelay = 5_000 - let ownerHeld = false const announce = (reason: "missing" | "version-mismatch", previousVersion?: string) => { if (announced) return @@ -65,7 +64,6 @@ export async function ensure(options: EnsureOptions = {}): Promise { const registration = await registered(options.file, true) if (registration.service !== undefined) { - ownerHeld = false spawnDelay = 5_000 const service = registration.service const compatible = !service.legacy && (options.version === undefined || service.version === options.version) @@ -82,7 +80,6 @@ export async function ensure(options: EnsureOptions = {}): Promise { if (failure !== undefined) throw failure const finished = [...contenders].filter(contenderFinished) if (finished.some((item) => item.child.exitCode === 0)) { - ownerHeld = true spawnDelay = Math.min(spawnDelay * 2, 30_000) } finished.forEach((item) => contenders.delete(item))