@camstack/addon-pipeline 1.2.41 → 1.2.42

package/dist/{event-loop-stall-monitor-a2u9e1qx.mjs → event-loop-stall-monitor-CWOJtTNc.mjs}

@@ -1,4 +1,4 @@
-import { D as hfModelUrl, a as COCO_TO_MACRO, i as COCO_80_LABELS, r as AUDIO_MACRO_LABELS } from "./dist-DvUTPZ8J.mjs";
+import { a as COCO_TO_MACRO, i as COCO_80_LABELS, k as hfModelUrl, r as AUDIO_MACRO_LABELS } from "./dist-COK_1Ot8.mjs";
 import { PerformanceObserver } from "node:perf_hooks";
 //#region src/detection-pipeline/registry/model-catalogs.ts
 var HF_REPO = "camstack/camstack-models";

package/dist/{event-loop-stall-monitor-C5P2bcwP.js → event-loop-stall-monitor-DNFRgNef.js}

@@ -1,4 +1,4 @@
-const require_dist = require("./dist-BS7wLqoV.js");
+const require_dist = require("./dist-BdDd3ql_.js");
 let node_perf_hooks = require("node:perf_hooks");
 //#region src/detection-pipeline/registry/model-catalogs.ts
 var HF_REPO = "camstack/camstack-models";

package/dist/motion-wasm/index.js

@@ -2,7 +2,7 @@ Object.defineProperties(exports, {
 	__esModule: { value: true },
 	[Symbol.toStringTag]: { value: "Module" }
 });
-const require_dist = require("../dist-BS7wLqoV.js");
+const require_dist = require("../dist-BdDd3ql_.js");
 const require_lazy_sharp = require("../lazy-sharp-CcYuXtsa.js");
 let node_fs = require("node:fs");
 let node_path = require("node:path");

package/dist/motion-wasm/index.mjs

@@ -1,4 +1,4 @@
-import { A as motionDetectionCapability, E as evaluateZoneRules, K as hydrateSchema, V as BaseAddon, W as DeviceType } from "../dist-DvUTPZ8J.mjs";
+import { M as motionDetectionCapability, O as evaluateZoneRules, W as BaseAddon, Y as hydrateSchema, q as DeviceType } from "../dist-COK_1Ot8.mjs";
 import { t as getSharp } from "../lazy-sharp-B-mb8uWx.mjs";
 import { readFileSync } from "node:fs";
 import { join } from "node:path";

package/dist/pipeline-runner/index.js

@@ -3,8 +3,8 @@ Object.defineProperties(exports, {
 	[Symbol.toStringTag]: { value: "Module" }
 });
 const require_chunk = require("../chunk-emK7D4bc.js");
-const require_dist = require("../dist-BS7wLqoV.js");
-const require_event_loop_stall_monitor = require("../event-loop-stall-monitor-C5P2bcwP.js");
+const require_dist = require("../dist-BdDd3ql_.js");
+const require_event_loop_stall_monitor = require("../event-loop-stall-monitor-DNFRgNef.js");
 const require_worker_protocol = require("../worker-protocol-DextwlTX.js");
 const require_hub_hostname = require("../hub-hostname-DAJXlOgV.js");
 const require_lazy_sharp = require("../lazy-sharp-CcYuXtsa.js");
@@ -1687,27 +1687,108 @@ function startStallSampler(opts = {}) {
 	};
 }
 //#endregion
