@torrent-tv/proxy 2.74.0 → 2.75.0

package/test/matroska-cues-track.test.js

@@ -1,192 +1,192 @@
-/**
- * @file The container's table names a track, and only the picture's entries are
- * cut points.
- *
- * Measured 2026-08-18 on the two files the field sessions were recorded from:
- * `Minions.and.Monsters.1080p.mkv` carries 2778 cue entries for its video track
- * — one every 2.002 s — and 4669 more across four subtitle tracks;
- * `Moana.2 … MegaPeer.mkv` carries 1055 for video and 5007 across five subtitle
- * tracks. Read without the track, both sets went into the cut list together,
- * and ffmpeg — which can only cut a copied picture at a real keyframe at or
- * after the time it is asked for — moved every such cut forward to the next
- * keyframe. That is the whole of the disagreement the field reported: 2.002 s
- * on the first file, a median of 6.3 s and a worst case of 21 s on the second,
- * and never once negative.
- */
-
-import test from "node:test";
-import assert from "node:assert/strict";
-import { readMatroskaKeyframeTimes } from "../services/container-index/matroska.js";
-
-const ID_EBML = 0x1a45dfa3;
-const ID_SEGMENT = 0x18538067;
-const ID_SEEK_HEAD = 0x114d9b74;
-const ID_SEEK = 0x4dbb;
-const ID_SEEK_ID = 0x53ab;
-const ID_SEEK_POSITION = 0x53ac;
-const ID_INFO = 0x1549a966;
-const ID_TIMESTAMP_SCALE = 0x2ad7b1;
-const ID_TRACKS = 0x1654ae6b;
-const ID_TRACK_ENTRY = 0xae;
-const ID_TRACK_NUMBER = 0xd7;
-const ID_TRACK_TYPE = 0x83;
-const ID_CUES = 0x1c53bb6b;
-const ID_CUE_POINT = 0xbb;
-const ID_CUE_TIME = 0xb3;
-const ID_CUE_TRACK_POSITIONS = 0xb7;
-const ID_CUE_TRACK = 0xf7;
-const ID_CUE_CLUSTER_POSITION = 0xf1;
-
-/** An element id, as the bytes the specification gives it. */
-function idBytes(id) {
-  const bytes = [];
-  let rest = id;
-  while (rest > 0) {
-    bytes.unshift(rest & 0xff);
-    rest = Math.floor(rest / 256);
-  }
-  return Buffer.from(bytes);
-}
-
-/** A size, as a four-byte EBML variable-length integer. */
-function sizeBytes(size) {
-  const buffer = Buffer.alloc(4);
-  buffer.writeUInt32BE(size, 0);
-  buffer[0] |= 0x10;
-  return buffer;
-}
-
-function element(id, payload) {
-  return Buffer.concat([idBytes(id), sizeBytes(payload.length), payload]);
-}
-
-/** An unsigned value, in as few bytes as it needs. */
-function uintElement(id, value) {
-  const bytes = [];
-  let rest = value;
-  do {
-    bytes.unshift(rest & 0xff);
-    rest = Math.floor(rest / 256);
-  } while (rest > 0);
-  return element(id, Buffer.from(bytes));
-}
-
-function cuePoint(timeMs, tracks) {
-  return element(ID_CUE_POINT, Buffer.concat([
-    uintElement(ID_CUE_TIME, timeMs),
-    ...tracks.map((track) => element(ID_CUE_TRACK_POSITIONS, Buffer.concat([
-      uintElement(ID_CUE_TRACK, track),
-      uintElement(ID_CUE_CLUSTER_POSITION, 4096)
-    ])))
-  ]));
-}
-
-/**
- * A file with the shape the field files have: one picture, one set of subtitles,
- * and a table that indexes both.
- *
- * @param {{ withTracks?: boolean }} [options]
- * @returns {Buffer}
- */
-function buildFile({ withTracks = true, cueTrack = null } = {}) {
-  const info = element(ID_INFO, uintElement(ID_TIMESTAMP_SCALE, 1_000_000));
-  const tracks = element(ID_TRACKS, Buffer.concat([
-    element(ID_TRACK_ENTRY, Buffer.concat([
-      uintElement(ID_TRACK_NUMBER, 1),
-      uintElement(ID_TRACK_TYPE, 1) // video
-    ])),
-    element(ID_TRACK_ENTRY, Buffer.concat([
-      uintElement(ID_TRACK_NUMBER, 2),
-      uintElement(ID_TRACK_TYPE, 17) // subtitles
-    ]))
-  ]));
-  const forPicture = cueTrack ?? 1;
-  const forSubtitles = cueTrack ?? 2;
-  const cues = element(ID_CUES, Buffer.concat([
-    cuePoint(0, [forPicture]),
-    cuePoint(1070, [forSubtitles]),
-    cuePoint(2002, [forPicture]),
-    cuePoint(3141, [forSubtitles]),
-    cuePoint(4004, [forPicture])
-  ]));
-
-  const seekEntry = (targetId, position) => element(ID_SEEK, Buffer.concat([
-    element(ID_SEEK_ID, idBytes(targetId)),
-    element(ID_SEEK_POSITION, (() => {
-      const buffer = Buffer.alloc(4);
-      buffer.writeUInt32BE(position, 0);
-      return buffer;
-    })())
-  ]));
-  // Positions are relative to the Segment's payload, so the SeekHead has to be
-  // measured before they can be stated. Its own length does not change when the
-  // placeholders become real values: every size and position here is written at
-  // a fixed width.
-  const draft = element(ID_SEEK_HEAD, Buffer.concat([
-    seekEntry(ID_INFO, 0),
-    ...(withTracks ? [seekEntry(ID_TRACKS, 0)] : []),
-    seekEntry(ID_CUES, 0)
-  ]));
-  const infoAt = draft.length;
-  const tracksAt = infoAt + info.length;
-  const cuesAt = withTracks ? tracksAt + tracks.length : infoAt + info.length;
-  const seekHead = element(ID_SEEK_HEAD, Buffer.concat([
-    seekEntry(ID_INFO, infoAt),
-    ...(withTracks ? [seekEntry(ID_TRACKS, tracksAt)] : []),
-    seekEntry(ID_CUES, cuesAt)
-  ]));
-
-  const segmentPayload = Buffer.concat(
-    withTracks ? [seekHead, info, tracks, cues] : [seekHead, info, cues]
-  );
-  return Buffer.concat([
-    element(ID_EBML, Buffer.from([0x42, 0x86, 0x81, 0x01])),
-    element(ID_SEGMENT, segmentPayload)
-  ]);
-}
-
-function readerOver(file) {
-  return async (start, end) => {
-    const last = Math.min(end, file.length - 1);
-    return start > last ? null : file.subarray(start, last + 1);
-  };
-}
-
-test("only the picture's entries become cut times", async () => {
-  const file = buildFile();
-
-  const times = await readMatroskaKeyframeTimes(readerOver(file), file.length);
-
-  assert.deepEqual(
-    times.map((time) => Number(time.toFixed(3))),
-    [0, 2.002, 4.004],
-    "the subtitle entries at 1.070 and 3.141 are not keyframes, and a cut asked for there lands late"
-  );
-});
-
-test("a table whose entries name no known track is still used", async () => {
-  // The cue points reference track 9, which Tracks never declares. Filtering
-  // leaves nothing — and returning nothing would put an even grid on a copied
-  // picture, the failure this reader exists to prevent.
-  const file = buildFile({ cueTrack: 9 });
-
-  const times = await readMatroskaKeyframeTimes(readerOver(file), file.length);
-
-  assert.deepEqual(
-    times.map((time) => Number(time.toFixed(3))),
-    [0, 1.07, 2.002, 3.141, 4.004],
-    "an unrecognised table beats no table at all"
-  );
-});
-
-test("a file whose tracks cannot be read keeps every entry", async () => {
-  const file = buildFile({ withTracks: false });
-
-  const times = await readMatroskaKeyframeTimes(readerOver(file), file.length);
-
-  assert.deepEqual(
-    times.map((time) => Number(time.toFixed(3))),
-    [0, 1.07, 2.002, 3.141, 4.004],
-    "with nothing to tell the tracks apart, the old behaviour is the only one available"
-  );
-});
+/**
+ * @file The container's table names a track, and only the picture's entries are
+ * cut points.
+ *
+ * Measured 2026-08-18 on the two files the field sessions were recorded from:
+ * `Minions.and.Monsters.1080p.mkv` carries 2778 cue entries for its video track
+ * — one every 2.002 s — and 4669 more across four subtitle tracks;
+ * `Moana.2 … MegaPeer.mkv` carries 1055 for video and 5007 across five subtitle
+ * tracks. Read without the track, both sets went into the cut list together,
+ * and ffmpeg — which can only cut a copied picture at a real keyframe at or
+ * after the time it is asked for — moved every such cut forward to the next
+ * keyframe. That is the whole of the disagreement the field reported: 2.002 s
+ * on the first file, a median of 6.3 s and a worst case of 21 s on the second,
+ * and never once negative.
+ */
+
+import test from "node:test";
+import assert from "node:assert/strict";
+import { MatroskaContainer } from "../services/container/MatroskaContainer.js";
+
+const ID_EBML = 0x1a45dfa3;
+const ID_SEGMENT = 0x18538067;
+const ID_SEEK_HEAD = 0x114d9b74;
+const ID_SEEK = 0x4dbb;
+const ID_SEEK_ID = 0x53ab;
+const ID_SEEK_POSITION = 0x53ac;
+const ID_INFO = 0x1549a966;
+const ID_TIMESTAMP_SCALE = 0x2ad7b1;
+const ID_TRACKS = 0x1654ae6b;
+const ID_TRACK_ENTRY = 0xae;
+const ID_TRACK_NUMBER = 0xd7;
+const ID_TRACK_TYPE = 0x83;
+const ID_CUES = 0x1c53bb6b;
+const ID_CUE_POINT = 0xbb;
+const ID_CUE_TIME = 0xb3;
+const ID_CUE_TRACK_POSITIONS = 0xb7;
+const ID_CUE_TRACK = 0xf7;
+const ID_CUE_CLUSTER_POSITION = 0xf1;
+
+/** An element id, as the bytes the specification gives it. */
+function idBytes(id) {
+  const bytes = [];
+  let rest = id;
+  while (rest > 0) {
+    bytes.unshift(rest & 0xff);
+    rest = Math.floor(rest / 256);
+  }
+  return Buffer.from(bytes);
+}
+
+/** A size, as a four-byte EBML variable-length integer. */
+function sizeBytes(size) {
+  const buffer = Buffer.alloc(4);
+  buffer.writeUInt32BE(size, 0);
+  buffer[0] |= 0x10;
+  return buffer;
+}
+
+function element(id, payload) {
+  return Buffer.concat([idBytes(id), sizeBytes(payload.length), payload]);
+}
+
+/** An unsigned value, in as few bytes as it needs. */
+function uintElement(id, value) {
+  const bytes = [];
+  let rest = value;
+  do {
+    bytes.unshift(rest & 0xff);
+    rest = Math.floor(rest / 256);
+  } while (rest > 0);
+  return element(id, Buffer.from(bytes));
+}
+
+function cuePoint(timeMs, tracks) {
+  return element(ID_CUE_POINT, Buffer.concat([
+    uintElement(ID_CUE_TIME, timeMs),
+    ...tracks.map((track) => element(ID_CUE_TRACK_POSITIONS, Buffer.concat([
+      uintElement(ID_CUE_TRACK, track),
+      uintElement(ID_CUE_CLUSTER_POSITION, 4096)
+    ])))
+  ]));
+}
+
+/**
+ * A file with the shape the field files have: one picture, one set of subtitles,
+ * and a table that indexes both.
+ *
+ * @param {{ withTracks?: boolean }} [options]
+ * @returns {Buffer}
+ */
+function buildFile({ withTracks = true, cueTrack = null } = {}) {
+  const info = element(ID_INFO, uintElement(ID_TIMESTAMP_SCALE, 1_000_000));
+  const tracks = element(ID_TRACKS, Buffer.concat([
+    element(ID_TRACK_ENTRY, Buffer.concat([
+      uintElement(ID_TRACK_NUMBER, 1),
+      uintElement(ID_TRACK_TYPE, 1) // video
+    ])),
+    element(ID_TRACK_ENTRY, Buffer.concat([
+      uintElement(ID_TRACK_NUMBER, 2),
+      uintElement(ID_TRACK_TYPE, 17) // subtitles
+    ]))
+  ]));
+  const forPicture = cueTrack ?? 1;
+  const forSubtitles = cueTrack ?? 2;
+  const cues = element(ID_CUES, Buffer.concat([
+    cuePoint(0, [forPicture]),
+    cuePoint(1070, [forSubtitles]),
+    cuePoint(2002, [forPicture]),
+    cuePoint(3141, [forSubtitles]),
+    cuePoint(4004, [forPicture])
+  ]));
+
+  const seekEntry = (targetId, position) => element(ID_SEEK, Buffer.concat([
+    element(ID_SEEK_ID, idBytes(targetId)),
+    element(ID_SEEK_POSITION, (() => {
+      const buffer = Buffer.alloc(4);
+      buffer.writeUInt32BE(position, 0);
+      return buffer;
+    })())
+  ]));
+  // Positions are relative to the Segment's payload, so the SeekHead has to be
+  // measured before they can be stated. Its own length does not change when the
+  // placeholders become real values: every size and position here is written at
+  // a fixed width.
+  const draft = element(ID_SEEK_HEAD, Buffer.concat([
+    seekEntry(ID_INFO, 0),
+    ...(withTracks ? [seekEntry(ID_TRACKS, 0)] : []),
+    seekEntry(ID_CUES, 0)
+  ]));
+  const infoAt = draft.length;
+  const tracksAt = infoAt + info.length;
+  const cuesAt = withTracks ? tracksAt + tracks.length : infoAt + info.length;
+  const seekHead = element(ID_SEEK_HEAD, Buffer.concat([
+    seekEntry(ID_INFO, infoAt),
+    ...(withTracks ? [seekEntry(ID_TRACKS, tracksAt)] : []),
+    seekEntry(ID_CUES, cuesAt)
+  ]));
+
+  const segmentPayload = Buffer.concat(
+    withTracks ? [seekHead, info, tracks, cues] : [seekHead, info, cues]
+  );
+  return Buffer.concat([
+    element(ID_EBML, Buffer.from([0x42, 0x86, 0x81, 0x01])),
+    element(ID_SEGMENT, segmentPayload)
+  ]);
+}
+
+function readerOver(file) {
+  return async (start, end) => {
+    const last = Math.min(end, file.length - 1);
+    return start > last ? null : file.subarray(start, last + 1);
+  };
+}
+
+test("only the picture's entries become cut times", async () => {
+  const file = buildFile();
+
+  const times = await MatroskaContainer.readKeyframeTimes(readerOver(file), file.length);
+
+  assert.deepEqual(
+    times.map((time) => Number(time.toFixed(3))),
+    [0, 2.002, 4.004],
+    "the subtitle entries at 1.070 and 3.141 are not keyframes, and a cut asked for there lands late"
+  );
+});
+
+test("a table whose entries name no known track is still used", async () => {
+  // The cue points reference track 9, which Tracks never declares. Filtering
+  // leaves nothing — and returning nothing would put an even grid on a copied
+  // picture, the failure this reader exists to prevent.
+  const file = buildFile({ cueTrack: 9 });
+
+  const times = await MatroskaContainer.readKeyframeTimes(readerOver(file), file.length);
+
+  assert.deepEqual(
+    times.map((time) => Number(time.toFixed(3))),
+    [0, 1.07, 2.002, 3.141, 4.004],
+    "an unrecognised table beats no table at all"
+  );
+});
+
+test("a file whose tracks cannot be read keeps every entry", async () => {
+  const file = buildFile({ withTracks: false });
+
+  const times = await MatroskaContainer.readKeyframeTimes(readerOver(file), file.length);
+
+  assert.deepEqual(
+    times.map((time) => Number(time.toFixed(3))),
+    [0, 1.07, 2.002, 3.141, 4.004],
+    "with nothing to tell the tracks apart, the old behaviour is the only one available"
+  );
+});

