Missing public properties in McpToolset

[davidjsanders]

Is your feature request related to a specific problem?

When using MCP tools with McpToolset, configuration information for the object is not exposed as object properties or attributes meaning that 'private' attributes need to be accessed, e.g. toolset._connection_params rather than toolset.connection_params

Describe the Solution You'd Like

Expose the attributes of the McpToolset class as properties.

Impact on your work

This issue is not critical and more affects static code analysis and typing hints than any major issue; however, it's fairly simple to implement and resolve.

Willingness to contribute

Are you interested in implementing this feature yourself or submitting a PR?
No

🟡 Recommended Information

Describe Alternatives You've Considered

Access the private properties of the class.

Proposed API / Implementation

@Property
def connection_params(self):
return self._connection_params

[wukath]

@davidjsanders what attributes would you want exposed as public properties? connection_params makes sense, but just curious what the use case is for exposing the other ones

[davidjsanders]

@wukath Hi Kathy, I think the following would be useful...

1. self._connection_params -> as already stated
2. self._header_provider -> it might be useful to change this dynamically (i.e. a property and setter)
3. self._auth_scheme and self._auth_credential -> useful for debugging to have these available

I think number 1 is the most important for me (to get the URL.) My use case was enabling a retrying async client on MCP communications (snippet below) and it was useful to get debugging output when implementing it.

class RetryingAsyncClient(httpx.AsyncClient):
    """
    An httpx.AsyncClient that automatically retries failed requests.
    """

    NETWORK_ERRORS = (
        httpx.ConnectError,
        httpx.ReadTimeout,
        httpx.ConnectTimeout,
        httpx.PoolTimeout,
        google_exceptions.ServiceUnavailable,
        google_exceptions.GatewayTimeout,
    )

    @staticmethod
    def is_retryable_error(exception: BaseException) -> bool:
        """
        Determines if an exception should trigger a retry.
        Retries on:
        - Network timeouts/connection errors
        - 5xx Server Errors
        - 429 Too Many Requests
        """
        if isinstance(exception, RetryingAsyncClient.NETWORK_ERRORS):
            return True
        
        if isinstance(exception, httpx.HTTPStatusError):
            return (
                exception.response.status_code == 429 or 
                exception.response.status_code >= 500
            )
            
        return False

    _retry_logger = logging.getLogger("RetryingAsyncClient")
    _retry_logger.setLevel(logging.WARNING)

    @retry(
        stop=stop_after_attempt(3),
        wait=wait_exponential(multiplier=1, min=2, max=10),
        retry=retry_if_exception(is_retryable_error),
        before_sleep=before_sleep_log(
            logger=_retry_logger,
            log_level=logging.WARNING,
        ),
    )
    async def send(
        self,
        request: httpx.Request,
        *,
        stream: bool = False,
        auth: httpx.Auth | None = None,
        follow_redirects: bool | None = None,
    ) -> httpx.Response:
        """
        Send a request with automatic retries.
        """
        return await super().send(
            request,
            stream=stream,
            auth=auth,
            follow_redirects=follow_redirects,
        )

[klateefa]

@davidjsanders Thanks for the update. The request is currently being reviewed, and I will let you know if anything further is needed. Thank you for your understanding.