Adds anonymous telemetry by schnie · Pull Request #2009 · astronomer/astro-cli

schnie · 2026-02-11T14:42:04Z

Summary

Add anonymous telemetry to track CLI command usage
Events are sent to the Astronomer telemetry API (/v1alpha1/telemetry) with source astro-cli
Fire-and-forget pattern using a detached subprocess ensures no impact on CLI responsiveness
Users can opt out via astro telemetry disable or ASTRO_TELEMETRY_DISABLED=1

Commands

astro telemetry - Show current telemetry status
astro telemetry enable - Enable telemetry
astro telemetry disable - Disable telemetry

Environment Variables

ASTRO_TELEMETRY_DISABLED=1 - Disable telemetry (overrides config)
ASTRO_TELEMETRY_API_URL - Override the telemetry API endpoint
ASTRO_TELEMETRY_DEBUG=1 - Run telemetry synchronously with debug output to stderr

Test plan

astro telemetry shows enabled/disabled state
astro telemetry disable / astro telemetry enable work correctly
ASTRO_TELEMETRY_DISABLED=1 overrides config
ASTRO_TELEMETRY_DEBUG=1 shows request/response on stderr
All existing and new tests pass

🤖 Generated with Claude Code

coveralls-official · 2026-02-11T14:48:01Z

Pull Request Test Coverage Report for Build cfbde458-1260-4070-97a6-d093d2b09807

Details

