Repository navigation
refactor: calculer moonbreak, expéditions et verrou de lune avec ogamejs 4.1 - #10
Merged
Merged
Conversation
…ejs 4.1 `!mb`, `!oge` et `!ogl` portaient leur propre copie du calcul, recopiée ensuite dans ogame-ui. Les trois vivent maintenant dans ogamejs 4.1.0 (`Fleets.getMoonbreak*`, `getExpeditionMaxFind`, `getCargoCapacity`, `getMoonLockShips`) : le bot et le site appellent le même code. Les fichiers ne gardent que la lecture de la commande et le message. Les réponses ne changent pas : 30 commandes rejouées avant et après, sorties identiques à l'octet près. `!oge` suppose toujours un explorateur avec un éclaireur, sans bonus de formes de vie, et lit désormais le bonus de classe dans serverData (`explorerBonusIncreasedExpeditionOutcome`, 50 % par défaut). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
L'aide et le message d'erreur annoncent « Entre 1 et 4 attaquant(s) au plus », comme une attaque groupée en jeu, mais rien ne le vérifiait : `!mb 8944 10 10 10 10 10` répondait une probabilité pour une attaque impossible. ogame-ui refusait déjà ce cas. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
!mb,!ogeet!oglavaient chacun leur propre copie du calcul, recopiée ensuite dans ogame-ui. Ces trois calculs sont maintenant dans ogamejs 4.1.0 : le bot et le site appellent le même code.1. Refactor, sans changement de réponse
mb.js: les vagues, la probabilité et les pertes viennent deFleets.getMoonbreakWaves,getMoonbreakChanceetgetMoonbreakLosses. Le fichier ne garde que la lecture de la commande et le message.expeditions.js:Fleets.getExpeditionMaxFindetgetCargoCapacity. La commande suppose toujours un explorateur avec un éclaireur, sans bonus de formes de vie. Le bonus de classe est maintenant lu dans serverData (explorerBonusIncreasedExpeditionOutcome, 50 % par défaut).create-link.js:Fleets.getMoonLockShips.J'ai rejoué 30 commandes avant et après (15
!mb, 10!oge, 5!ogl, cas limites et erreurs compris) : les sorties sont identiques au caractère près.2. Correctif :
!mbau-delà de 4 attaquantsL'aide et le message d'erreur annoncent « Entre 1 et 4 attaquant(s) au plus », mais rien ne le vérifiait :
!mb 8944 10 10 10 10 10répondait une probabilité pour une attaque impossible. La commande renvoie maintenant le message d'erreur, comme ogame-ui. Un test couvre ce cas.Lint OK, 21 tests sur 21 passent.
masterredéploie le bot en prod (GHCR, puis Scaleway).🤖 Generated with Claude Code