gossip: replace usize with u8 for relay limits

Signed-off-by: Yuki Kishimoto <[email protected]>

Author: yukibtc

crates/nostr-sdk/CHANGELOG.md

@@ -23,6 +23,12 @@
 
 -->
 
+## Unreleased
+
+### Breaking changes
+
+- Replace `usize` with `u8` for gossip relay limits
+
 ## v0.44.1 - 2025/11/09
 
 ### Fixed

crates/nostr-sdk/src/client/options.rs

@@ -16,15 +16,15 @@ use nostr_relay_pool::prelude::*;
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub struct GossipRelayLimits {
     /// Max number of **read** relays per user (default: 3)
-    pub read_relays_per_user: usize,
+    pub read_relays_per_user: u8,
     /// Max number of **write** relays per user (default: 3)
-    pub write_relays_per_user: usize,
+    pub write_relays_per_user: u8,
     /// Max number of **hint** relays per user (default: 1)
-    pub hint_relays_per_user: usize,
+    pub hint_relays_per_user: u8,
     /// Max number of **most used** relays per user (default: 1)
-    pub most_used_relays_per_user: usize,
+    pub most_used_relays_per_user: u8,
     /// Max number of NIP-17 relays per user (default: 3)
-    pub nip17_relays: usize,
+    pub nip17_relays: u8,
 }
 
 impl Default for GossipRelayLimits {

gossip/nostr-gossip-memory/CHANGELOG.md

@@ -25,6 +25,10 @@
 
 ## Unreleased
 
+### Breaking changes
+
+- Replace `usize` with `u8` for relay limits
+
 ### Changed
 
 - Replace `Mutex` with `RwLock` for better concurrency (https://github.com/rust-nostr/nostr/pull/1126)

gossip/nostr-gossip-memory/src/store.rs

@@ -288,7 +288,7 @@ impl NostrGossipMemory {
         tx: &LruCache<PublicKey, PkData>,
         public_key: &PublicKey,
         flag: GossipFlags,
-        limit: usize,
+        limit: u8,
     ) -> impl Iterator<Item = RelayUrl> + '_ {
         let mut relays: Vec<(RelayUrl, u64, Option<Timestamp>)> = Vec::new();
 
@@ -312,7 +312,10 @@ impl NostrGossipMemory {
         });
 
         // Take only the requested limit and extract relay URLs
-        relays.into_iter().take(limit).map(|(url, _, _)| url)
+        relays
+            .into_iter()
+            .take(limit as usize)
+            .map(|(url, _, _)| url)
     }
 }
 

gossip/nostr-gossip/src/lib.rs

@@ -59,38 +59,38 @@ pub enum BestRelaySelection {
     /// Get all the best relays for **reading** and **writing** events (NIP-65)
     All {
         /// Limit for read relays
-        read: usize,
+        read: u8,
         /// Limit for write relays
-        write: usize,
+        write: u8,
         /// Limit for hints
-        hints: usize,
+        hints: u8,
         /// Limit for most received relays
-        most_received: usize,
+        most_received: u8,
     },
     /// Get the best relays for **reading** events (NIP-65)
     Read {
         /// Limit
-        limit: usize,
+        limit: u8,
     },
     /// Get the best relays for **writing** events (NIP-65)
     Write {
         /// Limit
-        limit: usize,
+        limit: u8,
     },
     /// Get the best relays for **reading** and **writing** private messages (NIP-17)
     PrivateMessage {
         /// Limit
-        limit: usize,
+        limit: u8,
     },
     /// Relays found in hints
     Hints {
         /// Limit
-        limit: usize,
+        limit: u8,
     },
     /// Relays that received most events
     MostReceived {
         /// Limit
-        limit: usize,
+        limit: u8,
     },
 }