Skip to content

feat(code-review): track review runtime for agent health dashboard - #6974

Merged
eshurakov merged 3 commits into
mainfrom
eshurakov/agent-health-overview
Sep 29, 2026
Merged

eshurakov merged 3 commits into
mainfrom
eshurakov/agent-health-overview

Conversation

@eshurakov

@eshurakov eshurakov commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Emit completed-review run-time sum and sample count in the existing five-minute review outcome event, so the live Axiom dashboard can compute a weighted last-hour average excluding queue wait.
  • Test valid completed runtimes, empty windows, emitted aggregates, and query constraints.

Verification

  • Focused review-health-aggregate Jest suite: 18 passed.
  • Incremental typecheck, focused oxlint, oxfmt check, and git diff --check passed.

Rollout note

The dashboard has already been configured separately at https://app.axiom.co/kilo-code-nvjh/dashboards/agent-health. Its average run-time tile remains blank until this web telemetry change is deployed. The dashboard JSON definition is intentionally not part of this PR.

@kilo-code-bot

kilo-code-bot Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the completed-review runtime telemetry (sum and sample count) added to the existing five-minute review outcome event and its tests; the SQL filters to nonnegative completed runtimes and the aggregation sums weighted buckets correctly.

Files Reviewed (2 files)
  • apps/web/src/lib/code-reviews/telemetry/review-health-aggregate.ts
  • apps/web/src/lib/code-reviews/telemetry/review-health-aggregate.test.ts
Previous Review Summary

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • apps/web/src/lib/code-reviews/telemetry/review-health-aggregate.ts
  • apps/web/src/lib/code-reviews/telemetry/review-health-aggregate.test.ts
  • dev/axiom/dashboards/agent-health.json

Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0

Review guidance: REVIEW.md from base branch main

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