avbridge 2.2.0 → 2.2.1

package/dist/element.cjs

@@ -1,6 +1,6 @@
 'use strict';
 
-var chunkOE66B34H_cjs = require('./chunk-OE66B34H.cjs');
+var chunkUF2N5L63_cjs = require('./chunk-UF2N5L63.cjs');
 require('./chunk-HZLQNKFN.cjs');
 require('./chunk-G4APZMCP.cjs');
 require('./chunk-NZU7W256.cjs');
@@ -226,7 +226,7 @@ var AvbridgeVideoElement = class extends HTMLElementCtor {
     this._dispatch("loadstart", {});
     let player;
     try {
-      player = await chunkOE66B34H_cjs.createPlayer({
+      player = await chunkUF2N5L63_cjs.createPlayer({
         source,
         target: this._videoEl,
         // Honor the consumer's preferred initial strategy. "auto" means

package/dist/element.d.cts

@@ -1,4 +1,4 @@
-import { a as MediaInput, m as StrategyName, S as StrategyClass, U as UnifiedPlayer, d as AudioTrackInfo, n as SubtitleTrackInfo, D as DiagnosticsSnapshot } from './player-DUyvltvy.cjs';
+import { a as MediaInput, m as StrategyName, S as StrategyClass, U as UnifiedPlayer, d as AudioTrackInfo, n as SubtitleTrackInfo, D as DiagnosticsSnapshot } from './player-U2NPmFvA.cjs';
 
 /**
  * `<avbridge-video>` — `HTMLMediaElement`-compatible primitive backed by the

package/dist/element.d.ts

@@ -1,4 +1,4 @@
-import { a as MediaInput, m as StrategyName, S as StrategyClass, U as UnifiedPlayer, d as AudioTrackInfo, n as SubtitleTrackInfo, D as DiagnosticsSnapshot } from './player-DUyvltvy.js';
+import { a as MediaInput, m as StrategyName, S as StrategyClass, U as UnifiedPlayer, d as AudioTrackInfo, n as SubtitleTrackInfo, D as DiagnosticsSnapshot } from './player-U2NPmFvA.js';
 
 /**
  * `<avbridge-video>` — `HTMLMediaElement`-compatible primitive backed by the

package/dist/element.js

@@ -1,4 +1,4 @@
-import { createPlayer } from './chunk-C5VA5U5O.js';
+import { createPlayer } from './chunk-DMWARSEF.js';
 import './chunk-ILKDNBSE.js';
 import './chunk-5DMTJVIU.js';
 import './chunk-J5MCMN3S.js';

package/dist/index.cjs

@@ -1,6 +1,6 @@
 'use strict';
 
-var chunkOE66B34H_cjs = require('./chunk-OE66B34H.cjs');
+var chunkUF2N5L63_cjs = require('./chunk-UF2N5L63.cjs');
 var chunkHZLQNKFN_cjs = require('./chunk-HZLQNKFN.cjs');
 var chunkL4NPOJ36_cjs = require('./chunk-L4NPOJ36.cjs');
 require('./chunk-G4APZMCP.cjs');
@@ -21,7 +21,7 @@ var MEDIABUNNY_CONTAINERS = /* @__PURE__ */ new Set([
 async function remux(source, options = {}) {
   const outputFormat = options.outputFormat ?? "mp4";
   options.signal?.throwIfAborted();
-  const ctx = await chunkOE66B34H_cjs.probe(source);
+  const ctx = await chunkUF2N5L63_cjs.probe(source);
   options.signal?.throwIfAborted();
   validateRemuxEligibility(ctx, options.strict ?? false);
   if (MEDIABUNNY_CONTAINERS.has(ctx.container)) {
@@ -33,7 +33,7 @@ function validateRemuxEligibility(ctx, strict) {
   const video = ctx.videoTracks[0];
   const audio = ctx.audioTracks[0];
   if (video) {
-    const mbCodec = chunkOE66B34H_cjs.avbridgeVideoToMediabunny(video.codec);
+    const mbCodec = chunkUF2N5L63_cjs.avbridgeVideoToMediabunny(video.codec);
     if (!mbCodec) {
       throw new Error(
         `Cannot remux: video codec "${video.codec}" is not supported for remuxing. Use transcode() to re-encode to a modern codec.`
@@ -41,7 +41,7 @@ function validateRemuxEligibility(ctx, strict) {
     }
   }
   if (audio) {
-    const mbCodec = chunkOE66B34H_cjs.avbridgeAudioToMediabunny(audio.codec);
+    const mbCodec = chunkUF2N5L63_cjs.avbridgeAudioToMediabunny(audio.codec);
     if (!mbCodec) {
       throw new Error(
         `Cannot remux: audio codec "${audio.codec}" is not supported for remuxing. Use transcode() to re-encode to a modern codec.`
@@ -60,7 +60,7 @@ function validateRemuxEligibility(ctx, strict) {
 async function remuxViaMediAbunny(ctx, outputFormat, options) {
   const mb = await import('mediabunny');
   const input = new mb.Input({
-    source: await chunkOE66B34H_cjs.buildMediabunnySourceFromInput(mb, ctx.source),
+    source: await chunkUF2N5L63_cjs.buildMediabunnySourceFromInput(mb, ctx.source),
     formats: mb.ALL_FORMATS
   });
   const target = new mb.BufferTarget();
@@ -146,8 +146,8 @@ async function doLibavRemux(libav, filename, ctx, outputFormat, options) {
   const audioStream = streams.find((s) => s.codec_type === libav.AVMEDIA_TYPE_AUDIO) ?? null;
   const videoTrackInfo = ctx.videoTracks[0];
   const audioTrackInfo = ctx.audioTracks[0];
-  const mbVideoCodec = videoTrackInfo ? chunkOE66B34H_cjs.avbridgeVideoToMediabunny(videoTrackInfo.codec) : null;
-  const mbAudioCodec = audioTrackInfo ? chunkOE66B34H_cjs.avbridgeAudioToMediabunny(audioTrackInfo.codec) : null;
+  const mbVideoCodec = videoTrackInfo ? chunkUF2N5L63_cjs.avbridgeVideoToMediabunny(videoTrackInfo.codec) : null;
+  const mbAudioCodec = audioTrackInfo ? chunkUF2N5L63_cjs.avbridgeAudioToMediabunny(audioTrackInfo.codec) : null;
   const target = new mb.BufferTarget();
   const output = new mb.Output({
     format: createOutputFormat(mb, outputFormat),
@@ -358,7 +358,7 @@ async function transcode(source, options = {}) {
   const quality = options.quality ?? "medium";
   validateCodecCompatibility(outputFormat, videoCodec, audioCodec);
   options.signal?.throwIfAborted();
-  const ctx = await chunkOE66B34H_cjs.probe(source);
+  const ctx = await chunkUF2N5L63_cjs.probe(source);
   options.signal?.throwIfAborted();
   if (!MEDIABUNNY_CONTAINERS2.has(ctx.container)) {
     throw new Error(
@@ -370,7 +370,7 @@ async function transcode(source, options = {}) {
 async function attemptTranscode(ctx, outputFormat, videoCodec, audioCodec, quality, options) {
   const mb = await import('mediabunny');
   const input = new mb.Input({
-    source: await chunkOE66B34H_cjs.buildMediabunnySourceFromInput(mb, ctx.source),
+    source: await chunkUF2N5L63_cjs.buildMediabunnySourceFromInput(mb, ctx.source),
     formats: mb.ALL_FORMATS
   });
   const target = new mb.BufferTarget();
@@ -552,23 +552,23 @@ function qualityToMediabunny(mb, quality) {
 
 Object.defineProperty(exports, "UnifiedPlayer", {
   enumerable: true,
-  get: function () { return chunkOE66B34H_cjs.UnifiedPlayer; }
+  get: function () { return chunkUF2N5L63_cjs.UnifiedPlayer; }
 });
 Object.defineProperty(exports, "classify", {
   enumerable: true,
-  get: function () { return chunkOE66B34H_cjs.classifyContext; }
+  get: function () { return chunkUF2N5L63_cjs.classifyContext; }
 });
 Object.defineProperty(exports, "createPlayer", {
   enumerable: true,
-  get: function () { return chunkOE66B34H_cjs.createPlayer; }
+  get: function () { return chunkUF2N5L63_cjs.createPlayer; }
 });
 Object.defineProperty(exports, "probe", {
   enumerable: true,
-  get: function () { return chunkOE66B34H_cjs.probe; }
+  get: function () { return chunkUF2N5L63_cjs.probe; }
 });
 Object.defineProperty(exports, "srtToVtt", {
   enumerable: true,
-  get: function () { return chunkOE66B34H_cjs.srtToVtt; }
+  get: function () { return chunkUF2N5L63_cjs.srtToVtt; }
 });
 exports.remux = remux;
 exports.transcode = transcode;

package/dist/index.d.cts

@@ -1,5 +1,5 @@
-import { M as MediaContext, C as Classification, a as MediaInput, b as ConvertOptions, c as ConvertResult, T as TranscodeOptions } from './player-DUyvltvy.cjs';
-export { A as AudioCodec, d as AudioTrackInfo, e as ContainerKind, f as CreatePlayerOptions, D as DiagnosticsSnapshot, H as HardwareAccelerationHint, O as OutputAudioCodec, g as OutputFormat, h as OutputVideoCodec, P as PlaybackSession, i as PlayerEventMap, j as PlayerEventName, k as Plugin, l as ProgressInfo, S as StrategyClass, m as StrategyName, n as SubtitleTrackInfo, o as TranscodeQuality, U as UnifiedPlayer, V as VideoCodec, p as VideoTrackInfo, q as createPlayer } from './player-DUyvltvy.cjs';
+import { M as MediaContext, C as Classification, a as MediaInput, b as ConvertOptions, c as ConvertResult, T as TranscodeOptions } from './player-U2NPmFvA.cjs';
+export { A as AudioCodec, d as AudioTrackInfo, e as ContainerKind, f as CreatePlayerOptions, D as DiagnosticsSnapshot, H as HardwareAccelerationHint, O as OutputAudioCodec, g as OutputFormat, h as OutputVideoCodec, P as PlaybackSession, i as PlayerEventMap, j as PlayerEventName, k as Plugin, l as ProgressInfo, S as StrategyClass, m as StrategyName, n as SubtitleTrackInfo, o as TranscodeQuality, U as UnifiedPlayer, V as VideoCodec, p as VideoTrackInfo, q as createPlayer } from './player-U2NPmFvA.cjs';
 
 /**
  * Pure classification — no I/O, no async. Test-friendly.

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { M as MediaContext, C as Classification, a as MediaInput, b as ConvertOptions, c as ConvertResult, T as TranscodeOptions } from './player-DUyvltvy.js';
-export { A as AudioCodec, d as AudioTrackInfo, e as ContainerKind, f as CreatePlayerOptions, D as DiagnosticsSnapshot, H as HardwareAccelerationHint, O as OutputAudioCodec, g as OutputFormat, h as OutputVideoCodec, P as PlaybackSession, i as PlayerEventMap, j as PlayerEventName, k as Plugin, l as ProgressInfo, S as StrategyClass, m as StrategyName, n as SubtitleTrackInfo, o as TranscodeQuality, U as UnifiedPlayer, V as VideoCodec, p as VideoTrackInfo, q as createPlayer } from './player-DUyvltvy.js';
+import { M as MediaContext, C as Classification, a as MediaInput, b as ConvertOptions, c as ConvertResult, T as TranscodeOptions } from './player-U2NPmFvA.js';
+export { A as AudioCodec, d as AudioTrackInfo, e as ContainerKind, f as CreatePlayerOptions, D as DiagnosticsSnapshot, H as HardwareAccelerationHint, O as OutputAudioCodec, g as OutputFormat, h as OutputVideoCodec, P as PlaybackSession, i as PlayerEventMap, j as PlayerEventName, k as Plugin, l as ProgressInfo, S as StrategyClass, m as StrategyName, n as SubtitleTrackInfo, o as TranscodeQuality, U as UnifiedPlayer, V as VideoCodec, p as VideoTrackInfo, q as createPlayer } from './player-U2NPmFvA.js';
 
 /**
  * Pure classification — no I/O, no async. Test-friendly.

package/dist/index.js

@@ -1,5 +1,5 @@
-import { probe, avbridgeVideoToMediabunny, avbridgeAudioToMediabunny, buildMediabunnySourceFromInput } from './chunk-C5VA5U5O.js';
-export { UnifiedPlayer, classifyContext as classify, createPlayer, probe, srtToVtt } from './chunk-C5VA5U5O.js';
+import { probe, avbridgeVideoToMediabunny, avbridgeAudioToMediabunny, buildMediabunnySourceFromInput } from './chunk-DMWARSEF.js';
+export { UnifiedPlayer, classifyContext as classify, createPlayer, probe, srtToVtt } from './chunk-DMWARSEF.js';
 import { normalizeSource } from './chunk-ILKDNBSE.js';
 import { prepareLibavInput } from './chunk-WD2ZNQA7.js';
 import './chunk-5DMTJVIU.js';

package/dist/{player-DUyvltvy.d.cts → player-U2NPmFvA.d.cts}

@@ -324,6 +324,7 @@ declare class UnifiedPlayer {
     private lastProgressTime;
     private lastProgressPosition;
     private errorListener;
+    private endedListener;
     private switchingPromise;
     private subtitleResources;
     /**

package/dist/{player-DUyvltvy.d.ts → player-U2NPmFvA.d.ts}

@@ -324,6 +324,7 @@ declare class UnifiedPlayer {
     private lastProgressTime;
     private lastProgressPosition;
     private errorListener;
+    private endedListener;
     private switchingPromise;
     private subtitleResources;
     /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "avbridge",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "description": "Play and convert arbitrary video files in the browser. Native, remux, hybrid, fallback, and transcode — one API.",
   "license": "MIT",
   "author": "Keishi Hattori",

package/src/player.ts

@@ -34,6 +34,11 @@ export class UnifiedPlayer {
   private lastProgressPosition = -1;
   private errorListener: (() => void) | null = null;
 
+  // Bound so we can removeEventListener in destroy(); without this the
+  // listener outlives the player and accumulates on elements that swap
+  // source (e.g. <avbridge-video>).
+  private endedListener: (() => void) | null = null;
+
   // Serializes escalation / setStrategy calls
   private switchingPromise: Promise<void> = Promise.resolve();
 
@@ -121,12 +126,10 @@ export class UnifiedPlayer {
       // Try the primary strategy, falling through the chain on failure
       await this.startSession(decision.strategy, decision.reason);
 
-      // Apply subtitles for non-canvas strategies. Awaited so fetch/parse
-      // failures surface deterministically — but per-track failures are
-      // caught inside attachSubtitleTracks and logged via console.warn so
-      // a single bad sidecar doesn't break bootstrap. Subtitles are not
-      // load-bearing for playback. (Promoting this to a typed `subtitleerror`
-      // event is a nice-to-have follow-up.)
+      // Apply subtitles for non-canvas strategies. Per-track failures are
+      // caught inside attachSubtitleTracks and logged via console.warn —
+      // subtitles are not load-bearing, so a bad sidecar must not break
+      // bootstrap.
       if (this.session!.strategy !== "fallback" && this.session!.strategy !== "hybrid") {
         await attachSubtitleTracks(
           this.options.target,
@@ -146,7 +149,8 @@ export class UnifiedPlayer {
       });
 
       this.startTimeupdateLoop();
-      this.options.target.addEventListener("ended", () => this.emitter.emit("ended", undefined));
+      this.endedListener = () => this.emitter.emit("ended", undefined);
+      this.options.target.addEventListener("ended", this.endedListener);
       this.emitter.emitSticky("ready", undefined);
       const bootstrapElapsed = performance.now() - bootstrapStart;
       dbg.info("bootstrap", `ready in ${bootstrapElapsed.toFixed(0)}ms`);
@@ -492,6 +496,10 @@ export class UnifiedPlayer {
       this.timeupdateInterval = null;
     }
     this.clearSupervisor();
+    if (this.endedListener) {
+      this.options.target.removeEventListener("ended", this.endedListener);
+      this.endedListener = null;
+    }
     if (this.session) {
       await this.session.destroy();
       this.session = null;
@@ -508,14 +516,12 @@ export async function createPlayer(options: CreatePlayerOptions): Promise<Unifie
 }
 
 /**
- * Build a synthetic classification when the consumer asked for a specific
- * initial strategy. We ask `classify()` for the natural decision so we can
- * inherit the correct fallback chain, then override the strategy + class.
- *
- * Why this matters: hard-coding `class: "NATIVE"` (the old behavior) made
- * diagnostics lie for every initialStrategy that wasn't actually native, and
- * any downstream logic that trusted `strategyClass` could make the wrong
- * decision. We now derive the class from the picked strategy.
+ * Build a synthetic classification for an explicit `initialStrategy` override.
+ * The `class` is derived from the chosen strategy so diagnostics and any
+ * downstream consumer of `strategyClass` see the real strategy. The fallback
+ * chain is inherited from the natural classification but must never contain
+ * `initial` itself — otherwise `startSession` would retry the strategy that
+ * just failed before escalating.
  *
  * @internal — exported for unit tests; not part of the public API.
  */
@@ -525,11 +531,13 @@ export function buildInitialDecision(
 ): Classification {
   const natural = classify(ctx);
   const cls = strategyToClass(initial, natural);
+  const inherited = natural.fallbackChain ?? defaultFallbackChain(initial);
+  const fallbackChain = inherited.filter((s) => s !== initial);
   return {
     class: cls,
     strategy: initial,
     reason: `initial strategy "${initial}" requested via options.initialStrategy`,
-    fallbackChain: natural.fallbackChain ?? defaultFallbackChain(initial),
+    fallbackChain,
   };
 }
 

package/src/strategies/fallback/index.ts

@@ -93,6 +93,13 @@ export async function createFallbackSession(
       void doSeek(v);
     },
   });
+  // Mirror `paused` from the audio clock — the underlying <video> never
+  // has its own src, so its native `paused` is always true and would
+  // mislead doSetStrategy's wasPlaying capture on a backend switch.
+  Object.defineProperty(target, "paused", {
+    configurable: true,
+    get: () => !audio.isPlaying(),
+  });
   // Mirror duration so the demo's controls can use target.duration too.
   if (ctx.duration && Number.isFinite(ctx.duration)) {
     Object.defineProperty(target, "duration", {
@@ -244,6 +251,7 @@ export async function createFallbackSession(
       try {
         delete (target as unknown as Record<string, unknown>).currentTime;
         delete (target as unknown as Record<string, unknown>).duration;
+        delete (target as unknown as Record<string, unknown>).paused;
       } catch { /* ignore */ }
     },
 

package/src/strategies/fallback/video-renderer.ts

@@ -51,8 +51,12 @@ export class VideoRenderer {
     });
 
     this.canvas = document.createElement("canvas");
+    // object-fit:contain letterboxes the canvas bitmap (sized to
+    // frame.displayWidth × displayHeight in paint()) inside the stage so
+    // portrait / non-stage-aspect content isn't stretched. Canvas is a
+    // replaced element, so object-fit applies.
     this.canvas.style.cssText =
-      "position:absolute;left:0;top:0;width:100%;height:100%;background:black;";
+      "position:absolute;left:0;top:0;width:100%;height:100%;background:black;object-fit:contain;";
 
     // Attach the canvas next to the video. When the video lives inside an
     // `<avbridge-video>` shadow root, `target.parentElement` is the

package/src/strategies/hybrid/index.ts

@@ -45,12 +45,19 @@ export async function createHybridSession(
     throw err;
   }
 
-  // Patch <video> element for the unified player layer
+  // Patch <video> element for the unified player layer. `paused` is
+  // mirrored from the audio clock so callers that inspect target.paused
+  // (notably doSetStrategy capturing wasPlaying) see the real play state
+  // — the underlying <video> never has its own src and stays paused.
   Object.defineProperty(target, "currentTime", {
     configurable: true,
     get: () => audio.now(),
     set: (v: number) => { void doSeek(v); },
   });
+  Object.defineProperty(target, "paused", {
+    configurable: true,
+    get: () => !audio.isPlaying(),
+  });
   if (ctx.duration && Number.isFinite(ctx.duration)) {
     Object.defineProperty(target, "duration", {
       configurable: true,
@@ -129,6 +136,7 @@ export async function createHybridSession(
       try {
         delete (target as unknown as Record<string, unknown>).currentTime;
         delete (target as unknown as Record<string, unknown>).duration;
+        delete (target as unknown as Record<string, unknown>).paused;
       } catch { /* ignore */ }
     },
 

package/src/strategies/remux/index.ts

@@ -36,7 +36,19 @@ export async function createRemuxSession(
         await pipeline.start(video.currentTime || 0, true);
         return;
       }
-      await video.play();
+      // seek() may have already started the pump with autoPlay=false
+      // (strategy-switch flow calls seek before play). Flip the pipeline's
+      // pending autoPlay so the MseSink fires video.play() once buffered
+      // data lands, and also attempt an immediate video.play() in case the
+      // sink is already wired up. The immediate call can reject when
+      // video.src hasn't been set yet — that's fine, the deferred path will
+      // catch it.
+      pipeline.setAutoPlay(true);
+      try {
+        await video.play();
+      } catch {
+        /* sink not ready yet; setAutoPlay will handle playback on first buffered write */
+      }
     },
     pause() {
       wantPlay = false;

package/src/strategies/remux/pipeline.ts

@@ -24,6 +24,8 @@ import {
 export interface RemuxPipeline {
   start(fromTime?: number, autoPlay?: boolean): Promise<void>;
   seek(time: number, autoPlay?: boolean): Promise<void>;
+  /** Update the autoplay intent mid-flight — used when play() arrives after seek() but before the MseSink has been constructed. */
+  setAutoPlay(autoPlay: boolean): void;
   destroy(): Promise<void>;
   stats(): Record<string, unknown>;
 }
@@ -248,6 +250,10 @@ export async function createRemuxPipeline(
         console.error("[avbridge] remux pipeline reseek failed:", err);
       });
     },
+    setAutoPlay(autoPlay) {
+      pendingAutoPlay = autoPlay;
+      if (sink) sink.setPlayOnSeek(autoPlay);
+    },
     async destroy() {
       destroyed = true;
       pumpToken++;