@camstack/addon-post-analysis 1.2.44 → 1.2.46

package/dist/pipeline-analytics/index.js

@@ -12169,8 +12169,15 @@ async function rebuildObjectEmbeddings(deps, options = {}) {
 					failed += 1;
 					continue;
 				}
-				await deps.store(source, embedding);
-				rebuilt += 1;
+				const outcome = await deps.store(source, embedding);
+				if (outcome === "persisted") rebuilt += 1;
+				else {
+					failed += 1;
+					deps.logger.warn("embedding rebuild wrote nothing for a track", { meta: {
+						trackId,
+						outcome
+					} });
+				}
 			} catch (err) {
 				failed += 1;
 				deps.logger.warn("embedding rebuild failed for a track", { meta: {
@@ -20597,6 +20604,14 @@ var ObjectEmbeddingStore = class {
 	* write: the provider rejects an upsert into an index it has never seen. A
 	* dim change against a populated index is REFUSED rather than wiping it, so
 	* this failing is the signal that the encoder model changed.
+	*
+	* `declaredDim` caches the declaration, and that cache is per-PROCESS while
+	* the index lives in whichever addon currently provides `vector-store`. When
+	* the active provider changed under a running hub the cache stayed warm
+	* against an index that no longer existed, and every write from then on threw
+	* `index "object-clip" was never declared` — for hours, until the addon was
+	* restarted by hand. So a failed write clears the cache (see
+	* {@link upsertIfBetter}) and the next one re-declares.
 	*/
 	async ensureIndex(dim) {
 		if (this.declaredDim === dim) return true;
@@ -20621,31 +20636,56 @@ var ObjectEmbeddingStore = class {
 	* Upsert the best-confidence embedding for a track. At most ONE row per
 	* track: a write only replaces the stored row when its confidence is
 	* strictly higher, keeping the track's best crop over its lifetime.
+	*
+	* `force` BYPASSES that gate — it does not try to out-score it. A rebuild
+	* first attempted to win by writing `confidence: Infinity`, which worked
+	* exactly once: the stored row then held Infinity, `Infinity <= Infinity`
+	* short-circuited, and every later pass silently wrote nothing while
+	* reporting hundreds rebuilt. Observed live — 214 "rebuilt" moved the index
+	* by 4 rows.
+	*
+	* ## Why this returns an outcome instead of `void`
+	*
+	* It used to return `void` and turn every failure into a warn. The rebuild
+	* therefore counted a write the instant the call returned, and the second time
+	* this class of bug appeared it produced `rebuilt: 704` against `indexed: 0` —
+	* a progress number that was pure fiction while the real cause sat in a warn
+	* nobody was reading. The caller now has to distinguish "written" from "logged
+	* and dropped", because a counter that cannot reads as verification.
 	*/
 	async upsertIfBetter(input) {
-		if (input.embedding.length === 0) return;
-		if (!await this.ensureIndex(input.embedding.length)) return;
+		if (input.embedding.length === 0) return "failed";
+		if (!await this.ensureIndex(input.embedding.length)) return "failed";
 		try {
-			const stored = (await this.vectorStore.getByIds.query({
+			if (input.force !== true) {
+				const stored = (await this.vectorStore.getByIds.query({
+					index: OBJECT_CLIP_INDEX,
+					ids: [input.trackId]
+				})).items[0]?.metadata["confidence"];
+				if (typeof stored === "number" && input.confidence <= stored) return "skipped";
+			}
+			const result = await this.vectorStore.upsert.mutate({
 				index: OBJECT_CLIP_INDEX,
-				ids: [input.trackId]
-			})).items[0]?.metadata["confidence"];
-			if (typeof stored === "number" && input.confidence <= stored) return;
-			if ((await this.vectorStore.upsert.mutate({
-				index: "object-clip",
 				items: [{
 					id: input.trackId,
 					vector: require_dist.encodeVectorBase64(input.embedding),
 					metadata: metadataFor(input)
 				}]
-			})).rejected > 0) this.logger.warn("object-embedding upsert rejected", {
-				tags: { deviceId: input.deviceId },
-				meta: {
-					trackId: input.trackId,
-					dim: input.embedding.length
-				}
 			});
+			if (result.upserted === 0) {
+				this.logger.warn("object-embedding upsert rejected", {
+					tags: { deviceId: input.deviceId },
+					meta: {
+						trackId: input.trackId,
+						dim: input.embedding.length,
+						rejected: result.rejected
+					}
+				});
+				return "failed";
+			}
+			return "persisted";
 		} catch (err) {
+			this.declaredDim = null;
 			this.logger.warn("ObjectEmbeddingStore.upsertIfBetter failed", {
 				tags: { deviceId: input.deviceId },
 				meta: {
@@ -20653,6 +20693,7 @@ var ObjectEmbeddingStore = class {
 					error: String(err)
 				}
 			});
+			return "failed";
 		}
 	}
 	/**
@@ -27104,19 +27145,18 @@ var PipelineAnalyticsAddon = class extends require_dist.BaseAddon {
 					height
 				})).embedding;
 			},
-			store: async (source, embedding) => {
-				await store.upsertIfBetter({
-					trackId: source.trackId,
-					deviceId: source.deviceId,
-					timestamp: source.timestamp,
-					className: source.className,
-					embedding,
-					modelId,
-					confidence: Number.POSITIVE_INFINITY,
-					...source.mediaKey !== void 0 ? { mediaKey: source.mediaKey } : {},
-					...source.keyFrameMediaKey !== void 0 ? { keyFrameMediaKey: source.keyFrameMediaKey } : {}
-				});
-			},
+			store: async (source, embedding) => store.upsertIfBetter({
+				trackId: source.trackId,
+				deviceId: source.deviceId,
+				timestamp: source.timestamp,
+				className: source.className,
+				embedding,
+				modelId,
+				confidence: source.confidence,
+				force: true,
+				...source.mediaKey !== void 0 ? { mediaKey: source.mediaKey } : {},
+				...source.keyFrameMediaKey !== void 0 ? { keyFrameMediaKey: source.keyFrameMediaKey } : {}
+			}),
 			logger: {
 				info: (message, extras) => this.ctx.logger.info(message, extras),
 				warn: (message, extras) => this.ctx.logger.warn(message, extras)

package/dist/pipeline-analytics/index.mjs

@@ -12163,8 +12163,15 @@ async function rebuildObjectEmbeddings(deps, options = {}) {
 					failed += 1;
 					continue;
 				}
-				await deps.store(source, embedding);
-				rebuilt += 1;
+				const outcome = await deps.store(source, embedding);
+				if (outcome === "persisted") rebuilt += 1;
+				else {
+					failed += 1;
+					deps.logger.warn("embedding rebuild wrote nothing for a track", { meta: {
+						trackId,
+						outcome
+					} });
+				}
 			} catch (err) {
 				failed += 1;
 				deps.logger.warn("embedding rebuild failed for a track", { meta: {
@@ -20591,6 +20598,14 @@ var ObjectEmbeddingStore = class {
 	* write: the provider rejects an upsert into an index it has never seen. A
 	* dim change against a populated index is REFUSED rather than wiping it, so
 	* this failing is the signal that the encoder model changed.
+	*
+	* `declaredDim` caches the declaration, and that cache is per-PROCESS while
+	* the index lives in whichever addon currently provides `vector-store`. When
+	* the active provider changed under a running hub the cache stayed warm
+	* against an index that no longer existed, and every write from then on threw
+	* `index "object-clip" was never declared` — for hours, until the addon was
+	* restarted by hand. So a failed write clears the cache (see
+	* {@link upsertIfBetter}) and the next one re-declares.
 	*/
 	async ensureIndex(dim) {
 		if (this.declaredDim === dim) return true;
@@ -20615,31 +20630,56 @@ var ObjectEmbeddingStore = class {
 	* Upsert the best-confidence embedding for a track. At most ONE row per
 	* track: a write only replaces the stored row when its confidence is
 	* strictly higher, keeping the track's best crop over its lifetime.
+	*
+	* `force` BYPASSES that gate — it does not try to out-score it. A rebuild
+	* first attempted to win by writing `confidence: Infinity`, which worked
+	* exactly once: the stored row then held Infinity, `Infinity <= Infinity`
+	* short-circuited, and every later pass silently wrote nothing while
+	* reporting hundreds rebuilt. Observed live — 214 "rebuilt" moved the index
+	* by 4 rows.
+	*
+	* ## Why this returns an outcome instead of `void`
+	*
+	* It used to return `void` and turn every failure into a warn. The rebuild
+	* therefore counted a write the instant the call returned, and the second time
+	* this class of bug appeared it produced `rebuilt: 704` against `indexed: 0` —
+	* a progress number that was pure fiction while the real cause sat in a warn
+	* nobody was reading. The caller now has to distinguish "written" from "logged
+	* and dropped", because a counter that cannot reads as verification.
 	*/
 	async upsertIfBetter(input) {
-		if (input.embedding.length === 0) return;
-		if (!await this.ensureIndex(input.embedding.length)) return;
+		if (input.embedding.length === 0) return "failed";
+		if (!await this.ensureIndex(input.embedding.length)) return "failed";
 		try {
-			const stored = (await this.vectorStore.getByIds.query({
+			if (input.force !== true) {
+				const stored = (await this.vectorStore.getByIds.query({
+					index: OBJECT_CLIP_INDEX,
+					ids: [input.trackId]
+				})).items[0]?.metadata["confidence"];
+				if (typeof stored === "number" && input.confidence <= stored) return "skipped";
+			}
+			const result = await this.vectorStore.upsert.mutate({
 				index: OBJECT_CLIP_INDEX,
-				ids: [input.trackId]
-			})).items[0]?.metadata["confidence"];
-			if (typeof stored === "number" && input.confidence <= stored) return;
-			if ((await this.vectorStore.upsert.mutate({
-				index: "object-clip",
 				items: [{
 					id: input.trackId,
 					vector: encodeVectorBase64(input.embedding),
 					metadata: metadataFor(input)
 				}]
-			})).rejected > 0) this.logger.warn("object-embedding upsert rejected", {
-				tags: { deviceId: input.deviceId },
-				meta: {
-					trackId: input.trackId,
-					dim: input.embedding.length
-				}
 			});
+			if (result.upserted === 0) {
+				this.logger.warn("object-embedding upsert rejected", {
+					tags: { deviceId: input.deviceId },
+					meta: {
+						trackId: input.trackId,
+						dim: input.embedding.length,
+						rejected: result.rejected
+					}
+				});
+				return "failed";
+			}
+			return "persisted";
 		} catch (err) {
+			this.declaredDim = null;
 			this.logger.warn("ObjectEmbeddingStore.upsertIfBetter failed", {
 				tags: { deviceId: input.deviceId },
 				meta: {
@@ -20647,6 +20687,7 @@ var ObjectEmbeddingStore = class {
 					error: String(err)
 				}
 			});
+			return "failed";
 		}
 	}
 	/**
@@ -27098,19 +27139,18 @@ var PipelineAnalyticsAddon = class extends BaseAddon {
 					height
 				})).embedding;
 			},
-			store: async (source, embedding) => {
-				await store.upsertIfBetter({
-					trackId: source.trackId,
-					deviceId: source.deviceId,
-					timestamp: source.timestamp,
-					className: source.className,
-					embedding,
-					modelId,
-					confidence: Number.POSITIVE_INFINITY,
-					...source.mediaKey !== void 0 ? { mediaKey: source.mediaKey } : {},
-					...source.keyFrameMediaKey !== void 0 ? { keyFrameMediaKey: source.keyFrameMediaKey } : {}
-				});
-			},
+			store: async (source, embedding) => store.upsertIfBetter({
+				trackId: source.trackId,
+				deviceId: source.deviceId,
+				timestamp: source.timestamp,
+				className: source.className,
+				embedding,
+				modelId,
+				confidence: source.confidence,
+				force: true,
+				...source.mediaKey !== void 0 ? { mediaKey: source.mediaKey } : {},
+				...source.keyFrameMediaKey !== void 0 ? { keyFrameMediaKey: source.keyFrameMediaKey } : {}
+			}),
 			logger: {
 				info: (message, extras) => this.ctx.logger.info(message, extras),
 				warn: (message, extras) => this.ctx.logger.warn(message, extras)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/addon-post-analysis",
-  "version": "1.2.44",
+  "version": "1.2.46",
   "description": "Post-Analysis bundle — enrichment, embedding-encoder, pipeline-analytics. Multi-entry npm package shipping addons that consume pipeline output.",
   "keywords": [
     "camstack",
@@ -87,6 +87,12 @@
           },
           {
             "name": "videoclips"
+          },
+          {
+            "name": "face-gallery"
+          },
+          {
+            "name": "plate-gallery"
           }
         ],
         "storageLocations": [