@camstack/addon-post-analysis 1.2.45 → 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;
@@ -20628,33 +20643,49 @@ var ObjectEmbeddingStore = class {
 	* 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 {
 			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;
+				if (typeof stored === "number" && input.confidence <= stored) return "skipped";
 			}
-			if ((await this.vectorStore.upsert.mutate({
-				index: "object-clip",
+			const result = await this.vectorStore.upsert.mutate({
+				index: OBJECT_CLIP_INDEX,
 				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: {
@@ -20662,6 +20693,7 @@ var ObjectEmbeddingStore = class {
 					error: String(err)
 				}
 			});
+			return "failed";
 		}
 	}
 	/**
@@ -27113,20 +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: source.confidence,
-					force: true,
-					...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;
@@ -20622,33 +20637,49 @@ var ObjectEmbeddingStore = class {
 	* 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 {
 			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;
+				if (typeof stored === "number" && input.confidence <= stored) return "skipped";
 			}
-			if ((await this.vectorStore.upsert.mutate({
-				index: "object-clip",
+			const result = await this.vectorStore.upsert.mutate({
+				index: OBJECT_CLIP_INDEX,
 				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: {
@@ -20656,6 +20687,7 @@ var ObjectEmbeddingStore = class {
 					error: String(err)
 				}
 			});
+			return "failed";
 		}
 	}
 	/**
@@ -27107,20 +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: source.confidence,
-					force: true,
-					...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.45",
+  "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": [