@norskvideo/ctl-sdk 0.1.5 → 0.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@norskvideo/ctl-sdk",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "type": "module",
   "exports": {
     ".": {

package/product-service.d.ts

@@ -49,9 +49,11 @@ export type ImportProductTemplateBytesFn = (opts: {
         kind: "product-default";
     }>;
 }) => Promise<void>;
-/** The container operations stopAll/restoreAll depend on, behind an interface
- *  so tests can drive them without shelling out to Docker. Defaults wire
- *  straight to docker-runner + the readiness probe. */
+/** Every container operation this service performs, behind an interface so
+ *  tests can drive them without shelling out to Docker. Defaults wire straight
+ *  to docker-runner + the readiness probe. Reach for these rather than the
+ *  module functions: calling docker-runner directly re-creates the bypass that
+ *  made an injected fake silently inert on add/remove. */
 export interface ProductContainerOps {
     run(image: string, hostPort: number): Promise<string>;
     remove(containerId: string): Promise<void>;
@@ -72,8 +74,8 @@ export interface ProductServiceOptions {
     /** Liveness probe for dev-mode products. Injectable for tests; defaults to a
      *  fast timeout-bounded GET of the dev URL's `/manifest.json`. */
     isDevUrlAlive?: (baseUrl: string) => Promise<boolean>;
-    /** Container lifecycle ops used by stopAll/restoreAll. Injectable for tests;
-     *  defaults to the real docker-runner functions. */
+    /** Container lifecycle ops used by add/remove/restart/stopAll/restoreAll.
+     *  Injectable for tests; defaults to the real docker-runner functions. */
     containerOps?: ProductContainerOps;
 }
 export declare class ProductService {

package/product-service.js

@@ -76,25 +76,25 @@ export class ProductService {
             }
             port = allocated;
             logger.info(`Starting product container: ${spec.image} on host port ${port}`);
-            containerId = await dockerRun(spec.image, port);
+            containerId = await this.containerOps.run(spec.image, port);
             baseUrl = specBaseUrl(spec, port);
         }
         try {
-            await waitForReady(baseUrl);
+            await this.containerOps.waitForReady(baseUrl);
         }
         catch (e) {
             if (containerId)
-                await dockerRm(containerId);
+                await this.containerOps.remove(containerId);
             throw e;
         }
         const manifest = await fetchManifest(baseUrl).catch(async (e) => {
             if (containerId)
-                await dockerRm(containerId);
+                await this.containerOps.remove(containerId);
             throw e;
         });
         if (existing.some((p) => p.name === manifest.name)) {
             if (containerId)
-                await dockerRm(containerId);
+                await this.containerOps.remove(containerId);
             throw new ProductError("NAME_CONFLICT", `product '${manifest.name}' already registered`);
         }
         const warnings = [];
@@ -111,7 +111,7 @@ export class ProductService {
             licenseFile = resolved.file;
             if (resolved.fatal) {
                 if (containerId)
-                    await dockerRm(containerId);
+                    await this.containerOps.remove(containerId);
                 throw new ProductError("LICENSE_INVALID", resolved.fatal);
             }
             if (resolved.warning)
@@ -137,19 +137,19 @@ export class ProductService {
             const parsed = contents !== undefined ? parseLicenseContents(contents) : undefined;
             if (parsed?.kind === "invalid-v2") {
                 if (containerId)
-                    await dockerRm(containerId);
+                    await this.containerOps.remove(containerId);
                 throw new ProductError("LICENSE_INVALID", parsed.reason);
             }
             if (parsed?.kind === "legacy") {
                 if (containerId)
-                    await dockerRm(containerId);
+                    await this.containerOps.remove(containerId);
                 throw new ProductError("LICENSE_INVALID", notV2EnvelopeMessage("V1 licenses are no longer accepted at registration"));
             }
             if (parsed?.kind === "v2") {
                 const check = checkProductEntitlement(parsed, manifest.name, spec.kind === "container" ? spec.image : undefined);
                 if (!check.ok) {
                     if (containerId)
-                        await dockerRm(containerId);
+                        await this.containerOps.remove(containerId);
                     throw new ProductError("LICENSE_INVALID", check.reason);
                 }
                 warnings.push(...check.warnings);
@@ -159,7 +159,7 @@ export class ProductService {
         // stable, human-readable name (replacing docker's random alias). Tracked
         // by id, so a rename failure is harmless — hence best-effort.
         if (containerId)
-            await dockerRename(containerId, productContainerName(manifest.name));
+            await this.containerOps.rename(containerId, productContainerName(manifest.name));
         // Sanity-probe the config screen so we catch (a) URLs that don't
         // serve anything at the manifest-declared path, and (b) the common
         // "registered the vite dev port" mistake — both are hard rejections
@@ -170,7 +170,7 @@ export class ProductService {
             }
             catch (e) {
                 if (containerId)
-                    await dockerRm(containerId);
+                    await this.containerOps.remove(containerId);
                 throw e;
             }
         }
@@ -227,7 +227,7 @@ export class ProductService {
         if (!target)
             throw new ProductError("NOT_FOUND", `product '${name}' not registered`);
         if (target.containerId)
-            await dockerRm(target.containerId);
+            await this.containerOps.remove(target.containerId);
         await this.store.update((products) => products.filter((p) => p.name !== name));
         logger.info(`Product '${name}' removed`);
     }