Document-grafiekhydratatie kent geen 8 MiB-plafond en herschrijft CRLF #1666

New issue

Closed

opened 2026-08-21 12:02:02 +00:00 by brenno · 0 comments

brenno commented

2026-08-21 12:02:02 +00:00

Owner

Wat er misgaat

Twee gaten naast de symlink in #1640, in hetzelfde _inlineBlock:

Geen groottelimiet. Het deck-openpad weigert grafiekdata boven FileService.maxChartDataBytes (8 MiB). Documentexport leest het bestand volledig in het geheugen tijdens buildDocumentExportBundle.
CRLF. Herschreven fences zijn altijd ```chart\n…\n``` (LF). In een verder CRLF-getrouw document worden die blokken op LF gezet vóór de scan.

Waar het zit

lib/services/document_chart_hydration.dart. Cap staat in lib/services/file/file_service_open.dart bij _hydrateCharts.

Stappen

(1) source: data/huge.json van tientallen MiB, document exporteren → geheugenpiek. (2) CRLF-document met een chart-fence exporteren → die fence is LF in de gehydrateerde tussenstap.

Wat het met het formaat doet

De meester .md via Opslaan blijft byte-getrouw; de exportvoorbereiding niet. Een pathologisch databestand is een DoS op het exportpad.

Regressie

Zelfde 8 MiB-cap als deck-open; fence-einde van het brondocument hergebruiken. Tests naar document_chart_hydration_test.dart.

## Wat er misgaat Twee gaten naast de symlink in #1640, in hetzelfde `_inlineBlock`: 1. **Geen groottelimiet.** Het deck-openpad weigert grafiekdata boven `FileService.maxChartDataBytes` (8 MiB). Documentexport leest het bestand volledig in het geheugen tijdens `buildDocumentExportBundle`. 2. **CRLF.** Herschreven fences zijn altijd `` ```chart\n…\n``` `` (LF). In een verder CRLF-getrouw document worden die blokken op LF gezet vóór de scan. ## Waar het zit `lib/services/document_chart_hydration.dart`. Cap staat in `lib/services/file/file_service_open.dart` bij `_hydrateCharts`. ## Stappen (1) `source: data/huge.json` van tientallen MiB, document exporteren → geheugenpiek. (2) CRLF-document met een chart-fence exporteren → die fence is LF in de gehydrateerde tussenstap. ## Wat het met het formaat doet De meester `.md` via Opslaan blijft byte-getrouw; de exportvoorbereiding niet. Een pathologisch databestand is een DoS op het exportpad. ## Regressie Zelfde 8 MiB-cap als deck-open; fence-einde van het brondocument hergebruiken. Tests naar `document_chart_hydration_test.dart`.

brenno added the

bug

label

2026-08-21 12:02:02 +00:00

brenno referenced this issue

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

fix: bug-label batch — 15 issues aangepakt #1708

brenno referenced this issue from a commit

2026-08-22 01:40:03 +00:00

fix: grafiekhydratatie 8 MiB-plafond, CRLF-behoud en compacte fence (#1666, #1668)

brenno closed this issue

2026-08-22 01:46:33 +00:00