Repository navigation
Summons - #508
Merged
Merged
Summons#508
Conversation
Merged
Server: USE_SKILL casts a summon at the client's spot (within 250x150 of the owner), spending MP and itemCon and giving its buff; the field keeps summons, shows SPAWN/REMOVE/MOVE_SUMMON, checks SUMMON_ATTACK against mobCount and a 2x damage cap, breaks Puppets at their x HP, and brings followers along on a map change. HeavenClient: MapSummons draws and animates summons (v83 CSummoned); SummonRules holds follow/circle movement, placement and damage; Combat places the cast, targets and attacks on the owner's 3 s / 1.5 s clock, and plays others' attacks; chat lines on expiry. Harness: summons test. Docs: combat.md Summons. Report 158.
A local Puppet's spawn makes each mob this client moves that is chasing someone chase the Puppet for 3-4 minutes; its removal lets them go (FUN_00679847). A mob touching an unblinking Puppet rolls 40/(lv+70) times its attack times 10-85% of it, over 100 (FUN_00793687), shows it, and sends DAMAGE_SUMMON (0x7a627a).
The client heals its owner with Aura of the Beholder, else buffs one stat with Hex, each in its skill's time (FUN_007a4080, FUN_007a5ab4, FUN_007a5c99), and sends BEHOLDER (0xB2). The server checks the skills, holds each to its pace in the field, heals or gives the Hex buff, and shows the others SUMMON_SKILL (0xB3).
A summon's hits now carry its skill's status, as a player's attacks do. Silver Hawk and Golden Eagle stun at the level's prop (50% to 99%). Elquines and Frostprey always freeze; their data has no prop. Each status lasts the level's x seconds, inferred from the freezes' x growing from 1 to 3. The client only shows the status it is sent. The harness has an Arch Mage's Elquines freeze a mob for 3 s. The Beholder test now finds its own summon on the shared map.
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.
Summon skills (Puppet, Silver Hawk, Octopus, Elquines, the souls, Beholder...) now work on both sides.
What changed
itemConand giving its buff.mobCountand a 2x damage cap, breaks Puppets at theirxHP, and brings followers along on a map change.MapSummonsdraws and animates summons (v83CSummoned);SummonRulesholds follow/circle movement, placement and damage;FUN_00679847). A mob touching it rolls touch damage (FUN_00793687) and sends DAMAGE_SUMMON.FUN_007a4080,FUN_007a5ab4,FUN_007a5c99), and sends BEHOLDER (0xB2). The server checks the pace and shows the others SUMMON_SKILL (0xB3).xseconds.agent_docs/combat.mdgains a Summons section.Verification
summon_rules; new server unit tests for casting, attacks and the Puppet.summons, including the Beholder and an Arch Mage's Elquines freezing a mob for 3 s.Reviewed with sol-loop: 2 rounds with gpt-6.1-sol on high.
Stacked on #507