relay-pool: change RelayPool::batch_msg signature

Take a slice instead of a vector in `RelayPool::batch_msg` and `Client::batch_msg`.

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

Author: yukibtc

crates/nostr-relay-pool/CHANGELOG.md

@@ -28,6 +28,7 @@
 ### Breaking changes
 
 - Change `Relay::send_msg` and `Relay::batch_msg` signatures (https://github.com/rust-nostr/nostr/pull/1124)
+- Change `RelayPool::batch_msg` signature (https://github.com/rust-nostr/nostr/pull/1124)
 
 ## v0.44.0 - 2025/11/06
 

crates/nostr-relay-pool/src/pool/mod.rs

@@ -659,7 +659,7 @@ impl RelayPool {
         U: TryIntoUrl,
         Error: From<<U as TryIntoUrl>::Err>,
     {
-        self.batch_msg_to(urls, vec![msg]).await
+        self.batch_msg_to(urls, &[msg]).await
     }
 
     /// Send multiple client messages at once to specific relays
@@ -668,7 +668,7 @@ impl RelayPool {
     pub async fn batch_msg_to<I, U>(
         &self,
         urls: I,
-        msgs: Vec<ClientMessage<'_>>,
+        msgs: &[ClientMessage<'_>],
     ) -> Result<Output<()>, Error>
     where
         I: IntoIterator<Item = U>,
@@ -713,7 +713,7 @@ impl RelayPool {
         for url in set.into_iter() {
             let relay: &Relay = self.internal_relay(&relays, &url)?;
             urls.push(url);
-            futures.push(relay.batch_msg(msgs.clone()));
+            futures.push(relay.batch_msg(msgs));
         }
 
         // Join futures

crates/nostr-relay-pool/src/relay/inner.rs

@@ -86,11 +86,11 @@ impl RelayChannels {
 
     fn send_client_msgs(
         &self,
-        msgs: Vec<ClientMessage>,
+        msgs: &[ClientMessage],
         confirmation: Option<oneshot::Sender<()>>,
     ) -> Result<(), Error> {
         // Serialize messages to JSON
-        let msgs: Vec<ClientMessageJson> = msgs.into_iter().map(|msg| msg.as_json()).collect();
+        let msgs: Vec<ClientMessageJson> = msgs.iter().map(|msg| msg.as_json()).collect();
 
         // Build request
         let req: SendMessageRequest = SendMessageRequest { msgs, confirmation };
@@ -1325,10 +1325,10 @@ impl InnerRelay {
 
     #[inline]
     pub async fn send_msg(&self, msg: ClientMessage<'_>) -> Result<(), Error> {
-        self.batch_msg(vec![msg]).await
+        self.batch_msg(&[msg]).await
     }
 
-    pub async fn batch_msg(&self, msgs: Vec<ClientMessage<'_>>) -> Result<(), Error> {
+    pub async fn batch_msg(&self, msgs: &[ClientMessage<'_>]) -> Result<(), Error> {
         // Check if relay is operational
         self.ensure_operational()?;
 

crates/nostr-relay-pool/src/relay/mod.rs

@@ -411,7 +411,7 @@ impl Relay {
 
     /// Send multiple [`ClientMessage`] at once
     #[inline]
-    pub async fn batch_msg(&self, msgs: Vec<ClientMessage<'_>>) -> Result<(), Error> {
+    pub async fn batch_msg(&self, msgs: &[ClientMessage<'_>]) -> Result<(), Error> {
         self.inner.batch_msg(msgs).await
     }
 

crates/nostr-sdk/CHANGELOG.md

@@ -23,6 +23,12 @@
 
 -->
 
+## Unreleased
+
+### Breaking changes
+
+- Change `Client::batch_msg` signature (https://github.com/rust-nostr/nostr/pull/1124)
+
 ## v0.44.0 - 2025/11/06
 
 ### Breaking changes

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

@@ -935,7 +935,7 @@ impl Client {
     pub async fn batch_msg_to<I, U>(
         &self,
         urls: I,
-        msgs: Vec<ClientMessage<'_>>,
+        msgs: &[ClientMessage<'_>],
     ) -> Result<Output<()>, Error>
     where
         I: IntoIterator<Item = U>,