Skip to content

[core] Exclude mobs from Martial Arts calculations#9760

Merged
Xaver-DaRed merged 1 commit intoLandSandBoat:basefrom
UmeboshiXI:Martial_Arts_Mobs
Apr 11, 2026
Merged

[core] Exclude mobs from Martial Arts calculations#9760
Xaver-DaRed merged 1 commit intoLandSandBoat:basefrom
UmeboshiXI:Martial_Arts_Mobs

Conversation

@UmeboshiXI
Copy link
Copy Markdown
Contributor

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

Excludes mobs from Martial Arts auto attack delay reductions. This was a root cause of various "bugs" in past and currently where higher level MNK mobs will have what seems like permanent Hundred Fists since their delay becomes so low.

Tests to support that mobs don't benefit from Martial Arts:

  1. Go to Uleguerand Range and find a Molech near the top (I-5). Capture its attack delay/TP
  2. Go to Phomiuna Aqueducts and find a low level Taurus. Capture its attack delay/TP
  3. Both of their delays should be the same.

Steps to test these changes

  1. Go to I-5 Uleguerand Range and find a Molech. Use captain to record its attack delay. It should be around 390~ (What the mobs delay is set in mob_pools.sql). See that its attacks are not machine gunning you down faster than normal.

For reference: On current LSB(Not this PR), Molech hits at least 2 times (MNK H2H) at 240 Delay.

Note: Other references to Martial Arts in terms of TP lua scripts are already gated behind a player character check.

@Xaver-DaRed Xaver-DaRed merged commit 8e4fe56 into LandSandBoat:base Apr 11, 2026
11 checks passed
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.

2 participants