Skip to content

MW-11193: Handle deprecated Copilot Metrics API - #114

Merged
myleshenderson merged 2 commits into
masterfrom
myles/MW-11193-handle-copilot-deprecation
Apr 1, 2026
Merged

MW-11193: Handle deprecated Copilot Metrics API#114
myleshenderson merged 2 commits into
masterfrom
myles/MW-11193-handle-copilot-deprecation

Conversation

@myleshenderson

@myleshenderson myleshenderson commented Apr 1, 2026

Copy link
Copy Markdown

There's a follow on ticket to remove copilot code related to the deprecated enpoints.

This is just defensive to make sure that nothing blows up and give us time to do the other ticket at our convenience.

Summary

  • Catch all exceptions from the copilot_usage stream so the tap gracefully skips copilot metrics instead of crashing the entire sync
  • The GitHub Copilot metrics API endpoints (/orgs/{org}/copilot/metrics and /orgs/{org}/team/{team_slug}/copilot/metrics) are being shut down on April 2, 2026
  • Broad except Exception used at both org-level and team-level to handle any error GitHub returns (410 Gone, 500, or otherwise)

🤖 Generated with Claude Code

The GitHub Copilot metrics API endpoints are being shut down on April 2,
2026. Catch all exceptions from copilot_usage stream so the tap gracefully
skips copilot metrics instead of crashing, regardless of what error GitHub
returns after deprecation.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@myleshenderson
myleshenderson requested a review from aaboyd April 1, 2026 14:41
@myleshenderson myleshenderson added the low-risk Low risk changes not needing review label Apr 1, 2026
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@myleshenderson
myleshenderson merged commit 18249df into master Apr 1, 2026
1 check passed
@myleshenderson
myleshenderson deleted the myles/MW-11193-handle-copilot-deprecation branch April 1, 2026 14:48

@KBorders01 KBorders01 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good. FYI @aaboyd it looks like you missed this one originally.

@aaboyd

aaboyd commented Jun 30, 2026

Copy link
Copy Markdown

Looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-risk Low risk changes not needing review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants