@camstack/addon-provider-reolink 1.2.140 → 1.2.142

package/dist/addon.js

@@ -5386,7 +5386,7 @@ var ZodIssueCode = {
 var ZodFirstPartyTypeKind;
 ZodFirstPartyTypeKind || (ZodFirstPartyTypeKind = {});
 //#endregion
-//#region ../types/dist/sleep-vl6nBE8d.mjs
+//#region ../types/dist/sleep-Bs3xPtSh.mjs
 /**
 * The audio chunk plane's byte format, and the ONE expansion from a coded
 * window to float samples (D455).
@@ -12454,6 +12454,24 @@ var GetStreamWithCodecInputSchema = object({
 });
 var RtpSourceSchema = object({
 	url: string(),
+	/**
+	* The SENTRY counterpart of {@link url} — the dial that asks for the stream
+	* WITHOUT asking for the camera — or `null` when this camera offers none.
+	*
+	* The broker answers it because the broker is the only thing that can:
+	* eligibility is `sentryEligible` (a BATTERY camera that DECLARES the signals
+	* cap), asked of facts only the broker holds. A consumer re-deriving it from
+	* cap bindings would be a second authority, and two authorities on the same
+	* question is how a composite camera ends up dialling a sleeping camera
+	* because its own copy of the rule was a release behind.
+	*
+	* `null` is a real answer and is never a live url in disguise: a consumer
+	* that silently fell back to {@link url} would perform exactly the wake this
+	* field exists to prevent. Only a PASSTHROUGH acquisition can offer one — a
+	* transcode hands back an egress the broker is feeding itself, which has no
+	* sentry counterpart.
+	*/
+	sentryUrl: string().nullable().optional(),
 	videoCodec: _enum(["H264", "H265"]),
 	audioCodec: string(),
 	resolution: object({
@@ -29619,6 +29637,114 @@ var motionTriggerCapability = {
 	durability: "session"
 };
 /**
+* camera-grid-layout — the geometry of a COMPOSITE camera, on the camera's own
+* page.
+*
+* ## Why this is a capability and not an addon settings schema
+*
+* It was one, and it did not render. The addon declared the editor as a
+* `type: 'widget'` field inside its own `deviceSettingsSchema()`; the hub
+* returned that section correctly and `ConfigFormField` renders `type:'widget'`
+* perfectly well — and nothing ever asked camera-grid for it. The Cluster →
+* Pipeline → Device Overrides page interrogates a HAND-WRITTEN list of four
+* addons (`PIPELINE_CLUSTER_DEVICE_ADDONS`), whose own comment says an addon
+* not on it "falls off silently".
+*
+* Adding a fifth name to that list would have been the wrong fix twice over:
+* that page is per-camera DETECTION tuning, and a grid's geometry belongs
+* beside PTZ and motion zones on the camera itself — which is exactly where the
+* framework already puts a widget, when the widget is declared on a CAPABILITY.
+* `deviceConfig.ui` is the mechanism (`motion-zones`, `ptz`, `ptz-autotrack`
+* all use it): the framework emits the structural section and the widget
+* self-persists through the cap's own mutations.
+*
+* ## Why one addon may implement it
+*
+* It is a device-scoped NATIVE cap, registered by the grid camera device
+* itself. Nothing else declares a composite camera, so nothing else has a
+* layout — and the device-scoped route means the widget asks THE camera, not
+* "the camera-grid addon", which is what let the old custom-action pair be
+* reached only by a caller that already knew the addon id.
+*
+* ## The tab
+*
+* `streaming`, not a top-tab of its own. A grid's geometry IS what its stream
+* is, so the Streaming tab is where it belongs; a `grid` top-tab would need an
+* entry in `WELL_KNOWN_TAB_MAP` or the device page renders the raw id as the
+* label (measured on the robot camera, 2026-09-06 — a tab called "navigation"
+* next to "PTZ").
+*/
+/** A rectangle in normalized [0,1] coordinates of whatever contains it. */
+var GridNormalizedRectSchema = object({
+	x: number().min(0).max(1),
+	y: number().min(0).max(1),
+	width: number().gt(0).max(1),
+	height: number().gt(0).max(1)
+});
+/**
+* One source camera, the part of its picture taken, and where that part lands.
+*
+* Both rectangles are NORMALIZED (D519): a source camera can change resolution
+* — a profile switch, a firmware update, a substream that comes back different
+* — and a stored PIXEL rectangle would quietly start cutting the wrong region,
+* which is the class of bug nobody files.
+*/
+var GridLayoutCellSchema = object({
+	deviceId: number().int().positive(),
+	/** The part of the SOURCE taken, normalized against the source. */
+	source: GridNormalizedRectSchema,
+	/** Where it lands, normalized against the CANVAS. */
+	cell: GridNormalizedRectSchema
+});
+/**
+* Which profiles this grid can actually compose, and why not.
+*
+* A grid's `high` composes its sources' `high` and its `low` their `low`, so a
+* profile is on offer only when EVERY source can serve it. The refusal NAMES
+* the sources, because "this grid has no low" is not a finding — "615 has no
+* low" is, and it is the one an operator can act on.
+*/
+var GridProfileOfferSchema = object({
+	profile: _enum([
+		"high",
+		"mid",
+		"low"
+	]),
+	offered: boolean(),
+	/** Sources that cannot serve it. Empty when it is offered, or when there are no cells. */
+	missingSources: array(number().int().positive())
+});
+var GridLayoutViewSchema = object({
+	/** The persisted grid row this camera was declared from. */
+	instanceId: string(),
+	deviceId: number().int().nonnegative(),
+	name: string(),
+	/** The canvas the operator laid out — this grid's `high`. mid/low are scaled from it. */
+	width: number().int(),
+	height: number().int(),
+	fps: number().int(),
+	cells: array(GridLayoutCellSchema),
+	/** What the catalog will publish, and what it refuses to. Read-only. */
+	profiles: array(GridProfileOfferSchema)
+});
+var GridLayoutPatchSchema = object({
+	deviceId: number().int().nonnegative(),
+	name: string().min(1).max(160).optional(),
+	width: number().int().min(160).max(7680).optional(),
+	height: number().int().min(120).max(4320).optional(),
+	fps: number().int().min(1).max(60).optional(),
+	/**
+	* The whole cell list at once. A per-cell patch would need an ordering the
+	* editor does not have, and a half-applied layout is a picture nobody asked
+	* for.
+	*/
+	cells: array(GridLayoutCellSchema).max(16)
+});
+DeviceType.Camera, method(object({ deviceId: number().int().nonnegative() }), GridLayoutViewSchema.nullable(), { auth: "admin" }), method(GridLayoutPatchSchema, GridLayoutViewSchema, {
+	kind: "mutation",
+	auth: "admin"
+});
+/**
 * Motion-zones share the same MaskShape vocabulary as privacy-mask — the
 * on-camera motion-detection mask is a single `grid` region (a row-major
 * boolean cell lattice the camera's onboard VMD evaluates). Composing it as
@@ -31820,6 +31946,26 @@ var PtzHomePresetSchema = object({
 		"none"
 	])
 });
+/**
+* The camera's own idle-return behaviour: whether it goes home by itself, and
+* after how long.
+*
+* ONE shape, not two scalars, because on the camera they are ONE fact —
+* Reolink's guard point carries `benable` and `timeout` in the same element,
+* written by the same command. Reading them apart means two round-trips that
+* can disagree.
+*
+* `enabled` and `seconds` are INDEPENDENT: a camera can hold a 68 s delay with
+* the behaviour switched off, which is exactly what the vendor app leaves when
+* you turn the guard point off without clearing it. The first version of this
+* cap reported only the delay, so that state was invisible — a camera could be
+* configured to return after 68 seconds with no way to say whether it should
+* return at all.
+*/
+var PtzHomeReturnSchema = object({
+	enabled: boolean(),
+	seconds: number().int()
+});
 var ptzCapability = {
 	name: "ptz",
 	scope: "device",
@@ -31902,18 +32048,25 @@ var ptzCapability = {
 			auth: "admin"
 		}),
 		/**
-		* How long the camera waits, idle, before returning home on its own --
-		* `null` when it has no such behaviour.
+		* Whether the camera returns home on its own when left idle, and after how
+		* long. `null` when it has no such behaviour at all.
 		*
 		* Reolink's guard point carries one (`timeout`, freely settable in
 		* seconds). It is a real function of the camera that we did not expose at
 		* all, and it belongs next to the home it governs rather than in a vendor
 		* settings page nobody associates with Home.
 		*/
-		getHomeReturnSeconds: method(object({ deviceId: number() }), number().int().nullable()),
-		/** Set the idle-return delay. Refused by a camera that has no such timer. */
-		setHomeReturnSeconds: method(object({
+		getHomeReturn: method(object({ deviceId: number() }), PtzHomeReturnSchema.nullable()),
+		/**
+		* Set both together. Refused by a camera with no idle-return behaviour.
+		*
+		* They travel together for the same reason they are read together: the
+		* underlying write carries both, so setting one alone means reading the
+		* other back first and racing whoever else is writing.
+		*/
+		setHomeReturn: method(object({
 			deviceId: number(),
+			enabled: boolean(),
 			seconds: number().int().min(0).max(3600)
 		}), _void(), {
 			kind: "mutation",
@@ -37851,6 +38004,18 @@ Object.freeze({
 		addonId: null,
 		access: "view"
 	},
+	"cameraGridLayout.getLayout": {
+		capName: "camera-grid-layout",
+		capScope: "device",
+		addonId: null,
+		access: "view"
+	},
+	"cameraGridLayout.saveLayout": {
+		capName: "camera-grid-layout",
+		capScope: "device",
+		addonId: null,
+		access: "create"
+	},
 	"cameraStreams.getBrokerStreams": {
 		capName: "camera-streams",
 		capScope: "device",
@@ -41601,7 +41766,7 @@ Object.freeze({
 		addonId: null,
 		access: "view"
 	},
-	"ptz.getHomeReturnSeconds": {
+	"ptz.getHomeReturn": {
 		capName: "ptz",
 		capScope: "device",
 		addonId: null,
@@ -41661,7 +41826,7 @@ Object.freeze({
 		addonId: null,
 		access: "create"
 	},
-	"ptz.setHomeReturnSeconds": {
+	"ptz.setHomeReturn": {
 		capName: "ptz",
 		capScope: "device",
 		addonId: null,
@@ -43550,6 +43715,21 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
+	"cameraGridLayout.getLayout": [{
+		name: "deviceId",
+		form: "single",
+		optional: false
+	}],
+	"cameraGridLayout.saveLayout": [{
+		name: "cells",
+		form: "object-array",
+		optional: false,
+		itemField: "deviceId"
+	}, {
+		name: "deviceId",
+		form: "single",
+		optional: false
+	}],
 	"cameraStreams.getBrokerStreams": [{
 		name: "deviceId",
 		form: "single",
@@ -44850,7 +45030,7 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
-	"ptz.getHomeReturnSeconds": [{
+	"ptz.getHomeReturn": [{
 		name: "deviceId",
 		form: "single",
 		optional: false
@@ -44900,7 +45080,7 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
-	"ptz.setHomeReturnSeconds": [{
+	"ptz.setHomeReturn": [{
 		name: "deviceId",
 		form: "single",
 		optional: false
@@ -153888,21 +154068,28 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 					}, channel);
 					this.ctx.logger.info("reolink: guard point pinned to the current position", { tags: { deviceId: this.id } });
 				},
-				getHomeReturnSeconds: async ({ deviceId }) => {
+				getHomeReturn: async ({ deviceId }) => {
 					if (deviceId !== this.id) return null;
 					const guard = await this.readGuardStatus();
-					return guard === null ? null : guard.timeoutSeconds;
+					return guard === null ? null : {
+						enabled: guard.enabled,
+						seconds: guard.timeoutSeconds
+					};
 				},
-				setHomeReturnSeconds: async ({ deviceId, seconds }) => {
+				setHomeReturn: async ({ deviceId, enabled, seconds }) => {
 					if (deviceId !== this.id) return;
-					const current = await this.readGuardStatus();
-					const api = await this.ensureApi();
-					const channel = this.getChannel();
-					await api.setPtzGuard({
-						enabled: current?.enabled ?? true,
+					await (await this.ensureApi()).setPtzGuard({
+						enabled,
 						timeoutSeconds: seconds,
 						setPosition: false
-					}, channel);
+					}, this.getChannel());
+					this.ctx.logger.info("reolink: guard point idle-return updated", {
+						tags: { deviceId: this.id },
+						meta: {
+							enabled,
+							seconds
+						}
+					});
 				},
 				setHomePreset: async ({ deviceId, presetId }) => {
 					if (deviceId !== this.id) return;

package/dist/addon.mjs

@@ -5381,7 +5381,7 @@ var ZodIssueCode = {
 var ZodFirstPartyTypeKind;
 ZodFirstPartyTypeKind || (ZodFirstPartyTypeKind = {});
 //#endregion
-//#region ../types/dist/sleep-vl6nBE8d.mjs
+//#region ../types/dist/sleep-Bs3xPtSh.mjs
 /**
 * The audio chunk plane's byte format, and the ONE expansion from a coded
 * window to float samples (D455).
@@ -12449,6 +12449,24 @@ var GetStreamWithCodecInputSchema = object({
 });
 var RtpSourceSchema = object({
 	url: string(),
+	/**
+	* The SENTRY counterpart of {@link url} — the dial that asks for the stream
+	* WITHOUT asking for the camera — or `null` when this camera offers none.
+	*
+	* The broker answers it because the broker is the only thing that can:
+	* eligibility is `sentryEligible` (a BATTERY camera that DECLARES the signals
+	* cap), asked of facts only the broker holds. A consumer re-deriving it from
+	* cap bindings would be a second authority, and two authorities on the same
+	* question is how a composite camera ends up dialling a sleeping camera
+	* because its own copy of the rule was a release behind.
+	*
+	* `null` is a real answer and is never a live url in disguise: a consumer
+	* that silently fell back to {@link url} would perform exactly the wake this
+	* field exists to prevent. Only a PASSTHROUGH acquisition can offer one — a
+	* transcode hands back an egress the broker is feeding itself, which has no
+	* sentry counterpart.
+	*/
+	sentryUrl: string().nullable().optional(),
 	videoCodec: _enum(["H264", "H265"]),
 	audioCodec: string(),
 	resolution: object({
@@ -29614,6 +29632,114 @@ var motionTriggerCapability = {
 	durability: "session"
 };
 /**
+* camera-grid-layout — the geometry of a COMPOSITE camera, on the camera's own
+* page.
+*
+* ## Why this is a capability and not an addon settings schema
+*
+* It was one, and it did not render. The addon declared the editor as a
+* `type: 'widget'` field inside its own `deviceSettingsSchema()`; the hub
+* returned that section correctly and `ConfigFormField` renders `type:'widget'`
+* perfectly well — and nothing ever asked camera-grid for it. The Cluster →
+* Pipeline → Device Overrides page interrogates a HAND-WRITTEN list of four
+* addons (`PIPELINE_CLUSTER_DEVICE_ADDONS`), whose own comment says an addon
+* not on it "falls off silently".
+*
+* Adding a fifth name to that list would have been the wrong fix twice over:
+* that page is per-camera DETECTION tuning, and a grid's geometry belongs
+* beside PTZ and motion zones on the camera itself — which is exactly where the
+* framework already puts a widget, when the widget is declared on a CAPABILITY.
+* `deviceConfig.ui` is the mechanism (`motion-zones`, `ptz`, `ptz-autotrack`
+* all use it): the framework emits the structural section and the widget
+* self-persists through the cap's own mutations.
+*
+* ## Why one addon may implement it
+*
+* It is a device-scoped NATIVE cap, registered by the grid camera device
+* itself. Nothing else declares a composite camera, so nothing else has a
+* layout — and the device-scoped route means the widget asks THE camera, not
+* "the camera-grid addon", which is what let the old custom-action pair be
+* reached only by a caller that already knew the addon id.
+*
+* ## The tab
+*
+* `streaming`, not a top-tab of its own. A grid's geometry IS what its stream
+* is, so the Streaming tab is where it belongs; a `grid` top-tab would need an
+* entry in `WELL_KNOWN_TAB_MAP` or the device page renders the raw id as the
+* label (measured on the robot camera, 2026-09-06 — a tab called "navigation"
+* next to "PTZ").
+*/
+/** A rectangle in normalized [0,1] coordinates of whatever contains it. */
+var GridNormalizedRectSchema = object({
+	x: number().min(0).max(1),
+	y: number().min(0).max(1),
+	width: number().gt(0).max(1),
+	height: number().gt(0).max(1)
+});
+/**
+* One source camera, the part of its picture taken, and where that part lands.
+*
+* Both rectangles are NORMALIZED (D519): a source camera can change resolution
+* — a profile switch, a firmware update, a substream that comes back different
+* — and a stored PIXEL rectangle would quietly start cutting the wrong region,
+* which is the class of bug nobody files.
+*/
+var GridLayoutCellSchema = object({
+	deviceId: number().int().positive(),
+	/** The part of the SOURCE taken, normalized against the source. */
+	source: GridNormalizedRectSchema,
+	/** Where it lands, normalized against the CANVAS. */
+	cell: GridNormalizedRectSchema
+});
+/**
+* Which profiles this grid can actually compose, and why not.
+*
+* A grid's `high` composes its sources' `high` and its `low` their `low`, so a
+* profile is on offer only when EVERY source can serve it. The refusal NAMES
+* the sources, because "this grid has no low" is not a finding — "615 has no
+* low" is, and it is the one an operator can act on.
+*/
+var GridProfileOfferSchema = object({
+	profile: _enum([
+		"high",
+		"mid",
+		"low"
+	]),
+	offered: boolean(),
+	/** Sources that cannot serve it. Empty when it is offered, or when there are no cells. */
+	missingSources: array(number().int().positive())
+});
+var GridLayoutViewSchema = object({
+	/** The persisted grid row this camera was declared from. */
+	instanceId: string(),
+	deviceId: number().int().nonnegative(),
+	name: string(),
+	/** The canvas the operator laid out — this grid's `high`. mid/low are scaled from it. */
+	width: number().int(),
+	height: number().int(),
+	fps: number().int(),
+	cells: array(GridLayoutCellSchema),
+	/** What the catalog will publish, and what it refuses to. Read-only. */
+	profiles: array(GridProfileOfferSchema)
+});
+var GridLayoutPatchSchema = object({
+	deviceId: number().int().nonnegative(),
+	name: string().min(1).max(160).optional(),
+	width: number().int().min(160).max(7680).optional(),
+	height: number().int().min(120).max(4320).optional(),
+	fps: number().int().min(1).max(60).optional(),
+	/**
+	* The whole cell list at once. A per-cell patch would need an ordering the
+	* editor does not have, and a half-applied layout is a picture nobody asked
+	* for.
+	*/
+	cells: array(GridLayoutCellSchema).max(16)
+});
+DeviceType.Camera, method(object({ deviceId: number().int().nonnegative() }), GridLayoutViewSchema.nullable(), { auth: "admin" }), method(GridLayoutPatchSchema, GridLayoutViewSchema, {
+	kind: "mutation",
+	auth: "admin"
+});
+/**
 * Motion-zones share the same MaskShape vocabulary as privacy-mask — the
 * on-camera motion-detection mask is a single `grid` region (a row-major
 * boolean cell lattice the camera's onboard VMD evaluates). Composing it as
@@ -31815,6 +31941,26 @@ var PtzHomePresetSchema = object({
 		"none"
 	])
 });
+/**
+* The camera's own idle-return behaviour: whether it goes home by itself, and
+* after how long.
+*
+* ONE shape, not two scalars, because on the camera they are ONE fact —
+* Reolink's guard point carries `benable` and `timeout` in the same element,
+* written by the same command. Reading them apart means two round-trips that
+* can disagree.
+*
+* `enabled` and `seconds` are INDEPENDENT: a camera can hold a 68 s delay with
+* the behaviour switched off, which is exactly what the vendor app leaves when
+* you turn the guard point off without clearing it. The first version of this
+* cap reported only the delay, so that state was invisible — a camera could be
+* configured to return after 68 seconds with no way to say whether it should
+* return at all.
+*/
+var PtzHomeReturnSchema = object({
+	enabled: boolean(),
+	seconds: number().int()
+});
 var ptzCapability = {
 	name: "ptz",
 	scope: "device",
@@ -31897,18 +32043,25 @@ var ptzCapability = {
 			auth: "admin"
 		}),
 		/**
-		* How long the camera waits, idle, before returning home on its own --
-		* `null` when it has no such behaviour.
+		* Whether the camera returns home on its own when left idle, and after how
+		* long. `null` when it has no such behaviour at all.
 		*
 		* Reolink's guard point carries one (`timeout`, freely settable in
 		* seconds). It is a real function of the camera that we did not expose at
 		* all, and it belongs next to the home it governs rather than in a vendor
 		* settings page nobody associates with Home.
 		*/
-		getHomeReturnSeconds: method(object({ deviceId: number() }), number().int().nullable()),
-		/** Set the idle-return delay. Refused by a camera that has no such timer. */
-		setHomeReturnSeconds: method(object({
+		getHomeReturn: method(object({ deviceId: number() }), PtzHomeReturnSchema.nullable()),
+		/**
+		* Set both together. Refused by a camera with no idle-return behaviour.
+		*
+		* They travel together for the same reason they are read together: the
+		* underlying write carries both, so setting one alone means reading the
+		* other back first and racing whoever else is writing.
+		*/
+		setHomeReturn: method(object({
 			deviceId: number(),
+			enabled: boolean(),
 			seconds: number().int().min(0).max(3600)
 		}), _void(), {
 			kind: "mutation",
@@ -37846,6 +37999,18 @@ Object.freeze({
 		addonId: null,
 		access: "view"
 	},
+	"cameraGridLayout.getLayout": {
+		capName: "camera-grid-layout",
+		capScope: "device",
+		addonId: null,
+		access: "view"
+	},
+	"cameraGridLayout.saveLayout": {
+		capName: "camera-grid-layout",
+		capScope: "device",
+		addonId: null,
+		access: "create"
+	},
 	"cameraStreams.getBrokerStreams": {
 		capName: "camera-streams",
 		capScope: "device",
@@ -41596,7 +41761,7 @@ Object.freeze({
 		addonId: null,
 		access: "view"
 	},
-	"ptz.getHomeReturnSeconds": {
+	"ptz.getHomeReturn": {
 		capName: "ptz",
 		capScope: "device",
 		addonId: null,
@@ -41656,7 +41821,7 @@ Object.freeze({
 		addonId: null,
 		access: "create"
 	},
-	"ptz.setHomeReturnSeconds": {
+	"ptz.setHomeReturn": {
 		capName: "ptz",
 		capScope: "device",
 		addonId: null,
@@ -43545,6 +43710,21 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
+	"cameraGridLayout.getLayout": [{
+		name: "deviceId",
+		form: "single",
+		optional: false
+	}],
+	"cameraGridLayout.saveLayout": [{
+		name: "cells",
+		form: "object-array",
+		optional: false,
+		itemField: "deviceId"
+	}, {
+		name: "deviceId",
+		form: "single",
+		optional: false
+	}],
 	"cameraStreams.getBrokerStreams": [{
 		name: "deviceId",
 		form: "single",
@@ -44845,7 +45025,7 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
-	"ptz.getHomeReturnSeconds": [{
+	"ptz.getHomeReturn": [{
 		name: "deviceId",
 		form: "single",
 		optional: false
@@ -44895,7 +45075,7 @@ Object.freeze({
 		form: "single",
 		optional: false
 	}],
-	"ptz.setHomeReturnSeconds": [{
+	"ptz.setHomeReturn": [{
 		name: "deviceId",
 		form: "single",
 		optional: false
@@ -153883,21 +154063,28 @@ var ReolinkCamera = class ReolinkCamera extends BaseDevice {
 					}, channel);
 					this.ctx.logger.info("reolink: guard point pinned to the current position", { tags: { deviceId: this.id } });
 				},
-				getHomeReturnSeconds: async ({ deviceId }) => {
+				getHomeReturn: async ({ deviceId }) => {
 					if (deviceId !== this.id) return null;
 					const guard = await this.readGuardStatus();
-					return guard === null ? null : guard.timeoutSeconds;
+					return guard === null ? null : {
+						enabled: guard.enabled,
+						seconds: guard.timeoutSeconds
+					};
 				},
-				setHomeReturnSeconds: async ({ deviceId, seconds }) => {
+				setHomeReturn: async ({ deviceId, enabled, seconds }) => {
 					if (deviceId !== this.id) return;
-					const current = await this.readGuardStatus();
-					const api = await this.ensureApi();
-					const channel = this.getChannel();
-					await api.setPtzGuard({
-						enabled: current?.enabled ?? true,
+					await (await this.ensureApi()).setPtzGuard({
+						enabled,
 						timeoutSeconds: seconds,
 						setPosition: false
-					}, channel);
+					}, this.getChannel());
+					this.ctx.logger.info("reolink: guard point idle-return updated", {
+						tags: { deviceId: this.id },
+						meta: {
+							enabled,
+							seconds
+						}
+					});
 				},
 				setHomePreset: async ({ deviceId, presetId }) => {
 					if (deviceId !== this.id) return;

package/package.json

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