@pie-players/pie-assessment-toolkit 0.2.10 → 0.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@pie-players/pie-assessment-toolkit",
-	"version": "0.2.10",
+	"version": "0.3.0",
 	"type": "module",
 	"description": "PIE assessment toolkit: composable services + reference implementation for assessment players and tool coordination",
 	"license": "MIT",
@@ -56,7 +56,7 @@
 		"README.md"
 	],
 	"scripts": {
-		"build": "rm -rf dist tsconfig.tsbuildinfo && tsc -p tsconfig.json && node ./scripts/build-ce-components.mjs",
+		"build": "rm -rf dist tsconfig.tsbuildinfo && tsc -p tsconfig.json && bun ./scripts/build-ce-components.mjs",
 		"dev": "tsc -p tsconfig.json --watch",
 		"typecheck": "tsc -p tsconfig.json --noEmit",
 		"lint": "biome check .",
@@ -64,12 +64,12 @@
 		"test": "bun test"
 	},
 	"dependencies": {
-		"@pie-players/pie-calculator": "0.1.5",
-		"@pie-players/pie-calculator-desmos": "0.1.6",
-		"@pie-players/pie-context": "0.1.2",
-		"@pie-players/pie-players-shared": "0.2.6",
-		"@pie-players/pie-tts": "0.1.5",
-		"@pie-players/tts-client-server": "0.2.5",
+		"@pie-players/pie-calculator": "0.3.0",
+		"@pie-players/pie-calculator-desmos": "0.3.0",
+		"@pie-players/pie-context": "0.3.0",
+		"@pie-players/pie-players-shared": "0.3.0",
+		"@pie-players/pie-tts": "0.3.0",
+		"@pie-players/tts-client-server": "0.3.0",
 		"daisyui": "^5.5.18"
 	},
 	"peerDependencies": {
@@ -84,7 +84,7 @@
 		"url": "https://github.com/pie-framework/pie-players/issues"
 	},
 	"engines": {
-		"node": ">=18.0.0"
+		"node": ">=20.0.0"
 	},
 	"sideEffects": true
 }

package/src/components/ItemToolBar.svelte

@@ -170,17 +170,30 @@
 		const resolved = resolveToolsForLevel(effectiveToolsConfig, placementLevel);
 		return effectiveToolRegistry.normalizeToolIds(resolved).filter(Boolean);
 	});
+	const providerConfig = $derived(
+		(effectiveToolsConfig?.providers || {}) as Record<
+			string,
+			{ enabled?: boolean } | undefined
+		>
+	);
+	const isProviderEnabled = (toolId: string): boolean =>
+		providerConfig[toolId]?.enabled !== false;
 
 	// Pass 1: determine allowed tools
 	const allowedToolIds = $derived.by(() => {
 		const configuredTools =
 			normalizedExplicitTools.length > 0 ? normalizedExplicitTools : placementAllowedToolIds;
+		const providerEnabledConfiguredTools = configuredTools.filter((toolId) =>
+			isProviderEnabled(toolId)
+		);
 		if (pnpResolver && assessment && itemRef) {
 			const allowedByPnp = pnpResolver.getAllowedToolIds(assessment, itemRef);
-			if (configuredTools.length === 0) return allowedByPnp;
-			return allowedByPnp.filter((toolId) => configuredTools.includes(toolId));
+			if (providerEnabledConfiguredTools.length === 0) return allowedByPnp;
+			return allowedByPnp.filter((toolId) =>
+				providerEnabledConfiguredTools.includes(toolId)
+			);
 		}
-		return configuredTools;
+		return providerEnabledConfiguredTools;
 	});
 
 	const contentReady = $derived.by(() => {
@@ -285,6 +298,9 @@
 
 		return Array.from(visible);
 	});
+	const toolbarVisibleToolIds = $derived.by(() =>
+		effectiveToolRegistry.filterToolIdsByActivation(visibleToolIds, 'toolbar-toggle')
+	);
 
 	// Dynamically load whatever tools are currently visible.
 	// The registry owns module loader configuration by toolId.
@@ -292,7 +308,7 @@
 		if (!isBrowser) return;
 		let cancelled = false;
 		void effectiveToolRegistry
