Skip to content

feat(analytics): enhance command tracking with success and failure events#1551

Open
ramya18101 wants to merge 1 commit into
mainfrom
DXCDT-1862-enhance-command-tracking-events
Open

feat(analytics): enhance command tracking with success and failure events#1551
ramya18101 wants to merge 1 commit into
mainfrom
DXCDT-1862-enhance-command-tracking-events

Conversation

@ramya18101

@ramya18101 ramya18101 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

🔧 Changes

This pull request refactors and improves analytics tracking for command execution in the CLI. It replaces the previous generic command run tracking with more granular success/failure tracking, adds classification of failure types, and ensures command outcome is always tracked at the end of execution. The changes also include expanded test coverage for analytics and error classification logic.

📚 References

DXCDT-1864

🔬 Testing

Added necessary tests: TestGenerateSucceededEventName, TestGenerateFailureEventName

📝 Checklist

  • All new/changed/fixed functionality is covered by tests (or N/A)
  • I have added documentation for all new/changed functionality (or N/A)

@brth31

brth31 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

One thing to flag before merging: the event name changes from CLI - Apps - Create - Run to CLI - Apps - Create - Succeeded / ...Failed. Any Heap saved reports or queries filtering by event name will stop returning results at the cutover. Before merging, identify who owns the Heap CLI usage dashboards and give them the merge date so they can update their queries. No code change needed here.

Also: the PR description is blank — please add a summary of what changed and why before this goes to review.

@ramya18101 ramya18101 marked this pull request as ready for review June 18, 2026 00:33
@ramya18101 ramya18101 requested a review from a team as a code owner June 18, 2026 00:33
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