Pull Request Overview
This PR introduces an opt‐out telemetry system to capture anonymized usage data and performance metrics while preserving user privacy. Key changes include:
- Integration of PostHog for telemetry tracking and a new Telemetry service with singleton management.
- Implementation of comprehensive event tracking in MCPAgent via enhanced run and astream methods.
- Addition of a public_identifier property across connector implementations for better connector tracking.
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File |
Description |
| pyproject.toml |
Added posthog dependency |
| mcp_use/utils.py |
Added a singleton decorator for consistent instance management |
| mcp_use/telemetry/utils.py |
Added utilities for package version and model information extraction |
| mcp_use/telemetry/posthog.py |
Introduced the Telemetry class and related telemetry methods |
| mcp_use/telemetry/events.py |
Created telemetry event classes with comprehensive properties |
| mcp_use/connectors/*.py |
Added public_identifier properties to connector classes |
| mcp_use/connectors/base.py |
Defined an abstract public_identifier to enforce API consistency |
| mcp_use/agents/mcpagent.py |
Enhanced agent methods to call telemetry events after execution |
Comments suppressed due to low confidence (2)
mcp_use/connectors/stdio.py:83
- [nitpick] Consider replacing the ambiguous key "command&args" with separate keys such as "command" and "args" to improve clarity and consistency with other connectors.
return {"type": "stdio", "command&args": f"{self.command} {' '.join(self.args)}"}
mcp_use/connectors/base.py:41
- The abstract property is annotated to return a string, but the concrete implementations return a dictionary. Consider updating the return type in the abstract method to dict[str, str] for consistency.
def public_identifier(self) -> str:
Originally posted by @copilot-pull-request-reviewer in mcp-use/mcp-use#124 (review)
Pull Request Overview
This PR introduces an opt‐out telemetry system to capture anonymized usage data and performance metrics while preserving user privacy. Key changes include:
Reviewed Changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
Comments suppressed due to low confidence (2)
mcp_use/connectors/stdio.py:83
mcp_use/connectors/base.py:41
Originally posted by @copilot-pull-request-reviewer in mcp-use/mcp-use#124 (review)