Skip to content

Conversation

claudiacodacy
Copy link
Contributor

@claudiacodacy claudiacodacy commented Oct 2, 2025

Caution

This PR was entirely done using Cursor + Codex. Review with care. 🙈


Problem

After analysis, it was discovered that state change telemetry events were not being properly emitted by the IntelliJ extension. The extension was missing critical telemetry tracking for repository, pull request, and branch state transitions, which are essential for analytics and monitoring user behavior across different IDE extensions.

Solution

This PR implements comprehensive telemetry state change tracking to match the VS Code extension behavior and includes several improvements for better IDE and OS parameter normalization.

Changes Made

🔧 Telemetry State Change Events

  • Added Repository State Change tracking: Now emits events when repository state transitions (Initializing, NeedsAuthentication, NoGitRepository, Loaded, NoRepository)
  • Added Pull Request State Change tracking: Tracks PR state transitions (NoPullRequest, Loaded)
  • Modified state change detection: Events are only emitted when state actually changes, preventing duplicate telemetry
  • Standardized OS & IDE conventions

Missing components

  • Tracking branch state change is missing. Requires understanding analysed branches. we dont pull that info yet. might as well open a new PT
  • Missing OrgID for parity with VScode

Copy link
Contributor

@manufacturist manufacturist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I think. If we can test it locally and see events propagated, we're golden.

claudiacodacy and others added 2 commits October 3, 2025 09:17
We have repeat os/ide detection logic,
I have extracted it to a helper class.
Copy link
Contributor

@og-pixel og-pixel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Made some changes, I need clarification on this one if statement, I think its too convoluted

manufacturist
manufacturist previously approved these changes Oct 6, 2025
og-pixel
og-pixel previously approved these changes Oct 6, 2025
@og-pixel og-pixel dismissed stale reviews from manufacturist and themself via ff57ec1 October 6, 2025 16:02
@og-pixel og-pixel merged commit 174dde5 into master Oct 7, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants