@mulmoclaude/mulmoscript-plugin 4.4.0 → 4.5.1

package/dist/vue.js

@@ -1,5 +1,5 @@
-import { a as pluginCore, d as errorMessage, f as TOOL_DEFINITION, p as TOOL_NAME } from "./plugin-CAhxJ5WM.js";
-import { n as SCRIPT_CHANGED_EVENT, r as shouldReloadForScriptChange, t as GENERATION_EVENT } from "./contract-CIt1ZAtt.js";
+import { a as pluginCore, h as TOOL_NAME, m as TOOL_DEFINITION, p as errorMessage } from "./plugin-Jo8iGv4K.js";
+import { a as shouldReloadForScriptChange, i as sameRoot, n as SCRIPT_CHANGED_EVENT, t as GENERATION_EVENT } from "./contract-BnIl6w_f.js";
 import { mulmoBeatSchema, mulmoScriptSchema } from "@mulmocast/types";
 import { Fragment, computed, createBlock, createCommentVNode, createElementBlock, createElementVNode, createVNode, defineAsyncComponent, defineComponent, inject, normalizeClass, normalizeStyle, onBeforeUnmount, onMounted, openBlock, reactive, ref, renderList, toDisplayString, unref, vModelText, vShow, watch, withDirectives, withModifiers } from "vue";
 import { beatsOf, withBeats } from "@mulmocast/beat-editor";
@@ -215,6 +215,57 @@ function focusLeftContainer(container, movedTo) {
 	return !container.contains(movedTo);
 }
 //#endregion
+//#region src/vue/subscription.ts
+/** A caller that predates roots is asking for the host's default root. */
+var DEFAULT_ROOT_GETTER = () => void 0;
+/**
+* The object form is checked at runtime, not only by the type.
+*
+* This package is published, so a JavaScript consumer reaches these functions
+* with no call-site checking at all. An object missing `root` type-checks
+* nowhere and passed through here unchanged, and the failure surfaced much
+* later as `sub.root is not a function` — thrown inside a pubsub delivery,
+* where there is no caller left to catch it and the only symptom is a View
+* that silently stops updating (CodeRabbit on #3015).
+*
+* A missing `root` is the one field with a safe answer: absent means the
+* default root, which is exactly what a caller who did not think about roots
+* intended. A missing `filePath` or `handler` has no such answer, so it
+* throws HERE, at the subscribe call, where the stack points at the bug.
+*/
+function checkedSubscription(subscription, label) {
+	if (typeof subscription.filePath !== "function" || typeof subscription.handler !== "function") throw new TypeError(`${label}: filePath and handler must both be functions`);
+	if (subscription.root === void 0) return {
+		...subscription,
+		root: DEFAULT_ROOT_GETTER
+	};
+	if (typeof subscription.root !== "function") throw new TypeError(`${label}: root must be a function returning the root id, or be omitted`);
+	return subscription;
+}
+function normalizeGenerationSubscription(first, legacyHandler) {
+	if (typeof first !== "function") return checkedSubscription(first, "onGenerationEvent(subscription)");
+	if (!legacyHandler) throw new TypeError("onGenerationEvent(filePath, handler): handler is required");
+	return {
+		filePath: first,
+		root: DEFAULT_ROOT_GETTER,
+		handler: legacyHandler
+	};
+}
+function normalizeScriptChangedSubscription(first, legacyOwnOrigin, legacyHandler) {
+	if (typeof first !== "function") {
+		const checked = checkedSubscription(first, "onScriptChanged(subscription)");
+		if (typeof checked.ownOrigin !== "string") throw new TypeError("onScriptChanged(subscription): ownOrigin must be a string");
+		return checked;
+	}
+	if (typeof legacyOwnOrigin !== "string" || !legacyHandler) throw new TypeError("onScriptChanged(filePath, ownOrigin, handler): ownOrigin and handler are required");
+	return {
+		filePath: first,
+		root: DEFAULT_ROOT_GETTER,
+		ownOrigin: legacyOwnOrigin,
+		handler: legacyHandler
+	};
+}
+//#endregion
 //#region src/vue/transport.ts
 var GENERATION_EVENT_KINDS = /* @__PURE__ */ new Set([
 	"beatImage",
@@ -223,26 +274,53 @@ var GENERATION_EVENT_KINDS = /* @__PURE__ */ new Set([
 	"movie",
 	"pdf"
 ]);
+/**
+* A field that is absent, or a string.
+*
+* `undefined` is a legitimate answer for both `root` and `origin`, so a
+* present-but-wrong-typed value CANNOT be flattened into it: `root: 42` read
+* as "no root" makes a named root's event look like a default-root one, and
+* the subscriber then routes another repository's script to this View
+* (CodeRabbit on #3015 — the same shape the dispatch boundary was just fixed
+* for). Malformed means malformed; the caller rejects the payload.
+*/
+function optionalString(value) {
+	if (value === void 0) return {
+		ok: true,
+		value: void 0
+	};
+	return typeof value === "string" ? {
+		ok: true,
+		value
+	} : { ok: false };
+}
 function parseScriptChangedEvent(payload) {
 	if (!isRecord(payload)) return null;
-	const { filePath, origin } = payload;
+	const { filePath, origin, root } = payload;
 	if (typeof filePath !== "string") return null;
+	const parsedOrigin = optionalString(origin);
+	const parsedRoot = optionalString(root);
+	if (!parsedOrigin.ok || !parsedRoot.ok) return null;
 	return {
 		filePath,
-		...typeof origin === "string" ? { origin } : {}
+		...parsedOrigin.value !== void 0 ? { origin: parsedOrigin.value } : {},
+		...parsedRoot.value !== void 0 ? { root: parsedRoot.value } : {}
 	};
 }
 function parseGenerationEvent(payload) {
 	if (!isRecord(payload)) return null;
-	const { kind, filePath, key, done, error } = payload;
+	const { kind, filePath, key, done, error, root } = payload;
 	if (typeof kind !== "string" || !GENERATION_EVENT_KINDS.has(kind)) return null;
 	if (typeof filePath !== "string" || typeof key !== "string" || typeof done !== "boolean") return null;
+	const parsedRoot = optionalString(root);
+	if (!parsedRoot.ok) return null;
 	return {
 		kind,
 		filePath,
 		key,
 		done,
-		...typeof error === "string" ? { error } : {}
+		...typeof error === "string" ? { error } : {},
+		...parsedRoot.value !== void 0 ? { root: parsedRoot.value } : {}
 	};
 }
 function useMulmoScriptTransport() {
@@ -269,12 +347,13 @@ function useMulmoScriptTransport() {
 			data: result
 		};
 	}
-	function onGenerationEvent(filePath, handler) {
+	function onGenerationEvent(first, legacyHandler) {
+		const { filePath, root, handler } = normalizeGenerationSubscription(first, legacyHandler);
 		return runtime.pubsub.subscribe(GENERATION_EVENT, (payload) => {
 			const event = parseGenerationEvent(payload);
 			if (!event) return;
 			const current = filePath();
-			if (!current || event.filePath !== current) return;
+			if (!current || event.filePath !== current || !sameRoot(event.root, root())) return;
 			handler(event);
 		});
 	}
@@ -285,11 +364,12 @@ function useMulmoScriptTransport() {
 	* on them would rebuild the element the caret is in on every keystroke, so they are dropped
 	* here. A write from the agent carries no origin and always reaches the handler.
 	*/
-	function onScriptChanged(filePath, ownOrigin, handler) {
+	function onScriptChanged(first, legacyOwnOrigin, legacyHandler) {
+		const { filePath, root, ownOrigin, handler } = normalizeScriptChangedSubscription(first, legacyOwnOrigin, legacyHandler);
 		return runtime.pubsub.subscribe(SCRIPT_CHANGED_EVENT, (payload) => {
 			const event = parseScriptChangedEvent(payload);
 			if (!event) return;
-			if (!shouldReloadForScriptChange(event, filePath(), ownOrigin)) return;
+			if (!shouldReloadForScriptChange(event, filePath(), ownOrigin, root())) return;
 			handler();
 		});
 	}
@@ -646,9 +726,14 @@ function useDeckEditor({ api, filePath, effectiveScript, commitScript }) {
 	* flushing cannot clobber it out of order.
 	*/
 	function watchForeignWrites(reload) {
-		return api.onScriptChanged(() => filePath.value, EDITOR_ORIGIN, () => {
-			flushPendingDeckSave();
-			reload();
+		return api.onScriptChanged({
+			filePath: () => filePath.value,
+			root: () => void 0,
+			ownOrigin: EDITOR_ORIGIN,
+			handler: () => {
+				flushPendingDeckSave();
+				reload();
+			}
 		});
 	}
 	return {
@@ -2010,14 +2095,18 @@ var View_vue_vue_type_script_setup_true_lang_default = /*@__PURE__*/ defineCompo
 		}
 		onMounted(initializeScript);
 		watch(() => props.selectedResult, initializeScript);
-		const unsubscribeGenerationEvents = api.onGenerationEvent(() => filePath.value, (event) => {
-			if (!event.done) {
-				reflectGenerationStart(event);
-				return;
+		const unsubscribeGenerationEvents = api.onGenerationEvent({
+			filePath: () => filePath.value,
+			root: () => void 0,
+			handler: (event) => {
+				if (!event.done) {
+					reflectGenerationStart(event);
+					return;
+				}
+				reflectGenerationFinish(event).catch((err) => {
+					console.error("[presentMulmoScript] reload on finish failed:", err);
+				});
 			}
-			reflectGenerationFinish(event).catch((err) => {
-				console.error("[presentMulmoScript] reload on finish failed:", err);
-			});
 		});
 		function reflectGenerationStart(entry) {
 			if (entry.kind === "beatImage") {
@@ -2353,7 +2442,7 @@ var _plugin_vue_export_helper_default = (sfc, props) => {
 };
 //#endregion
 //#region src/vue/View.vue
-var View_default = /*#__PURE__*/ _plugin_vue_export_helper_default(View_vue_vue_type_script_setup_true_lang_default, [["__scopeId", "data-v-f9957eaf"]]);
+var View_default = /*#__PURE__*/ _plugin_vue_export_helper_default(View_vue_vue_type_script_setup_true_lang_default, [["__scopeId", "data-v-2d5f76ac"]]);
 //#endregion
 //#region src/vue/Preview.vue?vue&type=script&setup=true&lang.ts
 var _hoisted_1 = {