Reglage d equilibrage. A 8, un joueur seul en heroique atteignait environ x10 degats
pour du contenu calibre 5 joueurs. 6 laisse une marge d environ +40 pour cent, qui
couvre les mecaniques heroiques sans trivialiser le donjon.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Le bonus etait ressenti comme hardcore et incoherent. Quatre causes, toutes corrigees.
1. La taille du groupe etait calculee puis ignoree : elle ne servait qu'au message
d'annonce. Un joueur seul et un groupe de 5 recevaient le meme x5 chacun, donc le
solo ramait et le groupe complet ecrasait tout. Le bonus vaut desormais
difficulte / joueurs presents dans l'instance (5 joueurs en donjon 5 -> x1).
2. Seules les stats primaires etaient buffees. Les degats d'une attaque valent
(degats_arme + AP/3.5) et seule la moitie AP en profitait : le joueur encaissait x5
mais tapait a peine plus fort. Ajout d'un bonus sur UNIT_MOD_DAMAGE_MAINHAND/
OFFHAND/RANGED, dose par Solocraft.DamagePct.
3. Les familiers derivent leurs PV et leur AP du maitre mais ne recalculaient pas apres
le buff : un pet deja invoque restait a x1. Appel de UpdateAllStats sur pet et gardien.
4. Rejoindre ou quitter un groupe en pleine instance ne recalculait rien. Ajout d'un
GroupScript.
Aussi : plus de soin complet gratuit si le buff est recalcule en combat, logs passes de
INFO a DEBUG sur scripts.solocraft (ils saturaient dc-world.log), Solocraft.Mythic rend
le mythique configurable, acces au conteneur protege par mutex.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
La route ne vient plus de coordonnees deduites de la base mais du trajet
reellement marche par un joueur, de la route d Elwynn au trone d Anduin. La
trace brute de 69 releves a ete traitee : elagage des aller-retours, puis un
point tous les ~45 yards plus tout changement d etage de plus de 6 yards et
tout virage de plus de 50 degres. Resultat, 30 waypoints au lieu de 24.
L apport decisif est la montee dans le donjon, desormais en paliers reguliers
(105, 115, 125, 132, 136, 145, 152, 157) la ou la version deduite sautait d un
etage a l autre. Les bots s y coincaient dans les batiments et la console etait
inondee de messages de pathfinding en echec.
Point de rassemblement recale sur le depart du trajet.
Mesures apres application, assaut a 50 bots : la horde atteint le waypoint
29 sur 30 avec une trentaine de survivants, contre un plafond a 21 sur 24
auparavant.
Reste ouvert : le bot le plus proche, mort ou vif, s arrete a 94 yards du
trone. Anduin n a donc jamais ete engage, et l hypothese d une protection par
la faction est ecartee par la mesure : un essai avec sa faction forcee a 12,
attaquable puisque les bots tuent les civils de cette faction, n a rien change.
Le verrou est la garde royale de l etage, pas le dirigeant.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
E8 - dirigeant de la capitale. Au declenchement le module le retrouve par son
creature.guid (nouvelles cles siege_*_boss_spawn), force le chargement de sa
grille si aucun joueur n est present, retire les drapeaux bloquants reellement
presents, applique niveau et points de vie, et le designe comme objectif du
commandant. Sa mort est detectee a chaque tick et vaut victoire immediate. En
fin d evenement tout est restaure : drapeaux exactement ceux qui avaient ete
retires, niveau, points de vie, faction, et sortie de combat. Rien n est ecrit
en base, un arret du worldserver laisse donc la creature intacte.
Le reglage du dirigeant est absolu et par ville (siege_*_boss_level,
siege_*_boss_health, siege_*_boss_faction, 0 = ne pas toucher) et non plus un
multiplicateur global : les deux chefs n ont rien de comparable, Anduin depasse
le milliard de points de vie quand Vol jin, reste au niveau 83, en a quelques
milliers. Un multiplicateur de 50 sur Anduin donnait 69 milliards.
AcceptCommand() ne pilote que le deplacement : une cible d attaque prioritaire
(SetSiegeTarget) a ete ajoutee, sinon les bots atteignaient le trone et
passaient leur temps sur la garde rapprochee.
Correction du recrutement, defaut signale en jeu : des bots Alliance se
retrouvaient enroles dans l assaut sur Hurlevent. La cause est dans le core,
PlayerBotBaseInfo::GetRandomCharacterByFuction() se termine par
return characters.begin()->second et renvoie donc le premier personnage du
compte, faction ignoree, quand il n en trouve aucun de la faction demandee. Le
module choisit desormais lui-meme le personnage et le connecte par
BGSType_Online_GUID, avec verification de la faction apres connexion et
garde-fou dans CommandSiege::AddBot. Le quota est rendu sur tout refus, sinon
la horde arrivait incomplete. Verifie : 15 bots recrutes, 15 horde, 15 au
niveau 110.
.siege status affiche l etat du dirigeant en direct.
Reste ouvert : aucun assaut n a encore entame les points de vie d Anduin. La
horde entierement horde se fait laminer par la garde avant le trone, la ou les
essais precedents progressaient plus loin uniquement parce que des bots
Alliance traversaient la ville sans etre inquietes. A retester avec l effectif
complet de 50.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
E7 - recrutement de la horde. Le gestionnaire engage siege_spawn_rate bots par
seconde jusqu au quota : bot deja connecte et inoccupe en priorite, sinon
connexion d une session bot libre. Chaque recrue est remise a niveau par le
schedule BGSType_Settting existant, teleportee au point de rassemblement, puis
enrolee dans CommandSiege. Le plafond porte sur le total engage depuis le debut
et non sur l effectif present : un bot tue n est jamais remplace, il n y a pas
de vague sans fin. En fin d evenement tout l effectif est deconnecte, morts
compris, y compris les recrues encore en chemin.
Le premier essai en conditions reelles a fige la horde a son point d entree.
Quatre causes distinctes, toutes corrigees :
1. IsNotSelect() est aussi appele par IsNotMovement() avec le bot lui-meme en
parametre. La regle siege_pvp y repondait ne pas selectionner pour un bot
non flagge PvP, ce qui declenchait StopMoving(). La regle est desormais
restreinte aux joueurs adverses autres que soi.
2. BotWarlockAI::ProcessNormalSpell() renvoyait true hors champ de bataille, et
la boucle d IA fait if (!IsInCombat() && ProcessNormalSpell()) return : les
demonistes n atteignaient jamais leur deplacement et restaient plantes au
point de depart. Le mode siege passe maintenant cette garde.
3. Portee d engagement. BotBGAI cherche une cible a 32 yards ; en ville il y a
toujours un civil a portee, si bien que l IA ne se deplace jamais (elle ne
marche que sans cible selectionnee). Nouvelle cle siege_engage_range, 18
yards par defaut : on n engage que ce qui barre le passage.
4. Enlisement. Lacher la cible ne suffit pas, l IA en reselectionne une au tick
suivant. Nouvelle fenetre de marche forcee (siege_stall_timeout,
siege_advance_window) pendant laquelle le bot ignore les cibles et avance.
Les bots du siege sont aussi exemptes de la deconnexion d inactivite du module
BG BotFill, qui vidait la horde en pleine progression.
Verifie en conditions reelles : 5 bots niveau 110 partis de la Vallee des Heros
atteignent le waypoint 21 sur 24, soit les etages du donjon de Hurlevent, en
environ quatre minutes, en tuant des PNJ en chemin et en perdant deux des leurs.
Le nettoyage de fin deconnecte bien tout l effectif.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
E5 - CommandSiege (game/CapitalSiege) : meme role que CommandBG en champ de
bataille, il ne combat pas et distribue des ordres de deplacement. Il charge
sa route depuis aiwaypoints (plage d entrees configurable), fait avancer
chaque bot a son rythme de point en point, et bascule sur le dirigeant une
fois le dernier point atteint. Un bot au contact ne recoit aucun ordre : la
boucle de combat de BotBGAI garde la priorite, sinon la horde traverserait la
ville sans jamais rien tuer. Un bot tue est compte en perte et n est pas
remis en jeu.
La route est validee avant que le creneau du jour soit consomme : un assaut
sans waypoints est refuse et pourra se declencher normalement une fois la
route en base, au lieu de bruler la journee.
E6 - mode siege dans BotBGAI. Le bot garde toute son IA de combat mais cesse
de dependre d un Battleground : le verrou InBattleground() ne s applique plus,
l etat d assaut est impose, et la resurrection de champ de bataille est
desactivee. La regle siege_pvp est appliquee dans IsNotSelect() : par defaut
les bots n engagent que les joueurs deja flagges PvP.
Correction de fond au passage : SearchCreatureListFromRange() construisait un
searcher mais ne visitait jamais la grille (VisitNearbyGridObject n existe
plus dans ce core, l appel avait ete commente sans remplacement). La liste
restait vide, donc les bots ne voyaient aucune creature - en siege comme en
champ de bataille. Remplace par Cell::VisitGridObjects.
Commande GM .siege recruit ajoutee comme harnais de test en attendant le
spawner automatique (E7) : enrole le playerbot selectionne dans la horde.
Verifie apres deploiement : assaut sur Orgrimmar refuse faute de route (les
waypoints 2000-2099 arriveront en E10), assaut sur Hurlevent accepte avec la
route de 24 points chargee de l entree 1000 a 1023.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Nouvelle methode GetBotBGActivity() qui inspecte les sessions bots en ligne et
renvoie l instance de BG ou ils se battent deja, ainsi que le type de BG pour
lequel ils sont inscrits en file.
QueryBattlegroundRequirement() s en sert pour ne completer que l instance
deja servie, et pour n ouvrir une nouvelle file que si aucun engagement n est
en cours. Sans ce garde-fou, plusieurs BG simultanes se disputaient le meme
vivier de bots et aucun ne se remplissait correctement.
Vingt-quatre waypoints de la Vallee des Heros au trone d Anduin Wrynn, dans
la plage d entrees reservee 1000-1099 (2000-2099 pour Orgrimmar).
Aucune coordonnee inventee : chaque point vient soit d un trajet de patrouille
reel (waypoint_data des patrouilleurs de Hurlevent), soit de la position d un
PNJ effectivement spawne. Les points sont donc marchables par construction,
ce qui evite de valider la route a la main en jeu. Detour calcule le chemin
entre deux points consecutifs, la route n a pas besoin d etre dense.
Trace : pont de la Vallee des Heros, porte, place du Commerce, route du canal,
allee du Donjon, cour et perron, grand hall, salle du trone, escalier, galerie
des emissaires, antichambre, trone.
Le point de rassemblement passe de la valeur game_tele (-8833, 622) au
waypoint 1000 (-8904, 692) : l ancienne tombait deja sur le pont, a l interieur
du perimetre garde.
Verifie en conditions reelles apres deploiement : tirage quotidien d une heure
aleatoire, alternance de faction, declenchement et arret manuels, ouverture et
fermeture de la ligne d historique, verrou anti-rejeu du creneau quotidien.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Nouveau module d evenement quotidien : une horde de playerbots assaille la
capitale adverse et tente d abattre son dirigeant. Cette premiere tranche
livre le squelette, la persistance et l ordonnancement ; le spawner, l IA
d assaut et l eveil du dirigeant suivent (E4-E8).
Le serveur tourne sur DestinyCore, sans systeme de modules : le module prend
donc la forme d un script module TrinityCore (decouverte CMake automatique
via AddCapitalSiegeScripts()), avec le gestionnaire dans la bibliotheque game
comme PlayerBotMgr, puisque l IA d assaut devra y acceder depuis BotBGAI.
game/CapitalSiege/CapitalSiegeMgr : tirage quotidien d une heure aleatoire
dans une plage configurable, alternance stricte de la faction attaquante,
machine a etats et arret d urgence si le diff monde depasse le seuil.
Le jour est consomme au declenchement et non a la fin, donc un redemarrage
du worldserver en plein evenement ne peut pas le rejouer.
scripts/CapitalSiege : branchement par WorldScript (aucune modification de
World.cpp), et commandes GM .siege status/start/stop/history sous la
permission RBAC_PERM_COMMAND_EVENT existante.
Persistance dans dc_characters et non dc_world : il s agit d etat mutable
ecrit par le serveur, qu une reimportation de la base de contenu ne doit pas
effacer. Six requetes preparees ajoutees a CharacterDatabase. Toute ligne
d historique restee ouverte est refermee au demarrage du core.
MMaps verifies pour les deux capitales : carte 0 (592 tuiles) et carte 1
(715 tuiles), tuiles de Hurlevent et d Orgrimmar presentes. La navigation
des bots passe par PathGenerator/MotionMaster, donc Detour, pas les VMaps.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Ces 20 lignes de TC_LOG_ERROR servaient au diagnostic du remplissage BG
(tick BotBGAI, ordres de mouvement, occupation des bannieres, raccourcis de
pathfinding). Elles spamment Server.log a chaque tick de bot et a chaque
calcul de chemin. Retirees avant le chantier Siege des Capitales pour
repartir sur un log propre.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Issue TrinityCore #22243 (fermee sans solution) : Shang Xi disparait a
lacceptation, aucun dialogue, quete interminable. Trois causes cumulees,
identifiees par sondes temporaires et .debug phase :
1. PHASE : le PNJ de scene (56686) heritait de la phase du donneur (1527),
que le joueur PERD justement en acceptant la quete (il ne garde que
1327). Cree mais invisible. Mesure : summon=OK cibleVoitSummon=NON.
2. DIALOGUES : repliques et sons ciblaient invocateur (7) / proprietaire
(23) - non resolus dans les listes daction differees (source_type 9).
3. CREDIT : le sort 106625 (credit du PNJ 56688 = lobjectif) ciblait 23 et
sexecutait au meme instant que la disparition du PNJ.
Correctifs : retrait de phase avant invocation + le joueur invoque ;
le PNJ force sa phase sur 1327 ; textes/sons cibles sur soi ; credit
attribue directement (KilledMonster) au joueur le plus proche ; despawn
repousse de 3s. Patch SQL rejouable, valide en jeu de bout en bout.
Note : seul le patch SQL est commite - dautres fichiers du working tree
appartiennent a une session parallele (module bots/BG).
1. ProcessRegulation appelait TryOccupied* (cible = banniere) puis
Process*Requirement (cible = centre de la base) toutes les 500 ms : le
second ecrasait le premier, les bots stationnaient a quelques metres des
bannieres sans jamais les atteindre (zero capture constatee en jeu).
Un ordre de banniere/drapeau prime desormais 20 s sur un ordre de
position, dans les deux surcharges d AcceptCommand.
2. Distance de cast resserree a 4 m (le sort d ouverture 21651 echoue
au-dela de sa portee).
3. Un BG sans joueur reel depuis plus d une minute renvoie ses bots au
monde : un match Ile des Conquerants tournait a 40v39 bots pour personne.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- GetNearGameObjectFlag reecrit : l ancienne detection par objet d aura
(indices approximatifs, rayon 6 m) faisait cliquer les bots sur une
banniere a 30 m -> sort d ouverture hors portee, zero capture.
Renvoie desormais la banniere cliquable la plus proche (etat reel du
noeud) dans un rayon de 30 m.
- TryOccupiedABNode : le bot marche jusqu a la banniere (< 8 m) avant de
lancer le sort, et s arrete pour canaliser l ouverture.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
La sonde a revele des ordres de deplacement vers (0,0,0) : la tuile navmesh
de l origine n existant pas, le chemin retombait en ligne droite a travers
la carte (survol du lac d Arathi, allers-retours incoherents). Garde posee
au point de passage oblige MovementTo(x,y,z).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Filtre navmesh des bots sans NAV_WATER : fini l effet de glisse/vol sur les
lacs (UpdateFilter reajoute l eau si le bot y est plonge, pour en sortir)
- AcceptCommand : un bot conserve sa cible tant qu elle n est pas atteinte
(15 s max), sauf ordre drapeau. Le commandant realloue tout toutes les
500 ms : les bots a mi-chemin oscillaient entre deux objectifs.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1. Les tuiles navmesh ne se chargeaient qu avec les grilles occupees : tout
trajet de bot vers une zone vide retombait en BuildShortcut (ligne droite
a travers le decor). Prechargement des tuiles des 5 cartes BG au boot.
2. MovePathfinding empilait un PointMovementGenerator generatePath=false qui
relancait un deplacement en ligne droite au moindre changement de vitesse,
ecrasant la spline du chemin. Remplace par un generateur passif.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Le registre des commandants (PlayerStatus/PlayerGUIDs) etait indexe uint64 ;
ObjectGuid (128 bits) s y convertissait via operator bool() -> tous les
joueurs partageaient la cle 1. Premier bot enrole, tous les autres rejetes
comme doublons, et les recherches inverses (GetBotBGAI) ne trouvaient
personne : les commandants ne pilotaient aucun bot. Conteneurs et signatures
retypes en ObjectGuid dans CommandBG et les 5 commandants (WS/AB/EY/AV/IC).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
StartCommander (passage de l IA des bots en mode combat / assignation des
objectifs) n avait aucun site d appel : les commandants restaient en phase
preparation et les bots figes au spawn pendant tout le match.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
BGSType_EnterBG n avait aucun site d emission : les bots restaient invites
jusqu a expiration et le joueur entrait seul dans le BG. InviteGroupToBG
pousse desormais le schedule aux sessions bots, et ProcessEnterBG utilise
le slot de file reel comme Ticket.Id (attendu par HandleBattleFieldPort).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ProcessInBGQueue envoyait BattlemasterJoin avec QueueID=0 (ligne commentee du
portage) : toutes les inscriptions bots etaient rejetees (invalid bgtype 0).
Ajout de GetScheduledBotCount : tant que 8+ bots sont en cours d engagement
(login/re-level/inscription), le tick n en engage pas de nouveaux (on est
montes a 92 bots en ligne pour un 15v15).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Player::LoadFromDB refusait silencieusement les persos bots (noms avec
chiffres type Bot5951) et posait AT_LOGIN_RENAME en boucle. Les 181 bots
ont ete renommes en base via ai_playerbot_names, et les sessions bots ne
passent plus par CheckPlayerName.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Avec un seul personnage sur le compte bot, characters.size()/2 - 1 = -1
=> ASSERT max >= min => SIGSEGV des que le module BG BotFill engage un bot.
Le contournement historique etait pbotall=0 ; la borne est desormais sure.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
L UI PvP Legion n envoie que la file aleatoire, que le core bloquait en dur
(message 'This Battleground has been disabled'). On tire desormais au sort un
BG parmi WS/AB/EY/AV/IC dont le bracket existe pour le niveau du joueur.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Nouvelle cle pbotbg : active le remplissage des BG par bots (WS/AB/EY/AV/IC)
sans reveiller les bots de monde/LFG/arene (pbotall reste a 0).
- OnRealPlayerJoinBattlegroundQueue reecrit : bracket du joueur +
QueryNeedPlayerCount (besoin reel) au lieu de 5 bots fixes par faction
- pbotbg_maxperteam : plafond de taille d equipe vise par le remplissage
- pbotbg_idlelogout : deconnexion des bots inactifs apres N secondes
- broadcast chinois remplace par un log, printf de debug supprime
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Le serpent restait en vol stationnaire au centre de larene (714, 4168) hors
combat : MovePath(ZHAOREN_PATH) netait lance que dans EnterCombat. Or les
6 lanceurs de feux dartifice (64507) forment un anneau a 24-35m du centre et
leur tir exige que la cible soit AU-DESSUS du joueur -> aucun lanceur ne
pouvait latteindre, et le combat ne demarre que sur impact de fusee
(SpellHit SPELL_OVERPACKED_FIREWORK). Blocage circulaire : quete impossible.
Fix : MovePath(ZHAOREN_PATH, true) des Reset(), il tourne donc hors combat
comme sur retail. Le chemin (12 points) passe a 6-8m au-dessus des lanceurs.
Valide en jeu : dragon en rotation, combat et quete termines.
Les entrees spell_area conditionnees par un statut de quete (quest_start/
quest_end) n etaient reevaluees QUE lors d un changement de zone/aire.
Consequence : rendre une quete sans changer d aire ne donnait jamais l aura
attendue. Sur l Ile Vagabonde, la chaine utilise See Quest Invis 4/5 : apres
avoir rendu Dafeng (29785) sur place, le joueur n obtenait pas 104566, donc
Aysa (Quest Invisibility 4) restait INVISIBLE et Bataille pyrotechnique
(29786) ne pouvait jamais etre prise. Chaine morte sans deco/reco.
Fix : appel de UpdateAreaDependentAuras() en fin de RewardQuest et de
AddQuest (cas miroir a l acceptation).
Portee : toute chaine a invisibilite de quete du serveur.
Diagnostic : .gps du joueur (aire 5886 correcte) + lecture de
SpellArea::IsFitToRequirements ; valide en jeu (Aysa visible, quete offerte).
Le masque des locales disponibles etait initialise a 0 et JAMAIS renseigne
(cablage perdu dans ce fork custom) -> World::GetAvailableDbcLocale()
retombait toujours sur enUS -> GetSessionDbcLocale() forcait enUS ->
les hotfix de broadcast_text (dialogues PNJ, corps de gossip) etaient
pousses en ANGLAIS par-dessus les donnees FR natives du client.
Fix : DB2Manager::LoadStores expose le masque reel des locales (fichiers
disque + locales chargees depuis la base hotfix quand DB2.Hotfix.LoadAllLocales
est actif, broadcast_text_locale faisant foi), branche dans World apres
LoadStores. Necessite aussi DB2.Hotfix.LoadAllLocales=1 dans worldserver.conf.
Valide en jeu : tous les dialogues PNJ de l Ile Vagabonde passent en francais.
Portee : serveur entier (tout texte resolu via le systeme de hotfix).
Village peuple en 2 passes (guids 20540xxx + 21000xxx) se chevauchant a
l entree -> jusqu a 4 singes empiles sur le meme point. Retrait des 4
doublons < 2m, village conserve peuple (~30 Hozen, blizz adaptatif).
Le client 7.3.5 embarque 88k traductions frFR officielles mais TC lit
creature_text_locale (11 lignes) et gossip_menu_option_locale (18 lignes).
Pont via BroadcastTextId : +12 288 dialogues de creatures et +5 605 options
de gossip en francais, serveur entier. Recharge a chaud, aucun restart.
Decouvert en testant l Ile Vagabonde (dialogues d ambiance en anglais).
Degats des sorts de creatures verifies authentiques (sonde DMGDBG : melee
14-16, Back Kick 69 = ratio MoP retail) -> intacts. La deviation etait la
densite custom : 70 spawns la ou le retail en met ~30. Echantillonnage a
dispersion maximale, espacement final 28m > rayon d aggro (~17m) : le
multi-pull redevient un choix du joueur.
Philosophie actee avec l admin : blizz adaptatif = preserver les valeurs
Blizzard, corriger les deviations custom.
- Pretre Esprit gardien (47788) : SCHOOL_ABSORB sur EFFECT_2, pas EFFECT_1
— la sauvegarde anti-mort ne fonctionnait JAMAIS (2 hooks rebindes)
- Chasseur Filet du traqueur (200108) : MOD_ROOT_2 (reel) au lieu de
MOD_ROOT — le burst de vitesse post-racine ne partait jamais
- Guerrier Mort soudaine (52437) : ABILITY_IGNORE_AURASTATE (reel) au lieu
de DUMMY — le reset de Colossus Smash ne partait jamais
- Druide Instinct de tueur (108299) : DUMMY (reel) au lieu de
MOD_SHAPESHIFT (2 hooks)
- Dernier recours guerrier (12975) + familier (53478) : scripts debranches,
l effet natif APPLY_AURA/MOD_INCREASE_HEALTH fait deja tout (patch SQL)
Analyses sans action (pas de fix verifie transposable) : mage Flame On
(redesign charge-recovery), druide Omen of Clarity resto, pretre Vampiric
Touch proc, warrior Ravager (FIXME assume chez Argus aussi), voleur
nerve_strike/nightstalker, warlock life_tap/demonic_empowerment/darkglare
(designs differents, notes dans le suivi de chantier).
Demoniste : couverture complete cote features, 9 hooks morts restants notes.
- Lames demoniaques (203555) : hook proc sur PROC_TRIGGER_SPELL (reel) au
lieu de DUMMY — le talent Havoc ne proccait jamais [aa54261d]
- Annihilation (201428/227518) : EFFECT_1 rebinde WEAPON_PERCENT_DAMAGE
- Metamorphose (201453) : immunite du saut — hook rebinde EFFECT_0 +
APPLY_AURA (une valeur d aura 39 etait passee comme type d effet)
- Barrage gangrene : bindings SQL des deux scripts INVERSES (aura sur le
sort de degats et inversement) — echange [82dbceb4]
- Aura d immolation : ajout du binding Vengeance 219830 (seul Havoc
178740 etait couvert) [aa54261d]
Notes sans action : Glide (approche differente a evaluer en jeu), Marque
de feu 204022 (FIXME assume aussi chez Argus), fury refiller 203650
(pipeline de donnees, comme Soul Reaper DK).
- Elixir de soins (122280) : hook proc sur DUMMY (reel) au lieu de
PROC_TRIGGER_SPELL — le passif ne proccait jamais [603df7a1]
- Eclair de jade (117952) : script supprime, proc de chi 25%/tick = reliquat
MoP, le canal Legion ne genere rien [8ffc9d55]
- Effuse (116694) : relais de soin MoP mort retire, redirection Soothing
Mist conservee
- Stagger (124273/4/5) : hook remove rebinde EFFECT_1/DUMMY
- The de mana (197908) : debranche, effet natif Legion (patch SQL)
Preuve locale : erreurs did-not-match au boot pour chaque hook corrige.
Les paquets destines au socket instance absent (SMSG_PET_SPELLS_MESSAGE etc.)
etaient jetes silencieusement -> familiers inertes. Repli sur le socket realm,
seul garanti pour toute session connectee.
- Aura 373 (MOD_SPEED_NO_CONTROL) implementee dans UpdateSpeed : vitesse
verrouillee des dash (Roulade moine x2.75/x3.75, Fel Rush) au lieu d un
comportement non cape [aa54261d]
- AuraScript::_Validate accepte SPELL_EFFECT_APPLY_AURA_ON_PET dans les
7 verifications d effets d aura [d308bf0b]
Methodologie : 7 commits engine analyses, 5 N/A justifies (systemes absents
ou deja presents chez nous).
Deux bugs a la Mare des Reflets (Ile Vagabonde) empechaient de finir
la quete jouer 5x avec l Esprit de l eau :
- Shu jouable (65493) portait l aura d invisibilite 89304 (reservee au
Water Spout Bunny 60488) -> spawne mais invisible.
- spell_water_spout_quest_credit (invoque l Esprit de l eau au terme de
la quete) present dans le core mais lie a aucun sort -> PNJ final absent.
Patch SQL data : sql/sylvania/shu_water_spirit_fix.sql
Signale et valide en jeu par le joueur.
Le core upstream activait .cheat god a chaque connexion d un compte GM,
meme en mode joueur (.gm off). Consequence : les degats des creatures
etaient annonces au client (combat log) mais jamais appliques cote
serveur -> barre de vie client figee sur la derniere valeur, regen et
nourriture visuellement inertes (rien a regenerer cote serveur).
Diagnostic par sondes HPDBG sur SetHealth/RegenerateHealth/BuildValuesUpdate.
Le god mode reste disponible via .cheat god on explicite.
Le harnais bot a intercepte 3 crashs avant tout joueur : mon renommage de la passe 4
(GetBossSaveData -> GetSaveData, la suggestion du compilateur) avait transforme
l override GetSaveData en appel recursif infini -> stack overflow des qu une instance
1676 se cree. Fix racine : suppression des overrides GetSaveData/Load du module,
la persistance generique d InstanceScript (SetBossState) fait foi dans les deux sens.
LECON : ne jamais accepter aveuglement le did-you-mean du compilateur quand la
fonction suggeree est celle qu on est en train de definir.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
AVANT : 4 boss etaient des coquilles vides de 28-33 lignes (Hote affige, Jeune fille
de vigilance, Sassz ine, KIL JAEDEN le boss final) et AUCUN boss n etait binde en DB
(ScriptName vides sauf Goroth) -> le raid entier etait decoratif malgre 978 spawns.
APRES : module lignee uwow complet (12 161 lignes, 12 fichiers) transcrit vers nos
APIs en 7 passes de compatibilite (~880 erreurs de compilation resorbees) :
instance script complet (portes, frames, graveyards), 9 encounters scriptes,
trash/evenements, 74 bindings de sorts, ~60 ScriptName de creatures, ATs.
Ajouts CORE reutilisables (compat DestinyCoreNew, prepareront le futur port Antorus) :
- UnitAI : DoCastTopAggro, GetObjectData, hooks no-op (SpellFinishCast,
OnApplyOrRemoveAura, OnRemoveAuraTarget, OnAreaTriggerDespawn/Cast, OnInterruptCast,
IsDisableGenerateLoot, GetModifyedData)
- CreatureAI : Talk(guid), Talk(liste aleatoire), ZoneTalk
- ScriptedAI : CheckHomeDistToEvade + alias IsHeroicRaid/IsMythicRaid/IsLfrRaid/
IsNormalRaid/IsHeroicPlusRaid ; SummonList::GetCreature
- Unit : GetAnyOwner, CastSpellDelay x2 ; Creature : SetReactState(delay), StopAttack
- Map : ApplyOnEveryPlayer, IsHeroicPlusRaid ; GameObject : AddDelayedEvent (m_Events)
- Position : SimplePosXYRelocationByAngle, GenerateNonDuplicatePoints, IsLinesCross
(reecrit sans G3D::LineSegment2D, absent de notre g3dlite)
- MotionMaster : MoveSmoothFlyPath x2 ; AreaTrigger : GetRadius/SetSphereScale(no-op)/
isMoving/Despawn/GetProgressCompat ; AreaTriggerAI : IsValidTarget/CalculateSpline
Degradations douces assumees (documentees en commentaire dans le code) :
filtrage de cibles des ATs (IsValidTarget jamais appele par notre moteur), scale
dynamique des zones de Kil jaeden, conversations cinematiques, distribution de loot
personnelle, hooks d interruption/notification d aura, quelques approximations de
roles (SoloQ->ROLE_TANK) et de type de degats.
SQL : sql/sylvania/tomb_of_sargeras_bindings.sql (applique en DB).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Inspection des cores 7.3.5 actifs en 2026 hors ArgusCore : Legion-Studios (releases
ecrasees, inexploitable), levrules (locales russes), Psychostout (dump sans historique),
mudyx/DestinyCoreNew (meme famille que nous ; ses fixes ciblent sa propre dette, notre
Nythendra est deja correcte ; NOTE : il possede des scripts Antorus/Tombe de Sargeras
qu on n a pas -> gisement de contenu futur), Titans/LegionCore-Reforged (2 fixes retenus).
Portes depuis LegionCore-Reforged 814091a (verifies buggues chez nous) :
- PathGenerator::BuildPolyPath : dans la branche farFromPoly on ne recalait QUE le point
d arrivee ; le point de DEPART restait aberrant quand l unite est au-dessus du navmesh
(debout sur un GO : quai, caisse, plateforme) -> chemins aberrants des poursuivants.
Recalage du startPoint sur son poly ajoute.
- MMapManager : budget de noeuds Detour de la navmesh query 1024 -> 2048 : les tres longs
chemins echouaient faute de budget.
Ecartes apres verification (pieges evites) : displayId 39 -> 32290 aurait CREE un bug
(nos 1216 GO display 39 spawnes = points d assise volontairement invisibles, 0 coffre) ;
locales frFR (nos 104640 noms FR >> leurs 9880, schema different) ; criteria appearance
(deja gere, 4 sites CriteriaHandler).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Inspection des 111 nouveaux commits ArgusCore (26/07 -> 01/08). Seuls 2 bugs sur 5
candidats existaient reellement chez nous (les autres deja corrects, voir ci-dessous) :
- Bond heroique (6544) : la constante SPELL_WARRIOR_HEROIC_LEAP_DAMAGE (52174) etait
declaree mais JAMAIS castee -> le bond ne faisait aucun degat a l atterrissage.
Le champ TriggerSpell de l effet JUMP_DEST n est pas peuple dans ce client, donc on
caste explicitement. Compromis assume : part au lancement, pas a l impact exact. (32dfd577)
- Vallee d Alterac : m_CaptainBuffTimer initialise avec urand(0,4)*60 au lieu de *60000
-> buff des capitaines quasi immediat au lieu de 2-6 min. Une seule des deux occurrences
etait buguee (l initialisation), l autre etait deja correcte. (a4f5083d)
DEJA CORRECTS chez nous (verifies, rien a porter) : opcode SMSG_ACCOUNT_HEIRLOOM_UPDATE
0x25C5 (ArgusCore cite DestinyCore comme reference), constantes AV/IoC 600/300,
areatrigger_teleport a ID negatifs (notre conteneur est deja int64), map IDs des BG
(architecture differente, pas de battleground_scripts chez nous).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Cause racine : une invocation faite par la MAP (instance->SummonCreature) n herite
d AUCUNE phase, contrairement a un summon fait par une creature. Les joueurs etant mis
en phase 169 par OnPlayerEnter, toutes les vagues et PNJ de ces 2 scenarios etaient
invisibles/intangibles => etape 1 infranchissable (joueur Argus bloque sur la map 1460).
Ce defaut avait ete trouve par le harnais bot le 26/07 et corrige dans le runner
d artefacts UNIQUEMENT ; les 2 scenarios narratifs, ecrits avant, ne l avaient jamais recu.
- Rive Brisee : helpers Summon/SummonAt centralises -> phase 169 + Z cale au sol ;
aggro forcee sur les vagues de demons (compteur d etape) ; allies non touches
- Moine Serenity s End : helper SummonPhased (phase + Z), 22 appels map rediriges
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
dc_world n est pas couverte par le backup auto (auth+characters seulement, dump
compresse ~78 Mo). Ces patchs sont la trace versionnee hors-VPS des changements
data : framework scenarios d artefact, campagne moine, intro Rive Brisee, bindings
Ring of Peace/Chi-Ji, trainers, quetes, fixes divers.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Dernier manque moine de l audit 10/07. AreaTrigger id 718 (trouve via SpellEffect.db2
effet 179 CREATE_AREATRIGGER misc[0]=718). Struct at_monk_ring_of_peace : OnUnitEnter
ejecte tout ennemi valide hors de l anneau (KnockbackFrom depuis le centre), mecanique
Legion 7.3.5 confirmee (wowhead/wowpedia : anneau 8s, ennemis entrants ejectes).
Binding : areatrigger_scripts(718) -> ~/dc-patches/ring_of_peace.sql.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- LFG: quitter la file en solo n autorise que SON propre ticket, pas celui d un autre (28b8d962)
- Calendrier: modifier un evenement ou y ajouter une invitation reserve au proprietaire
ou a un invite moderateur (helper CalendarEventEditableBy sur UpdateEvent + Invite) (0c2fed59)
Non portes (deja OK/absents chez nous) : arene join utilise deja la vraie note (pas le
bug hardcode a 1) ; Chant de Chi-Ji areatrigger DEJA implemente (at_monk_song_of_chiji).
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
PathGenerator::ValidatePathAgainstCollision() appelee en fin de BuildPointPath :
valide chaque segment du chemin MMAP contre la collision vmap (isInLineOfSight,
ModelIgnoreFlags::Nothing). Le MMAP ne modelise pas les objets fins (clotures,
barrieres) -> un chemin nav-mesh valide peut les traverser. Si un segment est bloque,
on tronque le chemin (PATHFIND_INCOMPLETE) pour que l unite s arrete devant l obstacle.
Adapte a nos APIs : _sourceUnit, isInLineOfSight 8-params, log printf, halfHeight fixe.
Skippe les chemins non-MMAP (vol/nage/raccourcis).
ARBITRAGE reversible : plus realiste mais risque de sur-troncature (PNJ qui s arretent
devant une porte/rampe M2 vue comme obstacle). A surveiller : PNJ bloques en mouvement.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Les 9 verifications de LoS de sorts dans Spell.cpp passent de ModelIgnoreFlags::M2
(ignore les doodads M2) a ::Nothing (les prend en compte). Effet : on ne peut plus
lancer de sort a travers piliers/arbres/portes qui sont des modeles M2. Nos vmaps
contiennent bien la geometrie M2 (13545 modeles .m2.vmo) donc le changement est effectif.
Couvre : cibles de chaine, CheckCast (cible+destination), CheckEffectTarget, ecorchage/cadavre.
ARBITRAGE reversible : plus Blizzlike mais risque de faux blocages sur doodads a
collision imprecise. Verifie en scenario (combat OK, 0 erreur LoS anormale). Revert =
Nothing -> M2 sur ces 9 sites si les joueurs signalent des sorts bloques a tort.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Anti-crash (adaptes a nos APIs) :
- Spell: assert dur sur cible destination -> log+skip (a00b85b2)
- SpellAuraEffects: DoT weapon-percent, caster nul -> tick a 0 (b456b7f7)
- Battleground: PlayerScores.find sans garde -> continue si absent (e875f9f4)
- VehicleHandler: 4x siege nul deref -> return (392466b1)
- LootHandler: division par zero si liste vide -> return (e8196eed)
Anti-exploit :
- PetHandler: index action bar hors bornes (OOB write) + abandon reserve au proprietaire (fec060ab)
- Guild: onglet de banque reserve au chef (2288f107) ; attribution de rang ne peut viser/donner un rang >= au sien (f94d82dc)
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
SylvaniaCore deploy