@viostream/viostream-player-svelte 0.2.6 → 0.2.8

package/README.md

@@ -89,6 +89,7 @@ All embed options are optional and passed directly to the Viostream embed API.
 | `startTime` | `string` | Seek to a time (seconds) before playback. |
 | `transcriptDownload` | `boolean` | Allow transcript download. Default: `false`. |
 | `useSettingsMenu` | `boolean` | Enable the settings menu on the control bar. Default: `false`. |
+| `forceAspectRatio` | `number` | Force a specific aspect ratio (e.g. `1.7778` for 16:9). Disables dynamic sizing. Must be a finite positive number; invalid values are ignored. |
 
 #### Event Callbacks
 

package/dist/ViostreamPlayer.svelte

@@ -25,7 +25,8 @@
 -->
 <script lang="ts">
 	import { onMount, type Snippet } from 'svelte';
-	import { getViostreamApi, wrapRawPlayer } from '@viostream/viostream-player-core';
+	import Debug from 'debug';
+	import { getViostreamApi, wrapRawPlayer, normalizeForceAspectRatio } from '@viostream/viostream-player-core';
 	import type {
 		ViostreamEmbedOptions,
 		ViostreamPlayer,
@@ -35,6 +36,8 @@
 	import type { ViostreamPlayerProps } from './types.js';
 	import { SDK_NAME, SDK_VERSION } from './version.js';
 
+	const debug = Debug('viostream:svelte');
+
 	let {
 		// Required props
 		accountKey,
@@ -57,6 +60,7 @@
 		startTime,
 		transcriptDownload,
 		useSettingsMenu,
+		forceAspectRatio,
 
 		// Event callbacks
 		onplay,
@@ -133,42 +137,53 @@
 	];
 
 	onMount(() => {
+		debug('onMount publicKey=%s accountKey=%s containerId=%s', publicKey, accountKey, containerId);
+
 		let destroyed = false;
-		const unsubscribers: Array<() => void> = [];
 
 		async function init() {
 			try {
+				debug('init: getting embed API');
 				const api = getViostreamApi();
 
-				if (destroyed) return;
+				if (destroyed) {
+					debug('init: stale closure detected after getViostreamApi — aborting publicKey=%s', publicKey);
+					return;
+				}
 
 				const embedOpts = buildEmbedOptions();
-				const raw: RawViostreamPlayerInstance = api.embed(publicKey, containerId, embedOpts);
+				debug('init: calling api.embed publicKey=%s containerId=%s options=%o', publicKey, containerId, embedOpts);
+				const raw: RawViostreamPlayerInstance = api.embed(publicKey, containerId, embedOpts, normalizeForceAspectRatio(forceAspectRatio));
+				debug('init: api.embed returned raw player');
+
 				const wrappedPlayer = wrapRawPlayer(raw, containerId);
+				debug('init: wrapRawPlayer completed containerId=%s', containerId);
 
 				if (destroyed) {
+					debug('init: stale closure detected after wrapRawPlayer — destroying and aborting publicKey=%s', publicKey);
 					wrappedPlayer.destroy();
 					return;
 				}
 
 				player = wrappedPlayer;
 				isLoading = false;
+				debug('init: player set, isLoading -> false publicKey=%s', publicKey);
 
-				// Wire up event callbacks from props
-				for (const [eventName, getHandler] of EVENT_MAP) {
-					const handler = getHandler();
-					if (handler) {
-						const unsub = wrappedPlayer.on(eventName, handler);
-						unsubscribers.push(unsub);
-					}
-				}
+				// Event wiring is handled by the $effect block below, which
+				// runs reactively when `player` is set and re-wires when
+				// callback props change.
 
 				// Notify consumer that the player is ready
+				debug('init: firing onplayerready publicKey=%s', publicKey);
 				onplayerready?.(wrappedPlayer);
 			} catch (err) {
 				if (!destroyed) {
-					errorMsg = err instanceof Error ? err.message : String(err);
+					const msg = err instanceof Error ? err.message : String(err);
+					debug('init: error caught publicKey=%s error=%s', publicKey, msg);
+					errorMsg = msg;
 					isLoading = false;
+				} else {
+					debug('init: error caught but destroyed — ignoring publicKey=%s', publicKey);
 				}
 			}
 		}
@@ -176,10 +191,8 @@
 		init();
 
 		return () => {
+			debug('cleanup publicKey=%s hasPlayer=%s', publicKey, !!player);
 			destroyed = true;
-			for (const unsub of unsubscribers) {
-				unsub();
-			}
 			player?.destroy();
 			player = undefined;
 		};
@@ -188,19 +201,27 @@
 	// Re-wire event handlers reactively when callback props change
 	// This handles the case where a consumer conditionally provides callbacks
 	$effect(() => {
-		if (!player) return;
+		if (!player) {
+			debug('$effect event wiring skipped — no player');
+			return;
+		}
 
 		const currentUnsubscribers: Array<() => void> = [];
+		const wiredEvents: string[] = [];
 
 		for (const [eventName, getHandler] of EVENT_MAP) {
 			const handler = getHandler();
 			if (handler) {
 				const unsub = player.on(eventName, handler);
 				currentUnsubscribers.push(unsub);
+				wiredEvents.push(eventName);
 			}
 		}
 
+		debug('$effect event wiring: subscribed to [%s]', wiredEvents.join(', '));
+
 		return () => {
+			debug('$effect event wiring cleanup: unsubscribing %d events', currentUnsubscribers.length);
 			for (const unsub of currentUnsubscribers) {
 				unsub();
 			}

package/dist/version.d.ts

@@ -1,2 +1,2 @@
 export declare const SDK_NAME = "viostream-player-svelte";
-export declare const SDK_VERSION = "0.2.6";
+export declare const SDK_VERSION = "0.2.8";

package/dist/version.js

@@ -1,3 +1,3 @@
 // Auto-generated by scripts/sync-version.mjs — do not edit
 export const SDK_NAME = 'viostream-player-svelte';
-export const SDK_VERSION = '0.2.6';
+export const SDK_VERSION = '0.2.8';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@viostream/viostream-player-svelte",
-  "version": "0.2.6",
+  "version": "0.2.8",
   "description": "Svelte 5 SDK for the Viostream video player — embed, control, and listen to player events",
   "license": "MIT",
   "publishConfig": {
@@ -31,7 +31,8 @@
     "!dist/**/*.spec.*"
   ],
   "dependencies": {
-    "@viostream/viostream-player-core": "^0.2.6"
+    "@viostream/viostream-player-core": "^0.2.8",
+    "debug": "^4.4.3"
   },
   "peerDependencies": {
     "svelte": "^5.0.0"
@@ -42,6 +43,7 @@
     "@sveltejs/vite-plugin-svelte": "^5.0.0",
     "@testing-library/jest-dom": "^6.9.1",
     "@testing-library/svelte": "^5.3.1",
+    "@types/debug": "^4.1.13",
     "jsdom": "^28.1.0",
     "publint": "^0.3.0",
     "svelte": "^5.0.0",
@@ -59,4 +61,4 @@
     "embed",
     "sdk"
   ]
-}
+}