[intersecato]

Overview

Fixes #3151

PaperweightGetBlocks#internalCall copies nmsChunk.getBlockEntities() into a new HashMap and then removes block entities from the chunk while running off the main thread. If the server modifies the same map at the same time (e.g. block entities being added/removed by ticking or chunk loading), the copy can fail with:

java.lang.NullPointerException: Cannot read field "wrapped" because "this.wrapped" is null at PaperweightGetBlocks.internalCall(PaperweightGetBlocks.java:349)

Description

The copy of the block entity map and the removal of the existing tiles (including the beacon deactivation handling) now run inside TaskManager.taskManager().sync(...), which returns the list of removed beacons. Everything after this block is unchanged, so the later beacon handling still uses the same beacons list.

This follows the approach of #3152, which only covered the 1.20.2–1.21.4 adapters. This PR applies it to every adapter currently on main:

• 1.21 (v1_21_R1)
• 1.21.4
• 1.21.5
• 1.21.6
• 1.21.9
• 1.21.11
• 26.1
• 26.2
• 26.3

Submitter Checklist

• Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
• Ensure that the pull request title represents the desired changelog entry.
• New public fields and methods are annotated with @since TODO.
• I read and followed the contribution guidelines.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.