test(daemon-client): keep fixture startup errors when teardown fails; reflow package-move comments

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

Author: thymikee

packages/daemon-client/src/daemon-client-timeout.ts

@@ -154,8 +154,9 @@ function isAffirmativelyApplePlatform(platform: PlatformSelector | undefined): b
 // Exported for direct hint-matrix testing: handleRequestTimeout also runs the real liveness probe
 // and process-kill side effects, so its wording is verified through this pure sub-function
 // (see also the production-seam route tests in
-// packages/daemon-client/src/__tests__/daemon-client-timeout-route.test.ts, which prove the liveness-gated
-// recovery this route performs — the side of the contract a pure formatter test cannot reach).
+// packages/daemon-client/src/__tests__/daemon-client-timeout-route.test.ts, which prove the
+// liveness-gated recovery this route performs — the side of the contract a pure formatter test
+// cannot reach).
 export function resolveRequestTimeoutHint(params: {
   remote: boolean;
   resetDaemon: boolean;

scripts/layering/package-entry-surface.ts

@@ -48,7 +48,10 @@ function violation(file: string, message: string, line = 1): LayeringViolation {
   return { rule: PACKAGE_EXPORT_SURFACE_RULE, file, line, message };
 }
 
-/** Keeps test-kit, daemon and daemon-client on one named public entry; test-kit also runs the isolation walk. */
+/**
+ * Keeps test-kit, daemon and daemon-client on one named public entry; test-kit also runs the
+ * isolation walk.
+ */
 export function checkPackageEntrySurfaces(
   manifests: ParsedManifests,
   sources: ReadonlyMap<string, string>,

src/__tests__/test-utils/registered-http-daemon.ts

@@ -58,11 +58,11 @@ export async function startRegisteredHttpDaemon(): Promise<RegisteredHttpDaemon>
     const child = spawnRegisteredDaemonFixture(paths, await registrationFor(stub.port), undefined);
     await waitForRegisteredDaemonFixture(paths, child);
   } catch (error) {
-    try {
-      await finishRegisteredDaemonFixture(stateDir);
-    } finally {
-      await closeLoopbackServer(stub.server);
-    }
+    // Both teardown steps run, and neither failure replaces the startup error.
+    await Promise.allSettled([
+      finishRegisteredDaemonFixture(stateDir),
+      closeLoopbackServer(stub.server),
+    ]);
     throw error;
   }
   return {
@@ -90,7 +90,7 @@ export async function launchesFreshHttpDaemon(
   try {
     fields = await registrationFor(stub.port);
   } catch (error) {
-    await closeLoopbackServer(stub.server);
+    await Promise.allSettled([closeLoopbackServer(stub.server)]);
     throw error;
   }
   spawn.mockImplementation((_command, _args, options) =>