Fiabilise le core et met à jour les dépendances - #220
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Lramelot
force-pushed
the
chore/solid-base
branch
from
October 3, 2026 18:22
76d6fbf to
f9636f1
Compare
AntoineJT
approved these changes
Oct 3, 2026
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
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.
Part de #212 (
chore/consolidate-core), dont elle contient les commits, et la complète : bugs trouvés en relecture, durcissement du démarrage, et reprise des mises à jour Renovate/Dependabot en attente. Si cette PR est fusionnée, #212 l'est avec.Bug bloquant en prod
bootstrap/validate-env-vars.mjsa été renommé en.tsdans #212, mais leCMDdu Dockerfile lance toujours le.mjs: le conteneur de prod s'arrête dès le démarrage. Le scriptstartdépendait en plus detsx, absent de l'image..tsnativement, leCMDetstartl'utilisent.dockerdu CI ne faisait que construire les images. Il les démarre maintenant : migrator sur un Postgres vierge, puis le bot avec un faux token, qui doit aller jusqu'au contrôle du token Discord.Corrections du core
onInstall/onUninstalls'exécutent avant les appels à Discord. Avant, un hook en erreur laissait les commandes créées alors que le module restait désactivé.unhandledRejection/uncaughtException;onLoadéchoue est retiré complètement ;Guildsdéclaré par le core n'était jamais demandé :index.tsne collectait les intents que des modules.customIddéclaré par deux modules est signalé au démarrage./config(qui filtre aussi sur la saisie) et dans le menu de réinitialisation.Exploitation
infopar défaut en prod, logs lisibles avec le niveaudebugen dev.pino-prettypasse en devDependency etLOG_LEVELest transmis parcompose.prod.yaml.Dépendances
Reprend les PR #215, #158, #179, #175, #156 (actions et outils mise), ainsi que #97 (TypeScript 6), #216 (oxfmt 0.71), #160, #214, #213 et #218 (lockfile). Renovate et Dependabot devraient fermer ces PR une fois celle-ci fusionnée.
Vérifications
pnpm build,tsc --noEmit, oxlint, oxfmt, et les 287 tests passent. Pour chaque correction testée, le test a été vu rouge sans le correctif.pnpm devdémarre sur le serveur de dev sans warning : 3 modules, 4 commandes, rounds jd4h synchronisés.Pas encore vérifié contre Discord : les boutons et commandes n'ont pas été cliqués, seul le démarrage l'a été. Les écritures de config simultanées sont testées avec une base simulée.
Point restant, hors de cette PR
Une commande renommée ou supprimée dans un module reste sur les serveurs : la mise à jour ne supprime que les commandes encore déclarées. La correction propre serait un enregistrement groupé par serveur, comme le fait déjà le mode dev, ce qui change le mode d'enregistrement en prod. À discuter.
🤖 Generated with Claude Code