AI-Code-Review
OpenClaw ist dein Senior Engineer – 24/7 online, reviewt jede Code-Zeile gewissenhaft
Die Realität von Code Review
PR hochgeladen, der Senior hat mit anderen Projekten zu tun. Nach zwei Tagen Review-Kommentar: einfach nur „LGTM". Entweder warten ewig oder es wird nicht richtig gelesen – wozu braucht man so ein Review?
Noch blöder: jeder in der Team hat andere Standards. Eine Person sagt, das geht klar, die nächste sagt, muss geändert werden. Wen hörst du? Standards vereinbaren? Nach drei Meetings wird man sich einfach nicht einig.
Code an OpenClaw geben, es reviewt zeilenweise nach besten Praktiken: Sicherheitslücken, Performance-Bottlenecks, Wartbarkeit, Edge Cases – alles covered.
Keine Wartelists, keine Politik, nicht "zu beschäftigt" für ein Oberflächliche Review. Und der Standard ist jedes Mal identisch – keine Diskussionen mehr in der Gruppe, ob das zählt als Problem.
3 Review-Prompts, kopieren, fertig
Von Sicherheits-Audits bis Designprinzipien – je nach Bedarf aussuchen.
Bitte reviewe die folgende Code-Änderung (PR diff) mit Fokus auf:
1. Sicherheitslücken: SQL-Injection, XSS, CSRF, Leak sensitiver Daten, unsicheres Deserialisieren
2. Performance-Probleme: N+1-Queries, unnötige Memory-Allokation, nicht-indizierte Abfragen, blocking Ops
3. Edge Cases: Null-Handling, Race Conditions, extrem große Datenmengen
Für jedes Problem geben:
- Severity-Level (Critical / Warning / Suggestion)
- Exakte Stelle (Datei + Zeilennummer)
- Fix-Vorschlag mit Code-Beispiel
Am Ende Gesamt-Evaluation und ob du Merge empfiehlst.Bitte check, ob dieser Code die SOLID-Prinzipien einhält:
- S (Single Responsibility): Macht diese Klasse/Funktion zu viel?
- O (Open/Closed): Braucht man für neue Features bestehenden Code zu ändern?
- L (Liskov Substitution): Kann die Subclass die Parent sicher ersetzen?
- I (Interface Segregation): Ist das Interface zu dick?
- D (Dependency Inversion): Hängt High-Level-Code direkt an Low-Level-Implementation?
Für jeden Verstoß:
1. Sag genau was verletzt wurde, wo
2. Warum ist das ein Problem
3. Gib Refactor-Option und Code-BeispielAnalysiere die Database-Queries im folgenden Code:
1. Finde alle N+1-Query-Probleme:
- Mark Queries in Loops
- Berechne Worst-Case-Query-Anzahl
2. Gib Optimierungsoptionen:
- Was lässt sich mit JOIN zusammenfassen
- Was braucht Eager Loading / Preload
- Was braucht Caching
3. Schreib den optimierten Code
4. Schätz Performance-Differenz vorher/nachher
ORM-Framework ist [dein Framework, z.B. SQLAlchemy / Prisma / ActiveRecord].Code-Review: OpenClaw vs GitHub Copilot
Beide können Code reviewen – aber völlig unterschiedlich.
- Sieht den ganzen Projekt-Kontext, versteht Cross-File Business-Logic
- Prompt ist völlig customizable – review nach deinen Team-Regeln
- Kann Modelle wechseln: einfache Reviews mit GPT-4o, komplexe Architektur mit Opus 4.6
- Review-Ergebnisse lassen sich exportieren, archivieren, als Team-Knowledge-Base nutzen
- In GitHub PR-Interface integriert – triggern ist praktisch
- Review-Scope ist hauptsächlich Diff-Content – Cross-File-Verständnis begrenzt
- Modell ist fixiert – keine eigenen Review-Regeln
- Mit chinesischen Kommentaren und Variablennamen occasional Crash