package/test/video-facts.test.js

@@ -0,0 +1,102 @@
+/**
+ * @file The picture's facts, from the two readings that state them.
+ *
+ * Audio and subtitles have had this reconciliation since their flags were first
+ * read from the file. Video never did: every figure the encode is planned from
+ * came from ffmpeg's `-i` banner alone, and the `VideoTrack` the container
+ * declares was read and then used for nothing but a line in the log — so a file
+ * that states its bit depth or its HDR signalling, against a probe that does
+ * not print them, disagreed with nobody watching.
+ *
+ * What each check pins is WHICH reading answers, and why: the size and the
+ * frame rate are what the decoder will produce, so the probe answers; the bit
+ * depth and the HDR signalling are the file saying how its samples are to be
+ * read, so the container answers.
+ */
+
+import test from "node:test";
+import assert from "node:assert/strict";
+
+import { Container } from "../services/container/Container.js";
+
+const banner = (fields) => ({ width: null, height: null, fps: null, isHdr: false, bitDepth: null, ...fields });
+
+test("with no container reading the probe answers for everything", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 1920, height: 1080, fps: 23.976, isHdr: true, bitDepth: 10 }),
+    null
+  );
+  assert.equal(facts.width, 1920);
+  assert.equal(facts.height, 1080);
+  assert.equal(facts.fps, 23.976);
+  assert.equal(facts.isHdr, true);
+  assert.equal(facts.bitDepth, 10);
+  assert.deepEqual(facts.disagreements, []);
+});
+
+test("the size and the frame rate are the probe's, because that is what will be decoded", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 1920, height: 1080, fps: 24 }),
+    { width: 1440, height: 1080, fps: 25, displayWidth: 1920, displayHeight: 1080 }
+  );
+  assert.equal(facts.width, 1920, "the ladder and the scale filter are sized to the decoded frame");
+  assert.equal(facts.height, 1080);
+  assert.equal(facts.fps, 24);
+});
+
+test("the bit depth and the HDR signalling are the file's, because the file states them", () => {
+  // A ten-bit HEVC whose probe printed neither: this is the case that decides
+  // whether tone mapping runs and how the decode cost is priced.
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 3840, height: 2160, fps: 24 }),
+    { width: 3840, height: 2160, bitDepth: 10, isHdr: true }
+  );
+  assert.equal(facts.bitDepth, 10);
+  assert.equal(facts.isHdr, true);
+  assert.deepEqual(facts.disagreements, [], "one side saying nothing is not a disagreement");
+});
+
+test("a field only the probe states is still answered", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 1280, height: 720, fps: 30, bitDepth: 8 }),
+    { width: null, height: null, fps: null, bitDepth: null }
+  );
+  assert.equal(facts.width, 1280);
+  assert.equal(facts.bitDepth, 8);
+});
+
+test("a real disagreement is reported, with both values and which is which", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 1920, height: 1080, bitDepth: 8, isHdr: false }),
+    { width: 1280, height: 720, bitDepth: 10, isHdr: true }
+  );
+  assert.equal(facts.disagreements.length, 3);
+  assert.ok(facts.disagreements.some((line) => /width 1280 in the container against 1920 in the probe/.test(line)));
+  assert.ok(facts.disagreements.some((line) => /bit depth 10 in the container against 8 in the probe/.test(line)));
+  // HDR is not among them, and cannot be: see `mergeVideoFacts`.
+  assert.ok(!facts.disagreements.some((line) => /HDR/.test(line)));
+  // And the rule still decides: the probe for the size, the file for the rest.
+  assert.equal(facts.width, 1920);
+  assert.equal(facts.bitDepth, 10);
+  assert.equal(facts.isHdr, true);
+});
+
+test("the display size is the container's alone — the probe has no such field", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 1440, height: 1080 }),
+    { width: 1440, height: 1080, displayWidth: 1920, displayHeight: 1080 }
+  );
+  assert.equal(facts.displayWidth, 1920);
+  assert.equal(facts.displayHeight, 1080);
+});
+
+test("zero and nonsense are not values", () => {
+  const facts = Container.mergeVideoFacts(
+    banner({ width: 0, height: 0, fps: 0, bitDepth: 0 }),
+    { width: 1920, height: 1080, fps: 24, bitDepth: 8 }
+  );
+  assert.equal(facts.width, 1920, "a probe that printed nothing does not outrank a file that speaks");
+  assert.equal(facts.fps, 24);
+  assert.equal(facts.bitDepth, 8);
+  assert.deepEqual(facts.disagreements, []);
+});

