[Feature] Optimize WithOtherFields deserialization

[mattsse]

Component

rpc, serde

Describe the feature you would like

we currently use this workaround:

alloy/crates/serde/src/other/mod.rs

Lines 235 to 257 in 3866c39

	fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
	where
	D: serde::Deserializer<'de>,
	{
	#[derive(Deserialize)]
	struct WithOtherFieldsHelper<T> {
	#[serde(flatten)]
	inner: T,
	#[serde(flatten)]
	other: OtherFields,
	}
	let mut helper = WithOtherFieldsHelper::deserialize(deserializer)?;
	// remove all fields present in the inner struct from the other fields, this is to avoid
	// duplicate fields in the catch all other fields because serde flatten does not exclude
	// already deserialized fields when deserializing the other fields.
	if let Value::Object(map) =
	serde_json::to_value(&helper.inner).map_err(serde::de::Error::custom)?
	{
	for key in map.keys() {
	helper.other.remove(key);
	}
	}

however we can likely achieve the same thing if the have a manual deserialize impl + visitor and use serde's helper

see also expanded code

Additional context

No response

[Ayushdubey86]

I'll look into this

[PanGan21]

Hey I gave it a try in #2110 . Looking for reviews 🙂