Skip to content

fix(console): redesign run detail around trustworthy outcomes - #3242

Open
AbigailDeng wants to merge 4 commits into
feat/2026-08-04_workflow-activity-vnextfrom
fix/2026-08-06_run-detail-outcome-path-usage
Open

fix(console): redesign run detail around trustworthy outcomes#3242
AbigailDeng wants to merge 4 commits into
feat/2026-08-04_workflow-activity-vnextfrom
fix/2026-08-06_run-detail-outcome-path-usage

Conversation

@AbigailDeng

@AbigailDeng AbigailDeng commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Problem and solution

The run detail surface exposed runtime event names and raw IDs, treated ended steps as completed successes, and presented zero usage without measurement provenance.

This frontend slice reorganizes run detail into five goal-based surfaces:

  • Overview reconciles Attempted / Succeeded / Failed / Waiting and keeps Skipped as not reported until the contract provides it.
  • Steps use product-facing role/type labels, explicit outcomes, durations, and collapsed technical evidence.
  • Timeline maps machine events to localized user events, localizes absolute timestamps, and shows run-relative offsets.
  • Usage reports only values with evidence, does not invent measured zero or a currency, and labels absent facts as Not reported.
  • Execution path derives from the same versioned run step facts rather than the separately fetched graph, so the tabs cannot contradict one another.

Diagnostics, input, request parameters, machine events, state version, and raw run/step IDs remain copyable under collapsed Technical details. Retry/run-again behavior is preserved and remains outside this presentation-focused change.

References #3226

Impacted paths

  • apps/aevatar-console-web/src/pages/workflow-activity-vnext/activity/RunDetailPage.tsx
  • apps/aevatar-console-web/src/pages/workflow-activity-vnext/activity/runDetailPresentation.ts
  • focused tests, EN/ZH messages, and responsive vNext styles

Backend contract dependencies

The current run detail contract does not provide typed initiator, redacted input summary, usage measurement provenance, cost currency, skipped-step facts, workflow-node display names, or a graph state version. This PR does not infer those facts from IDs, routes, zero values, diagnostic strings, or a separately versioned graph. Those fields remain unavailable/not reported until the backend read model supplies authoritative typed values.

Local verification

  • Related page/presentation tests: pnpm exec jest src/pages/workflow-activity-vnext/activity/RunDetailPage.test.tsx src/pages/workflow-activity-vnext/activity/runDetailPresentation.test.ts --runInBand (18 passed)
  • Locale catalog/audit: pnpm exec jest src/locales/catalog.test.ts src/locales/hardcodedCopyAudit.test.ts --runInBand (14 passed)
  • Analyzer-selected related tests: 96 suites / 1184 tests passed; the sole initial locale audit failure identified the missing Run ID catalog entry, which was added and the guard rerun successfully
  • Changed-file static checks: pnpm exec biome check <7 changed files> (passed)
  • Test stability guard: bash tools/ci/test_stability_guards.sh (passed)
  • Workflow Activity vNext baseline: 17/17 frames byte-identical, SHA-256 30e74d7b410ae72c4c91432355436679033679c54c10b1702908435b001577de
  • Diff whitespace check: passed
  • A reliable affected-only TypeScript target is unavailable; local typecheck was skipped
  • Full frontend suite/typecheck/build: deferred to GitHub CI by personal local workflow policy

CI remediation (2026-08-07)

  • Merged latest base be74bab5302b0ec9b5d4b2d6eecb9156dbf829a0.
  • CI failure reproduction: CODEX_ALLOW_FULL_FRONTEND_VALIDATION=1 pnpm --dir apps/aevatar-console-web tsc - passed.
  • Related tests: pnpm exec jest src/pages/workflow-activity-vnext/activity/RunDetailPage.test.tsx src/pages/workflow-activity-vnext/activity/runDetailPresentation.test.ts --runInBand - 2 suites / 19 tests passed.
  • Changed-file static checks: pnpm exec biome check <7 analyzer-selected changed files> - 7 files passed.
  • Test stability guard: bash tools/ci/test_stability_guards.sh - passed.
  • Full frontend suite and production build are delegated to GitHub CI by the personal incremental frontend policy.

@AbigailDeng

AbigailDeng commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Local verification

  • Related tests: pnpm exec jest src/pages/workflow-activity-vnext/activity/RunDetailPage.test.tsx src/pages/workflow-activity-vnext/activity/runDetailPresentation.test.ts --runInBand — 2 suites, 19 tests passed
  • Changed-file static checks: pnpm exec biome lint src/locales/workflowActivityVNextMessages.en-US.ts src/locales/workflowActivityVNextMessages.zh-CN.ts src/pages/workflow-activity-vnext/activity/RunDetailPage.test.tsx src/pages/workflow-activity-vnext/activity/RunDetailPage.tsx src/pages/workflow-activity-vnext/activity/runDetailPresentation.test.ts src/pages/workflow-activity-vnext/activity/runDetailPresentation.ts src/pages/workflow-activity-vnext/styles.ts — 7 files passed
  • Test stability guard: bash tools/ci/test_stability_guards.sh — passed
  • Full frontend suite/typecheck/build: deferred to GitHub CI by personal local workflow policy

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