Skip to content

feat: enrich cli_command_push telemetry with actorId and wasCreated#1152

Merged
patrikbraborec merged 3 commits into
masterfrom
feat/push-telemetry-actor-id-was-created
May 27, 2026
Merged

feat: enrich cli_command_push telemetry with actorId and wasCreated#1152
patrikbraborec merged 3 commits into
masterfrom
feat/push-telemetry-actor-id-was-created

Conversation

@patrikbraborec
Copy link
Copy Markdown
Contributor

@patrikbraborec patrikbraborec commented May 26, 2026

Summary

  • Adds push.actorId and push.wasCreated to the unified cli_command telemetry payload so each cli_command_push event carries the Actor ID being pushed and whether the push created a new Actor vs. updated an existing one.
  • After Keboola ETL refresh, this flattens to push_actor_id / push_was_created columns in Snowflake (mirroring the existing create.* pattern), unlocking per-surface attribution and dedup'd new-Actor counts.

Closes #1151

Adds `push.actorId` and `push.wasCreated` to the unified cli_command
telemetry payload so we can dedupe pushes per Actor in Snowflake and
attribute new-Actor creation through the CLI vs. Console surfaces.

Closes #1151

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Comment thread test/api/commands/push.test.ts Outdated
Drop the `as unknown as { ... }` casts in favor of bracket-notation
access to the protected `telemetryData` property, matching the
codebase's existing pattern (e.g. `instance['_buildParseArgsOption']`).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@patrikbraborec patrikbraborec enabled auto-merge (squash) May 26, 2026 11:54
@patrikbraborec patrikbraborec disabled auto-merge May 26, 2026 13:19
@patrikbraborec patrikbraborec enabled auto-merge (squash) May 26, 2026 13:19
@patrikbraborec patrikbraborec disabled auto-merge May 27, 2026 06:19
@patrikbraborec patrikbraborec enabled auto-merge (squash) May 27, 2026 06:20
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added this to the 141st sprint - DX team milestone May 27, 2026
@github-actions github-actions Bot added t-dx Issues owned by the DX team. tested Temporary label used only programatically for some analytics. labels May 27, 2026
@patrikbraborec patrikbraborec merged commit a3931bf into master May 27, 2026
23 checks passed
@patrikbraborec patrikbraborec deleted the feat/push-telemetry-actor-id-was-created branch May 27, 2026 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-dx Issues owned by the DX team. tested Temporary label used only programatically for some analytics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enrich cli_command_push telemetry with actorId and wasCreated flag

3 participants