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

brenno commented

2026-08-02 08:45:17 +00:00

Owner

Wat er misgaat

make check (en make check-conventions) staat rood op main: vier bestanden en twee klassen zijn boven hun plafond gegroeid zonder dat de ratchet in tool/check_conventions.dart is bijgewerkt. Dit is binnengekomen met de Markdown-editor-merges #1103 en #1113 (er draait geen CI-runner, dus de poort ving het niet vóór de merge).

Vastgesteld op main (commit 10338914), losstaand van de wijziging die het aan het licht bracht (#1111 — die raakt geen van deze bestanden).

Overschrijdingen

Bestandsplafond (fileSizeBaseline / max 1000):

lib/widgets/editors/markdown_deck_editor.dart — 1512 regels (max 1000, niet in de basislijn)
lib/state/tabs_provider.dart — 1037 regels (max 1000, niet in de basislijn)
lib/widgets/app_shell_main_layout.dart — 1026 regels (plafond 1023)
lib/widgets/app_shell.dart — 841 regels (plafond 840)

Klasseplafond (classSizeBaseline):

lib/state/tabs_provider.dart#TabsNotifier — 2291 (plafond 2243)
lib/widgets/app_shell.dart#_MainLayoutState — 1490 (plafond 1457)

Richting

De repo-lijn is: haal gedrag eruit (top-level helper, service of losse klasse) i.p.v. de basislijn optrekken — zeker bij markdown_deck_editor.dart (+512 over 1000) en tabs_provider.dart (+37), die nog geen basislijn-invoer hebben. Voor de kleine overschrijdingen (app_shell.dart +1, app_shell_main_layout.dart +3) kan een bewuste basislijn-verhoging mét reden verdedigbaar zijn. De auteur van de Markdown-editor-wijziging is de aangewezen persoon; dit blokkeert intussen de poort voor iedereen.

## Wat er misgaat `make check` (en `make check-conventions`) staat **rood op main**: vier bestanden en twee klassen zijn boven hun plafond gegroeid zonder dat de ratchet in `tool/check_conventions.dart` is bijgewerkt. Dit is binnengekomen met de Markdown-editor-merges #1103 en #1113 (er draait geen CI-runner, dus de poort ving het niet vóór de merge). Vastgesteld op main (commit 10338914), losstaand van de wijziging die het aan het licht bracht (#1111 — die raakt geen van deze bestanden). ## Overschrijdingen Bestandsplafond (`fileSizeBaseline` / max 1000): - `lib/widgets/editors/markdown_deck_editor.dart` — 1512 regels (max 1000, niet in de basislijn) - `lib/state/tabs_provider.dart` — 1037 regels (max 1000, niet in de basislijn) - `lib/widgets/app_shell_main_layout.dart` — 1026 regels (plafond 1023) - `lib/widgets/app_shell.dart` — 841 regels (plafond 840) Klasseplafond (`classSizeBaseline`): - `lib/state/tabs_provider.dart#TabsNotifier` — 2291 (plafond 2243) - `lib/widgets/app_shell.dart#_MainLayoutState` — 1490 (plafond 1457) ## Richting De repo-lijn is: haal gedrag eruit (top-level helper, service of losse klasse) i.p.v. de basislijn optrekken — zeker bij `markdown_deck_editor.dart` (+512 over 1000) en `tabs_provider.dart` (+37), die nog geen basislijn-invoer hebben. Voor de kleine overschrijdingen (`app_shell.dart` +1, `app_shell_main_layout.dart` +3) kan een bewuste basislijn-verhoging mét reden verdedigbaar zijn. De auteur van de Markdown-editor-wijziging is de aangewezen persoon; dit blokkeert intussen de poort voor iedereen.

brenno added the

bug

label

2026-08-02 08:45:17 +00:00

brenno added the

in-progress

label

2026-08-02 08:57:45 +00:00

brenno commented

2026-08-02 08:57:45 +00:00

Author

Owner

Opgepakt. Tak: fix/convention-ratchet-markdown-editor-1117. Aanpak: de ratchet in tool/check_conventions.dart verzoenen met de werkelijke, bewust gemergede omvang uit #1103/#1113 — twee nieuwe fileSizeBaseline-invoeren (markdown_deck_editor.dart, tabs_provider.dart) en vier verhogingen (app_shell.dart, app_shell_main_layout.dart + de klassen TabsNotifier, _MainLayoutState), elk met reden. Diepere extractie (m.n. markdown_deck_editor.dart, +512) blijft wenselijk als opvolging voor de editor-auteur; dit deblokkeert intussen de poort. Raakt alleen tool/, geen feature-bestanden.

Rows
Columns

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

Wat er misgaat

Overschrijdingen

Richting