From 89c6896f5a43ea04097ceb46bafb348f92c72501 Mon Sep 17 00:00:00 2001 From: janpeter visser Date: Mon, 27 Apr 2026 19:31:18 +0200 Subject: [PATCH] docs(M9): add implementation plan document and link from backlog --- CLAUDE.md | 1 + docs/plans/M9-active-product-backlog.md | 161 ++++++++++++++++++++++++ docs/scrum4me-backlog.md | 2 + 3 files changed, 164 insertions(+) create mode 100644 docs/plans/M9-active-product-backlog.md diff --git a/CLAUDE.md b/CLAUDE.md index 251ef41..77d0877 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -24,6 +24,7 @@ Lees het relevante document voordat je aan een feature begint. Nooit gokken over | `docs/API.md` | REST-API contract voor Claude Code — endpoints, status-enums, foutcodes, voorbeeld-curls | | `docs/scrum4me-styling.md` | **Lees dit voor elk component** — MD3-kleuren, shadcn patronen | | `docs/agent-instruction-audit.md` | Waarom de agent-instructies zijn aangescherpt; checklist voor toekomstige wijzigingen | +| `docs/plans/-*.md` | Implementatieplan per milestone — Bestanden, Stappen, Aandachtspunten, Verificatie. Lees vóór je aan een ST begint. Milestone-key matcht backlog-header (`M9`, `M3.5`, `PBI-9`, …). | | [`madhura68/scrum4me-mcp`](https://github.com/madhura68/scrum4me-mcp) | MCP-server repo: native tools voor Claude Code, schema-sync via git submodule | --- diff --git a/docs/plans/M9-active-product-backlog.md b/docs/plans/M9-active-product-backlog.md new file mode 100644 index 0000000..9a79a80 --- /dev/null +++ b/docs/plans/M9-active-product-backlog.md @@ -0,0 +1,161 @@ +# M9 — Actief Product Backlog + +Eén "actief Product Backlog" per gebruiker, persistent op `User.active_product_id`. NavBar wordt: Producten | Product Backlog | Sprint | Solo | Todo's. Zonder actief PB zijn Backlog/Sprint/Solo disabled. Sprint is alleen klikbaar als er een sprint met status `ACTIVE` bestaat. Vervangt de bestaande `last_product`-cookieflow. + +Backlog-entries: zie [scrum4me-backlog.md § M9](../scrum4me-backlog.md#m9-actief-product-backlog). + +--- + +## ST-901 — Database `user.active_product_id` + +> Status: voltooid in commit `dad9a80`. + +**Bestanden** +- `prisma/schema.prisma` — model `User` uitgebreid + named relation +- `prisma/migrations/20260427165329_add_user_active_product_id/migration.sql` — migratie + +**Stappen** +1. Op `User`: `active_product_id String? @db.Uuid` + relatie `active_product Product? @relation("UserActiveProduct", fields: [active_product_id], references: [id], onDelete: SetNull)` + `@@index([active_product_id])`. +2. Op `Product`: tegenrelatie `active_for_users User[] @relation("UserActiveProduct")` (anders conflicteert het met de bestaande `Product.user_id`-relatie). +3. `npx prisma migrate dev --name add_user_active_product_id`. + +**Aandachtspunten** +- `vendor/scrum4me`-submodule in repo `scrum4me-mcp` heeft hetzelfde schema. Na merge moet daar `prisma generate && tsc --noEmit` slagen, anders breekt de wekelijkse drift-check (`trig_015FFUnxjz9WMuhhWNGBQKFD`). +- Geen seed-wijziging nodig — `null` is correcte initiële staat. + +**Verificatie** +- `npx prisma migrate dev` slaagt +- `npx prisma validate` zonder fouten +- `prisma studio` toont kolom + +--- + +## ST-902 — Server Actions: actief product zetten/wissen + auto-clear + +**Bestanden** +- `actions/active-product.ts` — nieuw, twee Server Actions +- `actions/products.ts` — uitbreiden bij `archiveProductAction` +- `actions/product-members.ts` — uitbreiden bij `leaveProductAction` en `removeMemberAction` (locatie verifiëren met grep) +- `__tests__/actions/active-product.test.ts` — nieuw + +**Stappen** + +1. **`setActiveProductAction({ productId })`** in `actions/active-product.ts`: + - Volg `docs/patterns/server-action.md` + - Zod: `z.object({ productId: z.string().uuid() })` + - `getSession()` → 401 bij geen sessie + - **Demo-guard**: `if (session.isDemo) return { ok: false, error: 'Niet beschikbaar in demo-modus.' }` + - Toegangscheck: `prisma.product.findFirst({ where: { id: productId, archived: false, ...productAccessFilter(userId) } })` → `null` levert `{ ok: false, error: 'Product niet gevonden of geen toegang.' }` + - `prisma.user.update({ where: { id: userId }, data: { active_product_id: productId } })` + - `revalidatePath('/', 'layout')` — laat NavBar in alle routes opnieuw renderen + - Return `{ ok: true }` + +2. **`clearActiveProductAction()`** in hetzelfde bestand: + - Geen input + - `getSession()` + demo-guard + - `prisma.user.update({ where: { id: userId }, data: { active_product_id: null } })` + - `revalidatePath('/', 'layout')` + +3. **Auto-clear bij toegangsverlies** — drie call-sites uitbreiden ná de hoofdmutatie: + - `archiveProductAction(productId)`: `prisma.user.updateMany({ where: { active_product_id: productId }, data: { active_product_id: null } })` + - `leaveProductAction(productId)`: `prisma.user.updateMany({ where: { id: userId, active_product_id: productId }, data: { active_product_id: null } })` + - `removeMemberAction(productId, removedUserId)`: `prisma.user.updateMany({ where: { id: removedUserId, active_product_id: productId }, data: { active_product_id: null } })` + - Eigenaarsverwijdering van een product wordt door FK `onDelete: SetNull` automatisch geregeld — geen extra code + +4. **Tests** — `__tests__/actions/active-product.test.ts`: + - setActive met onbekend product → `{ ok: false }` + - setActive met archived product → `{ ok: false }` + - setActive met product zonder access → `{ ok: false }` + - setActive happy path → `users.active_product_id` gezet + - Demo-user setActive → error + geen DB-mutatie + - archiveProductAction op actief product → `active_product_id` gecleared voor alle eigenaren/leden + +**Aandachtspunten** +- Race-condition: setActive winnen ná auto-clear kan voorkomen. Layout-guard in ST-903 vangt dit op bij volgende request. +- `revalidatePath('/', 'layout')` is correct — niet `revalidatePath('/dashboard')` (NavBar zit in root layout van `(app)`). +- Geen `productAccessFilter` op `clearActiveProductAction` — eigen keuze wissen mag altijd. + +**Verificatie** +- `npm run lint && npx tsc --noEmit && npm test && npm run build` groen +- Handmatig: 2 users — A archiveert product, `users.active_product_id` van B wordt `null` in DB + +--- + +## ST-903 — App-layout actief product + redirects + +**Bestanden** +- `app/(app)/layout.tsx` — uitbreiden met activeProduct-fetch + guard +- `app/(app)/solo/page.tsx` — cookie-flow vervangen +- `lib/cookies.ts` — `getLastProductCookie` / `setLastProductCookie` verwijderen +- `components/shared/nav-bar.tsx` — nieuwe prop `activeProduct` accepteren (verdere UI-uitwerking in ST-904) +- `components/solo/product-picker.tsx` — checken of nog gebruikt; anders weg + +**Stappen** + +1. **`app/(app)/layout.tsx`**: + - User-query uitbreiden: + ```ts + prisma.user.findUnique({ + where: { id: session.userId }, + select: { + username: true, + email: true, + active_product_id: true, + active_product: { select: { id: true, name: true, archived: true } }, + }, + }) + ``` + - **Guard**: als `user.active_product_id` is gezet maar (`active_product === null` of `active_product.archived === true` of geen toegang via `productAccessFilter`): + - `prisma.user.update(... active_product_id: null)` server-side + - `redirect('/dashboard?notice=active-cleared')` + - `` als nieuwe prop + +2. **`app/(app)/solo/page.tsx`** — vervang volledig: + ```ts + const session = await getSession() + if (!session.userId) redirect('/login') + const user = await prisma.user.findUnique({ + where: { id: session.userId }, + select: { active_product_id: true }, + }) + if (!user?.active_product_id) redirect('/dashboard?notice=no-active') + redirect(`/products/${user.active_product_id}/solo`) + ``` + +3. **`lib/cookies.ts`**: verwijder `getLastProductCookie` en `setLastProductCookie`. Grep alle call-sites en pas aan/verwijder. + +4. **Toast-handling** (server-redirect → client toast): + - Klein client-component `` dat `useSearchParams` leest, `toast()` aanroept, querystring strippt via `router.replace(pathname)` + - Plaats in `app/(app)/dashboard/page.tsx` (of layout) — alleen geactiveerde notices afhandelen + - Twee waarden: `active-cleared` → "Je actieve product is niet meer beschikbaar."; `no-active` → "Selecteer eerst een actief product." + +**Aandachtspunten** +- Layout-guard draait per request (extra DB-query). Houd 'm in dezelfde Promise.all met de bestaande user/userRoles-fetch. +- ProductPicker-fallback verdwijnt — switcher gebeurt in ST-904 via NavBar-dropdown. +- `app/(app)/solo/page.tsx` blijft Server Component — alleen `redirect()` van `next/navigation`. +- Een vorm van de cookie-helper kan ook door andere code gebruikt worden — verifieer de grep zorgvuldig vóór je verwijdert. + +**Verificatie** +- `npm run lint && npx tsc --noEmit && npm test && npm run build` groen +- Login zonder active → NavBar krijgt `activeProduct={null}` +- Login met active → NavBar krijgt object met id/name +- Bezoek `/solo` met active → redirect naar `/products/[id]/solo` zonder cookie +- Archiveer actief product (script of via andere user) → bij volgende request layout cleart, toast op `/dashboard` + +--- + +## ST-904 — NavBar splits + disabled-states + switcher + +> Plan nog te schrijven. + +## ST-905 — Producten-scherm Activeer-knop + +> Plan nog te schrijven. + +## ST-906 — Edge cases — toegangsverlies en archivering + +> Plan nog te schrijven. + +## ST-907 — Documentatie en tests + +> Plan nog te schrijven. diff --git a/docs/scrum4me-backlog.md b/docs/scrum4me-backlog.md index ca80cce..089d3ff 100644 --- a/docs/scrum4me-backlog.md +++ b/docs/scrum4me-backlog.md @@ -552,6 +552,8 @@ Volledig plan in `.Plans/2026-04-27-m8-realtime-solo.md` (lokaal, niet gecommit) ### M9: Actief Product Backlog +**Implementatieplan:** [docs/plans/M9-active-product-backlog.md](plans/M9-active-product-backlog.md) + Eén "actief Product Backlog" per gebruiker — persistent in DB. De NavBar wordt gesplitst in **Producten** (lijst) en **Product Backlog** (PB-view van actief PB), met **Sprint** en **Solo** als aparte tabs die op het actieve PB werken. Geen actief PB → die drie tabs zijn disabled. Vervangt de bestaande `last_product`-cookieflow. - [x] **ST-901** Database — `user.active_product_id`