Skip to content

[C++] [lua] Fix Repair Kit / Mana Tank Interactions#10326

Merged
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Skold177:Fix-Repair-Kit-/-Mana-Tank-Interactions
Jun 16, 2026
Merged

[C++] [lua] Fix Repair Kit / Mana Tank Interactions#10326
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Skold177:Fix-Repair-Kit-/-Mana-Tank-Interactions

Conversation

@Skold177

@Skold177 Skold177 commented Jun 15, 2026

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?

This PR fixes how Repair Kit 1-4 and Mana Tank 1-4 are handled for automatons. Previously these were just flat HP Percent / MP Percent mods that we slapped on after spawn. The issue with that is the HP/MP boost these attachments provide is supposed to be reflected in the equipment menu before you even summon them. The other issue is the HP / MP boosts we were giving was static between all frames when it was supposed to be frame specific. So...

  • Auto Repair Kit 1-4 HP Boost and the Mana Tank 1-4 MP Boost is now calculated in core with a lookup table living in automaton.lua that explains how the HP/MP boost works. Each frame has a divisor that calculates the contribution of the attachment. For example : Auto Repair Kit IV has a value of 4 and the Valoredge frame has a divisor of 24. That means it's HP boost = 4 / 24 = 16.66%~ HP boost. Since these are now calculated core, this also fixes the equipment window showing the HP / MP boost before you summon them.

  • Updated the puppetmaster equipment menu packet to update every time you switch attachments so this display works properly, before it was only updated when changing frames.

  • Includes a correction to Stormwaker melee skill, something I discovered I had made a mistake on while testing this PR. Changed from C to C_PLUS.

Side by side comparison of this calculation on retail vs my local

Valoredge with Repair Kit IV
image

Another spot check
Stormwaker with Repair Kit II and Mana Tank IV
image

Source

Repair Kit
https://wiki.ffo.jp/html/8619.html

Mana Tank
https://wiki.ffo.jp/html/5330.html

Steps to test these changes

Equip different frames, mix/match various frames/repair kits/mana tanks - see changes reflected in equipment window and HP/MP match retail.

Fixes the handling of repair kit and mana tank with captures from retail.
@Skold177 Skold177 force-pushed the Fix-Repair-Kit-/-Mana-Tank-Interactions branch from e443d2f to ca9bc14 Compare June 15, 2026 21:18
@Xaver-DaRed Xaver-DaRed merged commit ef9705c into LandSandBoat:base Jun 16, 2026
10 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