refactor+fix: DeckNotifier krijgt lucht, plus twee opruimbugs (#630, #639, #637) #663

Merged

brenno merged 1 commit from fix/kleine-bugs into main

2026-07-22 17:59:28 +00:00

brenno commented

2026-07-22 17:59:17 +00:00

Owner

Drie bugs, één tak. make check groen.

#630 — DeckNotifier stond op 1305 van 1305

Nul ruimte. De eerste bijdrage van buiten die iets aan de deck-provider toevoegt, kreeg een rode poort over klassegrootte die niets met zijn wijziging te maken had — plús de uitnodiging uit de foutmelding om de ratchet maar te verhogen. Dat is de kortste weg van "goede eerste indruk" naar "die basislijn is toch maar een getal".

Twee stukken eruit, allebei omdat ze er niet in hoorden:

De vier MIAUW-methodes deden alle vier hetzelfde — map kopiëren, één sleutel wijzigen, vastleggen — en dat vier keer uitgeschreven wás de omvang. Nu één top-level functie met een enum voor welke van de twee mappen. Dat is geen regels schuiven maar minder code.
_checklistForScope raakt geen enkel veld van de klasse. Alles komt binnen via parameters: een pure functie die per ongeluk een methode werd.

1305 → 1256, dus 49 regels lucht. De issue vroeg om vijftig. Vastgezet in de baseline.

#639 — vier golden-failure-PNG's stonden gecommit

Een map met de naam failures en een masked diff erin leest op publicatiedag als een openstaande visuele regressie, terwijl de fout allang weg was. Weg, en de map staat nu in .gitignore.

Het tweede-orde-effect uit de issue klopt en is het echte argument: zolang ze getrackt waren, meldde een échte nieuwe golden-fout zich als "modified" in plaats van als een nieuw bestand — precies wanneer je wilt zien dat er iets níéuws stuk is.

#637 — de padnamen waren al goed, de zin eronder niet

Alle 30 in GIT_STORAGE.md genoemde paden bestaan; dat deel was al door jou rechtgezet. Nagemeten met een script over het hele bestand, niet steekproefsgewijs.

Maar de regel eronder klopte niet, en op drie punten tegelijk: git_cli.dart heet daar "the only Process.run site" en bevat helemaal geen Process.. Het is Process.start, het staat in git_cli_io.dart, en er is een tweede site — de chmod 700 in lib/services/disk_traces.dart, die met reden apart staat en sinds #516 door network_sink_guard_test geteld wordt.

Closes #630
Closes #639
Closes #637

Drie bugs, één tak. `make check` groen. ## #630 — DeckNotifier stond op 1305 van 1305 Nul ruimte. De eerste bijdrage van buiten die iets aan de deck-provider toevoegt, kreeg een rode poort over klassegrootte die niets met zijn wijziging te maken had — plús de uitnodiging uit de foutmelding om de ratchet maar te verhogen. Dat is de kortste weg van "goede eerste indruk" naar "die basislijn is toch maar een getal". Twee stukken eruit, allebei omdat ze er niet in hoorden: - **De vier MIAUW-methodes deden alle vier hetzelfde** — map kopiëren, één sleutel wijzigen, vastleggen — en dat vier keer uitgeschreven wás de omvang. Nu één top-level functie met een enum voor welke van de twee mappen. Dat is geen regels schuiven maar minder code. - **`_checklistForScope` raakt geen enkel veld van de klasse.** Alles komt binnen via parameters: een pure functie die per ongeluk een methode werd. **1305 → 1256**, dus 49 regels lucht. De issue vroeg om vijftig. Vastgezet in de baseline. ## #639 — vier golden-failure-PNG's stonden gecommit Een map met de naam `failures` en een masked diff erin leest op publicatiedag als een openstaande visuele regressie, terwijl de fout allang weg was. Weg, en de map staat nu in `.gitignore`. Het tweede-orde-effect uit de issue klopt en is het echte argument: zolang ze getrackt waren, meldde een échte nieuwe golden-fout zich als "modified" in plaats van als een nieuw bestand — precies wanneer je wilt zien dat er iets níéuws stuk is. ## #637 — de padnamen waren al goed, de zin eronder niet Alle 30 in `GIT_STORAGE.md` genoemde paden bestaan; dat deel was al door jou rechtgezet. Nagemeten met een script over het hele bestand, niet steekproefsgewijs. Maar de regel eronder klopte niet, en op drie punten tegelijk: `git_cli.dart` heet daar *"the only `Process.run` site"* en bevat helemaal geen `Process.`. Het is `Process.start`, het staat in `git_cli_io.dart`, en er is een **tweede** site — de `chmod 700` in `lib/services/disk_traces.dart`, die met reden apart staat en sinds #516 door `network_sink_guard_test` geteld wordt. Closes #630 Closes #639 Closes #637

brenno added 1 commit

2026-07-22 17:59:18 +00:00

refactor(state): geef DeckNotifier lucht, en ruim drie kleine bugs op

CI / Gate (Linux) · Format · Analyze · Coverage (push) Failing after 26s

Details

CI / Web hardening (push) Failing after 23s

Details

CI / Docs links (push) Failing after 21s

Details

CI / Gate (Linux) · Format · Analyze · Coverage (pull_request) Failing after 20s

Details

CI / Supply-chain (Trivy · advisory) (push) Failing after 29s

Details

CI / Web hardening (pull_request) Failing after 31s

Details

CI / Docs links (pull_request) Failing after 31s

Details

CI / Supply-chain (Trivy · advisory) (pull_request) Failing after 21s

Details

CI / Test (macos-latest) (push) Has been cancelled

Details

CI / Test (windows-latest) (push) Has been cancelled

Details

CI / Test (macos-latest) (pull_request) Has been cancelled

Details

CI / Test (windows-latest) (pull_request) Has been cancelled

Details

ec70e733af

#630 — DeckNotifier stond op 1305 van 1305: nul ruimte. De eerste bijdrage van
buiten die iets aan de deck-provider toevoegt, kreeg een rode poort over
klassegrootte die niets met zijn wijziging te maken had, plús een uitnodiging
om de ratchet te verhogen. Dat is de kortste weg van "goede eerste indruk" naar
"die basislijn is toch maar een getal".

Twee stukken eruit, allebei omdat ze er niet in hoorden:

- De vier MIAUW-methodes deden alle vier hetzelfde — map kopiëren, één sleutel
  wijzigen, vastleggen — en dat vier keer uitgeschreven wás de omvang. Nu één
  top-level functie met een enum voor welke map. Geen regels geschoven maar
  minder code.
- `_checklistForScope` raakt geen enkel veld van de klasse. Alles komt binnen
  via parameters: een pure functie die per ongeluk een methode werd.

1305 -> 1256, dus 49 regels lucht, en vastgezet.

#639 — vier golden-failure-PNG's stonden gecommit. Een map met de naam
"failures" en een masked diff erin leest op publicatiedag als een openstaande
visuele regressie, terwijl de fout allang weg was. Weg, en de map staat nu in
.gitignore — zolang ze getrackt waren, meldde een échte nieuwe fout zich als
"modified" in plaats van als een nieuw bestand.

#637 — de drie verkeerde padnamen in GIT_STORAGE.md waren al rechtgezet; alle
30 genoemde paden bestaan. Maar de zin eronder klopte niet: git_cli.dart heet
"the only Process.run site" en bevat helemaal geen Process. Het is Process.start
in git_cli_io.dart, en er is een tweede site (de chmod in disk_traces.dart) die
sinds #516 door network_sink_guard_test geteld wordt.

Closes #630
Closes #639
Closes #637