Drupal Review banner
Effective-Bytes Effective-Bytes

Drupal Review

Git community

Description

KI-gestützte Code-Reviews für Drupal-Projekte als Claude Code Skill. Prüft Architektur und Standards, ändert keinen Code.

Installation

This entry records only its repository, not the path inside it, so there is no exact command to give. Open the source below and copy the folder into ~/.claude/skills/, or the file into ~/.claude/agents/.

README

AI Code-Review (Drupal) — Claude Code Skill

Dieses Repository ist ein **Claude Code Skill** für **KI-unterstützte Code-Reviews in Drupal-Projekten**. Ziel ist eine konsistente Architektur, klare Verantwortlichkeiten und verlässliche Review-Ergebnisse, um Wartbarkeit und Zuverlässigkeit zu fördern.

Was es ist

  • Ein global installierbarer Skill für Claude Code (drupal-review)
  • Review-Anweisungen für KI-Code-Reviews (rules/REVIEW.md)
  • Ein Kontext-Generator (generate_context.sh), der die nötigen Informationen bereitstellt:
    • vorverarbeiteter File-Tree
    • PR-Kontext (Beschreibung, Kommentare, Inline-Kommentare, Diff)
    • Commit-Liste und Merge-Konflikt-Analyse (combined diff, Composer-Dependency-Checks, Marker-Scan)
    • die Development-Standards, gegen die geprüft wird — standardmäßig `Effective-Bytes/development-standards`, umstellbar auf eigene (siehe Eigene Standards verwenden)

Was es tut

  • Definiert, wie Drupal-Code strukturiert und geprüft wird
  • Stellt sicher, dass Architektur- und Projektstandards eingehalten werden
  • Erzeugt Kontextdaten für präzise KI-Reviews — in einem .drupal-review/-Verzeichnis innerhalb des Projekts, damit Claude Code die Dateien ohne Einzelfreigaben lesen kann. Das Verzeichnis wird automatisch über .git/info/exclude ignoriert (keine .gitignore-Änderung, kein Commit) — das Zielprojekt bleibt unverändert.

Was es nicht tut

  • Es ändert keinen Code. Der Reviewer meldet Verstöße mit Belegstelle und Begründung; Patches oder umgeschriebene Dateien liefert er bewusst nicht.
  • Es ersetzt keine Linter. PHPCS, ESLint, Stylelint und die Testsuite laufen weiter in der CI. Geprüft wird, was ein Linter nicht sieht: Architektur, Schichtentrennung, Platzierung von Logik.
  • Es prüft nur, was im PR steht. Ohne offenen Pull Request bzw. Merge Request für den aktuellen Branch bricht der Generator ab.
  • **Es is