@camstack/addon-pipeline 1.2.98 → 1.2.100

package/dist/recorder/index.js

@@ -1374,6 +1374,17 @@ function encodeDirectory(payload) {
 }
 //#endregion
 //#region src/recorder/segment-store.ts
+/**
+* Below this a file cannot be a media segment: one fMP4 `moof` + `mdat`
+* carrying a single sample already exceeds this by an order of magnitude.
+* What it actually catches is the structural stub — ffmpeg opens the next
+* segment the instant it closes the previous one, so a killed writer always
+* leaves one just-opened file (observed: 28-byte bare `styp` boxes). The boot
+* recovery plans by NAME and would otherwise index it with a nominal duration,
+* painting the timeline over footage that decodes to nothing — the same
+* failure `MAX_GAP_FACTOR` in staging-reconcile exists to prevent via gaps.
+*/
+var MIN_INDEXABLE_SEGMENT_BYTES = 64;
 var SegmentStore = class SegmentStore {
 	deps;
 	constructor(deps) {
@@ -1397,7 +1408,16 @@ var SegmentStore = class SegmentStore {
 			});
 			return null;
 		}
-		if (bytes <= 0) return null;
+		if (bytes < MIN_INDEXABLE_SEGMENT_BYTES) {
+			this.deps.logger.warn("segment too small to index — left in staging", {
+				tags: { deviceId: input.deviceId },
+				meta: {
+					bytes,
+					path: input.flatAbsPath
+				}
+			});
+			return null;
+		}
 		const relPath = segmentRelPath(input.deviceId, input.profile, input.startMs, input.durMs, bytes);
 		const toAbs = `${input.locationRoot}/${relPath}`;
 		try {
@@ -7946,12 +7966,22 @@ var RecordingController = class {
 	}
 	/** Stop one profile's writer + watcher and release its broker lease. */
 	async teardownProfile(deviceId, r) {
+		const teardownStartMs = Date.now();
 		try {
 			await r.writer.stopAndWait();
 		} catch {}
+		const writerStopMs = Date.now() - teardownStartMs;
 		try {
 			await r.watcher.flush?.();
 		} catch {}
+		this.deps.logger.info("recorder: profile teardown drained", {
+			tags: { deviceId },
+			meta: {
+				profile: r.profile,
+				writerStopMs,
+				flushMs: Date.now() - teardownStartMs - writerStopMs
+			}
+		});
 		try {
 			r.watcher.stop();
 		} catch {}
@@ -9306,6 +9336,7 @@ async function recoverAllStagedOrphans(deps) {
 		return entries;
 	};
 	const walkStartedMs = Date.now();
+	const segmentSecondsMap = await deps.segmentSecondsByDevice();
 	const work = [];
 	for (const location of locations) {
 		const stagingRoot = `${location.root}/${STAGING_DIR_NAME}`;
@@ -9316,7 +9347,7 @@ async function recoverAllStagedOrphans(deps) {
 			if (!Number.isInteger(deviceId) || deviceId <= 0) continue;
 			const profileDirs = await listPaced(`${stagingRoot}/${deviceDir}`);
 			if (profileDirs === null) continue;
-			const segmentSeconds = await deps.segmentSecondsFor(deviceId);
+			const segmentSeconds = segmentSecondsMap.get(deviceId) ?? deps.defaultSegmentSeconds;
 			for (const profile of profileDirs) {
 				if (!KNOWN_PROFILES.has(profile)) continue;
 				work.push({
@@ -10359,13 +10390,14 @@ var RecorderV2Addon = class extends require_dist.BaseAddon {
 				},
 				logger: this.ctx.logger,
 				bootMs,
-				segmentSecondsFor: async (deviceId) => {
+				segmentSecondsByDevice: async () => {
+					const out = /* @__PURE__ */ new Map();
 					try {
-						return (await loadDeviceConfig(this.configStore(), deviceId)).segmentSeconds ?? this.config.segmentSeconds;
-					} catch {
-						return this.config.segmentSeconds;
-					}
+						for (const [id, config] of await readDeviceConfigs(this.configStore())) if (config.segmentSeconds !== void 0) out.set(id, config.segmentSeconds);
+					} catch {}
+					return out;
 				},
+				defaultSegmentSeconds: this.config.segmentSeconds,
 				shouldStop: () => this.segmentStore === null
 			});
 		} catch (err) {

package/dist/recorder/index.mjs

@@ -1373,6 +1373,17 @@ function encodeDirectory(payload) {
 }
 //#endregion
 //#region src/recorder/segment-store.ts
+/**
+* Below this a file cannot be a media segment: one fMP4 `moof` + `mdat`
+* carrying a single sample already exceeds this by an order of magnitude.
+* What it actually catches is the structural stub — ffmpeg opens the next
+* segment the instant it closes the previous one, so a killed writer always
+* leaves one just-opened file (observed: 28-byte bare `styp` boxes). The boot
+* recovery plans by NAME and would otherwise index it with a nominal duration,
+* painting the timeline over footage that decodes to nothing — the same
+* failure `MAX_GAP_FACTOR` in staging-reconcile exists to prevent via gaps.
+*/
+var MIN_INDEXABLE_SEGMENT_BYTES = 64;
 var SegmentStore = class SegmentStore {
 	deps;
 	constructor(deps) {
@@ -1396,7 +1407,16 @@ var SegmentStore = class SegmentStore {
 			});
 			return null;
 		}
-		if (bytes <= 0) return null;
+		if (bytes < MIN_INDEXABLE_SEGMENT_BYTES) {
+			this.deps.logger.warn("segment too small to index — left in staging", {
+				tags: { deviceId: input.deviceId },
+				meta: {
+					bytes,
+					path: input.flatAbsPath
+				}
+			});
+			return null;
+		}
 		const relPath = segmentRelPath(input.deviceId, input.profile, input.startMs, input.durMs, bytes);
 		const toAbs = `${input.locationRoot}/${relPath}`;
 		try {
@@ -7945,12 +7965,22 @@ var RecordingController = class {
 	}
 	/** Stop one profile's writer + watcher and release its broker lease. */
 	async teardownProfile(deviceId, r) {
+		const teardownStartMs = Date.now();
 		try {
 			await r.writer.stopAndWait();
 		} catch {}
+		const writerStopMs = Date.now() - teardownStartMs;
 		try {
 			await r.watcher.flush?.();
 		} catch {}
+		this.deps.logger.info("recorder: profile teardown drained", {
+			tags: { deviceId },
+			meta: {
+				profile: r.profile,
+				writerStopMs,
+				flushMs: Date.now() - teardownStartMs - writerStopMs
+			}
+		});
 		try {
 			r.watcher.stop();
 		} catch {}
@@ -9305,6 +9335,7 @@ async function recoverAllStagedOrphans(deps) {
 		return entries;
 	};
 	const walkStartedMs = Date.now();
+	const segmentSecondsMap = await deps.segmentSecondsByDevice();
 	const work = [];
 	for (const location of locations) {
 		const stagingRoot = `${location.root}/${STAGING_DIR_NAME}`;
@@ -9315,7 +9346,7 @@ async function recoverAllStagedOrphans(deps) {
 			if (!Number.isInteger(deviceId) || deviceId <= 0) continue;
 			const profileDirs = await listPaced(`${stagingRoot}/${deviceDir}`);
 			if (profileDirs === null) continue;
-			const segmentSeconds = await deps.segmentSecondsFor(deviceId);
+			const segmentSeconds = segmentSecondsMap.get(deviceId) ?? deps.defaultSegmentSeconds;
 			for (const profile of profileDirs) {
 				if (!KNOWN_PROFILES.has(profile)) continue;
 				work.push({
@@ -10358,13 +10389,14 @@ var RecorderV2Addon = class extends BaseAddon {
 				},
 				logger: this.ctx.logger,
 				bootMs,
-				segmentSecondsFor: async (deviceId) => {
+				segmentSecondsByDevice: async () => {
+					const out = /* @__PURE__ */ new Map();
 					try {
-						return (await loadDeviceConfig(this.configStore(), deviceId)).segmentSeconds ?? this.config.segmentSeconds;
-					} catch {
-						return this.config.segmentSeconds;
-					}
+						for (const [id, config] of await readDeviceConfigs(this.configStore())) if (config.segmentSeconds !== void 0) out.set(id, config.segmentSeconds);
+					} catch {}
+					return out;
 				},
+				defaultSegmentSeconds: this.config.segmentSeconds,
 				shouldStop: () => this.segmentStore === null
 			});
 		} catch (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/addon-pipeline",
-  "version": "1.2.98",
+  "version": "1.2.100",
   "description": "Pipeline bundle — runner, detection, motion, audio + stream broker. Multi-entry npm package shipping pipeline addons under a single bundle.",
   "keywords": [
     "camstack",