Prompt-filer gör code review repeterbar, konsistent och skalbar — utan att varje granskning kräver en erfaren arkitekt i rummet.
| AI kan granska | AI kan INTE granska |
|---|---|
| Arkitekturmönsterbrott (lagerbrott, felaktiga beroenden) | Runtime-beteende under lastförhållanden |
| Namnkonventionsavvikelser | Integrationsproblem mot externa system |
| SRP- och DRY-brott synliga i koden | Affärsregler som inte framgår av koden |
| Saknad nullhantering och felhantering | Prestandaflaskhalsar som kräver profiling |
| Uppenbara säkerhetsproblem (SQL injection, hardkodade nycklar) | Säkerhetsproblem i produktionskonfiguration |
| Avvikelser från instruktionsfilernas regler | Om koden uppfyller affärskraven (kräver domänkunskap) |
| Saknad testbarhet (hårdkodade beroenden, statiska metoder) | Race conditions och deadlocks som kräver runtime-observation |
---
description: "Arkitekturorienterad code review av en C#-fil"
---
# Arkitektur Code Review
Du är senior C#-arkitekt med fokus på Clean Architecture och DDD.
Granska #file:${input:Filsökväg, t.ex. src/Domain/Orders/Order.cs} mot:
- Projektreglerna i #file:.github/copilot-instructions.md
- Domänreglerna i #file:.github/instructions/domain.instructions.md
Rapportera fynd i detta format:
## Arkitekturbrott
[numrerad lista — brott mot lagerregler och beroendesriktning]
## Namnkonventioner
[numrerad lista — avvikelser från projektkonventionen]
## SRP och komplexitet
[numrerad lista — klasser/metoder med mer än ett ansvar]
## Felhantering
[numrerad lista — saknad Result<T>, nullable-problem, resurser som inte stängs]
## Testbarhet
[numrerad lista — hardkodade beroenden, statiska anrop, svårmockade beroenden]
## Sammanfattning
Ge en av: ✅ Godkänd | ⚠️ Kräver ändringar | ❌ Ej godkänd
Motivera i max 3 meningar.
Om du inte hittar fynd i en kategori, skriv "Inga fynd".---
description: "Säkerhetsgranskning av C#-kod mot OWASP Top 10"
---
# Säkerhetsgranskning
Du är en erfaren AppSec-ingenjör specialiserad på .NET och C#.
Granska #file:${input:Filsökväg} för säkerhetsproblem.
Fokusera på OWASP Top 10 relevanta för C#/.NET:
1. Injection (SQL, LINQ, kommandoinjektion)
2. Autentiserings- och sessionsbrister
3. Exponering av känslig data (hardkodade nycklar, connection strings i kod)
4. Insecure Direct Object Reference (IDOR)
5. Saknad validering av indata (input från API-parametrar, formulär)
6. Felkonfiguration av säkerhetsinställningar
7. XXE och deserialisering av otillförlitlig data
Rapportera per kategori:
- Allvarlighetsgrad: Kritisk / Hög / Medel / Låg
- Exakt rad eller metod
- Konkret åtgärd
Avsluta med: Totalt antal fynd per allvarlighetsgrad.| Granskningstyp | Rekommenderad modell | Anledning |
|---|---|---|
| Snabb stilgranskning (namnkonventioner, formatering) | GPT-4.1 / Claude Sonnet 4.6 | Snabbt, billigt, räcker för ytliga brott |
| Arkitekturgranskning (en fil, 50–200 rader) | Claude Sonnet 4.6 | Stort kontextfönster, bra kodförståelse |
| Djup arkitekturgranskning (hel modul, flera filer) | Claude Opus 4.7 eller Claude Sonnet 4.6 (extended thinking) | Kräver reasoning över komplexa beroenden |
| Säkerhetsgranskning | Claude Opus 4.7 eller Claude Sonnet 4.6 (extended thinking) | Kräver djup förståelse av attack-vektorer |
| Trade-off-analys (arkitekturval) | Claude Sonnet 4.6 (extended thinking) | Reasoning-modell bäst för jämförande analys |
GitHub Copilot har en inbyggd "Code Review"-funktion tillgänglig direkt i VS Code och på GitHub.com. Den kan:
Kombinerat med dina egna .prompt.md-granskningar får du en tvåstegsprocess: Copilots inbyggda granskning fångar det uppenbara, din anpassade prompt fångar det projektspecifika.
Lös övningarna självständigt. Det finns inget facit — lärandet sker i processen.
code-review.prompt.md baserat på mallen ovan och kör den. Hur väl matchar AI:s fynd med vad du själv ser som förbättringsområden? Notera ett fynd som förvånade dig.
.prompt.md-filer: (1) arkitekturgranskning, (2) säkerhetsgranskning, (3) testbarhetsgranskning. Kör alla tre på samma C#-fil och sammanställ resultaten i en granskningsrapport. Bedöm: täcker de tre sammanlagt det du vill granska?
.prompt.md-filer (arkitektur, säkerhet, testbarhet), en copilot-instructions.md med granskningspolicy, och en REVIEW-PROCESS.md som dokumenterar arbetsflödet. Kör systemet på minst fem befintliga C#-filer och dokumentera de viktigaste fynden.
pre-commit-skript (eller GitHub Actions-workflow) som automatiskt kör en av dina review-prompter på filer som ändrats i en commit. Skriptet ska logga resultatet och (valfritt) blockera commit vid kritiska fynd. Dokumentera implementationen och begränsningarna.