[Prestaties][P0] Voer presentatievalidatie en parsing buiten de UI-isolate uit

brenno commented

2026-07-26 09:06:29 +00:00

Owner

Probleem

PresentationImportService.prepare en de importers hebben een asynchrone API, maar formaatvalidatie, ZIP-decodering, XML-parsing, IWA/Snappy-decompressie, reconstructie en classificatie draaien synchroon op de aanroepende isolate. De UI roept deze route rechtstreeks aan. Voortgangscallbacks veranderen dat niet: tijdens een zware stap krijgt Flutter geen gelegenheid om frames of invoer te verwerken.

Risico

Bij middelgrote, complexe of vijandige presentaties reageert de interface secondenlang niet. De gebruiker kan niet betrouwbaar annuleren en kan het verschil tussen lang werk en een vastgelopen import niet zien.

Voorgestelde aanpak

Verplaats CPU- en geheugenintensieve validatie/import naar een worker-isolate.
Definieer een serialiseerbaar opdracht-, resultaat-, fout- en voortgangscontract.
Voeg coöperatieve annulering toe tussen begrensde werkeenheden.
Houd deckbouw/UI-keuzes op de passende laag; stuur alleen noodzakelijke data over de isolategrens.
Combineer dit met het centrale resourcebudget uit de gekoppelde securitybevinding.

Acceptatiecriteria

De UI blijft aantoonbaar responsief tijdens import van een representatief groot deck.
Voortgang blijft geordend en eindigt precies één keer in succes, fout of annulering.
Annuleren stopt het werk binnen een gedocumenteerde maximale tijd en schrijft geen gedeeltelijke uitvoer.
Bestaande ImportFailure-redenen en per-dia-resultaten blijven behouden.
Desktop en web hebben een expliciet en getest uitvoeringspad.

Verificatie

Draai make check, een prestatietest op een groot deck en een widget/integratietest die tijdens import invoer en annulering verwerkt.

Herkomst

Verplaatst en tegen de actuele OciDeck-port herijkt vanuit Keiko #2.

## Probleem `PresentationImportService.prepare` en de importers hebben een asynchrone API, maar formaatvalidatie, ZIP-decodering, XML-parsing, IWA/Snappy-decompressie, reconstructie en classificatie draaien synchroon op de aanroepende isolate. De UI roept deze route rechtstreeks aan. Voortgangscallbacks veranderen dat niet: tijdens een zware stap krijgt Flutter geen gelegenheid om frames of invoer te verwerken. ## Risico Bij middelgrote, complexe of vijandige presentaties reageert de interface secondenlang niet. De gebruiker kan niet betrouwbaar annuleren en kan het verschil tussen lang werk en een vastgelopen import niet zien. ## Voorgestelde aanpak - Verplaats CPU- en geheugenintensieve validatie/import naar een worker-isolate. - Definieer een serialiseerbaar opdracht-, resultaat-, fout- en voortgangscontract. - Voeg coöperatieve annulering toe tussen begrensde werkeenheden. - Houd deckbouw/UI-keuzes op de passende laag; stuur alleen noodzakelijke data over de isolategrens. - Combineer dit met het centrale resourcebudget uit de gekoppelde securitybevinding. ## Acceptatiecriteria - De UI blijft aantoonbaar responsief tijdens import van een representatief groot deck. - Voortgang blijft geordend en eindigt precies één keer in succes, fout of annulering. - Annuleren stopt het werk binnen een gedocumenteerde maximale tijd en schrijft geen gedeeltelijke uitvoer. - Bestaande `ImportFailure`-redenen en per-dia-resultaten blijven behouden. - Desktop en web hebben een expliciet en getest uitvoeringspad. ## Verificatie Draai `make check`, een prestatietest op een groot deck en een widget/integratietest die tijdens import invoer en annulering verwerkt. ## Herkomst Verplaatst en tegen de actuele OciDeck-port herijkt vanuit [Keiko #2](https://pawprint.vigilis.online/brenno/Keiko/issues/2).

brenno added the

bug

accepted

labels

2026-07-26 09:06:29 +00:00

brenno referenced this issue from a commit

2026-07-26 11:07:07 +00:00

