Skip to content

Send telemetry info using specific id's associated to users to calculate DAU/WAU/MAU in posthog#3097

Open
imrishabh18 wants to merge 12 commits into
mainfrom
telemetry-dau
Open

Send telemetry info using specific id's associated to users to calculate DAU/WAU/MAU in posthog#3097
imrishabh18 wants to merge 12 commits into
mainfrom
telemetry-dau

Conversation

@imrishabh18
Copy link
Copy Markdown
Member

No description provided.

@imrishabh18 imrishabh18 requested review from seveibar and removed request for seveibar May 24, 2026 11:44
Copy link
Copy Markdown
Contributor

@seveibar seveibar left a comment

Choose a reason for hiding this comment

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

This will make the cli slower (a lot slower), it needs to be done carefully so that telemetry does not make commands run more slowly. also it's capturing all the ci usage i think- we can probably do some basic checks so that ci usage doesnt get sent rather than filter on the posthog side

Do we have events in runframe yet? Runframe and circuit json viewer telemetry may be a bit better

Comment thread lib/telemetry/index.ts
"Content-Type": "application/json",
},
body: JSON.stringify(payload),
signal: AbortSignal.timeout(1_000),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Each cli command taking 1s is wayyyy too long. An abortsignal needs to be sent at the end of every command

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.

2 participants