M42 — Vis-design voor de Workers-app (PR B) #126

Merged

janpeter merged 8 commits from feat/m42-vis-design into main

2026-10-04 13:02:19 +02:00

janpeter commented

2026-10-04 12:59:39 +02:00

Owner

Plan: M42-vis-design-migratie.md, increment 2. Spec: Vis-design §4.4–4.8. Werkitems: ST-1625, T-1985–T-1989.

Verificatie: npm run verify groen (2.178 tests geslaagd, 18 overgeslagen; lint 0 fouten/1 bestaande waarschuwing), schone productiebuild groen. Theme.css en contrasttest zijn bytegelijk aan Scrum4Me a2831a7c; 761 contrasttests groen. Onafhankelijke reviewbevindingen over links/rondingen zijn opgelost; delta-review GO.

Bewijs: 38 definitieve licht/donkerbeelden plus 6 baselinebeelden, met metingen en SHA256-manifest. De afgesloten job en run zijn uitsluitend lokale proefdata op expliciet JP-akkoord. Er zijn geen jobs gestart of productiegegevens gewijzigd.

Bestaande beperking: React418 op Worker Logs/run-detail reproduceert ook op baseline e58a138 en is apart geregistreerd als Workers ISS-1. De datum-/tijdlogica is ongewijzigd. Lokale screenshots gebruiken een schone productiebuild wegens een HMR-handshakeprobleem in de devserver. Normale/succes/fouttoasts zijn niet allemaal als gebruikersflow opgewekt; wrapper en tokens zijn geverifieerd.

JP heeft push, PR, merge na groene CI en deployment naar workers.jp-visser.nl expliciet geautoriseerd.

