Skip to content

Guard navigation against unloaded target chunks#784

Open
Smorki wants to merge 1 commit into
Winds-Studio:ver/26.2from
Smorki:fix/navigation-unloaded-target-chunk-pr
Open

Guard navigation against unloaded target chunks#784
Smorki wants to merge 1 commit into
Winds-Studio:ver/26.2from
Smorki:fix/navigation-unloaded-target-chunk-pr

Conversation

@Smorki

@Smorki Smorki commented Jun 21, 2026

Copy link
Copy Markdown

No description provided.

@Smorki

Smorki commented Jun 21, 2026

Copy link
Copy Markdown
Author

This fixes a crash during async teleportation where navigiation can tick toward a target before the destination chunk is fully loaded. It adds a loaded-chunkguard before accepting the processed path.

@Smorki

Smorki commented Jul 5, 2026

Copy link
Copy Markdown
Author

Hey

@Smorki

Smorki commented Jul 5, 2026

Copy link
Copy Markdown
Author

İs anyone there?

@Dreeam-qwq

Copy link
Copy Markdown
Member

This fixes a crash during async teleportation where navigiation can tick toward a target before the destination chunk is fully loaded. It adds a loaded-chunkguard before accepting the processed path.

Hi, can you send the stacktrace of the crash, or the full log at that time?

Btw, I assume it is handled by the chunk system, so this loaded-chunk guard becomes useless under this scenario. Did you test in production that this get chunk condition can actually be met?

@RuleGaed

This comment was marked as off-topic.

1 similar comment
@toprakdevx

This comment was marked as off-topic.

@Smorki

Smorki commented Jul 7, 2026

Copy link
Copy Markdown
Author

I am currently away from my development setup, so I can't pull the exact stacktrace right now, but I will send it as soon as I can.
Under heavy load, async teleportation sometimes outruns the chunk loading threads, creating a race condition where the navigation ticks before the chunk system finishes registration. This guard was meant as a safety layer for that specific edge case.
I will double-check my environment logs and share the details with you soon!

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.

4 participants