Skip to content

Comments

chore(telemetry): include repoId and headCommitHash in repo telemetry for all users#3956

Open
zhichli wants to merge 3 commits intomainfrom
pleasant-swift
Open

chore(telemetry): include repoId and headCommitHash in repo telemetry for all users#3956
zhichli wants to merge 3 commits intomainfrom
pleasant-swift

Conversation

@zhichli
Copy link
Member

@zhichli zhichli commented Feb 24, 2026

Copilot AI review requested due to automatic review settings February 24, 2026 01:02
@zhichli zhichli self-assigned this Feb 24, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 24, 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 updates the repository telemetry system to send basic repository information (repoId and headCommitHash) to all users while continuing to send detailed information (including remoteUrl and diffsJSON) only to internal users. This change improves telemetry coverage by collecting high-level repository context from all users while maintaining privacy by restricting detailed code diff data to internal Microsoft staff.

Changes:

  • Modified RepoInfoTelemetry to call both sendMSFTTelemetryEvent (for all users) and sendInternalMSFTTelemetryEvent (for internal users only)
  • Removed the ICopilotTokenStore dependency from RepoInfoTelemetry constructor since internal user filtering now happens within the telemetry service itself
  • Updated all test cases to reflect the new dual-telemetry approach and removed copilotTokenStore parameter from all test instantiations

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/extension/prompt/node/repoInfoTelemetry.ts Removed ICopilotTokenStore dependency, added dual telemetry calls (external for all users with limited data, internal for full data), and updated repoId property
src/extension/prompt/node/test/repoInfoTelemetry.spec.ts Added mock for sendMSFTTelemetryEvent, updated test expectations to verify both telemetry methods are called, removed copilotTokenStore parameter from all RepoInfoTelemetry instantiations

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