fix: Validate region ownership for AI sensor targets - #491
Open
PluginsKers wants to merge 1 commit into
Open
Conversation
|
请使用英文 哦，看错了我以为自己的项目哪 |
|
不过Folia也用英文，中文没人看 |
Validate entity ownership at AI sensor collection and consumption boundaries.
PluginsKers
force-pushed
the
fix/issue-1-region-owned-sensor-targets
branch
from
August 9, 2026 10:25
ff26903 to
ef3b376
Compare
PluginsKers
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Root cause
NearestLivingEntitySensorstores directLivingEntityreferences in Brain memory. Sensors refresh independently at the default 20-tick interval, so region ownership can change beforeNearestVisibleLivingEntities,AdultSensorAnyType, orBabyFollowAdultconsumes a cached target. The Bukkit target event can also replace the target with an entity owned by another region.Changes
Validate ownership before candidate state reads, cached visibility checks, distance calculations, and event-replaced target access. Erase stale
NEAREST_VISIBLE_ADULTmemory before stopping the follow behavior.These checks preserve Folia's ownership boundary instead of bypassing
CraftEntity#getHandlethread enforcement.Validation
.\gradlew.bat applyAllPatches --console=plain.\gradlew.bat build --console=plainRefs #203