@camstack/addon-terminal 0.1.12 → 0.1.13

package/dist/addon.js

@@ -18428,24 +18428,28 @@ var snapshotCapability = {
 		*
 		* `getSnapshotOverview` is cache-only by contract: it answers from whatever
 		* the wrapper happens to hold and never captures. Under D93 the client
-		* versions its image URL on that answer, and an image REQUEST is what enrols
-		* a camera in the keep-warm loop. Both of those are satisfiable by the
-		* client's own image cache — `expo-image` is URL-keyed and never revalidates
-		* — so a URL painted in a previous session comes off disk with no network,
-		* no enrolment, and nothing warming. Measured on the live hub: reopening
-		* after two minutes idle painted 15 of 16 tiles at **168 s old** with zero
-		* HTTP requests, and the fleet only recovered because a later poll happened
-		* to observe a different identity.
+		* versions its image URL on that answer, and an image REQUEST was the only
+		* demand signal. Both of those are satisfiable by the client's own image
+		* cache — `expo-image` is URL-keyed and never revalidates — so a URL painted
+		* in a previous session comes off disk with no network, no demand, and no
+		* capture. Measured on the live hub: reopening after two minutes idle
+		* painted 15 of 16 tiles at **168 s old** with zero HTTP requests, and the
+		* fleet only recovered because a later poll happened to observe a different
+		* identity.
 		*
 		* ## The two properties that fix it
 		*
 		* **It is an RPC, so no client cache can answer it.** The demand signal
-		* always reaches the wrapper. This method therefore MAY create keep-warm
-		* subscriptions, where `getSnapshotOverview` must never (D93) — the
-		* distinction is not "one is newer" but that the overview poll is app-wide
-		* (a creating overview would warm every camera on the install) while this is
-		* called by a rendered surface naming the tiles it is actually painting, at
-		* the width it is painting them.
+		* always reaches the wrapper. This method therefore CAPTURES, where
+		* `getSnapshotOverview` must never (D93) — the distinction is not "one is
+		* newer" but that the overview poll is app-wide (a capturing overview would
+		* dial every camera on the install) while this is called by a rendered
+		* surface naming the tiles it is actually painting, at the width it is
+		* painting them.
+		*
+		* Since 2026-08-11 this is the ONLY thing that refreshes a snapshot: the
+		* server-side keep-warm loop was removed (operator directive — on-demand,
+		* always), so a camera nobody is looking at costs nothing at all.
 		*
 		* **It waits, briefly and boundedly, for the capture it triggered.** The
 		* returned `capturedAt` is the frame the link will serve, not the frame the
@@ -27414,9 +27418,10 @@ var DeclaredDevices = class {
 		}
 		const integrationId = spec.integrationId ?? await this.ensureIntegration(spec.integrationName);
 		const index = await this.readIndex();
