Skip to content

Add telemetry for assignable users#312104

Merged
alexr00 merged 2 commits intomainfrom
alexr00/divine-moose
Apr 23, 2026
Merged

Add telemetry for assignable users#312104
alexr00 merged 2 commits intomainfrom
alexr00/divine-moose

Conversation

@alexr00
Copy link
Copy Markdown
Member

@alexr00 alexr00 commented Apr 23, 2026

Co-authored-by: Copilot copilot@github.com

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 23, 2026 08:56
@alexr00 alexr00 enabled auto-merge (squash) April 23, 2026 08:56
@alexr00 alexr00 self-assigned this Apr 23, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

Screenshot Changes

Base: e2accb2b Current: c564bf0c

Changed (1)

editor/inlineCompletions/other/JumpToHint/Dark
Before After
before after

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

Adds Microsoft telemetry for failures when fetching assignable actors/users from GitHub, so the Copilot extension can better understand reliability across GitHub.com vs older GHES API capabilities.

Changes:

  • Tracks whether the suggestedActors GraphQL API was used vs falling back to assignableUsers.
  • Sends an MSFT telemetry error event on failures, including an extracted error code and the API path used.
  • Introduces a helper to derive an error/status code from a caught error object.
Show a summary per file
File Description
extensions/copilot/src/platform/github/common/octoKitServiceImpl.ts Emits telemetry on getAssignableActors failures and adds error-code extraction helper.

Copilot's findings

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

Comment thread extensions/copilot/src/platform/github/common/octoKitServiceImpl.ts
Comment thread extensions/copilot/src/platform/github/common/octoKitServiceImpl.ts Outdated
Comment thread extensions/copilot/src/platform/github/common/octoKitServiceImpl.ts
Co-authored-by: Copilot <copilot@github.com>
@alexr00 alexr00 merged commit e923ba6 into main Apr 23, 2026
26 checks passed
@alexr00 alexr00 deleted the alexr00/divine-moose branch April 23, 2026 13:23
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 23, 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