Add serialize method to ReflectSerialize (bevy_reflect)

[Mysvac]

Objective

This change allows direct use of .serialize(value, serializer) instead of the more verbose .get_serializable(value).serialize(serializer).

Currently, ReflectDeserialize, ReflectDeserializeWithRegistry, and ReflectSerializeWithRegistry all provide a deserialize/serialize method directly. It is inconsistent and somewhat odd that ReflectSerialize only provides get_serializable.

While the returned Serializable object does expose a serialize method, I believe ReflectSerialize should offer a direct serialize method for better ergonomics.

The existing get_serializable method can still be used to obtain the Serializable struct for testing or other specialized use cases. However, for typical serialization tasks, the direct .serialize(...) approach is clearer and more concise.

Solution

Add serialize function for ReflectSerialize.

Testing

• cd crates/bevy_reflect; cargo test
• cd crates/bevy_ecs; cargo test
• cd crates/bevy_asset; cargo test
• cargo run --example serialization

---

Showcase

Click to view showcase

// bevy_reflect  type_registry.rs
#[derive(Clone)]
pub struct ReflectSerialize {
    get_serializable: fn(value: &dyn Reflect) -> Serializable,
}

impl ReflectSerialize {
    // keep function `get_serializable()` unchanged .

    /// Serializes a reflected value.
    pub fn serialize<S>(&self, value: &dyn Reflect, serializer: S) -> Result<S::Ok, S::Error>
    where
        S: serde::Serializer,
    {
        (self.get_serializable)(value).serialize(serializer)
    }
}

// bevy_reflect  custom_serialization.rs
/*
Ok(reflect_serialize
    .get_serializable(value)
    .borrow()
    .serialize(serializer))
*/
Ok(reflect_serialize.serialize(value, serializer))