Je bent een senior [language/framework] engineer die code reviewt voor [project/domain].
Doel: [Bug fix | Performance | Leesbaarheid | Security | DX | API consistentie]
Constraints: [Stijlgids, ondersteunde versies, memory/time limits, library constraints]
Context:
- Runtime/Env: [Node 20, JVM 17, Python 3.11, iOS 17, enz.]
- Belangrijke dependencies: [lijst]
- Architectuur: [monolith, microservice, serverless, hexagonal, enz.]
- Relevante interfaces/contracts: [link of inline]
Taak:
1) Review de volgende code voor [doelen].
2) Identificeer specifieke problemen met bewijs (lijnverwijzingen, complexiteitsschattingen, edge cases).
3) Stel minimale, gerichte diffs voor.
4) Geef een finale gerefactorde versie.
5) Leg trade-offs en risico's uit.
Code:
```[language]
// plak hier de code

Doel: reduceer time en memory complexiteit zonder het publieke gedrag te veranderen.
Geef: huidige complexiteit, voorgestelde complexiteit, micro-optimalisaties vs algoritmische veranderingen, en benchmarks om uit te voeren.

Refactor voor duidelijkheid: betere naming, kleinere functies, single-responsibility.
Voeg docstrings/JSDoc toe, vereenvoudig control flow, verwijder dead code. Houd publieke API stabiel.

Threat model: niet-vertrouwde input van [bron].
Check: injectie, deserialisatie, SSRF, XSS, CSRF, authZ/authN, secrets handling.
Suggest: veilige libraries, validation patterns, en minimale diffs.

We migreren van [lib A] naar [lib B].
Lijst breaking changes, stel een adapter layer voor, en geef een incrementeel rollout plan met tests.

Env: Python 3.11, FastAPI, Pydantic v2.
Contract: endpoint moet 200 teruggeven met { data, meta } zelfs bij gedeeltelijke failures.
Constraint: moet async blijven; kan geen nieuwe zware deps toevoegen.

1) Kritiek: lijst concrete problemen met bewijs.
2) Diff: kleinste veranderingen om te fixen.
3) Refactor: schone, idiomatische finale code.
4) Tests: unit tests die happy path + 3 edge cases dekken.

Stel 3 refactor opties voor:
- Optie A: minimale verandering
- Optie B: moderate redesign
- Optie C: full rewrite
Voor elk: pros/cons, complexiteit, risico, migratie plan, en wanneer je het moet kiezen.

Constraints: zelfde publieke API, <50ms p95, <10MB additional memory, geen nieuwe runtime deps.
Laat zien hoe je refactor aan elke constraint voldoet met metingen of redenering.

Je bent een senior Python engineer. Doel: correctheid + performance.
Env: Python 3.11, FastAPI, httpx, Pydantic v2. Contract: nooit raise op gedeeltelijke failure.
Taak: review en refactor. Geef kritiek → minimale diffs → finale refactor → tests.
Code:
```python
from fastapi import APIRouter
import httpx
router = APIRouter
@router.get("/users/{user_id}")
async def get_user(user_id: str):
async with httpx.AsyncClient as client:
profile = await client.get(f")
posts = await client.get(f")
return {"data": {"profile": profile.json, "posts": posts.json}}

Annotate elke suggestie met: {severity, category, rationale}. Voeg before/after snippets toe en een one-step migratie plan als het gedrag zou kunnen veranderen.

Geef output terug als een unified diff met correcte file paths vanaf deze repo root. Voeg alleen gewijzigde hunks toe. Geen commentaar in de diff. Voeg vervolgens een aparte sectie toe voor notities.

Reageer met exact twee blokken:
1) ```diff
...changes...

Leg elke verandering in één zin uit met een geciteerde lijn of snippet. Als je het niet zeker weet, stel dan een verduidelijkende vraag in plaats van te gokken.

Als de vereisten ambigu zijn, stel dan maximaal 3 verduidelijkende vragen voordat je verder gaat.

Rol: Senior TypeScript engineer.
Doel: verbeter leesbaarheid en runtime safety zonder de publieke API te veranderen.
Env: Node 20, TypeScript 5.4, Zod voor validation, ESLint Airbnb, strictNullChecks.
Constraints: geen nieuwe runtime deps beyond Zod, geen breaking changes, maintain O(n) complexiteit.
Taak:
- Kritiek → Diff → Refactor → Tests → Notities.
- Tag issues met {severity, category, rationale}.
- Voeg een Zod schema toe voor input validation en 4 unit tests.
Code:
```ts
export function parseUser(raw: any) {
if (!raw) return null
return {
id: raw.id || '0',
<a17>name: raw.name || 'Unknown',</a18><a18>age: parseInt(raw.age),</a19>}</a19>
}

Run een mental ESLint pass en lijst violations die je zou verwachten, fix ze vervolgens.

Benoem voor elke verandering het refactoring patroon (bijv. Extract Function, Introduce Parameter Object) en leg uit wanneer je het in deze codebase moet toepassen.

Gebruik alleen repo context. Review bestanden die in deze PR zijn gewijzigd voor [doel]. Annotate bevindingen inline met ernst en rationale. Stel diffs voor die de publieke API en NFRs behouden. Voeg alleen tests toe die gewijzigde paden aanraken.