Skip to content

📈 Fix telemetry overcounting and reduce event volume#157

Merged
savannahostrowski merged 4 commits into
mainfrom
cleanup-telemetry
May 26, 2026
Merged

📈 Fix telemetry overcounting and reduce event volume#157
savannahostrowski merged 4 commits into
mainfrom
cleanup-telemetry

Conversation

@savannahostrowski
Copy link
Copy Markdown
Member

@savannahostrowski savannahostrowski commented May 26, 2026

Closes https://github.com/fastapilabs/cloud/issues/3952

This PR fixes a couple things to prevent overcounting of events:

  • Previously, we didn't seed the lastAuthState before polling which caused us to fire a spurious extension_cloud_sign_in on every activation for already-signed-in users. Now we seed lastAuthState from the current token state before starting the poll, so only real sign-in transitions fire the event.
  • Previously, we were tracking whether we were providing codelens to the user. This event was noisy and ultimately, unused. I've opted to remove it. We still track when a codelens was clicked, which IMO is a better signal anyway.
  • Previously, we were tracking whether we were able to detect an entrypoint as a distinct event. This was also pretty noisy due to reloads. Instead, we can fold the detection-method counts (detection_method_config/pyproject/heuristic) and folders_with_apps into extension_activated properties.

@savannahostrowski savannahostrowski added this to the 2026-Q2 milestone May 26, 2026
@savannahostrowski savannahostrowski self-assigned this May 26, 2026
@savannahostrowski savannahostrowski merged commit c2c46b6 into main May 26, 2026
13 of 14 checks passed
@savannahostrowski savannahostrowski deleted the cleanup-telemetry branch May 26, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant