Changeset 9eb6e1b in Klonkt for src/routes/search.js

Timestamp:

09/30/2026 06:41:45 AM (2 hours ago)

Author:

Robin <roboburr@…>

Branches:

main

Children:

86cfa1a

Parents:

3ceaef6

Message:

Een fediverse-post ziet eruit als een post, ook in de zoekbalk

Plak je een link in de zoekbalk, dan laadt de uitklaplijst nu de post zelf
in plaats van te zoeken naar de tekst van de link. Dezelfde kaart staat op de
zoekpagina en op /authorize_interaction, dus één render op drie plekken.

DE KAART. Wat eromheen stond is teruggebracht tot een kop met gezicht, de
inhoud, en één rij knoppen: Reageer, Like, Boost en het origineel. Weg zijn de
zin "je reageert op", de aparte knop naar het origineel, de twee grote pillen,
"of reageer:" en het antwoordvenster dat altijd openstond. Antwoorden klapt
open als je erom vraagt; melden staat onderaan en niet in de weg.

Foto's staan in een raster met vaste rijhoogtes. Met verhoudingen groeide het
mee met de breedte, en werd op een desktop de eerste van drie foto's zo hoog
dat de knoppen onder de vouw van de uitklaplijst verdwenen.

EEN INHOUDSWAARSCHUWING WORDT GERESPECTEERD. De oude render toonde alles open,
ook wat de schrijver achter een waarschuwing had gezet.

MOBIEL. Elke knop minstens 44 punten. Op een smal scherm vallen de woorden weg
en blijven de tekens staan, gelijk verdeeld; de woorden blijven bestaan voor
wie het scherm voorgelezen krijgt. Nagemeten op 375 pixels: geen horizontale
overloop, kleinste tikdoel 44, de knoppen in beeld zonder te scrollen.

DE PREVIEW IS COMPACT. Waarderen en boosten kan ter plekke; antwoorden en
melden gaan naar de volle pagina, met ?reply=1 of ?report=1 zodat dat deel
daar al openstaat. Een editor in een uitklaplijst is op een telefoon geen plek
om te schrijven. Zonder JS doet dezelfde link hetzelfde.

/search/remote levert het fragment, achter dezelfde grens als de volle
pagina: alleen wie de site beheert laat ophalen. Die grens weegt hier zwaarder,
want deze route wordt bij elke toetsaanslag geraakt; zonder hem is hij een
haalservice die iedereen deze server op adressen naar keuze laat afsturen.
Voor een bezoeker 204, en dan valt de zoekbalk terug op de gewone suggesties.
Elke toetsaanslag krijgt een volgnummer, zodat een traag antwoord op een oude
vraag niet overschrijft wat je inmiddels hebt getypt.

Waarderen en boosten ter plekke verhuisde van authorize-interaction.js naar
chrome.js: de kaart staat nu op elke pagina, via de zoekbalk. De zoekpagina
stuurde je bij een like eerder weg naar /authorize_interaction.

Onderweg: /authorize_interaction haalt nu ook ondertekend op, net als de
zoekbalk. Anders zegt die pagina "niet gevonden" over een post voor volgers die
de preview wel liet zien.

Getoetst: de preview met de post, ondertekend; wat er in de compacte kaart wel
en niet staat; dat een bezoeker via de preview niets laat ophalen; dat een
zoekterm niets ophaalt; de waarschuwing; het venster op verzoek. Tegenproef:
rechtencheck eruit, waarschuwing genegeerd, compact genegeerd en anoniem
ophalen laten elk hun eigen toets omvallen. Volledige suite 1283 groen.

Het gedrag in chrome.js is in de browser nagelopen met een nagebootste
server: herkennen, laden, invoegen, de volgorde bij een traag antwoord, de
terugval bij 204, waarderen en boosten heen en terug, en het antwoordvenster
open en dicht met de focus op de goede plek.

shaer-utpi

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

File:

• src/routes/search.js (modified) (4 diffs)

src/routes/search.js

@@ -27 +27 @@
 import { t as i18nT, resolveLang } from '../services/i18n.js';
 import ActivityPubService from '../services/ActivityPubService.js';
+import ejs from 'ejs';
+import path from 'path';
+import { fileURLToPath } from 'url';
 import PermissionsService from '../services/PermissionsService.js';
 
@@ -99 +102 @@
   const site = res.locals.site;
   return !!(site && PermissionsService.canAdminSite(u, site));
+}
+
+/**
+ * Haal het object achter een adres op: eerst als post, dan als profiel.
+ *
+ * Eén weg voor de zoekpagina en de live-preview. Staan ze los, dan verschilt er
+ * vroeg of laat iets -- de ondertekening bijvoorbeeld, en dan toont de preview
+ * een post als ontbrekend die de pagina wel vindt.
+ *
+ * ONDERTEKEND als de site: een post die alleen voor volgers zichtbaar is weigert
+ * een anonieme GET, en dan lijkt een bestaande post te ontbreken.
+ */
+async function lookUpRemote(remoteUri, site) {
+  let remote = null;
+  let remoteKind = null;
+  try { remote = await ActivityPubService.resolveRemoteNote(remoteUri, { asSlug: site.slug }); } catch { /* onbereikbaar */ }
+  if (remote) remoteKind = 'note';
+  if (!remote) {
+    try { remote = await ActivityPubService.resolveRemoteActor(remoteUri); } catch { /* onbereikbaar */ }
+    if (remote) remoteKind = 'actor';
+  }
+  const reacted = remoteKind === 'note'
+    ? ActivityPubService.getReaction(site.slug, remote.object_uri || remoteUri)
+    : { liked: false, boosted: false };
+  return { remote, remoteKind, reacted };
 }
 
@@ -214 +242 @@
   const remoteUri = lookupUri(rawQ);
   const mayLookup = remoteUri ? mayLookUp(req, res) : false;
-  let remote = null;
-  let remoteKind = null;
-  if (remoteUri && mayLookup) {
-    // Ondertekend als deze site: een post die alleen voor volgers zichtbaar is
-    // weigert een anonieme GET, en dan lijkt een bestaande post te ontbreken.
-    try { remote = await ActivityPubService.resolveRemoteNote(remoteUri, { asSlug: site.slug }); } catch { /* onbereikbaar */ }
-    if (remote) remoteKind = 'note';
-    if (!remote) {
-      try { remote = await ActivityPubService.resolveRemoteActor(remoteUri); } catch { /* onbereikbaar */ }
-      if (remote) remoteKind = 'actor';
-    }
-  }
+  const { remote, remoteKind, reacted: remoteReacted } = (remoteUri && mayLookup)
+    ? await lookUpRemote(remoteUri, site)
+    : { remote: null, remoteKind: null, reacted: { liked: false, boosted: false } };
 
   const r = searchSite(req, res, rawQ, { posts: 50, tracks: 25, events: 25, pages: 8 });
@@ -234 +253 @@
     total, queryError: r.queryError,
     remoteUri, remote, remoteKind, mayLookup,
-    remoteReacted: (remoteKind === 'note') ? ActivityPubService.getReaction(site.slug, remote.object_uri || remoteUri) : { liked: false, boosted: false },
+    remoteReacted,
     siteTitle: site.title || '',
   });
+});
+
+// ── De live-preview van een adres (HTML-fragment) ─────────────────────────────
+//
+// Plak je een link in de zoekbalk, dan haalt de uitklaplijst de post zelf op
+// in plaats van te zoeken naar de tekst van de link (shaer-utpi).
+//
+// Dezelfde grens als de volle pagina, en hier weegt hij zwaarder: deze route
+// wordt bij elke toetsaanslag geraakt, dus zonder die grens is hij een
+// haalservice die iedereen deze server op adressen naar keuze laat afsturen.
+// Mag je niet ophalen, of is het geen adres, dan 204: de zoekbalk valt dan
+// terug op de gewone suggesties. Geen 403, want voor een bezoeker is een
+// geplakte link gewoon een zoekterm en geen geweigerde handeling.
+const VIEWS_DIR = path.join(path.dirname(fileURLToPath(import.meta.url)), '..', 'views');
+
+router.get('/remote', async (req, res) => {
+  const site = res.locals.site;
+  const remoteUri = lookupUri(String(req.query.q || '').slice(0, 2048));
+  if (!site || !remoteUri || !mayLookUp(req, res)) return res.status(204).end();
+  const { remote, remoteKind, reacted } = await lookUpRemote(remoteUri, site);
+  // Dezelfde taal als de pagina eromheen, anders staat er een Engelse kaart
+  // in een Nederlandse zoekbalk.
+  const lang = resolveLang(req, { userLang: req.session?.user?.lang, defaultLang: getSetting('default_lang') });
+  const html = await ejs.renderFile(path.join(VIEWS_DIR, 'partials', 'remote-preview.ejs'), {
+    t: (k, vars) => i18nT(lang, k, vars),
+    remote, remoteKind, remoteUri, reacted,
+    siteTitle: site.title || '',
+  }, { async: false });
+  // Wat deze bezoeker ziet hangt van zijn rechten en zijn reacties af: niet
+  // bewaren, niet delen.
+  res.set('Cache-Control', 'private, no-store');
+  res.type('html').send(html);
 });