@torrent-tv/proxy 2.9.117 → 2.9.119

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## 2.9.119
+
+- **Fix**: The progress report says the height the viewer is actually watching, not only the one an encoder is producing. It was zero whenever the video was copied — which is most sessions — so the quality menu read a bare "Auto" in exactly the case it was built to explain. Copying reports the source height, re-encoding reports the rung the proxy has settled on.
+
+## 2.9.118
+
+- **New**: Suspending the encoder says what the decision was taken on — where the viewer is, how far the unbroken run of segments reaches from there, and how many segment files the session directory holds. Suspending stops the only thing that reads the input, so a wrong reading here stops the download as well: measured 2026-08-06, the log announced "135s ahead of the viewer" while three segments totalling 31 s lay on disk, and neither figure could be checked against the other because the line carried no evidence. The directory keeps the segments of every run a session has had, so which of them were counted is the whole question.
+
 ## 2.9.117
 
 - **Fix**: A seek backwards no longer kills playback. How far the encoder is ahead of the viewer was measured as the highest segment number lying in the session's directory, which equals the look-ahead only while a viewer moves forward through one run. Measured 2026-08-06: a seek forward left segments 662-665 on disk, the viewer seeked BACK to 646, and the limiter compared 6950 s of output against a viewer at 6700 s, called it "250s ahead" and suspended the new run **136 ms after it started**, before it had produced anything. With the encoder stopped nothing read the input, so no pieces were requested — `0 selection(s)` with 33 peers connected — and segment 646 was never made; the viewer sat on a spinner for four minutes while a stopped ffmpeg was held back for being too far ahead. The measure is now the unbroken run of segments starting where the viewer is, and a viewer whose own segment is missing is not "zero ahead" but waiting, which resumes the encoder instead of pausing it. Covered by tests, including the field case.

package/package.json

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

package/services/hls-session-manager.js

@@ -1909,7 +1909,8 @@ export class HlsSessionManager {
     // — `0 selection(s)` with 33 peers connected — and segment 646 was never
     // made. Segments beyond a hole are not look-ahead: the viewer cannot reach
     // them without the hole being filled first.
-    const aheadSeconds = this.#contiguousAheadSeconds(session, viewerSegment);
+    const reading = this.#contiguousAheadSeconds(session, viewerSegment);
+    const aheadSeconds = reading === null ? null : reading.seconds;
     if (aheadSeconds === null) {
       // The segment the viewer needs does not exist. Whatever else is on disk,
       // this encoder has work to do right now.
@@ -1942,7 +1943,19 @@ export class HlsSessionManager {
     }
 
     if (!session.encoderPaused && aheadSeconds > LOOKAHEAD_PAUSE_SECONDS) {
-      this.#pauseEncoder(session, `${Math.round(aheadSeconds)}s ahead of the viewer`);
+      // The decision names what it was taken on. Suspending the encoder stops
+      // the only thing that reads the input, so a wrong reading here stops the
+      // download too — measured 2026-08-06: the log said "135s ahead" while
+      // three segments totalling 31 s lay on disk, and neither figure could be
+      // checked against the other because the line carried no evidence. The
+      // directory holds segments from every run this session has had, so which
+      // ones were counted is the whole question.
+      this.#pauseEncoder(
+        session,
+        `${Math.round(aheadSeconds)}s ahead of the viewer ` +
+          `(viewer at #${viewerSegment}, unbroken through #${reading.lastCovered}, ` +
+          `${reading.total} segment file(s) present)`
+      );
     } else if (session.encoderPaused && aheadSeconds <= LOOKAHEAD_RESUME_SECONDS) {
       this.#resumeEncoder(session, `${Math.round(aheadSeconds)}s ahead of the viewer`);
     }
@@ -1957,7 +1970,7 @@ export class HlsSessionManager {
    *
    * @param {HlsSession} session
    * @param {number} viewerSegment
-   * @returns {number | null}
+   * @returns {{ seconds: number, lastCovered: number, total: number } | null}
    */
   #contiguousAheadSeconds(session, viewerSegment) {
     let present;
@@ -1984,7 +1997,7 @@ export class HlsSessionManager {
     if (!Number.isFinite(from) || !Number.isFinite(to)) {
       return null;
     }
-    return Math.max(0, to - from);
+    return { seconds: Math.max(0, to - from), lastCovered, total: present.size };
   }
 
   /**
@@ -3481,13 +3494,16 @@ export class HlsSessionManager {
       segmentDurationSec: this.segmentDurationSec,
       speed: session.progress.speed,
       outputMbps,
-      // The height being produced RIGHT NOW. On automatic quality the proxy
-      // steps down a rung when the host cannot keep up or the viewer's link
-      // cannot carry the stream, and the menu said only "Auto" — so the one
-      // question it raises, what am I actually watching, had no answer
-      // anywhere. Zero when the video is copied, because then nothing is being
-      // chosen: the source's own height is what plays.
-      currentHeight: session.transcodeVideo ? (session.encodeHeight ?? 0) : 0,
+      // The height the viewer is WATCHING right now, which is what the menu
+      // has to say next to "Auto". When the video is re-encoded that is the
+      // rung the proxy has settled on — it steps down when the host cannot keep
+      // up or the link cannot carry the stream. When the video is COPIED it is
+      // the source's own height, and reporting zero there was simply wrong:
+      // most sessions copy the video, so the menu read a bare "Auto" almost
+      // always, which is exactly the question it was supposed to answer.
+      currentHeight: session.transcodeVideo
+        ? (session.encodeHeight ?? session.sourceHeight ?? 0)
+        : (session.sourceHeight ?? 0),
       updatedAt: session.progress.updatedAt,
       error: session.state === "failed" ? session.lastError : ""
     };