@norskvideo/ctl-test-harness 0.1.59 → 0.1.61

package/container-logs.d.ts

@@ -28,6 +28,17 @@ export declare function formatContainerLogs(instanceId: string, entries: Contain
 export declare function collectInstanceContainerLogs(instanceId: string, tail?: number, spawn?: SpawnLike): ContainerLog[];
 /** The ready-to-print diagnostic block (empty-marker when nothing is found). */
 export declare function dumpInstanceContainerLogs(instanceId: string, tail?: number, spawn?: SpawnLike): string;
+/** Write every container of one instance to `<dest>/<container>.log`, for a CI
+ *  artifact. The companion to {@link preserveEngineLogs}, which keeps the media
+ *  container's bind-mounted engine tree: this keeps what ONLY Docker has, and
+ *  the studio container's stdout is where "Refusing to run workflow" is printed
+ *  — the line the gate failures point at and that teardown then destroys.
+ *
+ *  Call BEFORE cleanup. Best-effort throughout and never throws: a diagnostic
+ *  that fails the run it was meant to explain is worse than no diagnostic.
+ *  Returns the number of logs written, and creates nothing when there are none.
+ */
+export declare function preserveInstanceContainerLogs(instanceId: string, dest: string, tail?: number, spawn?: SpawnLike): number;
 export interface LogHit {
     container: string;
     line: string;

package/container-logs.js

@@ -8,7 +8,7 @@
 // Pure argv/parse/format helpers; only the two exported probes shell out,
 // through an injectable spawn.
 import { spawnSync } from "node:child_process";
-import { cpSync, existsSync, readdirSync, statSync } from "node:fs";
+import { cpSync, existsSync, mkdirSync, readdirSync, statSync, writeFileSync } from "node:fs";
 import { join, relative } from "node:path";
 const defaultSpawn = (cmd, args, opts) => {
     const r = spawnSync(cmd, args, opts);
@@ -94,6 +94,40 @@ export function collectInstanceContainerLogs(instanceId, tail = DEFAULT_TAIL, sp
 export function dumpInstanceContainerLogs(instanceId, tail = DEFAULT_TAIL, spawn = defaultSpawn) {
     return formatContainerLogs(instanceId, collectInstanceContainerLogs(instanceId, tail, spawn));
 }
+/** Write every container of one instance to `<dest>/<container>.log`, for a CI
+ *  artifact. The companion to {@link preserveEngineLogs}, which keeps the media
+ *  container's bind-mounted engine tree: this keeps what ONLY Docker has, and
+ *  the studio container's stdout is where "Refusing to run workflow" is printed
+ *  — the line the gate failures point at and that teardown then destroys.
+ *
+ *  Call BEFORE cleanup. Best-effort throughout and never throws: a diagnostic
+ *  that fails the run it was meant to explain is worse than no diagnostic.
+ *  Returns the number of logs written, and creates nothing when there are none.
+ */
+export function preserveInstanceContainerLogs(instanceId, dest, tail = DEFAULT_TAIL, spawn = defaultSpawn) {
+    const entries = collectInstanceContainerLogs(instanceId, tail, spawn);
+    if (entries.length === 0)
+        return 0;
+    let written = 0;
+    try {
+        mkdirSync(dest, { recursive: true });
+    }
+    catch {
+        return 0;
+    }
+    for (const e of entries) {
+        try {
+            // The state goes in the file, not just the name: an `exited` container
+            // with an empty log says something a missing file does not.
+            writeFileSync(join(dest, `${e.name}.log`), `# ${e.name} (${e.state})\n${e.log}`);
+            written++;
+        }
+        catch {
+            // Keep going: one unwritable file must not cost the others.
+        }
+    }
+    return written;
+}
 /** Copy every engine `logs` tree under an instance-runtime root to `dest`,
  *  preserving the relative structure. Why this and not `docker logs`:
  *  norsk-ctl bind-mounts each media container's `/var/log/norsk` to

package/demo/run.d.ts

@@ -315,6 +315,10 @@ export declare class DemoSession {
     /** Tear down in reverse: sources, instance, then — private only — the
      *  daemon (with its proxy), the dev backend, the store. Reuse leaves the
      *  developer's daemon, store and registration as they are. */
+    /** Keep each instance's container logs under NORSK_CI_LOG_DIR for the
+     *  workflow's artifact upload. No env var (local dev) means no-op; every
+     *  failure inside is swallowed by the helper. */
+    private preserveContainerLogs;
     teardown(opts: {
         instances: string[];
         handles: SourceHandle[];

package/demo/run.js

@@ -29,6 +29,7 @@ import { existsSync, lstatSync, mkdirSync, readdirSync, readFileSync, readlinkSy
 import { homedir } from "node:os";
 import { dirname, isAbsolute, join, resolve } from "node:path";
 import { parseManifestSeed, repoOf } from "@norskvideo/ctl-sdk/manifest-seed";
+import { preserveInstanceContainerLogs } from "../container-logs.js";
 import { DOCKER_NETWORK_NAME, ensureRunnerOnNetwork, netReachMode } from "../container-net.js";
 import { cleanupDaemon, requireLicenseFile, runCli, startDaemon, } from "../daemon.js";
 import { hashSlot } from "../harness-config.js";
@@ -653,7 +654,27 @@ export class DemoSession {
     /** Tear down in reverse: sources, instance, then — private only — the
      *  daemon (with its proxy), the dev backend, the store. Reuse leaves the
      *  developer's daemon, store and registration as they are. */
+    /** Keep each instance's container logs under NORSK_CI_LOG_DIR for the
+     *  workflow's artifact upload. No env var (local dev) means no-op; every
+     *  failure inside is swallowed by the helper. */
+    preserveContainerLogs(instances) {
+        const ciLogDir = process.env.NORSK_CI_LOG_DIR;
+        if (!ciLogDir)
+            return;
+        for (const id of instances) {
+            const n = preserveInstanceContainerLogs(id, join(ciLogDir, "containers", id));
+            if (n > 0)
+                this.deps.log(`preserved ${n} container log(s) for ${id} -> ${join(ciLogDir, "containers", id)}`);
+        }
+    }
     async teardown(opts) {
+        // FIRST, while the containers are still standing. Everything below removes
+        // them, and `docker logs` is the only copy of the studio container's stdout
+        // — the "Refusing to run workflow" line that the workflow-never-started
+        // gates tell you to go and read. Unconditional rather than on-failure,
+        // because teardown is not told the verdict, and a tail per container is
+        // cheap next to a re-run.
+        this.preserveContainerLogs(opts.instances);
         if (opts.handles.length)
             await this.deps.stopSources(opts.handles).catch(() => { });
         if (this.policy === "reuse") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@norskvideo/ctl-test-harness",
-  "version": "0.1.59",
+  "version": "0.1.61",
   "type": "module",
   "exports": {
     ".": {
@@ -85,9 +85,9 @@
   "devDependencies": {
     "@types/bun": "latest",
     "@types/node": "^22.10.0",
-    "@norskvideo/norsk-studio": "1.30.0-2026-09-24-fe25440e",
-    "@norskvideo/norsk-studio-built-ins": "1.30.0-2026-09-24-fe25440e",
-    "@norskvideo/norsk-sdk": "1.3.1-2026-09-25-2aec49ef"
+    "@norskvideo/norsk-studio": "1.30.0-2026-09-28-b92e8139",
+    "@norskvideo/norsk-studio-built-ins": "1.30.0-2026-09-28-b92e8139",
+    "@norskvideo/norsk-sdk": "1.3.1-2026-09-30-acca6a9d"
   },
   "peerDependenciesMeta": {
     "@norskvideo/norsk-studio": {

package/smoke.js

@@ -19,7 +19,7 @@
 // rule); a product wanting typed commands bridges with toArgv in its own spec.
 import { existsSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
-import { dumpInstanceContainerLogs, preserveEngineLogs } from "./container-logs.js";
+import { dumpInstanceContainerLogs, preserveEngineLogs, preserveInstanceContainerLogs } from "./container-logs.js";
 import { ensureRunnerOnNetwork, netReachMode, STUDIO_INTERNAL_PORT, studioBaseFrom, } from "./container-net.js";
 import { cleanupDaemon, requireLicenseFile, runCli, startDaemon, } from "./daemon.js";
 import { hashSlot } from "./harness-config.js";
@@ -434,6 +434,11 @@ export async function runProductSmoke(slug, spec, deps = defaultSmokeDeps) {
     if (ciLogDir) {
         const n = preserveEngineLogs(storeDir, join(ciLogDir, slug));
         deps.log(`${slug}: preserved ${n} engine log tree(s) -> ${join(ciLogDir, slug)}`);
+        // And the half the bind-mounted tree does not hold: the STUDIO container's
+        // stdout, where "Refusing to run workflow" is printed. Still standing at
+        // this point; everything below removes it.
+        const c = preserveInstanceContainerLogs(instanceId, join(ciLogDir, slug, "containers", instanceId));
+        deps.log(`${slug}: preserved ${c} container log(s) -> ${join(ciLogDir, slug, "containers", instanceId)}`);
     }
     if (handles.length)
         await deps.stopSources(handles).catch(() => { });