@camstack/system 1.2.89 → 1.2.91

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

@@ -3,7 +3,7 @@ Object.defineProperties(exports, {
 	[Symbol.toStringTag]: { value: "Module" }
 });
 const require_chunk = require("../../chunk-Cek0wNdY.js");
-const require_dist = require("../../dist-Dq0ebOuS.js");
+const require_dist = require("../../dist-9Gfk4KOk.js");
 let node_crypto = require("node:crypto");
 let node_fs = require("node:fs");
 let node_module = require("node:module");
@@ -1144,10 +1144,21 @@ var SqliteSettingsBackend = class SqliteSettingsBackend {
 			}
 		}
 		this.getDb().exec(`CREATE TABLE IF NOT EXISTS "${table}" (${colDefs.join(", ")})`);
-		if (schema.indexes) for (const idx of schema.indexes) {
-			const unique = idx.unique ? "UNIQUE " : "";
-			const cols = idx.columns.map((c) => `"${c}"`).join(", ");
-			this.getDb().exec(`CREATE ${unique}INDEX IF NOT EXISTS "${idx.name}" ON "${table}" (${cols})`);
+		if (schema.indexes) {
+			const pkName = schema.columns.find((c) => c.primaryKey === true)?.name ?? "id";
+			const nonPkColumns = schema.columns.filter((c) => c.name !== pkName).map((c) => c.name);
+			const kvBlobColumn = nonPkColumns.length === 1 && nonPkColumns[0] === "data" ? "data" : null;
+			const isRealColumn = (field) => field === pkName || nonPkColumns.includes(field);
+			const indexExpr = (field) => {
+				if (isRealColumn(field)) return `"${field}"`;
+				return kvBlobColumn === null ? null : `json_extract("${kvBlobColumn}", '$.${field}')`;
+			};
+			for (const idx of schema.indexes) {
+				const exprs = idx.columns.map((c) => indexExpr(c));
+				if (exprs.some((e) => e === null)) continue;
+				const unique = idx.unique ? "UNIQUE " : "";
+				this.getDb().exec(`CREATE ${unique}INDEX IF NOT EXISTS "${idx.name}" ON "${table}" (${exprs.join(", ")})`);
+			}
 		}
 		this.structuredTables.set(table, fingerprint);
 	}

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

@@ -1,4 +1,4 @@
-import { A as decodeVectorBase64, Dt as asJsonObject, Rt as parseJsonUnknown, _t as BaseAddon, f as RUNTIME_DEFAULTS, gt as errMsg, ht as vectorStoreCapability, k as dataStoreProviderCapability, mt as vectorDimFromBase64, w as bareAddonId } from "../../dist-Bij6agsS.mjs";
+import { A as decodeVectorBase64, Dt as asJsonObject, Rt as parseJsonUnknown, _t as BaseAddon, f as RUNTIME_DEFAULTS, gt as errMsg, ht as vectorStoreCapability, k as dataStoreProviderCapability, mt as vectorDimFromBase64, w as bareAddonId } from "../../dist-DhrTd8LL.mjs";
 import { createRequire } from "node:module";
 import { randomUUID } from "node:crypto";
 import { statSync } from "node:fs";
@@ -1138,10 +1138,21 @@ var SqliteSettingsBackend = class SqliteSettingsBackend {
 			}
 		}
 		this.getDb().exec(`CREATE TABLE IF NOT EXISTS "${table}" (${colDefs.join(", ")})`);
