@camstack/system 1.2.106 → 1.2.107

package/dist/builtins/device-manager/device-manager.addon.js

@@ -953,7 +953,7 @@ function resolveDeviceById(registry, deviceId) {
 //#region src/builtins/device-manager/device-queries.ts
 async function listPersistedByAddon(pctx, input) {
 	const { addonId } = input;
-	const [index, meta] = await Promise.all([pctx.metaStore.readIndex(), pctx.metaStore.readMeta()]);
+	const { index, meta } = await pctx.metaStore.readAll();
 	const stableIds = index[addonId] ?? [];
 	const byStableId = /* @__PURE__ */ new Map();
 	for (const m of Object.values(meta)) if (m.addonId === addonId) byStableId.set(m.stableId, m);
@@ -978,8 +978,7 @@ async function listAll(pctx, input) {
 	const camerasOnly = input.isCamera === true;
 	const results = [];
 	const seen = /* @__PURE__ */ new Set();
-	const meta = await pctx.metaStore.readMeta();
-	const metadataMap = await pctx.metaStore.readMetadataMap();
+	const { meta, metadata: metadataMap, index } = await pctx.metaStore.readAll();
 	if (pctx.registry) {
 		const liveEntries = addonId ? pctx.registry.getAllForAddon(addonId).map((device) => ({
 			addonId,
@@ -997,7 +996,6 @@ async function listAll(pctx, input) {
 			} : info);
 		}
 	}
-	const index = await pctx.metaStore.readIndex();
 	const metaByAddonStable = /* @__PURE__ */ new Map();
 	for (const m of Object.values(meta)) metaByAddonStable.set(`${m.addonId}${m.stableId}`, m);
 	const targetAddons = addonId ? [addonId] : Object.keys(index);
@@ -1095,11 +1093,7 @@ async function getChildren(pctx, input) {
 	}
 	const results = [];
 	const seen = /* @__PURE__ */ new Set();
-	const [index, meta, metadataMap] = await Promise.all([
-		pctx.metaStore.readIndex(),
-		pctx.metaStore.readMeta(),
-		pctx.metaStore.readMetadataMap()
-	]);
+	const { index, meta, metadata: metadataMap } = await pctx.metaStore.readAll();
 	if (pctx.registry) {
 		const liveChildren = pctx.registry.getChildren(parentDeviceId);
 		for (const device of liveChildren) {
@@ -3875,8 +3869,61 @@ var DeviceMetaStore = class {
 		this.settings = settings;
 		this.registry = registry;
 	}
+	/** The read currently in flight, or null. Never a settled value — see
+	*  {@link readStore}. */
+	inFlightRead = null;
+	/**
+	* The whole persisted addon store.
+	*
+	* **Concurrent callers join the read already in flight.** This is not a
+	* cache and nothing survives settlement: a caller that awaited the running
+	* promise could not have observed anything older than its result, so the
+	* only thing that changes is cost. What that cost was, measured on the
+	* 2026-08-19 boot: `readAddonStore` lands in `SqliteSettingsBackend
+	* .getAllAddon`, which reads and `JSON.parse`s this addon's rows —
+	* 625 KB on the live hub (`deviceMeta` 467 KB + `deviceMetadata` 87 KB +
+	* `deviceIndex` 70 KB) — synchronously, on the hub's event loop. A V8
+	* profile of that boot had hub-main's JS thread 99.9% busy with 64% of it
+	* inside `getAllAddon`, ~89% of that entered here, and every runner's first
+	* store read queued behind it (the notification centre's six parallel reads
+	* all resolved together at t+44.7 s).
+	*
+	* A rejection is NOT latched: the slot is cleared before the promise
+	* settles either way, so a failed read costs the joiners that one failure
+	* and the next caller reaches the store again.
+	*/
 	readStore = async () => {
-		return await this.settings.readAddonStore();
+		const existing = this.inFlightRead;
+		if (existing !== null) return existing;
+		const read = (async () => {
+			try {
+				return await this.settings.readAddonStore();
+			} finally {
+				this.inFlightRead = null;
+			}
+		})();
+		this.inFlightRead = read;
+		return read;
+	};
+	/**
+	* The three fleet projections from ONE read.
+	*
+	* `listAll` asked for `deviceMeta`, then `deviceMetadata`, then
+	* `deviceIndex` — three SEQUENTIAL awaits, which {@link readStore}'s
+	* in-flight join cannot collapse because each starts after the previous one
+	* settled. Three full 625 KB parses per call, on a call made once per device
+	* lifecycle event during a 974-device boot: 23% of hub-main's CPU.
+	*
+	* It is also ONE snapshot. Three separate reads could straddle a write and
+	* hand back an index that names a device the meta map no longer has.
+	*/
+	readAll = async () => {
+		const store = await this.readStore();
+		return {
+			index: store.deviceIndex ?? {},
+			meta: store.deviceMeta ?? {},
+			metadata: store.deviceMetadata ?? {}
+		};
 	};
 	readIndex = async () => {
 		return (await this.readStore()).deviceIndex ?? {};

package/dist/builtins/device-manager/device-manager.addon.mjs

@@ -948,7 +948,7 @@ function resolveDeviceById(registry, deviceId) {
 //#region src/builtins/device-manager/device-queries.ts
 async function listPersistedByAddon(pctx, input) {
 	const { addonId } = input;
-	const [index, meta] = await Promise.all([pctx.metaStore.readIndex(), pctx.metaStore.readMeta()]);
+	const { index, meta } = await pctx.metaStore.readAll();
 	const stableIds = index[addonId] ?? [];
 	const byStableId = /* @__PURE__ */ new Map();
 	for (const m of Object.values(meta)) if (m.addonId === addonId) byStableId.set(m.stableId, m);
@@ -973,8 +973,7 @@ async function listAll(pctx, input) {
 	const camerasOnly = input.isCamera === true;
 	const results = [];
 	const seen = /* @__PURE__ */ new Set();
-	const meta = await pctx.metaStore.readMeta();
-	const metadataMap = await pctx.metaStore.readMetadataMap();
+	const { meta, metadata: metadataMap, index } = await pctx.metaStore.readAll();
 	if (pctx.registry) {
 		const liveEntries = addonId ? pctx.registry.getAllForAddon(addonId).map((device) => ({
 			addonId,
@@ -992,7 +991,6 @@ async function listAll(pctx, input) {
 			} : info);
 		}
 	}
-	const index = await pctx.metaStore.readIndex();
 	const metaByAddonStable = /* @__PURE__ */ new Map();
 	for (const m of Object.values(meta)) metaByAddonStable.set(`${m.addonId}${m.stableId}`, m);
 	const targetAddons = addonId ? [addonId] : Object.keys(index);
@@ -1090,11 +1088,7 @@ async function getChildren(pctx, input) {
 	}
 	const results = [];
 	const seen = /* @__PURE__ */ new Set();
-	const [index, meta, metadataMap] = await Promise.all([
-		pctx.metaStore.readIndex(),
-		pctx.metaStore.readMeta(),
-		pctx.metaStore.readMetadataMap()
-	]);
+	const { index, meta, metadata: metadataMap } = await pctx.metaStore.readAll();
 	if (pctx.registry) {
 		const liveChildren = pctx.registry.getChildren(parentDeviceId);
 		for (const device of liveChildren) {
@@ -3870,8 +3864,61 @@ var DeviceMetaStore = class {
 		this.settings = settings;
 		this.registry = registry;
 	}
+	/** The read currently in flight, or null. Never a settled value — see
+	*  {@link readStore}. */
+	inFlightRead = null;
+	/**
+	* The whole persisted addon store.
+	*
+	* **Concurrent callers join the read already in flight.** This is not a
+	* cache and nothing survives settlement: a caller that awaited the running
+	* promise could not have observed anything older than its result, so the
+	* only thing that changes is cost. What that cost was, measured on the
+	* 2026-08-19 boot: `readAddonStore` lands in `SqliteSettingsBackend
+	* .getAllAddon`, which reads and `JSON.parse`s this addon's rows —
+	* 625 KB on the live hub (`deviceMeta` 467 KB + `deviceMetadata` 87 KB +
+	* `deviceIndex` 70 KB) — synchronously, on the hub's event loop. A V8
+	* profile of that boot had hub-main's JS thread 99.9% busy with 64% of it
+	* inside `getAllAddon`, ~89% of that entered here, and every runner's first
+	* store read queued behind it (the notification centre's six parallel reads
+	* all resolved together at t+44.7 s).
+	*
+	* A rejection is NOT latched: the slot is cleared before the promise
+	* settles either way, so a failed read costs the joiners that one failure
+	* and the next caller reaches the store again.
+	*/
 	readStore = async () => {
-		return await this.settings.readAddonStore();
+		const existing = this.inFlightRead;
+		if (existing !== null) return existing;
+		const read = (async () => {
+			try {
+				return await this.settings.readAddonStore();
+			} finally {
+				this.inFlightRead = null;
+			}
+		})();
+		this.inFlightRead = read;
+		return read;
+	};
+	/**
+	* The three fleet projections from ONE read.
+	*
+	* `listAll` asked for `deviceMeta`, then `deviceMetadata`, then
+	* `deviceIndex` — three SEQUENTIAL awaits, which {@link readStore}'s
+	* in-flight join cannot collapse because each starts after the previous one
+	* settled. Three full 625 KB parses per call, on a call made once per device
+	* lifecycle event during a 974-device boot: 23% of hub-main's CPU.
+	*
+	* It is also ONE snapshot. Three separate reads could straddle a write and
+	* hand back an index that names a device the meta map no longer has.
+	*/
+	readAll = async () => {
+		const store = await this.readStore();
+		return {
+			index: store.deviceIndex ?? {},
+			meta: store.deviceMeta ?? {},
+			metadata: store.deviceMetadata ?? {}
+		};
 	};
 	readIndex = async () => {
 		return (await this.readStore()).deviceIndex ?? {};

package/dist/builtins/device-manager/device-meta-store.d.ts

@@ -19,7 +19,47 @@ export declare class DeviceMetaStore {
     constructor(settings: DeviceManagerSettings, 
     /** Hub-process device registry (null when this node owns no registry). */
     registry: IDeviceRegistry | null);
+    /** The read currently in flight, or null. Never a settled value — see
+     *  {@link readStore}. */
+    private inFlightRead;
+    /**
+     * The whole persisted addon store.
+     *
+     * **Concurrent callers join the read already in flight.** This is not a
+     * cache and nothing survives settlement: a caller that awaited the running
+     * promise could not have observed anything older than its result, so the
+     * only thing that changes is cost. What that cost was, measured on the
+     * 2026-08-19 boot: `readAddonStore` lands in `SqliteSettingsBackend
+     * .getAllAddon`, which reads and `JSON.parse`s this addon's rows —
+     * 625 KB on the live hub (`deviceMeta` 467 KB + `deviceMetadata` 87 KB +
+     * `deviceIndex` 70 KB) — synchronously, on the hub's event loop. A V8
+     * profile of that boot had hub-main's JS thread 99.9% busy with 64% of it
+     * inside `getAllAddon`, ~89% of that entered here, and every runner's first
+     * store read queued behind it (the notification centre's six parallel reads
+     * all resolved together at t+44.7 s).
+     *
+     * A rejection is NOT latched: the slot is cleared before the promise
+     * settles either way, so a failed read costs the joiners that one failure
+     * and the next caller reaches the store again.
+     */
     readStore: () => Promise<AddonStore>;
+    /**
+     * The three fleet projections from ONE read.
+     *
+     * `listAll` asked for `deviceMeta`, then `deviceMetadata`, then
+     * `deviceIndex` — three SEQUENTIAL awaits, which {@link readStore}'s
+     * in-flight join cannot collapse because each starts after the previous one
+     * settled. Three full 625 KB parses per call, on a call made once per device
+     * lifecycle event during a 974-device boot: 23% of hub-main's CPU.
+     *
+     * It is also ONE snapshot. Three separate reads could straddle a write and
+     * hand back an index that names a device the meta map no longer has.
+     */
+    readAll: () => Promise<{
+        readonly index: Record<string, string[]>;
+        readonly meta: Record<string, PersistedDeviceMeta>;
+        readonly metadata: Record<string, Record<string, unknown>>;
+    }>;
     readIndex: () => Promise<Record<string, string[]>>;
     readMeta: () => Promise<Record<string, PersistedDeviceMeta>>;
     /** Hardware-identity metadata map. Lives in a sibling key on the

package/dist/builtins/sqlite-storage/sqlite-settings-backend.d.ts

@@ -1,4 +1,4 @@
-import { CollectionColumn, CollectionIndex, DataStoreEngineInfo, HistogramBucket, ISettingsBackend, SettingsCountInput, SettingsDeleteInput, SettingsGetInput, SettingsHistogramInput, SettingsInsertInput, SettingsIsEmptyInput, SettingsQueryInput, SettingsRecord, SettingsSetInput, SettingsUpdateInput, IScopedLogger } from '@camstack/types';
+import { CollectionColumn, CollectionIndex, DataStoreEngineInfo, HistogramBucket, IScopedLogger, ISettingsBackend, SettingsCountInput, SettingsDeleteInput, SettingsGetInput, SettingsHistogramInput, SettingsInsertInput, SettingsIsEmptyInput, SettingsQueryInput, SettingsRecord, SettingsSetInput, SettingsUpdateInput } from '@camstack/types';
 import { default as Database } from 'better-sqlite3';
 import { MutationFilterInput } from './filter-compiler.js';
 /** Input for {@link SqliteSettingsBackend.deleteWhere}. */
@@ -123,7 +123,29 @@ export declare class SqliteSettingsBackend implements ISettingsBackend {
     setSystem(key: string, value: unknown): void;
     /** Get all system settings as flat key-value */
     getAllSystem(): Record<string, unknown>;
-    /** Get all settings for an addon */
+    /**
+     * Get all settings for an addon.
+     *
+     * Selected by the SAME key range {@link setAllAddon} deletes and re-inserts
+     * — `prefixWhere("<addonId>.")` — and for the same reason `getAllScoped`
+     * uses it: `"<addonId>.<key>"` is the PRIMARY KEY, so the range is a
+     * `SEARCH … USING INDEX` while anything else is a full `SCAN`.
+     *
+     * It used to select on `json_extract(data, '$.addonId')`, which is a second
+     * authority for "this addon's rows" and disagreed with the writer in both
+     * directions: a row inside the JSON's idea of the addon but outside the key
+     * range was READ and never DELETED (a config key that could not be cleared),
+     * and `json_extract` was evaluated on EVERY row — so a four-key addon paid
+     * for the whole table and one unparseable neighbour aborted the statement,
+     * taking out every addon's config read at once.
+     *
+     * The cost was not theoretical. On the live hub `addon-settings` is 627 KB
+     * in 24 rows, 625 KB of it device-manager's three fleet blobs; a V8 profile
+     * of a boot (2026-08-19) had hub-main's JS thread 99.9% busy with **64% of
+     * it inside this method**, which is what put a ~45 s queue in front of every
+     * runner's first store read. Measured on that table: 0.947 ms → ~0.02 ms for
+     * a four-key addon, 2.90 ms → 2.13 ms for device-manager's own.
+     */
     getAllAddon(addonId: string): Record<string, unknown>;
     /** Bulk-set all settings for an addon */
     setAllAddon(addonId: string, config: Record<string, unknown>): void;

package/dist/builtins/sqlite-storage/sqlite-settings.addon.js

@@ -321,6 +321,27 @@ function compileFilter$1(filter, shape, mode, serialize) {
 	};
 }
 //#endregion
+//#region src/builtins/sqlite-storage/prefix-range.ts
+var MAX_ASCII = 127;
+/**
+* @returns the range for `prefix`, or `null` when the caller must fall back to
+*   a `LIKE` scan (empty prefix — which is the whole table and has no upper
+*   bound — or any byte outside ASCII).
+*/
+function prefixRange(prefix) {
+	if (prefix.length === 0) return null;
+	for (let i = 0; i < prefix.length; i++) {
+		const code = prefix.charCodeAt(i);
+		if (code === 0 || code > MAX_ASCII) return null;
+	}
+	const last = prefix.charCodeAt(prefix.length - 1);
+	if (last >= MAX_ASCII) return null;
+	return {
+		lo: prefix,
+		hi: prefix.slice(0, -1) + String.fromCharCode(last + 1)
+	};
+}
+//#endregion
 //#region src/builtins/sqlite-storage/query-bounds.ts
 /**
 * Defensive row bounds for `data-store-provider.query`.
@@ -442,27 +463,6 @@ function isImposedBound(source) {
 	return source !== "caller";
 }
 //#endregion
-//#region src/builtins/sqlite-storage/prefix-range.ts
-var MAX_ASCII = 127;
-/**
-* @returns the range for `prefix`, or `null` when the caller must fall back to
-*   a `LIKE` scan (empty prefix — which is the whole table and has no upper
-*   bound — or any byte outside ASCII).
-*/
-function prefixRange(prefix) {
-	if (prefix.length === 0) return null;
-	for (let i = 0; i < prefix.length; i++) {
-		const code = prefix.charCodeAt(i);
-		if (code === 0 || code > MAX_ASCII) return null;
-	}
-	const last = prefix.charCodeAt(prefix.length - 1);
-	if (last >= MAX_ASCII) return null;
-	return {
-		lo: prefix,
-		hi: prefix.slice(0, -1) + String.fromCharCode(last + 1)
-	};
-}
-//#endregion
 //#region src/builtins/sqlite-storage/sqlite-settings-backend.ts
 function parseRowData(raw) {
 	return require_dist.asJsonObject(require_dist.parseJsonUnknown(raw)) ?? {};
@@ -890,10 +890,33 @@ var SqliteSettingsBackend = class SqliteSettingsBackend {
 		const rows = this.getDb().prepare("SELECT id, data FROM \"system-settings\"").all();
 		return Object.fromEntries(rows.map((r) => [r.id, JSON.parse(r.data)]));
 	}
-	/** Get all settings for an addon */
+	/**
+	* Get all settings for an addon.
+	*
+	* Selected by the SAME key range {@link setAllAddon} deletes and re-inserts
+	* — `prefixWhere("<addonId>.")` — and for the same reason `getAllScoped`
+	* uses it: `"<addonId>.<key>"` is the PRIMARY KEY, so the range is a
+	* `SEARCH … USING INDEX` while anything else is a full `SCAN`.
+	*
+	* It used to select on `json_extract(data, '$.addonId')`, which is a second
+	* authority for "this addon's rows" and disagreed with the writer in both
+	* directions: a row inside the JSON's idea of the addon but outside the key
+	* range was READ and never DELETED (a config key that could not be cleared),
+	* and `json_extract` was evaluated on EVERY row — so a four-key addon paid
+	* for the whole table and one unparseable neighbour aborted the statement,
+	* taking out every addon's config read at once.
+	*
+	* The cost was not theoretical. On the live hub `addon-settings` is 627 KB
+	* in 24 rows, 625 KB of it device-manager's three fleet blobs; a V8 profile
+	* of a boot (2026-08-19) had hub-main's JS thread 99.9% busy with **64% of
+	* it inside this method**, which is what put a ~45 s queue in front of every
+	* runner's first store read. Measured on that table: 0.947 ms → ~0.02 ms for
+	* a four-key addon, 2.90 ms → 2.13 ms for device-manager's own.
+	*/
 	getAllAddon(addonId) {
 		this.requireDeclared("addon-settings");
-		const rows = this.getDb().prepare("SELECT id, data FROM \"addon-settings\" WHERE json_extract(data, '$.addonId') = ?").all(addonId);
+		const where = this.prefixWhere(`${addonId}.`);
+		const rows = this.getDb().prepare(`SELECT id, data FROM "addon-settings" WHERE ${where.sql}`).all(...where.params);
 		if (rows.length === 0) return {};
 		const result = {};
 		for (const row of rows) {

package/dist/builtins/sqlite-storage/sqlite-settings.addon.mjs

@@ -315,6 +315,27 @@ function compileFilter$1(filter, shape, mode, serialize) {
 	};
 }
 //#endregion
+//#region src/builtins/sqlite-storage/prefix-range.ts
+var MAX_ASCII = 127;
+/**
+* @returns the range for `prefix`, or `null` when the caller must fall back to
+*   a `LIKE` scan (empty prefix — which is the whole table and has no upper
+*   bound — or any byte outside ASCII).
+*/
+function prefixRange(prefix) {
+	if (prefix.length === 0) return null;
+	for (let i = 0; i < prefix.length; i++) {
+		const code = prefix.charCodeAt(i);
+		if (code === 0 || code > MAX_ASCII) return null;
+	}
+	const last = prefix.charCodeAt(prefix.length - 1);
+	if (last >= MAX_ASCII) return null;
+	return {
+		lo: prefix,
+		hi: prefix.slice(0, -1) + String.fromCharCode(last + 1)
+	};
+}
+//#endregion
 //#region src/builtins/sqlite-storage/query-bounds.ts
 /**
 * Defensive row bounds for `data-store-provider.query`.
@@ -436,27 +457,6 @@ function isImposedBound(source) {
 	return source !== "caller";
 }
 //#endregion
-//#region src/builtins/sqlite-storage/prefix-range.ts
-var MAX_ASCII = 127;
-/**
-* @returns the range for `prefix`, or `null` when the caller must fall back to
-*   a `LIKE` scan (empty prefix — which is the whole table and has no upper
-*   bound — or any byte outside ASCII).
-*/
-function prefixRange(prefix) {
-	if (prefix.length === 0) return null;
-	for (let i = 0; i < prefix.length; i++) {
-		const code = prefix.charCodeAt(i);
-		if (code === 0 || code > MAX_ASCII) return null;
-	}
-	const last = prefix.charCodeAt(prefix.length - 1);
-	if (last >= MAX_ASCII) return null;
-	return {
-		lo: prefix,
-		hi: prefix.slice(0, -1) + String.fromCharCode(last + 1)
-	};
-}
-//#endregion
 //#region src/builtins/sqlite-storage/sqlite-settings-backend.ts
 function parseRowData(raw) {
 	return asJsonObject(parseJsonUnknown(raw)) ?? {};
@@ -884,10 +884,33 @@ var SqliteSettingsBackend = class SqliteSettingsBackend {
 		const rows = this.getDb().prepare("SELECT id, data FROM \"system-settings\"").all();
 		return Object.fromEntries(rows.map((r) => [r.id, JSON.parse(r.data)]));
 	}
-	/** Get all settings for an addon */
+	/**
+	* Get all settings for an addon.
+	*
+	* Selected by the SAME key range {@link setAllAddon} deletes and re-inserts
+	* — `prefixWhere("<addonId>.")` — and for the same reason `getAllScoped`
+	* uses it: `"<addonId>.<key>"` is the PRIMARY KEY, so the range is a
+	* `SEARCH … USING INDEX` while anything else is a full `SCAN`.
+	*
+	* It used to select on `json_extract(data, '$.addonId')`, which is a second
+	* authority for "this addon's rows" and disagreed with the writer in both
+	* directions: a row inside the JSON's idea of the addon but outside the key
+	* range was READ and never DELETED (a config key that could not be cleared),
+	* and `json_extract` was evaluated on EVERY row — so a four-key addon paid
+	* for the whole table and one unparseable neighbour aborted the statement,
+	* taking out every addon's config read at once.
+	*
+	* The cost was not theoretical. On the live hub `addon-settings` is 627 KB
+	* in 24 rows, 625 KB of it device-manager's three fleet blobs; a V8 profile
+	* of a boot (2026-08-19) had hub-main's JS thread 99.9% busy with **64% of
+	* it inside this method**, which is what put a ~45 s queue in front of every
+	* runner's first store read. Measured on that table: 0.947 ms → ~0.02 ms for
+	* a four-key addon, 2.90 ms → 2.13 ms for device-manager's own.
+	*/
 	getAllAddon(addonId) {
 		this.requireDeclared("addon-settings");
-		const rows = this.getDb().prepare("SELECT id, data FROM \"addon-settings\" WHERE json_extract(data, '$.addonId') = ?").all(addonId);
+		const where = this.prefixWhere(`${addonId}.`);
+		const rows = this.getDb().prepare(`SELECT id, data FROM "addon-settings" WHERE ${where.sql}`).all(...where.params);
 		if (rows.length === 0) return {};
 		const result = {};
 		for (const row of rows) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/system",
-  "version": "1.2.106",
+  "version": "1.2.107",
   "description": "Core addon for CamStack — builtins, pipeline, process management, auth, logging, events",
   "keywords": [
     "camstack",