Refactor RawFeltVec

[cptartur]

Stack:

• Make oracles support not experimental #3926
• Run oracles tests on CI #3925
• Fix typos #3867
• Integrate oracles result handling #3850
• Refactor RawFeltVec #3849 ⬅
• Resolve #2074 #3848
• Add oracles test #3847

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

[ksew1]

Maybe we could name it SerializedValue? As this i think is only usefull if value is serialized

[cptartur]

Renaming

[ksew1]

I would also drop generic. Only valid representation i would say is pub struct SerializedValue(pub Vec<Felt>)

[cptartur]

There's one use case where we want to write serialized array of ResourceBounds structs into memory in serialized format and we do it as so

&SerializedValue::new(resource_bounds).serialize_to_vec(),

We can change serialized value to be a Vec<Felt> but we would either have to provide an constructor that allows creating it from other values that implement serialize (we would have to do the serialization at construction time though) or change the usage to something along the lines of

&resource_bounds
                .iter()
                .flat_map(conversions::serde::serialize::SerializeToFeltVec::serialize_to_vec)
                .collect::<Vec<_>>(),

which kind of duplicates the logic.

[ksew1]

Is this needed?

[cptartur]

Yes, we need it to .read() the already serialized data that oracle returns.