Skip to content

feat(dashboard): identify result rows by eval path#1543

Merged
christso merged 1 commit into
mainfrom
dashboard-eval-path-after-result-dir
Jun 27, 2026
Merged

feat(dashboard): identify result rows by eval path#1543
christso merged 1 commit into
mainfrom
dashboard-eval-path-after-result-dir

Conversation

@christso

Copy link
Copy Markdown
Collaborator

Summary

Dashboard run tables now use repo-relative eval_path as the visible Eval identity while preserving suite only as a legacy fallback for old run artifacts. Result drill-down links carry result_dir when available, falling back to eval-path-scoped identity so duplicate test_id rows from different eval files continue to open the correct artifact.

This keeps Experiments and Recent Runs centered on eval source identity without renaming unrelated benchmark concepts or the existing compatibility suite routes.

Validation

  • bun test apps/dashboard/src/lib/result-table.test.ts apps/dashboard/src/lib/navigation.test.ts apps/dashboard/src/lib/run-detail-context.test.ts
  • cd apps/dashboard && bun run build
  • bun test apps/cli/test/commands/results/serve.test.ts
  • bun --filter agentv typecheck
  • bunx biome check ...touched dashboard/CLI files...
  • git diff --check

Review Notes

  • Spawned a code-review subagent before PR. It found duplicate-test-id sidebar active/link issues; this PR includes the follow-up fix so sidebar links and active states use result_dir/eval_path identity instead of bare test_id.

Post-Deploy Monitoring & Validation

  • No runtime migration or service monitoring required. This is local Dashboard read-model/navigation behavior with legacy suite fallback retained for old artifacts.

Refs EntityProcess/agentv-beads#16 and bead av-504.4.


Compound Engineering
Codex

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2007054
Status: ✅  Deploy successful!
Preview URL: https://84347d1b.agentv.pages.dev
Branch Preview URL: https://dashboard-eval-path-after-re.agentv.pages.dev

View logs

@christso christso marked this pull request as ready for review June 27, 2026 09:46
@christso christso merged commit 48478db into main Jun 27, 2026
8 checks passed
@christso christso deleted the dashboard-eval-path-after-result-dir branch June 27, 2026 09:46
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.

1 participant