Skip to content

Conversation

@CriticalXI
Copy link
Contributor

@CriticalXI CriticalXI commented Nov 18, 2025

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?

  • Implements a mixin to control Fomor parties within various zones. Sets up two types of party behavior:
    • Patrol: Parties follow a set leader. If the leader dies, all idle party members despawn. If followers die, they do not respawn unless the leader dies.
    • Guard: Fomors do not roam and guard a set position. None of these party members respawns unless all members of the party die.
  • All mobs in a party superlink with each other.
  • Fomors do not link unless they are in a party with each other.
  • There was an issue with the follow pathing logic that followers would never stop roaming. Adjusted the logic to only path if the leader is roaming.

Steps to test these changes

  • Kill various fomors throughout Phomiuna Aqueducts on the listed fomor party list, see they reflect the listed changes.

@CriticalXI CriticalXI changed the title [lua, sql, c++] Fomor Party Mixin with Phominua Aqueduct Setup [lua, sql, c++] Fomor Party Mixin with Phomiuna Aqueduct Setup Nov 18, 2025
@CriticalXI CriticalXI force-pushed the phomiuma_fomor_audit branch 5 times, most recently from d6cbc04 to 835c86f Compare November 26, 2025 20:57
@CriticalXI
Copy link
Contributor Author

Rebased

@CriticalXI
Copy link
Contributor Author

Just following up on this one, is something missing on my end that I need to adjust?

@Xaver-DaRed Xaver-DaRed merged commit 07ead46 into LandSandBoat:base Dec 16, 2025
18 of 19 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.

3 participants