Changeset 4fbe2c1 in Klonkt for src/assets/js/audio-player.js

Timestamp:

08/22/2026 10:35:33 AM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

c0e463b

Parents:

8bb3b09

Message:

Een bandje loopt niet rond: het stopt aan het eind, en aan het begin

Robins eis (22-8). Drie omlopen zaten in de weg, en de stilste was de keten:
na het laatste nummer hing die nummer een er weer achter, en omdat de wachtrij
EEN doorlopende tijdlijn is merk je dat niet eens als een trackwissel -- de
band gaat gewoon door alsof er meer op stond.

• De keten haakt in bandmodus niets meer aan voorbij het laatste nummer.
• next() pauzeert daar in plaats van om te lopen. ended roept next() aan, dus dit is wat de band echt laat stoppen.
• prev() loopt aan het begin niet naar achteren om: terugspoelen voorbij de kop levert de kop op, niet het laatste nummer.
• Vooruit prefetchen stopt ook aan het eind.

De drie modulo's blijven staan voor een gewone wachtrij; die hoort wel te
blijven rondlopen. Daar staat een test op, anders sneuvelt dat gedrag stil.

TERUGSPOELEN IS NU ECHT SPOELEN, ook over de nummergrenzen. Op de MSE-motor
ging dat al vanzelf (de band is een tijdlijn). Op de blob-motor landde het in
de KOP van het vorige nummer, en dan kom je nooit ergens in het midden uit; nu
zet hij pendingSeek zodat je aan het EIND van dat nummer uitkomt.

EN EEN FOUT DIE IK ER ZELF MEE INTRODUCEERDE, gevonden door het te draaien en
niet door te lezen: bij een MediaSource vuurt ended pas als de stream dicht
is, en die ging alleen dicht bij een wachtrij van een. Sinds de keten aan het
eind niets meer aanhaakt bleef de band dus hangen op de laatste seconde --
teller stil, isPlaying() waar, spoelen draaiend. Gemeten: 125,6 van 125,7.
De stream sluit nu ook aan het eind van een bandje.

Spoelen stopt vanzelf bij de kop en de staart: beweegt de teller twee rondes
niet, dan is er geen band meer. Anders bleven de spoelen draaien terwijl er
niets gebeurde.

Nagemeten op dev: band loopt af -> laatste nummer, gestopt, teller op 125,7 van
125,7, spoelen stil, knop terug op Afspelen. next() op het laatste nummer
blijft staan; prev() op het eerste spoelt naar de kop.

audio-player v37, MOD_V 54.

Co-Authored-By: Claude Opus 5 <noreply@…>

File:

• src/assets/js/audio-player.js (modified) (5 diffs)

src/assets/js/audio-player.js

@@ -279 +279 @@
   function preloadNext() {
     if (queue.length < 2) return;
+    if (tapeMode && currentIndex >= queue.length - 1) return;   // einde band
     const ni = (currentIndex + 1) % queue.length;
     const t = queue[ni];
@@ -361 +362 @@
     const start = chain.length ? chain[chain.length - 1].end : (buffered.length ? buffered.start(0) : 0);
     chain.push({ qIndex, start, end: chainEnd });
-    if (queue.length === 1 && ms && ms.readyState === 'open') {
+    // DE STREAM MOET DICHT ALS ER NIETS MEER KOMT, anders vuurt `ended` nooit.
+    // Bij een wachtrij van een was dat al zo. Een BANDJE heeft nu hetzelfde
+    // nodig: sinds hij niet meer rondloopt haakt de keten na het laatste nummer
+    // niets meer aan, en dan bleef de band aan het eind hangen -- de teller
+    // stilstaand op de laatste seconde, isPlaying() waar, en de spoelen
+    // draaiend. Gemeten op dev (22-8): 125,6 van 125,7 en daar bleef hij.
+    const laatsteVanDeBand = tapeMode && qIndex >= queue.length - 1;
+    if ((queue.length === 1 || laatsteVanDeBand) && ms && ms.readyState === 'open') {
       // Single-track queue: close the stream so `ended` fires (which replays
-      // it, matching the old engine's behaviour).
+      // it, matching the old engine's behaviour). Bij een bandje stopt `ended`
+      // hem juist, want next() pauzeert daar aan het eind.
       try { ms.endOfStream(); } catch (e) {}
     }
@@ -374 +383 @@
     const seg = currentSegment();
     if (!seg || chain.length - 1 - chain.indexOf(seg) >= 1) return;  // already one ahead
-    const nextIdx = (chain[chain.length - 1].qIndex + 1) % queue.length;
+    // EEN BANDJE LOOPT NIET ROND (Robins eis, 22-8). Deze modulo is precies wat
+    // een cassette eindeloos maakte: na het laatste nummer hing hij nummer een
+    // er weer achter, en omdat het een doorlopende keten is merk je dat niet
+    // eens als een trackwissel -- de band gaat gewoon door.
+    const volgendeInRij = chain[chain.length - 1].qIndex + 1;
+    if (tapeMode && volgendeInRij > queue.length - 1) return;   // einde band
+    const nextIdx = volgendeInRij % queue.length;
     const t = queue[nextIdx];
     if (!t || !t.url) return;
@@ -642 +657 @@
   function next() {
     if (!queue.length) return;
+    // Aan het eind van een bandje: stoppen. `ended` roept deze functie aan, dus
+    // zonder deze tak begint de band na het laatste nummer weer vooraan.
+    if (tapeMode && currentIndex >= queue.length - 1) { pause(); return; }
     loadTrack((currentIndex + 1) % queue.length, true);
   }
   function prev() {
     if (!queue.length) return;
+    // En aan het begin ook niet omlopen. Terugspoelen voorbij het begin levert
+    // de kop van de band op, niet het laatste nummer.
+    if (tapeMode && currentIndex === 0) {
+      try { audio.currentTime = 0; } catch (e) { /* nog niets geladen */ }
+      return;
+    }
     loadTrack(currentIndex === 0 ? queue.length - 1 : currentIndex - 1, true);
   }
@@ -1157 +1181 @@
     const duur = audio.duration || 0;
     const nu = audio.currentTime || 0;
-    if (duur && nu + d >= duur) { next(); return; }
-    if (nu + d < 0) { prev(); return; }
+    if (duur && nu + d >= duur) {
+      // Vooruit voorbij het eind. In bandmodus stopt next() aan het eind van de
+      // band; daarbuiten loopt hij door naar het volgende nummer.
+      next();
+      return;
+    }
+    if (nu + d < 0) {
+      // TERUGSPOELEN VOORBIJ HET BEGIN moet in het vorige nummer landen aan het
+      // EIND, niet aan het begin -- dat is wat terugspoelen doet. Zonder dit
+      // sprong je bij elke druk naar de kop van het vorige nummer en kwam je
+      // nooit ergens in het midden uit.
+      if (!(tapeMode && currentIndex === 0)) pendingSeek = Number.MAX_SAFE_INTEGER;
+      prev();
+      return;
+    }
     try { audio.currentTime = Math.max(0, nu + d); } catch (e) {}
   }