@camstack/addon-provider-reolink 1.2.145 → 1.2.146

package/dist/addon.js

@@ -5390,7 +5390,7 @@ var ZodIssueCode = {
 var ZodFirstPartyTypeKind;
 ZodFirstPartyTypeKind || (ZodFirstPartyTypeKind = {});
 //#endregion
-//#region ../types/dist/sleep-CopaBJss.mjs
+//#region ../types/dist/sleep-7AN_Txmu.mjs
 /**
 * The audio chunk plane's byte format, and the ONE expansion from a coded
 * window to float samples (D455).
@@ -5590,12 +5590,6 @@ Object.fromEntries([
 		icon: "circle-dot",
 		order: 40
 	},
-	{
-		id: "clips",
-		label: "Clips",
-		icon: "clapperboard",
-		order: 41
-	},
 	{
 		id: "engine",
 		label: "Inference Engine",
@@ -25421,17 +25415,31 @@ var videoclipsCapability = {
 	* `device-config-widget-wrapped-binding.spec.ts` pins that a `kind:'wrapped'`
 	* binding entry derives the same section a native one does.
 	*
-	* `topTab` because the browser owns the whole pane (a day's clips, a source
-	* list and a player), exactly as `ptz` does; the widget itself is
-	* `host/clips-browser` in ui-library's `HOST_WIDGETS`.
+	* **It is a SECTION of the Recording tab, at the end of it — not a tab of
+	* its own.** It shipped as a `clips` top-tab and the operator rejected the
+	* placement: *"utilizzerei la stessa tab recordings, lì abbiamo già tutto il
+	* necessario, una nuova sezione alla fine per le clips"*. The Recording tab
+	* already holds the recorder's panel, the schedule bands and the unified
+	* retention policy; footage the camera itself holds is the same question,
+	* asked of a different store. `order: 100` puts it after all of them with
+	* room left in front. The `clips` entry in `WELL_KNOWN_TABS` went with it —
+	* a well-known id nobody declares is an invitation to mint the tab again.
+	*
+	* `topTab` stays: the browser owns a pane (a day's tiles, a source list and
+	* a player) and the Config tab's inner bar has no room for one. The widget
+	* itself is `host/clips-browser` in ui-library's `HOST_WIDGETS`, and because
+	* the admin Recordings page renders every `location:'top-tab'` +
+	* `tab:'recording'` section behind its camera picker
+	* (`CameraRecordingSettingsSection`), this declaration lands the section on
+	* BOTH surfaces with no second wiring.
 	*/
 	deviceConfig: { ui: {
 		kind: "widget",
 		widgetId: "host/clips-browser",
-		tab: "clips",
+		tab: "recording",
 		topTab: true,
 		label: "Clips",
-		order: 0
+		order: 100
 	} },
 	methods: {
 		listClips: method(object({
@@ -25440,13 +25448,23 @@ var videoclipsCapability = {
 			until: number(),
 			limit: number().int().positive().optional(),
 			/**
-			* View filter over {@link ClipSourceSchema.source} values — the
-			* picker's selection, forwarded so a provider need not list what
-			* nobody is looking at. ABSENT means every source this camera has,
-			* which is the honest default for a surface whose whole point is that
-			* nothing is hidden (D554 3). A provider with one source ignores it.
+			* WHICH provider to ask — the `addonId` a {@link ClipSourceSchema} row
+			* carries, never a source id and never a list.
+			*
+			* Absent means the device's BOUND provider, which is CamStack on every
+			* camera today because that is what `deviceManager.getBindings`
+			* answers with. A provider the device is not bound to is refused by
+			* name rather than answered by another one (D552's
+			* `rejectUnresolvedAddonPin` rule).
+			*
+			* It replaced `sources?: string[]`, a VIEW filter over source ids that
+			* assumed the answer was a fan-out over everything a camera has. The
+			* operator settled otherwise on 2026-09-20 — *"il get videoclips
+			* prendere un provider opzionale, di default va su camstack"*,
+			* *"l'utilizzatore è uno solo"* — so the list is asked of one provider
+			* at a time and there is nothing to filter out of it.
 			*/
-			sources: array(string()).optional()
+			provider: string().optional()
 		}), array(ClipSchema).readonly(), {
 			kind: "query",
 			auth: "protected"
@@ -147455,6 +147473,21 @@ var ClipArtifactStore = class {
 	}
 };
 //#endregion
+//#region src/clips/clip-addressing.ts
+/** Does addressing this camera's catalog REQUIRE a persisted uid? */
+function uidIsRequired(subject) {
+	return subject.isHubChild;
+}
+/**
+* Is this camera unaddressable for want of a uid? The gate every catalog,
+* still and byte read runs — and the only one that may answer `uid-missing`.
+*/
+function uidIsMissing(subject) {
+	return uidIsRequired(subject) && subject.uid === void 0;
+}
+/** The one wording for the refusal, so two surfaces cannot disagree about it. */
+var UID_MISSING_REASON = "uid-missing: the hub selects a child by uid, and this camera has none persisted.";
+//#endregion
 //#region src/clips/clip-catalog-cache.ts
 /**
 * The per-`(device, day)` clip catalog cache.
@@ -148633,7 +148666,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			return cached;
 		}
-		if (device.uid === void 0) {
+		if (uidIsMissing(device)) {
 			deps.logger.warn("videoclips: no uid for this camera — its catalog cannot be addressed", {
 				tags,
 				meta: {
@@ -148643,7 +148676,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			noteHealth(device.deviceId, CLIP_SOURCE_ONBOARD, {
 				state: "unreachable",
-				reason: "uid-missing: this camera has no persisted uid, and on a hub the uid is what selects the child (the hub’s own uid addresses no catalog)."
+				reason: UID_MISSING_REASON
 			});
 			return [];
 		}
@@ -148777,7 +148810,8 @@ function createReolinkVideoclipsProvider(deps) {
 	const readHubEvents = async (device, since, until) => {
 		const tags = { deviceId: device.deviceId };
 		if (!device.isHubChild) return [];
-		if (device.uid === void 0) {
+		const uid = device.uid;
+		if (uid === void 0) {
 			deps.logger.warn("videoclips: no uid for this camera — the hub cannot be asked about it", {
 				tags,
 				meta: {
@@ -148787,7 +148821,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			noteHealth(device.deviceId, CLIP_SOURCE_ONBOARD, {
 				state: "unreachable",
-				reason: "uid-missing: the hub selects a child by uid, and this camera has none persisted."
+				reason: UID_MISSING_REASON
 			});
 			return [];
 		}
@@ -148815,7 +148849,7 @@ function createReolinkVideoclipsProvider(deps) {
 		let joined = [];
 		try {
 			const rows = await listEventRows(api, {
-				uid: device.uid,
+				uid,
 				channel: device.channel,
 				timeZone: device.timeZone,
 				since,
@@ -148950,11 +148984,10 @@ function createReolinkVideoclipsProvider(deps) {
 				availability: await resolveAvailability(device, via)
 			}];
 		},
-		listClips: async ({ deviceId, since, until, limit, sources }) => {
+		listClips: async ({ deviceId, since, until, limit }) => {
 			const device = await deps.resolveDevice(deviceId);
 			if (device === null) return [];
 			const tags = { deviceId };
-			if (sources !== void 0 && !sources.includes("native:reolink:onboard")) return [];
 			const via = viaFor(device);
 			const ordered = (via === "events" ? await (async () => {
 				const events = await readHubEvents(device, since, until);
@@ -149022,17 +149055,17 @@ function createReolinkVideoclipsProvider(deps) {
 	async function resolveAvailability(device, via) {
 		const known = health.get(healthKey(device.deviceId, CLIP_SOURCE_ONBOARD))?.availability;
 		if (known !== void 0) return known;
-		if (via === "events") return device.uid === void 0 ? {
+		if (via === "events") return uidIsMissing(device) ? {
 			state: "unreachable",
-			reason: "uid-missing: the hub selects a child by uid, and this camera has none persisted."
+			reason: UID_MISSING_REASON
 		} : { state: "ok" };
 		if (device.isBattery && device.sleeping && !device.isHubChild) return {
 			state: "sleeping",
 			reason: "A read would wake this camera. Nothing has been listed yet."
 		};
-		if (device.uid === void 0) return {
+		if (uidIsMissing(device)) return {
 			state: "unreachable",
-			reason: "uid-missing: this camera has no persisted uid to address its catalog with."
+			reason: UID_MISSING_REASON
 		};
 		try {
 			if ((await readStorageState(await device.getApi())).state === "absent") return {
@@ -149085,9 +149118,9 @@ function createClipService(deps) {
 			ok: false,
 			reason: "sleeping: a still would wake this standalone battery camera"
 		};
-		if (device.uid === void 0) return {
+		if (uidIsMissing(device)) return {
 			ok: false,
-			reason: "uid-missing: this camera’s stills cannot be addressed"
+			reason: UID_MISSING_REASON
 		};
 		const row = rowFor(deviceId, clipKey);
 		if (row === null) return {
@@ -149104,7 +149137,7 @@ function createClipService(deps) {
 						time: new Date(row.startMs),
 						endTime: new Date(row.endMs),
 						snapType: "sub",
-						uid: device.uid,
+						...device.uid === void 0 ? {} : { uid: device.uid },
 						isNvr: device.isHubChild,
 						timeZone: device.timeZone
 					});

package/dist/addon.mjs

@@ -5385,7 +5385,7 @@ var ZodIssueCode = {
 var ZodFirstPartyTypeKind;
 ZodFirstPartyTypeKind || (ZodFirstPartyTypeKind = {});
 //#endregion
-//#region ../types/dist/sleep-CopaBJss.mjs
+//#region ../types/dist/sleep-7AN_Txmu.mjs
 /**
 * The audio chunk plane's byte format, and the ONE expansion from a coded
 * window to float samples (D455).
@@ -5585,12 +5585,6 @@ Object.fromEntries([
 		icon: "circle-dot",
 		order: 40
 	},
-	{
-		id: "clips",
-		label: "Clips",
-		icon: "clapperboard",
-		order: 41
-	},
 	{
 		id: "engine",
 		label: "Inference Engine",
@@ -25416,17 +25410,31 @@ var videoclipsCapability = {
 	* `device-config-widget-wrapped-binding.spec.ts` pins that a `kind:'wrapped'`
 	* binding entry derives the same section a native one does.
 	*
-	* `topTab` because the browser owns the whole pane (a day's clips, a source
-	* list and a player), exactly as `ptz` does; the widget itself is
-	* `host/clips-browser` in ui-library's `HOST_WIDGETS`.
+	* **It is a SECTION of the Recording tab, at the end of it — not a tab of
+	* its own.** It shipped as a `clips` top-tab and the operator rejected the
+	* placement: *"utilizzerei la stessa tab recordings, lì abbiamo già tutto il
+	* necessario, una nuova sezione alla fine per le clips"*. The Recording tab
+	* already holds the recorder's panel, the schedule bands and the unified
+	* retention policy; footage the camera itself holds is the same question,
+	* asked of a different store. `order: 100` puts it after all of them with
+	* room left in front. The `clips` entry in `WELL_KNOWN_TABS` went with it —
+	* a well-known id nobody declares is an invitation to mint the tab again.
+	*
+	* `topTab` stays: the browser owns a pane (a day's tiles, a source list and
+	* a player) and the Config tab's inner bar has no room for one. The widget
+	* itself is `host/clips-browser` in ui-library's `HOST_WIDGETS`, and because
+	* the admin Recordings page renders every `location:'top-tab'` +
+	* `tab:'recording'` section behind its camera picker
+	* (`CameraRecordingSettingsSection`), this declaration lands the section on
+	* BOTH surfaces with no second wiring.
 	*/
 	deviceConfig: { ui: {
 		kind: "widget",
 		widgetId: "host/clips-browser",
-		tab: "clips",
+		tab: "recording",
 		topTab: true,
 		label: "Clips",
-		order: 0
+		order: 100
 	} },
 	methods: {
 		listClips: method(object({
@@ -25435,13 +25443,23 @@ var videoclipsCapability = {
 			until: number(),
 			limit: number().int().positive().optional(),
 			/**
-			* View filter over {@link ClipSourceSchema.source} values — the
-			* picker's selection, forwarded so a provider need not list what
-			* nobody is looking at. ABSENT means every source this camera has,
-			* which is the honest default for a surface whose whole point is that
-			* nothing is hidden (D554 3). A provider with one source ignores it.
+			* WHICH provider to ask — the `addonId` a {@link ClipSourceSchema} row
+			* carries, never a source id and never a list.
+			*
+			* Absent means the device's BOUND provider, which is CamStack on every
+			* camera today because that is what `deviceManager.getBindings`
+			* answers with. A provider the device is not bound to is refused by
+			* name rather than answered by another one (D552's
+			* `rejectUnresolvedAddonPin` rule).
+			*
+			* It replaced `sources?: string[]`, a VIEW filter over source ids that
+			* assumed the answer was a fan-out over everything a camera has. The
+			* operator settled otherwise on 2026-09-20 — *"il get videoclips
+			* prendere un provider opzionale, di default va su camstack"*,
+			* *"l'utilizzatore è uno solo"* — so the list is asked of one provider
+			* at a time and there is nothing to filter out of it.
 			*/
-			sources: array(string()).optional()
+			provider: string().optional()
 		}), array(ClipSchema).readonly(), {
 			kind: "query",
 			auth: "protected"
@@ -147450,6 +147468,21 @@ var ClipArtifactStore = class {
 	}
 };
 //#endregion
+//#region src/clips/clip-addressing.ts
+/** Does addressing this camera's catalog REQUIRE a persisted uid? */
+function uidIsRequired(subject) {
+	return subject.isHubChild;
+}
+/**
+* Is this camera unaddressable for want of a uid? The gate every catalog,
+* still and byte read runs — and the only one that may answer `uid-missing`.
+*/
+function uidIsMissing(subject) {
+	return uidIsRequired(subject) && subject.uid === void 0;
+}
+/** The one wording for the refusal, so two surfaces cannot disagree about it. */
+var UID_MISSING_REASON = "uid-missing: the hub selects a child by uid, and this camera has none persisted.";
+//#endregion
 //#region src/clips/clip-catalog-cache.ts
 /**
 * The per-`(device, day)` clip catalog cache.
@@ -148628,7 +148661,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			return cached;
 		}
-		if (device.uid === void 0) {
+		if (uidIsMissing(device)) {
 			deps.logger.warn("videoclips: no uid for this camera — its catalog cannot be addressed", {
 				tags,
 				meta: {
@@ -148638,7 +148671,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			noteHealth(device.deviceId, CLIP_SOURCE_ONBOARD, {
 				state: "unreachable",
-				reason: "uid-missing: this camera has no persisted uid, and on a hub the uid is what selects the child (the hub’s own uid addresses no catalog)."
+				reason: UID_MISSING_REASON
 			});
 			return [];
 		}
@@ -148772,7 +148805,8 @@ function createReolinkVideoclipsProvider(deps) {
 	const readHubEvents = async (device, since, until) => {
 		const tags = { deviceId: device.deviceId };
 		if (!device.isHubChild) return [];
-		if (device.uid === void 0) {
+		const uid = device.uid;
+		if (uid === void 0) {
 			deps.logger.warn("videoclips: no uid for this camera — the hub cannot be asked about it", {
 				tags,
 				meta: {
@@ -148782,7 +148816,7 @@ function createReolinkVideoclipsProvider(deps) {
 			});
 			noteHealth(device.deviceId, CLIP_SOURCE_ONBOARD, {
 				state: "unreachable",
-				reason: "uid-missing: the hub selects a child by uid, and this camera has none persisted."
+				reason: UID_MISSING_REASON
 			});
 			return [];
 		}
@@ -148810,7 +148844,7 @@ function createReolinkVideoclipsProvider(deps) {
 		let joined = [];
 		try {
 			const rows = await listEventRows(api, {
-				uid: device.uid,
+				uid,
 				channel: device.channel,
 				timeZone: device.timeZone,
 				since,
@@ -148945,11 +148979,10 @@ function createReolinkVideoclipsProvider(deps) {
 				availability: await resolveAvailability(device, via)
 			}];
 		},
-		listClips: async ({ deviceId, since, until, limit, sources }) => {
+		listClips: async ({ deviceId, since, until, limit }) => {
 			const device = await deps.resolveDevice(deviceId);
 			if (device === null) return [];
 			const tags = { deviceId };
-			if (sources !== void 0 && !sources.includes("native:reolink:onboard")) return [];
 			const via = viaFor(device);
 			const ordered = (via === "events" ? await (async () => {
 				const events = await readHubEvents(device, since, until);
@@ -149017,17 +149050,17 @@ function createReolinkVideoclipsProvider(deps) {
 	async function resolveAvailability(device, via) {
 		const known = health.get(healthKey(device.deviceId, CLIP_SOURCE_ONBOARD))?.availability;
 		if (known !== void 0) return known;
-		if (via === "events") return device.uid === void 0 ? {
+		if (via === "events") return uidIsMissing(device) ? {
 			state: "unreachable",
-			reason: "uid-missing: the hub selects a child by uid, and this camera has none persisted."
+			reason: UID_MISSING_REASON
 		} : { state: "ok" };
 		if (device.isBattery && device.sleeping && !device.isHubChild) return {
 			state: "sleeping",
 			reason: "A read would wake this camera. Nothing has been listed yet."
 		};
-		if (device.uid === void 0) return {
+		if (uidIsMissing(device)) return {
 			state: "unreachable",
-			reason: "uid-missing: this camera has no persisted uid to address its catalog with."
+			reason: UID_MISSING_REASON
 		};
 		try {
 			if ((await readStorageState(await device.getApi())).state === "absent") return {
@@ -149080,9 +149113,9 @@ function createClipService(deps) {
 			ok: false,
 			reason: "sleeping: a still would wake this standalone battery camera"
 		};
-		if (device.uid === void 0) return {
+		if (uidIsMissing(device)) return {
 			ok: false,
-			reason: "uid-missing: this camera’s stills cannot be addressed"
+			reason: UID_MISSING_REASON
 		};
 		const row = rowFor(deviceId, clipKey);
 		if (row === null) return {
@@ -149099,7 +149132,7 @@ function createClipService(deps) {
 						time: new Date(row.startMs),
 						endTime: new Date(row.endMs),
 						snapType: "sub",
-						uid: device.uid,
+						...device.uid === void 0 ? {} : { uid: device.uid },
 						isNvr: device.isHubChild,
 						timeZone: device.timeZone
 					});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/addon-provider-reolink",
-  "version": "1.2.145",
+  "version": "1.2.146",
   "description": "Reolink camera device provider addon for CamStack — native Baichuan protocol",
   "keywords": [
     "camstack",