janpeter/Ops-dashboard
1
0
Fork 0
Code Issues 2 Pull requests Projects Releases Packages Wiki Activity Actions

M44 — Vis-design voor het Ops-dashboard (PR E) #280

Merged
janpeter merged 12 commits from feat/m44-vis-design into main 2026-10-04 16:17:39 +02:00
Conversation 1 Commits 12 Files changed 87 +725 -432
janpeter commented 2026-10-04 15:50:39 +02:00
Owner
Copy link

Het Ops-dashboard krijgt het Vis-thema dat Scrum4Me, scrum4me-workers en de Media-Organizer al hebben: warm papier, inkt-tekst, in licht en donker.

Wat verandert

  • app/styles/theme.css is een bytegelijke kopie van app/styles/theme.css uit Scrum4Me (sha256 e01085a3…947dd22d); de contrasttest is ongewijzigd overgenomen (761 gevallen) en staat in de CI-groep base.
  • Public Sans, Source Serif 4 en Geist Mono; themakleur en manifest op #fdf9f4.
  • Ruwe paletklassen van 492 naar 0; alle dark:-paren met ruwe kleuren vervallen. Statuskleuren houden hun betekenis (success, warning, error, info).
  • Knoppen in pilvorm. De 85 losse knoppen blijven <button> (refs, type en disabled-gedrag in bevestigingsdialogen ongemoeid) en krijgen de pil via hun klassen.
  • Ember alleen op inloggen en koppelen; alles wat iets op een server start of wijzigt blijft inkt (docs/m44-ember-lijst.md).
  • Invoervelden met border-control-border, onderstreepte tekstlinks.
  • Het terminalvenster en drie codevakken (auditlog-uitvoer, Caddyfile-weergave, configuratievoorvertoning) dragen een vaste dark-klasse.
  • docs/design/styling.md en een stylingregel in AGENTS.md.

Alleen klassen, lettertypen, twee kleurwaarden en documentatie; geen logica gewijzigd.

Verificatie

  • npm run typecheck en npm run build groen.
  • npm test lokaal op macOS, op de stand vóór de knop- en documentatiecommits: 5.555 geslaagd, 19 mislukt in 2 bestanden (db-access-policy-bundle-flow mist flock, caddy-write-wrapper verwacht Linux). De run op de eindstand liep nog bij het openen van deze PR; CI is leidend.
  • Paletcontrole over app components lib: 0.
  • Routecontrole in de browser met een lokale proefdatabase en een nagebootste ops-agent met vaste voorbeelddata; er is geen echte server aangesproken en niets uitgevoerd. Elf routes bekeken, vier ook in donker: docs/m44-bewijs/increment-2/controle.md. JP keurde de proef (increment 0) goed op 2026-10-04.

Niet in de browser bekeken

Detailpagina's (unit, container, repo, auditrun), control-room, releases, network, mirror, worker-insights, de overige flowpagina's, het terminalvenster met echte uitvoer en de copilot-lade. Ze zijn wel vertaald; de volledige lijst staat in controle.md.

Opgemerkt, niet door deze wijziging

  • Hydratiefout op /systemd en /docker: de tijd bij "updated" wordt op de server anders opgemaakt dan in de browser.
  • Doorstuurlus tussen /login en / met een sessiecookie waarvan de sessie niet meer bestaat.

Plan: Scrum4Me docs/plans/M44-vis-ops-en-media.md. PBI-24, ST-073, ST-074, T-193–T-203.

🤖 Generated with Claude Code

Het Ops-dashboard krijgt het Vis-thema dat Scrum4Me, scrum4me-workers en de Media-Organizer al hebben: warm papier, inkt-tekst, in licht en donker. ## Wat verandert - `app/styles/theme.css` is een bytegelijke kopie van `app/styles/theme.css` uit Scrum4Me (sha256 `e01085a3…947dd22d`); de contrasttest is ongewijzigd overgenomen (761 gevallen) en staat in de CI-groep `base`. - Public Sans, Source Serif 4 en Geist Mono; themakleur en manifest op `#fdf9f4`. - Ruwe paletklassen van 492 naar 0; alle `dark:`-paren met ruwe kleuren vervallen. Statuskleuren houden hun betekenis (success, warning, error, info). - Knoppen in pilvorm. De 85 losse knoppen blijven `<button>` (refs, `type` en `disabled`-gedrag in bevestigingsdialogen ongemoeid) en krijgen de pil via hun klassen. - Ember alleen op inloggen en koppelen; alles wat iets op een server start of wijzigt blijft inkt (`docs/m44-ember-lijst.md`). - Invoervelden met `border-control-border`, onderstreepte tekstlinks. - Het terminalvenster en drie codevakken (auditlog-uitvoer, Caddyfile-weergave, configuratievoorvertoning) dragen een vaste `dark`-klasse. - `docs/design/styling.md` en een stylingregel in `AGENTS.md`. Alleen klassen, lettertypen, twee kleurwaarden en documentatie; geen logica gewijzigd. ## Verificatie - `npm run typecheck` en `npm run build` groen. - `npm test` lokaal op macOS, op de stand vóór de knop- en documentatiecommits: 5.555 geslaagd, 19 mislukt in 2 bestanden (`db-access-policy-bundle-flow` mist `flock`, `caddy-write-wrapper` verwacht Linux). De run op de eindstand liep nog bij het openen van deze PR; CI is leidend. - Paletcontrole over `app components lib`: 0. - Routecontrole in de browser met een lokale proefdatabase en een nagebootste ops-agent met vaste voorbeelddata; er is geen echte server aangesproken en niets uitgevoerd. Elf routes bekeken, vier ook in donker: `docs/m44-bewijs/increment-2/controle.md`. JP keurde de proef (increment 0) goed op 2026-10-04. ## Niet in de browser bekeken Detailpagina's (unit, container, repo, auditrun), control-room, releases, network, mirror, worker-insights, de overige flowpagina's, het terminalvenster met echte uitvoer en de copilot-lade. Ze zijn wel vertaald; de volledige lijst staat in `controle.md`. ## Opgemerkt, niet door deze wijziging - Hydratiefout op `/systemd` en `/docker`: de tijd bij "updated" wordt op de server anders opgemaakt dan in de browser. - Doorstuurlus tussen `/login` en `/` met een sessiecookie waarvan de sessie niet meer bestaat. Plan: Scrum4Me `docs/plans/M44-vis-ops-en-media.md`. PBI-24, ST-073, ST-074, T-193–T-203. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
app/styles/theme.css is bytegelijk aan app/styles/theme.css op Scrum4Me
origin/main (sha256 e01085a3432d6059810c2f8a691e55714216b38b0f67c165c2532980947dd22d,
beide). De contrasttest is ongewijzigd overgenomen (761 gevallen) en staat
in de CI-groep base.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Login, dashboard-widgets en de systemd-lijst gebruiken tokens in plaats van
ruwe paletklassen; de dark:-paren zijn vervallen. Alleen klassen gewijzigd.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Vaste vertaaltabel uit het plan; dark:-paren vervallen. Altijd-donkere
codevakken dragen een vaste dark-klasse. Alleen klassen gewijzigd.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Statuskleuren houden hun betekenis; conflicted en deleted blijven
onderscheidbaar (vol vlak tegenover tint).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Invoervelden krijgen border-control-border, tekstlinks de Vis-onderstreping,
en de poolbadges idea en codex blijven verschillend.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
De 85 losse knoppen blijven button-elementen (refs, type en disabled-gedrag
ongemoeid) en krijgen de pilvorm via hun klassen; rij-, kaart- en
sluierknoppen niet. Alles wat iets op een server uitvoert blijft inkt;
de keuze staat in docs/m44-ember-lijst.md.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Vijftien beelden van elf routes (vier ook in donker) met de bevindingen in
docs/m44-bewijs/increment-2/controle.md; docs/design/styling.md beschrijft
het thema en de regels, CLAUDE.md en AGENTS.md verwijzen ernaar.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
docs(T-203): stylingregel in AGENTS.md
All checks were successful
CI / Select checks (pull_request) Successful in 43s
Details
CI / Ops-agent checks (pull_request) Successful in 39s
Details
CI / DB access operator (pull_request) Successful in 1m15s
Details
CI / Deploy artifact checks (pull_request) Successful in 37s
Details
CI / Docker image build (pull_request) Successful in 1m24s
Details
CI / Mac foundation hermetic checks (pull_request) Successful in 2m5s
Details
CI / Root app checks (pull_request) Successful in 8m49s
Details
CI / Required checks (pull_request) Successful in 29s
Details
1093f94307 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
s4m-codex-reviewer reviewed 2026-10-04 15:58:48 +02:00
s4m-codex-reviewer left a comment
Copy link

COMMENT

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

Findings

  • WARNING — docs/design/styling.md:3: De nieuwe Vis-richtlijn conflicteert met de actieve ProductDoc patterns/md3-theme-styling, die nog MD3-blauw, Geist en afgeronde rechthoekige knoppen voorschrijft. Werk de canonieke productdocumentatie bij of leg de vervangende beslissing vast, zodat volgende bijdragen niet terugmigreren naar het oude thema.
  • WARNING — docs/m44-bewijs/increment-2/controle.md:25: De visuele verificatie dekt veel gemigreerde routes, het terminalvenster met uitvoer en de copilot-lade niet; donkere modus is slechts voor vier routes vastgelegd. Controleer minimaal representatieve detailpagina's, bevestigingsdialogen, terminalstatussen en de copilot-lade in beide modi. De contrasttest controleert tokenwaarden, niet de uiteindelijke rendering.

Beoordeling en verificatie

Geen blokkerende codefout vastgesteld in de aangeleverde diff. De migratie behoudt handlers, bevestigingsflows en disabled-gedrag; de nieuwe semantische kleuren hebben Tailwind-mappings en ember blijft beperkt tot de gedocumenteerde login- en pairingacties. De keuze om bestaande native knoppen te behouden is gedocumenteerd.

PR-head opnieuw gecontroleerd: 1093f94307ae6415c2f4a046bbd27d30fce4df98. De 715 testgevallen van test/theme-contrast.test.ts zijn uitgevoerd via TypeScript-transpilatie met een kleine Node-assert-harness: 0 fouten. Dit is geen geslaagde reguliere Vitest-run: /tmp is hier noexec, waardoor de runner/native binding niet kan starten. De typecheck is geprobeerd maar kon niet groen worden bevestigd door ontbrekende gegenereerde Prisma-types en aanvullende projectafhankelijkheden. Build en volledige testsuite zijn niet bevestigd door deze reviewer. Het aangeleverde bewijs meldt bovendien 19 platformgebonden testfouten op macOS.

Verdict COMMENT wegens de documentatie-afwijking en resterende verificatieonzekerheid.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **WARNING** — docs/design/styling.md:3: De nieuwe Vis-richtlijn conflicteert met de actieve ProductDoc patterns/md3-theme-styling, die nog MD3-blauw, Geist en afgeronde rechthoekige knoppen voorschrijft. Werk de canonieke productdocumentatie bij of leg de vervangende beslissing vast, zodat volgende bijdragen niet terugmigreren naar het oude thema. - **WARNING** — docs/m44-bewijs/increment-2/controle.md:25: De visuele verificatie dekt veel gemigreerde routes, het terminalvenster met uitvoer en de copilot-lade niet; donkere modus is slechts voor vier routes vastgelegd. Controleer minimaal representatieve detailpagina's, bevestigingsdialogen, terminalstatussen en de copilot-lade in beide modi. De contrasttest controleert tokenwaarden, niet de uiteindelijke rendering. ## Beoordeling en verificatie Geen blokkerende codefout vastgesteld in de aangeleverde diff. De migratie behoudt handlers, bevestigingsflows en disabled-gedrag; de nieuwe semantische kleuren hebben Tailwind-mappings en ember blijft beperkt tot de gedocumenteerde login- en pairingacties. De keuze om bestaande native knoppen te behouden is gedocumenteerd. PR-head opnieuw gecontroleerd: `1093f94307ae6415c2f4a046bbd27d30fce4df98`. De 715 testgevallen van `test/theme-contrast.test.ts` zijn uitgevoerd via TypeScript-transpilatie met een kleine Node-assert-harness: 0 fouten. Dit is geen geslaagde reguliere Vitest-run: `/tmp` is hier `noexec`, waardoor de runner/native binding niet kan starten. De typecheck is geprobeerd maar kon niet groen worden bevestigd door ontbrekende gegenereerde Prisma-types en aanvullende projectafhankelijkheden. Build en volledige testsuite zijn niet bevestigd door deze reviewer. Het aangeleverde bewijs meldt bovendien 19 platformgebonden testfouten op macOS. Verdict COMMENT wegens de documentatie-afwijking en resterende verificatieonzekerheid.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
severity/s2

   
severity/s3
No labels
severity/s2
severity/s3
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/Ops-dashboard!280
No description provided.