-//#region src/pipeline-runner/detail-subtree.ts
-/** Relative padding applied to the parent bbox before cutting the crop — more context for the child model. */
-var DETAIL_CROP_PADDING_RATIO = .15;
+//#region src/pipeline-runner/crop-convention-source.ts
 /**
-* Pad `bbox` by {@link DETAIL_CROP_PADDING_RATIO} of its own size, then clamp
-* to `[0, frameWidth] x [0, frameHeight]`. Pure — always returns a new rect.
+* Where the runner gets the ONE detail-crop convention.
+*
+* ## Why the runner, and only the runner
+*
+* Every detail crop is now cut inside `runDetailSubtree` — the live plane from
+* a frame handle, the embedding rebuild from a stored key frame it ships in as
+* `frameJpeg`. One cutter means one reader: nothing else needs the value, so
+* nothing else can drift from it.
+*
+* ## Why a settings read and not the attach config
+*
+* `RunnerCameraConfig` is the obvious orchestrator→runner channel, and it is
+* the wrong one here. Eleven separate call sites construct a
+* `RunnerCameraConfig` (session dispatch, standing dispatch, zone refresh,
+* node reconnect, placement correction, …). A cluster-wide constant threaded
+* through eleven builders is a constant that eventually reaches ten of them,
+* and the failure is invisible: the cameras attached by the forgotten builder
+* quietly cut a different crop and write it into the same index. That is the
+* bug this whole change exists to remove, so it must not be reintroduced as
+* the delivery mechanism.
+*
+* Instead the runner reads the OWNER addon's global settings directly, the way
+* both decoder addons read their backend selector
+* (`decoder-backend.ts`) — hub-routed by `AddonCallGateway`, so an agent runner
+* transparently reads the hub-central store with no placement branching.
+*
+* ## Deliberately NOT node-scoped
+*
+* The read passes no `nodeId`. With one, `addon-settings` would project that
+* node's scoped value and two nodes could cut different crops into the same
+* `object-clip` index — the original defect, relocated to the transport.
 */
-function padAndClampFrameBbox(bbox, frameWidth, frameHeight) {
-	const padX = DETAIL_CROP_PADDING_RATIO * bbox.w;
-	const padY = DETAIL_CROP_PADDING_RATIO * bbox.h;
-	const x1 = Math.max(0, bbox.x - padX);
-	const y1 = Math.max(0, bbox.y - padY);
-	const x2 = Math.min(frameWidth, bbox.x + bbox.w + padX);
-	const y2 = Math.min(frameHeight, bbox.y + bbox.h + padY);
-	return {
-		x: x1,
-		y: y1,
-		w: Math.max(0, x2 - x1),
-		h: Math.max(0, y2 - y1)
-	};
-}
+/**
+* The addon whose GLOBAL settings own the convention. `pipeline-orchestrator`
+* is hub-resident and neutral — it already owns the per-node decoder-backend
+* selector for the same reason (neither consumer should own a value the other
+* depends on).
+*/
+var CROP_CONVENTION_OWNER_ADDON_ID = "pipeline-orchestrator";
+/**
+* TTL-cached read of the cluster crop convention.
+*
+* {@link current} is synchronous because the crop derivation is, and because a
+* value that is never missing is easier to reason about than one that is
+* sometimes a promise. {@link refresh} is what actually talks to the hub;
+* the runner awaits it at the top of a detail dispatch.
+*/
+var CropConventionSource = class {
+	convention = require_dist.DEFAULT_DETAIL_CROP_CONVENTION;
+	lastReadAtMs = Number.NEGATIVE_INFINITY;
+	inFlight = null;
+	logger;
+	now;
+	ttlMs;
+	constructor(options) {
+		this.logger = options.logger;
+		this.now = options.now ?? (() => Date.now());
+		this.ttlMs = options.ttlMs ?? 6e4;
+	}
+	/** The convention in force right now — the default until a read lands. */
+	current() {
+		return this.convention;
+	}
+	/**
+	* Re-read if the cached value has expired. Concurrent callers share one
+	* in-flight read; a failure keeps the LAST KNOWN value rather than reverting
+	* to the default, because a mid-pass revert would silently change what every
+	* remaining vector of that pass means.
+	*/
+	async refresh(api) {
+		if (api === void 0) return this.convention;
+		if (this.now() - this.lastReadAtMs < this.ttlMs) return this.convention;
+		this.inFlight ??= this.read(api).finally(() => {
+			this.inFlight = null;
+		});
+		await this.inFlight;
+		return this.convention;
+	}
+	async read(api) {
+		try {
+			const next = require_dist.pickDetailCropConvention(await api.addonSettings.getGlobalSettings.query({ addonId: CROP_CONVENTION_OWNER_ADDON_ID }));
+			this.lastReadAtMs = this.now();
+			if (next.paddingRatio !== this.convention.paddingRatio || next.square !== this.convention.square) {
+				this.logger.info("detail crop convention changed — stored vectors need a rebuild", { meta: {
+					from: { ...this.convention },
+					to: { ...next }
+				} });
+				this.convention = next;
+			}
+		} catch (err) {
+			this.logger.warn("detail crop convention read failed — keeping the last known value", { meta: {
+				owner: CROP_CONVENTION_OWNER_ADDON_ID,
+				inUse: { ...this.convention },
+				error: err instanceof Error ? err.message : String(err)
+			} });
+		}
+	}
+};
+//#endregion
+//#region src/pipeline-runner/detail-subtree.ts
 /** Normalize a frame-space pixel rect to `[0,1]` by the given frame dims. */
 function normalizeRect(rect, frameWidth, frameHeight) {
 	return {
@@ -1806,15 +1887,52 @@ function toDetailResult(detection, stepId, crop) {
 		...detection.nativeFaceShortSidePx !== void 0 ? { nativeFaceShortSidePx: detection.nativeFaceShortSidePx } : {}
 	};
 }
-async function resolveCrop(deps, input) {
+/**
+* Cut the crop out of a caller-supplied FULL FRAME, applying the one crop
+* convention. This is the branch that made the rebuild comparable with live
+* traffic: before it existed the only way to feed stored pixels in was a
+* pre-cut tile, so the caller decided the rectangle and every caller decided
+* differently.
+*/
+async function cutFromFullFrame(frameJpegBase64, bbox, convention) {
+	const image = (await require_lazy_sharp.getSharp())(Buffer.from(frameJpegBase64, "base64"));
+	const meta = await image.metadata();
+	const frameWidth = meta.width ?? 0;
+	const frameHeight = meta.height ?? 0;
+	if (frameWidth <= 0 || frameHeight <= 0) return null;
+	const rect = require_dist.deriveDetailCropRect(bbox, frameWidth, frameHeight, convention);
+	const left = Math.max(0, Math.min(Math.round(rect.x), frameWidth - 1));
+	const top = Math.max(0, Math.min(Math.round(rect.y), frameHeight - 1));
+	const width = Math.max(1, Math.min(Math.round(rect.w), frameWidth - left));
+	const height = Math.max(1, Math.min(Math.round(rect.h), frameHeight - top));
+	return {
+		jpeg: await image.extract({
+			left,
+			top,
+			width,
+			height
+		}).jpeg({ quality: 90 }).toBuffer(),
+		width,
+		height,
+		frameSpace: {
+			x: left,
+			y: top,
+			w: width,
+			h: height
+		},
+		source: "derived"
+	};
+}
+async function resolveCrop(deps, input, convention) {
 	if (input.frameHandle) {
-		const normalized = normalizeRect(padAndClampFrameBbox(input.parent.bbox, input.frameHandle.width, input.frameHandle.height), input.frameHandle.width, input.frameHandle.height);
+		const normalized = normalizeRect(require_dist.deriveDetailCropRect(input.parent.bbox, input.frameHandle.width, input.frameHandle.height, convention), input.frameHandle.width, input.frameHandle.height);
 		const native = await deps.getNativeCrop(input.deviceId, input.frameHandle, normalized);
 		if (native) return {
 			...native,
 			source: "native"
 		};
 	}
+	if (input.frameJpeg) return cutFromFullFrame(input.frameJpeg, input.parent.bbox, convention);
 	if (input.cropJpeg) {
 		const jpeg = Buffer.from(input.cropJpeg, "base64");
 		const meta = await (await require_lazy_sharp.getSharp())(jpeg).metadata();
@@ -1849,11 +1967,42 @@ async function resolveCrop(deps, input) {
 */
 async function runDetailSubtree(deps, input) {
 	const matched = deps.resolveChildSteps(input.deviceId, input.parent.className, input.steps);
-	if (!matched || matched.length === 0) return null;
+	if (!matched) {
+		deps.logger.warn("runDetailSubtree: camera is not attached — no detail steps to run", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				...input.steps ? { requestedSteps: [...input.steps] } : {}
+			}
+		});
+		return null;
+	}
+	if (matched.length === 0) {
+		deps.logger.warn("runDetailSubtree: no matching detail step is enabled for this parent class", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				...input.steps ? { requestedSteps: [...input.steps] } : {},
+				hint: "the step is either absent from this camera step tree or disabled on it"
+			}
+		});
+		return null;
+	}
 	const requested = input.steps;
 	const steps = requested === void 0 ? matched : matched.map((step) => pruneChildStepsToRequested(step, requested));
-	const crop = await resolveCrop(deps, input);
-	if (!crop) return null;
+	const crop = await resolveCrop(deps, input, deps.resolveCropConvention(input.deviceId));
+	if (!crop) {
+		deps.logger.warn("runDetailSubtree: no crop source resolved — the dispatch produced nothing", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				hadFrameHandle: input.frameHandle !== void 0,
+				hadFrameJpeg: input.frameJpeg !== void 0,
+				hadCropJpeg: input.cropJpeg !== void 0
+			}
+		});
+		return null;
+	}
 	const nativeCropRef = input.frameHandle !== void 0 && crop.source === "native" ? {
 		handle: input.frameHandle,
 		cropFrameSpace: crop.frameSpace
@@ -4605,14 +4754,30 @@ var PipelineRunnerAddon = class PipelineRunnerAddon extends require_dist.BaseAdd
 	* the frame's owning process), and the node-local `pipelineExecutor`.
 	*/
 	async runDetailSubtree(input) {
+		const convention = await this.cropConvention.refresh(this.ctxIfReady?.api);
 		return runDetailSubtree({
 			resolveChildSteps: (deviceId, parentClassName, filter) => this.resolveDetailChildSteps(deviceId, parentClassName, filter),
 			getNativeCrop: (deviceId, handle, paddedBbox) => this.getNativeCropForDetail(deviceId, handle, paddedBbox),
 			runPipeline: (steps, imageJpeg, deviceId, deviceKey, nativeCropRef) => this.runDetailPipeline(steps, imageJpeg, deviceId, deviceKey, nativeCropRef),
-			resolveStepDevice: (deviceId, step) => this.resolveDetailStepDevice(deviceId, step)
+			resolveStepDevice: (deviceId, step) => this.resolveDetailStepDevice(deviceId, step),
+			resolveCropConvention: () => convention,
+			logger: this.detailSubtreeLogger()
 		}, input);
 	}
 	/**
+	* The cluster-wide detail-crop convention — read from the orchestrator's
+	* global settings, NOT from the attach config. See `crop-convention-source.ts`
+	* for why eleven attach builders make the attach config the wrong carrier.
+	*/
+	cropConvention = new CropConventionSource({ logger: {
+		info: (message, extras) => this.ctxIfReady?.logger.info(message, extras),
+		warn: (message, extras) => this.ctxIfReady?.logger.warn(message, extras)
+	} });
+	/** A warn sink that survives a not-yet-ready ctx (boot-order dispatches). */
+	detailSubtreeLogger() {
+		return { warn: (message, extras) => this.ctxIfReady?.logger.warn(message, extras) };
+	}
+	/**
 	* Step-tree device jump (phase 1) — pick which same-node inference device an
 	* enrichment `step` runs on. AUTO-jumps only when the camera's elected device
 	* cannot run the step's model (its format has no build); an operator

package/dist/pipeline-runner/index.mjs

@@ -1,5 +1,5 @@
-import { B as errMsg, G as createEvent, M as pipelineRunnerCapability, S as defineCustomActions, V as BaseAddon, X as nodePin, at as lazy, b as customAction, ct as object, ft as EventCategory, nt as array, ot as literal, rt as boolean, s as DEVICE_BACKEND_TO_FORMAT, st as number, tt as _enum, ut as string } from "../dist-DvUTPZ8J.mjs";
-import { c as getStepDefinition, t as attributeStall } from "../event-loop-stall-monitor-a2u9e1qx.mjs";
+import { $ as nodePin, C as defineCustomActions, F as pipelineRunnerCapability, J as createEvent, N as pickDetailCropConvention, U as errMsg, W as BaseAddon, at as array, c as DEVICE_BACKEND_TO_FORMAT, ct as lazy, dt as object, ht as EventCategory, it as _enum, lt as literal, ot as boolean, pt as string, s as DEFAULT_DETAIL_CROP_CONVENTION, ut as number, w as deriveDetailCropRect, x as customAction } from "../dist-COK_1Ot8.mjs";
+import { c as getStepDefinition, t as attributeStall } from "../event-loop-stall-monitor-CWOJtTNc.mjs";
 import { t as isWorkerReply } from "../worker-protocol-D7RzZIla.mjs";
 import { t as resolveHubHostname } from "../hub-hostname-cCknRYKj.mjs";
 import { t as getSharp } from "../lazy-sharp-B-mb8uWx.mjs";
@@ -1679,27 +1679,108 @@ function startStallSampler(opts = {}) {
 	};
 }
 //#endregion
-//#region src/pipeline-runner/detail-subtree.ts
-/** Relative padding applied to the parent bbox before cutting the crop — more context for the child model. */
-var DETAIL_CROP_PADDING_RATIO = .15;
+//#region src/pipeline-runner/crop-convention-source.ts
 /**
-* Pad `bbox` by {@link DETAIL_CROP_PADDING_RATIO} of its own size, then clamp
-* to `[0, frameWidth] x [0, frameHeight]`. Pure — always returns a new rect.
+* Where the runner gets the ONE detail-crop convention.
+*
+* ## Why the runner, and only the runner
+*
+* Every detail crop is now cut inside `runDetailSubtree` — the live plane from
+* a frame handle, the embedding rebuild from a stored key frame it ships in as
+* `frameJpeg`. One cutter means one reader: nothing else needs the value, so
+* nothing else can drift from it.
+*
+* ## Why a settings read and not the attach config
+*
+* `RunnerCameraConfig` is the obvious orchestrator→runner channel, and it is
+* the wrong one here. Eleven separate call sites construct a
+* `RunnerCameraConfig` (session dispatch, standing dispatch, zone refresh,
+* node reconnect, placement correction, …). A cluster-wide constant threaded
+* through eleven builders is a constant that eventually reaches ten of them,
+* and the failure is invisible: the cameras attached by the forgotten builder
+* quietly cut a different crop and write it into the same index. That is the
+* bug this whole change exists to remove, so it must not be reintroduced as
+* the delivery mechanism.
+*
+* Instead the runner reads the OWNER addon's global settings directly, the way
+* both decoder addons read their backend selector
+* (`decoder-backend.ts`) — hub-routed by `AddonCallGateway`, so an agent runner
+* transparently reads the hub-central store with no placement branching.
+*
+* ## Deliberately NOT node-scoped
+*
+* The read passes no `nodeId`. With one, `addon-settings` would project that
+* node's scoped value and two nodes could cut different crops into the same
+* `object-clip` index — the original defect, relocated to the transport.
 */
-function padAndClampFrameBbox(bbox, frameWidth, frameHeight) {
-	const padX = DETAIL_CROP_PADDING_RATIO * bbox.w;
-	const padY = DETAIL_CROP_PADDING_RATIO * bbox.h;
-	const x1 = Math.max(0, bbox.x - padX);
-	const y1 = Math.max(0, bbox.y - padY);
-	const x2 = Math.min(frameWidth, bbox.x + bbox.w + padX);
-	const y2 = Math.min(frameHeight, bbox.y + bbox.h + padY);
-	return {
-		x: x1,
-		y: y1,
-		w: Math.max(0, x2 - x1),
-		h: Math.max(0, y2 - y1)
-	};
-}
+/**
+* The addon whose GLOBAL settings own the convention. `pipeline-orchestrator`
+* is hub-resident and neutral — it already owns the per-node decoder-backend
+* selector for the same reason (neither consumer should own a value the other
+* depends on).
+*/
+var CROP_CONVENTION_OWNER_ADDON_ID = "pipeline-orchestrator";
+/**
+* TTL-cached read of the cluster crop convention.
+*
+* {@link current} is synchronous because the crop derivation is, and because a
+* value that is never missing is easier to reason about than one that is
+* sometimes a promise. {@link refresh} is what actually talks to the hub;
+* the runner awaits it at the top of a detail dispatch.
+*/
+var CropConventionSource = class {
+	convention = DEFAULT_DETAIL_CROP_CONVENTION;
+	lastReadAtMs = Number.NEGATIVE_INFINITY;
+	inFlight = null;
+	logger;
+	now;
+	ttlMs;
+	constructor(options) {
+		this.logger = options.logger;
+		this.now = options.now ?? (() => Date.now());
+		this.ttlMs = options.ttlMs ?? 6e4;
+	}
+	/** The convention in force right now — the default until a read lands. */
+	current() {
+		return this.convention;
+	}
+	/**
+	* Re-read if the cached value has expired. Concurrent callers share one
+	* in-flight read; a failure keeps the LAST KNOWN value rather than reverting
+	* to the default, because a mid-pass revert would silently change what every
+	* remaining vector of that pass means.
+	*/
+	async refresh(api) {
+		if (api === void 0) return this.convention;
+		if (this.now() - this.lastReadAtMs < this.ttlMs) return this.convention;
+		this.inFlight ??= this.read(api).finally(() => {
+			this.inFlight = null;
+		});
+		await this.inFlight;
+		return this.convention;
+	}
+	async read(api) {
+		try {
+			const next = pickDetailCropConvention(await api.addonSettings.getGlobalSettings.query({ addonId: CROP_CONVENTION_OWNER_ADDON_ID }));
+			this.lastReadAtMs = this.now();
+			if (next.paddingRatio !== this.convention.paddingRatio || next.square !== this.convention.square) {
+				this.logger.info("detail crop convention changed — stored vectors need a rebuild", { meta: {
+					from: { ...this.convention },
+					to: { ...next }
+				} });
+				this.convention = next;
+			}
+		} catch (err) {
+			this.logger.warn("detail crop convention read failed — keeping the last known value", { meta: {
+				owner: CROP_CONVENTION_OWNER_ADDON_ID,
+				inUse: { ...this.convention },
+				error: err instanceof Error ? err.message : String(err)
+			} });
+		}
+	}
+};
+//#endregion
+//#region src/pipeline-runner/detail-subtree.ts
 /** Normalize a frame-space pixel rect to `[0,1]` by the given frame dims. */
 function normalizeRect(rect, frameWidth, frameHeight) {
 	return {
@@ -1798,15 +1879,52 @@ function toDetailResult(detection, stepId, crop) {
 		...detection.nativeFaceShortSidePx !== void 0 ? { nativeFaceShortSidePx: detection.nativeFaceShortSidePx } : {}
 	};
 }
-async function resolveCrop(deps, input) {
+/**
+* Cut the crop out of a caller-supplied FULL FRAME, applying the one crop
+* convention. This is the branch that made the rebuild comparable with live
+* traffic: before it existed the only way to feed stored pixels in was a
+* pre-cut tile, so the caller decided the rectangle and every caller decided
+* differently.
+*/
+async function cutFromFullFrame(frameJpegBase64, bbox, convention) {
+	const image = (await getSharp())(Buffer.from(frameJpegBase64, "base64"));
+	const meta = await image.metadata();
+	const frameWidth = meta.width ?? 0;
+	const frameHeight = meta.height ?? 0;
+	if (frameWidth <= 0 || frameHeight <= 0) return null;
+	const rect = deriveDetailCropRect(bbox, frameWidth, frameHeight, convention);
+	const left = Math.max(0, Math.min(Math.round(rect.x), frameWidth - 1));
+	const top = Math.max(0, Math.min(Math.round(rect.y), frameHeight - 1));
+	const width = Math.max(1, Math.min(Math.round(rect.w), frameWidth - left));
+	const height = Math.max(1, Math.min(Math.round(rect.h), frameHeight - top));
+	return {
+		jpeg: await image.extract({
+			left,
+			top,
+			width,
+			height
+		}).jpeg({ quality: 90 }).toBuffer(),
+		width,
+		height,
+		frameSpace: {
+			x: left,
+			y: top,
+			w: width,
+			h: height
+		},
+		source: "derived"
+	};
+}
+async function resolveCrop(deps, input, convention) {
 	if (input.frameHandle) {
-		const normalized = normalizeRect(padAndClampFrameBbox(input.parent.bbox, input.frameHandle.width, input.frameHandle.height), input.frameHandle.width, input.frameHandle.height);
+		const normalized = normalizeRect(deriveDetailCropRect(input.parent.bbox, input.frameHandle.width, input.frameHandle.height, convention), input.frameHandle.width, input.frameHandle.height);
 		const native = await deps.getNativeCrop(input.deviceId, input.frameHandle, normalized);
 		if (native) return {
 			...native,
 			source: "native"
 		};
 	}
+	if (input.frameJpeg) return cutFromFullFrame(input.frameJpeg, input.parent.bbox, convention);
 	if (input.cropJpeg) {
 		const jpeg = Buffer.from(input.cropJpeg, "base64");
 		const meta = await (await getSharp())(jpeg).metadata();
@@ -1841,11 +1959,42 @@ async function resolveCrop(deps, input) {
 */
 async function runDetailSubtree(deps, input) {
 	const matched = deps.resolveChildSteps(input.deviceId, input.parent.className, input.steps);
-	if (!matched || matched.length === 0) return null;
+	if (!matched) {
+		deps.logger.warn("runDetailSubtree: camera is not attached — no detail steps to run", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				...input.steps ? { requestedSteps: [...input.steps] } : {}
+			}
+		});
+		return null;
+	}
+	if (matched.length === 0) {
+		deps.logger.warn("runDetailSubtree: no matching detail step is enabled for this parent class", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				...input.steps ? { requestedSteps: [...input.steps] } : {},
+				hint: "the step is either absent from this camera step tree or disabled on it"
+			}
+		});
+		return null;
+	}
 	const requested = input.steps;
 	const steps = requested === void 0 ? matched : matched.map((step) => pruneChildStepsToRequested(step, requested));
-	const crop = await resolveCrop(deps, input);
-	if (!crop) return null;
+	const crop = await resolveCrop(deps, input, deps.resolveCropConvention(input.deviceId));
+	if (!crop) {
+		deps.logger.warn("runDetailSubtree: no crop source resolved — the dispatch produced nothing", {
+			tags: { deviceId: input.deviceId },
+			meta: {
+				parentClass: input.parent.className,
+				hadFrameHandle: input.frameHandle !== void 0,
+				hadFrameJpeg: input.frameJpeg !== void 0,
+				hadCropJpeg: input.cropJpeg !== void 0
+			}
+		});
+		return null;
+	}
 	const nativeCropRef = input.frameHandle !== void 0 && crop.source === "native" ? {
 		handle: input.frameHandle,
 		cropFrameSpace: crop.frameSpace
@@ -4597,14 +4746,30 @@ var PipelineRunnerAddon = class PipelineRunnerAddon extends BaseAddon {
 	* the frame's owning process), and the node-local `pipelineExecutor`.
 	*/
 	async runDetailSubtree(input) {
+		const convention = await this.cropConvention.refresh(this.ctxIfReady?.api);
 		return runDetailSubtree({
 			resolveChildSteps: (deviceId, parentClassName, filter) => this.resolveDetailChildSteps(deviceId, parentClassName, filter),
 			getNativeCrop: (deviceId, handle, paddedBbox) => this.getNativeCropForDetail(deviceId, handle, paddedBbox),
 			runPipeline: (steps, imageJpeg, deviceId, deviceKey, nativeCropRef) => this.runDetailPipeline(steps, imageJpeg, deviceId, deviceKey, nativeCropRef),
-			resolveStepDevice: (deviceId, step) => this.resolveDetailStepDevice(deviceId, step)
+			resolveStepDevice: (deviceId, step) => this.resolveDetailStepDevice(deviceId, step),
+			resolveCropConvention: () => convention,
+			logger: this.detailSubtreeLogger()
 		}, input);
 	}
 	/**
+	* The cluster-wide detail-crop convention — read from the orchestrator's
+	* global settings, NOT from the attach config. See `crop-convention-source.ts`
+	* for why eleven attach builders make the attach config the wrong carrier.
+	*/
+	cropConvention = new CropConventionSource({ logger: {
+		info: (message, extras) => this.ctxIfReady?.logger.info(message, extras),
+		warn: (message, extras) => this.ctxIfReady?.logger.warn(message, extras)
+	} });
+	/** A warn sink that survives a not-yet-ready ctx (boot-order dispatches). */
+	detailSubtreeLogger() {
+		return { warn: (message, extras) => this.ctxIfReady?.logger.warn(message, extras) };
+	}
+	/**
 	* Step-tree device jump (phase 1) — pick which same-node inference device an
 	* enrichment `step` runs on. AUTO-jumps only when the camera's elected device
 	* cannot run the step's model (its format has no build); an operator

package/dist/recorder/index.js

@@ -1,5 +1,5 @@
 const require_chunk = require("../chunk-emK7D4bc.js");
-const require_dist = require("../dist-BS7wLqoV.js");
+const require_dist = require("../dist-BdDd3ql_.js");
 const require_hub_hostname = require("../hub-hostname-DAJXlOgV.js");
 const require_model_download_service_Cp9f4dk6 = require("../model-download-service-Cp9f4dk6-fsdDExML.js");
 let node_child_process = require("node:child_process");

package/dist/recorder/index.mjs

@@ -1,4 +1,4 @@
-import { $ as selectAssignedProfileSlots, B as errMsg, C as deriveRecordingMode, I as storageEvictableCapability, K as hydrateSchema, N as recordingCapability, P as recordingExportCapability, V as BaseAddon, W as DeviceType, X as nodePin, c as EVENT_PAD_MS, f as OpsLogEntrySchema, ft as EventCategory, lt as record, nt as array, p as RecordingConfigSchema, u as ExportRecordSchema, ut as string } from "../dist-DvUTPZ8J.mjs";
+import { $ as nodePin, I as recordingCapability, L as recordingExportCapability, T as deriveRecordingMode, U as errMsg, W as BaseAddon, Y as hydrateSchema, at as array, d as ExportRecordSchema, ft as record, ht as EventCategory, l as EVENT_PAD_MS, m as RecordingConfigSchema, nt as selectAssignedProfileSlots, p as OpsLogEntrySchema, pt as string, q as DeviceType, z as storageEvictableCapability } from "../dist-COK_1Ot8.mjs";
 import { t as resolveHubHostname } from "../hub-hostname-cCknRYKj.mjs";
 import { n as createFileDataPlaneHandler, s as parseRangeHeader, t as contentTypeFor } from "../model-download-service-Cp9f4dk6-CwUb5v3Y.mjs";
 import { spawn } from "node:child_process";

package/dist/stream-broker/_stub.js

@@ -1,8 +1,8 @@
-import { a as e, c as t, d as n, i as r, l as i, n as a, o, r as s, s as c, t as l, u } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare___mf_0_camstack_mf_1_ui_mf_2_library__loadShare__.js-CciPNnL7.mjs";
+import { a as e, c as t, d as n, i as r, l as i, n as a, o, r as s, s as c, t as l, u } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare___mf_0_camstack_mf_1_ui_mf_2_library__loadShare__.js-DAjAxXHm.mjs";
 import { a as d, i as f, n as p, o as m, r as h, t as g } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare__react__loadShare__.js-C9j-2lBe.mjs";
 import { n as _, r as v, t as y } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare__react_mf_1_jsx_mf_2_runtime__loadShare__.js-XO0-Pyu6.mjs";
 import { n as b, t as x } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare___mf_0_tanstack_mf_1_react_mf_2_query__loadShare__.js-BO7TIbJV.mjs";
-import { t as S } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare___mf_0_camstack_mf_1_types__loadShare__.js-CDzxDzcv.mjs";
+import { t as S } from "./_virtual_mf___mfe_internal__addon_stream_broker_widgets__loadShare___mf_0_camstack_mf_1_types__loadShare__.js-SU_6F3Y7.mjs";
 //#region ../../node_modules/lucide-react/dist/esm/shared/src/utils.js
 var C = (e) => e.replace(/([a-z0-9])([A-Z])/g, "$1-$2").toLowerCase(), w = (e) => e.replace(/^([A-Z])|[\s-_]+(\w)/g, (e, t, n) => n ? n.toUpperCase() : t.toLowerCase()), T = (e) => {
 	let t = w(e);

package/dist/stream-broker/{_virtual_mf-localSharedImportMap___mfe_internal__addon_stream_broker_widgets-CS82x0lt.mjs → _virtual_mf-localSharedImportMap___mfe_internal__addon_stream_broker_widgets-C8ovZN0g.mjs}

@@ -18,7 +18,7 @@ var e = {
 	},
 	"@camstack/types": {
 		name: "@camstack/types",
-		version: "1.2.31",
+		version: "1.2.41",
 		scope: ["default"],
 		loaded: !1,
 		from: "addon_stream_broker_widgets",
@@ -33,7 +33,7 @@ var e = {
 	},
 	"@camstack/ui-library": {
 		name: "@camstack/ui-library",
-		version: "1.2.25",
+		version: "1.2.29",
 		scope: ["default"],
 		loaded: !1,
 		from: "addon_stream_broker_widgets",