feat: laatst gebruikt in API-tokenoverzicht (IDEA-227) #108
No reviewers
Labels
No labels
severity/s4
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
janpeter/scrum4me-workers!108
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "codex/idea-227-last-used-spec"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Toont datum en tijd van het laatste geslaagde tokenverzoek. Null toont exact “Nog geen gebruik geregistreerd”. Sortering start nieuwste-bovenaan, houdt nulls onderaan in beide richtingen en bewaart de bestaande voorkeur. UTC wordt na hydratatie lokaal weergegeven.
Een equivalente positieve nodejs-guard in instrumentation voorkomt daarnaast een bestaande edge-bundelfout op de push-listener, gevonden tijdens de echte browserproef.
Validatie: npm run verify: 1307 tests, 48 gerichte token-tests; echte browserproef met synthetische admin en via MCP gebruikt token, inclusief sorteerbehoud. Mac:claude eindreview GO zonder Critical/Major. Rapport en screenshot staan in docs/superpowers/reviews en docs/evidence.
Afhankelijk van shared #64, Scrum4Me #267 en MCP #166. Eerst kolom/policy, dan consumers. Geen merge, migratie of deployment in productie uitgevoerd.
Reviewfollow-up: echte renderToString→hydrateRoot-proef met UTC op de server en vaste Europe/Amsterdam-tijdzone op de client, inclusief datumwisseling, onveranderd dateTime-attribuut/nulltekst en geen hydrationfouten. Een tijdelijke onveilige SSR-mutatie liet de nieuwe test aantoonbaar falen; runtimecode bleef ongewijzigd.
WIP: feat: laatst gebruikt in API-tokenoverzicht (IDEA-227)to feat: laatst gebruikt in API-tokenoverzicht (IDEA-227)COMMENT
Findings
__tests__/components/api-tokens/token-list.test.tsx:158: de nieuwe weergavetest rendert alleen client-side en gebruikt de tijdzone van de testrunner. Daarmee verifieert hij niet de in het gekoppelde plan vereiste SSR/hydration-overgang of een vaste testtijdzone. Voeg een SSR→hydrate-test en een expliciet ingestelde tijdzone toe, zodat de hydrationveilige UTC-placeholder en lokale datum/tijd regressievast zijn.De implementatie zelf volgt verder de productstandaard: UTC/null-serialisatie, lokale datum+tijd na hydratatie, nulltekst, null-last sortering en behoud van de bestaande opslagkey zijn aanwezig.
APPROVED
Geen blocker- of error-severity findings.
De diff implementeert de Workers-laag van IDEA-227 plan-conform: het gedeelde schema wordt gebumpt,
last_used_atwordt uitsluitend als UTC-ISO/null naar de admin-only client gemapt, en de tabel toont lokale datum én tijd pas na hydratatie. De nieuwe kolom sorteert bij eerste selectie aflopend, bewaart nullwaarden onderaan in beide richtingen en behoudt de bestaande opslagkey en id-tiebreaker. Tests behandelen mapping zonder hash-lek, nulltekst, lokale tijd, SSR/hydratatie en beide sorteerrichtingen; het bewijsdocument registreert de volledige verify- en browserproef.