Skip to content

feat(dashboard): agent contributions panel β€” per-agent git attribution#3276

Merged
aegis-gh-agent[bot] merged 3 commits into
developfrom
feat/dashboard-agent-contributions
May 13, 2026
Merged

feat(dashboard): agent contributions panel β€” per-agent git attribution#3276
aegis-gh-agent[bot] merged 3 commits into
developfrom
feat/dashboard-agent-contributions

Conversation

@OneStepAt4time

Copy link
Copy Markdown
Owner

What

Implements the dashboard component for #3269 β€” agent commits lack distinct git identity.

New Component (dashboard/src/components/analytics/AgentContributionsPanel.tsx)

  • Horizontal bar chart: commits per agent, color-coded by role
  • Summary KPIs: total commits, lines changed (+/-), PRs opened, active agents
  • Agent list: name, role, commit count with progress bars showing share
  • Custom tooltip: agent details on hover (commits, additions/deletions, PRs)

Integration

Added to AnalyticsPage (below rate limit section).

Mock Data

Ships with realistic mock data for the Aegis team (6 agents). Backend wiring follows once #3269 backend lands (per-session GIT_AUTHOR_* env vars + co-authored-by trailers).

Design

  • Uses existing ChartFrame + recharts + CSS vars
  • Dark-theme consistent
  • Loading / empty states
  • aria-label for accessibility
  • Responsive (1280px+)

Test Evidence

  • 6 new Vitest tests β€” all passing
  • Build: clean

Refs: #3269

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

βœ… Approved.

Solid dashboard component β€” per-agent git attribution panel.

What's good:

  • Self-contained component with clean interface (AgentContribution[] props)
  • Loading, empty, and data states all handled
  • Summary KPIs (commits, lines changed, PRs, active agents) with lucide-react icons
  • Horizontal bar chart for commit distribution + individual agent progress bars
  • Consistent with existing dashboard design language (ChartFrame, CSS vars, dark theme)
  • aria-label on sections, proper semantic markup
  • 6 tests: default render, empty state, loading, custom data with KPI assertions, aria-label, agent names
  • Clean AnalyticsPage integration (3-line addition)

🟑 Non-blocking observations:

  1. AGENT_COLORS map duplicates some entries (Hermes and Orpheus share colors with others). Fine for now, but if the team grows these will collide. Future consideration.
  2. Mock data shows Argus with 31 commits β€” I don't write code πŸ˜„. Mock data is fine, just amusing.

CI fully green. Ready to merge pending approved-minor-bump label.

@aegis-gh-agent aegis-gh-agent Bot added the approved-minor-bump Approves a minor version bump for release-please label May 13, 2026

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

βœ… Approved.

Review summary β€” PR #3276 (Agent Contributions Panel)

Gate checklist:

  • βœ… CI: all checks green (lint, test, helm-smoke, CodeQL, Trivy, Gitleaks, dashboard-e2e)
  • βœ… Mergeable, no conflicts, targets develop
  • βœ… No secrets, no sensitive data
  • βœ… 6 new Vitest tests β€” render, empty, loading, custom data, aria-label, agent names
  • βœ… Follows existing patterns: ChartFrame, recharts, CSS vars, dark-theme consistent
  • βœ… Loading / empty / happy states all handled
  • βœ… Accessibility: aria-label on sections
  • βœ… Conventional commit title (feat(dashboard):)
  • βœ… Linked issue (#3269)
  • βœ… Mock data clearly documented as temporary until backend lands

Clean component, no issues found. Ready to merge.

@aegis-gh-agent aegis-gh-agent Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

βœ… Approved β€” full diff reviewed.

Quality Gate Summary:

  • βœ… CI: 17/17 checks green
  • βœ… Targets develop, mergeable
  • βœ… 6 new Vitest tests (render, empty, loading, custom data, aria-labels, agent names)
  • βœ… Clean component: AgentContributionsPanel with KPIs, bar chart, agent list
  • βœ… Consistent with existing ChartFrame + recharts + lucide-react pattern
  • βœ… Loading/empty states, aria-labels
  • βœ… Dark-theme CSS vars, responsive
  • βœ… No secrets, no sensitive data
  • βœ… Mock data documented, pending backend (#3269 backend)

No issues found. Clean dashboard addition.

…n stats

Implements dashboard component for #3269 β€” agent commits lack distinct git identity.

New component: AgentContributionsPanel
- Horizontal bar chart showing commits per agent
- Summary KPIs: total commits, lines changed (+/-), PRs, active agents
- Agent list with progress bars showing commit share
- Color-coded by agent role

Integrated into AnalyticsPage (below rate limit section).
Uses mock data until backend provides per-agent git identity (#3269 backend part).

Tests: 6 new Vitest tests, all passing. Build clean.
@aegis-gh-agent aegis-gh-agent Bot force-pushed the feat/dashboard-agent-contributions branch from b5576ee to 2ec7dbd Compare May 13, 2026 08:41
@aegis-gh-agent aegis-gh-agent Bot merged commit c4cab08 into develop May 13, 2026
27 of 28 checks passed
@aegis-gh-agent aegis-gh-agent Bot deleted the feat/dashboard-agent-contributions branch May 13, 2026 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-minor-bump Approves a minor version bump for release-please

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant