Visitar URL original
Record CAPI X-GitHub-Copilot-Request-Te on GitHub telemetry events by rohit489 · Pull Request #339087 · microsoft/vscode · GitHub
Skip to content

Record CAPI X-GitHub-Copilot-Request-Te on GitHub telemetry events - #339087

Merged
Vijay Upadya (vijayupadya) merged 2 commits into
microsoft:mainfrom
rohit489:rohit489/gitHubCopilotRequestTe
Oct 1, 2026
Merged

Vijay Upadya (vijayupadya) merged 2 commits into
microsoft:mainfrom
rohit489:rohit489/gitHubCopilotRequestTe

Conversation

@rohit489

Copy link
Copy Markdown
Contributor

Capture the raw X-GitHub-Copilot-Request-Te response header once per model
call and forward it unchanged as gitHubCopilotRequestTe on GitHub-bound
telemetry. VS Code never interprets, normalizes or defaults the value, and
the property is omitted when the header is absent (e.g. BYOK and non-CAPI
endpoints).

Copilot AI balanced review requested due to automatic review settings October 1, 2026 12:17
@vs-code-engineering

Copy link
Copy Markdown
Contributor

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

Ulugbek Abdullaev (@ulugbekna)

Matched files:

  • extensions/copilot/src/extension/completions-core/vscode-node/lib/src/telemetry.ts
  • extensions/copilot/src/extension/inlineEdits/node/nextEditProviderTelemetry.ts
  • extensions/copilot/src/extension/inlineEdits/vscode-node/inlineCompletionProvider.ts

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Empty headers are mishandled, aggregate review actions can be misattributed, and inline-chat follow-up telemetry loses the correlation value.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Captures the raw CAPI request telemetry header and propagates it through GitHub telemetry across HTTP, WebSocket, chat, editing, review, and completion workflows.

Changes:

  • Adds header extraction and per-model-call correlation.
  • Propagates the value to downstream GitHub telemetry.
  • Adds transport and telemetry coverage.
File Description
platform/​telemetry/​common/​telemetryData.ts Adds request header telemetry state.
platform/​review/​common/​reviewService.ts Associates review comments with model calls.
platform/​openai/​node/​fetch.ts Carries the header on cancellations.
platform/​networking/​test/​node/​fetch.spec.ts Tests extraction and omission.
platform/​networking/​node/​test/​chatWebSocketManager.spec.ts Tests WebSocket envelope capture.
platform/​networking/​node/​chatWebSocketTelemetry.ts Adds WebSocket event properties.
platform/​networking/​node/​chatWebSocketManager.ts Captures per-turn envelope values.
platform/​networking/​node/​chatStream.ts Adds compaction telemetry correlation.
platform/​networking/​common/​fetch.ts Defines header extraction helpers.
platform/​multiFileEdit/​common/​multiFileEditQualityTelemetry.ts Correlates edit-quality telemetry.
platform/​endpoint/​test/​node/​messagesApi.spec.ts Tests Messages API propagation.
platform/​endpoint/​node/​responsesApi.ts Propagates Responses API values.
platform/​endpoint/​node/​messagesApi.ts Propagates Messages API values.
platform/​endpoint/​node/​chatEndpoint.ts Propagates chat completion values.
platform/​chat/​common/​commonTypes.ts Extends response error metadata.
extension/​tools/​node/​skillTool.ts Correlates skill-read telemetry.
extension/​tools/​node/​readFileTool.tsx Correlates skill file reads.
extension/​tools/​node/​applyPatchTool.tsx Correlates patch telemetry.
extension/​tools/​node/​abstractReplaceStringTool.tsx Correlates replacement telemetry.
extension/​tools/​common/​skillTelemetry.ts Adds GitHub skill telemetry property.
extension/​prompts/​node/​codeMapper/​codeMapperService.ts Correlates edit-survival events.
extension/​prompts/​node/​codeMapper/​codeMapper.ts Tracks single-call mapper attribution.
extension/​prompt/​test/​node/​toolCallRound.spec.ts Tests round correlation helpers.
extension/​prompt/​node/​test/​feedbackGenerator.spec.ts Tests review telemetry propagation.
extension/​prompt/​node/​test/​chatMLFetcherRetry.spec.ts Tests retry-specific values.
extension/​prompt/​node/​test/​chatMLFetcherResponseApiTelemetry.spec.ts Tests HTTP and WebSocket telemetry.
extension/​prompt/​node/​intentDetector.tsx Correlates intent telemetry.
extension/​prompt/​node/​feedbackGenerator.ts Tags review comments and actions.
extension/​prompt/​node/​defaultIntentRequestHandler.ts Stores response-call metadata.
extension/​prompt/​node/​chatParticipantTelemetry.ts Adds chat and tool telemetry.
extension/​prompt/​node/​chatMLFetcherTelemetry.ts Adds model-response event properties.
extension/​prompt/​node/​chatMLFetcher.ts Propagates values through fetch outcomes.
extension/​prompt/​common/​toolCallRound.ts Adds in-memory round correlation.
extension/​prompt/​common/​conversation.ts Adds in-memory turn metadata.
extension/​intents/​node/​toolCallingLoop.ts Associates model calls with rounds.
extension/​inlineEdits/​vscode-node/​inlineCompletionProvider.ts Correlates survival telemetry.
extension/​inlineEdits/​node/​nextEditProviderTelemetry.ts Adds NES telemetry correlation.
extension/​inlineChat2/​node/​inlineChatIntent.ts Passes inline response correlation.
extension/​externalAgents/​node/​oaiLanguageModelServer.ts Propagates pass-through response values.
extension/​conversation/​vscode-node/​userActions.ts Correlates feedback telemetry.
extension/​completions-core/​vscode-node/​lib/​src/​telemetry.ts Adds completion telemetry state.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread extensions/copilot/src/extension/prompt/node/feedbackGenerator.ts Outdated
@vijayupadya
Vijay Upadya (vijayupadya) merged commit 3bf9306 into microsoft:main Oct 1, 2026
35 checks passed
@vs-code-engineering vs-code-engineering Bot added this to the 1.141.0 milestone Oct 1, 2026
@rohit489
Rohit Agrawal - MSFT (rohit489) deleted the rohit489/gitHubCopilotRequestTe branch October 2, 2026 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants