fix: resolve compilation errors after rebase

- Use addr() instead of peer() for live_tx_tracker methods (expect SocketAddr)
- Remove duplicate get_peer_location_by_addr function

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: sanity

crates/core/src/ring/connection_manager.rs

@@ -373,34 +373,6 @@ impl ConnectionManager {
         None
     }
 
-    /// Look up a PeerKeyLocation by socket address from connections_by_location or transient connections.
-    /// Used for connection-based routing when we need full peer info from just an address.
-    pub fn get_peer_location_by_addr(&self, addr: SocketAddr) -> Option<PeerKeyLocation> {
-        // Check connections by location
-        let connections = self.connections_by_location.read();
-        for conns in connections.values() {
-            for conn in conns {
-                if conn.location.addr() == addr {
-                    return Some(conn.location.clone());
-                }
-            }
-        }
-        drop(connections);
-
-        // Check transient connections - construct PeerKeyLocation from PeerId
-        if let Some((peer, entry)) = self
-            .transient_connections
-            .iter()
-            .find(|e| e.key().addr == addr)
-            .map(|e| (e.key().clone(), e.value().clone()))
-        {
-            let mut pkl = PeerKeyLocation::new(peer.pub_key, peer.addr);
-            pkl.location = entry.location;
-            return Some(pkl);
-        }
-        None
-    }
-
     pub fn is_gateway(&self) -> bool {
         self.is_gateway
     }

crates/core/src/ring/mod.rs

@@ -355,7 +355,7 @@ impl Ring {
 
     pub async fn prune_connection(&self, peer: PeerId) {
         tracing::debug!(%peer, "Removing connection");
-        self.live_tx_tracker.prune_transactions_from_peer(&peer);
+        self.live_tx_tracker.prune_transactions_from_peer(peer.addr);
         // This case would be when a connection is being open, so peer location hasn't been recorded yet and we can ignore everything below
         let Some(loc) = self.connection_manager.prune_alive_connection(&peer) else {
             return;
@@ -473,7 +473,7 @@ impl Ring {
                 .map(|(loc, conns)| {
                     let conns: Vec<_> = conns
                         .iter()
-                        .filter(|conn| !live_tx_tracker.has_live_connection(&conn.location.peer()))
+                        .filter(|conn| !live_tx_tracker.has_live_connection(conn.location.addr()))
                         .cloned()
                         .collect();
                     (*loc, conns)
@@ -653,7 +653,7 @@ impl Ring {
             op_manager.connect_forward_estimator.clone(),
         );
 
-        live_tx_tracker.add_transaction(query_target.peer().clone(), tx);
+        live_tx_tracker.add_transaction(query_target.addr(), tx);
         op_manager
             .push(tx, OpEnum::Connect(Box::new(op)))
             .await