make check staat rood op main: voorbestaande drift over vijf poorten (geen per-PR-poort) #1118

New issue

Closed

opened 2026-08-02 09:51:48 +00:00 by brenno · 0 comments

brenno commented

2026-08-02 09:51:48 +00:00

Owner

Symptoom

make check staat rood op main (de26b0fa), los van welke feature-branch dan
ook. Vijf poorten faalden achter elkaar (elke poort verbergt de volgende tot hij
groen is):

conventions — bestands-/klasseplafonds: markdown_deck_editor.dart 1512,
tabs_provider.dart 1037 (TabsNotifier 2291), app_shell.dart 841
(_MainLayoutState 1490), app_shell_main_layout.dart 1026.
method-length — question_editor.dart _QuestionEditorState.build 163 (max 150).
hardcoded-text — expanded_markdown_dialog.dart:396 " · Esc".
source-map — drie editor-libraries zonder regel in docs/SOURCE_MAP.md
(editor_slide_preview.dart, expanded_markdown_dialog.dart, markdown_editor_field.dart).
coverage-per-file — editor_slide_preview.dart op 0% (0/9 regels).

Oorzaak

De statische poorten (en de per-bestand-coverage-vloer) draaien alleen op een
v*-tag via de release-CI, niet per PR. Recente merges (o.a. de Markdown-editor-UX
en de vertaal-merges) lieten deze plafonds groeien en de registraties achter zonder
dat een poort het per PR ving; zo dreef main tussen releases stil rood.

Aanpak (deze PR)

Alles gedrag-behoudend en door te krimpen / registreren, geen enkele baseline
verhoogd: part-splitsingen + top-level-helper-extracties voor de plafonds, build
ingekort, de Esc-toetshint via l10n.d() + unchangedInEnglish/AllLanguages,
drie SOURCE_MAP-regels, en een gerichte test voor editor_slide_preview. Volledige
make check + make l10n-check groen.

Vervolg (los van deze fix)

Overweeg een lichte per-PR-poort met alleen de snelle statische checks
(format-check analyze check-conventions check-method-length check-hardcoded-text check-audience-boundary + de source-map-test), zonder de trage coverage-run, zodat
main niet opnieuw stil rood kan worden. Bewust hier alleen gemeld, niet gebouwd.

## Symptoom `make check` staat **rood op `main`** (de26b0fa), los van welke feature-branch dan ook. Vijf poorten faalden achter elkaar (elke poort verbergt de volgende tot hij groen is): 1. **conventions** — bestands-/klasseplafonds: `markdown_deck_editor.dart` 1512, `tabs_provider.dart` 1037 (`TabsNotifier` 2291), `app_shell.dart` 841 (`_MainLayoutState` 1490), `app_shell_main_layout.dart` 1026. 2. **method-length** — `question_editor.dart` `_QuestionEditorState.build` 163 (max 150). 3. **hardcoded-text** — `expanded_markdown_dialog.dart:396` `" · Esc"`. 4. **source-map** — drie editor-libraries zonder regel in `docs/SOURCE_MAP.md` (`editor_slide_preview.dart`, `expanded_markdown_dialog.dart`, `markdown_editor_field.dart`). 5. **coverage-per-file** — `editor_slide_preview.dart` op 0% (0/9 regels). ## Oorzaak De statische poorten (en de per-bestand-coverage-vloer) draaien alleen op een `v*`-tag via de release-CI, niet per PR. Recente merges (o.a. de Markdown-editor-UX en de vertaal-merges) lieten deze plafonds groeien en de registraties achter zonder dat een poort het per PR ving; zo dreef `main` tussen releases stil rood. ## Aanpak (deze PR) Alles gedrag-behoudend en door te **krimpen / registreren**, geen enkele baseline verhoogd: part-splitsingen + top-level-helper-extracties voor de plafonds, `build` ingekort, de `Esc`-toetshint via `l10n.d()` + `unchangedInEnglish/AllLanguages`, drie SOURCE_MAP-regels, en een gerichte test voor `editor_slide_preview`. Volledige `make check` + `make l10n-check` groen. ## Vervolg (los van deze fix) Overweeg een lichte **per-PR-poort** met alleen de snelle statische checks (`format-check analyze check-conventions check-method-length check-hardcoded-text check-audience-boundary` + de source-map-test), zonder de trage coverage-run, zodat `main` niet opnieuw stil rood kan worden. Bewust hier alleen gemeld, niet gebouwd.

brenno referenced this issue from a commit

2026-08-02 09:53:08 +00:00

refactor: breng bestanden en klassen onder de ratchet-plafonds (#1118)

brenno referenced this issue from a commit

2026-08-02 09:53:08 +00:00

refactor(question-editor): kort build() in onder de methodelengte-grens (#1118)

brenno referenced this issue from a commit

2026-08-02 09:53:08 +00:00

fix(l10n): toon de Esc-toetshint via d() in plaats van hardgecodeerd (#1118)

brenno referenced this issue from a commit

2026-08-02 09:53:08 +00:00

test+docs: dek editor_slide_preview en registreer drie editor-libraries (#1118)