Skip to content

Render Chat run lifecycle inline - #3126

Open
AbigailDeng wants to merge 2 commits into
devfrom
codex/issue-3119-console-vnext-5-render-the-run-and-permission-lifecycle-inline
Open

Render Chat run lifecycle inline#3126
AbigailDeng wants to merge 2 commits into
devfrom
codex/issue-3119-console-vnext-5-render-the-run-and-permission-lifecycle-inline

Conversation

@AbigailDeng

Copy link
Copy Markdown
Contributor

Summary

  • Render structured Run lifecycle state, current steps, Run Detail navigation, and explicit recovery inline in Chat without deriving runtime truth from assistant prose.
  • Restore tool approval and runtime intervention actions with pending-command duplicate prevention, and map structured permission codes to account-access remediation.
  • Reconcile settled live Runs through the durable Run summary API while preserving Run and actor identities from stream events.

Verification

  • pnpm --dir apps/aevatar-console-web exec jest --runInBand --runTestsByPath src/pages/chat/index.test.tsx src/pages/chat/chatRunLifecycle.test.ts: passed (47 tests)
  • pnpm --dir apps/aevatar-console-web tsc: passed
  • pnpm --dir apps/aevatar-console-web exec biome lint src/pages/chat/index.tsx src/pages/chat/index.test.tsx src/pages/chat/chatRunLifecycle.tsx src/pages/chat/chatRunLifecycle.test.ts src/pages/chat/chatTypes.ts src/locales/en-US.ts src/locales/zh-CN.ts --max-diagnostics=100: passed
  • bash tools/ci/test_stability_guards.sh: passed
  • git diff --check: passed

Risks

  • Existing persisted Chat history does not yet expose a durable Run correlation contract, so history entries without a Run ID intentionally show recovery guidance instead of an inferred terminal state.
  • The complete frontend test suite was not run under the repository incremental test policy.

Closes #3119

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.56%. Comparing base (3f548ff) to head (414baec).

@@           Coverage Diff           @@
##              dev    #3126   +/-   ##
=======================================
  Coverage   84.56%   84.56%           
=======================================
  Files        1503     1503           
  Lines      140015   140015           
  Branches    17180    17180           
=======================================
+ Hits       118397   118401    +4     
+ Misses      13752    13749    -3     
+ Partials     7866     7865    -1     
Flag Coverage Δ
ci 84.56% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

[Console vNext][5] Render the Run and permission lifecycle inline in Chat

1 participant