wrap-controller-fixes-and-gametests into main #1

Merged

Flexhd merged 2 commits from wrap-controller-fixes-and-gametests into main

2026-08-05 17:22:20 +00:00

Flexhd commented

2026-08-05 17:22:08 +00:00

Owner

No description provided.

Flexhd added 2 commits

2026-08-05 17:22:08 +00:00

Switch wrapping to WorldWrapController and cover it with gametests 9bb7c69dad

Enable WorldWrapController and delete PlayerEntityMixin. The mixin compared
X against getBoundNorth/getBoundSouth (Z bounds) and wrapped via x * -1, so
it only worked on a border centred on the origin; it treated fluid as a safe
landing, checked a single block for a two-block-tall player, never fired for
riding players because travel() is not called while mounted, and did no chunk
preloading.

Fix the controller's own bugs along the way:

- An axis with no crossing was snapped to the low wall instead of being
  carried through, so walking north over the border also yanked the player
  to the west wall and preloaded the wrong chunk.
- Use RegistryKey instead of System.identityHashCode for world identity.
- Release preload state on disconnect, on server stop, and on leaving the
  band; it previously leaked an entry per player.
- Refresh the approach ticket at half its lifetime; the refresh interval
  equalled the ticket lifetime, leaving a window where the destination
  could unload.
- The "synchronous fallback" chunk lookup discarded its result and forced
  nothing, and the final getTopY fallback could read an unloaded heightmap.
- Make the dry-landing ring scan linear in the radius rather than cubic.

Defer instead of blocking when the destination is not resident. wrap() now
checks readiness without forcing, raises a ticket, and postpones; the player
keeps moving into the opaque fog and the wall opens once generation catches
up. The wait is bounded so a destination that never loads cannot pin anyone
against the border. Blocking is now only reachable after three seconds.

Extract WrapBounds so the wrap can be driven over an explicit box, since the
world border is global state a test cannot safely reshape.

Add 44 gametests: wall and corner geometry, off-centre borders, inset
ordering across sizes, arrival never re-entering a trigger band, lava, water,
low ceilings, pillars, flight and spectator altitude, velocity, yaw/pitch,
fall distance, boats with riders, preloading, and deferral.

Not covered: client-side smoothness of the velocity-carrying teleport packet,
elytra gliding, the END_WORLD_TICK wiring against a real border, and the
bounded-wait failsafe.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Improve border fog and add teleport API 9e89753425

Flexhd merged commit 2eb7fc1418 into main

2026-08-05 17:22:20 +00:00

Flexhd referenced this pull request from a commit

2026-08-05 17:22:22 +00:00

Merge pull request 'wrap-controller-fixes-and-gametests into main' (#1) from wrap-controller-fixes-and-gametests into main