sandeshp1995/claude-code-team-kit

Überprüfen Sie Codeänderungen auf Korrektheit, Lesbarkeit, Tests und offensichtliche Sicherheitsprobleme. Verwenden Sie dies, wenn der Benutzer eine Überprüfung seiner Änderungen, eines Diffs oder eines Branches anfordert, oder bevor er einen Merge Request erstellt.

Was ist claude-code-team-kit?

Dieses Skill führt eine strukturierte Code-Review-Checkliste für KI-Agenten wie Claude Code, Cursor oder Codex durch. Es identifiziert die zu überprüfenden Änderungen, indem es automatisch den Basis-Branch (origin/main, origin/master oder origin/develop) oder unbestätigte Änderungen ermittelt. Der Skill arbeitet eine Checkliste mit den Kategorien Korrektheit, Tests, Lesbarkeit, Sicherheit und Konsistenz ab und bewertet nur echte Befunde. Die Ergebnisse werden nach Schweregrad gruppiert (Muss behoben, Sollte behoben, Kosmetik) und enthalten Datei:Zeile, Problembeschreibung und eine konkrete Verbesserungsvorschlag. Der Skill endet mit einem zusammenfassenden Urteil, ob die Änderungen bereit zum Mergen sind, nach Behebung der Muss-Fixes bereit sind oder eine weitere Überarbeitung benötigen. Dies ist besonders nützlich für Entwicklerteams, die konsistente, qualitativ hochwertige Code-Reviews in ihren Arbeitsablauf integrieren möchten.

Funktioniert mit✓Claude Code~Codex CLI~Cursor

npx skills add https://github.com/sandeshp1995/claude-code-team-kit/tree/HEAD/.claude/skills/code-review-basics

Dokumentation

Grundlagen der Code-Review

Überprüfen Sie die Änderungen, auf die der Benutzer zeigt. Wenn keine Angabe erfolgt, überprüfen Sie den aktuellen Branch gegen seinen Basis-Branch:

Finden Sie die Basis: versuchen Sie git merge-base HEAD origin/main, dann origin/master, dann origin/develop. Wenn keine existiert, überprüfen Sie unbestätigte Änderungen (git diff HEAD).
Lesen Sie den vollständigen Diff, öffnen Sie dann den umgebenden Code für jede Änderung, die Sie allein aus dem Diff nicht beurteilen können.

Checkliste

Arbeiten Sie jeden Punkt durch. Melden Sie nur Punkte mit einem echten Befund.

Korrektheit

Tut der Code das, was die Änderung beabsichtigt? Achten Sie auf Off-by-one-Fehler, falsche Bedingungen, unbehandelte None/null und geschluckte Ausnahmen.
Sind Grenzfälle behandelt: leere Eingabe, sehr große Eingabe, gleichzeitige Aufrufe, Wiederholungen?

Tests

Ist jedes Verhaltensänderung durch einen Test abgedeckt, der ohne sie fehlschlagen würde?
Überprüfen die Tests das Verhalten, nicht die Implementierungsdetails?

Lesbarkeit

Sind die Namen zutreffend? Würde ein neues Teammitglied dies beim ersten Lesen verstehen?
Ist toter Code, Debug-Ausgabe oder auskommentierter Code zurückgelassen worden?

Sicherheit (Grundlagen)

Keine Geheimnisse, Token oder Anmeldedaten in Code, Konfiguration oder Logs.
Benutzereingaben werden validiert und Abfragen sind parametrisiert, niemals durch Zeichenkettenverkettung erstellt.

Konsistenz

Folgt es den Konventionen in CLAUDE.md und dem umgebenden Code?

Ausgabe

Gruppieren Sie Befunde nach Schweregrad:

Muss behoben: Fehler, Sicherheitsprobleme, fehlende Tests für riskante Logik.
Sollte behoben: Wartbarkeitsprobleme, die später Zeit kosten werden.
Kosmetik: optionale Verbesserungen. Halten Sie diese wenige.

Geben Sie für jeden Befund datei:zeile, das Problem und einen konkreten Verbesserungsvorschlag an. Beenden Sie mit einem einzeiligen Urteil: bereit zum Mergen, bereit nach Muss-Fixes oder benötigt weitere Überarbeitung. Wenn Sie in einer Kategorie nichts finden, lassen Sie die Kategorie weg. Erfinden Sie keine Probleme, um Platz zu füllen.

Die Pro-Edition fügt vollständige Checklisten für Sicherheit, Leistung, Barrierefreiheit und API-Design sowie einen /review-Befehl hinzu, der die passenden für den Diff auswählt.