context: remove seed_addr from account states (#377)

Author: mjain-jump

Cargo.lock

@@ -518,7 +518,7 @@ winapi = "0.3.8"
 x509-parser = "0.14.0"
 zeroize = { version = "1.7", default-features = false }
 zstd = "0.13.3"
-protosol = {git = "https://github.com/firedancer-io/protosol", tag = "v1.0.4"}
+protosol = {git = "https://github.com/firedancer-io/protosol", tag = "v1.0.5"}
 solfuzz-agave-macro = { path = "macro" }
 once_cell = "1.21.3"
 lazy_static = "1.5.0"

Cargo.toml

@@ -12,7 +12,7 @@
 import subprocess
 
 # NOTE: this needs bumped with schema version upgrades of the protocol
-PROTOSOL_VERSION_TAG = "v1.0.4"
+PROTOSOL_VERSION_TAG = "v1.0.5"
 
 def replace_path_with_git_rev(toml_data, git_url, rev):
     """
@@ -131,11 +131,11 @@ def main():
     # Add protosol dependency (deduped if already present)
     if "dependencies" not in toml_data:
         toml_data["dependencies"] = table()
-    
+
     # Remove existing protosol if present to ensure we use the correct version
     if "protosol" in toml_data["dependencies"]:
         del toml_data["dependencies"]["protosol"]
-    
+
     # Add the required protosol dependency
     protosol_dep = inline_table()
     protosol_dep["git"] = "https://github.com/firedancer-io/protosol"

scripts/generate_cargo.py

@@ -497,7 +497,6 @@ impl From<InstrEffects> for proto::InstrEffects {
                     lamports: account.lamports,
                     data: account.data.to_vec(),
                     executable: account.executable,
-                    seed_addr: None,
                 })
                 .collect(),
             cu_avail: val.cu_avail,
@@ -1127,7 +1126,6 @@ mod tests {
             lamports: 1,
             data: bincode::serialize(&sysvar).unwrap(),
             executable: false,
-            seed_addr: None,
         }
     }
 
@@ -1166,23 +1164,20 @@ mod tests {
                     lamports: 1000,
                     data: vec![],
                     executable: false,
-                    seed_addr: None,
                 },
                 proto::AcctState {
                     address: vec![2u8; 32],
                     owner: vec![0u8; 32],
                     lamports: 0,
                     data: vec![],
                     executable: false,
-                    seed_addr: None,
                 },
                 proto::AcctState {
                     address: vec![0u8; 32],
                     owner: native_loader_id.clone(),
                     lamports: 10000000,
                     data: b"Solana Program".to_vec(),
                     executable: true,
-                    seed_addr: None,
                 },
             ]),
             instr_accounts: vec![
@@ -1219,23 +1214,20 @@ mod tests {
                         lamports: 999,
                         data: vec![],
                         executable: false,
-                        seed_addr: None,
                     },
                     proto::AcctState {
                         address: vec![2u8; 32],
                         owner: vec![0u8; 32],
                         lamports: 1,
                         data: vec![],
                         executable: false,
-                        seed_addr: None,
                     },
                     proto::AcctState {
                         address: vec![0u8; 32],
                         owner: native_loader_id.clone(),
                         lamports: 10000000,
                         data: b"Solana Program".to_vec(),
                         executable: true,
-                        seed_addr: None,
                     },
                 ]),
                 cu_avail: 9850u64,

src/lib.rs

@@ -152,7 +152,6 @@ impl From<TransactionAccount> for proto::AcctState {
             data: value.1.data().to_vec(),
             executable: value.1.executable(),
             owner: value.1.owner().to_bytes().to_vec(),
-            seed_addr: None,
         }
     }
 }

src/txn_fuzzer.rs

@@ -54,7 +54,6 @@ fn get_clock_sysvar_account() -> AcctState {
         data: bincode::serialize(&clock).unwrap(),
         executable: false,
         owner: native_loader::id().to_bytes().to_vec(),
-        seed_addr: None,
     }
 }
 
@@ -72,7 +71,6 @@ fn get_epoch_schedule_sysvar_account() -> AcctState {
         data: bincode::serialize(&epoch_schedule).unwrap(),
         executable: false,
         owner: native_loader::id().to_bytes().to_vec(),
-        seed_addr: None,
     }
 }
 
@@ -88,7 +86,6 @@ fn get_rent_sysvar_account() -> AcctState {
         data: bincode::serialize(&rent).unwrap(),
         executable: false,
         owner: native_loader::id().to_bytes().to_vec(),
-        seed_addr: None,
     }
 }
 
@@ -119,7 +116,6 @@ fn deploy_program(name: String) -> [(Pubkey, AcctState); 2] {
         data: bincode::serialize(&state).unwrap(),
         executable: true,
         owner: bpf_loader_upgradeable::id().to_bytes().to_vec(),
-        seed_addr: None,
     };
 
     let state = UpgradeableLoaderState::ProgramData {
@@ -144,7 +140,6 @@ fn deploy_program(name: String) -> [(Pubkey, AcctState); 2] {
         data: header,
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     [
@@ -196,7 +191,6 @@ fn test_txn_execute_clock() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let mut program_info = deploy_program("clock-sysvar".to_string());
@@ -315,7 +309,6 @@ fn test_simple_transfer() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let sender = Pubkey::new_unique();
@@ -325,7 +318,6 @@ fn test_simple_transfer() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let recipient = Pubkey::new_unique();
@@ -335,7 +327,6 @@ fn test_simple_transfer() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let mut program_info = deploy_program("simple-transfer".to_string());
@@ -470,7 +461,6 @@ fn test_lookup_table() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let sender = Pubkey::new_unique();
@@ -480,7 +470,6 @@ fn test_lookup_table() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let recipient = Pubkey::new_unique();
@@ -490,7 +479,6 @@ fn test_lookup_table() {
         data: vec![],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let extra_account = Pubkey::new_unique();
@@ -500,7 +488,6 @@ fn test_lookup_table() {
         data: vec![5, 0, 0, 0, 0, 0, 0, 0],
         executable: false,
         owner: vec![0; 32],
-        seed_addr: None,
     };
 
     let mut program_info = deploy_program("complex-transfer".to_string());
@@ -534,7 +521,6 @@ fn test_lookup_table() {
         data: alut_data,
         executable: false,
         owner: address_lookup_table::id().to_bytes().to_vec(),
-        seed_addr: None,
     };
 
     let blockhash_queue = vec![