Derive Serde for StarkProof Struct

[AbdelStark]

Title

Derive Serde for StarkProof Struct

Description

Currently, the StarkProof struct does not implement serialization and deserialization via serde, which is necessary for our application to easily convert proofs to and from JSON format. Implementing serde will also allow for better interoperability with other services that consume or produce JSON data.

Expected Behavior

After implementing serde:

• The StarkProof struct should be serializable to JSON format.
• The StarkProof struct should be deserializable from JSON format.

Possible Solution

• Add serde crate to Cargo.toml if not already included:

  [dependencies]
  serde = { version = "1.0", features = ["derive"] }
  serde_json = "1.0"

• Modify the StarkProof struct definition to derive Serialize and Deserialize:

  #[derive(Serialize, Deserialize)]
  pub struct StarkProof {
    // struct fields
  }

• Ensure that all fields within the StarkProof struct are also compatible with serde.

Additional Context

• This change will facilitate the development team in handling the proof data more efficiently, especially when interfacing with web applications.
• It's important to ensure that the serialization and deserialization process preserves all critical information within the struct without any data loss.

[manoahLinks]

can i hop on this ?

[MayowaObisesan]

Hi @AbdelStark Can I take up this issue?

[MSghais]

Can I jump on this, please? I go through the repo a bit and check what needs to be changed also on the attributes for Deserialization/Serialization. I think I can give an ETA in few hours

[AbdelStark]

Can I jump on this, please? I go through the repo a bit and check what needs to be changed also on the attributes for Deserialization/Serialization. I think I can give an ETA in few hours

Ok please give it a try.

[AbdelStark]

can i hop on this ?

Assigned to @MSghais for now. Please contribute to another Starknet related repo meanwhile.

[AbdelStark]

Hi @AbdelStark Can I take up this issue?

Assigned to @MSghais for now. Please contribute to another Starknet related repo meanwhile.

[MayowaObisesan]

Alright @AbdelStark . Thanks