De Workers-webapp krijgt het goedgekeurde Vis-thema: dezelfde tokenwaarden als Scrum4Me, Public Sans/Source Serif 4/Geist Mono, pillknoppen, zichtbare veldranden, onderstreepte tekstlinks en een toaster met thematokens. De ruwe paletklassen zijn van 41 naar 0 gegaan. Auth-, route- en jobgedrag zijn behouden. Plan: [M42-vis-design-migratie.md](https://git.jp-visser.nl/janpeter/Scrum4Me/src/commit/a2831a7cbaad615fcabc3baba4c7175c595f434d/docs/plans/M42-vis-design-migratie.md), increment 2. Spec: [Vis-design](https://git.jp-visser.nl/janpeter/Scrum4Me/src/commit/a2831a7cbaad615fcabc3baba4c7175c595f434d/docs/specs/2026-10-03-vis-design-migratie-design.md) §4.4–4.8. Werkitems: ST-1625, T-1985–T-1989. Verificatie: npm run verify groen (2.178 tests geslaagd, 18 overgeslagen; lint 0 fouten/1 bestaande waarschuwing), schone productiebuild groen. Theme.css en contrasttest zijn bytegelijk aan Scrum4Me a2831a7c; 761 contrasttests groen. Onafhankelijke reviewbevindingen over links/rondingen zijn opgelost; delta-review GO. Bewijs: [38 definitieve licht/donkerbeelden plus 6 baselinebeelden](https://git.jp-visser.nl/janpeter/scrum4me-workers/src/commit/b1107248f7ea6a4eb5f027a4fe5938e1c4761bc2/docs/m42-bewijs/controle.md), met metingen en SHA256-manifest. De afgesloten job en run zijn uitsluitend lokale proefdata op expliciet JP-akkoord. Er zijn geen jobs gestart of productiegegevens gewijzigd. Bestaande beperking: React418 op Worker Logs/run-detail reproduceert ook op baseline e58a138 en is apart geregistreerd als Workers ISS-1. De datum-/tijdlogica is ongewijzigd. Lokale screenshots gebruiken een schone productiebuild wegens een HMR-handshakeprobleem in de devserver. Normale/succes/fouttoasts zijn niet allemaal als gebruikersflow opgewekt; wrapper en tokens zijn geverifieerd. JP heeft push, PR, merge na groene CI en deployment naar workers.jp-visser.nl expliciet geautoriseerd.

janpeter added 8 commits

2026-10-04 12:59:39 +02:00

feat(ST-1625): neem Vis-thema en contrastcontrole over 8b086459f8

Theme.css is byte-identiek aan Scrum4Me a2831a7c; diff leeg. Contrastcontrole: 233 failures op oud thema, 761 tests groen op Vis.

feat(ST-1625): laad Vis-lettertypen en pas PWA-kleuren aan 3d18e31a4c

feat(ST-1625): pas Workers-controls en toaster aan voor Vis 9c052e97fc

Pillen en ember voor Inloggen, Nieuwe job en Nieuwe token/aanmaken (wederzijds exclusief). Overige schermen behouden ink-acties. Alle veldranden volgen control-border, ook klasseconstanten en comboboxes. Login gebruikt de gedeelde tokens in plaats van ontbrekende md-sys aliases.

feat(ST-1625): vervang ruwe Workers-paletkleuren door tokens 1263260c11

Paletklassentelling: 41 vóór, 0 na (actuele main bevat nieuwere dispatch-UI dan de 25 uit het plan). Semantische info/success/warning/error volgen beide thema’s; teksttinten uitsluitend 10/15/20 procent.

test(ST-1625): laat kleurasserties de Vis-tokens controleren 5d5a592948

fix(ST-1625): voltooi Vis-links en tekstknoprondingen 889b762edc

style(ST-1625): normaliseer linkdecoratieklassen 55babbb64e

docs(ST-1625): leg Workers Vis-schermcontrole en review vast

CI / Verify (pull_request) Successful in 2m6s

Details

b1107248f7

38 definitieve licht/donkerbeelden en 6 baselinebeelden, SHA256-manifest, font/veldmetingen, lokale proefdata en bestaande React418-waarschuwing. Finale verify2178passed18skip; schone build groen; onafhankelijke delta-GO.

janpeter merged commit bd23523c73 into main

2026-10-04 13:02:19 +02:00

janpeter referenced this pull request from a commit

2026-10-04 13:02:19 +02:00

Merge pull request 'M42 — Vis-design voor de Workers-app (PR B)' (#126) from feat/m42-vis-design into main

s4m-codex-reviewer reviewed

2026-10-04 13:03:44 +02:00

s4m-codex-reviewer left a comment

Verdict: COMMENT

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

Minor — docs/m42-bewijs/review.md:7: Het bewijs noemt .superpowers/m42/spec.md en .superpowers/m42/plan.md bindend, maar beide ontbreken in de PR-tree. Voeg een duurzame bron of de relevante specificatie toe, zodat de geclaimde M42-conformiteit reproduceerbaar beoordeeld kan worden.

Geen blokkerende codebevindingen gevonden. De wijzigingen gebruiken centrale semantische tokens en bestaande UI-patronen; login-, API- en joblogica blijven functioneel ongewijzigd. De productdocumentatie voor Login en PWA vermeldt nog de oude tokens/kleuren en verdient actualisatie.

Verificatie

Beoordeeld op head b1107248f7ea6a4eb5f027a4fe5938e1c4761bc2, overeenkomstig de payload. ESLint: 0 fouten, 1 bestaande waarschuwing; typecheck: geslaagd. Alle 761 contrastasserties uit de nieuwe test slagen via directe TypeScript-transpilatie met synchrone assertion-adapter. De screenshotbestanden komen overeen met alle geregistreerde SHA256-hashes; het loginbeeld is visueel gecontroleerd.

De volledige Vitest-suite kon in deze runtime niet starten: de native Rollup-module kan vanuit de tijdelijke werkmap niet worden geladen (failed to map segment from shared object). Dit is geen aangetoonde PR-regressie, maar de suite en productiebuild zijn niet onafhankelijk bevestigd. De eerder vastgelegde groene uitvoering in de PR-documentatie is auteursbewijs. Daarom COMMENT, geen volledige goedkeuring.

# Verdict: COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **Minor — docs/m42-bewijs/review.md:7:** Het bewijs noemt `.superpowers/m42/spec.md` en `.superpowers/m42/plan.md` bindend, maar beide ontbreken in de PR-tree. Voeg een duurzame bron of de relevante specificatie toe, zodat de geclaimde M42-conformiteit reproduceerbaar beoordeeld kan worden. Geen blokkerende codebevindingen gevonden. De wijzigingen gebruiken centrale semantische tokens en bestaande UI-patronen; login-, API- en joblogica blijven functioneel ongewijzigd. De productdocumentatie voor Login en PWA vermeldt nog de oude tokens/kleuren en verdient actualisatie. ## Verificatie Beoordeeld op head `b1107248f7ea6a4eb5f027a4fe5938e1c4761bc2`, overeenkomstig de payload. ESLint: 0 fouten, 1 bestaande waarschuwing; typecheck: geslaagd. Alle 761 contrastasserties uit de nieuwe test slagen via directe TypeScript-transpilatie met synchrone assertion-adapter. De screenshotbestanden komen overeen met alle geregistreerde SHA256-hashes; het loginbeeld is visueel gecontroleerd. De volledige Vitest-suite kon in deze runtime niet starten: de native Rollup-module kan vanuit de tijdelijke werkmap niet worden geladen (`failed to map segment from shared object`). Dit is geen aangetoonde PR-regressie, maar de suite en productiebuild zijn niet onafhankelijk bevestigd. De eerder vastgelegde groene uitvoering in de PR-documentatie is auteursbewijs. Daarom COMMENT, geen volledige goedkeuring.

janpeter referenced this pull request

2026-10-04 13:50:32 +02:00

fix(M42): lijstrijen geen pilvorm en codex-poolbadge onderscheidbaar #127