+		const live = await this.readLiveByStableId();
 		const outcomes = [];
 		for (const declaration of spec.devices) {
-			const outcome = await this.applyDeclaration(declaration, integrationId, index);
+			const outcome = await this.applyDeclaration(declaration, integrationId, index, live);
 			if (outcome !== null) outcomes.push(outcome);
 		}
 		return {
@@ -27462,6 +27467,26 @@ var DeclaredDevices = class {
 		return new Map(rows.map((row) => [row.stableId, row]));
 	}
 	/**
+	* Devices this kernel already has CONSTRUCTED, by stableId.
+	*
+	* Distinct from {@link readIndex}, and the distinction is the bug: the index
+	* is persisted rows, this is live objects. A row without an object must be
+	* adopted; an object must be left exactly as it is.
+	*
+	* Failure is non-fatal and deliberately so — an empty map degrades to the
+	* previous behaviour (attempt the adopt) rather than skipping a device that
+	* genuinely needs bringing up.
+	*/
+	async readLiveByStableId() {
+		try {
+			const devices = await this.ports.devices.getAll();
+			return new Map(devices.map((device) => [device.stableId, device]));
+		} catch (err) {
+			this.ports.logger.warn("could not read live devices — falling back to adopt-by-row", { meta: { error: err instanceof Error ? err.message : String(err) } });
+			return /* @__PURE__ */ new Map();
+		}
+	}
+	/**
 	* One declaration: adopt what exists, create what does not.
 	*
 	* The create branch is the destructive one — it seeds `initialMeta`, and
@@ -27470,8 +27495,15 @@ var DeclaredDevices = class {
 	* the declared name over the operator's rename. D49: that branch needs a
 	* second read to agree.
 	*/
-	async applyDeclaration(declaration, integrationId, index) {
+	async applyDeclaration(declaration, integrationId, index, live) {
 		try {
+			const alreadyLive = live.get(declaration.stableId);
+			if (alreadyLive !== void 0) return {
+				stableId: declaration.stableId,
+				deviceId: alreadyLive.id,
+				device: alreadyLive,
+				created: false
+			};
 			let existing = index.get(declaration.stableId);
 			if (existing === void 0) {
 				existing = (await this.readIndex()).get(declaration.stableId);
@@ -33478,9 +33510,20 @@ function needsTerminalCameraConfigMigration(persistedConfig, declaration) {
 function escapeXml(value) {
 	return value.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll("\"", "&quot;").replaceAll("'", "&apos;");
 }
-/** Render already-interpreted terminal rows into a compact MJPEG frame. */
+/**
+* Render already-interpreted terminal rows into a compact MJPEG frame.
+*
+* `xml:space="preserve"` is load-bearing, not tidiness. SVG `<text>` collapses
+* runs of whitespace by default, and a terminal's entire column alignment IS
+* runs of whitespace — Glances pads every field with spaces. Without it the
+* frame drew each line at roughly half its true width, crammed into the
+* top-left of a mostly-black image, while the SAME session over `attach`
+* looked perfect — which is exactly how the operator reported it. Measured in
+* the hub's own rasterizer 2026-08-11: `AAA<10 spaces>BBB<3>CCC` drew 92 px
+* collapsed against 178 px preserved.
+*/
 async function renderTerminalJpeg(lines) {
-	const renderedLines = lines.slice(0, 40).map((line, index) => `<text x="8" y="${String(18 + index * 15)}">${escapeXml(line.slice(0, 120))}</text>`).join("");
+	const renderedLines = lines.slice(0, 40).map((line, index) => `<text xml:space="preserve" x="8" y="${String(18 + index * 15)}">${escapeXml(line.slice(0, 120))}</text>`).join("");
 	const svg = `<svg xmlns="http://www.w3.org/2000/svg" width="${String(960)}" height="${String(640)}"><rect width="100%" height="100%" fill="#0b0d10"/><g fill="#d7dce2" font-family="DejaVu Sans Mono,monospace" font-size="13">${renderedLines}</g></svg>`;
 	return (0, sharp.default)(Buffer.from(svg)).jpeg({
 		quality: 82,
@@ -39608,12 +39651,46 @@ var TerminalAddon = class extends BaseAddon {
 					this.migratedTerminalCameraConfigIds.add(outcome.device.id);
 				}
 			}
+			if (outcome.created) await this.silenceAnalysisFor(outcome.device.id);
 			const nodeId = outcome.device.config.get("nodeId");
 			const profileId = outcome.device.config.get("profileId");
 			const profileAvailable = this.cameraProfilesByNode.get(nodeId)?.some((profile) => profile.profileId === profileId) ?? false;
 			outcome.device.setNodeOnline((onlineByNode.get(nodeId) ?? false) && profileAvailable);
 		}
 	}
+	/**
+	* A Terminal camera is a rendered screen. Object detection on it finds
+	* nothing, forever, at full cost.
+	*
+	* Measured on the hub 2026-08-11, for ONE terminal camera: 19 frames per 10 s
+	* through the detection pipeline at ~61 ms of inference each, plus 115
+	* capture-scheduler requests a minute — against `detections=0`. Multiply by
+	* one terminal per node and it is a standing tax on a hub that was already
+	* shedding 86 % of its capture queue.
+	*
+	* Written through `setCameraSwitch`, which is the authority that already owns
+	* this function — [D62] forbids a second store that disagrees with it. And
+	* written ONLY on creation: an operator who deliberately turns detection back
+	* on for a terminal must win, and a reconcile that re-asserted every pass
+	* would silently overrule them once a minute.
+	*/
+	async silenceAnalysisFor(deviceId) {
+		for (const switchId of ["object-detection", "audio-analysis"]) try {
+			await this.ctx.api.pipelineOrchestrator.setCameraSwitch.mutate({
+				deviceId,
+				switchId,
+				enabled: false
+			});
+		} catch (err) {
+			this.ctx.logger.warn("could not switch off analysis for a Terminal camera", {
+				tags: { deviceId },
+				meta: {
+					switchId,
+					error: err instanceof Error ? err.message : String(err)
+				}
+			});
+		}
+	}
 	terminalInstanceControl() {
 		return {
 			listInstances: async () => this.terminalInstances().map((instance) => this.instanceInfo(instance)),

package/dist/addon.mjs

@@ -18406,24 +18406,28 @@ var snapshotCapability = {
 		*
 		* `getSnapshotOverview` is cache-only by contract: it answers from whatever
 		* the wrapper happens to hold and never captures. Under D93 the client
-		* versions its image URL on that answer, and an image REQUEST is what enrols
-		* a camera in the keep-warm loop. Both of those are satisfiable by the
-		* client's own image cache — `expo-image` is URL-keyed and never revalidates
-		* — so a URL painted in a previous session comes off disk with no network,
-		* no enrolment, and nothing warming. Measured on the live hub: reopening
-		* after two minutes idle painted 15 of 16 tiles at **168 s old** with zero
-		* HTTP requests, and the fleet only recovered because a later poll happened
-		* to observe a different identity.
+		* versions its image URL on that answer, and an image REQUEST was the only
+		* demand signal. Both of those are satisfiable by the client's own image
+		* cache — `expo-image` is URL-keyed and never revalidates — so a URL painted
+		* in a previous session comes off disk with no network, no demand, and no
+		* capture. Measured on the live hub: reopening after two minutes idle
+		* painted 15 of 16 tiles at **168 s old** with zero HTTP requests, and the
+		* fleet only recovered because a later poll happened to observe a different
+		* identity.
 		*
 		* ## The two properties that fix it
 		*
 		* **It is an RPC, so no client cache can answer it.** The demand signal
-		* always reaches the wrapper. This method therefore MAY create keep-warm
-		* subscriptions, where `getSnapshotOverview` must never (D93) — the
-		* distinction is not "one is newer" but that the overview poll is app-wide
-		* (a creating overview would warm every camera on the install) while this is
-		* called by a rendered surface naming the tiles it is actually painting, at
-		* the width it is painting them.
+		* always reaches the wrapper. This method therefore CAPTURES, where
+		* `getSnapshotOverview` must never (D93) — the distinction is not "one is
+		* newer" but that the overview poll is app-wide (a capturing overview would
+		* dial every camera on the install) while this is called by a rendered
+		* surface naming the tiles it is actually painting, at the width it is
+		* painting them.
+		*
+		* Since 2026-08-11 this is the ONLY thing that refreshes a snapshot: the
+		* server-side keep-warm loop was removed (operator directive — on-demand,
+		* always), so a camera nobody is looking at costs nothing at all.
 		*
 		* **It waits, briefly and boundedly, for the capture it triggered.** The
 		* returned `capturedAt` is the frame the link will serve, not the frame the
@@ -27392,9 +27396,10 @@ var DeclaredDevices = class {
 		}
 		const integrationId = spec.integrationId ?? await this.ensureIntegration(spec.integrationName);
 		const index = await this.readIndex();
+		const live = await this.readLiveByStableId();
 		const outcomes = [];
 		for (const declaration of spec.devices) {
-			const outcome = await this.applyDeclaration(declaration, integrationId, index);
+			const outcome = await this.applyDeclaration(declaration, integrationId, index, live);
 			if (outcome !== null) outcomes.push(outcome);
 		}
 		return {
@@ -27440,6 +27445,26 @@ var DeclaredDevices = class {
 		return new Map(rows.map((row) => [row.stableId, row]));
 	}
 	/**
+	* Devices this kernel already has CONSTRUCTED, by stableId.
+	*
+	* Distinct from {@link readIndex}, and the distinction is the bug: the index
+	* is persisted rows, this is live objects. A row without an object must be
+	* adopted; an object must be left exactly as it is.
+	*
+	* Failure is non-fatal and deliberately so — an empty map degrades to the
+	* previous behaviour (attempt the adopt) rather than skipping a device that
+	* genuinely needs bringing up.
+	*/
+	async readLiveByStableId() {
+		try {
+			const devices = await this.ports.devices.getAll();
+			return new Map(devices.map((device) => [device.stableId, device]));
+		} catch (err) {
+			this.ports.logger.warn("could not read live devices — falling back to adopt-by-row", { meta: { error: err instanceof Error ? err.message : String(err) } });
+			return /* @__PURE__ */ new Map();
+		}
+	}
+	/**
 	* One declaration: adopt what exists, create what does not.
 	*
 	* The create branch is the destructive one — it seeds `initialMeta`, and
@@ -27448,8 +27473,15 @@ var DeclaredDevices = class {
 	* the declared name over the operator's rename. D49: that branch needs a
 	* second read to agree.
 	*/
-	async applyDeclaration(declaration, integrationId, index) {
+	async applyDeclaration(declaration, integrationId, index, live) {
 		try {
+			const alreadyLive = live.get(declaration.stableId);
+			if (alreadyLive !== void 0) return {
+				stableId: declaration.stableId,
+				deviceId: alreadyLive.id,
+				device: alreadyLive,
+				created: false
+			};
 			let existing = index.get(declaration.stableId);
 			if (existing === void 0) {
 				existing = (await this.readIndex()).get(declaration.stableId);
@@ -33456,9 +33488,20 @@ function needsTerminalCameraConfigMigration(persistedConfig, declaration) {
 function escapeXml(value) {
 	return value.replaceAll("&", "&amp;").replaceAll("<", "&lt;").replaceAll(">", "&gt;").replaceAll("\"", "&quot;").replaceAll("'", "&apos;");
 }
-/** Render already-interpreted terminal rows into a compact MJPEG frame. */
+/**
+* Render already-interpreted terminal rows into a compact MJPEG frame.
+*
+* `xml:space="preserve"` is load-bearing, not tidiness. SVG `<text>` collapses
+* runs of whitespace by default, and a terminal's entire column alignment IS
+* runs of whitespace — Glances pads every field with spaces. Without it the
+* frame drew each line at roughly half its true width, crammed into the
+* top-left of a mostly-black image, while the SAME session over `attach`
+* looked perfect — which is exactly how the operator reported it. Measured in
+* the hub's own rasterizer 2026-08-11: `AAA<10 spaces>BBB<3>CCC` drew 92 px
+* collapsed against 178 px preserved.
+*/
 async function renderTerminalJpeg(lines) {
-	const renderedLines = lines.slice(0, 40).map((line, index) => `<text x="8" y="${String(18 + index * 15)}">${escapeXml(line.slice(0, 120))}</text>`).join("");
+	const renderedLines = lines.slice(0, 40).map((line, index) => `<text xml:space="preserve" x="8" y="${String(18 + index * 15)}">${escapeXml(line.slice(0, 120))}</text>`).join("");
 	const svg = `<svg xmlns="http://www.w3.org/2000/svg" width="${String(960)}" height="${String(640)}"><rect width="100%" height="100%" fill="#0b0d10"/><g fill="#d7dce2" font-family="DejaVu Sans Mono,monospace" font-size="13">${renderedLines}</g></svg>`;
 	return sharp(Buffer.from(svg)).jpeg({
 		quality: 82,
@@ -39586,12 +39629,46 @@ var TerminalAddon = class extends BaseAddon {
 					this.migratedTerminalCameraConfigIds.add(outcome.device.id);
 				}
 			}
+			if (outcome.created) await this.silenceAnalysisFor(outcome.device.id);
 			const nodeId = outcome.device.config.get("nodeId");
 			const profileId = outcome.device.config.get("profileId");
 			const profileAvailable = this.cameraProfilesByNode.get(nodeId)?.some((profile) => profile.profileId === profileId) ?? false;
 			outcome.device.setNodeOnline((onlineByNode.get(nodeId) ?? false) && profileAvailable);
 		}
 	}
+	/**
+	* A Terminal camera is a rendered screen. Object detection on it finds
+	* nothing, forever, at full cost.
+	*
+	* Measured on the hub 2026-08-11, for ONE terminal camera: 19 frames per 10 s
+	* through the detection pipeline at ~61 ms of inference each, plus 115
+	* capture-scheduler requests a minute — against `detections=0`. Multiply by
+	* one terminal per node and it is a standing tax on a hub that was already
+	* shedding 86 % of its capture queue.
+	*
+	* Written through `setCameraSwitch`, which is the authority that already owns
+	* this function — [D62] forbids a second store that disagrees with it. And
+	* written ONLY on creation: an operator who deliberately turns detection back
+	* on for a terminal must win, and a reconcile that re-asserted every pass
+	* would silently overrule them once a minute.
+	*/
+	async silenceAnalysisFor(deviceId) {
+		for (const switchId of ["object-detection", "audio-analysis"]) try {
+			await this.ctx.api.pipelineOrchestrator.setCameraSwitch.mutate({
+				deviceId,
+				switchId,
+				enabled: false
+			});
+		} catch (err) {
+			this.ctx.logger.warn("could not switch off analysis for a Terminal camera", {
+				tags: { deviceId },
+				meta: {
+					switchId,
+					error: err instanceof Error ? err.message : String(err)
+				}
+			});
+		}
+	}
 	terminalInstanceControl() {
 		return {
 			listInstances: async () => this.terminalInstances().map((instance) => this.instanceInfo(instance)),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/addon-terminal",
-  "version": "0.1.12",
+  "version": "0.1.13",
   "description": "Interactive terminal sessions (pty + xterm) as a CamStack addon",
   "keywords": [
     "camstack",