refactor: extract tool fetch loop body for safe cleanup

[IbrahimAhmed8]

In internal/node/mcp_handlers.go, fetchRemoteToolCatalogue now ensures cleanup occurs per iteration.

The previous implementation was functionally correct but introduced a latent risk. If session.ListTools panics or if a conditional return is introduced later, active streams and sessions would leak

Changes:
Extracted the loop body into a helper method named fetchToolsForRemoteService. This scopes the defer statement to the iteration scope rather than the parent function scope.

A standard defer inside the loop was intentionally avoided to prevent holding all network streams open until function return

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!