[aronwk-aaron]

On behalf of @aronwk-aaron (written by Claude).

Small fixes for property models and brick by brick, each checked against the 1.10.64 client.

• PlaceModelResponse writes the model's rotation. The client's PlaceModelResponse::Deserialize (0x00dc0170) reads the rotation as an optional w, x, y, z quaternion, but the server wrote the 4-byte response after the rotation flag, so any model placed with a rotation other than identity sent a message the client ran out of data reading.
• Naming a model no longer renames the property. The model info window sends UpdatePropertyOrModelForFilterCheck with isProperty false (client 0x00cc42e0); only the property editor sends it true (0x00c2e710). Storing a model's name and description is a follow-up.
• A model item keeps its blueprint across saves. The blueprintid config of a brick built model item wasn't saved in the item XML, so a picked up model couldn't be placed again after a reload.
• Model behaviors are written only for property and inventory models. The client reads the behavior info only when the object's config has propertyObjectID or inInventory (ObjectLoader2::LoadModelBehaviorsComponent); for other models it read those bits as the next component's data.
• The model vault uses the models' inventory groups. The client maps VAULT_MODELS to MODELS for groups (m_VaultToPlayerInventory); the server refused group updates from the vault.
• Tests: the server mock now keeps a copy of the last bitstream sent (it kept a pointer to the sender's stack). New tests for the place model response layout, model behavior info and vault groups.

🤖 Generated with Claude Code

[aronwk-aaron]

reviewed code myself and tested locally