Skip to content

feat: Add Copilot app metrics to usage metrics structs - #4488

Open
Tens1des wants to merge 2 commits into
google:masterfrom
Tens1des:feat/copilot-app-metrics
Open

feat: Add Copilot app metrics to usage metrics structs#4488
Tens1des wants to merge 2 commits into
google:masterfrom
Tens1des:feat/copilot-app-metrics

Conversation

@Tens1des

Copy link
Copy Markdown
Contributor

Fixes #4487

Adds support for Copilot app usage metrics fields documented at:
https://docs.github.com/en/copilot/reference/copilot-usage-metrics/copilot-usage-metrics

Test plan

  • go test ./github/... -run Copilot
  • ./script/generate.sh

Add used_copilot_app, used_copilot_cloud_agent, totals_by_copilot_app,
and daily_active_copilot_app_users to match the Copilot usage metrics API.

Fixes google#4487
@gmlewis gmlewis added the NeedsReview PR is awaiting a review before merging. label Aug 25, 2026
@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.51%. Comparing base (27beda9) to head (4192472).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4488   +/-   ##
=======================================
  Coverage   98.51%   98.51%           
=======================================
  Files         195      195           
  Lines       17691    17691           
=======================================
  Hits        17429    17429           
  Misses        262      262           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gmlewis gmlewis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you, @Tens1des!
LGTM.
Awaiting second LGTM+Approval from any other contributor to this repo before merging.

cc: @stevehipwell - @alexandear - @Not-Dhananjay-Mishra

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

Labels

NeedsReview PR is awaiting a review before merging.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Copilot app metrics fields to Copilot usage metrics structs

2 participants