eb-player 2.5.3 → 2.5.4

package/dist/build/types/src/core/browser-support.d.ts

@@ -0,0 +1,29 @@
+/**
+ * Runtime feature floor of the player.
+ *
+ * The bundle's *syntax* floor is ES2017 (see scripts/check-syntax-floor.ts),
+ * which parses on Chrome 55+, Safari 10.1+ and Firefox 52+. Some of those
+ * engines still lack runtime APIs the player cannot work without:
+ *
+ *   - Proxy           — the reactive PlayerState store (Chrome 49, Safari 10)
+ *   - AbortController — every subscription/listener cleanup (Chrome 66,
+ *                       Safari 12.1, Firefox 57)
+ *
+ * On such a browser `start()` used to throw a bare ReferenceError from deep
+ * inside PlayerController's constructor, leaving the integrator's container
+ * as an empty white box. This module lets start() detect the gap up front
+ * and show a readable message instead, using only pre-ES2015 DOM APIs.
+ */
+export declare const REQUIRED_RUNTIME_FEATURES: readonly ["Proxy", "AbortController"];
+/**
+ * Returns the names of the required runtime features that are not
+ * constructors on `scope` (defaults to `window`).
+ */
+export declare function findMissingRuntimeFeatures(scope?: Record<string, unknown>): string[];
+/**
+ * Replaces the container's content with the skin's error overlay markup
+ * (same class names as ErrorMessage, so the shipped CSS styles it) carrying
+ * the translated "browser not supported" text. Plain DOM calls only: this
+ * runs precisely on the browsers where lit-html/PlayerState cannot.
+ */
+export declare function renderUnsupportedBrowser(container: HTMLElement, lang?: string): void;

package/dist/build/types/src/engines/base-engine.d.ts

