Code-review-feedback aus etappe 2.9/2.10:
1. cli.ts: deletedCoords im cache wird ab jetzt akkumuliert statt
ersetzt. Vorher wurden bei einem run nur die aktuell von relays
gelieferten kind:5-coords geschrieben — wenn ein relay beim
naechsten run die alten deletions nicht mehr liefert (GC,
relay-tausch), waere die geschichte verloren und newDeletionsCount
im naechsten lauf wieder "neu" -> false-positive hard-fail im
drop-check.
2. relays.ts: kommentar zum belt-and-suspenders-setTimeout neben dem
RxJS-timeout-operator, damit der zweck (handle-cleanup falls beide
subscribe-callbacks verschluckt werden) klar ist.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Vorher konnten malformed tags wie ['t'] (ohne second element)
undefined ins string[]-array werfen, das im JSON als null landete.
Code-review-feedback aus etappe 2.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Per NIP-09 darf ein deletion nur events mit created_at <= deletion.created_at
loeschen. Vorher wurde ein re-publizierter post nach geloeschtem vorgaenger
stumm wegfiltern. Code-review-feedback aus etappe 2.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Jörg Lohrer