Skip to content

Sync eng/common directory with azure-sdk-tools for PR 14737#7029

Merged
azure-sdk merged 3 commits intomainfrom
sync-eng/common-telemetry_script_vscode_support-14737
Mar 26, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 14737#7029
azure-sdk merged 3 commits intomainfrom
sync-eng/common-telemetry_script_vscode_support-14737

Conversation

@azure-sdk
Copy link
Collaborator

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#14737 See eng/common workflow

@azure-sdk azure-sdk requested a review from a team as a code owner March 26, 2026 00:51
@azure-sdk azure-sdk added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Mar 26, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR syncs eng/common content with the upstream azure-sdk-tools changes from azure-sdk-tools PR 14737, updating the telemetry hook script to accept multiple possible JSON property names emitted by different clients.

Changes:

  • Add compatibility parsing for toolName vs tool_name.
  • Add compatibility parsing for sessionId vs session_id.
  • Add compatibility parsing for toolArgs vs tool_input, with explicit null initialization.

@azure-sdk azure-sdk force-pushed the sync-eng/common-telemetry_script_vscode_support-14737 branch from 0766fc4 to d393c9d Compare March 26, 2026 00:57
@azure-sdk azure-sdk force-pushed the sync-eng/common-telemetry_script_vscode_support-14737 branch from d393c9d to 9c389ee Compare March 26, 2026 01:05
@azure-sdk azure-sdk merged commit 5b2f92a into main Mar 26, 2026
44 checks passed
@azure-sdk azure-sdk deleted the sync-eng/common-telemetry_script_vscode_support-14737 branch March 26, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants