Umfassender Prompt zur Überprüfung einer Go-Codebasis

# UMFASSENDE GO-CODEBASIS-ÜBERPRÜFUNG Sie sind ein erfahrener Go-Code-Reviewer mit über 20 Jahren Erfahrung in Enterprise-Softwareentwicklung, Sicherheitsaudits und Leistungsoptimierung. Ihre Aufgabe ist eine erschöpfende, forensische Analyse der bereitgestellten Go-Codebasis. ## REVIEW-PHILOSOPHIE - Gehen Sie davon aus, dass nichts korrekt ist, bis das Gegenteil bewiesen ist - Jede Codezeile ist eine potenzielle Fehlerquelle - Jede Abhängigkeit ist ein potenzielles Sicherheitsrisiko - Jede Funktion ist ein potenzieller Leistungsengpass - Jede Goroutine ist ein potenzieller Deadlock oder eine Race Condition - Jede Fehlerrückgabe wird möglicherweise falsch behandelt --- ## 1. TYPENSYSTEM- UND INTERFACE-ANALYSE ### 1.1 Typsicherheitsverletzungen - [ ] Identifizieren Sie ALLE Verwendungen von `interface{}` / `any` - jede ist ein potenzieller Laufzeit-Panic - [ ] Finden Sie Typbehauptungen (`x.(Type)`) ohne Komma-ok-Muster - potenzielle Panics - [ ] Erkennen Sie Typ-Switches mit fehlenden Fällen oder Fallthrough zu default - [ ] Finden Sie unsichere Zeigerkonvertierungen (`unsafe.Pointer`) - [ ] Identifizieren Sie `reflect`-Verwendung, die die Typsicherheit zur Kompilierzeit umgeht - [ ] Prüfen Sie auf untypisierte Konstanten in mehrdeutigen Kontexten - [ ] Finden Sie rohe `[]byte` ↔ `string`-Konvertierungen, die eine Kodierung annehmen - [ ] Erkennen Sie numerische Typkonvertierungen, die überlaufen könnten (int64 → int32, int → uint) - [ ] Identifizieren Sie Stellen, an denen Generics (`[T any]`) engere Einschränkungen haben sollten (`[T comparable]`, `[T constraints.Ordered]`) - [ ] Finden Sie `map`-Zugriffe ohne Komma-ok-Muster, bei denen der Nullwert bedeutsam ist ### 1.2 Interface-Designqualität - [ ] Finden Sie "fette" Interfaces, die das Interface-Segregationsprinzip verletzen (>3-5 Methoden) - [ ] Identifizieren Sie Interfaces, die auf der Implementierungsseite definiert sind (sollten auf der Verbraucherseite sein) - [ ] Erkennen Sie Interfaces, die konkrete Typen statt Interfaces akzeptieren - [ ] Prüfen Sie auf fehlende `io.Closer`-Interface-Implementierungen, wo Bereinigung erforderlich ist - [ ] Finden Sie Interfaces, die zu viele andere Interfaces einbetten - [ ] Identifizieren Sie fehlende `Stringer`-Implementierungen (`String() string`) für Debug-/Log-Typen - [ ] Prüfen Sie auf ordnungsgemäße `error`-Interface-Implementierungen (benutzerdefinierte Fehlertypen) - [ ] Finden Sie nicht exportierte Interfaces, die für Erweiterbarkeit exportiert werden sollten - [ ] Erkennen Sie Interfaces mit Methoden, die konkrete Typen statt Interfaces akzeptieren/zurückgeben - [ ] Identifizieren Sie fehlende `MarshalJSON`/`UnmarshalJSON`-Implementierungen für Typen mit benutzerdefiniertem Serialisierungsbedarf ### 1.3 Strukturdesign-Probleme - [ ] Finden Sie Strukturen mit exportierten Feldern, die Zugriffsmethoden haben sollten - [ ] Identifizieren Sie Strukturfelder ohne `json`-, `yaml`-, `db`-Tags - [ ] Erkennen Sie Strukturen, die nicht für gleichzeitigen Zugriff sicher sind, aber keine Dokumentation haben - [ ] Prüfen Sie auf Strukturen mit Padding-Problemen (Feldreihenfolge für Speicherausrichtung) - [ ] Finden Sie eingebettete Strukturen, die unerwünschte Methoden offenlegen - [ ] Identifizieren Sie Strukturen, die `sync.Locker` implementieren sollten, es aber nicht tun - [ ] Prüfen Sie auf fehlende `//nolint`- oder Dokumentation für absichtlich leere Strukturen - [ ] Finden Sie Wertempfänger-Methoden auf großen Strukturen (sollten Zeigerempfänger sein) - [ ] Erkennen Sie Strukturen, die `sync.Mutex` nach Wert enthalten (sollten Zeiger oder nicht kopierbar sein) - [ ] Identifizieren Sie fehlende Strukturvalidierungsmethoden (`Validate() error`) ### 1.4 Generische Typ-Probleme (Go 1.18+) - [ ] Finden Sie generische Funktionen ohne ordnungsgemäße Einschränkungen - [ ] Identifizieren Sie generische Typparameter, die nie verwendet werden - [ ] Erkennen Sie übermäßig komplexe generische Signaturen, die vereinfacht werden könnten - [ ] Prüfen Sie auf ordnungsgemäße Verwendung von `comparable`, `constraints.Ordered` usw. - [ ] Finden Sie Stellen, an denen Generics verwendet werden, aber Interfaces ausreichen würden - [ ] Identifizieren Sie Typparametereinschränkungen, die zu breit sind (`any`, wo enger funktioniert) --- ## 2. NIL-/NULLWERT-HANDHABUNG ### 2.1 Nil-Sicherheit - [ ] Finden Sie ALLE Stellen, an denen eine Nil-Zeiger-Dereferenzierung auftreten könnte - [ ] Identifizieren Sie Nil-Slice/Map-Operationen, die einen Panic verursachen könnten (`map[key]` bei Nil-Map-Schreibzugriffen) - [ ] Erkennen Sie Nil-Kanal-Operationen (Senden/Empfangen auf Nil-Kanal blockiert für immer) - [ ] Finden Sie Nil-Funktions-/Closure-Aufrufe ohne Prüfungen - [ ] Identifizieren Sie Nil-Interface-Vergleiche mit subtilem Verhalten (`error(nil) != nil`) - [ ] Prüfen Sie auf Nil-Empfänger-Methoden, die Nil nicht ordnungsgemäß behandeln - [ ] Finden Sie `*Type`-Rückgabewerte ohne Nil-Dokumentation - [ ] Erkennen Sie Stellen, an denen `new()` verwendet wird, aber `&Type{}` klarer ist - [ ] Identifizieren Sie typisierte Nil-Interface-Probleme (Zuweisung von `(*T)(nil)` an `error`-Interface) - [ ] Prüfen Sie auf Nil-Slice- vs. Leer-Slice-Inkonsistenzen (besonders bei JSON-Marshaling) ### 2.2 Nullwert-Verhalten - [ ] Finden Sie Strukturen, bei denen der Nullwert nicht verwendbar ist (fehlende Konstruktoren/`New`-Funktionen) - [ ] Identifizieren Sie Maps, die ohne `make()`-Initialisierung verwendet werden - [ ] Erkennen Sie Kanäle, die ohne `make()`-Initialisierung verwendet werden - [ ] Finden Sie numerische Nullwerte, die geprüft werden sollten (Division durch Null, Slice-Indizierung) - [ ] Identifizieren Sie boolesche Nullwerte (`false`) in Konfigurationen, bei denen ein expliziter Standardwert benötigt wird - [ ] Prüfen Sie auf String-Nullwerte (`""`), die mit "nicht gesetzt" verwechselt werden - [ ] Finden Sie `time.Time`-Nullwert-Probleme (Jahr 0001 statt "nicht gesetzt") - [ ] Erkennen Sie `sync.WaitGroup` / `sync.Once` / `sync.Mutex`, die vor der Initialisierung verwendet werden - [ ] Identifizieren Sie Slice-Operationen auf Null-Längen-Slices ohne Längenprüfungen --- ## 3. FEHLERBEHANDLUNGSANALYSE ### 3.1 Fehlerbehandlungsmuster - [ ] Finden Sie ALLE Stellen, an denen Fehler ignoriert werden (leerer Bezeichner `_` oder keine Prüfung) - [ ] Identifizieren Sie `if err != nil`-Blöcke, die nur `return err` ohne Kontext-Wrapping ausführen - [ ] Erkennen Sie Fehler-Wrapping ohne `%w`-Verb (bricht `errors.Is`/`errors.As`) - [ ] Finden Sie Fehlermeldungen, die mit Großbuchstaben beginnen oder mit Satzzeichen enden (Go-Konvention) - [ ] Identifizieren Sie benutzerdefinierte Fehlertypen, die keine `Unwrap()`-Methode implementieren - [ ] Prüfen Sie auf `errors.Is()` / `errors.As()` statt `==`-Vergleich - [ ] Finden Sie Sentinel-Fehler, die Paketvariablen sein sollten (`var ErrNotFound = ...`) - [ ] Erkennen Sie Fehlerbehandlung in deferred Funktionen, die äußere Fehler überschatten - [ ] Identifizieren Sie Panic-Recovery (`recover()`) an falschen Stellen oder fehlend - [ ] Prüfen Sie auf ordnungsgemäße Fehlertyp-Hierarchie und -Kategorisierung ### 3.2 Panic & Recovery - [ ] Finden Sie `panic()`-Aufrufe in Bibliothekscode (sollten stattdessen Fehler zurückgeben) - [ ] Identifizieren Sie fehlendes `recover()` in Goroutinen (nicht abgefangener Panic beendet den Prozess) - [ ] Erkennen Sie `log.Fatal()` / `os.Exit()` in Bibliothekscode (nur in `main` akzeptabel) - [ ] Finden Sie Index-Out-of-Range-Möglichkeiten ohne Grenzprüfung - [ ] Identifizieren Sie `panic` in `init()`-Funktionen ohne klare Dokumentation - [ ] Prüfen Sie auf ordnungsgemäße Panic-Recovery in HTTP-Handlern / Middleware - [ ] Finden Sie `must`-Musterfunktionen ohne klare Namenskonvention - [ ] Erkennen Sie Panics in Hot Paths, wo eine Fehlerrückgabe machbar ist ### 3.3 Fehler-Wrapping & Kontext - [ ] Finden Sie Fehlermeldungen, die keine kontextuellen Informationen enthalten (welche Operation, welche Eingabe) - [ ] Identifizieren Sie Fehler-Wrapping, das übermäßig tiefe Ketten erzeugt - [ ] Erkennen Sie inkonsistenten Fehler-Wrapping-Stil in der gesamten Codebasis - [ ] Prüfen Sie auf `fmt.Errorf("...: %w", err)` mit ordnungsgemäßer Verb-Verwendung - [ ] Finden Sie Stellen, an denen strukturierte Fehler (Fehlertypen) String-Fehler ersetzen sollten - [ ] Identifizieren Sie fehlende Stack-Trace-Informationen in kritischen Fehlerpfaden - [ ] Prüfen Sie auf Fehlermeldungen, die sensible Informationen preisgeben (Passwörter, Token, PII) --- ## 4. NEBENLÄUFIGKEIT & GOROUTINEN ### 4.1 Goroutine-Verwaltung - [ ] Finden Sie Goroutine-Leaks (gestartete, aber nie beendete Goroutinen) - [ ] Identifizieren Sie Goroutinen ohne ordnungsgemäßen Herunterfahrmechanismus (Kontext-Abbruch) - [ ] Erkennen Sie Goroutinen, die in Schleifen gestartet werden, ohne die Nebenläufigkeit zu steuern - [ ] Finden Sie Fire-and-Forget-Goroutinen ohne Fehlerberichterstattung - [ ] Identifizieren Sie Goroutinen, die die Funktion, die sie erstellt hat, überleben - [ ] Prüfen Sie auf `go func()`, die Schleifenvariablen erfasst (Go <1.22-Problem) - [ ] Finden Sie Goroutine-Pools, die unbegrenzt wachsen - [ ] Erkennen Sie Goroutinen ohne `recover()` für Panic-Sicherheit - [ ] Identifizieren Sie fehlende `sync.WaitGroup` für die Verfolgung des Goroutine-Abschlusses - [ ] Prüfen Sie auf ordnungsgemäße Verwendung von `errgroup.Group` für fehlerpropagierende Goroutine-Gruppen ### 4.2 Kanal-Probleme - [ ] Finden Sie ungepufferte Kanäle, die Deadlocks verursachen könnten - [ ] Identifizieren Sie Kanäle, die nie geschlossen werden (potenzielle Goroutine-Leaks) - [ ] Erkennen Sie Doppel-Schließen auf Kanälen (Laufzeit-Panic) - [ ] Finden Sie Senden auf geschlossenem Kanal (Laufzeit-Panic) - [ ] Identifizieren Sie fehlendes `select` mit `default` für nicht-blockierende Operationen - [ ] Prüfen Sie auf fehlenden `context.Done()`-Fall in select-Anweisungen - [ ] Finden Sie fehlende Kanalrichtung in