-		if (schema.indexes) for (const idx of schema.indexes) {
-			const unique = idx.unique ? "UNIQUE " : "";
-			const cols = idx.columns.map((c) => `"${c}"`).join(", ");
-			this.getDb().exec(`CREATE ${unique}INDEX IF NOT EXISTS "${idx.name}" ON "${table}" (${cols})`);
+		if (schema.indexes) {
+			const pkName = schema.columns.find((c) => c.primaryKey === true)?.name ?? "id";
+			const nonPkColumns = schema.columns.filter((c) => c.name !== pkName).map((c) => c.name);
+			const kvBlobColumn = nonPkColumns.length === 1 && nonPkColumns[0] === "data" ? "data" : null;
+			const isRealColumn = (field) => field === pkName || nonPkColumns.includes(field);
+			const indexExpr = (field) => {
+				if (isRealColumn(field)) return `"${field}"`;
+				return kvBlobColumn === null ? null : `json_extract("${kvBlobColumn}", '$.${field}')`;
+			};
+			for (const idx of schema.indexes) {
+				const exprs = idx.columns.map((c) => indexExpr(c));
+				if (exprs.some((e) => e === null)) continue;
+				const unique = idx.unique ? "UNIQUE " : "";
+				this.getDb().exec(`CREATE ${unique}INDEX IF NOT EXISTS "${idx.name}" ON "${table}" (${exprs.join(", ")})`);
+			}
 		}
 		this.structuredTables.set(table, fingerprint);
 	}

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

@@ -1,2 +1,2 @@
-import { StorageOrchestratorAddon, n as StorageMigrationCoordinator, r as SqliteLocationStore, t as StorageOrchestratorService } from "./storage-orchestrator.addon.mjs";
+import { StorageOrchestratorAddon, i as SqliteLocationStore, r as StorageMigrationCoordinator, t as StorageOrchestratorService } from "./storage-orchestrator.addon.mjs";
 export { SqliteLocationStore, StorageMigrationCoordinator, StorageOrchestratorAddon, StorageOrchestratorAddon as default, StorageOrchestratorService };

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

@@ -3,14 +3,18 @@ Object.defineProperties(exports, {
 	[Symbol.toStringTag]: { value: "Module" }
 });
 const require_chunk = require("../../chunk-Cek0wNdY.js");
-const require_dist = require("../../dist-Dq0ebOuS.js");
+const require_dist = require("../../dist-9Gfk4KOk.js");
 let node_crypto = require("node:crypto");
 let node_fs_promises = require("node:fs/promises");
 node_fs_promises = require_chunk.__toESM(node_fs_promises);
 let node_path = require("node:path");
 node_path = require_chunk.__toESM(node_path);
-let _camstack_system = require("@camstack/system");
 //#region src/builtins/storage-orchestrator/location-store.ts