package/services/container-index/avi.js

@@ -1,167 +0,0 @@
-/**
- * @file Keyframe index for AVI, read without downloading the file.
- *
- * AVI ends with an `idx1` chunk: one fixed-size entry per stream chunk, each
- * carrying a flags word whose keyframe bit says whether that chunk starts a
- * keyframe. Frame number times the video stream's frame duration gives the
- * time, so the index alone is enough — no media has to be read.
- *
- * `idx1` lives at the end of the file and the top-level chunk headers state
- * their sizes, so it is reached by stepping over headers (typically two hops:
- * `LIST hdrl`, `LIST movi`), not by scanning.
- *
- * Still relevant despite the format's age: older releases are largely XviD in
- * AVI, and those are exactly the files that get copied rather than re-encoded.
- */
-
-const HEADER_BYTES = 8;
-const PROBE_BYTES = 4096;
-// Keyframe flag in an idx1 entry's flags word (AVIIF_KEYFRAME).
-const KEYFRAME_FLAG = 0x10;
-const IDX1_ENTRY_BYTES = 16;
-// Cap on the idx1 read. One entry per chunk, 16 bytes each — a long film runs
-// to a few MB; beyond this is not a normal index.
-const MAX_IDX1_BYTES = 64 * 1024 * 1024;
-
-/**
- * Whether this looks like AVI: a RIFF container whose form type is `AVI `.
- *
- * @param {Buffer} head
- * @returns {boolean}
- */
-export function isAvi(head) {
-  return (
-    head.length >= 12 &&
-    head.toString("latin1", 0, 4) === "RIFF" &&
-    head.toString("latin1", 8, 12) === "AVI "
-  );
-}
-
-/**
- * Microseconds per frame and the video stream's chunk id prefix, from the main
- * header. Both live in the `hdrl` list near the file start.
- *
- * @param {Buffer} head
- * @returns {{ microsecondsPerFrame: number } | null}
- */
-function readMainHeader(head) {
-  // Top-level: "RIFF" size "AVI " then chunks. `avih` sits inside `LIST hdrl`.
-  let offset = 12;
-  while (offset + HEADER_BYTES <= head.length) {
-    const id = head.toString("latin1", offset, offset + 4);
-    const size = head.readUInt32LE(offset + 4);
-    if (size <= 0) {
-      return null;
-    }
-    if (id === "LIST") {
-      // Descend: list type follows the header, then its own chunks.
-      const listType = head.toString("latin1", offset + 8, offset + 12);
-      if (listType === "hdrl") {
-        let inner = offset + 12;
-        while (inner + HEADER_BYTES <= Math.min(head.length, offset + 8 + size)) {
-          const innerId = head.toString("latin1", inner, inner + 4);
-          const innerSize = head.readUInt32LE(inner + 4);
-          if (innerSize <= 0) {
-            return null;
-          }
-          if (innerId === "avih" && inner + 8 + 4 <= head.length) {
-            return { microsecondsPerFrame: head.readUInt32LE(inner + 8) };
-          }
-          inner += HEADER_BYTES + innerSize + (innerSize % 2);
-        }
-      }
-      offset += HEADER_BYTES + 4 + (size - 4) + ((size - 4) % 2);
-      continue;
-    }
-    offset += HEADER_BYTES + size + (size % 2);
-  }
-  return null;
-}
-
-/**
- * Step over top-level chunks to find `idx1`.
- *
- * @param {(start: number, end: number) => Promise<Buffer | null>} readRange
- * @param {number} fileSize
- * @returns {Promise<{ offset: number, size: number } | null>}
- */
-async function findIdx1(readRange, fileSize) {
-  let offset = 12; // Past "RIFF" size "AVI ".
-  while (offset + HEADER_BYTES < fileSize) {
-    const probe = await readRange(offset, Math.min(fileSize - 1, offset + HEADER_BYTES - 1));
-    if (!probe || probe.length < HEADER_BYTES) {
-      return null;
-    }
-    const id = probe.toString("latin1", 0, 4);
-    const size = probe.readUInt32LE(4);
-    if (size <= 0) {
-      return null;
-    }
-    if (id === "idx1") {
-      return { offset: offset + HEADER_BYTES, size };
-    }
-    // Chunks are word-aligned; a LIST carries its type inside the payload, so
-    // the same size arithmetic covers both cases.
-    offset += HEADER_BYTES + size + (size % 2);
-  }
-  return null;
-}
-
-/**
- * Read the keyframe times of an AVI file.
- *
- * @param {(start: number, end: number) => Promise<Buffer | null>} readRange
- * @param {number} fileSize
- * @returns {Promise<number[] | null>} Ascending seconds, or null when the file
- *   has no `idx1` (OpenDML-only index, interrupted write, damaged upload).
- */
-export async function readAviKeyframeTimes(readRange, fileSize) {
-  const head = await readRange(0, Math.min(PROBE_BYTES - 1, fileSize - 1));
-  if (!head || !isAvi(head)) {
-    return null;
-  }
-  const mainHeader = readMainHeader(head);
-  if (!mainHeader || !mainHeader.microsecondsPerFrame) {
-    return null;
-  }
-
-  const idx1 = await findIdx1(readRange, fileSize);
-  if (!idx1 || idx1.size > MAX_IDX1_BYTES) {
-    return null;
-  }
-
-  const table = await readRange(idx1.offset, Math.min(fileSize - 1, idx1.offset + idx1.size - 1));
-  if (!table || table.length < IDX1_ENTRY_BYTES) {
-    return null;
-  }
-
-  const secondsPerFrame = mainHeader.microsecondsPerFrame / 1e6;
-  const times = [];
-  let videoFrame = 0;
-  for (let at = 0; at + IDX1_ENTRY_BYTES <= table.length; at += IDX1_ENTRY_BYTES) {
-    const chunkId = table.toString("latin1", at, at + 4);
-    // Video chunks are "##db" (uncompressed) or "##dc" (compressed); audio is
-    // "##wb" and must not advance the frame counter.
-    const isVideo = chunkId.endsWith("db") || chunkId.endsWith("dc");
-    if (!isVideo) {
-      continue;
-    }
-    const flags = table.readUInt32LE(at + 4);
-    if ((flags & KEYFRAME_FLAG) !== 0) {
-      times.push(videoFrame * secondsPerFrame);
-    }
-    videoFrame += 1;
-  }
-  if (times.length === 0) {
-    return null;
-  }
-  // AVI names a keyframe by its FRAME NUMBER, and the time above is that number
-  // multiplied by the frame duration the header declares. The frames are the
-  // right ones — measured 2026-08-21 against the files themselves, 1196 index
-  // entries against 1196 real keyframes and 901 against 901, exactly — but the
-  // names are 10-44 ms away from the presentation times the demuxer computes,
-  // always under one frame. So the caller is told how far a time here may be
-  // from the instant it refers to, and can ask for a seek late enough that it
-  // still lands on the frame rather than on the one before it.
-  return { times, tolerance: secondsPerFrame };
-}