Luijaa/web-template

Pre-PR-Check

Führe alle Schritte aus und berichte in einer Tabelle mit Bestehen/Nicht-Bestehen sowie Beweisen (Befehl und Ergebnisauszug).

1. make check muss vollständig bestehen (docs-check, lint + SAST, test, build, license) — außer in 2 Fällen:

   • Nur-Dokumentations-PR (docs(design): …, ADR, docs): make docs-check reicht aus
   • Repository, das noch keinen Stack gewählt hat (Target gibt not configured aus — ADR-0001 noch nicht akzeptiert): dieses Target wird als N/A (noch nicht konfiguriert) gemeldet, nicht als bestanden, und ist kein Grund zum Anhalten — bearbeite kein Rezept, um exit 0 zu setzen

   Und make e2e, wenn Benutzersichtbares geändert wurde (e2e-Tests hinzufügen/ändern), sowie make dast, wenn Middleware, Header, Cookies, Fehlerseiten oder neue Routen berührt wurden

2. git diff origin/main...HEAD --stat — prüfe, ob nur arbeitsrelevante Dateien geändert wurden, keine versehentlichen Dateien (Build-Output, .env, persönliche Dateien)

3. Temporäre Logs, die vergessen wurden zu löschen: git diff origin/main...HEAD | grep -n '\[DBG-' darf nichts finden (fix-bug Punkt 3)

4. Suche nach Secrets im Diff: git diff origin/main...HEAD | grep -nEi '(api[_-]?key|secret|token|password|BEGIN .*PRIVATE KEY)' — wenn gefunden, stoppen und melden

5. Neue Umgebungsvariablen sind vollständig in .env.example vorhanden

6. Hinzugefügte/hochgestufte Abhängigkeiten (prüfe Diff von Lockfile, go.mod, Dockerfile FROM): Version und Lizenz entsprechen docs/dependencies.md und sind in der PR-Beschreibung aufgeführt

7. Neue Migrationen (falls vorhanden) sind ausführbar und rückgängig machbar

8. Dokumentation: README/docs/ADR (docs/adr/) wurde entsprechend der Änderungen aktualisiert; neue Domänenbegriffe befinden sich in CONTEXT.md und die Namen im Code stimmen mit CONTEXT.md überein

9. Überprüfung gemäß Skill review (Claude Code: Subagent reviewer; andere Agenten: lies .agents/skills/review/SKILL.md und überprüfe den Diff selbst) und behebe alle „muss behoben werden“-Punkte zuerst

10. Commit-Nachricht ist Conventional Commits und der Branch ist nicht main

11. PR-Beschreibung enthält Closes #<issue> (wenn kein Issue vorhanden, erstelle eines zuerst) sowie die Überschrift ## CHANGELOG (Kategorie + 1 Zeile aus Benutzersicht auf Thai, oder -) — bearbeite CHANGELOG.md nicht selbst (docs/release.md)

12. UI berührt: keine hartcodierten Texte, Schlüssel in allen Sprachen vollständig, keine direkten Farbwerte, sowohl th/en × light/dark geprüft (docs/ui.md)

Wenn ein Punkt nicht bestanden wird, darf kein PR erstellt werden. Korrigiere zuerst oder melde, woran es hängt.