Skip to content

Add advanced code review metrics view - #851

Draft
dayland wants to merge 5 commits into
mainfrom
feature/code-review-advanced-metrics
Draft

Add advanced code review metrics view#851
dayland wants to merge 5 commits into
mainfrom
feature/code-review-advanced-metrics

Conversation

@dayland

@dayland dayland commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Restores the PR Review diagnostics that were previously trimmed and adds a dedicated advanced dashboard for engineering analysis.

  • Persists raw usage telemetry, coverage, BCQuality article counts, and sub-skill diagnostics
  • Aggregates partial telemetry with explicit token and credit coverage
  • Records BC-Bench commit/version, Copilot CLI version, BC-ALAgents commit, and BCQuality commit/content version as aggregate identity
  • Displays linked harness pins on the default and advanced dashboards
  • Keeps the default leaderboard focused on headline metrics
  • Preserves historical runs by treating unavailable diagnostics and identity as missing rather than zero

Depends on microsoft/BC-ALAgents#62 for persisted sub-results used by complete article and executed-sub-skill diagnostics.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 57308020-6803-4e62-b08e-57fa875bff89
@dayland
dayland marked this pull request as draft September 3, 2026 12:50
@dayland

dayland commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator Author

Marking this draft while the advanced data contract is completed. BC-ALAgents currently logs BCQuality filtering/consumption but does not persist article retained, pruned, cited/used, sub-skill execution, or usage-completeness diagnostics into BC-Bench result artifacts, so this page cannot yet show the requested full performance set.

Restore raw usage and BCQuality diagnostics to result data, aggregate them with explicit telemetry coverage, and expose them in the advanced code-review view.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 57308020-6803-4e62-b08e-57fa875bff89
Include immutable BC-Bench, Copilot CLI, BC-ALAgents, and BCQuality identity in summaries and aggregate grouping, and display linked pins on both code-review dashboards.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 57308020-6803-4e62-b08e-57fa875bff89
Track the current carry-over-focused IsHandled article name so dataset coverage remains aligned with BCQuality main.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 57308020-6803-4e62-b08e-57fa875bff89
Keep historical direct-agent results available while making BC PR Review the baseline and performance comparison surface.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 57308020-6803-4e62-b08e-57fa875bff89

@haoranpb Sun Haoran (haoranpb) 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.

dayland I think lots of the changes here is not needed, if we strictly follow the versioning policy of BC-Bench.

A version of BC-Bench, e.g. v0.10.0, specifically list the version of all agent harness.

So a fixed version of BC-Bench should point to a fixed version of all agent harnesses, which should then points to fixed versions of bcquality etc

Note: the version of BC-Bench is already collected as part of the results

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