refactor(mcp): remove redundant entry-level logs and cross-reference list* methods

[Copilot]

listTools, listResources, and listPrompts each fired a logConn.Printf entry log that duplicated the logging already emitted by listMCPItems after session validation. The three functions were also undocumented as intentionally parallel, creating a silent maintenance contract.

Changes

• internal/mcp/connection_methods.go
  • Removed the redundant per-function logConn.Printf from listTools, listResources, and listPrompts — equivalent logging continues inside listMCPItems ("list%s: requesting %s list from backend serverID=%s")
  • Added Godoc cross-reference comments to each function documenting the intentional parallel structure:

// listTools fetches all tools from the backend via paginated SDK calls.
// See also: listResources, listPrompts — these three follow the same structure.
func (c *Connection) listTools() (*Response, error) {
    return listSDKItems(c, "tools", ...)
}

[Copilot]

Pull request overview

This PR refactors MCP connection list operations to reduce redundant logging and clarify intentional parallel structure among the list* methods that fetch paginated data via the SDK.

Changes:

• Removed per-function entry logs from listTools, listResources, and listPrompts, relying on the centralized logging in listMCPItems after session validation.
• Added GoDoc cross-reference comments documenting that listTools, listResources, and listPrompts intentionally share the same structure.

Show a summary per file

File	Description
internal/mcp/connection_methods.go	Removes redundant list-entry logs and adds GoDoc “See also” cross-references for the parallel list methods.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

• Files reviewed: 1/1 changed files
• Comments generated: 0