Feat/issue 208 contributor onboarding

.github/workflows/ci.yml

@@ -34,6 +34,8 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
+      - name: Install system dependencies
+        run: sudo apt-get update && sudo apt-get install -y libudev-dev
       - name: Build
         run: cargo build --locked
       - name: Test
@@ -47,6 +49,8 @@ jobs:
       - uses: dtolnay/rust-toolchain@stable
         with:
           components: clippy
+      - name: Install system dependencies
+        run: sudo apt-get update && sudo apt-get install -y libudev-dev
       - name: Run Clippy
         run: cargo clippy --all-targets --all-features --locked -- -D warnings
 
@@ -56,6 +60,8 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
+      - name: Install system dependencies
+        run: sudo apt-get update && sudo apt-get install -y libudev-dev
       - name: Build
         run: cargo build --locked
       - name: Rust smoke tests

Cargo.toml

@@ -45,13 +45,14 @@ tracing-subscriber = { version = "0.3", features = ["env-filter", "json", "fmt"]
 tracing-appender = "0.2"
 bip39 = { version = "2", features = ["rand", "std"] }
 wasmparser = "0.116.0"
-wasmprinter = "0.3"
+wasmprinter = "0.252"
 wat = "1.0"
 hmac = "0.12"
 rustyline = "14.0.0"
 zip = "0.6"
 tempfile = "3.8"
 zeroize = { version = "1.9.0", features = ["derive"] }
+similar = "2.2"
 
 [features]
 hardware-wallet = ["dep:hidapi", "dep:trezor-client"]

benches/benchmarks.rs

@@ -277,7 +277,7 @@ fn bench_basic(c: &mut Criterion) {
         b.iter(|| {
             let mut acc: u64 = 0;
             for i in 0..10_000u64 {
-                acc = acc.wrapping_add((i & 0xff) as u64);
+                acc = acc.wrapping_add(i & 0xff);
             }
             black_box(acc);
         })

deny.toml

@@ -37,6 +37,7 @@ allow = [
     "Zlib",
     "OpenSSL",
     "CC0-1.0",
+    "BSL-1.0",
 ]
 
 # ring 0.16.x has no license field; clarify it manually.

src/commands/deploy.rs

@@ -1,43 +1,3 @@
-use zeroize::{Zeroizing, Zeroize};
-use std::fs::File;
-use zeroize::{Zeroizing, Zeroize};
-use std::os::unix::fs::PermissionsExt;
-use std::io::Write;
-// Decrypt the key into a Zeroizing wrapper
-let decrypted_secret: Zeroizing<String> = decrypt_wallet_key(&wallet_name)?;
-
-// Create temp file for the secret with 600 permissions
-let mut temp_key_file = tempfile::NamedTempFile::new()?;
-std::fs::set_permissions(temp_key_file.path(), std::fs::Permissions::from_mode(0o600))?;
-write!(temp_key_file, "{}", *decrypted_secret)?;
-
-// Execute command securely
-let mut child = std::process::Command::new("stellar")
-    .args(["contract", "deploy", "--source-account", temp_key_file.path().to_str().unwrap()])
-    .env_clear() 
-    .spawn()?;
-
-child.wait()?;
-// Memory is automatically zeroed by Zeroizing<String> when it goes out of scope here
-use std::os::unix::fs::PermissionsExt;
-use std::io::Write;
-
-// Inside handle() function:
-let decrypted_secret: Zeroizing<String> = decrypt_wallet_key(&wallet_name)?;
-
-// Create temp file for the secret with 600 permissions
-let mut temp_key_file = tempfile::NamedTempFile::new()?;
-std::fs::set_permissions(temp_key_file.path(), std::fs::Permissions::from_mode(0o600))?;
-write!(temp_key_file, "{}", *decrypted_secret)?;
-
-// Execute command securely
-let mut child = std::process::Command::new("stellar")
-    .args(["contract", "deploy", "--source-account", temp_key_file.path().to_str().unwrap()])
-    .env_clear() // Prevents leakage of existing env vars
-    .spawn()?;
-
-child.wait()?;
-// Memory is automatically zeroed by Zeroizing<String> when scope ends
 use crate::utils::{config, confirmation, horizon, optimizer, print as p, soroban};
 use anyhow::Result;
 use clap::Args;
@@ -402,8 +362,8 @@ pub fn handle(args: DeployArgs) -> Result<()> {
         args.network.clone(),
         risk_level,
     )
-    .add("WASM file", &wasm_path.display().to_string())
-    .add("WASM size", &format!("{:.1} KB", wasm_size_kb))
+    .add("WASM file", wasm_path.display().to_string())
+    .add("WASM size", format!("{:.1} KB", wasm_size_kb))
     .add("WASM hash", &wasm_hash)
     .add("Wallet", &wallet.name)
     .add("Public Key", &wallet.public_key)

src/commands/invoke.rs

@@ -136,14 +136,14 @@ pub fn handle(args: InvokeArgs) -> Result<()> {
         .add("Wallet", &args.wallet)
         .add(
             "Estimated Fee",
-            &format!("{} stroops", outcome.simulation.fee),
+            format!("{} stroops", outcome.simulation.fee),
         )
         .add("Return Value", &outcome.simulation.return_value);
 
         // Add arguments to summary if present
         if !arg_list.is_empty() {
             for (i, (arg, arg_type)) in arg_list.iter().zip(arg_type_list.iter()).enumerate() {
-                summary = summary.add(&format!("Arg [{}] {}", i, arg_type), arg);
+                summary = summary.add(format!("Arg [{}] {}", i, arg_type), arg);
             }
         }