Skip to content
This repository was archived by the owner on Sep 20, 2026. It is now read-only.

Unify Run Detail loading and action states - #32

Merged
cauyxy merged 4 commits into
masterfrom
yxinyux/issue-24-run-detail-state
Jul 18, 2026
Merged

cauyxy merged 4 commits into
masterfrom
yxinyux/issue-24-run-detail-state

Conversation

@cauyxy

@cauyxy cauyxy commented Jul 18, 2026

Copy link
Copy Markdown
Member

Summary

  • model Run Detail loading, not-found, blocking-error, ready, and preserved refresh-failure states explicitly
  • expose semantic native problems for detail reads and screenshot/video actions, with localized target-scoped retry UI
  • serialize detail actions through one visible gate, keep uncancellable video deletion non-dismissible, and format replay metadata by locale
  • update generated bindings, focused behavior tests, and current truth documentation

Testing

  • npm run verify -- --source-only (local macOS)
  • focused Run Detail Vitest coverage (page state, action gate, problem presentation, formatting, confirmation dismissal)
  • Browser Preview smoke at http://127.0.0.1:14207/history/preview-run in Chinese and English
  • npm run tauri dev startup smoke on macOS; native process launched and resolved the local installation/database, but desktop automation could not attach to the unbundled dev window
  • destructive native video deletion was not exercised because no disposable local fixture was available

Closes #24

Release Notes:

  • 战绩详情现在明确区分加载、未找到与错误状态,并在操作失败时保留可用内容和安全重试。

@cauyxy
cauyxy merged commit eb9493f into master Jul 18, 2026
2 of 4 checks passed
@cauyxy
cauyxy deleted the yxinyux/issue-24-run-detail-state branch July 18, 2026 18:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

统一 Run Detail 的加载状态与操作可用性

1 participant