@torrent-tv/proxy 2.89.5 → 2.89.7

package/test/seek-target-not-superseded.test.js

@@ -1,195 +1,195 @@
-/**
- * @file The segment the viewer just seeked TO must not be refused as stale.
- *
- * A seek bumps the wait epoch so that requests made for the position being left
- * stop being held. The request for the position being ARRIVED at races that
- * bump: hls.js asks for the new segment within milliseconds, and if the epoch
- * moves underneath it, the epoch check alone cannot tell the two apart.
- *
- * Measured 2026-08-18: the viewer seeked to 1061.0 s, `segment-00101` — the
- * segment at that very position — was answered 503 twice within 80 ms, the
- * player never asked for it again, and instead looped `a/0/segment-00103` and
- * `a/0/segment-00104` 737 and 736 times over 149 seconds while the picture
- * stood at `t=1061.0s readyState=1`. The session ended without another frame.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { SourceFile } from "../services/media/SourceFile.js";
-import { Timeline } from "../services/encode/output/Timeline.js";
-import { mkdtemp, rm } from "node:fs/promises";
-import os from "node:os";
-import path from "node:path";
-import { wireOutputs } from "../services/server/wire-outputs.js";
-import { fmp4Format } from "../services/encode/segment-formats/fmp4.js";
-
-const SESSION_ID = "aaaaaaaa11112222";
-const SEGMENT_SECONDS = 10.4;
-const SEEK_TO_SECONDS = 1061;
-const SEGMENT_AT_SEEK = Math.floor(SEEK_TO_SECONDS / SEGMENT_SECONDS);
-
-/**
- * A session whose viewer has just landed at {@link SEEK_TO_SECONDS}.
- *
- * @returns {Promise<{ manager: object, dirPath: string }>}
- */
-async function managerAfterSeek() {
-  const dirPath = await mkdtemp(path.join(os.tmpdir(), "seek-target-"));
-  const manager = wireOutputs({
-    enabled: true,
-    ffmpegBin: "ffmpeg",
-    localBindHost: "127.0.0.1",
-    localPort: 9090
-  });
-  const session = {
-    id: SESSION_ID,
-    dirPath,
-    // Where this file is cut, held by the file. A fixture that stated it
-    // on the session was describing what production no longer does.
-    timeline: new Timeline({
-      boundaries: Array.from({ length: 400 }, (_, index) => index * SEGMENT_SECONDS),
-      cutGrid: "uniform"
-    }),
-    state: "ready",
-    file: new SourceFile({ sourceKey: "source-1", fileIndex: 0, name: "film.mkv" }),
-    // An ordinary session reads its own file, and its sound is inside it. The
-    // three differ only for a soundtrack shipped as a file of its own.
-    get inputFile() { return this.file; },
-    get audioFile() { return this.file; },
-    startedAt: Date.now(),
-    lastAccessedAt: Date.now(),
-    consumers: new Set(),
-    segmentFormat: fmp4Format,
-    transcodeVideo: false,
-    useSyntheticPlaylist: true,
-    segmentCount: 399,
-    encodeStartIndex: SEGMENT_AT_SEEK,
-    waitEpoch: 1,
-    runState: "PRODUCING"
-  };
-  manager.outputs.set(SESSION_ID, session);
-  manager.viewers.of(session, "viewer").moveTo(SEEK_TO_SECONDS);
-  manager.viewers.of(session, "viewer").playing = false;
-  return { manager, dirPath };
-}
-
-test("the segment at the seek target is still wanted", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK).padStart(5, "0")}.mp4`),
-    true,
-    "this is the segment the viewer is waiting for; refusing it is what froze the field session"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("a segment behind the viewer is not wanted any more", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK - 3).padStart(5, "0")}.mp4`),
-    false,
-    "a request for the position the viewer left is exactly what the epoch exists to release"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("a segment far beyond what the run will reach is not wanted", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK + 200).padStart(5, "0")}.mp4`),
-    false,
-    "nothing will produce it before the viewer moves again"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("a playlist belongs to no position and is never stale", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-
-  assert.equal(manager.serving.requestStillWanted(SESSION_ID, "index.m3u8"), true);
-  assert.equal(manager.serving.requestStillWanted(SESSION_ID, "init.mp4"), true);
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("a request as deep as the cushion the browser is told to hold is still wanted", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-
-  // The browser sizes its forward buffer from the proxy's own look-ahead, so
-  // the deepest request it can make is one for the segment at the far edge of
-  // that cushion. Refusing it would be refusing the very depth this proxy
-  // asked for: at the old width — eight segments ahead of the viewer — three
-  // quarters of a full cushion's requests were stale by definition.
-  const edge = Math.floor((SEEK_TO_SECONDS + manager.lookaheadSeconds) / SEGMENT_SECONDS);
-
-  assert.ok(edge - SEGMENT_AT_SEEK > 8, "the case only exists past the old eight-segment width");
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(edge).padStart(5, "0")}.mp4`),
-    true,
-    "the far edge of the cushion the proxy itself keeps produced"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("the viewer who made the request is the one it is judged against", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-  const session = manager.outputs.get(SESSION_ID);
-  session.viewers = new Map();
-  // Two people watching one copied picture: one at the seek target, one a
-  // hundred segments back and waiting for a segment there. Asked about nobody
-  // in particular the answer is the FURTHEST of them, because what lies behind
-  // that one has already been made.
-  const behind = SEGMENT_AT_SEEK - 100;
-  manager.viewers.of(session, "behind").moveTo(behind * SEGMENT_SECONDS);
-  manager.viewers.of(session, "behind").playing = false;
-  manager.viewers.of(session, "ahead").moveTo(SEEK_TO_SECONDS);
-  manager.viewers.of(session, "ahead").playing = false;
-
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(behind).padStart(5, "0")}.mp4`, "behind"),
-    true,
-    "held for the viewer who is there, whatever the viewer in front is doing"
-  );
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(behind).padStart(5, "0")}.mp4`),
-    false,
-    "unnamed, it is judged against the furthest viewer, which is the leader"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
-
-test("a seek moves the seeking viewer's own head, and nobody else's", async () => {
-  const { manager, dirPath } = await managerAfterSeek();
-  const session = manager.outputs.get(SESSION_ID);
-  session.viewers = new Map();
-  const staying = SEGMENT_AT_SEEK - 40;
-  manager.viewers.of(session, "staying").position = {
-    segment: staying,
-    seconds: staying * SEGMENT_SECONDS,
-    at: Date.now()
-  };
-
-  const jumpTo = 120;
-  manager.viewerRequests.requestSeek(SESSION_ID, jumpTo, "jumping");
-
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(Math.floor(jumpTo / SEGMENT_SECONDS)).padStart(5, "0")}.mp4`, "jumping"),
-    true,
-    "the segment at the seek target — the request that raced the epoch in 2026-08-18"
-  );
-  assert.equal(
-    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(staying).padStart(5, "0")}.mp4`, "staying"),
-    true,
-    "somebody else's seek does not move where this viewer is"
-  );
-
-  await rm(dirPath, { recursive: true, force: true });
-});
+/**
+ * @file The segment the viewer just seeked TO must not be refused as stale.
+ *
+ * A seek bumps the wait epoch so that requests made for the position being left
+ * stop being held. The request for the position being ARRIVED at races that
+ * bump: hls.js asks for the new segment within milliseconds, and if the epoch
+ * moves underneath it, the epoch check alone cannot tell the two apart.
+ *
+ * Measured 2026-08-18: the viewer seeked to 1061.0 s, `segment-00101` — the
+ * segment at that very position — was answered 503 twice within 80 ms, the
+ * player never asked for it again, and instead looped `a/0/segment-00103` and
+ * `a/0/segment-00104` 737 and 736 times over 149 seconds while the picture
+ * stood at `t=1061.0s readyState=1`. The session ended without another frame.
+ */
+
+import test from "node:test";
+import assert from "node:assert/strict";
+import { SourceFile } from "../services/media/SourceFile.js";
+import { Timeline } from "../services/encode/output/Timeline.js";
+import { mkdtemp, rm } from "node:fs/promises";
+import os from "node:os";
+import path from "node:path";
+import { wireOutputs } from "../services/server/wire-outputs.js";
+import { fmp4Format } from "../services/encode/segment-formats/fmp4.js";
+
+const SESSION_ID = "aaaaaaaa11112222";
+const SEGMENT_SECONDS = 10.4;
+const SEEK_TO_SECONDS = 1061;
+const SEGMENT_AT_SEEK = Math.floor(SEEK_TO_SECONDS / SEGMENT_SECONDS);
+
+/**
+ * A session whose viewer has just landed at {@link SEEK_TO_SECONDS}.
+ *
+ * @returns {Promise<{ manager: object, dirPath: string }>}
+ */
+async function managerAfterSeek() {
+  const dirPath = await mkdtemp(path.join(os.tmpdir(), "seek-target-"));
+  const manager = wireOutputs({
+    enabled: true,
+    ffmpegBin: "ffmpeg",
+    localBindHost: "127.0.0.1",
+    localPort: 9090
+  });
+  const session = {
+    id: SESSION_ID,
+    dirPath,
+    // Where this file is cut, held by the file. A fixture that stated it
+    // on the session was describing what production no longer does.
+    timeline: new Timeline({
+      boundaries: Array.from({ length: 400 }, (_, index) => index * SEGMENT_SECONDS),
+      cutGrid: "uniform"
+    }),
+    state: "ready",
+    file: new SourceFile({ sourceKey: "source-1", fileIndex: 0, name: "film.mkv" }),
+    // An ordinary session reads its own file, and its sound is inside it. The
+    // three differ only for a soundtrack shipped as a file of its own.
+    get inputFile() { return this.file; },
+    get audioFile() { return this.file; },
+    startedAt: Date.now(),
+    lastAccessedAt: Date.now(),
+    consumers: new Set(),
+    segmentFormat: fmp4Format,
+    transcodeVideo: false,
+    useSyntheticPlaylist: true,
+    segmentCount: 399,
+    encodeStartIndex: SEGMENT_AT_SEEK,
+    waitEpoch: 1,
+    runState: "PRODUCING"
+  };
+  manager.outputs.set(SESSION_ID, session);
+  manager.viewers.of(session, "viewer").moveTo(SEEK_TO_SECONDS);
+  manager.viewers.of(session, "viewer").playing = false;
+  return { manager, dirPath };
+}
+
+test("the segment at the seek target is still wanted", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK).padStart(5, "0")}.mp4`),
+    true,
+    "this is the segment the viewer is waiting for; refusing it is what froze the field session"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("a segment behind the viewer is not wanted any more", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK - 3).padStart(5, "0")}.mp4`),
+    false,
+    "a request for the position the viewer left is exactly what the epoch exists to release"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("a segment far beyond what the run will reach is not wanted", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(SEGMENT_AT_SEEK + 200).padStart(5, "0")}.mp4`),
+    false,
+    "nothing will produce it before the viewer moves again"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("a playlist belongs to no position and is never stale", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+
+  assert.equal(manager.serving.requestStillWanted(SESSION_ID, "index.m3u8"), true);
+  assert.equal(manager.serving.requestStillWanted(SESSION_ID, "init.mp4"), true);
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("a request as deep as the cushion the browser is told to hold is still wanted", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+
+  // The browser sizes its forward buffer from the proxy's own look-ahead, so
+  // the deepest request it can make is one for the segment at the far edge of
+  // that cushion. Refusing it would be refusing the very depth this proxy
+  // asked for: at the old width — eight segments ahead of the viewer — three
+  // quarters of a full cushion's requests were stale by definition.
+  const edge = Math.floor((SEEK_TO_SECONDS + manager.lookaheadSeconds) / SEGMENT_SECONDS);
+
+  assert.ok(edge - SEGMENT_AT_SEEK > 8, "the case only exists past the old eight-segment width");
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(edge).padStart(5, "0")}.mp4`),
+    true,
+    "the far edge of the cushion the proxy itself keeps produced"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("the viewer who made the request is the one it is judged against", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+  const session = manager.outputs.get(SESSION_ID);
+  session.viewers = new Map();
+  // Two people watching one copied picture: one at the seek target, one a
+  // hundred segments back and waiting for a segment there. Asked about nobody
+  // in particular the answer is the FURTHEST of them, because what lies behind
+  // that one has already been made.
+  const behind = SEGMENT_AT_SEEK - 100;
+  manager.viewers.of(session, "behind").moveTo(behind * SEGMENT_SECONDS);
+  manager.viewers.of(session, "behind").playing = false;
+  manager.viewers.of(session, "ahead").moveTo(SEEK_TO_SECONDS);
+  manager.viewers.of(session, "ahead").playing = false;
+
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(behind).padStart(5, "0")}.mp4`, "behind"),
+    true,
+    "held for the viewer who is there, whatever the viewer in front is doing"
+  );
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(behind).padStart(5, "0")}.mp4`),
+    false,
+    "unnamed, it is judged against the furthest viewer, which is the leader"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});
+
+test("a seek moves the seeking viewer's own head, and nobody else's", async () => {
+  const { manager, dirPath } = await managerAfterSeek();
+  const session = manager.outputs.get(SESSION_ID);
+  session.viewers = new Map();
+  const staying = SEGMENT_AT_SEEK - 40;
+  manager.viewers.of(session, "staying").position = {
+    segment: staying,
+    seconds: staying * SEGMENT_SECONDS,
+    at: Date.now()
+  };
+
+  const jumpTo = 120;
+  manager.viewerRequests.requestSeek(SESSION_ID, jumpTo, "jumping");
+
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(Math.floor(jumpTo / SEGMENT_SECONDS)).padStart(5, "0")}.mp4`, "jumping"),
+    true,
+    "the segment at the seek target — the request that raced the epoch in 2026-08-18"
+  );
+  assert.equal(
+    manager.serving.requestStillWanted(SESSION_ID, `segment-${String(staying).padStart(5, "0")}.mp4`, "staying"),
+    true,
+    "somebody else's seek does not move where this viewer is"
+  );
+
+  await rm(dirPath, { recursive: true, force: true });
+});

package/test/segment-cut-times.test.js

@@ -1,63 +1,63 @@
-/**
- * The playlist and the real segments must describe the same thing.
- *
- * They did not. The playlist was built from the container index while ffmpeg
- * chose its own cut points from a target duration, and the two only ever agreed
- * by luck — on a field file the index listed 1902 keyframes, ffmpeg found about
- * twice as many, and segment #876 meant 1:26:50 to the player and roughly
- * minute 58 to ffmpeg. Seeks landed nowhere near where they were aimed.
- *
- * These tests pin the arithmetic that ties the two together.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { segmentCutTimesFrom } from "../services/encode/run-command.js";
-
-test("cut times are the interior boundaries, rebased on the run start", () => {
-  // Segments 0-10, 10-20, 20-30, 30-40 (file ends at 40).
-  const boundaries = [0, 10, 20, 30, 40];
-
-  assert.deepEqual(
-    segmentCutTimesFrom(boundaries, 0),
-    [10, 20, 30],
-    "from the start, the cuts are every boundary except 0 and the file end"
-  );
-
-  // Restarting at segment 2 means the run begins at 20 s; a cut at 30 s is 10 s
-  // into the run. Sending 30 would put it at 50 — `-segment_times` counts from
-  // the run, which is what made a seek land in the wrong place.
-  assert.deepEqual(
-    segmentCutTimesFrom(boundaries, 2),
-    [10],
-    "after a restart the times must be relative to where the run starts"
-  );
-});
-
-test("uneven, real-world boundaries survive the rebasing exactly", () => {
-  // Keyframes never land on round numbers; rounding here is what would push a
-  // cut past its keyframe and silently double a segment.
-  // Starting at segment 1 leaves segments 1, 2 and 3 — three segments, so two
-  // cuts between them, each measured from the run start at 10.427.
-  const boundaries = [0, 10.427, 20.854, 31.281, 41.708];
-  assert.deepEqual(segmentCutTimesFrom(boundaries, 1), [10.427, 20.854]);
-});
-
-test("boundaries that cannot serve are refused rather than half-used", () => {
-  assert.equal(segmentCutTimesFrom(null, 0), null);
-  assert.equal(segmentCutTimesFrom([], 0), null);
-  assert.equal(segmentCutTimesFrom([0], 0), null, "a single boundary describes no segment");
-  assert.equal(
-    segmentCutTimesFrom([0, 10, 20], 2),
-    null,
-    "a start index at or past the last boundary has no run to describe"
-  );
-});
-
-test("the final run has no interior cuts and asks for none", () => {
-  assert.deepEqual(
-    segmentCutTimesFrom([0, 10, 20], 1),
-    [],
-    "starting at the last segment leaves nothing to cut, which is not an error"
-  );
-});
+/**
+ * The playlist and the real segments must describe the same thing.
+ *
+ * They did not. The playlist was built from the container index while ffmpeg
+ * chose its own cut points from a target duration, and the two only ever agreed
+ * by luck — on a field file the index listed 1902 keyframes, ffmpeg found about
+ * twice as many, and segment #876 meant 1:26:50 to the player and roughly
+ * minute 58 to ffmpeg. Seeks landed nowhere near where they were aimed.
+ *
+ * These tests pin the arithmetic that ties the two together.
+ */
+
+import test from "node:test";
+import assert from "node:assert/strict";
+import { segmentCutTimesFrom } from "../services/encode/run-command.js";
+
+test("cut times are the interior boundaries, rebased on the run start", () => {
+  // Segments 0-10, 10-20, 20-30, 30-40 (file ends at 40).
+  const boundaries = [0, 10, 20, 30, 40];
+
+  assert.deepEqual(
+    segmentCutTimesFrom(boundaries, 0),
+    [10, 20, 30],
+    "from the start, the cuts are every boundary except 0 and the file end"
+  );
+
+  // Restarting at segment 2 means the run begins at 20 s; a cut at 30 s is 10 s
+  // into the run. Sending 30 would put it at 50 — `-segment_times` counts from
+  // the run, which is what made a seek land in the wrong place.
+  assert.deepEqual(
+    segmentCutTimesFrom(boundaries, 2),
+    [10],
+    "after a restart the times must be relative to where the run starts"
+  );
+});
+
+test("uneven, real-world boundaries survive the rebasing exactly", () => {
+  // Keyframes never land on round numbers; rounding here is what would push a
+  // cut past its keyframe and silently double a segment.
+  // Starting at segment 1 leaves segments 1, 2 and 3 — three segments, so two
+  // cuts between them, each measured from the run start at 10.427.
+  const boundaries = [0, 10.427, 20.854, 31.281, 41.708];
+  assert.deepEqual(segmentCutTimesFrom(boundaries, 1), [10.427, 20.854]);
+});
+
+test("boundaries that cannot serve are refused rather than half-used", () => {
+  assert.equal(segmentCutTimesFrom(null, 0), null);
+  assert.equal(segmentCutTimesFrom([], 0), null);
+  assert.equal(segmentCutTimesFrom([0], 0), null, "a single boundary describes no segment");
+  assert.equal(
+    segmentCutTimesFrom([0, 10, 20], 2),
+    null,
+    "a start index at or past the last boundary has no run to describe"
+  );
+});
+
+test("the final run has no interior cuts and asks for none", () => {
+  assert.deepEqual(
+    segmentCutTimesFrom([0, 10, 20], 1),
+    [],
+    "starting at the last segment leaves nothing to cut, which is not an error"
+  );
+});