Просмотр исходного кода

Pre-livegang code-audit: bevindingen in TASKS.md

Browserloze audit op een verse export. Verse export geeft 0 dart-analyze-errors.

Twee openstaande punten gevonden:
- categorielabel in de horeca-agenda staat op neongroen #42E31C i.p.v. het
  merkgroen #09B34A (Tertiary-token)
- P1-17 heeft nog precies 6 ontbrekende EN-vertalingen, allemaal in
  FilterBalkComponent; tcyc3idz bestaat niet meer

Plus: de lokale repo bevat 15 mappen die de builder niet meer genereert,
goed voor 2 lokale analyze-errors die de verse export niet heeft.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
bob 12 часов назад
Родитель
Сommit
e8ab747379
1 измененных файлов с 72 добавлено и 4 удалено
  1. 72 4
      TASKS.md

+ 72 - 4
TASKS.md

@@ -41,7 +41,7 @@ Google Play en één doorloop die alleen Bob kan doen.
 | **G** | Google Play-checklist G0–G4 | Volledige checklist hieronder |
 | **G** | Google Play-checklist G0–G4 | Volledige checklist hieronder |
 | **65** | De **ingelogde** doorloop | Kernfuncties zijn nooit op een release-build getest. Claude kan dit niet (geen wachtwoorden) |
 | **65** | De **ingelogde** doorloop | Kernfuncties zijn nooit op een release-build getest. Claude kan dit niet (geen wachtwoorden) |
 | **AdMob** | `Show Test Ads` uit | App Settings → AdMob. Wacht op de Google-registratie |
 | **AdMob** | `Show Test Ads` uit | App Settings → AdMob. Wacht op de Google-registratie |
-| **P1-17** | Laatste vertaling(en) | ✅ `1k52hayg` → `End date` en `22xoyrqe` → `Menu` gedaan (2026-09-19 avond, export-geverifieerd). Rest: `tcyc3idz` zodra "Mijn aanmeldingen" af is |
+| **P1-17** | Laatste vertalingen | Nog **precies 6** gaten in de hele levende app, allemaal in `FilterBalkComponent` — sleutels + voorgestelde EN staan in het audit-blok onderaan (2026-09-20). `tcyc3idz` bestaat niet meer |
 | **Drupal** | taak 56-restant · taak 28/109 | Cron-description op "xx days" ✅ gedaan · events zonder categorie gaat via de importer |
 | **Drupal** | taak 56-restant · taak 28/109 | Cron-description op "xx days" ✅ gedaan · events zonder categorie gaat via de importer |
 | **Bij livegang** | Schakelaars en opruimklussen | Eigen blok hieronder |
 | **Bij livegang** | Schakelaars en opruimklussen | Eigen blok hieronder |
 
 
@@ -352,9 +352,10 @@ niet op de toegestane lijst", niet "display bestaat niet".
   `nav.dart` en ~30 kB in de app. Niet opnieuw agenderen. Moet in de **builder**, niet met
   `nav.dart` en ~30 kB in de app. Niet opnieuw agenderen. Moet in de **builder**, niet met
   `git rm` — anders staat alles na de volgende export terug.
   `git rm` — anders staat alles na de volgende export terug.
 
 
-- **P1-17 · laatste vertaling(en).** `1k52hayg` → `End date` en `22xoyrqe` →
-  `Menu` zijn gedaan (2026-09-19 avond). Rest: `tcyc3idz` zodra "Mijn
-  aanmeldingen" af is (nu een lege NL-tekst).
+- **P1-17 · laatste vertalingen.** `1k52hayg` → `End date` en `22xoyrqe` →
+  `Menu` zijn gedaan (2026-09-19 avond). **Restant, exact gemeten 2026-09-20:
+  zes sleutels in `FilterBalkComponent`** — zie het audit-blok onderaan dit
+  bestand. `tcyc3idz` bestaat niet meer in de export.
   ⚠️ Tel geen sleutels mee waar EN == NL terwijl dat correct is (`Select...`,
   ⚠️ Tel geen sleutels mee waar EN == NL terwijl dat correct is (`Select...`,
   `Website`, `Facebook`, `Media`, `", "` — identiek in beide talen).
   `Website`, `Facebook`, `Media`, `", "` — identiek in beide talen).
 
 
@@ -614,3 +615,70 @@ bouwt er ooit iemand een lijst op deze view, dan komt zo'n node 14x in beeld.
 De oorzaak is het klassieke Views-patroon "een veld met meerdere waarden krijgt
 De oorzaak is het klassieke Views-patroon "een veld met meerdere waarden krijgt
 een eigen rij" — kijk op node `214439` welk veld 14 waarden heeft en zet
 een eigen rij" — kijk op node `214439` welk veld 14 waarden heeft en zet
 *Multiple field settings → Display all values in the same row* aan.
 *Multiple field settings → Display all values in the same row* aan.
+
+### 🔍 Pre-livegang code-audit — UITGEVOERD 2026-09-20 (Claude, browserloos)
+
+Verse export (`/tmp/ff-audit`) + de patroonchecks uit `CLAUDE.md`.
+**`fvm dart analyze lib/` op de verse export: 0 errors** (2331 info/warning).
+
+**Twee dingen om op te pakken — allebei builder-werk:**
+
+- 🟡 **Neongroen categorielabel in de horeca-agenda.** Eigenaar: Bob (of Claude,
+  na akkoord). `HorecagelegenheidEventTabelComponentCurrent` zet het
+  categorielabel op `Color(0xFF42E31C)` — een fel, ander groen dan het merkgroen
+  `#09B34A` dat `tagCategorieComponent` en `HorecagelegenheidoverzichtKaart`
+  gebruiken. Zichtbaar op elke zaakpagina met een gevulde agenda (testzaak nid
+  `75459`). Fix: die `Container` → Fill Color → thema-token **Tertiary**.
+  Export-check: `grep -n "0xFF42E31C" <widget>.dart` hoort leeg te zijn.
+  *(De roze `#F265A0` op de deelknop en de `#C8B1B1`/`#C28080` van Entree/Contact
+  stáán nog in de lokale working tree maar zijn in de builder al weg — die twee
+  waren dus terecht afgevinkt, de lokale export liep achter. Zie het punt over
+  stale mappen hieronder.)*
+
+- 🟡 **P1-17 is bijna klaar: nog precies ZES ontbrekende EN-vertalingen in de
+  hele levende app**, allemaal in de nieuwe `FilterBalkComponent` (taak 113).
+  Gemeten over alle 218 i18n-sleutels die in bereikbare widgets gerenderd worden;
+  alles daarbuiten is óf een dode kopie, óf EN==NL waar dat correct is
+  (`Home`, `Info`, `Website`, `Select...`, `Media`, `", "`).
+
+  | sleutel | NL | voorstel EN |
+  |---|---|---|
+  | `lv9f2gi8` | Zoek op titel | Search by title |
+  | `2dzc7lc6` | Alles | All |
+  | `mrkaqmb4` | Vandaag | Today |
+  | `kqsuit8r` | Weekend | Weekend |
+  | `ssmpizqo` | Deze week | This week |
+  | `u4ve4egb` | Alles | All |
+
+  ⚠️ `FilterBalkKanweg` en `FilterBalkComponentCopy` hebben dezelfde zes gaten —
+  die zijn wegwerp (taak 114d), niet vertalen.
+  ✅ **`tcyc3idz` bestaat niet meer** in de verse export; dat restpunt uit het
+  P1-17-blok hierboven kan weg.
+
+**Eén ding voor de git-kant:**
+
+- 🟠 **De lokale repo loopt achter op de builder: 15 mappen onder `lib/` die de
+  verse export niet meer produceert.** `export-code` schrijft alleen, verwijdert
+  nooit. Daardoor geeft `dart analyze` **lokaal 2 errors** (in
+  `kanweg/kanweghorecagelegenheid_current_copy` en `shared/drawer_component_copy`,
+  allebei `Undefined name 'KanwegHorecagelegenhedenOverzichtWidget'`) terwijl de
+  verse export er 0 heeft. **De build valt er niet over** — niets importeert die
+  twee bestanden — maar het maakt de standaard "0 errors"-check onbetrouwbaar en
+  het is precies het patroon dat eerder 161 errors gaf.
+  **Advies: commit de huidige working tree niet as-is; draai een verse export en
+  commit die.** Claude verwijdert niets — de mappen:
+  `favorieten_copy` · `favorieten_copy2` · `kanweg/` (8 bestanden) ·
+  `kanweg_test_upload` · `shared/drawer_component_copy` ·
+  `shared/geen_evenementen_component` · `shared/header_current_copy` ·
+  `horecagelegenhedenoverzicht/horecagelegenheid_event_tabel_component_copy` ·
+  `uitgaanspaginas/home_copy2` · `uitgaanspaginas/home_uitgaan_slider_component_copy` ·
+  `uitgaanspaginas/home_uitgaantabel_kaart_component_copy`.
+
+**Nagemeten en schoon — niet opnieuw doen:**
+`! ??` in models (0) · `getJsonField(...).toList()` zonder `true` (0) ·
+lege responsive-tak `return 0;` (0) · bron/pad-mismatch bij `getJsonField`
+(0) · ellipsis zonder `maxLines` (alleen in twee weeskomponenten) ·
+ongelezen component-parameters (alleen de drie al bekende) ·
+iOS `UsageDescription` staat op precies 3.
+**`showsTestAd: true` staat nog aan** — het restant van G0.6 (`tel-04-horeca.png`)
+is dus nog steeds geblokkeerd op de AdMob-schakelaar.