Extract loader-instruction crate

Cargo.toml

@@ -143,6 +143,7 @@ members = [
     "sdk/instructions-sysvar",
     "sdk/keccak-hasher",
     "sdk/keypair",
+    "sdk/loader-instruction",
     "sdk/logger",
     "sdk/macro",
     "sdk/message",
@@ -514,6 +515,7 @@ solana-keypair = { path = "sdk/keypair", version = "=2.2.0" }
 solana-last-restart-slot = { path = "sdk/last-restart-slot", version = "=2.2.0" }
 solana-lattice-hash = { path = "lattice-hash", version = "=2.2.0" }
 solana-ledger = { path = "ledger", version = "=2.2.0" }
+solana-loader-instruction = { path = "sdk/loader-instruction", version = "=2.2.0" }
 solana-loader-v4-program = { path = "programs/loader-v4", version = "=2.2.0" }
 solana-local-cluster = { path = "local-cluster", version = "=2.2.0" }
 solana-log-collector = { path = "log-collector", version = "=2.2.0" }

sdk/loader-instruction/Cargo.toml

@@ -0,0 +1,31 @@
+[package]
+name = "solana-loader-instruction"
+description = "Solana non-upgradable BPF loader instructions."
+documentation = "https://docs.rs/solana-loader-instruction"
+version = { workspace = true }
+authors = { workspace = true }
+repository = { workspace = true }
+homepage = { workspace = true }
+license = { workspace = true }
+edition = { workspace = true }
+
+[dependencies]
+serde = { workspace = true, optional = true }
+serde_bytes = { workspace = true, optional = true }
+serde_derive = { workspace = true, optional = true }
+solana-instruction = { workspace = true, features = ["bincode", "std"], optional = true }
+solana-pubkey = { workspace = true }
+solana-sdk-ids = { workspace = true }
+
+[features]
+bincode = ["dep:solana-instruction", "serde"]
+dev-context-only-utils = ["bincode"]
+serde = ["dep:serde", "dep:serde_bytes", "dep:serde_derive"]
+
+[package.metadata.docs.rs]
+targets = ["x86_64-unknown-linux-gnu"]
+all-features = true
+rustdoc-args = ["--cfg=docsrs"]
+
+[lints]
+workspace = true

sdk/program/src/loader_instruction.rs → sdk/loader-instruction/src/lib.rs

@@ -1,14 +1,18 @@
-//! Instructions for the [non-upgradable BPF loader][nubpfl].
-//!
-//! [nubpfl]: crate::bpf_loader
+//! Instructions for the non-upgradable BPF loader.
+#![cfg_attr(docsrs, feature(doc_auto_cfg))]
 
-use crate::{
-    instruction::{AccountMeta, Instruction},
-    pubkey::Pubkey,
-    sysvar::rent,
+#[cfg(feature = "bincode")]
+use {
+    solana_instruction::{AccountMeta, Instruction},
+    solana_pubkey::Pubkey,
+    solana_sdk_ids::sysvar::rent,
 };
 
-#[derive(Serialize, Deserialize, Debug, PartialEq, Eq, Clone)]
+#[cfg_attr(
+    feature = "serde",
+    derive(serde_derive::Deserialize, serde_derive::Serialize)
+)]
+#[derive(Debug, PartialEq, Eq, Clone)]
 pub enum LoaderInstruction {
     /// Write program data into an account
     ///
@@ -19,7 +23,7 @@ pub enum LoaderInstruction {
         offset: u32,
 
         /// Serialized program data
-        #[serde(with = "serde_bytes")]
+        #[cfg_attr(feature = "serde", serde(with = "serde_bytes"))]
         bytes: Vec<u8>,
     },
 
@@ -34,6 +38,7 @@ pub enum LoaderInstruction {
     Finalize,
 }
 
+#[cfg(feature = "bincode")]
 pub fn write(
     account_pubkey: &Pubkey,
     program_id: &Pubkey,
@@ -48,6 +53,7 @@ pub fn write(
     )
 }
 
+#[cfg(feature = "bincode")]
 pub fn finalize(account_pubkey: &Pubkey, program_id: &Pubkey) -> Instruction {
     let account_metas = vec![
         AccountMeta::new(*account_pubkey, true),

sdk/program/Cargo.toml

@@ -52,6 +52,7 @@ solana-instruction = { workspace = true, default-features = false, features = [
 solana-instructions-sysvar = { workspace = true }
 solana-keccak-hasher = { workspace = true, features = ["sha3"] }
 solana-last-restart-slot = { workspace = true, features = ["serde", "sysvar"] }
+solana-loader-instruction = { workspace = true, features = ["bincode"] }
 solana-message = { workspace = true, features = ["bincode", "blake3"] }
 solana-msg = { workspace = true }
 solana-native-token = { workspace = true }

sdk/program/src/lib.rs

@@ -485,7 +485,6 @@ pub mod hash;
 pub mod incinerator;
 pub mod instruction;
 pub mod lamports;
-pub mod loader_instruction;
 pub mod loader_upgradeable_instruction;
 pub mod loader_v4;
 pub mod loader_v4_instruction;
@@ -528,6 +527,11 @@ pub use solana_fee_calculator as fee_calculator;
 pub use solana_keccak_hasher as keccak;
 #[deprecated(since = "2.1.0", note = "Use `solana-last-restart-slot` crate instead")]
 pub use solana_last_restart_slot as last_restart_slot;
+#[deprecated(
+    since = "2.2.0",
+    note = "Use `solana-loader-instruction` crate instead"
+)]
+pub use solana_loader_instruction as loader_instruction;
 #[deprecated(since = "2.2.0", note = "Use `solana-message` crate instead")]
 pub use solana_message as message;
 #[deprecated(since = "2.1.0", note = "Use `solana-program-memory` crate instead")]

transaction-status/Cargo.toml

@@ -24,6 +24,7 @@ solana-account-decoder = { workspace = true }
 solana-clock = { workspace = true }
 solana-hash = { workspace = true }
 solana-instruction = { workspace = true }
+solana-loader-instruction = { workspace = true, features = ["bincode"] }
 solana-message = { workspace = true }
 solana-program = { workspace = true }
 solana-pubkey = { workspace = true }

transaction-status/src/parse_bpf_loader.rs

@@ -5,11 +5,9 @@ use {
     base64::{prelude::BASE64_STANDARD, Engine},
     bincode::deserialize,
     serde_json::json,
+    solana_loader_instruction::LoaderInstruction,
     solana_message::{compiled_instruction::CompiledInstruction, AccountKeys},
-    solana_program::{
-        loader_instruction::LoaderInstruction,
-        loader_upgradeable_instruction::UpgradeableLoaderInstruction,
-    },
+    solana_program::loader_upgradeable_instruction::UpgradeableLoaderInstruction,
 };
 
 pub fn parse_bpf_loader(
@@ -224,12 +222,8 @@ mod test {
         let account_keys = vec![fee_payer, account_pubkey];
         let missing_account_keys = vec![account_pubkey];
 
-        let instruction = solana_program::loader_instruction::write(
-            &account_pubkey,
-            &program_id,
-            offset,
-            bytes.clone(),
-        );
+        let instruction =
+            solana_loader_instruction::write(&account_pubkey, &program_id, offset, bytes.clone());
         let mut message = Message::new(&[instruction], Some(&fee_payer));
         assert_eq!(
             parse_bpf_loader(
@@ -258,8 +252,7 @@ mod test {
         )
         .is_err());
 
-        let instruction =
-            solana_program::loader_instruction::finalize(&account_pubkey, &program_id);
+        let instruction = solana_loader_instruction::finalize(&account_pubkey, &program_id);
         let mut message = Message::new(&[instruction], Some(&fee_payer));
         assert_eq!(
             parse_bpf_loader(