feat: restriction fine du bot dev/test par liste de salons et/ou catégorie#89
Merged
Merged
Conversation
…gorie (#88) RESTRICT_TO_CHANNEL_ID accepte désormais une liste de salons séparés par des virgules (un seul identifiant, sans virgule, reste rétrocompatible). Nouvelle variable RESTRICT_TO_CATEGORY_ID : tout salon rangé dans cette catégorie Discord est autorisé, en plus de ceux listés dans RESTRICT_TO_CHANNEL_ID. Permet d'ajouter de nouveaux salons de dev/test à la catégorie sans redéployer ni reconfigurer le bot. Les deux variables sont indépendantes et optionnelles ; ni l'une ni l'autre définie (cas de la prod) laisse le comportement inchangé. ctx.channel.category_id est lu via getattr avec une valeur par défaut pour ne pas planter sur les DM (qui n'ont pas cette catégorie). docker-compose.dev.yml utilise maintenant RESTRICT_TO_CATEGORY_ID pointant sur la catégorie de salons de test dédiée. Closes #88
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Résumé
Closes #88
Revue
Implémenté et revu via un workflow (implémentation + revue adversariale automatisée). La revue a trouvé un bug réel avant merge :
if restrict_channel_ids or restrict_category_id:traitaitRESTRICT_TO_CATEGORY_ID=0comme "non défini" (0 est falsy en Python), désactivant silencieusement toute restriction. Corrigé (is not None) et vérifié par 12 cas de test unitaires couvrant tous les combos (prod, liste avec espaces, catégorie seule, DM, les deux définies, etc.).Test plan
Restreint aux catégorie ....$edittesté dans un salon de la catégorie autorisée : sans réponse (erreur), en réponse à un message non-bot (erreur) — les deux correctement colorés.$prez/$cal/$site/$jdrre-testés dans les salons existants de la catégorie, toujours fonctionnels.