feat: add metadata_hash field to Escrow struct for IPFS agreement storage

apps/onchain/src/fee_tests.rs

@@ -1,5 +1,5 @@
 use crate::{Error, Milestone, MilestoneStatus, VaultixEscrow, VaultixEscrowClient};
-use soroban_sdk::symbol_short;
+use soroban_sdk::{symbol_short, BytesN};
 /// Comprehensive tests for the Configurable Fee Model feature (#93)
 /// Tests cover:
 /// - Default global fee behavior (no overrides)
@@ -139,6 +139,7 @@ fn test_release_milestone_uses_global_fee_by_default() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -195,6 +196,7 @@ fn test_release_milestone_uses_token_fee_override() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -255,6 +257,7 @@ fn test_release_milestone_uses_escrow_fee_override() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -314,6 +317,7 @@ fn test_fee_precedence_escrow_over_token_and_global() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -367,6 +371,7 @@ fn test_cancel_escrow_uses_token_fee_override() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -423,6 +428,7 @@ fn test_refund_expired_uses_escrow_fee_override() {
         &token_address,
         &milestones,
         &deadline,
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     token_client.approve(&depositor, &contract_id, &10_000, &200);
@@ -482,6 +488,7 @@ fn test_zero_fee_valid() {
         &token_address,
         &milestones,
         &(env.ledger().timestamp() + 3600),
+        &BytesN::from_array(&env, &[0u8; 32]),
     );
 
     // Approve contract to transfer depositor's tokens, then deposit

apps/onchain/src/lib.rs

@@ -2,8 +2,8 @@
 #![no_std]
 #![allow(unexpected_cfgs)]
 use soroban_sdk::{
-    contract, contracterror, contractimpl, contracttype, symbol_short, token, Address, Env, Symbol,
-    Vec,
+    contract, contracterror, contractimpl, contracttype, symbol_short, token, Address, BytesN, Env,
+    Symbol, Vec,
 };
 
 impl VaultixEscrow {
@@ -89,6 +89,7 @@ pub struct Escrow {
     pub threshold_amount: i128, // Threshold amount for multi-sig requirement
     pub required_signatures: u32, // Number of signatures required for release
     pub collected_signatures: Vec<Address>, // Addresses that have signed for release
+    pub metadata_hash: BytesN<32>, // IPFS metadata hash for the escrow agreement
 }
 
 #[contracttype]
@@ -106,6 +107,7 @@ struct EscrowEntryV2 {
     required_signatures: u32,
     collected_signatures: Vec<Address>,
     fee_override_bps: i128,
+    metadata_hash: BytesN<32>,
 }
 
 #[contracttype]
@@ -117,6 +119,7 @@ pub struct CreateEscrowRequest {
     pub token_address: Address,
     pub milestones: Vec<Milestone>,
     pub deadline: u64,
+    pub metadata_hash: BytesN<32>,
 }
 
 #[contracttype]
@@ -484,6 +487,7 @@ impl VaultixEscrow {
         Ok(())
     }
 
+    #[allow(clippy::too_many_arguments)]
     pub fn create_escrow(
         env: Env,
         escrow_id: u64,
@@ -492,6 +496,7 @@ impl VaultixEscrow {
         token_address: Address,
         milestones: Vec<Milestone>,
         deadline: u64,
+        metadata_hash: BytesN<32>,
     ) -> Result<(), Error> {
         depositor.require_auth();
         ensure_not_paused(&env)?;
@@ -545,6 +550,7 @@ impl VaultixEscrow {
             required_signatures: 1,
             collected_signatures: Vec::new(&env),
             fee_override_bps,
+            metadata_hash: metadata_hash.clone(),
         };
 
         store_escrow_entry_v2(&env, escrow_id, &escrow);
@@ -556,7 +562,14 @@ impl VaultixEscrow {
                 Symbol::new(&env, "EscrowCreated"),
                 escrow_id,
             ),
-            (depositor, recipient, token_address, total_amount, deadline),
+            (
+                depositor,
+                recipient,
+                token_address,
+                total_amount,
+                deadline,
+                metadata_hash,
+            ),
         );
 
         Ok(())
@@ -581,6 +594,7 @@ impl VaultixEscrow {
             let token_address = request.token_address.clone();
             let milestones = request.milestones.clone();
             let deadline = request.deadline;
+            let metadata_hash = request.metadata_hash.clone();
 
             if depositor == recipient {
                 return Err(Error::SelfDealing);
@@ -645,6 +659,7 @@ impl VaultixEscrow {
                 required_signatures: 1,
                 collected_signatures: Vec::new(&env),
                 fee_override_bps,
+                metadata_hash,
             };
 
             pending_entries.push_back((escrow_id, escrow, fee_override_bps >= 0));
@@ -1669,6 +1684,7 @@ fn load_escrow_entry_v2(env: &Env, escrow_id: u64) -> Result<EscrowEntryV2, Erro
         required_signatures: legacy.required_signatures,
         collected_signatures: legacy.collected_signatures,
         fee_override_bps,
+        metadata_hash: BytesN::from_array(env, &[0u8; 32]),
     };
 
     env.storage().persistent().remove(&legacy_key);
@@ -1696,6 +1712,7 @@ fn escrow_entry_to_public(escrow: EscrowEntryV2) -> Escrow {
         threshold_amount: escrow.threshold_amount,
         required_signatures: escrow.required_signatures,
         collected_signatures: escrow.collected_signatures,
+        metadata_hash: escrow.metadata_hash,
     }
 }