@camstack/system 1.2.156 → 1.2.157

package/dist/builtins/storage-orchestrator/storage-orchestrator.addon.js

@@ -425,8 +425,8 @@ var StorageMigrationCoordinator = class {
 				finishedAt: null,
 				error: null
 			};
-			this.active = job;
 			await this.persist(job);
+			this.active = job;
 			this.runPromise = this.run(job);
 			this.runPromise;
 			return job.jobId;

package/dist/builtins/storage-orchestrator/storage-orchestrator.addon.mjs

@@ -418,8 +418,8 @@ var StorageMigrationCoordinator = class {
 				finishedAt: null,
 				error: null
 			};
-			this.active = job;
 			await this.persist(job);
+			this.active = job;
 			this.runPromise = this.run(job);
 			this.runPromise;
 			return job.jobId;

package/dist/index.js

@@ -9612,20 +9612,63 @@ var ConfigManager = class ConfigManager {
 		if (this.settingsStore === null) return;
 		this.settingsStore.clearDeviceRuntimeState(deviceId);
 	}
+	/**
+	* The addon's `ctx.settings`.
+	*
+	* The backend is chosen on EVERY call, never frozen when the view is built.
+	* That is not a nicety: `settings-store` is owned by `storage-orchestrator`,
+	* so the door is wired by `registerProvider` STRICTLY AFTER that addon's
+	* context — and therefore its settings view — already exists. A view bound at
+	* construction handed the door's own owner the sync-store backend, whose
+	* writes throw for the whole life of the process once `sqlite-settings` runs
+	* isolated (D181: no sync `ISettingsStore` reaches hub-main, only the async
+	* door). Reads were worse than the writes: they answered `{}` in silence.
+	* That is what made `storageMigration.start` fail on its first durable write
+	* while every other addon's persistence worked. See D294.
+	*/
 	createSettingsView(addonId) {
-		if (this.settingsStore === null && this.settingsDoor !== null) {
+		const storeBacked = this.createStoreSettingsView(addonId);
+		let doorBacked = null;
+		let doorBackedSource = null;
+		const active = () => {
+			if (this.settingsStore !== null) return storeBacked;
 			const door = this.settingsDoor;
-			return createDoorSettingsView(addonId, door, {
-				getSection: (section) => this.getSection(section),
-				setSection: async (section, patch) => {
-					for (const [key, value] of Object.entries(patch)) await door.set({
-						collection: "system-settings",
-						key: `${section}.${key}`,
-						value
-					});
-				}
-			});
-		}
+			if (door === null) return storeBacked;
+			if (doorBacked === null || doorBackedSource !== door) {
+				doorBackedSource = door;
+				doorBacked = this.createDoorBackedSettingsView(addonId, door);
+			}
+			return doorBacked;
+		};
+		return {
+			readAddonStore: () => active().readAddonStore(),
+			writeAddonStore: (patch) => active().writeAddonStore(patch),
+			readDeviceStore: (deviceId) => active().readDeviceStore(deviceId),
+			readDeviceStoreBatch: (deviceIds) => active().readDeviceStoreBatch(deviceIds),
+			writeDeviceStore: (deviceId, patch) => active().writeDeviceStore(deviceId, patch),
+			clearDeviceStore: (deviceId) => active().clearDeviceStore(deviceId),
+			getSection: (section) => active().getSection(section),
+			setSection: (section, patch) => active().setSection(section, patch),
+			readDeviceRuntimeState: (deviceId) => active().readDeviceRuntimeState(deviceId),
+			writeDeviceRuntimeState: (deviceId, data) => active().writeDeviceRuntimeState(deviceId, data),
+			clearDeviceRuntimeState: (deviceId) => active().clearDeviceRuntimeState(deviceId)
+		};
+	}
+	/** The async-door backend of {@link createSettingsView}. */
+	createDoorBackedSettingsView(addonId, door) {
+		return createDoorSettingsView(addonId, door, {
+			getSection: (section) => this.getSection(section),
+			setSection: async (section, patch) => {
+				for (const [key, value] of Object.entries(patch)) await door.set({
+					collection: "system-settings",
+					key: `${section}.${key}`,
+					value
+				});
+			}
+		});
+	}
+	/** The sync-`ISettingsStore` backend of {@link createSettingsView}. */
+	createStoreSettingsView(addonId) {
 		const cm = this;
 		return {
 			async readAddonStore() {
@@ -9641,6 +9684,15 @@ var ConfigManager = class ConfigManager {
 			async readDeviceStore(deviceId) {
 				return cm.getAddonDevice(addonId, String(deviceId));
 			},
+			async readDeviceStoreBatch(deviceIds) {
+				const out = /* @__PURE__ */ new Map();
+				for (const deviceId of deviceIds) try {
+					out.set(deviceId, cm.getAddonDevice(addonId, String(deviceId)));
+				} catch {
+					out.set(deviceId, {});
+				}
+				return out;
+			},
 			async writeDeviceStore(deviceId, patch) {
 				const key = String(deviceId);
 				const existing = cm.getAddonDevice(addonId, key);

package/dist/index.mjs

@@ -9605,20 +9605,63 @@ var ConfigManager = class ConfigManager {
 		if (this.settingsStore === null) return;
 		this.settingsStore.clearDeviceRuntimeState(deviceId);
 	}
+	/**
+	* The addon's `ctx.settings`.
+	*
+	* The backend is chosen on EVERY call, never frozen when the view is built.
+	* That is not a nicety: `settings-store` is owned by `storage-orchestrator`,
+	* so the door is wired by `registerProvider` STRICTLY AFTER that addon's
+	* context — and therefore its settings view — already exists. A view bound at
+	* construction handed the door's own owner the sync-store backend, whose
+	* writes throw for the whole life of the process once `sqlite-settings` runs
+	* isolated (D181: no sync `ISettingsStore` reaches hub-main, only the async
+	* door). Reads were worse than the writes: they answered `{}` in silence.
+	* That is what made `storageMigration.start` fail on its first durable write
+	* while every other addon's persistence worked. See D294.
+	*/
 	createSettingsView(addonId) {
-		if (this.settingsStore === null && this.settingsDoor !== null) {
+		const storeBacked = this.createStoreSettingsView(addonId);
+		let doorBacked = null;
+		let doorBackedSource = null;
+		const active = () => {
+			if (this.settingsStore !== null) return storeBacked;
 			const door = this.settingsDoor;
-			return createDoorSettingsView(addonId, door, {
-				getSection: (section) => this.getSection(section),
-				setSection: async (section, patch) => {
-					for (const [key, value] of Object.entries(patch)) await door.set({
-						collection: "system-settings",
-						key: `${section}.${key}`,
-						value
-					});
-				}
-			});
-		}
+			if (door === null) return storeBacked;
+			if (doorBacked === null || doorBackedSource !== door) {
+				doorBackedSource = door;
+				doorBacked = this.createDoorBackedSettingsView(addonId, door);
+			}
+			return doorBacked;
+		};
+		return {
+			readAddonStore: () => active().readAddonStore(),
+			writeAddonStore: (patch) => active().writeAddonStore(patch),
+			readDeviceStore: (deviceId) => active().readDeviceStore(deviceId),
+			readDeviceStoreBatch: (deviceIds) => active().readDeviceStoreBatch(deviceIds),
+			writeDeviceStore: (deviceId, patch) => active().writeDeviceStore(deviceId, patch),
+			clearDeviceStore: (deviceId) => active().clearDeviceStore(deviceId),
+			getSection: (section) => active().getSection(section),
+			setSection: (section, patch) => active().setSection(section, patch),
+			readDeviceRuntimeState: (deviceId) => active().readDeviceRuntimeState(deviceId),
+			writeDeviceRuntimeState: (deviceId, data) => active().writeDeviceRuntimeState(deviceId, data),
+			clearDeviceRuntimeState: (deviceId) => active().clearDeviceRuntimeState(deviceId)
+		};
+	}
+	/** The async-door backend of {@link createSettingsView}. */
+	createDoorBackedSettingsView(addonId, door) {
+		return createDoorSettingsView(addonId, door, {
+			getSection: (section) => this.getSection(section),
+			setSection: async (section, patch) => {
+				for (const [key, value] of Object.entries(patch)) await door.set({
+					collection: "system-settings",
+					key: `${section}.${key}`,
+					value
+				});
+			}
+		});
+	}
+	/** The sync-`ISettingsStore` backend of {@link createSettingsView}. */
+	createStoreSettingsView(addonId) {
 		const cm = this;
 		return {
 			async readAddonStore() {
@@ -9634,6 +9677,15 @@ var ConfigManager = class ConfigManager {
 			async readDeviceStore(deviceId) {
 				return cm.getAddonDevice(addonId, String(deviceId));
 			},
+			async readDeviceStoreBatch(deviceIds) {
+				const out = /* @__PURE__ */ new Map();
+				for (const deviceId of deviceIds) try {
+					out.set(deviceId, cm.getAddonDevice(addonId, String(deviceId)));
+				} catch {
+					out.set(deviceId, {});
+				}
+				return out;
+			},
 			async writeDeviceStore(deviceId, patch) {
 				const key = String(deviceId);
 				const existing = cm.getAddonDevice(addonId, key);

package/dist/kernel/config-manager.d.ts

@@ -31,6 +31,19 @@ export interface AddonSettingsView {
     writeDeviceRuntimeState(deviceId: number, data: Record<string, unknown>): Promise<void>;
     clearDeviceRuntimeState(deviceId: number): Promise<void>;
 }
+/**
+ * A CONCRETE backend behind {@link ConfigManager.createSettingsView} — the
+ * sync-`ISettingsStore` one or the async-door one.
+ *
+ * `readDeviceStoreBatch` is required here and optional on
+ * {@link AddonSettingsView} because both of these can answer it, so the view
+ * the kernel hands an addon never has to decide whether the method exists.
+ * The optionality on the public interface stays for the implementation that
+ * genuinely cannot batch: a forked runner's UDS-backed view.
+ */
+export interface SettingsViewBackend extends AddonSettingsView {
+    readDeviceStoreBatch(deviceIds: readonly number[]): Promise<ReadonlyMap<number, Record<string, unknown>>>;
+}
 export interface ISettingsStore {
     getSystem(key: string): unknown;
     setSystem(key: string, value: unknown): void;
@@ -181,7 +194,25 @@ export declare class ConfigManager {
     setDeviceRuntimeState(deviceId: string, blob: Record<string, unknown>): void;
     /** Forget the device's runtime state. No-op before the store is wired. */
     clearDeviceRuntimeState(deviceId: string): void;
+    /**
+     * The addon's `ctx.settings`.
+     *
+     * The backend is chosen on EVERY call, never frozen when the view is built.
+     * That is not a nicety: `settings-store` is owned by `storage-orchestrator`,
+     * so the door is wired by `registerProvider` STRICTLY AFTER that addon's
+     * context — and therefore its settings view — already exists. A view bound at
+     * construction handed the door's own owner the sync-store backend, whose
+     * writes throw for the whole life of the process once `sqlite-settings` runs
+     * isolated (D181: no sync `ISettingsStore` reaches hub-main, only the async
+     * door). Reads were worse than the writes: they answered `{}` in silence.
+     * That is what made `storageMigration.start` fail on its first durable write
+     * while every other addon's persistence worked. See D294.
+     */
     createSettingsView(addonId: string): AddonSettingsView;
+    /** The async-door backend of {@link createSettingsView}. */
+    private createDoorBackedSettingsView;
+    /** The sync-`ISettingsStore` backend of {@link createSettingsView}. */
+    private createStoreSettingsView;
     /** Get all system-wide activation overrides. */
     getSystemActivation(): Readonly<Record<string, boolean>>;
     /** Set system-wide activation for a capability/addon pair. */

package/dist/kernel/settings-door-view.d.ts

@@ -1,7 +1,7 @@
 import { ISettingsStoreProvider } from '@camstack/types';
-import { AddonSettingsView } from './config-manager.js';
+import { SettingsViewBackend } from './config-manager.js';
 export interface DoorSectionAccess {
     getSection(section: string): Record<string, unknown>;
     setSection(section: string, patch: Record<string, unknown>): void | Promise<void>;
 }
-export declare function createDoorSettingsView(addonId: string, door: ISettingsStoreProvider, sections: DoorSectionAccess): AddonSettingsView;
+export declare function createDoorSettingsView(addonId: string, door: ISettingsStoreProvider, sections: DoorSectionAccess): SettingsViewBackend;

package/package.json

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