@mulmoclaude/core 2.1.0 → 3.0.0

package/dist/{discovery-B_sHig7n.cjs → discovery-CtiZMlsX.cjs}

@@ -45,6 +45,22 @@ function createForwardingLogger(getLogger) {
 }
 //#endregion
 //#region src/collection/server/host.ts
+/** `err.code` on the throw from `getWorkspaceRoot()` under an explicit-root
+*  binding: this CALL is missing its `workspaceRoot` option. Exported because
+*  the fix differs from a watcher root conflict — that one means another root's
+*  watcher is already running — and a host catching both in one place should
+*  not have to match on message text to tell them apart. */
+var COLLECTION_ROOT_REQUIRED = "COLLECTION_ROOT_REQUIRED";
+/** Build a change payload, attaching `root` only when the engine call carried
+*  an explicit one. Centralised so every publish site states the root the same
+*  way, and so a single-workspace host's payload shape stays byte-identical to
+*  what it saw before multi-root support. */
+function collectionChangePayload(base, root) {
+	return root === void 0 ? base : {
+		...base,
+		root
+	};
+}
 var hostSlot = createHostSlot("@mulmoclaude/core/collection/server: configureCollectionHost()");
 var changePublisher = null;
 /** Wire the engine to a host. Call once at server startup, before any
@@ -73,9 +89,23 @@ function publishCollectionChange(payload) {
 function requireHost() {
 	return hostSlot.get();
 }
-/** The configured workspace root. Throws if the host never configured one. */
+/** The configured workspace root. Throws if the host never configured one —
+*  and, under an explicit-root binding (`workspaceRoot: null`), throws rather
+*  than guessing, which is the whole point of that mode: on a multi-root host
+*  a missing `opts.workspaceRoot` must fail loudly instead of silently
+*  resolving against some other project. */
 function getWorkspaceRoot() {
-	return requireHost().workspaceRoot;
+	const root = requireHost().workspaceRoot;
+	if (root === null) throw Object.assign(/* @__PURE__ */ new Error("@mulmoclaude/core/collection/server: the host is bound in explicit-root mode (workspaceRoot: null), so there is no ambient workspace root — pass an explicit `workspaceRoot` in this call's options."), { code: COLLECTION_ROOT_REQUIRED });
+	return root;
+}
+/** The configured workspace root, or `null` under an explicit-root binding /
+*  before the host configures one. Never throws — for callers that need to
+*  COMPARE roots (is this the one we are already running for?) rather than
+*  operate on one. Anything that will touch the filesystem wants
+*  `getWorkspaceRoot()` and its loud failure instead. */
+function peekWorkspaceRoot() {
+	return hostSlot.peek()?.workspaceRoot ?? null;
 }
 function userSkillsDir() {
 	return requireHost().paths.userSkillsDir;
@@ -92,10 +122,11 @@ function skillsStagingDir(workspaceRoot) {
 function archiveDir() {
 	return requireHost().paths.archiveDir;
 }
-/** Absolute path to the configured workspace's `collections-registries.json`. */
-function collectionsRegistriesConfigPath() {
-	const host = requireHost();
-	return host.paths.collectionsRegistriesConfig(host.workspaceRoot);
+/** Absolute path to a workspace's `collections-registries.json`. Takes the
+*  root explicitly — reading the ambient one here would throw under an
+*  explicit-root binding and take the Discover tab down with it. */
+function collectionsRegistriesConfigPath(workspaceRoot) {
+	return requireHost().paths.collectionsRegistriesConfig(workspaceRoot);
 }
 function isPresetSlug(slug) {
 	return requireHost().isPresetSlug(slug);
@@ -174,11 +205,6 @@ function isContainedInRoot(absPath, rootPath) {
 	if (ancestorReal === rootReal) return true;
 	return ancestorReal.startsWith(rootReal + node_path.default.sep);
 }
-/** Workspace-bound convenience over `isContainedInRoot`. Production
-*  callers use this; the tests exercise the pure helper. */
-function isContainedInWorkspace(absPath) {
-	return isContainedInRoot(absPath, getWorkspaceRoot());
-}
 /** Resolve a schema-declared dataPath against `rootPath` (default:
 *  the live workspace), refusing anything that escapes — absolute
 *  paths, `..`-segments, empty string, or symlinks pointing outside
@@ -386,11 +412,11 @@ async function writeItem(dataDir, itemId, item, opts = {}) {
 			await handle.close();
 		}
 	} else await require_atomic.writeFileAtomic(filePath, payload);
-	if (opts.slug) publishCollectionChange({
+	if (opts.slug) publishCollectionChange(collectionChangePayload({
 		slug: opts.slug,
 		ids: [safeId],
 		op: "upsert"
-	});
+	}, opts.workspaceRoot));
 	return {
 		kind: "ok",
 		itemId: safeId,
@@ -410,11 +436,11 @@ async function deleteItem(dataDir, itemId, opts = {}) {
 	const filePath = itemFilePath(dataDir, safeId);
 	try {
 		await (0, node_fs_promises.unlink)(filePath);
-		if (opts.slug) publishCollectionChange({
+		if (opts.slug) publishCollectionChange(collectionChangePayload({
 			slug: opts.slug,
 			ids: [safeId],
 			op: "delete"
-		});
+		}, opts.workspaceRoot));
 		return {
 			kind: "ok",
 			itemId: safeId
@@ -1240,11 +1266,11 @@ async function sqliteWrite(absPath, itemId, item, opts) {
 			item
 		};
 	});
-	if (outcome.kind === "ok" && opts.slug) publishCollectionChange({
+	if (outcome.kind === "ok" && opts.slug) publishCollectionChange(collectionChangePayload({
 		slug: opts.slug,
 		ids: [safeId],
 		op: "upsert"
-	});
+	}, opts.publishRoot));
 	return outcome;
 }
 async function sqliteDelete(absPath, itemId, opts) {
@@ -1269,11 +1295,11 @@ async function sqliteDelete(absPath, itemId, opts) {
 			itemId: safeId
 		};
 	});
-	if (outcome.kind === "ok" && opts.slug) publishCollectionChange({
+	if (outcome.kind === "ok" && opts.slug) publishCollectionChange(collectionChangePayload({
 		slug: opts.slug,
 		ids: [safeId],
 		op: "delete"
-	});
+	}, opts.publishRoot));
 	return outcome;
 }
 /** Best-effort full WAL checkpoint so the MAIN db file alone is a
@@ -1304,6 +1330,7 @@ function sqliteStoreFor(collection, opts) {
 	const key = collection.schema.primaryKey;
 	const slug = opts.slug ?? collection.slug;
 	const root = () => opts.workspaceRoot ?? getWorkspaceRoot();
+	const publishRoot = opts.workspaceRoot;
 	if (file === void 0) return {
 		capabilities: {
 			writable: false,
@@ -1329,11 +1356,13 @@ function sqliteStoreFor(collection, opts) {
 		read: (itemId) => sqliteRead(file, itemId, root()),
 		write: (itemId, item, writeOpts = {}) => sqliteWrite(file, itemId, item, {
 			workspaceRoot: root(),
+			publishRoot,
 			slug,
 			refuseOverwrite: writeOpts.refuseOverwrite
 		}),
 		delete: (itemId) => sqliteDelete(file, itemId, {
 			workspaceRoot: root(),
+			publishRoot,
 			slug
 		}),
 		watch: async (onChange) => closerFor(await watchSingleFile(file, (base, name) => name.startsWith(base), () => onChange({ kind: "collection" })))
@@ -2730,6 +2759,12 @@ Object.defineProperty(exports, "BackendUnavailableError", {
 		return BackendUnavailableError;
 	}
 });
+Object.defineProperty(exports, "COLLECTION_ROOT_REQUIRED", {
+	enumerable: true,
+	get: function() {
+		return COLLECTION_ROOT_REQUIRED;
+	}
+});
 Object.defineProperty(exports, "CollectionQueryZ", {
 	enumerable: true,
 	get: function() {
@@ -2790,6 +2825,12 @@ Object.defineProperty(exports, "checkpointSqliteDatabase", {
 		return checkpointSqliteDatabase;
 	}
 });
+Object.defineProperty(exports, "collectionChangePayload", {
+	enumerable: true,
+	get: function() {
+		return collectionChangePayload;
+	}
+});
 Object.defineProperty(exports, "collectionWritable", {
 	enumerable: true,
 	get: function() {
@@ -2892,12 +2933,6 @@ Object.defineProperty(exports, "isContainedInRoot", {
 		return isContainedInRoot;
 	}
 });
-Object.defineProperty(exports, "isContainedInWorkspace", {
-	enumerable: true,
-	get: function() {
-		return isContainedInWorkspace;
-	}
-});
 Object.defineProperty(exports, "isPresetSlug", {
 	enumerable: true,
 	get: function() {
@@ -2946,6 +2981,12 @@ Object.defineProperty(exports, "pageFromFullRead", {
 		return pageFromFullRead;
 	}
 });
+Object.defineProperty(exports, "peekWorkspaceRoot", {
+	enumerable: true,
+	get: function() {
+		return peekWorkspaceRoot;
+	}
+});
 Object.defineProperty(exports, "projectItemFields", {
 	enumerable: true,
 	get: function() {
@@ -3055,4 +3096,4 @@ Object.defineProperty(exports, "writeItem", {
 	}
 });
 
-//# sourceMappingURL=discovery-B_sHig7n.cjs.map
+//# sourceMappingURL=discovery-CtiZMlsX.cjs.map