@torrent-tv/proxy 2.14.2 → 2.14.3

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 2.14.3
+
+- **Fix**: A separately published audio track begins where the PICTURE is, measured rather than guessed. The position this class keeps is the read head, and the viewer sits behind it by whatever the player has buffered — a figure the browser already reports with every link report, so the playhead is one subtraction away (less one segment of margin, since the report can be ten seconds old). 2.14.2 subtracted the whole look-ahead instead, which was safe but made the encoder produce up to two minutes of audio nobody would hear before reaching the part that was wanted. A report older than fifteen seconds is ignored — a viewer may have seeked since — and then the whole look-ahead is subtracted as before.
+- **Fix**: A request behind the encode run is acted on when the player ASKS AGAIN, not after three seconds of waiting. Repetition is the player saying it still needs that exact segment; a delay only says time has passed, and those three seconds were part of the twenty a track change cost. A scan is told apart by what else is being asked for — more than three distinct segments behind the run within two seconds is the player sweeping the playlist, and moving the encoder to one of them would be moving it to a number picked at random.
+- **Fix**: That scan count is taken over a two-second window rather than over the life of the run. Accumulated, it would have crossed the threshold on any long session and disabled the repair for good — silently, since nothing about a repair that never fires is logged.
+
 ## 2.14.2
 
 - **Fix**: A separately published audio track starts BEHIND the picture's read head, and a request behind its run is answered as before. Two mistakes compounded in 2.14.1 and left the viewer on a spinner that never ended. The position this class keeps is written by the segments a session serves — the READ head — while the viewer's picture sits behind it by everything they have buffered, so the track was started AHEAD of them: field 2026-08-15, the run placed at segment #16 while the player asked for #10. On top of that, 2.14.1 had begun answering such a request "not found" at once instead of holding it, which turned a condition the encoder used to correct in twenty seconds into a permanent refusal: hls.js retried #10 for a minute and a half, raised a fatal network error, recovered, and retried it again. The prompt refusal is withdrawn — it was written for a probe and met a real request — and the track now starts a whole look-ahead behind the read head, which is exactly how far apart the two can be. The price is audio the player already holds: at ten to twenty times realtime and 75 KB a piece, a second or two of work.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@torrent-tv/proxy",
