storgae clean up

contracts/escrow/src/errors.rs

@@ -55,6 +55,8 @@ impl core::fmt::Display for Error {
             Error::ApprovalRequestAlreadyExecuted => write!(f, "Large-transfer approval request already executed"),
             Error::ApprovalRequestCancelled => write!(f, "Large-transfer approval request was cancelled"),
             Error::LargeTransferApprovalRequired => write!(f, "Transfer requires multi-step approval due to large amount"),
+            Error::FeeRateTooHigh => write!(f, "Fee rate exceeds maximum allowed (1000 bps = 10%)"),
+            Error::InvalidFeeAmount => write!(f, "Fee calculation resulted in an invalid amount"),
         }
     }
 }
@@ -107,6 +109,12 @@ impl ContractError for Error {
             Error::LargeTransferApprovalRequired => {
                 propchain_traits::errors::escrow_codes::LARGE_TRANSFER_APPROVAL_REQUIRED
             }
+            Error::FeeRateTooHigh => {
+                propchain_traits::errors::escrow_codes::FEE_RATE_TOO_HIGH
+            }
+            Error::InvalidFeeAmount => {
+                propchain_traits::errors::escrow_codes::INVALID_FEE_AMOUNT
+            }
         }
     }
 

contracts/escrow/src/lib.rs

@@ -76,6 +76,10 @@ mod propchain_escrow {
         fee_rate_bps: u16,
         /// Fee recipient account
         fee_recipient: Option<AccountId>,
+        /// Escrow analytics data
+        analytics: EscrowAnalytics,
+        /// Unique participant tracking for analytics
+        analytics_participants: Mapping<AccountId, bool>,
     }
 
     // Events
@@ -323,12 +327,13 @@ mod propchain_escrow {
                 large_transfer_requests: Mapping::default(),
                 large_transfer_request_count: 0,
                 escrow_active_large_transfer: Mapping::default(),
-                // 0 means "use global constant from propchain_traits::constants"
                 large_transfer_threshold: 0,
                 very_large_transfer_threshold: 0,
                 tax_compliance_contract,
                 fee_rate_bps: 0,
                 fee_recipient: None,
+                analytics: EscrowAnalytics::default(),
+                analytics_participants: Mapping::default(),
             }
         }
 

contracts/escrow/src/types.rs

@@ -223,3 +223,22 @@ pub struct EscrowAnalytics {
     /// Number of unique participants (buyers + sellers)
     pub unique_participants: u64,
 }
+
+impl Default for EscrowAnalytics {
+    fn default() -> Self {
+        Self {
+            total_created: 0,
+            total_released: 0,
+            total_refunded: 0,
+            total_disputed: 0,
+            total_active: 0,
+            total_volume: 0,
+            total_released_volume: 0,
+            total_fees_collected: 0,
+            average_escrow_amount: 0,
+            average_dispute_resolution_time: 0,
+            total_disputes_resolved: 0,
+            unique_participants: 0,
+        }
+    }
+}

contracts/staking/src/tests.rs

@@ -1588,5 +1588,4 @@ fn set_early_withdrawal_penalty_max_cap() {
         assert_eq!(vesting_after_claim.claimable_at_block(200), 0);
         assert_eq!(vesting_after_claim.claimable_at_block(300), 500);
     }
-}