fix(ratchet): verzoen conventiepoort met Markdown-editor-omvang (#1117) #1122

Merged

brenno merged 1 commit from fix/convention-ratchet-markdown-editor-1117 into main

2026-08-02 12:33:29 +00:00

brenno commented

2026-08-02 12:33:22 +00:00

Owner

Wat

make check (via make check-conventions) stond rood op main: de Markdown-
editor-merges #1103 en #1113 lieten vier bestanden en twee klassen boven hun
plafond komen zonder dat de ratchet in tool/check_conventions.dart meegroeide.
Er draait geen CI-runner, dus de poort ving het pas achteraf — en blokkeerde
intussen elke andere wijziging.

Closes #1117.

Hoe

Verzoent de basislijnen met de bewust gemergede omvang, elk met een reden in de
code (zoals de bestaande +N (#issue)-invoeren):

fileSizeBaseline

nieuw: markdown_deck_editor.dart 1512, tabs_provider.dart 1037
verhoogd: app_shell.dart 840→841 (de markdown_editor_field-import),
app_shell_main_layout.dart 1023→1026 (async verwerp-dialoog voor niet-
toegepaste markdown-wijzigingen)

classSizeBaseline

TabsNotifier 2243→2291, _MainLayoutState 1457→1490

Geen feature-code aangeraakt — de diff raakt alleen tool/check_conventions.dart
(commentaar + int-literalen), dus minimaal conflictrisico met lopend
Markdown-editor-werk.

Afweging (bewaker-relevant: publieke belofte)

De repo-lijn is extractie boven basislijn optrekken. Hier is bewust voor
verzoening gekozen omdat (1) de omvang legitieme, al-gemergede feature-code is,
(2) die bestanden actief in ontwikkeling zijn (extractie erop = conflictrisico en
een veel zwaardere validatielast), en (3) twee van de zes overschrijdingen
klasse-plafonds zijn, die een part-splitsing niet oplost. Diepere extractie
blijft wenselijke opvolging — met name markdown_deck_editor.dart (+512 over
1000: één part plus zeven losse hulpwidgets op regels ~1010–1512 die zich lenen
voor een tweede part). Dat is genoteerd als schuld in de code én hier, niet als
eindstand.

Poort

make check groen op de tak (op verse origin/main): check_conventions weer
binnen alle plafonds (geen "shrank"-tips voor de zes invoeren → exact op maat),
en de rest van de poort (format, analyze, coverage, l10n, docs, sbom) groen.
make ratchets / check_ratchet_trend is adviserend en geen onderdeel van
make check.

## Wat `make check` (via `make check-conventions`) stond **rood op main**: de Markdown- editor-merges #1103 en #1113 lieten vier bestanden en twee klassen boven hun plafond komen zonder dat de ratchet in `tool/check_conventions.dart` meegroeide. Er draait geen CI-runner, dus de poort ving het pas achteraf — en blokkeerde intussen elke andere wijziging. Closes #1117. ## Hoe Verzoent de basislijnen met de bewust gemergede omvang, elk met een reden in de code (zoals de bestaande `+N (#issue)`-invoeren): **`fileSizeBaseline`** - nieuw: `markdown_deck_editor.dart` 1512, `tabs_provider.dart` 1037 - verhoogd: `app_shell.dart` 840→841 (de `markdown_editor_field`-import), `app_shell_main_layout.dart` 1023→1026 (async verwerp-dialoog voor niet- toegepaste markdown-wijzigingen) **`classSizeBaseline`** - `TabsNotifier` 2243→2291, `_MainLayoutState` 1457→1490 Geen feature-code aangeraakt — de diff raakt alleen `tool/check_conventions.dart` (commentaar + int-literalen), dus minimaal conflictrisico met lopend Markdown-editor-werk. ## Afweging (bewaker-relevant: publieke belofte) De repo-lijn is *extractie boven basislijn optrekken*. Hier is bewust voor verzoening gekozen omdat (1) de omvang legitieme, al-gemergede feature-code is, (2) die bestanden actief in ontwikkeling zijn (extractie erop = conflictrisico en een veel zwaardere validatielast), en (3) twee van de zes overschrijdingen **klasse**-plafonds zijn, die een `part`-splitsing niet oplost. Diepere extractie blijft wenselijke opvolging — met name `markdown_deck_editor.dart` (+512 over 1000: één `part` plus zeven losse hulpwidgets op regels ~1010–1512 die zich lenen voor een tweede `part`). Dat is genoteerd als schuld in de code én hier, niet als eindstand. ## Poort `make check` groen op de tak (op verse `origin/main`): `check_conventions` weer binnen alle plafonds (geen "shrank"-tips voor de zes invoeren → exact op maat), en de rest van de poort (format, analyze, coverage, l10n, docs, sbom) groen. `make ratchets` / `check_ratchet_trend` is adviserend en geen onderdeel van `make check`.

brenno added 1 commit

2026-08-02 12:33:23 +00:00

fix(ratchet): verzoen conventiepoort met Markdown-editor-omvang (#1117 )

scans / scans (pull_request) Successful in 3m42s

Details

292a4139b1

De Markdown-editor-merges #1103/#1113 lieten vier bestanden en twee klassen
boven hun plafond komen zonder dat de ratchet in tool/check_conventions.dart
meegroeide — er draait geen CI-runner, dus make check ving het pas achteraf.
De poort stond daardoor rood op main en blokkeerde elke andere wijziging.

Verzoent de basislijnen met de bewust gemergede omvang, elk met reden:
- fileSizeBaseline: nieuw markdown_deck_editor.dart 1512 en tabs_provider.dart
  1037; verhoogd app_shell.dart 840→841 (markdown_editor_field-import),
  app_shell_main_layout.dart 1023→1026 (async verwerp-dialoog).
- classSizeBaseline: TabsNotifier 2243→2291, _MainLayoutState 1457→1490.

Geen feature-code aangeraakt. Diepere extractie (m.n. markdown_deck_editor.dart,
+512 over 1000: zeven losse hulpwidgets lenen zich voor een tweede part) blijft
wenselijke opvolging voor de editor-auteur; dit deblokkeert intussen de poort.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit de766052b7 into main

2026-08-02 12:33:29 +00:00

brenno referenced this pull request from a commit

2026-08-02 12:33:30 +00:00

Merge pull request 'fix(ratchet): verzoen conventiepoort met Markdown-editor-omvang (#1117)' (#1122) from fix/convention-ratchet-markdown-editor-1117 into main

brenno referenced this pull request

2026-08-02 12:33:56 +00:00

Conventiepoort rood op main: Markdown-editor-bestanden boven hun plafond #1117