Skip to content

Add end to end passrate per skill table to Dashboard#1755

Open
msalaman wants to merge 2 commits intomainfrom
masalama/dashboardUpdates
Open

Add end to end passrate per skill table to Dashboard#1755
msalaman wants to merge 2 commits intomainfrom
masalama/dashboardUpdates

Conversation

@msalaman
Copy link
Copy Markdown
Contributor

@msalaman msalaman commented Apr 7, 2026

Description

Add end to end passrate per skill table to Dashboard
image

Checklist

  • Tests pass locally (cd tests && npm test)
  • If modifying skill descriptions: verified routing correctness with integration tests (npm run test:skills:integration -- <skill>)
  • If modifying skill USE FOR / DO NOT USE FOR / PREFER OVER clauses: confirmed no routing regressions for competing skills
  • Version bumped in skill frontmatter (if skill files changed)

Related Issues

@msalaman msalaman marked this pull request as ready for review April 7, 2026 18:31
Copilot AI review requested due to automatic review settings April 7, 2026 18:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds a new Dashboard panel that surfaces end-to-end (integration) pass rate per skill, including a visual percent bar and pass/fail grouping around a 70% threshold.

Changes:

  • Adds a new “End-to-End Pass Rate per Skill” panel to the dashboard layout.
  • Implements client-side fetching/aggregation/sorting of integration test results into per-skill pass rates.
  • Adds styling for an inline percent bar with a 70% threshold marker.
Show a summary per file
File Description
dashboard/index.html Adds a new panel container for the E2E pass-rate table.
dashboard/assets/style.css Introduces styles for the E2E pass-rate bar, value coloring, and threshold marker.
dashboard/assets/dashboard.js Fetches latest test results, computes per-skill E2E pass rates, and renders a filterable/collapsible panel.

Copilot's findings

Comments suppressed due to low confidence (1)

dashboard/assets/dashboard.js:4

  • The 70% threshold is hard-coded in multiple places (comparisons + UI labels). Please centralize it (e.g., a single constant like E2E_THRESHOLD = 0.7 and derive thresholdPct = Math.round(E2E_THRESHOLD * 100)) to avoid future drift and make threshold changes safer.
  • Files reviewed: 3/3 changed files
  • Comments generated: 8

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