@@ -26,6 +26,14 @@ export declare abstract class BaseEngine extends EngineStateSync {
      */
     protected loadSourceUrl: string;
     private watchdog;
+    /**
+     * Every native video listener bound by bindVideoEvents(), so detach() can
+     * remove them itself. They are also registered with `{ signal }`, but
+     * Chromium < 90, Safari < 15 and Firefox < 86 silently ignore that option:
+     * on those engines the listeners outlived destroy() and kept writing a
+     * disposed PlayerState on every timeupdate (Sentry FORJA-FRONTWEB-5B9/5BA).
+     */
+    private videoListeners;
     /**
      * Promise that resolves when the engine driver (hls.js/dash.js) is created
      * and ready for integration (e.g., P2P SDK). Resolved inside onAttach().
@@ -49,6 +57,11 @@ export declare abstract class BaseEngine extends EngineStateSync {
      */
     setLoadSource(src: string): void;
     protected bindVideoEvents(video: HTMLVideoElement, state: PlayerState, signal: AbortSignal): void;
+    /**
+     * Removes every listener bound by bindVideoEvents() — the explicit half of
+     * the cleanup that `{ signal }` was supposed to do on its own.
+     */
+    private unbindVideoEvents;
     protected abstract recoverFromStall(attempt: number): void;
     protected abstract onAttach(): Promise<void> | void;
     protected abstract onDetach(): void;

package/dist/build/types/src/skin/base-component.d.ts

@@ -63,6 +63,16 @@ export declare abstract class BaseComponent {
     /**
      * Renders the component into this.el using lit-html.
      * No-op if el is null (e.g., after disconnect()).
+     *
+     * A render that throws is contained: lit-html keeps its parts alive through
+     * comment markers inside the component's subtree, and a DOM rewriter
+     * outside the player (page translation, an extension) that rebuilds that
+     * markup without the markers breaks the part for good — every later render
+     * throws the same `this._$AA.nextSibling is null` and, with a component
+     * re-rendering on each state tick, that is thousands of identical errors
+     * per session (Sentry FORJA-FRONTWEB-5EZ..5F1). The failure is logged once,
+     * the view is unsubscribed from state/bus updates, and render() becomes a
+     * no-op. disconnect() stays safe to call afterwards.
      */
     protected render(): void;
 }

package/dist/build/types/src/skin/controls/volume-control.d.ts

@@ -4,17 +4,27 @@ import { BaseComponent } from '../base-component';
  * Volume control with mute toggle button and draggable slider.
  *
  * - Mute button emits 'mute-toggle'
- * - Slider uses Pointer Events API (pointerdown/pointermove/pointerup)
+ * - Slider uses Pointer Events API (pointerdown/pointermove/pointerup/pointercancel)
  * - Slider fill width reflects current volume (0% to 100%)
  * - When muted, fill shows 0% visually
+ *
+ * The document-level drag listeners are stable instance methods, never
+ * per-drag closures: a second pointerdown landing during a drag (a second
+ * finger on a phone) used to overwrite the stored closure, so the first
+ * drag's listeners could never be removed again and kept calling
+ * eventToVolume() against a track rect that pointerup had already nulled
+ * (Sentry FORJA-FRONTWEB-5E3/5E4). With one function reference per event
+ * type, addEventListener is idempotent and removeEventListener always pairs.
  */
 export declare class VolumeControl extends BaseComponent {
     private trackRect;
-    private onPointerMove;
-    private onPointerUp;
+    private readonly onDocumentPointerMove;
+    private readonly onDocumentPointerUp;
+    private readonly onDocumentPointerCancel;
     protected onConnect(): void;
     private eventToVolume;
     private handlePointerDown;
+    private endDrag;
     private getVolumeIcon;
     protected template(): TemplateResult;
 }

package/dist/build/types/src/utils/play-quietly.d.ts

@@ -0,0 +1,17 @@
+/**
+ * Call `video.play()` and swallow its rejection.
+ *
+ * play() returns a promise that rejects with `AbortError` whenever pause() —
+ * or a new load — interrupts the pending play request. That is the normal
+ * outcome of a fast pause after play, an ad break, a cast hand-over or a
+ * source switch, never something the caller can act on, and left unhandled
+ * it reaches the page as an unhandled promise rejection (Sentry issues 18
+ * and W, ~420 users/day on forja.ma). Browsers predating the promise return
+ * (and jsdom) return undefined, so the catch is only attached when there is
+ * something to attach it to.
+ */
+/** Anything with a media-element-shaped play(): HTMLVideoElement, or the poster loop's minimal media handle. */
+export interface Playable {
+    play(): unknown;
+}
+export declare function playQuietly(video: Playable): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eb-player",
-  "version": "2.5.3",
+  "version": "2.5.4",
   "packageManager": "pnpm@11.24.0",
   "scripts": {
     "generate-demo": "pnpm generate --brand default --variant demo && cp dist/players/default/default.js test_player/src/demo.js",
@@ -12,13 +12,14 @@
     "lint": "eslint src/ test/",
     "lint:fix": "eslint src/ test/ --fix",
     "typecheck": "tsc --noEmit",
-    "build": "NODE_ENV=production rollup -c rollup.config.ts --configPlugin typescript",
-    "postbuild": "cp dist/build/eb-player.css dist/ && cp src/core/css/theme-*.css dist/build/ && cp src/core/css/theme-*.css dist/ && tsx scripts/size-check.ts && tsx scripts/check-no-vue.ts && tsx scripts/check-api-surface.ts",
+    "build": "NODE_ENV=production rollup -c rollup.config.ts --configPlugin \"typescript={tsconfig:'tsconfig.rollup.json'}\"",
+    "postbuild": "cp dist/build/eb-player.css dist/ && cp src/core/css/theme-*.css dist/build/ && cp src/core/css/theme-*.css dist/ && tsx scripts/size-check.ts && tsx scripts/check-no-vue.ts && tsx scripts/check-api-surface.ts && tsx scripts/check-syntax-floor.ts",
     "dev": "node --import tsx scripts/dev-server.ts",
     "generate": "node --import tsx scripts/generate.ts",
     "size-check": "tsx scripts/size-check.ts",
     "check-no-vue": "tsx scripts/check-no-vue.ts",
     "check-api-surface": "tsx scripts/check-api-surface.ts",
+    "check-syntax-floor": "tsx scripts/check-syntax-floor.ts",
     "prepare": "husky"
   },
   "devDependencies": {
@@ -30,6 +31,7 @@
     "@rollup/plugin-virtual": "^3.0.2",
     "@types/node": "^26.4.0",
     "@vitest/coverage-v8": "^4.1.8",
+    "acorn": "^8.16.0",
     "concurrently": "^10.0.3",
     "eslint": "^10.4.1",
     "eslint-plugin-lit": "^2.3.1",