fix(import): één centraal resourcebudget voor de presentatie-import (#874)

brenno referenced this issue from a commit

2026-07-26 11:31:16 +00:00

fix(import): één centraal resourcebudget voor de presentatie-import (#874)

brenno referenced this issue

2026-07-26 11:31:31 +00:00

fix(import): één centraal resourcebudget voor de presentatie-import (#874) #884

brenno referenced this issue

2026-07-26 11:32:48 +00:00

[Security][P0] Maak het resourcebudget van presentatie-import volledig #874

brenno referenced this issue

2026-07-26 14:02:10 +00:00

[Security][P1] Neutraliseer Markdown- en YAML-injectie uit presentatie-import #876

brenno added the

in-progress

label

2026-07-26 15:31:43 +00:00

brenno commented

2026-07-26 15:31:44 +00:00

Author

Owner

Opgepakt. Tak: fix/875-import-off-isolate (schone worktree vanaf main). Reikwijdte: nieuw serialiseerbaar import-taakcontract + gedeelde parse-kern (lib/services/import/pipeline/import_task.dart), platform-gesplitste isolate-runner (import_runner{,_io,_web}.dart, Isolate.spawn+kill op desktop, in-process met coöperatieve annulering + yields op web), herbedrading van PresentationImportService.prepare en de bulk-runner, en een klein annuleerbaar voortgangsvenster voor de enkelvoudige import. Deckbouw blijft op de hoofd-isolate (WebAssetStore + l10n-translate). Tests: responsiviteit op groot deck, voortgang-exact-één, annuleren binnen grens zonder gedeeltelijke uitvoer, expliciet web-pad, regressie op failures en per-dia-resultaten.

brenno referenced this issue from a commit

2026-07-26 16:44:48 +00:00

feat(import): tijddeadline + serialiseerbaar taakcontract + platform-runner (#875)

brenno referenced this issue from a commit

2026-07-26 16:44:49 +00:00

refactor(import): lees en classificeer op een worker-isolate via de runner (#875)

brenno referenced this issue from a commit

2026-07-26 16:44:49 +00:00

feat(import): annuleerbaar voortgangsvenster voor de enkelvoudige import (#875)

brenno referenced this issue from a commit

2026-07-26 16:44:49 +00:00

test(import): dek het isolate-, annuleer-, deadline- en web-pad (#875)

brenno referenced this issue from a commit

2026-07-26 16:44:49 +00:00

docs(import): documenteer de off-isolate import en annulering (#875)

brenno referenced this issue

2026-07-26 16:45:17 +00:00

fix(import): voer presentatievalidatie en parsing buiten de UI-isolate uit (#875) #902

brenno referenced this issue from a commit

2026-07-26 16:46:20 +00:00

Merge pull request 'fix(import): voer presentatievalidatie en parsing buiten de UI-isolate uit (#875)' (#902) from fix/875-import-off-isolate into main

brenno commented

2026-07-26 16:46:48 +00:00

Author

Owner

Opgelost en op main via PR #902 (merge b897eef9). De import draait nu buiten de UI-isolate: een serialiseerbaar taakcontract + gedeelde parse-kern, een worker-isolate op desktop (kill bij annuleren) en een in-process pad met yields op web. Alleen de deckbouw blijft op de hoofd-isolate (WebAssetStore + l10n). Annuleren stopt binnen een gedocumenteerde tijd zonder gedeeltelijke uitvoer, met een klein annuleerbaar voortgangsvenster voor de enkelvoudige import en midden-in-bestand-stoppen voor de wachtrij. ImportBudget kreeg de maxDuration-deadline uit #874. make check volledig groen (6720 tests, dekking 86,6%). Alle acceptatiecriteria afgedekt met tests.

Opgelost en op main via PR #902 (merge b897eef9). De import draait nu buiten de UI-isolate: een serialiseerbaar taakcontract + gedeelde parse-kern, een worker-isolate op desktop (kill bij annuleren) en een in-process pad met yields op web. Alleen de deckbouw blijft op de hoofd-isolate (WebAssetStore + l10n). Annuleren stopt binnen een gedocumenteerde tijd zonder gedeeltelijke uitvoer, met een klein annuleerbaar voortgangsvenster voor de enkelvoudige import en midden-in-bestand-stoppen voor de wachtrij. ImportBudget kreeg de maxDuration-deadline uit #874. make check volledig groen (6720 tests, dekking 86,6%). Alle acceptatiecriteria afgedekt met tests.

brenno closed this issue

2026-07-26 16:46:48 +00:00

Rows
Columns

[Prestaties][P0] Voer presentatievalidatie en parsing buiten de UI-isolate uit #875

Probleem

Risico

Voorgestelde aanpak

Acceptatiecriteria

Verificatie

Herkomst