Removed code to start connection upon addition of new peers.

[alexeykiselev]

Log messages added.

[Copilot]

Pull Request Overview

This PR removes the auto-connection logic when adding a new peer and instead relies on manual connection management while adding additional debug logging to better trace connection events.

• Removed asynchronous spawning of outgoing connections in AddAddress.
• Added debug logging for spawning and tracking peer connections.
• Updated function signatures in peer management and node initialization to reflect these changes.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
pkg/node/peers/peer_manager.go	Removed connection spawn in AddAddress and added debug logging for connection metrics.
pkg/node/node.go	Removed unused SpawnOutgoingConnection function.
pkg/node/network/network.go	Added debug logging for established and disconnected connection events.
cmd/node/node.go	Updated spawnPeersByAddresses call to match the new AddAddress signature.

Comments suppressed due to low confidence (2)

pkg/node/peers/peer_manager.go:425

• The removal of the context parameter and the asynchronous outgoing connection spawn in AddAddress is a key design change. Please ensure that the process for initiating connections is handled elsewhere and that the new signature aligns with the overall connection management strategy.

func (a *PeerManagerImpl) AddAddress(addr proto.TCPAddr) error {

cmd/node/node.go:440

• The call to spawnPeersByAddresses has been updated to remove the context parameter. Verify that context handling for asynchronous operations is re-integrated at an appropriate location if needed.

	if pErr := spawnPeersByAddresses(conf.Addresses, peerManager); pErr != nil {