Madhura68
0a58557e9d
Story 6 van PBI "Alle dialogen conform docs/patterns/dialog.md". - batch-enqueue-blocker-dialog: entityDialog* layout-classes - task-detail-dialog: entityDialog* layout-classes (rest van interne layout blijft custom — hybride detail+blur-save view) - docs/specs/dialogs/task-detail.md — profiel dat het blur-save + PATCH-route patroon documenteert (afwijking van klassieke Server-Action+form flow) - docs/specs/dialogs/batch-enqueue-blocker.md — profiel voor informational confirm-dialog Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
65 lines
3 KiB
Markdown
65 lines
3 KiB
Markdown
---
|
|
title: "TaskDetailDialog Profiel"
|
|
status: active
|
|
audience: [ai-agent, contributor]
|
|
language: nl
|
|
last_updated: 2026-05-04
|
|
---
|
|
|
|
# TaskDetailDialog Profiel
|
|
|
|
> Volgt `docs/patterns/dialog.md`. Dit document beschrijft alleen de Solo-specifieke afwijkingen.
|
|
|
|
> **Niet te verwarren met `TaskDialog`** (`app/_components/tasks/task-dialog.tsx`) — dat is de classic create/edit-dialog voor backlog-taken. **`TaskDetailDialog`** is een solo-board-specifieke detail+edit-overlay die een lopende taak laat zien terwijl de Claude-agent eraan werkt.
|
|
|
|
## Doel
|
|
|
|
Vanuit het solo-board kan de gebruiker op een task-card klikken om:
|
|
- Het implementation_plan te lezen / bewerken (markdown, blur-save)
|
|
- Verify-instellingen te wijzigen (`verify_only`, `verify_required`)
|
|
- Claude-job-status en branch-link te zien
|
|
- De huidige taak naar Claude te sturen / te annuleren
|
|
|
|
## Velden
|
|
|
|
| Veld | Type | Validatie |
|
|
|---|---|---|
|
|
| `implementation_plan` | string \| null | max 10000, markdown, blur-save |
|
|
| `verify_only` | boolean | toggle, direct opgeslagen |
|
|
| `verify_required` | enum `'ALIGNED' \| 'ALIGNED_OR_PARTIAL' \| 'ANY'` | radio, direct opgeslagen |
|
|
|
|
## URL- of state-pattern
|
|
|
|
- Gekozen: **state-based** — `task: SoloTask | null` prop uit `solo-board`. `null` = dialog gesloten.
|
|
- Reden: solo-board is one-page; de detail-dialog is altijd in context.
|
|
|
|
## Persistence
|
|
|
|
**Geen klassiek form-submit.** Wijzigingen schrijven via `fetch('/api/tasks/:id', { method: 'PATCH' })` (route handler), getriggerd door:
|
|
- Plan-textarea: blur of debounced auto-save
|
|
- Verify-toggles: direct bij click
|
|
|
|
Dit valt buiten de standaard "Server Action met form" flow van `docs/patterns/dialog.md` § 7. Reden: het zijn fine-grained edits van een lopende taak, geen save-dan-sluit-flow.
|
|
|
|
## Drielaagse demo-policy
|
|
|
|
- **Laag 1 (proxy.ts):** `/api/tasks/[id]`-route is via `apiAuth`-helper beschermd; demo-write zou geblokkeerd moeten worden in de route handler zelf
|
|
- **Laag 2 (route handler):** `session.isDemo`-check in `app/api/tasks/[id]/route.ts` (PATCH) — verifieer dat dit aanwezig is
|
|
- **Laag 3 (UI):** `<DemoTooltip show={isDemo}>` rond plan-textarea en verify-toggles; `readOnly` resp `disabled`-state op de controls
|
|
|
|
## Layout
|
|
|
|
Gebruikt `entityDialogContentClasses` voor responsive sizing (§4 spec). Body-layout intern is custom (gegroepeerde secties in plaats van form-fields) want het is een hybride detail+edit-view, niet een klassiek form.
|
|
|
|
## Bewust NIET in v1
|
|
|
|
- ❌ **Klassiek save-dan-sluit-form** — blur-save is bewust gekozen omdat de gebruiker tussendoor de plan-tekst herziet terwijl Claude bezig is.
|
|
- ❌ **Dirty-close-guard** — niet relevant zonder klassiek submit-form; wijzigingen worden direct gepersisteerd.
|
|
- ❌ **Cmd/Ctrl+Enter shortcut** — geen submit, dus n.v.t.
|
|
- ❌ **422-fieldErrors** — fine-grained PATCH-route geeft simpele 200/400/403; geen veldgewijze rendering nodig in deze UX.
|
|
|
|
## Gerelateerde bestanden
|
|
|
|
- `components/solo/task-detail-dialog.tsx` — implementatie
|
|
- `app/api/tasks/[id]/route.ts` — PATCH-handler
|
|
- `stores/solo-store.ts` — client-state
|