+/**
+* @durable class=config owner=storage-orchestrator
+*   write="an operator creates or edits a location (basePath, minFreePercent, enabled, node pin); plus the boot reconcile — declared defaults seeded on a fresh install, and in-memory-only locations backfilled when the store is late-wired"
+*   retention="none — a row goes only when the operator deletes the location, or when the declaration pass prunes one whose declaring addon is gone. Bounded by the storage targets a human configures."
+*/
 var STORAGE_LOCATIONS = "storage_locations";
 /**
 * Reserved key under which the cluster `nodeId` (SP1) is persisted INSIDE
@@ -577,6 +581,50 @@ function isTerminal(job) {
 	return job.phase === "done" || job.phase === "failed" || job.phase === "cancelled";
 }
 //#endregion
+//#region src/kernel/storage-location-registry.ts
+/**
+* Aggregates per-addon {@link StorageLocationDeclaration} arrays into a single
+* {@link StorageLocationRegistry}.
+*
+* Merge rules:
+* - Declarations are keyed by `id`; iteration order follows first occurrence.
+* - **First declarer wins** for metadata (`displayName`, `description`,
+*   `defaultsTo`): if a later addon declares the same `id`, its metadata is
+*   silently ignored.
+* - **Cardinality conflict throws**: if two addons declare the same `id` with
+*   different `cardinality` values an `Error` is thrown immediately (message
+*   contains the words "cardinality" and the conflicting `id`).
+*
+* @param perAddon - One entry per addon; each entry is that addon's
+*   `storageLocations` declaration array.
+*/
+function buildStorageLocationRegistry(perAddon) {
+	const map = /* @__PURE__ */ new Map();
+	for (const addonDeclarations of perAddon) for (const declaration of addonDeclarations) {
+		const existing = map.get(declaration.id);
+		if (existing !== void 0) {
+			if (existing.cardinality !== declaration.cardinality) throw new Error(`Storage location cardinality conflict for id "${declaration.id}": "${existing.cardinality}" (first declarer) vs "${declaration.cardinality}" (later declarer). All addons declaring the same storage location id must agree on cardinality.`);
+			continue;
+		}
+		map.set(declaration.id, declaration);
+	}
+	const frozen = Object.freeze([...map.values()]);
+	return {
+		get(id) {
+			return map.get(id);
+		},
+		has(id) {
+			return map.has(id);
+		},
+		cardinalityOf(id) {
+			return map.get(id)?.cardinality ?? null;
+		},
+		list() {
+			return frozen;
+		}
+	};
+}
+//#endregion
 //#region src/builtins/storage-orchestrator/data-store-dispatch.ts
 /**
 * Pick the engine to serve a call.
@@ -756,11 +804,23 @@ var StorageOrchestratorService = class {
 	*   2. backfill — in-memory locations the DB doesn't have yet (the
 	*      pre-store seed defaults on a fresh install) are persisted, so the
 	*      store becomes the durable source of truth from here on.
+	*
+	* **The re-entry guard latches on a SUCCESSFUL hydrate, not on the attempt** —
+	* `loadAll()` is awaited BEFORE `this.locationStore` is assigned, and the
+	* order is load-bearing. Latching first meant one transient read error
+	* answered `hasStore()` true forever, so the addon's `ensureStoreWired` gate
+	* skipped every retry; the service then ran the whole process lifetime on
+	* pre-store seed defaults while `upsert` mirrored them onto rows it had never
+	* read, silently replacing the operator's persisted `minFreePercent`. Failing
+	* back to store-less is the safe direction: an in-memory-only service writes
+	* nothing over what is on disk, and the next `capability:provider-registered`
+	* event tries again. Held by
+	* `__tests__/storage-orchestrator-persistence.spec.ts`.
 	*/
 	async attachStore(store) {
 		if (this.locationStore) return;
-		this.locationStore = store;
 		const rows = await store.loadAll();
+		this.locationStore = store;
 		const dbIds = new Set(rows.map((r) => r.id));
 		for (const loc of rows) {
 			const upgraded = !loc.isSystem && loc.id === `${loc.type}:default` ? {
@@ -1925,7 +1985,7 @@ var StorageOrchestratorAddon = class extends require_dist.BaseAddon {
 	async seedFromDeclarations() {
 		if (!this.service) return;
 		await this.refreshNodeLocalCache();
-		const registry = (0, _camstack_system.buildStorageLocationRegistry)(this.ctx.kernel.listStorageLocationDeclarations?.() ?? []);
+		const registry = buildStorageLocationRegistry(this.ctx.kernel.listStorageLocationDeclarations?.() ?? []);
 		this.service.setRegistry(registry);
 		this.service.pruneUndeclaredSystemLocations();
 		const mediaRoot = process.env.CAMSTACK_MEDIA_ROOT?.trim();
@@ -1963,3 +2023,4 @@ exports.StorageMigrationCoordinator = StorageMigrationCoordinator;
 exports.StorageOrchestratorAddon = StorageOrchestratorAddon;
 exports.default = StorageOrchestratorAddon;
 exports.StorageOrchestratorService = StorageOrchestratorService;
+exports.buildStorageLocationRegistry = buildStorageLocationRegistry;

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

@@ -1,9 +1,13 @@
-import { Lt as parseJsonObject, _t as BaseAddon, g as StorageMigrationJobSchema, h as StorageLocationTypeSchema, lt as storageCapability, st as settingsStoreCapability, ut as storageMigrationCapability } from "../../dist-Bij6agsS.mjs";
+import { Lt as parseJsonObject, _t as BaseAddon, g as StorageMigrationJobSchema, h as StorageLocationTypeSchema, lt as storageCapability, st as settingsStoreCapability, ut as storageMigrationCapability } from "../../dist-DhrTd8LL.mjs";
 import { randomUUID } from "node:crypto";
 import * as fs from "node:fs/promises";
 import * as path$1 from "node:path";
-import { buildStorageLocationRegistry } from "@camstack/system";
 //#region src/builtins/storage-orchestrator/location-store.ts
+/**
+* @durable class=config owner=storage-orchestrator
+*   write="an operator creates or edits a location (basePath, minFreePercent, enabled, node pin); plus the boot reconcile — declared defaults seeded on a fresh install, and in-memory-only locations backfilled when the store is late-wired"
+*   retention="none — a row goes only when the operator deletes the location, or when the declaration pass prunes one whose declaring addon is gone. Bounded by the storage targets a human configures."
+*/
 var STORAGE_LOCATIONS = "storage_locations";
 /**
 * Reserved key under which the cluster `nodeId` (SP1) is persisted INSIDE
@@ -570,6 +574,50 @@ function isTerminal(job) {
 	return job.phase === "done" || job.phase === "failed" || job.phase === "cancelled";
 }
 //#endregion
+//#region src/kernel/storage-location-registry.ts
+/**
+* Aggregates per-addon {@link StorageLocationDeclaration} arrays into a single
+* {@link StorageLocationRegistry}.
+*
+* Merge rules:
+* - Declarations are keyed by `id`; iteration order follows first occurrence.
+* - **First declarer wins** for metadata (`displayName`, `description`,
+*   `defaultsTo`): if a later addon declares the same `id`, its metadata is
+*   silently ignored.
+* - **Cardinality conflict throws**: if two addons declare the same `id` with
+*   different `cardinality` values an `Error` is thrown immediately (message
+*   contains the words "cardinality" and the conflicting `id`).
+*
+* @param perAddon - One entry per addon; each entry is that addon's
+*   `storageLocations` declaration array.
+*/
+function buildStorageLocationRegistry(perAddon) {
+	const map = /* @__PURE__ */ new Map();
+	for (const addonDeclarations of perAddon) for (const declaration of addonDeclarations) {
+		const existing = map.get(declaration.id);
+		if (existing !== void 0) {
+			if (existing.cardinality !== declaration.cardinality) throw new Error(`Storage location cardinality conflict for id "${declaration.id}": "${existing.cardinality}" (first declarer) vs "${declaration.cardinality}" (later declarer). All addons declaring the same storage location id must agree on cardinality.`);
+			continue;
+		}
+		map.set(declaration.id, declaration);
+	}
+	const frozen = Object.freeze([...map.values()]);
+	return {
+		get(id) {
+			return map.get(id);
+		},
+		has(id) {
+			return map.has(id);
+		},
+		cardinalityOf(id) {
+			return map.get(id)?.cardinality ?? null;
+		},
+		list() {
+			return frozen;
+		}
+	};
+}
+//#endregion
 //#region src/builtins/storage-orchestrator/data-store-dispatch.ts
 /**
 * Pick the engine to serve a call.
@@ -749,11 +797,23 @@ var StorageOrchestratorService = class {
 	*   2. backfill — in-memory locations the DB doesn't have yet (the
 	*      pre-store seed defaults on a fresh install) are persisted, so the
 	*      store becomes the durable source of truth from here on.
+	*
+	* **The re-entry guard latches on a SUCCESSFUL hydrate, not on the attempt** —
+	* `loadAll()` is awaited BEFORE `this.locationStore` is assigned, and the
+	* order is load-bearing. Latching first meant one transient read error
+	* answered `hasStore()` true forever, so the addon's `ensureStoreWired` gate
+	* skipped every retry; the service then ran the whole process lifetime on
+	* pre-store seed defaults while `upsert` mirrored them onto rows it had never
+	* read, silently replacing the operator's persisted `minFreePercent`. Failing
+	* back to store-less is the safe direction: an in-memory-only service writes
+	* nothing over what is on disk, and the next `capability:provider-registered`
+	* event tries again. Held by
+	* `__tests__/storage-orchestrator-persistence.spec.ts`.
 	*/
 	async attachStore(store) {
 		if (this.locationStore) return;
-		this.locationStore = store;
 		const rows = await store.loadAll();
+		this.locationStore = store;
 		const dbIds = new Set(rows.map((r) => r.id));
 		for (const loc of rows) {
 			const upgraded = !loc.isSystem && loc.id === `${loc.type}:default` ? {
@@ -1951,4 +2011,4 @@ var StorageOrchestratorAddon = class extends BaseAddon {
 	}
 };
 //#endregion
-export { StorageOrchestratorAddon, StorageOrchestratorAddon as default, StorageMigrationCoordinator as n, SqliteLocationStore as r, StorageOrchestratorService as t };
+export { StorageOrchestratorAddon, StorageOrchestratorAddon as default, SqliteLocationStore as i, buildStorageLocationRegistry as n, StorageMigrationCoordinator as r, StorageOrchestratorService as t };

package/dist/builtins/storage-orchestrator/storage-orchestrator.service.d.ts

@@ -139,6 +139,18 @@ export declare class StorageOrchestratorService {
      *   2. backfill — in-memory locations the DB doesn't have yet (the
      *      pre-store seed defaults on a fresh install) are persisted, so the
      *      store becomes the durable source of truth from here on.
+     *
+     * **The re-entry guard latches on a SUCCESSFUL hydrate, not on the attempt** —
+     * `loadAll()` is awaited BEFORE `this.locationStore` is assigned, and the
+     * order is load-bearing. Latching first meant one transient read error
+     * answered `hasStore()` true forever, so the addon's `ensureStoreWired` gate
+     * skipped every retry; the service then ran the whole process lifetime on
+     * pre-store seed defaults while `upsert` mirrored them onto rows it had never
+     * read, silently replacing the operator's persisted `minFreePercent`. Failing
+     * back to store-less is the safe direction: an in-memory-only service writes
+     * nothing over what is on disk, and the next `capability:provider-registered`
+     * event tries again. Held by
+     * `__tests__/storage-orchestrator-persistence.spec.ts`.
      */
     attachStore(store: ILocationStore): Promise<void>;
     /**

package/dist/builtins/system-config/system-config.addon.js

@@ -3,7 +3,7 @@ Object.defineProperties(exports, {
 	[Symbol.toStringTag]: { value: "Module" }
 });
 require("../../chunk-Cek0wNdY.js");
-const require_dist = require("../../dist-Dq0ebOuS.js");
+const require_dist = require("../../dist-9Gfk4KOk.js");
 //#region src/builtins/system-config/system-config.addon.ts
 /**
 * Built-in `system-config` addon — Phase 4 of the settings redesign.

package/dist/builtins/system-config/system-config.addon.mjs

@@ -1,4 +1,4 @@
-import { Pt as hydrateSchema, _t as BaseAddon, gt as errMsg } from "../../dist-Bij6agsS.mjs";
+import { Pt as hydrateSchema, _t as BaseAddon, gt as errMsg } from "../../dist-DhrTd8LL.mjs";
 //#region src/builtins/system-config/system-config.addon.ts
 /**
 * Built-in `system-config` addon — Phase 4 of the settings redesign.

package/dist/builtins/winston-logging/index.js

@@ -3,7 +3,7 @@ Object.defineProperties(exports, {
 	[Symbol.toStringTag]: { value: "Module" }
 });
 const require_chunk = require("../../chunk-Cek0wNdY.js");
-const require_dist = require("../../dist-Dq0ebOuS.js");
+const require_dist = require("../../dist-9Gfk4KOk.js");
 const require_formatter = require("../../formatter-DqAKDlvN.js");
 let node_path = require("node:path");
 node_path = require_chunk.__toESM(node_path);

package/dist/builtins/winston-logging/index.mjs

@@ -1,4 +1,4 @@
-import { J as logDestinationCapability, _t as BaseAddon } from "../../dist-Bij6agsS.mjs";
+import { J as logDestinationCapability, _t as BaseAddon } from "../../dist-DhrTd8LL.mjs";
 import { t as formatLogLine } from "../../formatter-B7qW8bPJ.mjs";
 import * as path$1 from "node:path";
 import path from "node:path";