Skip to content

Fix MaceDMG reliability with Killaura, spear hits, and AttributeSwap swaps + spear aura fix#8

Open
Altidias wants to merge 3 commits into
cev-api:masterfrom
Altidias:master
Open

Fix MaceDMG reliability with Killaura, spear hits, and AttributeSwap swaps + spear aura fix#8
Altidias wants to merge 3 commits into
cev-api:masterfrom
Altidias:master

Conversation

@Altidias

@Altidias Altidias commented Jun 9, 2026

Copy link
Copy Markdown

MaceDMG only smashed when your crosshair was on the target and when itsattack listener happened to run after a weapon swap, so it was unreliable with Killaura, with spears, and alongside AttributeSwap. Separately, aura couldn't hit with a spear at all, because it used the wrong attack method.

  • MaceDmgHack
    • Removed the MC.hitResult == ENTITY gate. The PlayerAttacksEntityEvent
      already means an entity is being hit, requiring the crosshair on it
      suppressed the smash for Killaura (which attacks via gameMode.attack
      without you looking at the target).
    • Extracted the smash into a public doSmash() guarded by a once-per-tick
      check (lastSmashTick), so it can be triggered externally without
      double-firing.
  • AttributeSwapHack
    • Implemented LeftClickListener so the swap also fires on the attack
      input, covering spear / left-click hits that don't go through the normal
      attack path.
    • After swapping, calls maceDmgHack.doSmash() so the smash lands right
      after the mace is equipped.
  • KillauraHack
    • Made aura spear aware: when the held item has a PIERCING_WEAPON
      component, attacks route through gameMode.piercingAttack() (the STAB
      action packet) instead of gameMode.attack().

@Altidias Altidias changed the title Fix MaceDMG reliability with Killaura, spear hits, and AttributeSwap swaps Fix MaceDMG reliability with Killaura, spear hits, and AttributeSwap swaps + spear aura fix Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant