Salut les développeurs Gladys !
Comme vous avez dû sûrement voir, j’ai ajouté les reviews automatiques de Cursor, avec le prompt suivant :
You need to review the current pull request.
Read through the code and leave comments (in English) wherever you think there are issues or potential problems in the PR.
If you believe the PR is risky to merge (for example, it touches sensitive parts of Gladys or could introduce production issues), add the « risk:high » label to the PR.
If the PR adds new categories or device types to server/constants.js, make sure they are generic enough and not tied to a specific brand. The goal of Gladys is to harmonize device behavior across different brands. Exceptions are acceptable for unique or unusual devices that have no equivalent in other ecosystems.
If you think a human should review the PR to ensure it aligns with the project’s philosophy, add the « needs:human-review » label and request a review from the contributor who seems the most appropriate.
If the label « needs:human-review » was already present and no longer necessary, remove it.If you think the PR is ready for merge, approve PR.
When the review is finished, remove the label « needs:cursor-review » if present on the PR.
Vous voyez des points manquants ?
En sachant que l’IA est très smart donc elle voit déjà beaucoup de choses sans qu’on le mentionne, ce que j’ai mentionné c’est vraiment le comportement qu’on veut au niveau du « flow de merge » de PR ![]()
Objectif : des PRs mergés à un rythme rapide !