Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 27, 2025

Description

This makes the telemetry service more extensible.


Important

Refactors telemetry service for extensibility, introduces RefreshTimer utility, and updates task management.

  • Telemetry Refactor:
    • Introduces PostHogTelemetryClient in PostHogTelemetryClient.ts for handling telemetry events.
    • Refactors TelemetryService in TelemetryService.ts to support multiple telemetry clients.
    • Adds BaseTelemetryClient in BaseTelemetryClient.ts as an abstract class for telemetry clients.
    • Updates ClineProvider in ClineProvider.ts to implement TelemetryPropertiesProvider.
    • Modifies webviewMessageHandler.ts to handle telemetry settings updates.
  • New Utility:
    • Adds RefreshTimer in refresh-timer.ts for executing callbacks with retry behavior.
    • Includes tests for RefreshTimer in refresh-timer.test.ts.
  • Testing:
    • Adds tests for PostHogTelemetryClient in PostHogTelemetryClient.test.ts.
  • Miscellaneous:
    • Updates extension.ts to initialize telemetry service with context.
    • Removes PostHogClient.ts as it is replaced by the new client architecture.

This description was created by Ellipsis for 8d5a1c3. You can customize this summary. It will automatically update as commits are pushed.

cte and others added 3 commits May 26, 2025 20:12
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@cte cte merged commit 1384077 into main May 27, 2025
10 checks passed
@cte cte deleted the cte/telemetry-refactor branch May 27, 2025 03:22
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 27, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants