Skip to content

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

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

Sync eng/common directory with azure-sdk-tools for PR 14737#37839
azure-sdk merged 3 commits into
mainfrom
sync-eng/common-telemetry_script_vscode_support-14737

Conversation

@azure-sdk
Copy link
Copy Markdown
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
Copy Markdown
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

Syncs the eng/common telemetry hook script with the corresponding changes from azure-sdk-tools PR 14737, improving compatibility with differing JSON payload shapes from various clients.

Changes:

  • Normalize extraction of toolName, sessionId, and tool input by supporting multiple property names (toolName vs tool_name, etc.).
  • Minor cleanup of extraneous whitespace in the script.

Comment thread eng/common/scripts/azsdk_tool_telemetry.ps1
Comment thread eng/common/scripts/azsdk_tool_telemetry.ps1 Outdated
@azure-sdk azure-sdk force-pushed the sync-eng/common-telemetry_script_vscode_support-14737 branch from af2020c to c09688c Compare March 26, 2026 00:58
@azure-sdk azure-sdk force-pushed the sync-eng/common-telemetry_script_vscode_support-14737 branch from c09688c to 7fc32f5 Compare March 26, 2026 01:07
@azure-sdk azure-sdk merged commit ccb46a6 into main Mar 26, 2026
13 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.

4 participants