Skip to content

Conversation

@jwajgelt
Copy link
Contributor

Adds telemetry events:

  • on extension activation, with the editor type (vscode vs cursor vs windsurf)
  • on maestro tests starting/completing/failing, with the device family (iPhone/iPad simulator, physical/emulated Android)

How Has This Been Tested:

What?

How Has This Change Been Documented:

Internal

@jwajgelt jwajgelt requested a review from latekvo December 10, 2025 14:43
@jwajgelt jwajgelt self-assigned this Dec 10, 2025
@vercel
Copy link

vercel bot commented Dec 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
radon-ide Ready Ready Preview Comment Dec 10, 2025 2:43pm

Comment on lines +58 to +68
if (deviceInfo.platform === DevicePlatform.IOS) {
if (deviceInfo.deviceType === DeviceType.Tablet) {
return DeviceFamily.IPAD_SIMULATOR;
}
return DeviceFamily.IPHONE_SIMULATOR;
}

if (deviceInfo.emulator) {
return DeviceFamily.ANDROID_EMULATOR;
}
return DeviceFamily.ANDROID_PHYSICAL;
Copy link
Member

@latekvo latekvo Dec 10, 2025

Choose a reason for hiding this comment

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

Just to be sure, we don't want to be checking deviceInfo.deviceType on android devices? (feel free to ignore if this is a dumb question)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We don't set it for Androids, it's always Phone (at this time, at least)

@jwajgelt jwajgelt merged commit cbd58df into main Dec 10, 2025
9 checks passed
@jwajgelt jwajgelt deleted the @jwajgelt/add_telemetry_tidbits branch December 10, 2025 15:04
jwajgelt added a commit that referenced this pull request Dec 12, 2025
Adds telemetry events:
- on extension activation, with the editor type (vscode vs cursor vs
windsurf)
- on maestro tests starting/completing/failing, with the device family
(iPhone/iPad simulator, physical/emulated Android)

What?

Internal
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