chore(deps): upgrade Alloy and remove incompatible side-by-side benchmark

Cargo.toml

@@ -30,10 +30,10 @@ simd-json = "0.13"
 dotenvy = "0.15"
 
 # Ethereum and crypto
-alloy-primitives = "0.8.26"
-alloy-sol-types = { version = "0.8.26", features = ["eip712-serde", "json"] }
-alloy-signer = { version = "0.7", features = ["eip712"] }
-alloy-signer-local = { version = "0.7", features = ["eip712"] }
+alloy-primitives = "1.5"
+alloy-sol-types = { version = "1.5", features = ["eip712-serde", "json"] }
+alloy-signer = { version = "1.6", features = ["eip712"] }
+alloy-signer-local = { version = "1.6", features = ["eip712"] }
 
 # Numeric types
 rust_decimal = { version = "1.36", features = ["serde-with-str"] }
@@ -70,7 +70,6 @@ tokio-test = "0.4"
 mockito = "1.0"
 proptest = "1.0"
 env_logger = "0.10"
-polymarket-rs-client = "0.1.1"
 
 [features]
 default = ["stream"]

README.md

@@ -55,7 +55,7 @@ End-to-end performance with Polymarket's API, including network latency, JSON pa
 - **32.5% more consistent** 
 - **4.2x faster** than Official Python Client
 
-**Benchmark Methodology:** All benchmarks run side-by-side on the same machine, same network, same time using 20 iterations, 100ms delay between requests, /simplified-markets endpoint. Best performance achieved with connection keep-alive enabled. See `examples/side_by_side_benchmark.rs` for the complete benchmark implementation.
+**Benchmark Methodology:** All benchmarks run side-by-side on the same machine, same network, same time using 20 iterations, 100ms delay between requests, /simplified-markets endpoint. Best performance achieved with connection keep-alive enabled. See `examples/side_by_side_benchmark.rs` in commit `a63a170`: https://github.com/floor-licker/polyfill-rs/blob/a63a170/examples/side_by_side_benchmark.rs for the complete benchmark implementation.
 
 **Computational Performance (pure CPU, no I/O)**
 

examples/side_by_side_benchmark.rs