-  "version": "2.14.2",
+  "version": "2.14.3",
   "description": "Torrent proxy client that exposes webseed-like HTTP stream endpoint.",
   "license": "GPL-3.0-or-later",
   "publishConfig": {

package/services/hls-session-manager.js

@@ -354,6 +354,10 @@ const READ_WINDOW_SECONDS = 30;
 const READ_WINDOW_MIN_BYTES = 16 * 1024 * 1024;
 const READ_WINDOW_MAX_BYTES = 96 * 1024 * 1024;
 const LOOKAHEAD_PAUSE_SECONDS = 120;
+// How old a viewer's link report may be and still describe where they are. It
+// is sent every 10 s, and a seek in between moves them somewhere this cannot
+// predict — so anything older is treated as no report at all.
+const NET_REPORT_FRESH_MS = 15_000;
 const LOOKAHEAD_RESUME_SECONDS = 60;
 // Seek debounce. A far (out-of-window) segment request is a server-side seek.
 // Rather than restart ffmpeg on the first one, wait a short quiet period:
@@ -389,7 +393,18 @@ const SEEK_SETTLE_MS = 300;
 // seek settles on its own — the seek is what should move the encoder — and
 // short enough that a session cannot sit on an unanswerable request, which
 // measured two minutes forty-one before a viewer gave up.
-const BEHIND_HEAD_REPAIR_MS = 3_000;
+// A request behind the run is acted on once it has been REPEATED, not once it
+// has waited: repetition is the player saying it still needs this exact
+// segment, while a delay only says time has passed. The floor below stays as a
+// last guard against acting on a single stray poll.
+const BEHIND_HEAD_REPAIR_MIN_ASKS = 2;
+// More distinct indices than this behind the head at once is the player
+// scanning the playlist rather than waiting for a frame.
+const BEHIND_HEAD_SCAN_INDICES = 3;
+// The window the count above is taken over. A player's scan lands inside half a
+// second (field log 2026-08-02); a viewer waiting asks every few seconds.
+const BEHIND_HEAD_SCAN_WINDOW_MS = 2_000;
+const BEHIND_HEAD_REPAIR_MS = 400;
 // How far behind the run a request may be and still be treated as the encoder
 // standing in the wrong place rather than as a player scanning the playlist. A
 // misplaced run is out by at most the buffer the player was holding — measured
@@ -2886,6 +2901,7 @@ export class HlsSessionManager {
     // for the seek that should move the encoder. It also stops the map growing
     // for the life of a session.
     session.firstWantedAt = new Map();
+    session.behindHeadAsks = new Map();
     const generation = ++session.encodeRunGeneration;
     const previousFfmpeg = session.ffmpeg;
     // A suspended process does not act on SIGTERM until it is continued, so the
@@ -3461,6 +3477,16 @@ export class HlsSessionManager {
     if (!session.firstWantedAt.has(index)) {
       session.firstWantedAt.set(index, Date.now());
     }
+    // How often each index behind the run has been asked for, and how many
+    // distinct ones there are. The repair reads both: one index asked twice is
+    // a viewer waiting, a dozen asked once each is the player scanning. Kept
+    // only for what is behind the head — everything ahead is ordinary
+    // read-ahead — and cleared with each run, like the record above.
+    if (index < session.encodeStartIndex) {
+      session.behindHeadAsks ??= new Map();
+      const asked = session.behindHeadAsks.get(index);
+      session.behindHeadAsks.set(index, { count: (asked?.count ?? 0) + 1, at: Date.now() });
+    }
     if (!session || session.state === "disposed" || index < 0) {
       return;
     }
@@ -3580,6 +3606,37 @@ export class HlsSessionManager {
     if (session.seekSettleTimer != null) {
       return;
     }
+    // What separates a request the viewer is waiting for from the player
+    // scanning the playlist is not TIME but what else it is asking for. On a
+    // seek hls.js fires dozens of DIFFERENT indices within half a second (field
+    // log: #178, #681, #725, #807, #74, #245, #387) and abandons them all; a
+    // viewer waiting for audio asks for the SAME one, over and over, because it
+    // is the only thing that will let playback continue.
+    //
+    // So: this index has been asked for at least twice, and it is the only
+    // thing behind the head being asked for. Both are facts about the traffic,
+    // available at once, where a delay is a guess about it — and it was three
+    // seconds of the twenty a track change cost on 2026-08-15.
+    const asked = session.behindHeadAsks?.get(index)?.count ?? 0;
+    if (asked < BEHIND_HEAD_REPAIR_MIN_ASKS) {
+      return;
+    }
+    // Counted over a WINDOW, not over the run: a scan is many indices at once,
+    // while the same map left to accumulate would eventually hold every
+    // behind-head request a long run ever saw and switch the repair off for
+    // good.
+    const scanSince = Date.now() - BEHIND_HEAD_SCAN_WINDOW_MS;
+    let distinctBehind = 0;
+    for (const record of session.behindHeadAsks?.values() ?? []) {
+      if (record.at >= scanSince) {
+        distinctBehind += 1;
+      }
+    }
+    if (distinctBehind > BEHIND_HEAD_SCAN_INDICES) {
+      // A scan, not a wait. Moving the encoder to one of these is moving it to
+      // a number the player picked at random.
+      return;
+    }
     const wantedAt = session.firstWantedAt?.get(index);
     if (!Number.isFinite(wantedAt) || Date.now() - wantedAt < BEHIND_HEAD_REPAIR_MS) {
       return;
@@ -4751,6 +4808,37 @@ export class HlsSessionManager {
     return this.#viewerPositionOf(this.#activeVariant(base));
   }
 
+  /**
+   * Where to start a separately published audio track, in seconds.
+   *
+   * The player, on changing track, discards the audio it holds and refills from
+   * the PICTURE onwards — so that is where the encoder has to begin. What this
+   * class knows directly is the read head, which runs ahead of the picture by
+   * the player's own buffer; the browser reports that buffer with every link
+   * report, so the picture is the one subtraction below.
+   *
+   * One segment of margin, because the report is up to ten seconds old and the
+   * picture has moved on since — a run that begins a little early costs a
+   * segment of audio nobody plays, while one that begins a little late is
+   * behind the viewer and can only be fixed by restarting it.
+   *
+   * With no fresh report, the whole look-ahead is subtracted instead: it is the
+   * furthest the two can be apart, so it cannot leave the run ahead of them.
+   *
+   * @param {HlsSession} base
+   * @returns {number}
+   */
+  #audioStartSecondsFor(base) {
+    const watching = this.#activeVariant(base);
+    const readHead = this.#viewerPositionOf(watching);
+    const report = watching.netReport;
+    const reportAge = Number.isFinite(report?.at) ? Date.now() - report.at : Number.POSITIVE_INFINITY;
+    const buffered = reportAge <= NET_REPORT_FRESH_MS && Number.isFinite(report?.bufferedAheadSec)
+      ? report.bufferedAheadSec
+      : LOOKAHEAD_PAUSE_SECONDS;
+    return Math.max(0, readHead - buffered - this.segmentDurationSec);
+  }
+
   #viewerPositionOf(session) {
     if (Number.isFinite(session.viewerPositionSeconds) && session.viewerPositionSeconds > 0) {
       return session.viewerPositionSeconds;
@@ -5334,26 +5422,23 @@ export class HlsSessionManager {
       // while the player is asking for segment #537 — and the audio would begin
       // at zero and never catch up, since nothing treats a far request as a
       // seek. The accessor falls back to the last segment actually requested.
-      // Behind where the picture has been READ to, by the whole look-ahead.
+      // Where the PICTURE is, not where it has been read to.
       //
       // The position this class keeps is written by the segments a session
-      // serves, so it is the READ head, and the player's picture sits behind it
-      // by everything it has buffered — up to the look-ahead cap. Starting the
-      // audio at the read head therefore starts it AHEAD of the viewer, and
-      // every request they then make is behind a run that only moves forward.
-      // Field 2026-08-15: the run was placed at #16, the player asked for #10,
-      // and the audio never arrived until the encoder was dragged back.
+      // serves, so it is the READ head, and the viewer's picture sits behind it
+      // by everything the player has buffered. Started at the read head, the
+      // audio run begins AHEAD of the viewer, and every request they then make
+      // is behind a run that only moves forward — field 2026-08-15, placed at
+      // #16 while the player asked for #10, and the audio arrived only after
+      // the encoder was dragged back.
       //
-      // The cap is exactly how far apart the two can be, so subtracting it
-      // cannot leave the run ahead of the viewer. The price is audio the player
-      // already has — for a track being encoded at ten to twenty times realtime
-      // and served in 75 KB pieces, that is a second or two of work.
-      startPositionSeconds: Math.max(
-        0,
-        Math.floor(
-          (this.#viewerPositionOf(this.#activeVariant(base)) - LOOKAHEAD_PAUSE_SECONDS) / 10
-        ) * 10
-      ),
+      // The distance is measured, not assumed: the browser reports how many
+      // seconds it holds ahead of the picture with every link report, so the
+      // playhead is one subtraction away. A stale report is no use — a viewer
+      // who seeked since then is somewhere else entirely — so an old one is
+      // ignored and the whole look-ahead is subtracted instead, which cannot
+      // leave the run ahead of them.
+      startPositionSeconds: this.#audioStartSecondsFor(base),
       segmentFormatId: base.segmentFormat.id,
       // Cut where the picture is cut. Two streams meant to be played together
       // have to be divided at the same times, and the grid is the base's — the

package/test/behind-head-repair.test.js

@@ -71,16 +71,23 @@ async function managerWithRunAhead() {
   return { manager, session, dirPath, restarts };
 }
 
-test("a request behind the run is repaired once it has waited", async (t) => {
+test("a request behind the run is repaired once the player asks again", async (t) => {
   const { manager, session, dirPath } = await managerWithRunAhead();
   t.after(async () => {
     await manager.disposeAll();
     await rm(dirPath, { recursive: true, force: true });
   });
-  // Asked for four seconds ago and still unanswerable.
-  session.firstWantedAt.set(WANTED, Date.now() - 4000);
+  // Asked for a moment ago and still unanswerable. What decides is that the
+  // player comes BACK for the same segment: it is the only one it wants, and no
+  // amount of waiting could say that as clearly.
+  session.firstWantedAt.set(WANTED, Date.now() - 1000);
+  const name = fmp4Format.segmentFileName(WANTED);
 
-  await manager.getFileStream(SESSION_ID, fmp4Format.segmentFileName(WANTED), { requestSeq: 1 });
+  await manager.getFileStream(SESSION_ID, name, { requestSeq: 1 });
+
+  assert.equal(session.seekTarget, null, "one poll is not yet evidence");
+
+  await manager.getFileStream(SESSION_ID, name, { requestSeq: 2 });
 
   assert.equal(
     session.seekTarget,
@@ -89,6 +96,33 @@ test("a request behind the run is repaired once it has waited", async (t) => {
   );
 });
 
+test("a burst of different segments behind the run is a scan, and moves nothing", async (t) => {
+  const { manager, session, dirPath } = await managerWithRunAhead();
+  t.after(async () => {
+    await manager.disposeAll();
+    await rm(dirPath, { recursive: true, force: true });
+  });
+  // What hls.js does on a seek: dozens of indices within half a second, each
+  // abandoned. Field log 2026-08-02: #178, #681, #725, #807, #74, #245, #387.
+  const scanned = [WANTED, WANTED - 40, WANTED - 120, WANTED - 200, WANTED - 300];
+  for (const index of scanned) {
+    session.firstWantedAt.set(index, Date.now() - 1000);
+  }
+  // Interleaved, as they arrive on the wire: the player opens them together
+  // rather than finishing with one before opening the next.
+  for (const seq of [1, 2]) {
+    for (const index of scanned) {
+      await manager.getFileStream(SESSION_ID, fmp4Format.segmentFileName(index), { requestSeq: seq });
+    }
+  }
+
+  assert.equal(
+    session.seekTarget,
+    null,
+    "moving the encoder to one of these would be moving it to a number the player picked at random"
+  );
+});
+
 test("a request behind the run is left alone at first", async (t) => {
   const { manager, session, dirPath } = await managerWithRunAhead();
   t.after(async () => {

package/test/quality-variants.test.js

@@ -638,19 +638,18 @@ test("a rung served by copy stays offered while a re-encoded rung is on screen",
   );
 });
 
-test("a separately published audio track starts behind the picture's read head", async (t) => {
+test("a separately published audio track starts where the picture is, from the reported buffer", async (t) => {
   const { manager, base, dirPath } = await managerWithBase();
   t.after(async () => {
     await manager.disposeAll();
     await rm(dirPath, { recursive: true, force: true });
   });
-  // What broke playback on 2026-08-15: the position this class keeps is where
-  // segments have been SERVED to, and the viewer's picture is behind it by
-  // everything they have buffered. Started at the read head, the audio run sat
-  // ahead of the viewer and every request they made was behind a run that only
-  // moves forward.
-  base.viewerPositionSeconds = 140;
   base.audioSeparate = true;
+  // Served up to 140 s, and the browser says it holds 40 s ahead of the
+  // picture — so the viewer is at 100 s, and that, less a segment of margin,
+  // is where the track has to begin.
+  base.viewerPositionSeconds = 140;
+  base.netReport = { linkMbps: 20, bufferedAheadSec: 40, at: Date.now() };
   manager.getCachedAudioTracks = () => [
     { index: 0, language: "rus", title: "", isDefault: true },
     { index: 1, language: "eng", title: "", isDefault: false }
@@ -663,13 +662,44 @@ test("a separately published audio track starts behind the picture's read head",
     return { sessionId: VARIANT_ID, session: rendition };
   };
 
-  // A segment, not the playlist: the playlist is answered from the base and
-  // deliberately starts no encoder.
   await manager.resolveAudioRenditionFile(BASE_ID, 1, "segment-00010.mp4");
 
   assert.equal(created.length, 1, "the track's own session was made");
-  assert.ok(
-    created[0].startPositionSeconds <= 20,
-    `started behind the picture, not at the read head — got ${created[0].startPositionSeconds}s for a read head of 140s`
+  assert.equal(
+    created[0].startPositionSeconds,
+    96,
+    "140 s served, less the 40 s the player holds, less one segment of margin"
+  );
+});
+
+test("a stale buffer report is not used to place an audio track", async (t) => {
+  const { manager, base, dirPath } = await managerWithBase();
+  t.after(async () => {
+    await manager.disposeAll();
+    await rm(dirPath, { recursive: true, force: true });
+  });
+  base.audioSeparate = true;
+  base.viewerPositionSeconds = 300;
+  // Sent a minute ago: the viewer may have seeked anywhere since, so it says
+  // nothing about where they are now.
+  base.netReport = { linkMbps: 20, bufferedAheadSec: 5, at: Date.now() - 60_000 };
+  manager.getCachedAudioTracks = () => [
+    { index: 0, language: "rus", title: "", isDefault: true },
+    { index: 1, language: "eng", title: "", isDefault: false }
+  ];
+  const created = [];
+  manager.createOrGetSession = async (params) => {
+    created.push(params);
+    const rendition = fakeSession({ id: VARIANT_ID, encodeHeight: 0, dirPath });
+    rendition.audioOnly = true;
+    return { sessionId: VARIANT_ID, session: rendition };
+  };
+
+  await manager.resolveAudioRenditionFile(BASE_ID, 1, "segment-00010.mp4");
+
+  assert.equal(
+    created[0].startPositionSeconds,
+    176,
+    "the whole look-ahead is subtracted instead — it cannot leave the run ahead of the viewer"
   );
 });