Skip to content

Fix Folia border task world access#737

Open
GenOpenDiesel wants to merge 1 commit into
ChanceSD:masterfrom
GenOpenDiesel:fix-folia-border-task-thread-check
Open

Fix Folia border task world access#737
GenOpenDiesel wants to merge 1 commit into
ChanceSD:masterfrom
GenOpenDiesel:fix-folia-border-task-thread-check

Conversation

@GenOpenDiesel

Copy link
Copy Markdown

Summary

  • Run Folia anti-border-hopping checks on each player's scheduler before reading world/region state.
  • Use a concurrent last-location map because checks may now execute from different region threads.

Test plan

  • java -cp gradle\wrapper\gradle-wrapper.jar org.gradle.wrapper.GradleWrapperMain build

Run border-hopping region checks on the player scheduler under Folia so world reads happen on the owning region thread.

Co-authored-by: Cursor <cursoragent@cursor.com>
@deepsource-io

deepsource-io Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

DeepSource Code Review

We reviewed changes in f637ccc...e3d274d on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Java Jun 27, 2026 10:03a.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

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.

1 participant