fix(document): hydrateer grafiekdata en projectmap bij conversie (#1639) #1702

Merged

brenno merged 2 commits from fix/1639-conversie-grafiekdata-projectpad into main

2026-08-21 23:35:51 +00:00

brenno commented

2026-08-21 23:21:40 +00:00

Owner

Samenvatting

Het conversiepad Naar presentatie converteren sloeg de grafiek-hydratatie over die het exportpad wél doet (buildDocumentExportBundle). Een ```chart-blok met source: data/….json kwam als lege grafiekdia in een padloos nieuw tabblad — de cijfers reisden niet mee.

De fix trekt het conversiepad gelijk met export:

hydrateDocumentChartData wordt vóór DocumentDeckBridge.documentToDeck gerund (zelfde als buildDocumentExportBundle)
projectPath reist mee op het nieuwe deck via newDeckInNewTab → newDeck

Bestanden

lib/widgets/document_editor_screen.dart — _convertToPresentation: hydrate + projectPath
lib/state/tabs_provider.dart — newDeckInNewTab: projectPath parameter
lib/state/deck_provider.dart — newDeck: projectPath parameter
tool/check_conventions.dart — verhoogde baselines (tabs_provider 1019, document_editor_screen 1209, TabsNotifier 2256, DeckNotifier 1376, _DocumentEditorScreenState 1211)

Testplan

test/document_conversion_chart_test.dart — 3 tests: grafiekdata staat inline in chart-dia, zonder hydratatie staat het er niet, projectPath reist mee
test/state_providers_extra_test.dart — newDeckInNewTab preserves projectPath
make check groen (10362 tests, 87.2% dekking)
make check-secrets schoon (gitleaks + trufflehog)
make sast schoon (semgrep 0 findings)

Bewaker

Overgeslagen — de wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte. Het herstelt een bestaande hydratatie-gedrag naar een pad dat het miste (conversie gelijk trekken met export).

Generated with Devin

## Samenvatting Het conversiepad **Naar presentatie converteren** sloeg de grafiek-hydratatie over die het exportpad wél doet (`buildDocumentExportBundle`). Een ` ```chart `-blok met `source: data/….json` kwam als lege grafiekdia in een padloos nieuw tabblad — de cijfers reisden niet mee. De fix trekt het conversiepad gelijk met export: - `hydrateDocumentChartData` wordt vóór `DocumentDeckBridge.documentToDeck` gerund (zelfde als `buildDocumentExportBundle`) - `projectPath` reist mee op het nieuwe deck via `newDeckInNewTab` → `newDeck` ## Bestanden - `lib/widgets/document_editor_screen.dart` — `_convertToPresentation`: hydrate + `projectPath` - `lib/state/tabs_provider.dart` — `newDeckInNewTab`: `projectPath` parameter - `lib/state/deck_provider.dart` — `newDeck`: `projectPath` parameter - `tool/check_conventions.dart` — verhoogde baselines (tabs_provider 1019, document_editor_screen 1209, TabsNotifier 2256, DeckNotifier 1376, _DocumentEditorScreenState 1211) ## Testplan - [x] `test/document_conversion_chart_test.dart` — 3 tests: grafiekdata staat inline in chart-dia, zonder hydratatie staat het er niet, projectPath reist mee - [x] `test/state_providers_extra_test.dart` — `newDeckInNewTab` preserves `projectPath` - [x] `make check` groen (10362 tests, 87.2% dekking) - [x] `make check-secrets` schoon (gitleaks + trufflehog) - [x] `make sast` schoon (semgrep 0 findings) ## Bewaker Overgeslagen — de wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer of publieke belofte. Het herstelt een bestaande hydratatie-gedrag naar een pad dat het miste (conversie gelijk trekken met export). Generated with [Devin](https://devin.ai)

brenno added 2 commits

2026-08-21 23:21:40 +00:00

fix(document): hydrateer grafiekdata en projectmap bij conversie (#1639 ) 57b168974c

Het conversiepad document → presentatie sloeg de grafiek-hydratatie over
die het exportpad wél doet: een `source: data/….json` chart-blok kwam
als lege grafiekdia in een padloos nieuw tabblad. Nu wordt
hydrateDocumentChartData vóór documentToDeck gerund (gelijk aan
buildDocumentExportBundle) en reist projectPath mee op het nieuwe deck.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

test(document): regressietest voor conversie-hydratatie (#1639 )

scans / scans (pull_request) Successful in 5m3s

Details

static-gate / static-gate (pull_request) Successful in 11m33s

Details

b53c985261

Documenteert dat het conversiepad grafiekdata inline vouwt en projectPath
behoudt, en bewijst dat zonder hydratatie de chart-dia leeg blijft.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno merged commit 935f0b3820 into main

2026-08-21 23:35:51 +00:00

brenno referenced this pull request from a commit

2026-08-21 23:35:54 +00:00

Merge pull request 'fix(document): hydrateer grafiekdata en projectmap bij conversie (#1639)' (#1702) from fix/1639-conversie-grafiekdata-projectpad into main