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

Merged

janpeter merged 2 commits from fix/m42-review-minors into main

2026-10-04 14:01:42 +02:00

janpeter commented

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

Owner

Twee reviewpunten uit de nacontrole van M42 PR B (#126).

De volle-breedte lijstrij in de MCP-tester en de wis-rij in de multi-select hadden rounded-full; de Vis-spec vraagt de pilvorm alleen voor knoppen. Ze krijgen rounded-lg.
De poolbadges idea en codex in Worker Logs waren allebei bg-info/15 text-info en daardoor niet te onderscheiden. Codex krijgt bg-secondary-container text-secondary (keuze van JP; tertiary is in Vis dezelfde kleur als info).

Lokaal niet getest: de worktree had geen node_modules. CI doet de controle.

PBI-177, ST-1625.

🤖 Generated with Claude Code

Twee reviewpunten uit de nacontrole van M42 PR B (#126). - De volle-breedte lijstrij in de MCP-tester en de wis-rij in de multi-select hadden `rounded-full`; de Vis-spec vraagt de pilvorm alleen voor knoppen. Ze krijgen `rounded-lg`. - De poolbadges `idea` en `codex` in Worker Logs waren allebei `bg-info/15 text-info` en daardoor niet te onderscheiden. Codex krijgt `bg-secondary-container text-secondary` (keuze van JP; `tertiary` is in Vis dezelfde kleur als `info`). Lokaal niet getest: de worktree had geen `node_modules`. CI doet de controle. PBI-177, ST-1625. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 2 commits

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

fix(M42): lijstrijen geen pilvorm (tool-list, multi-select) 55b4f3aacd

De spec vraagt de pilvorm alleen voor knoppen; volle-breedte rijen krijgen rounded-lg.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

fix(M42): codex-poolbadge onderscheidbaar van idea

CI / Verify (pull_request) Successful in 2m5s

Details

d25bc5362d

Beide waren bg-info/15 text-info; codex krijgt de neutrale secondary-container.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

s4m-codex-reviewer reviewed

2026-10-04 13:58:24 +02:00

s4m-codex-reviewer left a comment

COMMENT

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

Findings

INFO — components/worker-logs/worker-logs-view.tsx:63: De visuele regressieverificatie is niet afgerond. Vitest start in deze omgeving niet doordat de native Rollup-module niet geladen kan worden (failed to map segment from shared object). De bestaande componenttests controleren niet expliciet het kleuronderscheid tussen idea en codex. Geen aangetoonde codefout; wegens deze onzekerheid geen APPROVED.

Beoordeling

De opgehaalde PR-head komt overeen met d25bc5362d3860ef842be94f6bb1aa44556c9f97; de repositorydiff komt exact overeen met de aangeleverde drie wijzigingen. De twee rijacties gebruiken nu rounded-lg; bestaande callbacks en selectiegedrag blijven intact. De Codex-badge hergebruikt bestaande thema-tokens en verschilt van de idea-badge. Het berekende tekstcontrast op de nieuwe achtergrond is 8,39:1 (licht) en 8,68:1 (donker). Geen blokkerende bevindingen in codekwaliteit of architectuur; geen nieuwe publieke contracten of documentatieplichtige architectuurwijzigingen. Getoetst aan productdoc architecture/overview en de agent-guide.

Verificatie: git diff --check groen; ESLint 0 fouten en één bestaande waarschuwing buiten de diff; Prisma-generatie en TypeScript-typecheck groen. Tests en browsercontrole niet succesvol uitgevoerd; bestaande M42-reviewrapporten gelden niet als nieuw bewijs voor deze head.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - **INFO — components/worker-logs/worker-logs-view.tsx:63:** De visuele regressieverificatie is niet afgerond. Vitest start in deze omgeving niet doordat de native Rollup-module niet geladen kan worden (`failed to map segment from shared object`). De bestaande componenttests controleren niet expliciet het kleuronderscheid tussen `idea` en `codex`. Geen aangetoonde codefout; wegens deze onzekerheid geen APPROVED. ## Beoordeling De opgehaalde PR-head komt overeen met `d25bc5362d3860ef842be94f6bb1aa44556c9f97`; de repositorydiff komt exact overeen met de aangeleverde drie wijzigingen. De twee rijacties gebruiken nu `rounded-lg`; bestaande callbacks en selectiegedrag blijven intact. De Codex-badge hergebruikt bestaande thema-tokens en verschilt van de idea-badge. Het berekende tekstcontrast op de nieuwe achtergrond is 8,39:1 (licht) en 8,68:1 (donker). Geen blokkerende bevindingen in codekwaliteit of architectuur; geen nieuwe publieke contracten of documentatieplichtige architectuurwijzigingen. Getoetst aan productdoc `architecture/overview` en de agent-guide. Verificatie: `git diff --check` groen; ESLint 0 fouten en één bestaande waarschuwing buiten de diff; Prisma-generatie en TypeScript-typecheck groen. Tests en browsercontrole niet succesvol uitgevoerd; bestaande M42-reviewrapporten gelden niet als nieuw bewijs voor deze head.

janpeter merged commit 45fe4e091a into main

2026-10-04 14:01:42 +02:00

janpeter referenced this pull request from a commit

2026-10-04 14:01:44 +02:00

Merge pull request 'fix(M42): lijstrijen geen pilvorm en codex-poolbadge onderscheidbaar' (#127) from fix/m42-review-minors into main