When a wind-chime is part of a contraption and that contraption is disassembled back into blocks it crashes the world, and continues to do so whenever the world is loaded;
I think that is a result of this method

@Override
public boolean onSyncedBlockEvent(BlockState state, World world, BlockPos pos, int type, int data) {
    return world.getBlockEntity(pos).onSyncedBlockEvent(type, data);
}

with getBlockEntity returning null and attempting to call onSyncedBlockEvent on the null object causing the exception. Not sure why the block entity isn't present in the world properly though, especially after the first crash.

https://mclo.gs/TRl9EBW

The crash only occurs with some types of contraptions (including minecart assemblers). Windmills apparently don't crash, but behave very weirdly: https://cdn.discordapp.com/attachments/1101506920268963871/1133030068915032154/Does_Not_Like.mp4

Rotating bearings crash: https://cdn.discordapp.com/attachments/1101506920268963871/1133031785333276793/Does_Not_Like_2.mp4

my suggestion would be adding a null-check there

To reproduce, put a windchime on a minecart assembler, assemble it, then disassemble it. Assembling works fine, but disassembly causes a ticking entity crash, which effectively softlocks the world because it continues crashing when the chunks are loaded.

Versions:

Minecraft: 1.19.2
Quilt Loader: 0.18.10
QFAPI: 4.0.0-beta.30_qsl-3.0.0-beta.29_fapi-0.76.0
Windchimes: 1.2.1
(We also have a bunch of other mods, it's a custom modpack: https://github.com/HexxyCraft/modpack)

Logs:

Server: https://mclo.gs/Sph488x
Singleplayer: https://mclo.gs/TRl9EBW

i mean... i wrote the code against vanilla, and vanilla doesn't remove block entities without removing the block...

on it

@hibiii Sorry to bug you again, but is this patch available for 1.19.2 or do we need to fork and backport it for our modpack? (I'm fine with doing that, there'd just be no point if you're planning to release it officially.)

you could apply d9d0091's diff onto v1.2.1 and compile locally