Changeset 8ed36de in Klonkt for src/services/music/index.js

Timestamp:

08/21/2026 04:57:20 PM (3 weeks ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

b0ccd0d

Parents:

4e5fe29

Message:

Een verwijderde track kondigt zichzelf aan, net als een post

Een post stuurt bij verwijderen een Delete(Tombstone) naar zijn volgers; een
track deed dat niet. De rij ging weg, het Audio-object gaf 404, en elke server
die hem had geindexeerd bleef ernaar wijzen. Op de hub stond het er vandaag als
een kaartje met een dode link, en het viel alleen op doordat robo het zag.

Sinds shaer-0nh is een track een eersterangs Audio-object met een eigen id, dus
hij hoort ook een eigen afmelding te krijgen.

• deliverObjectDelete(site, objectId) is de gedeelde romp; deliverDelete en het nieuwe deliverTrackDelete leunen er allebei op. Het object-id komt van de aanroeper, want bij verwijderen is de rij vaak al weg.
• trackUri() staat nu op EEN plek in music/index.js. Het formaat werd eerder alleen door de bouwkant gekend, en een tweede plek die het opnieuw in elkaar zet is precies hoe je een Delete stuurt die de ontvanger niet herkent.
• Beide verwijderroutes in admin-audio melden af: de losse verwijdering en de opruiming van wezen. Een wees is voor ONS een track zonder bestand, maar de buitenwereld heeft een gewoon Audio-object opgeslagen.

Twee tests door de echte route. Een testhaak op deliver() bestaat niet, dus de
proef kijkt in ap_delivery: de volger krijgt een inbox op een dichte poort, de
directe poging faalt, en de activiteit hoort in de wachtrij te belanden. Die
tabel was in het echte geval leeg, dus dat is precies het bewijs. Controleproef
gedraaid: zonder de aankondiging valt de test om.

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

File:

• src/services/music/index.js (modified) (2 diffs)

src/services/music/index.js

@@ -165 +165 @@
 }
 
+/**
+ * De identiteit van een track op de draad.
+ *
+ * Staat apart omdat hij op TWEE momenten nodig is die ver uit elkaar liggen:
+ * bij het bouwen van het Audio-object, en bij het verwijderen ervan, wanneer de
+ * rij al weg is en er dus niets meer te bouwen valt. Toen dit nog inline stond,
+ * kende alleen de bouwkant de vorm en ging er bij verwijderen geen Delete uit --
+ * elke server die de track had geindexeerd hield hem voor altijd (21-8).
+ */
+export function trackUri(base, site, id) {
+  return `${actorId(base, site.slug)}/tracks/${encodeURIComponent(id)}`;
+}
+
 export function buildTrackAudio(base, site, r, opts = {}) {
   const abs = (u) => !u ? null : (/^https?:/i.test(u) ? u : `${base}${u.startsWith('/') ? '' : '/'}${u}`);
@@ -187 +200 @@
   const a = {
     ...(opts.standalone ? { '@context': AP_CONTEXT } : {}),
-    id: `${actorId(base, site.slug)}/tracks/${encodeURIComponent(r.id)}`,
+    id: trackUri(base, site, r.id),
     type: 'Audio',
     name: r.title || 'Audio',