Skip to content

✨ dashboard: Evidence panel 1 — outcome by author kind (merged / closed / reverted-14d; human · agent · bot) - #12385

Merged
hivecommons-hive[bot] merged 1 commit into
v5from
feat/evidence-panel1-12364
Oct 16, 2026
Merged

hivecommons-hive[bot] merged 1 commit into
v5from
feat/evidence-panel1-12364

Conversation

@clubanderson

Copy link
Copy Markdown
Member

Closes #12364 — RFC #12062, proposal B (API + dashboard), panel 1 — Outcome by author kind.

What

  • New evPanel card Outcome by author kind in the Throughput section's Evidence sub-panel, rendered first (RFC panel order) beside panels 2 / 3 / 8. Same prt-card / ev-panel chrome, inline SVG, no new deps, no inline style=.
  • One grouped bar per author kind (human · agent · bot, plus any other kind the API emits) for merged, closed without merge and reverted within 14 d of merge, scaled against the largest kind; n= per row and the panel total. Tooltip carries merged rate of decided PRs and reverted rate of merged PRs (open PRs excluded).
  • "What this does not show" note: reverted = reverted_by link within 14 d of merge only; schema v1 bundles do not record close-without-merge so closed may read 0; open PRs excluded from rates; merged ≠ correct.
  • Honours the existing window / repo filter wiring (fetchEvidenceSummary is unchanged; the panel reads dto.outcomes_by_author_kind).
  • Empty state when no kind has n > 0.
  • Docs: src/docs/review-evidence.md gains a panel 1 bullet. Changelog fragment added.
  • Tests: evidence_card_ui_test.go pins the new helper, panel key, bar classes, note text, CSS and panel order; evOutcomesSvg added to the ReferenceError guard.

Guardrail (RFC #12062 Non-goals)

Record/read-path only. Nothing in this PR reads into or alters merge eligibility or auto-merge; the API is unchanged and tests touch only index.html markup.

cc @mrbobbytables

Adds the Outcome by author kind evPanel card to the Throughput Evidence
sub-panel, drawn from outcomes_by_author_kind in GET /api/evidence/summary:
grouped merged / closed / reverted-14d bars per human · agent · bot with
n= and a what-this-does-not-show note. Reporting only; nothing reads or
alters merge eligibility.

Closes #12364

Signed-off-by: Andy Anderson <andy@clubanderson.com>
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Oct 16, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow Bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Oct 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Open PR overlap check

Base branch: v5

Conflicts are checked with every PR merged onto the current tip of v5, so a PR that only needs a rebase is not reported as conflicting with its siblings.

PR Title Relation Files
#12346 ✨ nous: clarify passive baseline snapshots in Strategy Lab same files src/pkg/dashboard/static/index.html
#12332 ✨ feature: Add cadence templates for quickly setting cadence for all agents (#12300) same files src/pkg/dashboard/static/index.html
#12328 ✨ feat(dashboard): close settings dialog when clicking Save & close with no changes same files src/pkg/dashboard/static/index.html

@hivecommons-hive

Copy link
Copy Markdown
Contributor

merged by Hive trusted-author auto-merge: author @clubanderson holds role owner and GitHub permission admin on repo; CI green at a668b05.

@kubestellar-prow
kubestellar-prow Bot deleted the feat/evidence-panel1-12364 branch October 16, 2026 13:02
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

We'd love to hear how your experience was: share feedback

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

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ dashboard: Evidence panel 1 — outcome by author kind (merged / closed / reverted-14d; human · agent · bot)

1 participant