@editframe/elements 0.18.20-beta.0 → 0.18.22-beta.0

package/src/elements/SampleBuffer.ts

@@ -53,7 +53,6 @@ export class SampleBuffer {
   }
 
   find(desiredSeekTimeMs: number): MediaSample | undefined {
-    // Take snapshot to avoid concurrent modification during iteration
     const currentBuffer = [...this.buffer];
 
     if (currentBuffer.length === 0) return undefined;
@@ -70,7 +69,7 @@ export class SampleBuffer {
       const sampleEndMs = roundToMilliseconds(sampleStartMs + sampleDurationMs);
 
       // Check if the desired time falls within this sample's time span [start, end], inclusive of end
-      if (targetTimeMs >= sampleStartMs && targetTimeMs <= sampleEndMs) {
+      if (targetTimeMs >= sampleStartMs && targetTimeMs < sampleEndMs) {
         return sample;
       }
     }

package/src/gui/ContextMixin.browsertest.ts

@@ -1,4 +1,4 @@
-import { LitElement } from "lit";
+import { html, LitElement } from "lit";
 import { customElement } from "lit/decorators/custom-element.js";
 import { afterEach, beforeEach, describe, expect, test, vi } from "vitest";
 
@@ -13,7 +13,7 @@ class TestContext extends ContextMixin(LitElement) {}
 @customElement("test-context-reactivity")
 class TestContextElement extends ContextMixin(LitElement) {
   render() {
-    return document.createElement("ef-timegroup");
+    return html`<slot></slot>`;
   }
 }
 
@@ -631,6 +631,9 @@ describe("ContextMixin", () => {
       await timegroup.updateComplete;
       await newChild.updateComplete;
 
+      // Wait for next animation frame to ensure temporal cache is cleared
+      await new Promise((resolve) => requestAnimationFrame(resolve));
+
       // Duration should now be 15s (the max of all children)
       expect(element.targetTimegroup?.durationMs).toBe(15000);
 

package/src/gui/ContextMixin.ts

@@ -250,6 +250,13 @@ export function ContextMixin<T extends Constructor<LitElement>>(superClass: T) {
           if (newTimegroup !== this.targetTimegroup) {
             this.targetTimegroup = newTimegroup;
             shouldUpdate = true;
+          } else if (
+            mutation.target instanceof Element &&
+            (mutation.target.tagName === "EF-TIMEGROUP" ||
+              mutation.target.closest("ef-timegroup"))
+          ) {
+            // Handle childList changes within existing timegroups
+            shouldUpdate = true;
           }
         } else if (mutation.type === "attributes") {
           // Watch for attribute changes that might affect duration

package/test/EFVideo.framegen.browsertest.ts

@@ -50,40 +50,10 @@ describe("EFVideo Frame Generation", () => {
     expect(video.durationMs).toBe(0);
   });
 
-  test("effective mode is 'asset' if mode is not specified", async ({
-    expect,
-  }) => {
-    expect(video.effectiveMode).toBe("asset");
-  });
-
-  test("assetIndexLoader is pending", async ({ expect }) => {
-    expect(video.assetIndexLoader.status).toEqual(TaskStatus.PENDING);
-  });
-
-  test("assetSegmentKeysTask is pending", ({ expect }) => {
-    expect(video.assetSegmentKeysTask.status).toEqual(TaskStatus.PENDING);
-  });
-
-  test("assetInitSegmentsTask is pending", ({ expect }) => {
-    expect(video.assetInitSegmentsTask.status).toEqual(TaskStatus.PENDING);
-  });
-
-  test("assetSegmentLoader is pending", ({ expect }) => {
-    expect(video.assetSegmentLoader.status).toEqual(TaskStatus.PENDING);
-  });
-
   test("fragmentIndexTask is pending", ({ expect }) => {
     expect(video.fragmentIndexTask.status).toEqual(TaskStatus.PENDING);
   });
 
-  test("mediaSegmentsTask is pending", ({ expect }) => {
-    expect(video.mediaSegmentsTask.status).toEqual(TaskStatus.PENDING);
-  });
-
-  test("seekTask is pending", ({ expect }) => {
-    expect(video.seekTask.status).toEqual(TaskStatus.PENDING);
-  });
-
   // Note: Timing-dependent tests disabled due to seek range issues
   // The test asset data starts at 80ms but component initializes at 0ms
   // These tests validate task completion after data loading but fail on seek timing
@@ -96,32 +66,8 @@ describe("EFVideo Frame Generation", () => {
       expect(video.durationMs).toBeCloseTo(10_085, 0);
     });
 
-    test("assetIndexLoader is fulfilled", ({ expect }) => {
-      expect(video.assetIndexLoader.status).toEqual(TaskStatus.COMPLETE);
-    });
-
-    test("assetSegmentKeysTask is fulfilled", ({ expect }) => {
-      expect(video.assetSegmentKeysTask.status).toEqual(TaskStatus.COMPLETE);
-    });
-
-    test("assetInitSegmentsTask is fulfilled", ({ expect }) => {
-      expect(video.assetInitSegmentsTask.status).toEqual(TaskStatus.COMPLETE);
-    });
-
-    test("assetSegmentLoader is fulfilled", ({ expect }) => {
-      expect(video.assetSegmentLoader.status).toEqual(TaskStatus.COMPLETE);
-    });
-
     test("fragmentIndexTask is fulfilled", ({ expect }) => {
       expect(video.fragmentIndexTask.status).toEqual(TaskStatus.COMPLETE);
     });
-
-    test("mediaSegmentsTask is fulfilled", ({ expect }) => {
-      expect(video.mediaSegmentsTask.status).toEqual(TaskStatus.COMPLETE);
-    });
-
-    test("seekTask is fulfilled", ({ expect }) => {
-      expect(video.seekTask.status).toEqual(TaskStatus.COMPLETE);
-    });
   });
 });