Skip to content

Telemetry: add RequestId and ConversationId to toolCallDetails and response.success events#310281

Merged
bhavyaus merged 1 commit intomainfrom
anisha/requestids-telemetry
Apr 16, 2026
Merged

Telemetry: add RequestId and ConversationId to toolCallDetails and response.success events#310281
bhavyaus merged 1 commit intomainfrom
anisha/requestids-telemetry

Conversation

@24anisha
Copy link
Copy Markdown
Contributor

Currently, the toolCallDetails events don't contain the relevant RequestId, and response.success events don't contain the ConversationId. This makes it difficult to complete some of our desired telemetry data analysis.

Copilot AI review requested due to automatic review settings April 15, 2026 22:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

Screenshot Changes

Base: ca3b9bfb Current: 66b88ecf

Changed (2)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark
Before After
before after
chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Light
Before After
before after

blocks-ci screenshots changed

Replace the contents of test/componentFixtures/blocks-ci-screenshots.md with:

Updated blocks-ci-screenshots.md
<!-- auto-generated by CI — do not edit manually -->

#### editor/codeEditor/CodeEditor/Dark
![screenshot](https://hediet-screenshots.azurewebsites.net/images/cb32a3e854b5734fe5aaca2318f2e0a42ee821b05ea97883ea42c5ba95edb3c3)

#### editor/codeEditor/CodeEditor/Light
![screenshot](https://hediet-screenshots.azurewebsites.net/images/42624fbba5e0db7f32c224b5eb9c5dd3b08245697ae2e7d2a88be0d7c287129b)

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

This PR enhances Copilot chat telemetry so analysts can reliably correlate tool-calling behavior and successful responses with the correct per-turn request and conversation identifiers.

Changes:

  • Adds requestId to the toolCallDetails telemetry event payload (and updates its GDPR schema accordingly).
  • Threads conversationId into response.success telemetry by propagating it into the fetcher’s base telemetry and emitting it in the success event (with GDPR schema update).
Show a summary per file
File Description
extensions/copilot/src/extension/prompt/node/chatParticipantTelemetry.ts Adds requestId to toolCallDetails event properties and declares it in the GDPR schema.
extensions/copilot/src/extension/prompt/node/chatMLFetcherTelemetry.ts Adds conversationId to the response.success GDPR schema and includes it in emitted success telemetry properties.
extensions/copilot/src/extension/prompt/node/chatMLFetcher.ts Ensures conversationId is included in the base telemetry data used downstream by response.success.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 0

@bhavyaus bhavyaus force-pushed the anisha/requestids-telemetry branch from 82ed4e6 to 035a5a7 Compare April 16, 2026 05:32
@bhavyaus bhavyaus enabled auto-merge (squash) April 16, 2026 05:33
@bhavyaus bhavyaus merged commit 866d1d8 into main Apr 16, 2026
25 of 26 checks passed
@bhavyaus bhavyaus deleted the anisha/requestids-telemetry branch April 16, 2026 06:05
@vs-code-engineering vs-code-engineering bot added this to the 1.117.0 milestone Apr 16, 2026
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.

3 participants