-			.ensureToolModulesLoaded(visibleToolIds)
+			.ensureToolModulesLoaded(toolbarVisibleToolIds)
 			.then(() => {
 				if (!cancelled) moduleLoadVersion += 1;
 			})
@@ -377,7 +393,7 @@
 		moduleLoadVersion;
 
 		const rendered: ToolToolbarRenderResult[] = [];
-		for (const toolId of visibleToolIds) {
+		for (const toolId of toolbarVisibleToolIds) {
 			const result = effectiveToolRegistry.renderForToolbar(toolId, renderContext, toolbarContext);
 			if (result) {
 				rendered.push(result);

package/src/components/PieAssessmentToolkit.svelte

@@ -38,18 +38,26 @@
 	import { connectAssessmentToolkitHostRuntimeContext } from "../context/runtime-context-consumer.js";
 	import { ToolkitCoordinator } from "../services/ToolkitCoordinator.js";
 	import {
-		PIE_ITEM_SESSION_CHANGED_EVENT,
+		PIE_INTERNAL_CONTENT_LOADED_EVENT,
+		PIE_INTERNAL_ITEM_SESSION_CHANGED_EVENT,
+		PIE_INTERNAL_ITEM_PLAYER_ERROR_EVENT,
 		PIE_REGISTER_EVENT,
 		PIE_UNREGISTER_EVENT,
-		type ItemSessionChangedDetail,
+		type InternalContentLoadedDetail,
+		type InternalItemSessionChangedDetail,
+		type InternalItemPlayerErrorDetail,
 		type RuntimeRegistrationDetail,
 	} from "../runtime/registration-events.js";
 	import { dispatchCrossBoundaryEvent } from "../runtime/tool-host-contract.js";
 	import { SectionRuntimeEngine } from "../runtime/SectionRuntimeEngine.js";
 	import {
 		createRuntimeId,
-		shouldHandleBySourceRuntime,
 	} from "../runtime/runtime-event-guards.js";
+	import {
+		createSessionEmitPolicyState,
+		resetSessionEmitPolicyState,
+		shouldEmitCanonicalSessionEvent,
+	} from "../runtime/session-event-emitter-policy.js";
 
 	type SessionChangedLike = {
 		eventDetail?: unknown;
@@ -106,6 +114,7 @@ const DEFAULT_ENV = {
 	let pendingCompositionModel: unknown = null;
 	let pendingCompositionEmit = false;
 	let compositionEmitRaf: number | null = null;
+	const sessionEmitPolicyState = createSessionEmitPolicyState();
 
 	function getHostElement(): HTMLElement | null {
 		if (!anchor) return null;
@@ -357,12 +366,22 @@ const DEFAULT_ENV = {
 	}) {
 		const normalized =
 			(args.result as SessionChangedLike | null)?.eventDetail || args.fallbackSession;
-		emit("session-changed", {
+		const payload = {
 			...(normalized as Record<string, unknown>),
 			itemId: args.itemId,
 			canonicalItemId: args.canonicalItemId || args.itemId,
 			sourceRuntimeId: runtimeId,
-		});
+		} as Record<string, unknown>;
+		if (
+			!shouldEmitCanonicalSessionEvent({
+				state: sessionEmitPolicyState,
+				itemId: args.itemId,
+				payload,
+			})
+		) {
+			return;
+		}
+		emit("session-changed", payload);
 	}
 
 	function isLocalToCurrentRuntime(eventTarget: EventTarget | null): boolean {
@@ -401,6 +420,14 @@ const DEFAULT_ENV = {
 		}
 	});
 
+	$effect(() => {
+		const currentSectionId = effectiveSectionId;
+		const currentAttemptId = attemptId || "";
+		void currentSectionId;
+		void currentAttemptId;
+		resetSessionEmitPolicyState(sessionEmitPolicyState);
+	});
+
 	$effect(() => {
 		if (!host || !hostRuntimeContextValue) return;
 		hostRuntimeProvider = new ContextProvider(host, {
@@ -489,6 +516,11 @@ const DEFAULT_ENV = {
 			})
 			.catch((error) => {
 				runtimeError = error;
+				sectionEngine.reportSectionError({
+					source: "section-runtime",
+					error,
+					timestamp: Date.now(),
+				});
 				emit("runtime-error", {
 					runtimeId,
 					error,
@@ -509,39 +541,78 @@ const DEFAULT_ENV = {
 			const detail = (event as CustomEvent<RuntimeRegistrationDetail>).detail;
 			if (!detail?.element || !detail?.itemId) return;
 			const changed = sectionEngine.register(detail);
+			sectionEngine.handleContentRegistered(detail);
 			if (changed) emitCompositionChanged();
 		};
 
 		const onUnregister = (event: Event) => {
 			if (!isLocalToCurrentRuntime(event.target)) return;
 			const detail = (event as CustomEvent<RuntimeRegistrationDetail>).detail;
+			if (!detail?.itemId) return;
 			const changed = detail?.element
 				? sectionEngine.unregister(detail.element)
 				: false;
+			sectionEngine.handleContentUnregistered(detail);
 			if (changed) emitCompositionChanged();
 		};
 
 		const onItemSessionChanged = (event: Event) => {
 			if (!isLocalToCurrentRuntime(event.target)) return;
-			const detail = (event as CustomEvent<ItemSessionChangedDetail>).detail;
+			const detail = (event as CustomEvent<InternalItemSessionChangedDetail>).detail;
 			if (!detail?.itemId) return;
-			if (!shouldHandleBySourceRuntime(detail.sourceRuntimeId, runtimeId)) return;
 			const result = sectionEngine.handleItemSessionChanged(detail.itemId, detail.session);
 			emitNormalizedSessionChanged({
 				itemId: detail.itemId,
-				canonicalItemId: detail.canonicalItemId,
+				canonicalItemId: sectionEngine.getCanonicalItemId(detail.itemId),
 				result,
 				fallbackSession: detail.session,
 			});
 		};
+		const onContentLoaded = (event: Event) => {
+			if (!isLocalToCurrentRuntime(event.target)) return;
+			const detail = (event as CustomEvent<InternalContentLoadedDetail>).detail;
+			if (!detail?.itemId) return;
+			sectionEngine.handleContentLoaded({
+				itemId: detail.itemId,
+				canonicalItemId: detail.canonicalItemId,
+				contentKind: detail.contentKind,
+				detail: detail.detail,
+				timestamp: Date.now(),
+			});
+		};
+		const onItemPlayerError = (event: Event) => {
+			if (!isLocalToCurrentRuntime(event.target)) return;
+			const detail = (event as CustomEvent<InternalItemPlayerErrorDetail>).detail;
+			if (!detail?.itemId) return;
+			sectionEngine.handleItemPlayerError({
+				itemId: detail.itemId,
+				canonicalItemId: detail.canonicalItemId,
+				contentKind: detail.contentKind,
+				error: detail.error,
+				timestamp: Date.now(),
+			});
+		};
 
 		localHost.addEventListener(PIE_REGISTER_EVENT, onRegister);
 		localHost.addEventListener(PIE_UNREGISTER_EVENT, onUnregister);
-		localHost.addEventListener(PIE_ITEM_SESSION_CHANGED_EVENT, onItemSessionChanged);
+		localHost.addEventListener(PIE_INTERNAL_ITEM_SESSION_CHANGED_EVENT, onItemSessionChanged);
+		localHost.addEventListener(PIE_INTERNAL_CONTENT_LOADED_EVENT, onContentLoaded);
+		localHost.addEventListener(PIE_INTERNAL_ITEM_PLAYER_ERROR_EVENT, onItemPlayerError);
 		return () => {
 			localHost.removeEventListener(PIE_REGISTER_EVENT, onRegister);
 			localHost.removeEventListener(PIE_UNREGISTER_EVENT, onUnregister);
-			localHost.removeEventListener(PIE_ITEM_SESSION_CHANGED_EVENT, onItemSessionChanged);
+			localHost.removeEventListener(
+				PIE_INTERNAL_ITEM_SESSION_CHANGED_EVENT,
+				onItemSessionChanged,
+			);
+			localHost.removeEventListener(
+				PIE_INTERNAL_CONTENT_LOADED_EVENT,
+				onContentLoaded,
+			);
+			localHost.removeEventListener(
+				PIE_INTERNAL_ITEM_PLAYER_ERROR_EVENT,
+				onItemPlayerError,
+			);
 		};
 	});