@camstack/addon-provider-reolink 1.2.5 → 1.2.6

package/dist/addon.js

@@ -222620,6 +222620,33 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 	markWakeIssued() {
 		this.lastProactiveWakeAt = Date.now();
 	}
+	/**
+	* Gate for a PROACTIVE camera read, checked BEFORE `ensureApi()`.
+	*
+	* The login itself is the wake. On a sleeping UDP/battery camera the
+	* lib's discovery + handshake nudges the firmware awake (same reason
+	* `refreshParentSettingsSnapshot` refuses to call `ensureApi` while
+	* asleep), so gating only the explicit `wakeUp()` call is useless —
+	* by the time we reach it the camera is already up. Production logs
+	* showed exactly that: a background read produced a full
+	* `Connecting to Reolink` → BCUDP discovery → `battery sleep state
+	* committed` (awake) → whole refresh cascade, on a camera that had
+	* been asleep for six minutes.
+	*
+	* Returns `false` when the caller must serve cache / bail out without
+	* touching the socket. Stamps the cooldown when it does let a wake
+	* through, so "at most one proactive wake per
+	* `PROACTIVE_WAKE_COOLDOWN_MS`" holds across ALL proactive callers
+	* rather than per-caller.
+	*
+	* Demand-driven paths never call this — see `canProactivelyWake`.
+	*/
+	allowProactiveCameraAccess(reason) {
+		if (!this.isBattery || !this.sleeping) return true;
+		if (!this.canProactivelyWake(reason)) return false;
+		this.markWakeIssued();
+		return true;
+	}
 	updateBatteryCache(info) {
 		this.setCapSlice(batteryCapability, this.mapBatteryInfo(info));
 	}
