Vijf punten: opslag volgt de herkomst, acties wordt een tabel, twee module-gaten dicht #417

Merged
brenno merged 7 commits from fix/vijf-punten into main 2026-07-20 15:47:14 +00:00
Owner

Vijf losse punten uit een terugkoppeling, elk met zijn eigen commit.

1. Openen en opslaan volgen nu de herkomst. saveDeckWithDestination keek
alleen naar filePath en wist niets van herkomst: een deck van WebDAV belandde
bij Ctrl+S als lokaal bestand. De server hield de oude versie, de laptop de
nieuwe, en niets meldde dat de twee uit elkaar liepen. Nu gaat opslaan terug naar
dezelfde plek, hetzelfde pad, hetzelfde formaat. De zes menu-items voor openen en
opslaan zijn er twee geworden ("Openen uit…" / "Opslaan naar…"), met de
verbindingsvraag die wegvalt bij één verbinding.

Onderweg gevonden: een tabblad droeg drie losse herkomstvelden met een comment
erbij dat een deck er hooguit één zou hebben, terwijl niets dat handhaafde. Van
WebDAV openen en naar S3 wegschrijven leverde er twee. Dat is nu één veld
(StorageOrigin), anders heeft "terug naar de herkomst" geen eenduidig antwoord.

2. Afronden & verzegelen zit achter de informatieveiligheidsmodule. Het
menu-item keek alleen of het deck al verzegeld was, terwijl zijn buurman in
hetzelfde menu de module-check wél had en het RFC3161-tijdstempel dat ná het
verzegelen komt er ook al achter zat. De knop op de ondertekeningsdia is
meegegaan — die was de laatste weg eromheen.

3. "Gebundelde standaarden" heet "Standaarden en methodieken". MIAUW staat in
dat lijstje en is een methodiek, geen standaard.

4. Het slidetype "Acties en besluiten" is opgeheven. Het was een tabel met een
vaste kop en een formulier eroverheen: geen plakken uit een spreadsheet, geen
kolom bijzetten, geen live bewerken tijdens presenteren. De rijen stonden al als
gewone Markdown-tabel op schijf, dus migreren is gratis — de parser leest het
oude class-token als table. Het gemak komt terug als eigenschap van het
tabeltype zelf: een preset voor de kolommen, en een markering van verlopen
datums die wordt afgeleid tegen de dag van presenteren (standaard uit, want een
tabel met historische datums zou volledig rood kleuren).

5. Aanvalsoppervlak hoort bij de module. Het miste category: in
slideTypeMeta en viel terug op "algemeen", dus het stond altijd in de kiezer.

Toetsing

make check groen: 4108 tests, dekking 79,3%. CI blijft pending — er is geen
runner.

Twee bestanden liepen tegen de groottegrens aan; die zijn gesplitst
(shell_actions_connections.dart, een part voor TabInfo/TabsState) in plaats
van de baseline op te rekken.

🤖 Generated with Claude Code

Vijf losse punten uit een terugkoppeling, elk met zijn eigen commit. **1. Openen en opslaan volgen nu de herkomst.** `saveDeckWithDestination` keek alleen naar `filePath` en wist niets van herkomst: een deck van WebDAV belandde bij Ctrl+S als lokaal bestand. De server hield de oude versie, de laptop de nieuwe, en niets meldde dat de twee uit elkaar liepen. Nu gaat opslaan terug naar dezelfde plek, hetzelfde pad, hetzelfde formaat. De zes menu-items voor openen en opslaan zijn er twee geworden ("Openen uit…" / "Opslaan naar…"), met de verbindingsvraag die wegvalt bij één verbinding. Onderweg gevonden: een tabblad droeg drie losse herkomstvelden met een comment erbij dat een deck er hooguit één zou hebben, terwijl niets dat handhaafde. Van WebDAV openen en naar S3 wegschrijven leverde er twee. Dat is nu één veld (`StorageOrigin`), anders heeft "terug naar de herkomst" geen eenduidig antwoord. **2. Afronden & verzegelen zit achter de informatieveiligheidsmodule.** Het menu-item keek alleen of het deck al verzegeld was, terwijl zijn buurman in hetzelfde menu de module-check wél had en het RFC3161-tijdstempel dat ná het verzegelen komt er ook al achter zat. De knop op de ondertekeningsdia is meegegaan — die was de laatste weg eromheen. **3. "Gebundelde standaarden" heet "Standaarden en methodieken".** MIAUW staat in dat lijstje en is een methodiek, geen standaard. **4. Het slidetype "Acties en besluiten" is opgeheven.** Het was een tabel met een vaste kop en een formulier eroverheen: geen plakken uit een spreadsheet, geen kolom bijzetten, geen live bewerken tijdens presenteren. De rijen stonden al als gewone Markdown-tabel op schijf, dus migreren is gratis — de parser leest het oude class-token als `table`. Het gemak komt terug als eigenschap van het tabeltype zelf: een preset voor de kolommen, en een markering van verlopen datums die wordt afgeleid tegen de dag van presenteren (standaard uit, want een tabel met historische datums zou volledig rood kleuren). **5. Aanvalsoppervlak hoort bij de module.** Het miste `category:` in `slideTypeMeta` en viel terug op "algemeen", dus het stond altijd in de kiezer. ## Toetsing `make check` groen: 4108 tests, dekking 79,3%. CI blijft pending — er is geen runner. Twee bestanden liepen tegen de groottegrens aan; die zijn gesplitst (`shell_actions_connections.dart`, een part voor `TabInfo`/`TabsState`) in plaats van de baseline op te rekken. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
brenno force-pushed fix/vijf-punten from b786abdfd2
Some checks failed
CI / Gate (Linux) · Format · Analyze · Coverage (push) Has been cancelled
CI / Test (macos-latest) (push) Has been cancelled
CI / Test (windows-latest) (push) Has been cancelled
CI / Web hardening (push) Has been cancelled
CI / Docs links (push) Has been cancelled
CI / Supply-chain (Trivy · advisory) (push) Has been cancelled
CI / Gate (Linux) · Format · Analyze · Coverage (pull_request) Has been cancelled
CI / Test (macos-latest) (pull_request) Has been cancelled
CI / Test (windows-latest) (pull_request) Has been cancelled
CI / Web hardening (pull_request) Has been cancelled
CI / Docs links (pull_request) Has been cancelled
CI / Supply-chain (Trivy · advisory) (pull_request) Has been cancelled
to de1641c858
Some checks failed
CI / Gate (Linux) · Format · Analyze · Coverage (push) Has been cancelled
CI / Test (macos-latest) (push) Has been cancelled
CI / Test (windows-latest) (push) Has been cancelled
CI / Web hardening (push) Has been cancelled
CI / Docs links (push) Has been cancelled
CI / Supply-chain (Trivy · advisory) (push) Has been cancelled
CI / Gate (Linux) · Format · Analyze · Coverage (pull_request) Has been cancelled
CI / Test (macos-latest) (pull_request) Has been cancelled
CI / Test (windows-latest) (pull_request) Has been cancelled
CI / Web hardening (pull_request) Has been cancelled
CI / Docs links (pull_request) Has been cancelled
CI / Supply-chain (Trivy · advisory) (pull_request) Has been cancelled
2026-07-20 15:46:59 +00:00
Compare
brenno merged commit 75cc7e053d into main 2026-07-20 15:47:14 +00:00
Sign in to join this conversation.
No description provided.