feat: meld een overgeslagen sidecar aan de gebruiker (#564) #568

Merged

brenno merged 1 commit from feat/sidecar-waarschuwing into main

2026-07-22 16:06:45 +00:00

brenno commented

2026-07-22 16:06:34 +00:00

Owner

De bevinding uit de bewaker-ronde op #560, die ik toen bewust niet meenam omdat hij een eigen kanaal en 31 vertalingen kost.

Het bestand blijft heel en wordt bij opslaan niet overschreven, dus er is niets kwijt — maar de gebruiker zag alleen dat zijn strepen er niet waren, en dat leest als "die heb ik nooit gemaakt". Nu staat er een melding, in foutkleur, mét de geruststelling die het verschil maakt tussen "weg" en "even niet geladen".

Eigen kanaal en niet dat van de grafiekdata: de tekst is anders, en het gaat hier over werk van de gebruiker zélf in plaats van over een ontbrekend databestand.

Wat de ratchets zeiden

Drie vielen er om, en ze wezen naar iets echts in plaats van naar mijn regels commentaar. Het resultaat van openDeckDetailed was een anoniem record dat bij elke nieuwe mededeling groeide — eerst het deck, toen de weigeringsreden, toen de grafiekwaarschuwingen, nu de overgeslagen lagen. Elk veld erbij kostte een regel op negen returnplekken, en dat zag je pas doordat de klassenratchet omviel.

Het heet nu DeckOpenResult, met een fabriek voor de weigering, en het woont in de part die over het openen gaat. Twee identieke meldblokken in TabsNotifier zijn één top-level functie geworden.

Netto krimpen beide klassen onder waar ze stonden: FileService 2879 → 2864, TabsNotifier 2403 → 2395. Allebei vastgezet in plaats van het plafond te verhogen.

Regressietest legt beide kanten vast: een te grote laag wordt gemeld, een gezond deck meldt niets. Eén nieuwe l10n-string in alle 31 talen. make check groen.

Closes #564

De bevinding uit de bewaker-ronde op #560, die ik toen bewust niet meenam omdat hij een eigen kanaal en 31 vertalingen kost. Het bestand blijft heel en wordt bij opslaan niet overschreven, dus er is niets kwijt — maar de gebruiker zag alleen dat zijn strepen er niet waren, en dat leest als "die heb ik nooit gemaakt". Nu staat er een melding, in foutkleur, mét de geruststelling die het verschil maakt tussen "weg" en "even niet geladen". Eigen kanaal en niet dat van de grafiekdata: de tekst is anders, en het gaat hier over werk van de gebruiker zélf in plaats van over een ontbrekend databestand. ## Wat de ratchets zeiden Drie vielen er om, en ze wezen naar iets echts in plaats van naar mijn regels commentaar. Het resultaat van `openDeckDetailed` was een anoniem record dat bij elke nieuwe mededeling groeide — eerst het deck, toen de weigeringsreden, toen de grafiekwaarschuwingen, nu de overgeslagen lagen. Elk veld erbij kostte een regel op negen returnplekken, en dat zag je pas doordat de klassenratchet omviel. Het heet nu `DeckOpenResult`, met een fabriek voor de weigering, en het woont in de part die over het openen gaat. Twee identieke meldblokken in `TabsNotifier` zijn één top-level functie geworden. Netto krimpen beide klassen onder waar ze stonden: FileService 2879 → 2864, TabsNotifier 2403 → 2395. Allebei vastgezet in plaats van het plafond te verhogen. Regressietest legt beide kanten vast: een te grote laag wordt gemeld, een gezond deck meldt niets. Eén nieuwe l10n-string in alle 31 talen. `make check` groen. Closes #564

brenno added 1 commit

2026-07-22 16:06:34 +00:00

feat(sidecar): meld een overgeslagen laag aan de gebruiker

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

Details

CI / Web hardening (push) Failing after 4s

Details

CI / Docs links (push) Failing after 4s

Details

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

Details

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

Details

CI / Web hardening (pull_request) Failing after 4s

Details

CI / Docs links (pull_request) Failing after 4s

Details

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

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

65a9bd6283

De bevinding uit de bewaker-ronde op #560, bewust toen niet meegenomen. Het
bestand blijft heel en wordt bij opslaan niet overschreven, dus er is niets
kwijt — maar de gebruiker zag alleen dat zijn strepen er niet waren, en dat
leest als "die heb ik nooit gemaakt".

Eigen kanaal, niet dat van de grafiekdata: de tekst is anders en het gaat hier
over werk van de gebruiker zélf. In foutkleur, met de geruststelling erbij die
het verschil maakt tussen "weg" en "even niet geladen".

Drie ratchets vielen onderweg om, en die wezen naar iets echts: het resultaat
van `openDeckDetailed` was een anoniem record dat bij elke nieuwe mededeling
groeide — eerst het deck, toen de weigeringsreden, toen de
grafiekwaarschuwingen, nu de overgeslagen lagen. Elk veld erbij kostte een
regel op negen returnplekken. Het heet nu `DeckOpenResult`, met een fabriek
voor de weigering, en woont in de part die over het openen gaat.

Twee identieke meldblokken in TabsNotifier zijn één top-level functie geworden.
Netto krimpen beide klassen onder waar ze stonden: FileService 2879 -> 2864,
TabsNotifier 2403 -> 2395, allebei vastgezet.

Eén nieuwe l10n-string, in alle 31 talen.

Closes #564

brenno merged commit 91b11d0922 into main

2026-07-22 16:06:45 +00:00

brenno referenced this pull request from a commit

2026-07-22 16:06:47 +00:00

Merge pull request 'feat: meld een overgeslagen sidecar aan de gebruiker (#564)' (#568) from feat/sidecar-waarschuwing into main