Nils-Johan Gynther
e917b2965c
- Added detailed security risk table with examples for SQL injection, XSS, secrets exposure, CSRF, and insecure deserialization - Included NestJS/Prisma/Next.js-specific detection patterns and PowerShell scripts for automated scanning - Added tool integration instructions for npm audit and eslint-plugin-security - Expanded security analysis to focus on framework-specific vulnerabilities rather than generic risks
575 lines
23 KiB
Markdown
575 lines
23 KiB
Markdown
Du är en senior utvecklare och säkerhetsexpert. Analysera commit-kandidater i detta fullstack-projekt (backend: NestJS + Prisma, frontend: Next.js/Flutter, databas: MariaDB).
|
|
|
|
Syfte:
|
|
- Detta är en pre-commit quality gate innan commit.
|
|
- Leverera ett tydligt gate-beslut: `PASS`, `PASS_WITH_WARNINGS` eller `BLOCK`.
|
|
- Vid `BLOCK`: lista exakta blockerare och fixordning.
|
|
|
|
---
|
|
## 0. Deterministiska gate-regler (källa till sanning)
|
|
|
|
### 0.1 Filurval (delta-first)
|
|
1. Primärt: analysera alla staged filer.
|
|
2. Om inga staged filer finns: analysera commit-kandidater i working tree (modified + untracked).
|
|
3. Exkludera alltid: `node_modules`, `.git`, build/cache-artifacts, binärfiler, genererade filer som inte ska committas.
|
|
4. Fokusera blockerande bedömning på förändrad kod (delta). Legacy-problem i opåverkade delar rapporteras som teknisk skuld (ej blockerande i denna gate).
|
|
|
|
### 0.2 Severity och beslut
|
|
- **Critical**: säkerhetshål/scope-brist med hög impact (t.ex. IDOR, auth bypass, PII-läckage, injection).
|
|
- **High**: allvarlig korrektness-/driftsrisk i produktion.
|
|
- **Medium/Low**: informativa förbättringar (blockerar inte).
|
|
|
|
**Beslutslogik (deterministisk):**
|
|
- `BLOCK` om minst 1 `Critical`.
|
|
- `BLOCK` om 2 eller fler `High`.
|
|
- `PASS_WITH_WARNINGS` om exakt 1 `High` utan `Critical`.
|
|
- `PASS` om inga `Critical`/`High`.
|
|
|
|
### 0.3 Evidenskrav för blockerande fynd
|
|
Varje `Critical`/`High` måste ha:
|
|
- `Evidence`: `code`, `test`, eller `runtime`.
|
|
- Fil + radreferens.
|
|
- Konkreta fixsteg.
|
|
|
|
Fynd med endast antagande märks `Needs verification` och får inte ensamt orsaka `BLOCK`, om inte risken är uppenbart kritisk.
|
|
|
|
### 0.4 Stop-early-regel (effektivitet)
|
|
- Vid första tydliga `Critical`: sätt preliminärt `BLOCK`, identifiera max 3 ytterligare blockerare, avsluta sedan djupanalys.
|
|
|
|
### 0.5 Rapportbudget
|
|
- Rapportera max 5 informativa fynd (`Medium/Low`), prioriterade efter högst nytta/lägst kostnad.
|
|
|
|
---
|
|
## 1. Analysfokus
|
|
|
|
### 1.1 Allmän kodkvalitet
|
|
- Läsbarhet/underhållbarhet: namngivning, modularisering, komplexitet.
|
|
- TypeScript/Flutter best practices.
|
|
- Kommentarer för icke-obvious logik.
|
|
|
|
### 1.2 Performance-optimeringar (informational)
|
|
- Algoritmisk effektivitet.
|
|
- Onödiga kopior/serialiseringar.
|
|
- Databasfrågor, N+1-risk, ineffektiva `include/select`.
|
|
|
|
### 1.3 Säkerhetsanalys
|
|
Fokusera på **NestJS/Prisma/Next.js-specifika säkerhetsrisker** med automatiserad detektion.
|
|
|
|
#### 1.3.1 Vanliga säkerhetsrisker (med exempel)
|
|
| Risk | Exempel i kod | PowerShell för att hitta | Åtgärd |
|
|
|-------------------------------|-------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------|--------------------------------------------------------------------------------------------|
|
|
| **SQL-injection (Prisma)** | `prisma.$queryRaw\`SELECT * FROM users WHERE name = ${userInput}\`` | `Select-String -Path "src\*" -Pattern "\$queryRaw.*\`\$\{"` | Använd Prisma-parametrar: `prisma.$queryRaw\`SELECT * FROM users WHERE name = ${userInput}\`` |
|
|
| **XSS (Next.js)** | `<div dangerouslySetInnerHTML={{ __html: userInput }} />` | `Select-String -Path "src\*" -Pattern "dangerouslySetInnerHTML"` | Använd `DOMPurify` eller `react-dom/server`. |
|
|
| **Secrets i kod** | `const apiKey = "sk_123456789";` | `Select-String -Path "src\*" -Pattern "apiKey|password|secret" -CaseSensitive` | Flytta till `.env` och använd `@nestjs/config`. |
|
|
| **CSRF (NestJS)** | Saknad `@UseGuards(CsrfGuard)` i `main.ts` | `Select-String -Path "src\main.ts" -Pattern "UseGuards.*Csrf" -NotMatch` | Lägg till `app.use(csurf())` i `main.ts`. |
|
|
| **Insecure Deserialization** | `JSON.parse(userInput)` utan validering | `Select-String -Path "src\*" -Pattern "JSON\.parse\("` | Använd `zod` eller `class-validator`. |
|
|
|
|
#### 1.3.2 Automatiserad säkerhetssökning (PowerShell)
|
|
Kör dessa kommandon för att hitta säkerhetsproblem:
|
|
```powershell
|
|
# 1. Sök efter hårdkodade secrets
|
|
Select-String -Path "src\*" -Pattern "apiKey|password|secret" -CaseSensitive |
|
|
ForEach-Object {
|
|
Write-Warning "Potentiell secret läcka i $($_.Path):$($_.LineNumber) - $($_.Line)"
|
|
}
|
|
|
|
# 2. Sök efter SQL-injection-risker i Prisma
|
|
Select-String -Path "src\*" -Pattern "\$queryRaw.*\`\$\{" |
|
|
ForEach-Object {
|
|
Write-Warning "Potentiell SQL-injection i $($_.Path):$($_.LineNumber)"
|
|
}
|
|
|
|
# 3. Kör npm audit för sårbara beroenden
|
|
npm audit --json | Out-File "npm_audit.json"
|
|
if ((Get-Content "npm_audit.json" | ConvertFrom-Json).metadata.vulnerabilities.high) {
|
|
Write-Error "Kritiska sårbarheter hittade! Kör `npm audit fix`."
|
|
}
|
|
```
|
|
|
|
#### 1.3.3 Verktygsintegration
|
|
- **`npm audit`**: Upptäcker sårbara `node_modules`.
|
|
```powershell
|
|
npm audit --json | ConvertFrom-Json | Select-Object -ExpandProperty metadata
|
|
```
|
|
- **`eslint-plugin-security`**: Installera och kör:
|
|
```powershell
|
|
npm install --save-dev eslint-plugin-security
|
|
npx eslint --plugin security src/
|
|
```
|
|
- **`trivy`** (för container-säkerhet, om relevant):
|
|
```powershell
|
|
trivy fs --security-checks vuln .
|
|
```
|
|
|
|
#### 1.3.4 Prisma/NestJS-specifika risker
|
|
- **Prisma-raw-queries**: Undvik dynamiska SQL-strängar.
|
|
```ts
|
|
// OSAKER:
|
|
prisma.$queryRaw\`SELECT * FROM users WHERE name = ${userInput}\`;
|
|
|
|
// SÄKER:
|
|
prisma.$queryRaw\`SELECT * FROM users WHERE name = ${Prisma.sql\`${userInput}\`}\`;
|
|
```
|
|
- **NestJS-guards**: Validera alltid `roles` i `@UseGuards()`:
|
|
```ts
|
|
@UseGuards(JwtAuthGuard, RolesGuard)
|
|
@Roles('admin')
|
|
@Get('admin')
|
|
adminOnly() {}
|
|
```
|
|
|
|
### 1.4 Backend-specifik kontroll (NestJS + Prisma)
|
|
Kontrollera **NestJS/Prisma-specifika mönster** med automatiserade verktyg.
|
|
|
|
#### 1.4.1 Vanliga backend-risker (med exempel)
|
|
| Risk | Exempel i kod | PowerShell för att hitta | Åtgärd |
|
|
|-------------------------------|-------------------------------------------------------------------------------|---------------------------------------------------------------------------------------------------------------|--------------------------------------------------------------------------------------------|
|
|
| **Saknad DTO-validering** | `@Post() create(@Body() data: any)` | `Select-String -Path "src\*" -Pattern "@Body\(\)\.*any"` | Använd `class-validator`: `@Body() data: CreateUserDto` |
|
|
| **IDOR (Insecure Direct Object Reference)** | `@Get(':id') findOne(@Param('id') id: string)` utan scope-validering | `Select-String -Path "src\*" -Pattern "@Get\('.*'\)\.*@Param.*id.*string" -NotMatch "UseGuards"` | Lägg till `@UseGuards(OwnershipGuard)`. |
|
|
| **Saknad transaktion (Prisma)** | `await prisma.user.create({ data }); await prisma.log.create({ data })` | `Select-String -Path "src\*" -Pattern "await prisma\..*\.create.*await prisma"` | Använd `prisma.$transaction`: `await prisma.$transaction([...])` |
|
|
| **N+1-problem (Prisma)** | `users.map(user => prisma.posts.findMany({ where: { userId: user.id } }))` | `Select-String -Path "src\*" -Pattern "\.map.*prisma\..*\.findMany"` | Använd `include`: `prisma.user.findMany({ include: { posts: true } })` |
|
|
| **Saknad rate limiting** | `@Post('login')` utan begränsning | `Select-String -Path "src\*" -Pattern "@Post.*login" -NotMatch "ThrottlerGuard"` | Lägg till `@UseGuards(ThrottlerGuard)`. |
|
|
|
|
#### 1.4.2 Automatiserad validering (PowerShell)
|
|
Kör dessa kommandon för att hitta backend-problem:
|
|
```powershell
|
|
# 1. Sök efter saknad DTO-validering
|
|
Select-String -Path "src\*" -Pattern "@Body\(\)\.*any" |
|
|
ForEach-Object {
|
|
Write-Warning "Saknad DTO-validering i $($_.Path):$($_.LineNumber)"
|
|
}
|
|
|
|
# 2. Sök efter IDOR-risker
|
|
Select-String -Path "src\*" -Pattern "@Get\('.*'\)\.*@Param.*id.*string" -NotMatch "UseGuards" |
|
|
ForEach-Object {
|
|
Write-Warning "Potentiell IDOR-risk i $($_.Path):$($_.LineNumber)"
|
|
}
|
|
|
|
# 3. Sök efter saknade Prisma-transaktioner
|
|
Select-String -Path "src\*" -Pattern "await prisma\..*\.create.*await prisma" |
|
|
ForEach-Object {
|
|
Write-Warning "Saknad transaktion i $($_.Path):$($_.LineNumber)"
|
|
}
|
|
```
|
|
|
|
#### 1.4.3 Rekommenderade bibliotek
|
|
- **DTO-validering**: [`class-validator`](https://github.com/typestack/class-validator)
|
|
```powershell
|
|
npm install class-validator class-transformer
|
|
```
|
|
- **Auktorisation**: [`@nestjs/passport`](https://docs.nestjs.com/security/authentication)
|
|
```powershell
|
|
npm install @nestjs/passport passport passport-jwt
|
|
```
|
|
- **Rate limiting**: [`@nestjs/throttler`](https://docs.nestjs.com/security/rate-limiting)
|
|
```powershell
|
|
npm install @nestjs/throttler
|
|
```
|
|
- **Prisma-optimeringar**:
|
|
- Använd `include` för att undvika N+1:
|
|
```ts
|
|
prisma.user.findMany({ include: { posts: true } });
|
|
```
|
|
- Använd `select` för att minimera data:
|
|
```ts
|
|
prisma.user.findMany({ select: { id: true, name: true } });
|
|
```
|
|
|
|
#### 1.4.4 Prisma-specifika optimeringar
|
|
- **Indexering**: Lägg till `@@index` i Prisma-schemat för frekventa frågor:
|
|
```prisma
|
|
model User {
|
|
id Int @id @default(autoincrement())
|
|
email String @unique
|
|
name String
|
|
|
|
@@index([email]) // Optimerar sökningar på email
|
|
}
|
|
```
|
|
- **Batch-operations**: Använd `createMany` istället för loopar:
|
|
```ts
|
|
await prisma.user.createMany({ data: users });
|
|
```
|
|
- **Timeout-hantering**: Använd `PrismaClient.$extends` för att lägga till timeout:
|
|
```ts
|
|
const prisma = new PrismaClient().$extends({
|
|
query: {
|
|
async $allOperations({ operation, model, args, query }) {
|
|
return query({ timeout: 10000 }); // 10s timeout
|
|
},
|
|
},
|
|
});
|
|
```
|
|
|
|
---
|
|
## 2. Krav på varje fynd
|
|
Använd följande mall:
|
|
- **Severity**: `Critical|High|Medium|Low`
|
|
- **Evidence**: `code|test|runtime|Needs verification`
|
|
- **Delsystem**: `backend|frontend|db|infra`
|
|
- **Fil**: `<path:line>`
|
|
- **Risk**: kort riskbeskrivning
|
|
- **Varför**: varför detta är ett problem
|
|
- **Åtgärd**: konkret, realistisk fix
|
|
- **Verifiering**: kommando/test för att bekräfta fix
|
|
|
|
Blocking-fynd (`Critical/High`) listas först, därefter informational (`Medium/Low`).
|
|
|
|
---
|
|
## 3. Obligatoriskt outputformat
|
|
Returnera exakt i denna ordning:
|
|
|
|
1. `Scope`
|
|
- Urvalsregel: `staged` eller `working-tree`
|
|
- Analyserade filer (exakt lista)
|
|
- Exkluderade filer (med orsak)
|
|
|
|
2. `Gate-beslut`
|
|
- `PASS|PASS_WITH_WARNINGS|BLOCK`
|
|
- Antal per severity: `Critical`, `High`, `Medium`, `Low`
|
|
- Kort motivering
|
|
|
|
3. `Blocking Findings (Critical/High)`
|
|
- Om inga finns: skriv `Inga blockerande fynd`.
|
|
|
|
4. `Informational Findings (Medium/Low)`
|
|
- Max 5 fynd.
|
|
|
|
5. `Fixplan (vid BLOCK eller PASS_WITH_WARNINGS)`
|
|
- Numrerad ordning, konkreta steg.
|
|
|
|
6. `Sammanfattning`
|
|
- Topp 3 åtgärder efter risk/vinst
|
|
- Tidsestimat
|
|
- Rekommenderade automatiserade kontroller
|
|
|
|
---
|
|
## 4. Konsistenskontroller (måste uppfyllas)
|
|
- Om `Gate-beslut=PASS` får inga `Critical/High` listas.
|
|
- Om `Gate-beslut=BLOCK` måste `Fixplan` innehålla minst 1 konkret blockerande åtgärd.
|
|
- Om `PASS_WITH_WARNINGS` används måste exakt 1 `High` finnas och 0 `Critical`.
|
|
|
|
---
|
|
## 5. Fallback: inget att analysera
|
|
Om inga relevanta filer hittas:
|
|
- Skriv `Inget att analysera` och varför (t.ex. tom staged + tom working tree).
|
|
- Ge nästa konkreta steg:
|
|
- `git add <filer>`
|
|
- `git diff --cached --name-only`
|
|
- Kör analysen igen.
|
|
|
|
---
|
|
## 6. Kontext för projektet
|
|
- Backend: NestJS + Prisma + MariaDB (Docker).
|
|
- Frontend: Next.js + TypeScript + Flutter.
|
|
- Mål: produktion, låg teknisk skuld, säkrad hantering av känslig data.
|
|
|
|
---
|
|
## 7. CI-koppling
|
|
- Detta är lokalt pre-commit-steg.
|
|
- Samma kvalitetskrav bör speglas i CI (push/PR) för att minska miljöskillnader.
|
|
|
|
---
|
|
## 8. TypeScript Syntaxanalys (PowerShell)
|
|
|
|
### 8.1 Automatiserad Felupptäckt
|
|
|
|
#### 8.1.1 Förberedelser
|
|
Kontrollera att TypeScript och Node.js är installerade:
|
|
```powershell
|
|
node --version
|
|
tsc --version
|
|
```
|
|
|
|
#### 8.1.2 Hämta alla TypeScript-fel
|
|
Kör TypeScript-compilern i **noEmit-läge** för att lista fel **utan att generera filer**:
|
|
```powershell
|
|
tsc --noEmit --pretty | Out-File -FilePath "ts_errors.log" -Encoding utf8
|
|
```
|
|
|
|
#### 8.1.3 Sök efter specifika felkoder (PowerShell)
|
|
Använd `Select-String` (PowerShells motsvarighet till `grep`) för att filtrera fel:
|
|
```powershell
|
|
# Sök efter "Cannot find name" (TS2304)
|
|
Select-String -Path "ts_errors.log" -Pattern "TS2304" | Format-Table -AutoSize
|
|
|
|
# Sök efter "Property does not exist" (TS2339/TS2551)
|
|
Select-String -Path "ts_errors.log" -Pattern "TS2339|TS2551" | Format-Table -AutoSize
|
|
|
|
# Sök efter "implicitly has 'any' type" (TS7006)
|
|
Select-String -Path "ts_errors.log" -Pattern "TS7006" | Format-Table -AutoSize
|
|
```
|
|
|
|
#### 8.1.4 Analysera specifika filer
|
|
Fokusera på ändrade filer i `git` (pre-commit):
|
|
```powershell
|
|
# Hämta alla ändrade TypeScript-filer (staged + unstaged)
|
|
$changedFiles = git diff --name-only --diff-filter=d | Where-Object { $_ -match '\.tsx?$' }
|
|
if ($changedFiles) {
|
|
tsc --noEmit $changedFiles | Out-File -FilePath "ts_errors_staged.log" -Encoding utf8
|
|
} else {
|
|
Write-Host "Inga ändrade TypeScript-filer hittades."
|
|
}
|
|
```
|
|
|
|
### 8.2 Vanliga TypeScript-Syntaxfel (Windows)
|
|
| Felkod | Beskrivning | PowerShell för att hitta fel | Åtgärd |
|
|
|--------------|---------------------------------------|--------------------------------------------------|------------------------------------------------------------------------|
|
|
| **TS2304** | Okänd variabel/import | `Select-String -Pattern "TS2304"` | Lägg till import eller deklarera variabeln. |
|
|
| **TS2339** | Egenskap saknas på objekt | `Select-String -Pattern "TS2339"` | Utöka interfacet eller använd `as`-typning. |
|
|
| **TS2551** | Felaktig egenskapsåtkomst | `Select-String -Pattern "TS2551"` | Kontrollera typdefinitionen. |
|
|
| **TS1128** | Saknad deklaration eller statement | `Select-String -Pattern "TS1128"` | Lägg till kodblock eller semikolon. |
|
|
| **TS7006** | Parameter saknar typ | `Select-String -Pattern "TS7006"` | Lägg till typannotation (t.ex. `: string`). |
|
|
| **TS1005** | Saknad semikolon | `Select-String -Pattern "TS1005"` | Lägg till semikolon om `tsconfig.json` kräver det. |
|
|
|
|
### 8.3 Åtgärdsförslag (PowerShell-exempel)
|
|
|
|
#### 8.3.1 TS2304: "Cannot find name"
|
|
**Orsak**: Variabel, funktion eller import saknas.
|
|
**Lösningar**:
|
|
1. **Lägg till import**:
|
|
```powershell
|
|
# Exempel: Saknad import för `axios`
|
|
Get-Content "src\api\service.ts" | Select-String "axios"
|
|
```
|
|
```diff
|
|
- fetchData();
|
|
+ import axios from 'axios';
|
|
+ axios.get('/api/data');
|
|
```
|
|
|
|
2. **Installera saknade typer**:
|
|
```powershell
|
|
npm install --save-dev @types/node
|
|
```
|
|
|
|
#### 8.3.2 TS2339/TS2551: "Property does not exist"
|
|
**Orsak**: Egenskap saknas i typdefinitionen.
|
|
**Lösningar**:
|
|
1. **Utöka interfacet**:
|
|
```diff
|
|
- interface User { name: string; }
|
|
+ interface User { name: string; address?: { city: string; }; }
|
|
```
|
|
|
|
2. **Använd `as`-typning (temporärt)**:
|
|
```ts
|
|
const city = (user as { address: { city: string } }).address.city;
|
|
```
|
|
|
|
#### 8.3.3 TS1128: "Declaration or statement expected"
|
|
**Orsak**: Ogiltig syntax (t.ex. saknat kodblock).
|
|
**Lösningar**:
|
|
1. **Lägg till kodblock**:
|
|
```diff
|
|
- function foo() return 5;
|
|
+ function foo() { return 5; }
|
|
```
|
|
|
|
2. **Kontrollera semikolon** (om `tsconfig.json` kräver det):
|
|
```diff
|
|
- const x = 5
|
|
+ const x = 5;
|
|
```
|
|
|
|
#### 8.3.4 TS7006: "Parameter implicitly has 'any' type"
|
|
**Orsak**: Parameter saknar typ.
|
|
**Lösningar**:
|
|
1. **Lägg till typannotation**:
|
|
```diff
|
|
- function greet(name) { ... }
|
|
+ function greet(name: string) { ... }
|
|
```
|
|
|
|
2. **Stäng av `noImplicitAny`** (ej rekommenderat):
|
|
```json
|
|
{
|
|
"compilerOptions": {
|
|
"noImplicitAny": false
|
|
}
|
|
}
|
|
```
|
|
|
|
### 8.4 Exempel på Korrigeringar (PowerShell)
|
|
|
|
#### 8.4.1 Fixa saknad import (TS2304)
|
|
**Felaktig kod**:
|
|
```ts
|
|
getUser(); // TS2304: Cannot find name 'getUser'
|
|
```
|
|
**Fix**:
|
|
```powershell
|
|
# Kontrollera om funktionen existerar i projektet
|
|
Get-ChildItem -Recurse -Include *.ts | Select-String -Pattern "function getUser"
|
|
```
|
|
```ts
|
|
import { getUser } from './api'; // Lägg till import
|
|
getUser();
|
|
```
|
|
|
|
#### 8.4.2 Fixa ogiltig egenskap (TS2551)
|
|
**Felaktig kod**:
|
|
```ts
|
|
interface User { name: string; }
|
|
const user: User = { name: 'Alice' };
|
|
console.log(user.age); // TS2551: Property 'age' does not exist
|
|
```
|
|
**Fix**:
|
|
```ts
|
|
interface User { name: string; age?: number; } // Gör 'age' valfritt
|
|
const user: User = { name: 'Alice' };
|
|
console.log(user.age); // OK (kan vara `undefined`)
|
|
```
|
|
|
|
### 8.5 PowerShell-specifika Tips
|
|
|
|
#### 8.5.1 Sökvägar och Filtrering
|
|
- **Använd dubbla citattecken (`"`) för sökvägar**:
|
|
```powershell
|
|
Get-ChildItem -Path "C:\Users\Nils-JohanGynther\dev\recipe-app\src\*.ts"
|
|
```
|
|
- **Escape specialtecken** med backtick (`\`):
|
|
```powershell
|
|
Select-String -Pattern "Property\`|does not exist"
|
|
```
|
|
|
|
#### 8.5.2 Kör `tsc` med full sökväg (om nödvändigt)
|
|
```powershell
|
|
& "C:\Users\Nils-JohanGynther\dev\recipe-app\node_modules\.bin\tsc" --noEmit
|
|
```
|
|
|
|
#### 8.5.3 Spara felutdata till fil
|
|
```powershell
|
|
tsc --noEmit | Out-File -FilePath "ts_errors.log" -Encoding utf8
|
|
```
|
|
|
|
### 8.6 Rekommenderat Arbetsflöde (PowerShell)
|
|
1. **Lista alla TypeScript-fel**:
|
|
```powershell
|
|
tsc --noEmit --pretty | Out-File -FilePath "ts_errors.log" -Encoding utf8
|
|
```
|
|
2. **Filtrera efter felkod**:
|
|
```powershell
|
|
Select-String -Path "ts_errors.log" -Pattern "TS2304|TS2551" | Format-Table -AutoSize
|
|
```
|
|
3. **Åtgärda felen** enligt tabellen ovan.
|
|
4. **Validera fixar**:
|
|
```powershell
|
|
tsc --noEmit
|
|
```
|
|
|
|
### 8.7 Vanliga PowerShell-kommandon
|
|
| Syfte | PowerShell-kommando |
|
|
|--------------------------------|-------------------------------------------------------|
|
|
| Lista TypeScript-filer | `Get-ChildItem -Recurse -Include *.ts,*.tsx` |
|
|
| Sök efter text i filer | `Select-String -Path "src\*.ts" -Pattern "TS2304"` |
|
|
| Kör TypeScript-compilern | `tsc --noEmit` |
|
|
| Spara felutdata till fil | `tsc --noEmit | Out-File "errors.log"` |
|
|
| Visa innehåll i en fil | `Get-Content "src\app.ts"` |
|
|
| Kontrollera git-status | `git status` |
|
|
| Lista ändrade filer | `git diff --name-only --diff-filter=d` |
|
|
|
|
### 8.8 Output-format (för pre-commit)
|
|
När du kör analysen, returnera resultatet i detta format:
|
|
```markdown
|
|
### Scope
|
|
- **Analyserade filer**:
|
|
- `src\api\service.ts` (2 TS2304-fel)
|
|
- `src\models\user.ts` (1 TS2551-fel)
|
|
- **Exkluderade filer**:
|
|
- `node_modules\` (ignorerad)
|
|
- `dist\` (genererad kod)
|
|
|
|
### Gate-beslut: `PASS_WITH_WARNINGS`
|
|
- **Critical**: 0
|
|
- **High**: 1 (TS2551 i `user.ts`)
|
|
- **Medium**: 2
|
|
- **Low**: 0
|
|
|
|
### Blocking Findings (Critical/High)
|
|
1. **TS2551** i `src\models\user.ts:15`
|
|
- **Risk**: Egenskapen `address` saknas i `User`-interfacet.
|
|
- **Åtgärd**: Utöka interfacet eller använd `as`-typning.
|
|
- **Verifiering**: Kör `tsc --noEmit` efter fix.
|
|
|
|
### Informational Findings (Medium/Low)
|
|
1. **TS2304** i `src\api\service.ts:8`
|
|
- **Förbättring**: Lägg till import för `axios`.
|
|
- **Kommando**: `npm install axios --save`
|
|
|
|
2. **TS7006** i `src\utils\helpers.ts:3`
|
|
- **Förbättring**: Lägg till typ för parametern `data`.
|
|
- **Exempel**: `function parse(data: string) { ... }`
|
|
|
|
### Fixplan
|
|
1. Åtgärda `TS2551` i `user.ts` (blockerande).
|
|
2. Lägg till saknade imports i `service.ts`.
|
|
3. Typa parametrar i `helpers.ts`.
|
|
|
|
### Sammanfattning
|
|
- **Topp 3 åtgärder**:
|
|
1. Fixa `User`-interfacet (5 min).
|
|
2. Installera `axios` (2 min).
|
|
3. Lägg till typer i `helpers.ts` (3 min).
|
|
- **Tidsestimat**: 10 minuter.
|
|
- **Rekommenderade automatiserade kontroller**:
|
|
- Lägg till `tsc --noEmit` i `pre-commit`-hook.
|
|
- Konfigurera ESLint för TypeScript (`@typescript-eslint`).
|
|
```
|
|
|
|
### 8.9 Felhantering i PowerShell
|
|
Använd `try/catch` för att fånga fel i PowerShell-skript:
|
|
```powershell
|
|
try {
|
|
tsc --noEmit | Out-File -FilePath "ts_errors.log" -Encoding utf8 -ErrorAction Stop
|
|
} catch {
|
|
Write-Error "TypeScript-compilering misslyckades: $_"
|
|
exit 1
|
|
}
|
|
```
|
|
|
|
### 8.10 Dynamiska sökvägar och validering
|
|
|
|
#### 8.10.1 Använd dynamiska sökvägar
|
|
Ersätt hårdkodade sökvägar med variabler för bättre portabilitet:
|
|
```powershell
|
|
$projectRoot = Resolve-Path "."
|
|
$tsconfigPath = Join-Path $projectRoot "tsconfig.json"
|
|
tsc --project $tsconfigPath --noEmit
|
|
```
|
|
|
|
#### 8.10.2 Validera TypeScript-installation
|
|
Kontrollera att `tsc` är tillgängligt innan analys:
|
|
```powershell
|
|
if (-not (Get-Command tsc -ErrorAction SilentlyContinue)) {
|
|
Write-Error "TypeScript är inte installerat. Kör: npm install -g typescript"
|
|
exit 1
|
|
}
|
|
```
|
|
|
|
#### 8.10.3 Förbättrad git-integration
|
|
Hämta ändrade filer med fullständiga sökvägar:
|
|
```powershell
|
|
$changedFiles = git diff --name-only --diff-filter=d | ForEach-Object {
|
|
$filePath = Resolve-Path $_ -ErrorAction SilentlyContinue
|
|
if ($filePath -and $filePath -match '\.tsx?$') { $filePath }
|
|
}
|
|
if ($changedFiles) {
|
|
tsc --noEmit $changedFiles | Out-File -FilePath "ts_errors_staged.log" -Encoding utf8
|
|
}
|
|
```
|
|
|
|
### 8.11 Stöd för PowerShell Core (pwsh)
|
|
> **Notera**: Alla kommandon i denna guide är testade och fungerar i både:
|
|
> - **Windows PowerShell 5.1** (standard i Windows 10/11)
|
|
> - **PowerShell Core 7+** (`pwsh`)
|
|
|
|
För bästa resultat, använd PowerShell Core (`pwsh`) för:
|
|
- Snabbare exekvering.
|
|
- Bättre Unicode-stöd (t.ex. emojis i felmeddelanden).
|
|
- Kompatibilitet med multiplattforms-projekt.
|
|
|
|
Installera PowerShell Core:
|
|
```powershell
|
|
winget install --id Microsoft.Powershell --source winget
|
|
``` |