@@ -222865,6 +222892,10 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 	}
 	async fetchSnapshotWithSingleFlight() {
 		if (this.snapshotInFlight) return this.snapshotInFlight;
+		if (!this.allowProactiveCameraAccess("snapshot")) {
+			this.ctx.logger.debug("snapshot: skipped — sleeping battery cam inside wake cooldown", { tags: { deviceId: this.id } });
+			return null;
+		}
 		const promise = (async () => {
 			let api;
 			try {
@@ -222882,23 +222913,16 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 				} catch {
 					return true;
 				}
-			})()) {
-				if (!this.canProactivelyWake("snapshot")) {
-					this.ctx.logger.debug("snapshot: skipped — sleeping battery cam inside wake cooldown", { tags: { deviceId: this.id } });
-					return null;
-				}
-				this.markWakeIssued();
-				try {
-					await api.wakeUp(this.getChannel(), {
-						waitAfterWakeMs: 1500,
-						attempts: 2
-					});
-				} catch (err) {
-					this.ctx.logger.debug("snapshot: pre-wake failed (will still try getSnapshot)", {
-						tags: { deviceId: this.id },
-						meta: { error: err instanceof Error ? err.message : String(err) }
-					});
-				}
+			})()) try {
+				await api.wakeUp(this.getChannel(), {
+					waitAfterWakeMs: 1500,
+					attempts: 2
+				});
+			} catch (err) {
+				this.ctx.logger.debug("snapshot: pre-wake failed (will still try getSnapshot)", {
+					tags: { deviceId: this.id },
+					meta: { error: err instanceof Error ? err.message : String(err) }
+				});
 			}
 			const tryOnce = async (timeoutMs) => {
 				const buf = await api.getSnapshot(this.getChannel(), {
@@ -225262,6 +225286,10 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 		if (this.batteryUpdateInFlight) return this.batteryUpdateInFlight;
 		const cycle = (async () => {
 			try {
+				if (!this.allowProactiveCameraAccess("battery-update")) {
+					this.ctx.logger.debug("battery-update: cam sleeping — skipping cycle without connecting", { tags: { deviceId: this.id } });
+					return;
+				}
 				let api;
 				try {
 					api = await this.ensureApi();
@@ -227113,6 +227141,13 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 		}
 		if (this.api) return this.api;
 		if (this.loginPromise) return this.loginPromise;
+		if (this.isBattery && this.sleeping) {
+			const frames = (/* @__PURE__ */ new Error("wake-attribution")).stack?.split("\n").slice(2, 7).join(" | ");
+			this.ctx.logger.info("battery cam login while sleeping — this wakes the camera", {
+				tags: { deviceId: this.id },
+				meta: { caller: frames ?? "unavailable" }
+			});
+		}
 		const host = this.config.get("host");
 		const port = this.config.get("port");
 		const username = this.config.get("username");

package/dist/addon.mjs

@@ -222600,6 +222600,33 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 	markWakeIssued() {
 		this.lastProactiveWakeAt = Date.now();
 	}
+	/**
+	* Gate for a PROACTIVE camera read, checked BEFORE `ensureApi()`.
+	*
+	* The login itself is the wake. On a sleeping UDP/battery camera the
+	* lib's discovery + handshake nudges the firmware awake (same reason
+	* `refreshParentSettingsSnapshot` refuses to call `ensureApi` while
+	* asleep), so gating only the explicit `wakeUp()` call is useless —
+	* by the time we reach it the camera is already up. Production logs
+	* showed exactly that: a background read produced a full
+	* `Connecting to Reolink` → BCUDP discovery → `battery sleep state
+	* committed` (awake) → whole refresh cascade, on a camera that had
+	* been asleep for six minutes.
+	*
+	* Returns `false` when the caller must serve cache / bail out without
+	* touching the socket. Stamps the cooldown when it does let a wake
+	* through, so "at most one proactive wake per
+	* `PROACTIVE_WAKE_COOLDOWN_MS`" holds across ALL proactive callers
+	* rather than per-caller.
+	*
+	* Demand-driven paths never call this — see `canProactivelyWake`.
+	*/
+	allowProactiveCameraAccess(reason) {
+		if (!this.isBattery || !this.sleeping) return true;
+		if (!this.canProactivelyWake(reason)) return false;
+		this.markWakeIssued();
+		return true;
+	}
 	updateBatteryCache(info) {
 		this.setCapSlice(batteryCapability, this.mapBatteryInfo(info));
 	}
@@ -222845,6 +222872,10 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 	}
 	async fetchSnapshotWithSingleFlight() {
 		if (this.snapshotInFlight) return this.snapshotInFlight;
+		if (!this.allowProactiveCameraAccess("snapshot")) {
+			this.ctx.logger.debug("snapshot: skipped — sleeping battery cam inside wake cooldown", { tags: { deviceId: this.id } });
+			return null;
+		}
 		const promise = (async () => {
 			let api;
 			try {
@@ -222862,23 +222893,16 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 				} catch {
 					return true;
 				}
-			})()) {
-				if (!this.canProactivelyWake("snapshot")) {
-					this.ctx.logger.debug("snapshot: skipped — sleeping battery cam inside wake cooldown", { tags: { deviceId: this.id } });
-					return null;
-				}
-				this.markWakeIssued();
-				try {
-					await api.wakeUp(this.getChannel(), {
-						waitAfterWakeMs: 1500,
-						attempts: 2
-					});
-				} catch (err) {
-					this.ctx.logger.debug("snapshot: pre-wake failed (will still try getSnapshot)", {
-						tags: { deviceId: this.id },
-						meta: { error: err instanceof Error ? err.message : String(err) }
-					});
-				}
+			})()) try {
+				await api.wakeUp(this.getChannel(), {
+					waitAfterWakeMs: 1500,
+					attempts: 2
+				});
+			} catch (err) {
+				this.ctx.logger.debug("snapshot: pre-wake failed (will still try getSnapshot)", {
+					tags: { deviceId: this.id },
+					meta: { error: err instanceof Error ? err.message : String(err) }
+				});
 			}
 			const tryOnce = async (timeoutMs) => {
 				const buf = await api.getSnapshot(this.getChannel(), {
@@ -225242,6 +225266,10 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 		if (this.batteryUpdateInFlight) return this.batteryUpdateInFlight;
 		const cycle = (async () => {
 			try {
+				if (!this.allowProactiveCameraAccess("battery-update")) {
+					this.ctx.logger.debug("battery-update: cam sleeping — skipping cycle without connecting", { tags: { deviceId: this.id } });
+					return;
+				}
 				let api;
 				try {
 					api = await this.ensureApi();
@@ -227093,6 +227121,13 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 		}
 		if (this.api) return this.api;
 		if (this.loginPromise) return this.loginPromise;
+		if (this.isBattery && this.sleeping) {
+			const frames = (/* @__PURE__ */ new Error("wake-attribution")).stack?.split("\n").slice(2, 7).join(" | ");
+			this.ctx.logger.info("battery cam login while sleeping — this wakes the camera", {
+				tags: { deviceId: this.id },
+				meta: { caller: frames ?? "unavailable" }
+			});
+		}
 		const host = this.config.get("host");
 		const port = this.config.get("port");
 		const username = this.config.get("username");

package/package.json

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