Skip to content

[Phoenix] NM Respawn Persistence Module - #201

Open
KiplingFFXI wants to merge 3 commits into
betafrom
Timed-NM-Respawn-Persistence
Open

[Phoenix] NM Respawn Persistence Module#201
KiplingFFXI wants to merge 3 commits into
betafrom
Timed-NM-Respawn-Persistence

Conversation

@KiplingFFXI

Copy link
Copy Markdown

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 creates a module which keeps NM respawn windows across map server restarts. Right now a restart spawns every down NM fresh, so maintenance hands out free pops.

Each NM saves its remaining window when it goes down and registers it again at the next boot, so held NMs never spawn and pop on their own when the window ends. Covers 30 open world NMs with windows of an hour or more. Odqan, Carmine Dobsonfly and the Weeping Willow / Lumber Jack chain get their own handling since they share windows between mobs. WotG NMs are left out for now since those entities don't exist while that content is off.

Steps to test these changes

Load the module. Kill NMs. See NMs stay down even after map restarts.

@KiplingFFXI
KiplingFFXI force-pushed the Timed-NM-Respawn-Persistence branch from ed4768c to 2482ff5 Compare September 2, 2026 05:52

@Skold177 Skold177 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are all going to error on startup because we disable them - the script won't load even if you have ensure table. You can disable them for now and add them back later when Besieged is enabled.

Lamie_No7
Medusa
Merrow_No5
Dorgerwor_the_Astute
Darting_Kachaal_Ja
Dragonscaled_Bugaal_Ja
Gulool_Ja_Ja
Hundredfaced_Hapool_Ja

@KiplingFFXI
KiplingFFXI force-pushed the Timed-NM-Respawn-Persistence branch from 2482ff5 to 1ac844e Compare September 2, 2026 18:40
@KiplingFFXI

Copy link
Copy Markdown
Author

These are all going to error on startup because we disable them - the script won't load even if you have ensure table. You can disable them for now and add them back later when Besieged is enabled.

Lamie_No7 Medusa Merrow_No5 Dorgerwor_the_Astute Darting_Kachaal_Ja Dragonscaled_Bugaal_Ja Gulool_Ja_Ja Hundredfaced_Hapool_Ja

Fixed. Left detailed TODOs for all Besieged and WOTG NMs so we know what to update on this module later.

@KiplingFFXI
KiplingFFXI marked this pull request as draft September 2, 2026 22:07
@KiplingFFXI
KiplingFFXI marked this pull request as ready for review September 2, 2026 23:13
@KiplingFFXI
KiplingFFXI marked this pull request as draft September 2, 2026 23:46
@KiplingFFXI
KiplingFFXI force-pushed the Timed-NM-Respawn-Persistence branch from 1ac844e to 0d4686e Compare September 3, 2026 02:10
@KiplingFFXI KiplingFFXI added the hold Do not merge this PR label Sep 3, 2026
@KiplingFFXI
KiplingFFXI force-pushed the Timed-NM-Respawn-Persistence branch from 055d997 to be2f6d5 Compare September 6, 2026 16:14
@KiplingFFXI
KiplingFFXI marked this pull request as ready for review September 6, 2026 16:14
@KiplingFFXI KiplingFFXI removed the hold Do not merge this PR label Sep 6, 2026
@KiplingFFXI

Copy link
Copy Markdown
Author

This is good for another review.

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