Skip to content

fix(#8479): recast elytra on aiStep tail - #8545

Open
Pro0101-2b2fr wants to merge 3 commits into
CCBlueX:nextgenfrom
Pro0101-2b2fr:fix/elytra-recast-handler
Open

fix(#8479): recast elytra on aiStep tail#8545
Pro0101-2b2fr wants to merge 3 commits into
CCBlueX:nextgenfrom
Pro0101-2b2fr:fix/elytra-recast-handler

Conversation

@Pro0101-2b2fr

Copy link
Copy Markdown
Contributor

Summary

Fixes #8479 — ElytraRecast module did literally nothing when enabled because it had no event handler to drive its logic.

Changes

  • Added a GameTickEvent handler (tickHandler) that calls recastElytra() each tick.
  • The shouldRecast predicate and recastElytra() method already existed — they just were never invoked.

Root cause

The module defined the recast logic and conditions but had no event subscription to actually trigger them. This was likely a regression from a refactor that removed the original handler without replacement.


This PR was developed with AI assistance.

@Pro0101-2b2fr
Pro0101-2b2fr force-pushed the fix/elytra-recast-handler branch from f420665 to e47b839 Compare June 22, 2026 22:13
@MukjepScarlet

Copy link
Copy Markdown
Contributor

recastElytra was called in MixinLivingEntity. Have you confirmed the issue?

@Pro0101-2b2fr
Pro0101-2b2fr force-pushed the fix/elytra-recast-handler branch from e47b839 to 0ddb0a3 Compare June 23, 2026 14:25
@Pro0101-2b2fr

Pro0101-2b2fr commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

recastElytra was called in MixinLivingEntity. Have you confirmed the issue?

It didn't work I don't know why, but mine is working now.
It flags grim "ElytraB no release" but I don't know how to fix it...
I also wasn't able to test on other anticheat.
Here is the video https://uploadnow.io/f/k83jNsB

@MukjepScarlet MukjepScarlet added the ❓ information needed Further information is requested label Jun 25, 2026
@MukjepScarlet
MukjepScarlet requested a review from 1zun4 June 25, 2026 06:25
@MukjepScarlet MukjepScarlet removed the ❓ information needed Further information is requested label Aug 11, 2026
@MukjepScarlet MukjepScarlet changed the title fix(#8479): add GameTickEvent handler to ElytraRecast module fix(#8479): recast elytra on aiStep tail Aug 11, 2026
@MukjepScarlet

Copy link
Copy Markdown
Contributor

@Pro0101-2b2fr Test again when you are available.

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.

[BUG] elytra recast

2 participants