@norskvideo/ctl-sdk 0.1.22 → 0.1.23

package/manifest-fetch.d.ts

@@ -17,7 +17,18 @@ export declare function specBaseUrl(spec: ProductSpec, port?: number, reachHost?
  *  so a recorded reachHost is never forgotten. */
 export declare function productBaseUrl(reg: Pick<ProductRegistration, "spec" | "port" | "reachHost">): string;
 export declare function fetchManifest(baseUrl: string): Promise<Manifest>;
-export declare function waitForReady(baseUrl: string): Promise<void>;
+/** Poll a product's manifest until it answers.
+ *
+ *  The timeout names `baseUrl`. That is the whole point of the message: the
+ *  daemon dials a container product on loopback, and where that loopback is not
+ *  the daemon's own (a docker-outside-of-docker runner, where it belongs to the
+ *  docker host — see product-reach.ts) the product is perfectly healthy and
+ *  simply unreachable. Without the address, that reads identically to a product
+ *  that genuinely failed to start. */
+export declare function waitForReady(baseUrl: string, opts?: {
+    timeoutMs?: number;
+    intervalMs?: number;
+}): Promise<void>;
 /**
  * Probe the product's configScreenUrl after registration. Two hard
  * rejections:

package/manifest-fetch.js

@@ -56,8 +56,18 @@ export async function fetchManifest(baseUrl) {
     }
     return parsed.data;
 }
-export async function waitForReady(baseUrl) {
-    const deadline = Date.now() + READINESS_TIMEOUT_MS;
+/** Poll a product's manifest until it answers.
+ *
+ *  The timeout names `baseUrl`. That is the whole point of the message: the
+ *  daemon dials a container product on loopback, and where that loopback is not
+ *  the daemon's own (a docker-outside-of-docker runner, where it belongs to the
+ *  docker host — see product-reach.ts) the product is perfectly healthy and
+ *  simply unreachable. Without the address, that reads identically to a product
+ *  that genuinely failed to start. */
+export async function waitForReady(baseUrl, opts = {}) {
+    const timeoutMs = opts.timeoutMs ?? READINESS_TIMEOUT_MS;
+    const intervalMs = opts.intervalMs ?? READINESS_INTERVAL_MS;
+    const deadline = Date.now() + timeoutMs;
     while (Date.now() < deadline) {
         try {
             const r = await fetch(`${baseUrl}/manifest.json`);
@@ -67,9 +77,9 @@ export async function waitForReady(baseUrl) {
         catch {
             // not ready yet
         }
-        await new Promise((resolve) => setTimeout(resolve, READINESS_INTERVAL_MS));
+        await new Promise((resolve) => setTimeout(resolve, intervalMs));
     }
-    throw new ProductError("READINESS_TIMEOUT", `product did not become ready within ${READINESS_TIMEOUT_MS}ms`);
+    throw new ProductError("READINESS_TIMEOUT", `product did not become ready within ${timeoutMs}ms — nothing answered ${baseUrl}/manifest.json`);
 }
 /**
  * Probe the product's configScreenUrl after registration. Two hard

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@norskvideo/ctl-sdk",
-  "version": "0.1.22",
+  "version": "0.1.23",
   "type": "module",
   "exports": {
     ".": {
@@ -37,7 +37,7 @@
   "types": "./index.d.ts",
   "dependencies": {
     "@norskvideo/ctl-foundation": "^0.1.0",
-    "@norskvideo/ctl-product-template-schema": "^0.1.5",
+    "@norskvideo/ctl-product-template-schema": "^0.1.12",
     "express": "5",
     "lucide-react": "^0.483.0",
     "react": "^19.0.0",

package/product-service.js

@@ -127,7 +127,7 @@ export class ProductService {
             }
             port = allocated;
             await this.refreshImage(spec.image);
-            logger.info(`Starting product container: ${spec.image} on host port ${port}`);
+            logger.info(`Starting product container: ${spec.image} on host port ${port} (reach: ${this.reach})`);
             containerId = await this.containerOps.run(spec.image, port);
             reachHost = await this.reachHostFor(containerId);
             baseUrl = specBaseUrl(spec, port, reachHost);