112 of 251 (44.62%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 35.131%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
cmd/root.go	10	12	83.33%
cmd/telemetry_send.go	10	12	83.33%
cmd/telemetry.go	52	62	83.87%
pkg/telemetry/send.go	0	35	0.0%
pkg/telemetry/telemetry.go	40	130	30.77%

Totals
Change from base Build 96e87396-dad3-4edc-9a17-8881e17d45df:	0.03%
Covered Lines:	23090
Relevant Lines:	65726

💛 - Coveralls

Add Segment-based telemetry to track CLI usage anonymously. Users can opt-out via `astro telemetry disable` or ASTRO_TELEMETRY_DISABLED=1. Features: - Fire-and-forget pattern using detached subprocess - Tracks command paths (not arguments/values) - Detects agent (Claude Code, Cursor) and CI environments - Anonymous user ID stored in config Co-Authored-By: Claude <noreply@anthropic.com>

for more information, see https://pre-commit.ci

Copilot

Pull request overview

Introduces anonymous telemetry to the Astro CLI, including opt-in/out controls and an internal sender command that posts command-usage events to the Astronomer telemetry API.

Changes:

Add pkg/telemetry for enablement checks, context detection, payload construction, and async “fire-and-forget” sending via a detached subprocess.
Add astro telemetry {enable|disable} plus hidden internal _telemetry-send command.
Add config keys for telemetry enabled state and anonymous ID persistence.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
pkg/telemetry/telemetry.go	Core telemetry logic (enablement, anonymous ID, context detection, async spawn).
pkg/telemetry/send.go	HTTP sender + stdin-driven sender entrypoint for subprocess.
pkg/telemetry/telemetry_test.go	Unit tests for enablement, context detection, URL override, anonymous ID, and command path parsing.
cmd/telemetry.go	User-facing `astro telemetry` command and subcommands.
cmd/telemetry_send.go	Hidden internal `_telemetry-send` command wiring.
cmd/telemetry_test.go	Tests for telemetry command wiring and status output.
cmd/root.go	Hooks telemetry tracking into the root command and registers telemetry commands.
config/types.go	Adds new config fields for telemetry.
config/config.go	Registers defaults for telemetry config keys.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

cmd/telemetry_send.go

pkg/telemetry/telemetry.go

pkg/telemetry/send.go

pkg/telemetry/telemetry_test.go

cmd/telemetry_test.go

pkg/telemetry/send.go

pkg/telemetry/telemetry_test.go

cmd/root.go

- Skip PersistentPreRunE chain for _telemetry-send subprocess to avoid network I/O, token validation, and recursive telemetry tracking - Use ordered slices instead of maps in DetectContext to ensure deterministic results (generic CI checked last) - Accept full 2xx range in Send() instead of only 200 - Collect real OS version via uname/ver instead of duplicating GOOS/GOARCH Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Already identified by the top-level source field on the payload. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

The value is runtime.GOARCH (cpu architecture), not a platform. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

- Set SilenceUsage/SilenceErrors at construction time on _telemetry-send command instead of inside RunE - Write minimal config to in-memory fs before InitConfig in tests to avoid touching the real filesystem Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Replace test-only extractCommandPath/splitCommandPath helpers with actual cobra commands so the test exercises the production code path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

- Bound stdin read in _telemetry-send to 64KB via io.LimitReader - Drain response body before close to allow HTTP connection reuse - Replace fragile cmd.Name() string match with cobra annotation for PersistentPreRunE bypass - Log subprocess spawn failures in debug mode - Add one-time first-run notice to stderr informing users about telemetry and how to opt out Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

- Check return value of io.Copy (errcheck) - Extract magic number to named constant maxStdinBytes (mnd) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

schnie requested a review from a team as a code owner February 11, 2026 14:42

schnie marked this pull request as draft February 13, 2026 17:20

schnie and others added 2 commits February 20, 2026 12:57

[pre-commit.ci] auto fixes from pre-commit.com hooks

186b0cc

for more information, see https://pre-commit.ci

schnie force-pushed the telemetry branch from e6ad366 to 186b0cc Compare February 20, 2026 18:06

schnie changed the title ~~feat: add anonymous telemetry tracking~~ Adds anonymous telemetry tracking Feb 20, 2026

Swaps in astro endpoint

32ec21c

schnie force-pushed the telemetry branch 3 times, most recently from 3dee135 to a283e63 Compare February 20, 2026 19:04

Adds debug mode

67bad37

schnie force-pushed the telemetry branch from a283e63 to 67bad37 Compare February 20, 2026 19:48

schnie marked this pull request as ready for review February 20, 2026 19:51

schnie self-assigned this Feb 23, 2026

schnie requested a review from Copilot February 23, 2026 16:19

Copilot started reviewing on behalf of schnie February 23, 2026 16:20 View session

Copilot AI reviewed Feb 23, 2026

View reviewed changes

schnie and others added 3 commits February 23, 2026 12:34

Remove redundant cli_name from telemetry properties

ffad595

Already identified by the top-level source field on the payload. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Rename telemetry property platform to architecture

fefacff

The value is runtime.GOARCH (cpu architecture), not a platform. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

schnie force-pushed the telemetry branch from b4b6ddf to fefacff Compare February 23, 2026 18:02

schnie force-pushed the telemetry branch from 16fdff4 to df26937 Compare February 23, 2026 18:19

Test GetCommandPath with real cobra command tree

4abc192

Replace test-only extractCommandPath/splitCommandPath helpers with actual cobra commands so the test exercises the production code path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

schnie changed the title ~~Adds anonymous telemetry tracking~~ Adds anonymous telemetry Feb 23, 2026

jlaneve and others added 2 commits February 25, 2026 13:37

fix: address lint errors in telemetry send

39a855f

- Check return value of io.Copy (errcheck) - Extract magic number to named constant maxStdinBytes (mnd) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

jlaneve approved these changes Feb 25, 2026

View reviewed changes

schnie merged commit 80a727b into main Feb 25, 2026
3 of 5 checks passed

schnie deleted the telemetry branch February 25, 2026 19:19

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adds anonymous telemetry#2009

Adds anonymous telemetry#2009
schnie merged 11 commits intomainfrom
telemetry

schnie commented Feb 11, 2026 •

edited

Loading

Uh oh!

coveralls-official bot commented Feb 11, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

schnie commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Commands

Environment Variables

Test plan

Uh oh!

coveralls-official bot commented Feb 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request Test Coverage Report for Build cfbde458-1260-4070-97a6-d093d2b09807

Details

💛 - Coveralls

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

schnie commented Feb 11, 2026 •

edited

Loading

coveralls-official bot commented Feb 11, 2026 •

edited

Loading