Hallo, liebe Gladys-Entwickler!
Wie ihr sicherlich schon gesehen habt, habe ich die automatischen Reviews von Cursor hinzugefügt, mit folgendem Prompt:
Du musst den aktuellen Pull Request überprüfen.
Lies den Code durch und hinterlasse Kommentare (auf Englisch) überall dort, wo du Probleme oder potenzielle Probleme im PR siehst.
Wenn du denkst, dass der PR riskant zum Mergen ist (z. B. er betrifft sensible Teile von Gladys oder könnte Produktionsprobleme einführen), füge das Label « risk:high » zum PR hinzu.
Wenn der PR neue Kategorien oder Gerätetypen zu server/constants.js hinzufügt, stelle sicher, dass sie generisch genug sind und nicht an eine bestimmte Marke gebunden sind. Das Ziel von Gladys ist es, das Geräteverhalten über verschiedene Marken hinweg zu harmonisieren. Ausnahmen sind akzeptabel für einzigartige oder ungewöhnliche Geräte, die in anderen Ökosystemen kein Äquivalent haben.
Wenn du denkst, dass ein Mensch den PR überprüfen sollte, um sicherzustellen, dass er mit der Philosophie des Projekts übereinstimmt, füge das Label « needs:human-review » hinzu und bitte den Beitragenden, der am geeignetsten erscheint, um eine Überprüfung. Wenn das Label « needs:human-review » bereits vorhanden war und nicht mehr notwendig ist, entferne es.
Wenn du denkst, dass der PR zum Mergen bereit ist, approviere den PR.
Wenn die Überprüfung abgeschlossen ist, entferne das Label « needs:cursor-review », falls es im PR vorhanden ist.
Seht ihr fehlende Punkte?
Wobei die KI sehr schlau ist und bereits vieles sieht, ohne dass wir es erwähnen müssen. Was ich erwähnt habe, ist wirklich das Verhalten, das wir im « Merge-Flow » der PRs wollen 
Ziel: PRs werden in einem schnellen Tempo gemerged!