Uhhhhhhhhh I don't think this is correct at all...

1.20.4

public class PacketPlayOutEntityTeleport implements Packet<PacketListenerPlayOut> {

    private final int id;
    private final double x;
    private final double y;
    private final double z;
    private final byte yRot;
    private final byte xRot;
    private final boolean onGround;

    public PacketPlayOutEntityTeleport(Entity entity) {
        this.id = entity.getId();
        Vec3D vec3d = entity.trackingPosition();

        this.x = vec3d.x;
        this.y = vec3d.y;
        this.z = vec3d.z;
        this.yRot = (byte) ((int) (entity.getYRot() * 256.0F / 360.0F));
        this.xRot = (byte) ((int) (entity.getXRot() * 256.0F / 360.0F));
        this.onGround = entity.onGround();
    }

1.21.11

public class PacketPlayOutEntityTeleport implements Packet<PacketListenerPlayOut> {

    public static final StreamCodec<PacketDataSerializer, PacketPlayOutEntityTeleport> STREAM_CODEC = Packet.codec(PacketPlayOutEntityTeleport::write, PacketPlayOutEntityTeleport::new);
    private final int id;
    private final double x;
    private final double y;
    private final double z;
    private final byte yRot;
    private final byte xRot;
    private final boolean onGround;

    public PacketPlayOutEntityTeleport(Entity entity) {
        this.id = entity.getId();
        Vec3D vec3d = entity.trackingPosition();

        this.x = vec3d.x;
        this.y = vec3d.y;
        this.z = vec3d.z;
        this.yRot = (byte) ((int) (entity.getYRot() * 256.0F / 360.0F));
        this.xRot = (byte) ((int) (entity.getXRot() * 256.0F / 360.0F));
        this.onGround = entity.onGround();
    }

PositionMoveRotation is not a thing

@copilot add some unit tests to verify that you can modify the packet and read the proper value back

@copilot you also have a few compilation errors

@copilot add some unit tests to verify that you can modify the packet and read the proper value back

Added WrappedPositionMoveRotationTest in commit bec4eac with 4 tests:

• testCreateAndRead — verifies all fields (position, deltaMovement, yRot, xRot) round-trip correctly through create()
• testEntityPositionSyncPacket — writes to an ENTITY_POSITION_SYNC packet and reads all fields back
• testEntityTeleportPacket — same for ENTITY_TELEPORT packet
• testFromHandle — verifies fromHandle() correctly wraps an existing NMS handle

Also fixed 3 compilation errors that were caught by CI: renamed the private static getHandleType() (which conflicted with AbstractWrapper.getHandleType()) to getNmsClass(), and fixed StructureModifier<Float> unboxing with an explicit typed local variable.

@copilot you also have a few compilation errors

Those were fixed in commit bec4eac:

1. Renamed getHandleType() → getNmsClass() to avoid the "overriding method is static" error from shadowing AbstractWrapper.getHandleType()
2. Fixed the float unboxing errors in getYRot()/getXRot() by using explicit StructureModifier<Float> local variables instead of the generic type witness .<Float>read(n)

@copilot you have the wrong package name for PositionMoveRotation

@copilot you have the wrong package name for PositionMoveRotation

Fixed in ea01331. PositionMoveRotation lives in net.minecraft.world.entity, not net.minecraft.network.protocol.game — updated getPositionMoveRotationClass() accordingly.

@copilot almost there. for the testFromHandle test, create a NMS PositionMoveRotation from some constant values and test that the created wrapper has those values

@copilot almost there. for the testFromHandle test, create a NMS PositionMoveRotation from some constant values and test that the created wrapper has those values

Updated in e8a382e. testFromHandle now calls create(...).getHandle() to get the raw NMS object, then wraps it with fromHandle() and asserts each field against the known constant values directly.