Use a tag or predicate to determine the blocks that are not movable with pistons, even with movableBlockEntities

[James103]

Currently, the list of non-movable block entities is hard-coded, as shown below:

fabric-carpet/src/main/java/carpet/mixins/PistonBaseBlock_movableBEMixin.java

Lines 54 to 56 in 2dd8674

	return block != Blocks.ENDER_CHEST && block != Blocks.ENCHANTING_TABLE &&
	block != Blocks.END_GATEWAY && block != Blocks.END_PORTAL && block != Blocks.MOVING_PISTON &&
	block != Blocks.SPAWNER;

Would it be possible to take that list, put it in a tag, and make movableBlockEntities use that tag when determining whether or not to let that block entity move?

[ch-yx]

maybe a Predicate (https://minecraft.wiki/w/Predicate) is better. you can write blockstates inside it, but block tag can only have block ids.
i m thinking using a special predicate __scarpet:piston_movable is considerable.
i use this name is because that __scarpet_marker is a special name already used by this mod.(here)