Skip to content

Commit c8e52b9

Browse files
committed
fix(cli): stop dropping CI/agent telemetry, suppress HeyGen CI at workflow level
The CI=true early-exit in shouldTrack() was hiding most modern usage (coding agents in Codespaces, CI pipelines, agent sandboxes). Remove it. Each event still carries is_ci/is_docker/is_tty from system.ts, so CI vs laptop traffic can be separated in PostHog without being dropped at ingestion. HeyGen's own CI is suppressed via HYPERFRAMES_NO_TELEMETRY=1 added to each workflow that exercises the CLI.
1 parent 16d2966 commit c8e52b9

8 files changed

Lines changed: 35 additions & 5 deletions

File tree

.github/workflows/catalog-previews.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: Catalog Previews
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
on:
712
pull_request:
813
branches: [main]

.github/workflows/ci.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,11 @@ permissions:
44
contents: read
55
pull-requests: read
66

7+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
8+
# External users' CI continues to emit telemetry unless they set this themselves.
9+
env:
10+
HYPERFRAMES_NO_TELEMETRY: "1"
11+
712
on:
813
pull_request:
914
# `edited` is required so the workflow re-fires when a PR's base ref is

.github/workflows/docs.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: Docs
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
on:
712
pull_request:
813
branches: [main]

.github/workflows/player-perf.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: Player perf
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
on:
712
pull_request:
813
push:

.github/workflows/preview-regression.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: preview-regression
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
on:
712
pull_request:
813
push:

.github/workflows/regression.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: regression
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
on:
712
pull_request:
813
push:

.github/workflows/windows-render.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@ name: Windows render verification
33
permissions:
44
contents: read
55

6+
# Suppress hyperframes CLI telemetry from HeyGen's own CI runs.
7+
# External users' CI continues to emit telemetry unless they set this themselves.
8+
env:
9+
HYPERFRAMES_NO_TELEMETRY: "1"
10+
611
# Manually triggered smoke test that renders a HyperFrames composition on a
712
# real Windows runner. Proves the PR #336 `where ffmpeg` fix actually works
813
# end-to-end: FFmpeg is discovered natively on Windows, Chrome is installed

packages/cli/src/telemetry/client.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,6 @@ export function shouldTrack(): boolean {
4040
return false;
4141
}
4242

43-
if (process.env["CI"] === "true" || process.env["CI"] === "1") {
44-
telemetryEnabled = false;
45-
return false;
46-
}
47-
4843
if (isDevMode()) {
4944
telemetryEnabled = false;
5045
return false;

0 commit comments

Comments
 (0)