Copy-paste helpers binnen lib/xmpp/ consolideren (_childByNs 5x, _split, _asObject) #2081

New issue

Closed

opened 2026-09-13 08:07:16 +00:00 by brenno · 0 comments

brenno commented

2026-09-13 08:07:16 +00:00

Owner

Probleem

De XMPP-laag (lib/xmpp/) is de fundering voor Jitsi-videogesprekken (F3-milestone, NATIVE_CALLS.md §5) en NIET legacy. Maar binnen de laag zelf staan meerdere exacte copy-paste helpers die per bestand zijn gedupliceerd.

Gevonden duplicaten

1. `_childByNs` (5x exact gekopieerd)

Dezelfde helper om een child-element op namespace te vinden staat in 5 bestanden:

lib/xmpp/xmpp_chat.dart:297-303
lib/xmpp/xmpp_presence_beacon.dart:212-218
lib/xmpp/xmpp_snapshot.dart:314-320
lib/xmpp/xmpp_key_exchange.dart:310-316
lib/xmpp/xmpp_transport.dart:562-568

2. `_split` (2x)

lib/collab/matrix_snapshot.dart:198-205
lib/xmpp/xmpp_snapshot.dart:322-329

3. `_asObject` (2x)

lib/collab/matrix_relay_transport.dart:323-327
lib/xmpp/xmpp_transport.dart:575-579

Voorstel

Verplaats _childByNs naar lib/xmpp/xmpp_stanza.dart (dat al bestaat en XML-stanza helpers biedt).
Overweeg _split en _asObject in een gedeelde collab_codec.dart of utils/ helper, aangezien ze in zowel Matrix als XMPP voorkomen.

Schatting

~50 regels verwijderbaar.

Labels

duplicate, enhancement

## Probleem De XMPP-laag (lib/xmpp/) is de fundering voor Jitsi-videogesprekken (F3-milestone, NATIVE_CALLS.md §5) en NIET legacy. Maar binnen de laag zelf staan meerdere exacte copy-paste helpers die per bestand zijn gedupliceerd. ## Gevonden duplicaten ### 1. `_childByNs` (5x exact gekopieerd) Dezelfde helper om een child-element op namespace te vinden staat in 5 bestanden: - `lib/xmpp/xmpp_chat.dart:297-303` - `lib/xmpp/xmpp_presence_beacon.dart:212-218` - `lib/xmpp/xmpp_snapshot.dart:314-320` - `lib/xmpp/xmpp_key_exchange.dart:310-316` - `lib/xmpp/xmpp_transport.dart:562-568` ### 2. `_split` (2x) - `lib/collab/matrix_snapshot.dart:198-205` - `lib/xmpp/xmpp_snapshot.dart:322-329` ### 3. `_asObject` (2x) - `lib/collab/matrix_relay_transport.dart:323-327` - `lib/xmpp/xmpp_transport.dart:575-579` ## Voorstel 1. Verplaats `_childByNs` naar `lib/xmpp/xmpp_stanza.dart` (dat al bestaat en XML-stanza helpers biedt). 2. Overweeg `_split` en `_asObject` in een gedeelde `collab_codec.dart` of `utils/` helper, aangezien ze in zowel Matrix als XMPP voorkomen. ## Schatting ~50 regels verwijderbaar. ## Labels duplicate, enhancement

brenno added the

duplicate

enhancement

labels

2026-09-13 08:07:16 +00:00

brenno referenced this issue from a commit

2026-09-13 10:05:59 +00:00

refactor: verplaats _childByNs naar Stanza.childByNs (#2081)

brenno referenced this issue

2026-09-13 10:06:07 +00:00

refactor: dupliceer-codes opruimen (13 issues) #2084

brenno referenced this issue from a commit