refactor: cleanup

Signed-off-by: Pablo Maldonado <[email protected]>

Author: md0x

Diff for: programs/svm-spoke/src/instructions/deposit.rs

@@ -36,7 +36,7 @@ pub struct Deposit<'info> {
     )]
     pub state: Account<'info, State>,
 
-    /// CHECK: PDA derived with seeds ["delegate", delegate_seed_hash]; used as a CPI signer. No account data is read or written.
+    /// CHECK: PDA derived with seeds ["delegate", delegate_seed_hash]; used as a CPI signer.
     pub delegate: UncheckedAccount<'info>,
 
     #[account(

Diff for: programs/svm-spoke/src/instructions/fill.rs

@@ -182,8 +182,6 @@ fn unwrap_fill_relay_params(
     repayment_address: Option<Pubkey>,
     account: &Option<Account<FillRelayParams>>,
 ) -> FillRelayParams {
-    msg!("unwrap_fill_relay_params: repayment_chain_id: {:?}", repayment_chain_id);
-    msg!("unwrap_fill_relay_params: repayment_address: {:?}", repayment_address);
     match (relay_data, repayment_chain_id, repayment_address) {
         (Some(relay_data), Some(repayment_chain_id), Some(repayment_address)) => {
             FillRelayParams { relay_data, repayment_chain_id, repayment_address }

Diff for: programs/svm-spoke/src/utils/deposit_utils.rs

@@ -48,7 +48,6 @@ pub fn derive_deposit_delegate_seed_hash(
         exclusivity_parameter,
         message,
     };
-
     let serialized = data_struct.try_to_vec().unwrap();
 
     keccak::hash(&serialized).to_bytes()

Diff for: programs/svm-spoke/src/utils/fill_utils.rs

@@ -14,14 +14,6 @@ pub fn derive_fill_delegate_seed_hash(
 ) -> [u8; 32] {
     let data_struct = FillDelegateSeedData { relay_hash, repayment_chain_id, repayment_address };
     let serialized = data_struct.try_to_vec().unwrap();
-    // Log the raw bytes being hashed
-    msg!("derive_fill_delegate_seed_hash: relay_hash: {:?}", relay_hash);
-    msg!("derive_fill_delegate_seed_hash: repayment_chain_id: {:?}", repayment_chain_id);
-    msg!("derive_fill_delegate_seed_hash: repayment_address: {:?}", repayment_address);
 
-    // Compute keccak256
-    let hash_bytes = keccak::hash(&serialized).to_bytes();
-    // Log the resulting hash
-
-    hash_bytes
+    keccak::hash(&serialized).to_bytes()
 }

Diff for: src/svm/web3-v1/helpers.ts

@@ -25,7 +25,7 @@ export const isSolanaDevnet = (provider: AnchorProvider): boolean => {
 };
 
 /**
- * Borsh‐serializable struct matching your Rust DelegateSeedData
+ * Deposit Delegate Seed Data
  */
 class DepositDelegateSeedData {
   depositor!: Uint8Array;
@@ -56,7 +56,7 @@ class DepositDelegateSeedData {
 }
 
 /**
- * Borsh schema for DelegateSeedData
+ * Borsh schema for DepositDelegateSeedData
  */
 const delegateSeedSchema = new Map([
   [
@@ -93,17 +93,14 @@ export function getDepositDelegateSeedHash(depositData: DepositData): Uint8Array
     // Borsh will automatically prefix this with a 4‑byte LE length
     message: Uint8Array.from(depositData.message),
   });
-
-  // Serialize with borsh
-  const serialized = serialize(delegateSeedSchema, ds); // Uint8Array
+  const serialized = serialize(delegateSeedSchema, ds);
   const hashHex = ethers.utils.keccak256(serialized);
-  const seedHash = Buffer.from(hashHex.slice(2), "hex");
-  return seedHash;
+
+  return Buffer.from(hashHex.slice(2), "hex");
 }
 
 /**
- * Returns the delegate PDA for a deposit, Borsh‐serializing exactly the same fields
- * and ordering as your Rust `derive_deposit_delegate_seed_hash`.
+ * Returns the delegate PDA for a deposit.
  */
 export function getDepositDelegatePda(depositData: DepositData, programId: PublicKey): PublicKey {
   const seedHash = getDepositDelegateSeedHash(depositData);
@@ -113,8 +110,9 @@ export function getDepositDelegatePda(depositData: DepositData, programId: Publi
 
   return pda;
 }
+
 /**
- * Fill‐delegate seed data for relays
+ * Fill Delegate Seed Data
  */
 class FillDelegateSeedData {
   relayHash: Uint8Array;
@@ -145,8 +143,7 @@ const fillDelegateSeedSchema = new Map<any, any>([
 ]);
 
 /**
- * Computes the delegate seed hash for a fill relay operation.
- * Must match the Rust `derive_fill_delegate_seed_hash` logic exactly.
+ * Returns the fill delegate seed hash.
  */
 
 export function getFillRelayDelegateSeedHash(
@@ -160,12 +157,12 @@ export function getFillRelayDelegateSeedHash(
     repaymentAddress: repaymentAddress.toBuffer(),
   });
   const serialized = serialize(fillDelegateSeedSchema, ds);
+
   return Buffer.from(ethers.utils.keccak256(serialized).slice(2), "hex");
 }
 
 /**
- * Derives the program address (PDA) for a fill relay delegate.
- * Returns only the PDA; bump can be retrieved via `findProgramAddressSync` if needed.
+ * Returns the fill delegate PDA.
  */
 export function getFillRelayDelegatePda(
   relayHash: Uint8Array,
@@ -175,5 +172,6 @@ export function getFillRelayDelegatePda(
 ): { seedHash: Uint8Array; pda: PublicKey } {
   const seedHash = getFillRelayDelegateSeedHash(relayHash, repaymentChainId, repaymentAddress);
   const [pda] = PublicKey.findProgramAddressSync([Buffer.from("delegate"), seedHash], programId);
+
   return { seedHash, pda };
 }