Add noise suppression support for rule yeetUpdateSuppressionCrash

[VelizarBG]

For context (if needed): noise suppression has been released.
I'm gonna leave backporting to you since you know your setup better than I do.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
carpet-tis-addition	✅ Ready (Inspect)	Visit Preview	Aug 7, 2024 7:27pm

[Fallen-Breath]

Nice. Are these 2 @WrapOperation targets the only places where the blockEntity-blockState mismatch will happen for the sculk sensor IllegalArgumentException suppression thing?

[VelizarBG]

Yep, at least according to what I could find.

[Fallen-Breath]

Yep, at least according to what I could find.

Just a quick view, other blocks like sculk shrieker have similiar codes in net.minecraft.block.entity.SculkShriekerBlockEntity.VibrationCallback#accepts, maybe they are exploitable as well?

[VelizarBG]

I don't think the others are useful. The sculk shrieker BE only accepts the SCULK_SENSOR_TENDRILS_CLICKING game event and that one can only be emitted during the BE tick, making it only useful for crashing a server. The sculk catalyst can suppress only living entity death but you need to hook it up to an existing suppressor anyway.

[Fallen-Breath]

Fair. Just wait until other IEA application scenarios emerge. cuz if we patch it directly at net.minecraft.state.State#get, the potential impact will be too large