feat: complete E2.S5 Input → Block wiring
- Implementeer handleSubmit met API call naar /api/intent/classify - Voeg error handling en fallback naar dagnotitie toe - Integreer openBlock en addRecentAction - Update bouwplan: Epic 2 compleet (33 SP done, 46%)
This commit is contained in:
colinislit
700
docs/swift/bouwplan-swift-diagnostiek-workflow-beoordeling.md
Normal file
700
docs/swift/bouwplan-swift-diagnostiek-workflow-beoordeling.md
Normal file
@@ -0,0 +1,700 @@
|
||||
# 📋 Beoordeling Bouwplan Swift: Diagnostiek Workflow
|
||||
|
||||
**Datum:** 23-12-2024
|
||||
**Beoordelaar:** AI Code Review
|
||||
**Versie:** v1.0
|
||||
|
||||
---
|
||||
|
||||
## Executive Summary
|
||||
|
||||
**Algemene Beoordeling:** ✅ **HAALBAAR met enkele kritieke aanvullingen**
|
||||
|
||||
Het bouwplan is goed gestructureerd en bouwt slim voort op bestaande componenten. De geschatte 21 story points voor 2 weken zijn realistisch, maar er zijn enkele belangrijke technische hiaten die eerst opgelost moeten worden voordat de implementatie kan starten.
|
||||
|
||||
**Kritieke Bevindingen:**
|
||||
1. ❌ **Report type "diagnostiek" ontbreekt** in `REPORT_TYPES` enum
|
||||
2. ❌ **Swift intent types** moeten uitgebreid worden met nieuwe intents
|
||||
3. ⚠️ **Entity extraction** voor datum/tijd parsing ontbreekt in plan
|
||||
4. ⚠️ **Intake selectie** voor diagnoses niet duidelijk uitgewerkt
|
||||
5. ✅ **Hergebruik componenten** is goed geanalyseerd en realistisch
|
||||
|
||||
**Blokkerende Bevindingen (Swift Foundation - 24-12-2024):**
|
||||
6. 🔴 **E2.S5 Input → Block wiring ontbreekt** - handleSubmit is placeholder
|
||||
7. 🔴 **CanvasArea block rendering ontbreekt** - Blocks worden niet gerenderd
|
||||
8. 🔴 **P1 Blocks (E3) niet geïmplementeerd** - DagnotatieBlock, ZoekenBlock, OverdrachtBlock bestaan niet
|
||||
9. ⚠️ **Type duplicatie** - SwiftIntent in zowel types.ts als swift-store.ts
|
||||
|
||||
---
|
||||
|
||||
## 1. Compleetheid Analyse
|
||||
|
||||
### 1.1 ✅ Sterke Punten
|
||||
|
||||
**Goed Gedocumenteerd:**
|
||||
- Duidelijke epic/story breakdown met acceptatiecriteria
|
||||
- Realistische effort schattingen (21 SP voor 2 weken)
|
||||
- Goede referenties naar bestaande code
|
||||
- Testplan met concrete checklists
|
||||
- Risico analyse met mitigaties
|
||||
|
||||
**Technische Analyse:**
|
||||
- Correcte identificatie van hergebruikbare componenten
|
||||
- Wrapper pattern goed uitgelegd
|
||||
- API routes strategie is logisch
|
||||
|
||||
### 1.2 ❌ Ontbrekende Elementen
|
||||
|
||||
#### Kritiek: Report Type "diagnostiek"
|
||||
|
||||
**Probleem:**
|
||||
Het bouwplan noemt het toevoegen van report type "diagnostiek" (regel 179), maar dit type bestaat niet in de `REPORT_TYPES` enum.
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// lib/types/report.ts
|
||||
export const REPORT_TYPES = [
|
||||
'voortgang', 'observatie', 'incident', 'medicatie', 'contact',
|
||||
'crisis', 'intake', 'behandeladvies', 'vrije_notitie', 'verpleegkundig'
|
||||
] as const;
|
||||
```
|
||||
|
||||
**Impact:**
|
||||
- Database validatie zal falen bij POST `/api/reports` met type "diagnostiek"
|
||||
- Zod schema moet uitgebreid worden
|
||||
- Mogelijk database constraint check nodig
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg story toe: **E-D1.S0: Report type "diagnostiek" toevoegen** (1 SP)
|
||||
- Update `REPORT_TYPES` enum
|
||||
- Update Zod schema `CreateReportSchema`
|
||||
- Database migratie indien nodig (check constraints)
|
||||
|
||||
#### Kritiek: Swift Intent Types Uitbreiding
|
||||
|
||||
**Probleem:**
|
||||
Het bouwplan beschrijft nieuwe intent patterns, maar de `SwiftIntent` type definitie moet uitgebreid worden.
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// lib/swift/types.ts
|
||||
export type SwiftIntent = 'dagnotitie' | 'zoeken' | 'overdracht' | 'unknown';
|
||||
```
|
||||
|
||||
**Nodig:**
|
||||
```typescript
|
||||
export type SwiftIntent =
|
||||
| 'dagnotitie'
|
||||
| 'zoeken'
|
||||
| 'overdracht'
|
||||
| 'afspraak_maken' // NIEUW
|
||||
| 'rapportage' // NIEUW
|
||||
| 'diagnose_bekijken' // NIEUW
|
||||
| 'diagnose_toevoegen' // NIEUW
|
||||
| 'diagnose_wijzigen' // NIEUW
|
||||
| 'unknown';
|
||||
```
|
||||
|
||||
**Impact:**
|
||||
- TypeScript compile errors zonder deze uitbreiding
|
||||
- Block configs moeten uitgebreid worden
|
||||
- Intent classifier moet nieuwe types ondersteunen
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg toe aan E-D1.S1 en E-D2.S1:
|
||||
- Update `SwiftIntent` type
|
||||
- Update `BLOCK_CONFIGS` met nieuwe block types
|
||||
- Update `BlockType` type (exclude 'unknown')
|
||||
|
||||
#### Waarschuwing: Entity Extraction voor Datum/Tijd
|
||||
|
||||
**Probleem:**
|
||||
Het bouwplan beschrijft intent patterns die datum/tijd moeten extraheren (bijv. "morgen 10:00"), maar er is geen plan voor entity extraction van deze waarden.
|
||||
|
||||
**Voorbeeld uit plan:**
|
||||
```
|
||||
/^afspraak\s+(\w+)\s+(morgen|vandaag)\s+(\d{1,2}:\d{2})/i
|
||||
```
|
||||
|
||||
**Ontbrekend:**
|
||||
- Functie om "morgen" → Date object te converteren
|
||||
- Functie om "10:00" → tijd te parseren
|
||||
- Validatie van datum/tijd combinaties
|
||||
- Fallback naar date picker bij onduidelijke input
|
||||
|
||||
**Huidige situatie:**
|
||||
`lib/swift/entity-extractor.ts` bestaat, maar bevat alleen patient name extraction.
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg toe aan E-D1.S1:
|
||||
- `extractDateTime(input: string): { date?: Date; time?: string }`
|
||||
- Integratie met date-fns voor Nederlandse datum parsing
|
||||
- Fallback logica voor onduidelijke input
|
||||
|
||||
#### Waarschuwing: Intake Selectie voor Diagnoses
|
||||
|
||||
**Probleem:**
|
||||
Het bouwplan beschrijft diagnose toevoegen, maar `DiagnosisDetailForm` vereist een `intakeId` (encounter_id). Het plan beschrijft niet hoe dit wordt bepaald vanuit Swift context.
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// app/epd/patients/[id]/diagnose/components/diagnosis-detail-form.tsx
|
||||
// Vereist: intakes array en selectedIntakeId
|
||||
```
|
||||
|
||||
**Vragen:**
|
||||
- Moet Swift automatisch de laatste intake selecteren?
|
||||
- Moet Swift een intake selector tonen?
|
||||
- Kan diagnose zonder intake (direct encounter koppeling)?
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg toe aan E-D2.S3:
|
||||
- Beslissing: automatisch laatste intake of selector tonen
|
||||
- Documenteer in acceptatiecriteria
|
||||
- Update DiagnoseFormBlock implementatie
|
||||
|
||||
### 1.3 ⚠️ Onduidelijkheden
|
||||
|
||||
#### Appointment Modal Hergebruik
|
||||
|
||||
**Vraag:**
|
||||
Het plan zegt "80% hergebruik" van `AppointmentModal`, maar deze component is een Dialog met veel interne state. Hoe wordt dit geïntegreerd in Swift blocks?
|
||||
|
||||
**Huidige situatie:**
|
||||
- `AppointmentModal` is een volledig Dialog component
|
||||
- Swift blocks gebruiken `BlockContainer` (geen Dialog)
|
||||
- Dark theme styling moet aangepast worden
|
||||
|
||||
**Aanbeveling:**
|
||||
Clarificeer in E-D1.S2:
|
||||
- Option A: Extract form logica naar shared component, wrapper in beide contexts
|
||||
- Option B: Hergebruik AppointmentModal maar wrap in BlockContainer (mogelijk styling issues)
|
||||
- Option C: Nieuwe Swift-specifieke component met shared business logic
|
||||
|
||||
#### Encounter ID Callback Pattern
|
||||
|
||||
**Vraag:**
|
||||
Het plan zegt "encounter_id teruggeven via callback", maar hoe wordt dit gebruikt voor de volgende stap (rapportage)?
|
||||
|
||||
**Scenario:**
|
||||
1. User: "afspraak diagnostiek jan morgen 10:00"
|
||||
2. AfspraakBlock → encounter_id = "abc-123"
|
||||
3. User: "rapportage diagnostiek gesprek met jan"
|
||||
4. Hoe weet RapportageBlock dat encounter_id "abc-123" moet gebruiken?
|
||||
|
||||
**Aanbeveling:**
|
||||
Clarificeer:
|
||||
- Option A: Swift store houdt laatste encounter_id bij per patient
|
||||
- Option B: User moet encounter expliciet selecteren
|
||||
- Option C: AI fallback om encounter te matchen op datum/tijd
|
||||
|
||||
---
|
||||
|
||||
## 2. Haalbaarheid Analyse
|
||||
|
||||
### 2.1 ✅ Realistische Schattingen
|
||||
|
||||
**Story Points Breakdown:**
|
||||
- E-D1.S1: 2 SP (intent patterns) → **Realistisch**
|
||||
- E-D1.S2: 5 SP (AfspraakBlock) → **Realistisch** (met hergebruik)
|
||||
- E-D1.S3: 4 SP (RapportageBlock) → **Realistisch**
|
||||
- E-D2.S1: 2 SP (intent patterns) → **Realistisch**
|
||||
- E-D2.S2: 3 SP (DiagnoseBlock) → **Realistisch**
|
||||
- E-D2.S3: 5 SP (DiagnoseFormBlock) → **Realistisch**
|
||||
|
||||
**Totaal: 21 SP voor 2 weken = ~10 SP/week**
|
||||
Dit is haalbaar voor 1 developer met goede focus.
|
||||
|
||||
### 2.2 ⚠️ Risico's op Vertraging
|
||||
|
||||
**Hoog Risico:**
|
||||
1. **Appointment Modal Integratie** (E-D1.S2)
|
||||
- Hergebruik Dialog component in Block context kan complex zijn
|
||||
- Styling aanpassingen kunnen meer tijd kosten dan geschat
|
||||
- **Mitigatie:** Start met proof-of-concept, pas schatting aan indien nodig
|
||||
|
||||
2. **Datum/Tijd Parsing** (E-D1.S1)
|
||||
- Nederlandse datum parsing ("morgen", "volgende week") kan edge cases hebben
|
||||
- Tijdzone handling voor afspraken
|
||||
- **Mitigatie:** Gebruik date-fns met Nederlandse locale, test grondig
|
||||
|
||||
**Middel Risico:**
|
||||
1. **ICD-10 Zoeker Verbetering** (E-D2.S3)
|
||||
- Plan zegt "fuzzy search verbeteren" maar geeft geen specificaties
|
||||
- Bestaande zoeker werkt al redelijk goed
|
||||
- **Mitigatie:** Eerst testen of verbetering nodig is, anders scope verkleinen
|
||||
|
||||
2. **Encounter Koppeling** (E-D1.S3)
|
||||
- Automatische koppeling tussen afspraak en rapportage kan complex zijn
|
||||
- **Mitigatie:** Start met optionele koppeling, voeg automatische matching later toe
|
||||
|
||||
### 2.3 ✅ Goede Foundation
|
||||
|
||||
**Bestaande Componenten:**
|
||||
- ✅ AppointmentModal bestaat en is goed gestructureerd
|
||||
- ✅ ReportComposer bestaat en ondersteunt al `linkedEncounterId`
|
||||
- ✅ DiagnosisDetailForm bestaat met ICD-10 zoeker
|
||||
- ✅ Actions bestaan en zijn herbruikbaar
|
||||
- ✅ ICD-10 zoeker werkt al met fuzzy search
|
||||
|
||||
**Bestaande Infrastructuur:**
|
||||
- ✅ Swift foundation (Command Center, Intent Classification) werkt
|
||||
- ✅ Block system bestaat (`BlockContainer`)
|
||||
- ✅ API routes pattern is duidelijk
|
||||
- ✅ Supabase RLS policies zijn al geïmplementeerd
|
||||
|
||||
---
|
||||
|
||||
## 3. Technische Aanbevelingen
|
||||
|
||||
### 3.1 Kritieke Toevoegingen
|
||||
|
||||
#### Story E-D1.S0: Report Type "diagnostiek" (NIEUW - 1 SP)
|
||||
|
||||
**Beschrijving:**
|
||||
Voeg report type "diagnostiek" toe aan het systeem.
|
||||
|
||||
**Acceptatiecriteria:**
|
||||
- [ ] `REPORT_TYPES` enum bevat "diagnostiek"
|
||||
- [ ] Zod schema `CreateReportSchema` accepteert "diagnostiek"
|
||||
- [ ] Database constraint check (indien nodig)
|
||||
- [ ] QuickActions component toont "diagnostiek" optie (indien van toepassing)
|
||||
|
||||
**Technical Notes:**
|
||||
```typescript
|
||||
// lib/types/report.ts
|
||||
export const REPORT_TYPES = [
|
||||
// ... bestaande types
|
||||
'diagnostiek', // NIEUW
|
||||
] as const;
|
||||
```
|
||||
|
||||
#### Story E-D1.S1 Uitbreiding: Entity Extraction
|
||||
|
||||
**Toevoegen aan E-D1.S1:**
|
||||
- [ ] `extractDateTime(input: string)` functie
|
||||
- [ ] Nederlandse datum parsing ("morgen", "vandaag", "volgende week")
|
||||
- [ ] Tijd parsing ("10:00", "14:30")
|
||||
- [ ] Fallback naar date picker bij onduidelijke input
|
||||
|
||||
**Technical Notes:**
|
||||
```typescript
|
||||
// lib/swift/entity-extractor.ts
|
||||
export function extractDateTime(input: string): {
|
||||
date?: Date;
|
||||
time?: string;
|
||||
confidence: number;
|
||||
} {
|
||||
// Parse "morgen 10:00" → { date: tomorrow, time: "10:00" }
|
||||
// Parse "vandaag 14:30" → { date: today, time: "14:30" }
|
||||
// Return confidence voor fallback beslissing
|
||||
}
|
||||
```
|
||||
|
||||
#### Story E-D2.S3 Uitbreiding: Intake Selectie
|
||||
|
||||
**Clarificatie nodig:**
|
||||
- [ ] Beslissing: automatisch laatste intake of selector?
|
||||
- [ ] Documenteer in acceptatiecriteria
|
||||
- [ ] Implementeer gekozen aanpak
|
||||
|
||||
**Aanbeveling:**
|
||||
Automatisch laatste intake selecteren, met optie om te wijzigen:
|
||||
```typescript
|
||||
// DiagnoseFormBlock
|
||||
const intakes = await getPatientIntakes(patientId);
|
||||
const defaultIntakeId = intakes[0]?.id; // Laatste intake
|
||||
// Toon dropdown indien meerdere intakes beschikbaar
|
||||
```
|
||||
|
||||
### 3.2 Verbeteringen
|
||||
|
||||
#### Intent Patterns Verbeteren
|
||||
|
||||
**Huidige patterns zijn te specifiek:**
|
||||
```typescript
|
||||
// Te specifiek - mist veel variaties
|
||||
/^afspraak\s+(diagnostiek|behandeling)\s+(\w+)/i
|
||||
```
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg meer variaties toe:
|
||||
```typescript
|
||||
afspraak_maken: [
|
||||
// Basis patterns
|
||||
{ pattern: /^afspraak\s+(diagnostiek|behandeling)\s+(\w+)/i, weight: 1.0 },
|
||||
{ pattern: /^plan\s+(diagnostiek|behandeling)\s+(\w+)/i, weight: 0.95 },
|
||||
|
||||
// Met datum/tijd
|
||||
{ pattern: /^afspraak\s+(\w+)\s+(morgen|vandaag)\s+(\d{1,2}:\d{2})/i, weight: 1.0 },
|
||||
{ pattern: /^plan\s+(\w+)\s+(morgen|vandaag)\s+(\d{1,2}:\d{2})/i, weight: 0.95 },
|
||||
|
||||
// Zonder type (default diagnostiek)
|
||||
{ pattern: /^afspraak\s+(\w+)\s+(morgen|vandaag)/i, weight: 0.85 },
|
||||
|
||||
// Alleen "afspraak" met patient naam
|
||||
{ pattern: /^afspraak\s+(\w+)/i, weight: 0.7 },
|
||||
],
|
||||
```
|
||||
|
||||
#### API Routes Specificatie
|
||||
|
||||
**Huidige beschrijving is te vaag:**
|
||||
```typescript
|
||||
// app/api/appointments/route.ts
|
||||
export async function POST(request: NextRequest) {
|
||||
// Wrapper rond app/epd/agenda/actions.ts createEncounter
|
||||
// Retourneert encounter_id voor volgende stap
|
||||
}
|
||||
```
|
||||
|
||||
**Aanbeveling:**
|
||||
Specificeer volledige API contract:
|
||||
```typescript
|
||||
// POST /api/appointments
|
||||
// Request body:
|
||||
{
|
||||
patientId: string;
|
||||
periodStart: string; // ISO 8601
|
||||
periodEnd?: string;
|
||||
typeCode: 'diagnostiek' | 'behandeling' | ...;
|
||||
typeDisplay: string;
|
||||
classCode: 'AMB' | 'HH' | 'VR';
|
||||
classDisplay: string;
|
||||
notes?: string;
|
||||
}
|
||||
|
||||
// Response:
|
||||
{
|
||||
success: boolean;
|
||||
data?: {
|
||||
id: string; // encounter_id
|
||||
// ... andere encounter velden
|
||||
};
|
||||
error?: string;
|
||||
}
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## 4. Testplan Verbeteringen
|
||||
|
||||
### 4.1 Ontbrekende Test Cases
|
||||
|
||||
**Entity Extraction Tests:**
|
||||
- [ ] "morgen 10:00" → correcte datum + tijd
|
||||
- [ ] "vandaag 14:30" → correcte datum + tijd
|
||||
- [ ] "volgende week maandag" → correcte datum
|
||||
- [ ] "afspraak jan" → alleen patient, geen datum → fallback date picker
|
||||
- [ ] "afspraak jan morgen" → patient + datum, geen tijd → fallback time picker
|
||||
|
||||
**Encounter Koppeling Tests:**
|
||||
- [ ] Afspraak aanmaken → encounter_id opgeslagen
|
||||
- [ ] Rapportage met encounter_id → correct gekoppeld
|
||||
- [ ] Rapportage zonder encounter_id → optioneel (geen error)
|
||||
- [ ] Meerdere encounters opzelfde dag → juiste selectie
|
||||
|
||||
**Diagnose Intake Tests:**
|
||||
- [ ] Patiënt met 1 intake → automatisch geselecteerd
|
||||
- [ ] Patiënt met meerdere intakes → laatste geselecteerd
|
||||
- [ ] Patiënt zonder intakes → error/fallback
|
||||
|
||||
### 4.2 Integration Test Scenarios
|
||||
|
||||
**End-to-End Flow:**
|
||||
```
|
||||
1. "afspraak diagnostiek jan morgen 10:00"
|
||||
→ AfspraakBlock opent
|
||||
→ Patient "jan" gevonden
|
||||
→ Datum: morgen
|
||||
→ Tijd: 10:00
|
||||
→ Type: diagnostiek
|
||||
→ Opslaan → encounter_id = "abc-123"
|
||||
|
||||
2. "rapportage diagnostiek gesprek met jan"
|
||||
→ RapportageBlock opent
|
||||
→ Patient "jan" gevonden
|
||||
→ Type: diagnostiek
|
||||
→ Encounter: "abc-123" (laatste encounter van jan)
|
||||
→ Content invoeren
|
||||
→ Opslaan → report gekoppeld aan encounter
|
||||
|
||||
3. "diagnose jan"
|
||||
→ DiagnoseBlock opent
|
||||
→ Overzicht diagnoses van jan
|
||||
→ Filter: Actief
|
||||
|
||||
4. "diagnose toevoegen jan F41.1"
|
||||
→ DiagnoseFormBlock opent
|
||||
→ Patient: jan
|
||||
→ ICD-10: F41.1 (pre-filled)
|
||||
→ Intake: laatste intake (automatisch)
|
||||
→ Type: nevendiagnose (default)
|
||||
→ Opslaan → diagnose toegevoegd
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## 5. Swift Foundation Status (24-12-2024)
|
||||
|
||||
> **BELANGRIJK:** De diagnostiek workflow bouwt voort op de Swift foundation (bouwplan-swift-v1.md).
|
||||
> De foundation is nog niet compleet, waardoor de diagnostiek workflow nog niet kan starten.
|
||||
|
||||
### 5.1 Foundation Status Overzicht
|
||||
|
||||
| Epic | Status | Impact op Diagnostiek |
|
||||
|------|--------|----------------------|
|
||||
| E0 Setup & Foundation | ✅ Done | Geen blokkade |
|
||||
| E1 Command Center | ✅ Done | Geen blokkade |
|
||||
| E2 Intent Classification | ⏳ In Progress | 🔴 **BLOKKADE** |
|
||||
| E3 P1 Blocks | ⏳ To Do | 🔴 **BLOKKADE** |
|
||||
| E4 Navigation & Auth | ⏳ To Do | Geen blokkade |
|
||||
| E5 Polish & Testing | ⏳ To Do | Geen blokkade |
|
||||
|
||||
### 5.2 Blokkerende Issues
|
||||
|
||||
#### 🔴 Blokkade 1: E2.S5 Input → Block Wiring
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// components/swift/command-center/command-input.tsx:120
|
||||
const handleSubmit = async (e: React.FormEvent) => {
|
||||
// TODO: Process intent (E2)
|
||||
console.log('Submit:', inputValue);
|
||||
clearInput();
|
||||
};
|
||||
```
|
||||
|
||||
**Probleem:**
|
||||
De intent classificatie API bestaat (`/api/intent/classify`), maar wordt niet aangeroepen.
|
||||
Het resultaat wordt niet gebruikt om `openBlock()` aan te roepen.
|
||||
|
||||
**Impact:**
|
||||
Zonder deze wiring kan geen enkel block geopend worden via tekst/spraak input.
|
||||
|
||||
**Story toegevoegd aan bouwplan-swift-v1.md (v1.5):**
|
||||
```markdown
|
||||
| E2.S5 | Input → Block wiring | CommandInput submit → API → openBlock | ⏳ | E2.S4 | 2 |
|
||||
```
|
||||
|
||||
#### 🔴 Blokkade 2: CanvasArea Block Rendering
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// components/swift/command-center/canvas-area.tsx:18
|
||||
{activeBlock ? (
|
||||
<div className="text-slate-400">Block: {activeBlock}</div> // ← Placeholder!
|
||||
) : (
|
||||
<EmptyState />
|
||||
)}
|
||||
```
|
||||
|
||||
**Probleem:**
|
||||
De CanvasArea toont alleen placeholder tekst, niet de daadwerkelijke block componenten.
|
||||
Er is geen switch/case of dynamic import voor de block types.
|
||||
|
||||
**Impact:**
|
||||
Zelfs als E2.S5 werkt, worden blocks niet zichtbaar.
|
||||
|
||||
**Aanbeveling:**
|
||||
Voeg story E3.S0 toe aan bouwplan-swift-v1.md:
|
||||
```markdown
|
||||
| E3.S0 | CanvasArea block rendering | Switch/case voor block types, prefill doorgeven | ⏳ | E2.S5 | 2 |
|
||||
```
|
||||
|
||||
#### 🔴 Blokkade 3: P1 Blocks Bestaan Niet
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// components/swift/blocks/index.ts
|
||||
export { BlockContainer } from './block-container';
|
||||
|
||||
// Commented out - bestaan niet:
|
||||
// export { DagnotatieBlock } from './dagnotitie-block';
|
||||
// export { ZoekenBlock } from './zoeken-block';
|
||||
// export { OverdrachtBlock } from './overdracht-block';
|
||||
```
|
||||
|
||||
**Probleem:**
|
||||
De diagnostiek workflow voegt nieuwe blocks toe (AfspraakBlock, RapportageBlock, DiagnoseBlock),
|
||||
maar de basis P1 blocks waar deze op voortbouwen bestaan nog niet.
|
||||
|
||||
**Impact:**
|
||||
- DagnotatieBlock is nodig als referentie voor RapportageBlock
|
||||
- ZoekenBlock is nodig voor patient selectie in alle flows
|
||||
- BlockContainer bestaat wel, maar is nog niet getest met echte content
|
||||
|
||||
**Aanbeveling:**
|
||||
Implementeer eerst E3.S2 (DagnotatieBlock) als proof-of-concept voordat diagnostiek workflow start.
|
||||
|
||||
#### ⚠️ Waarschuwing: Type Duplicatie
|
||||
|
||||
**Huidige situatie:**
|
||||
```typescript
|
||||
// lib/swift/types.ts
|
||||
export type SwiftIntent = 'dagnotitie' | 'zoeken' | 'overdracht' | 'unknown';
|
||||
|
||||
// stores/swift-store.ts (DUPLICAAT)
|
||||
export type SwiftIntent = 'dagnotitie' | 'zoeken' | 'overdracht' | 'unknown';
|
||||
```
|
||||
|
||||
**Impact:**
|
||||
Bij uitbreiding van intents voor diagnostiek workflow moet dit op twee plekken bijgewerkt worden.
|
||||
Vergeten leidt tot TypeScript errors.
|
||||
|
||||
**Aanbeveling:**
|
||||
Verwijder duplicaat uit `swift-store.ts`, importeer uit `types.ts`.
|
||||
|
||||
### 5.3 Dependency Chain voor Diagnostiek
|
||||
|
||||
```
|
||||
Swift Foundation (moet eerst af)
|
||||
├── E2.S5 Input → Block wiring ← BLOKKADE
|
||||
│ └── handleSubmit() roept API aan
|
||||
│ └── openBlock() bij succes
|
||||
│
|
||||
├── E3.S0 CanvasArea block rendering ← BLOKKADE (ontbreekt in plan)
|
||||
│ └── Switch/case voor block types
|
||||
│ └── Prefill data doorgeven
|
||||
│
|
||||
├── E3.S2 DagnotatieBlock ← BLOKKADE
|
||||
│ └── Referentie voor RapportageBlock
|
||||
│ └── Test case voor block rendering
|
||||
│
|
||||
└── E3.S3 Patient Search API ← BLOKKADE
|
||||
└── Nodig voor patient selectie
|
||||
|
||||
Diagnostiek Workflow (kan dan starten)
|
||||
├── E-D1.S0 Report type "diagnostiek"
|
||||
├── E-D1.S1 Intent patterns uitbreiden
|
||||
├── E-D1.S2 AfspraakBlock
|
||||
├── E-D1.S3 RapportageBlock
|
||||
├── E-D2.S1 Diagnose intent patterns
|
||||
├── E-D2.S2 DiagnoseBlock
|
||||
└── E-D2.S3 DiagnoseFormBlock
|
||||
```
|
||||
|
||||
### 5.4 Geschatte Effort voor Foundation Completion
|
||||
|
||||
| Story | SP | Status |
|
||||
|-------|----|----|
|
||||
| E2.S5 Input → Block wiring | 2 | ⏳ Nieuw toegevoegd |
|
||||
| E3.S0 CanvasArea rendering | 2 | ⏳ Aanbevolen |
|
||||
| E3.S2 DagnotatieBlock | 5 | ⏳ Gepland |
|
||||
| E3.S3 Patient Search API | 3 | ⏳ Gepland |
|
||||
| **Subtotaal** | **12** | |
|
||||
|
||||
**Totale effort voor werkende demo:**
|
||||
- Swift Foundation completion: 12 SP
|
||||
- Diagnostiek Workflow: 22 SP
|
||||
- **Totaal: 34 SP**
|
||||
|
||||
---
|
||||
|
||||
## 6. Conclusie & Aanbevelingen
|
||||
|
||||
### 6.1 Algemene Beoordeling
|
||||
|
||||
**Compleetheid Diagnostiek Plan:** ⚠️ **7/10**
|
||||
- Goede structuur en breakdown
|
||||
- Enkele kritieke technische details ontbreken
|
||||
- Entity extraction niet volledig uitgewerkt
|
||||
|
||||
**Compleetheid Swift Foundation:** ⚠️ **5/10**
|
||||
- E0-E2.S4 solide geïmplementeerd
|
||||
- E2.S5 wiring ontbreekt (kritiek)
|
||||
- E3 blocks niet gebouwd (kritiek)
|
||||
- CanvasArea rendering niet in plan
|
||||
|
||||
**Haalbaarheid:** ⚠️ **6/10** (was 8/10)
|
||||
- Foundation moet eerst af voordat diagnostiek kan starten
|
||||
- Extra 12 SP nodig voor foundation completion
|
||||
- Totale effort: 34 SP i.p.v. 22 SP
|
||||
|
||||
### 6.2 Aanbevelingen voor Start
|
||||
|
||||
**Voor Start:**
|
||||
1. ✅ Voeg E-D1.S0 toe: Report type "diagnostiek" (1 SP)
|
||||
2. ✅ Uitbreid E-D1.S1: Entity extraction voor datum/tijd (extra 1 SP)
|
||||
3. ✅ Clarificeer E-D2.S3: Intake selectie strategie
|
||||
4. ✅ Update `SwiftIntent` type definitie
|
||||
5. ✅ Proof-of-concept AppointmentModal integratie
|
||||
|
||||
**Tijdens Implementatie:**
|
||||
1. Start met E-D1.S0 (report type) - basis voor alles
|
||||
2. Test entity extraction grondig voordat je verder gaat
|
||||
3. Itereer op AppointmentModal integratie (mogelijk meer tijd nodig)
|
||||
4. Houd rekening met edge cases in datum parsing
|
||||
|
||||
**Na Implementatie:**
|
||||
1. Uitgebreide end-to-end tests
|
||||
2. Performance testen (intent classification snelheid)
|
||||
3. UX feedback verzamelen
|
||||
4. Documentatie updaten
|
||||
|
||||
### 6.3 Aangepaste Story Breakdown
|
||||
|
||||
**Swift Foundation (moet eerst af):**
|
||||
- E2.S5: Input → Block wiring (2 SP) ⭐ NIEUW
|
||||
- E3.S0: CanvasArea block rendering (2 SP) ⭐ AANBEVOLEN
|
||||
- E3.S2: DagnotatieBlock (5 SP)
|
||||
- E3.S3: Patient Search API (3 SP)
|
||||
- **Subtotaal Foundation: 12 SP**
|
||||
|
||||
**Epic D1 — Afspraak & Rapportage (12 SP totaal):**
|
||||
- E-D1.S0: Report type "diagnostiek" toevoegen (1 SP) ⭐ NIEUW
|
||||
- E-D1.S1: Intent patterns + entity extraction (3 SP) ⬆️ +1 SP
|
||||
- E-D1.S2: AfspraakBlock (5 SP)
|
||||
- E-D1.S3: RapportageBlock uitbreiden (4 SP)
|
||||
|
||||
**Epic D2 — Diagnose Beheer (10 SP totaal):**
|
||||
- E-D2.S1: Intent patterns uitbreiden (2 SP)
|
||||
- E-D2.S2: DiagnoseBlock (3 SP)
|
||||
- E-D2.S3: DiagnoseFormBlock (5 SP)
|
||||
|
||||
**Totalen:**
|
||||
- Swift Foundation completion: 12 SP
|
||||
- Diagnostiek Workflow: 22 SP
|
||||
- **Totaal: 34 SP**
|
||||
|
||||
### 6.4 Finale Oordeel
|
||||
|
||||
⚠️ **HOLD - Foundation eerst afronden**
|
||||
|
||||
Het diagnostiek bouwplan is goed gestructureerd, maar kan nog niet starten omdat de Swift foundation incompleet is.
|
||||
|
||||
**Kritieke blokkades:**
|
||||
1. E2.S5 (handleSubmit wiring) - toegevoegd aan bouwplan v1.5
|
||||
2. E3.S0 (CanvasArea rendering) - moet nog toegevoegd worden
|
||||
3. E3.S2 (DagnotatieBlock) - moet gebouwd worden als referentie
|
||||
4. E3.S3 (Patient Search API) - nodig voor alle patient selectie
|
||||
|
||||
**Aanbevolen volgorde:**
|
||||
```
|
||||
Week 1: Foundation completion (12 SP)
|
||||
├── E2.S5 Input → Block wiring
|
||||
├── E3.S0 CanvasArea rendering
|
||||
├── E3.S2 DagnotatieBlock
|
||||
└── E3.S3 Patient Search API
|
||||
|
||||
Week 2-3: Diagnostiek Workflow (22 SP)
|
||||
├── E-D1.S0 Report type
|
||||
├── E-D1.S1 Intent patterns
|
||||
├── E-D1.S2 AfspraakBlock
|
||||
├── E-D1.S3 RapportageBlock
|
||||
├── E-D2.S1 Diagnose patterns
|
||||
├── E-D2.S2 DiagnoseBlock
|
||||
└── E-D2.S3 DiagnoseFormBlock
|
||||
```
|
||||
|
||||
**GO voor implementatie** zodra foundation stories zijn afgerond.
|
||||
|
||||
---
|
||||
|
||||
**Versiehistorie:**
|
||||
|
||||
| Versie | Datum | Auteur | Wijziging |
|
||||
|--------|-------|--------|-----------|
|
||||
| v1.0 | 23-12-2024 | AI Review | Initiële beoordeling |
|
||||
| v1.1 | 24-12-2024 | Claude | Swift Foundation status toegevoegd (sectie 5), blokkerende issues geïdentificeerd, finale oordeel aangepast naar HOLD |
|
||||
|
||||
Reference in New Issue
Block a user