make check nóg rood op main: methodelengte, hardgecodeerde tekst en SOURCE_MAP na Markdown-editor-merge #1123

New issue

Closed

opened 2026-08-02 12:34:20 +00:00 by brenno · 1 comment

brenno commented

2026-08-02 12:34:20 +00:00

Owner

Wat er misgaat

Na het verzoenen van de omvang-ratchets (#1117) is make check nog steeds rood op main: dezelfde Markdown-editor-merge (commit 6f74b41a "Improve Markdown editing workflow", familie van #1103/#1113) tript nog drie ándere poorten. Geen CI-runner ving het.

Per bestand geverifieerd byte-identiek tussen main en de merge; dit is pre-existing, losstaand van #1117 (dat alleen de bestand-/klasseplafonds dekte).

De drie breuken

check-method-length — lib/widgets/editors/question_editor.dart:180: _QuestionEditorState.build is 163 regels (max 150), niet in methodLengthBaseline.
check-hardcoded-text — lib/widgets/editors/expanded_markdown_dialog.dart:396: de zichtbare tekst '$previewShortcut · Esc' gaat niet door l10n.d('…'). Een nieuwe l10n.d vraagt 31 vertalingen (make add-l10n).
coverage/source_map — test/source_map_coverage_test.dart faalt: drie nieuwe editor-libs staan niet in docs/SOURCE_MAP.md: editor_slide_preview.dart, expanded_markdown_dialog.dart, markdown_editor_field.dart.

Richting

Elk vraagt de repo-eigen fix: build() splitsen (of methodLengthBaseline mét reden), de shortcut-hint door l10n.d + 31 vertalingen, en de drie libs in docs/SOURCE_MAP.md. Blokkeert de poort voor iedereen tot alle drie weg zijn.

## Wat er misgaat Na het verzoenen van de omvang-ratchets (#1117) is `make check` **nog steeds rood op main**: dezelfde Markdown-editor-merge (commit 6f74b41a "Improve Markdown editing workflow", familie van #1103/#1113) tript nog drie ándere poorten. Geen CI-runner ving het. Per bestand geverifieerd byte-identiek tussen main en de merge; dit is pre-existing, losstaand van #1117 (dat alleen de bestand-/klasseplafonds dekte). ## De drie breuken 1. **check-method-length** — `lib/widgets/editors/question_editor.dart:180`: `_QuestionEditorState.build` is 163 regels (max 150), niet in `methodLengthBaseline`. 2. **check-hardcoded-text** — `lib/widgets/editors/expanded_markdown_dialog.dart:396`: de zichtbare tekst `'$previewShortcut · Esc'` gaat niet door `l10n.d('…')`. Een nieuwe `l10n.d` vraagt 31 vertalingen (`make add-l10n`). 3. **coverage/source_map** — `test/source_map_coverage_test.dart` faalt: drie nieuwe editor-libs staan niet in `docs/SOURCE_MAP.md`: `editor_slide_preview.dart`, `expanded_markdown_dialog.dart`, `markdown_editor_field.dart`. ## Richting Elk vraagt de repo-eigen fix: build() splitsen (of methodLengthBaseline mét reden), de shortcut-hint door `l10n.d` + 31 vertalingen, en de drie libs in `docs/SOURCE_MAP.md`. Blokkeert de poort voor iedereen tot alle drie weg zijn.

brenno added the

bug

label

2026-08-02 12:34:20 +00:00

brenno commented

2026-08-02 12:49:50 +00:00

Author

Owner

Al opgelost op main — de drie breuken zijn parallel weggewerkt door #1118/#1119 (merge 8a24e400 'herstel groene make check op main — drift over vijf poorten'), gemaakt terwijl mijn ~80-min poortwachter-run een oudere basis (de26b0fa) mat. Geverifieerd op de huidige main (de766052):

check-method-length — question_editor.build() is nu 123 regels (ad65ddd6 haakte _questionKindField in). ✓
check-hardcoded-text — regel 396 is nu '$previewShortcut · ${context.l10n.d('Esc')}' (4ca3ecbb). ✓
source_map — editor_slide_preview.dart, expanded_markdown_dialog.dart en markdown_editor_field.dart staan nu in docs/SOURCE_MAP.md (fd651ecd). ✓

Bovendien loste #1119 de omvang-ratchets (mijn #1117) óók op, maar via de repo-voorkeur — extractie i.p.v. basislijn optrekken — waardoor die bestanden nu ónder hun oorspronkelijke plafonds staan. Mijn #1117-basislijnverhogingen werden daardoor overbodige, opgeblazen plafonds; die ruim ik op in een aparte kleine PR (tak fix/markdown-editor-gate-followup-1123). Dit issue is een duplicaat van #1118 en gaat dicht.

Al opgelost op main — de drie breuken zijn parallel weggewerkt door #1118/#1119 (merge 8a24e400 'herstel groene make check op main — drift over vijf poorten'), gemaakt terwijl mijn ~80-min poortwachter-run een oudere basis (de26b0fa) mat. Geverifieerd op de huidige main (de766052): 1. check-method-length — question_editor.build() is nu 123 regels (ad65ddd6 haakte _questionKindField in). ✓ 2. check-hardcoded-text — regel 396 is nu '$previewShortcut · ${context.l10n.d('Esc')}' (4ca3ecbb). ✓ 3. source_map — editor_slide_preview.dart, expanded_markdown_dialog.dart en markdown_editor_field.dart staan nu in docs/SOURCE_MAP.md (fd651ecd). ✓ Bovendien loste #1119 de omvang-ratchets (mijn #1117) óók op, maar via de repo-voorkeur — extractie i.p.v. basislijn optrekken — waardoor die bestanden nu ónder hun oorspronkelijke plafonds staan. Mijn #1117-basislijnverhogingen werden daardoor overbodige, opgeblazen plafonds; die ruim ik op in een aparte kleine PR (tak fix/markdown-editor-gate-followup-1123). Dit issue is een duplicaat van #1118 en gaat dicht.

brenno

2026-08-02 12:49:51 +00:00