@norskvideo/ctl-test-harness 0.1.28 → 0.1.29

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@norskvideo/ctl-test-harness",
-  "version": "0.1.28",
+  "version": "0.1.29",
   "type": "module",
   "exports": {
     ".": {

package/smoke.js

@@ -329,12 +329,13 @@ export async function runProductSmoke(slug, spec, deps = defaultSmokeDeps) {
     }
     if (handles.length)
         await deps.stopSources(handles).catch(() => { });
-    // The control plane is a singleton that deliberately outlives its daemon, so
-    // stopping the daemon strands it holding 127.0.0.1:14321 and every later job
-    // on that runner fails to bind. Ask the daemon to remove it while there is
-    // still a daemon to ask. Best-effort: a teardown that throws here would mask
-    // the journey's own error, and a SIGKILLed job never reaches this line at all
-    // -- the workflow's reap covers that half.
+    // The daemon reaps its own control-plane containers on shutdown (daemon.ts,
+    // model B), but only on the graceful path -- and cleanupDaemon escalates to
+    // SIGKILL 3s after SIGTERM, which can cut a slow `docker rm` short. Removing
+    // the product here does it while the daemon is calm rather than racing its
+    // own teardown. Best-effort: a throw here would mask the journey's own error,
+    // and a killed job never reaches this line -- the workflow's reap is what
+    // heals a runner already holding the port.
     if (registered)
         await cli(["product", "remove", spec.product.name]).catch(() => { });
     // cleanupDaemon ends with rmSync(storeDir); Docker's root-owned bind-mount