perf(media): herschrijf duplicaatverwijzingen in één deckpass #1052

New issue

Closed

opened 2026-08-01 08:26:10 +00:00 by brenno · 1 comment

brenno commented

2026-08-01 08:26:10 +00:00

Owner

Impact
Veel duplicaten en veel decks vermenigvuldigen elkaar. In het maximum van 20000 bij 20000 ontstaan honderden miljoenen vergelijkingen of volledige decklezingen.

Relevante code

lib/services/image_reference_service.dart:74 en 135
lib/widgets/dialogs/parts/image_carousel_picker_actions.dart:405

Acceptatie

Bouw één genormaliseerde target- of replacement-map.
Lees ieder deckbestand eenmaal, tel/vervang alle doelen in die pass en schrijf hooguit eenmaal wanneer iets veranderde.
Behoud relatieve POSIX-paden en atomair schrijven.
Schaaltest telt decklezingen/schrijfacties en bewijst O(decks + verwijzingen + targets) in plaats van het huidige product.

Context countReferences vergelijkt iedere gevonden afbeeldingsreferentie met ieder doelpad. Bij toepassen loopt _applyDedupePlan vervolgens voor iedere te verwijderen kopie opnieuw door ieder deckbestand en leest/schrijft dat bestand opnieuw. Impact Veel duplicaten en veel decks vermenigvuldigen elkaar. In het maximum van 20000 bij 20000 ontstaan honderden miljoenen vergelijkingen of volledige decklezingen. Relevante code - lib/services/image_reference_service.dart:74 en 135 - lib/widgets/dialogs/parts/image_carousel_picker_actions.dart:405 Acceptatie - Bouw één genormaliseerde target- of replacement-map. - Lees ieder deckbestand eenmaal, tel/vervang alle doelen in die pass en schrijf hooguit eenmaal wanneer iets veranderde. - Behoud relatieve POSIX-paden en atomair schrijven. - Schaaltest telt decklezingen/schrijfacties en bewijst O(decks + verwijzingen + targets) in plaats van het huidige product.

brenno added the

enhancement

accepted

labels

2026-08-01 08:26:10 +00:00

brenno referenced this issue from a commit

2026-08-01 20:43:49 +00:00

perf(media): herschrijf duplicaatverwijzingen in één deckpass (#1052)

brenno referenced this issue from a commit

2026-08-01 20:43:49 +00:00

test(media): dek het #1052-ontdubbelpad met een widgettest (#1052)

brenno referenced this issue

2026-08-01 20:44:03 +00:00

perf(media): herschrijf duplicaatverwijzingen in één deckpass (#1052) #1093

brenno referenced this issue from a commit

2026-08-01 20:44:08 +00:00

Merge pull request 'perf(media): herschrijf duplicaatverwijzingen in één deckpass (#1052)' (#1093) from fix/media-dedupe-onepass-1052 into main

brenno commented

2026-08-01 20:44:15 +00:00

Author

Owner

Opgelost op main (PR #1093, merge in 78f2dc2d). countReferences gebruikt nu één canonieke lookup-map; het toepassen bouwt één replacement-map en leest/schrijft elk deck hooguit één keer via het nieuwe replaceReferencesMulti (relatieve POSIX-paden en atomair schrijven behouden) — O(decks + verwijzingen + doelen). Tests: veel-doelen-telling, één-pass-herschrijving, geen-schrijf-zonder-treffer en het volledige ontdubbelpad.

Opgelost op main (PR #1093, merge in 78f2dc2d). countReferences gebruikt nu één canonieke lookup-map; het toepassen bouwt één replacement-map en leest/schrijft elk deck hooguit één keer via het nieuwe replaceReferencesMulti (relatieve POSIX-paden en atomair schrijven behouden) — O(decks + verwijzingen + doelen). Tests: veel-doelen-telling, één-pass-herschrijving, geen-schrijf-zonder-treffer en het volledige ontdubbelpad.

brenno closed this issue

2026-08-01 20:44:16 +00:00