@@ -1,241 +0,0 @@
-//! Side-by-side benchmark comparing polyfill-rs vs polymarket-rs-client
-//!
-//! To run this benchmark, uncomment the polymarket-rs-client dependency in Cargo.toml:
-//! ```toml
-//! [dev-dependencies]
-//! polymarket-rs-client = { path = "external/polymarket-rs-client" }
-//! ```
-
-use std::time::Instant;
-
-#[tokio::main]
-async fn main() -> Result<(), Box<dyn std::error::Error>> {
-    println!("═══════════════════════════════════════════════════════");
-    println!("          SIDE-BY-SIDE BENCHMARK");
-    println!("═══════════════════════════════════════════════════════");
-    println!("\nTesting both clients on:");
-    println!("  - Same machine");
-    println!("  - Same network");
-    println!("  - Same time");
-    println!("  - Same API endpoint (/simplified-markets)");
-    println!("  - 20 iterations each");
-    println!("  - 100ms delay between requests\n");
-
-    // Test 1: polymarket-rs-client
-    println!("══════════════════════════════════════");
-    println!("Test 1: polymarket-rs-client");
-    println!("══════════════════════════════════════");
-
-    let their_client = polymarket_rs_client::ClobClient::new("https://clob.polymarket.com");
-
-    let mut their_times = Vec::new();
-    for i in 1..=20 {
-        let start = Instant::now();
-        match their_client.get_simplified_markets(Some("MA==")).await {
-            Ok(_markets) => {
-                let elapsed = start.elapsed();
-                their_times.push(elapsed);
-
-                if i <= 3 || i > 17 {
-                    println!(
-                        "  Request {:2}: {:.1} ms",
-                        i,
-                        elapsed.as_micros() as f64 / 1000.0
-                    );
-                } else if i == 4 {
-                    println!("  ...");
-                }
-            },
-            Err(e) => {
-                println!("  Request {:2}: ERROR - {}", i, e);
-            },
-        }
-
-        tokio::time::sleep(std::time::Duration::from_millis(100)).await;
-    }
-
-    // Small break between tests
-    tokio::time::sleep(std::time::Duration::from_millis(500)).await;
-
-    // Test 2: polyfill-rs (with keep-alive)
-    println!("\n══════════════════════════════════════");
-    println!("Test 2: polyfill-rs (with keep-alive)");
-    println!("══════════════════════════════════════");
-
-    let our_client = polyfill_rs::ClobClient::new("https://clob.polymarket.com");
-    our_client
-        .start_keepalive(std::time::Duration::from_secs(30))
-        .await;
-    tokio::time::sleep(std::time::Duration::from_millis(500)).await; // Let keep-alive establish
-
-    let mut our_times = Vec::new();
-    for i in 1..=20 {
-        let start = Instant::now();
-        match our_client
-            .http_client
-            .get(format!(
-                "{}/simplified-markets?next_cursor=MA==",
-                our_client.base_url
-            ))
-            .send()
-            .await
-        {
-            Ok(response) => match response.json::<serde_json::Value>().await {
-                Ok(_json) => {
-                    let elapsed = start.elapsed();
-                    our_times.push(elapsed);
-
-                    if i <= 3 || i > 17 {
-                        println!(
-                            "  Request {:2}: {:.1} ms",
-                            i,
-                            elapsed.as_micros() as f64 / 1000.0
-                        );
-                    } else if i == 4 {
-                        println!("  ...");
-                    }
-                },
-                Err(e) => {
-                    println!("  Request {:2}: PARSE ERROR - {}", i, e);
-                },
-            },
-            Err(e) => {
-                println!("  Request {:2}: NETWORK ERROR - {}", i, e);
-            },
-        }
-
-        tokio::time::sleep(std::time::Duration::from_millis(100)).await;
-    }
-
-    our_client.stop_keepalive().await;
-
-    // Calculate statistics
-    fn calc_stats(times: &[std::time::Duration]) -> (f64, f64, f64, f64, f64) {
-        if times.is_empty() {
-            return (0.0, 0.0, 0.0, 0.0, 0.0);
-        }
-
-        let values: Vec<f64> = times
-            .iter()
-            .map(|d| d.as_micros() as f64 / 1000.0)
-            .collect();
-        let mean = values.iter().sum::<f64>() / values.len() as f64;
-        let variance = values.iter().map(|v| (v - mean).powi(2)).sum::<f64>() / values.len() as f64;
-        let std_dev = variance.sqrt();
-        let mut sorted = values.clone();
-        sorted.sort_by(|a, b| a.partial_cmp(b).unwrap());
-        let min = sorted[0];
-        let max = sorted[sorted.len() - 1];
-        let median = sorted[sorted.len() / 2];
-        (mean, std_dev, min, max, median)
-    }
-
-    let (their_mean, their_std, their_min, their_max, their_median) = calc_stats(&their_times);
-    let (our_mean, our_std, our_min, our_max, our_median) = calc_stats(&our_times);
-
-    // Results
-    println!("\n\n");
-    println!("═══════════════════════════════════════════════════════");
-    println!("                   HONEST RESULTS                      ");
-    println!("═══════════════════════════════════════════════════════\n");
-
-    println!("polymarket-rs-client:");
-    println!("  Mean:     {:.1} ms ± {:.1} ms", their_mean, their_std);
-    println!("  Median:   {:.1} ms", their_median);
-    println!("  Range:    {:.1} - {:.1} ms", their_min, their_max);
-    println!("  Variance: {:.1}%", (their_std / their_mean) * 100.0);
-    println!("  Success:  {}/20 requests", their_times.len());
-    println!("\n  (They claimed in README: 404.5 ms ± 22.9 ms)");
-
-    println!("\n");
-    println!("polyfill-rs (with keep-alive):");
-    println!("  Mean:     {:.1} ms ± {:.1} ms", our_mean, our_std);
-    println!("  Median:   {:.1} ms", our_median);
-    println!("  Range:    {:.1} - {:.1} ms", our_min, our_max);
-    println!("  Variance: {:.1}%", (our_std / our_mean) * 100.0);
-    println!("  Success:  {}/20 requests", our_times.len());
-    println!("\n  (We claimed in README: 368.6 ms ± 67.1 ms)");
-
-    println!("\n");
-    println!("═══════════════════════════════════════════════════════");
-
-    if our_times.is_empty() || their_times.is_empty() {
-        println!("ERROR: Not enough successful requests to compare");
-    } else {
-        let diff = our_mean - their_mean;
-        let pct = (diff.abs() / their_mean) * 100.0;
-
-        if diff < 0.0 {
-            println!(
-                "✅ polyfill-rs is {:.1}% FASTER ({:.1} ms faster)",
-                pct, -diff
-            );
-        } else {
-            println!(
-                "❌ polymarket-rs-client is {:.1}% faster ({:.1} ms faster)",
-                pct, diff
-            );
-        }
-    }
-
-    println!("═══════════════════════════════════════════════════════");
-
-    // Detailed variance comparison
-    println!("\n\nVariance Analysis:");
-    println!("────────────────────────────────────────────────────");
-    println!(
-        "  polymarket-rs-client: ±{:.1} ms ({:.1}% variance)",
-        their_std,
-        (their_std / their_mean) * 100.0
-    );
-    println!(
-        "  polyfill-rs:          ±{:.1} ms ({:.1}% variance)",
-        our_std,
-        (our_std / our_mean) * 100.0
-    );
-    println!();
-
-    if our_std < their_std {
-        let improvement = ((their_std - our_std) / their_std) * 100.0;
-        println!("  ✅ polyfill-rs is {:.1}% more consistent", improvement);
-    } else {
-        let diff = ((our_std - their_std) / their_std) * 100.0;
-        println!("  ⚠️  polymarket-rs-client is {:.1}% more consistent", diff);
-    }
-
-    // Claims validation
-    println!("\n\nClaims Validation:");
-    println!("────────────────────────────────────────────────────");
-
-    let their_claimed_mean = 404.5;
-    let their_claimed_std = 22.9;
-    let our_claimed_mean = 368.6;
-    let our_claimed_std = 67.1;
-
-    let their_mean_diff = ((their_mean - their_claimed_mean).abs() / their_claimed_mean) * 100.0;
-    let their_std_diff = ((their_std - their_claimed_std).abs() / their_claimed_std) * 100.0;
-    let our_mean_diff = ((our_mean - our_claimed_mean).abs() / our_claimed_mean) * 100.0;
-    let our_std_diff = ((our_std - our_claimed_std).abs() / our_claimed_std) * 100.0;
-
-    println!("polymarket-rs-client claimed vs actual:");
-    println!(
-        "  Mean:     {:.1} ms vs {:.1} ms ({:.1}% difference)",
-        their_claimed_mean, their_mean, their_mean_diff
-    );
-    println!(
-        "  Variance: ±{:.1} ms vs ±{:.1} ms ({:.1}% difference)",
-        their_claimed_std, their_std, their_std_diff
-    );
-
-    println!("\npolyfill-rs claimed vs actual:");
-    println!(
-        "  Mean:     {:.1} ms vs {:.1} ms ({:.1}% difference)",
-        our_claimed_mean, our_mean, our_mean_diff
-    );
-    println!(
-        "  Variance: ±{:.1} ms vs ±{:.1} ms ({:.1}% difference)",
-        our_claimed_std, our_std, our_std_diff
-    );
-
-    Ok(())
-}