PR #84057: [codex] Fix Control UI terminal run status recovery

• Repository: openclaw/openclaw
• Author: NianJiuZst
• State: closed | merged: False
• Link: https://github.com/openclaw/openclaw/pull/84057

Description (problem / solution / changelog)

Summary

• stop treating terminal session rows as active runs when hasActiveRun lingers after completion
• let terminal chat run status win over stale abortable UI so the Control UI returns to done instead of staying in progress
• add regressions for abort recovery, terminal session refresh, chat rendering, and session badges

Closes #84041.

Verification

• node scripts/run-vitest.mjs ui/src/ui/app-chat.test.ts ui/src/ui/controllers/sessions.test.ts ui/src/ui/views/chat.test.ts ui/src/ui/views/sessions.test.ts
• node scripts/run-oxlint.mjs --tsconfig config/tsconfig/oxlint.core.json ui/src/ui/session-run-state.ts ui/src/ui/app-chat.ts ui/src/ui/chat/run-lifecycle.ts ui/src/ui/views/chat.ts ui/src/ui/views/sessions.ts ui/src/ui/app-chat.test.ts ui/src/ui/controllers/sessions.test.ts ui/src/ui/views/chat.test.ts ui/src/ui/views/sessions.test.ts
• node scripts/run-tsgo.mjs -p test/tsconfig/tsconfig.test.ui.json --incremental --tsBuildInfoFile .artifacts/tsgo-cache/test-ui-issue-84041.tsbuildinfo
• git diff --check -- ui/src/ui/session-run-state.ts ui/src/ui/app-chat.ts ui/src/ui/chat/run-lifecycle.ts ui/src/ui/views/chat.ts ui/src/ui/views/sessions.ts ui/src/ui/app-chat.test.ts ui/src/ui/controllers/sessions.test.ts ui/src/ui/views/chat.test.ts ui/src/ui/views/sessions.test.ts

Real behavior proof

Behavior addressed: Control UI now returns to the terminal Done state when a session row still carries stale hasActiveRun: true after completion, so stale Stop and reading UI do not linger and context compaction becomes available again.

Real environment tested: Local macOS source checkout rendered the current branch Control UI chat surface with current ui/src/ui/views/chat.ts code, current Control UI CSS, and a real Chromium page.

Exact steps or command run after this patch: Ran a one-off local tsx proof script from the current checkout that rendered the current renderChat() output for status: "done" plus stale hasActiveRun: true, loaded that HTML into Chromium, queried the visible Control UI DOM, and saved a screenshot.

Evidence after fix (screenshot, recording, terminal capture, console output, redacted runtime log, linked artifact, or copied live output): Browser console output / terminal capture from the Chromium render:

runStatusLabel=Done
hasDoneStatus=true
hasInProgressStatus=false
stopButtonCount=0
readingIndicatorCount=0
compactDisabled=false
compactLabel=Compact
visibleText="Val Ready to chat Type a message below · / for commands What can you do? Summarize my recent sessions Help me configure a channel Check system health 95% context used 190k / 200k Compact Attach file Start Talk Done New session Export Send"

Observed result after fix: The current branch render shows the terminal Done chip, no in-progress chip, no Stop button, no reading indicator, and an enabled Compact action even when the same session row still reports hasActiveRun: true.

What was not tested: I did not run the reporter's exact custom local provider setup or capture a full live network/provider round-trip; this proof is a real Chromium render of the current Control UI surface for the stale terminal session state that triggered the bug.

Changed files

• ui/src/ui/app-chat.test.ts (modified, +15/-0)
• ui/src/ui/app-chat.ts (modified, +2/-1)
• ui/src/ui/chat/run-lifecycle.ts (modified, +2/-1)
• ui/src/ui/controllers/sessions.test.ts (modified, +1/-1)
• ui/src/ui/session-run-state.ts (added, +13/-0)
• ui/src/ui/views/chat.test.ts (modified, +45/-0)
• ui/src/ui/views/chat.ts (modified, +9/-4)
• ui/src/ui/views/sessions.test.ts (modified, +15/-1)
• ui/src/ui/views/sessions.ts (modified, +2/-1)

PR #84112: [codex] Fix Control UI terminal run status recovery

• Repository: openclaw/openclaw
• Author: clawsweeper[bot]
• State: closed | merged: True
• Link: https://github.com/openclaw/openclaw/pull/84112

Description (problem / solution / changelog)

Makes https://github.com/openclaw/openclaw/pull/84057 merge-ready for the ClawSweeper automerge loop. The edit pass should inspect the live PR diff, review comments, and failing checks; rebase if needed; keep the contributor branch credited; and stop only when validation is green or an external blocker is proven.

ClawSweeper 🐠 replacement reef notes:

• Cluster: automerge-openclaw-openclaw-84057
• Source PRs: https://github.com/openclaw/openclaw/pull/84057
• Credit: Source PR: https://github.com/openclaw/openclaw/pull/84057
• Validation: pnpm check:changed
• Replacement reason: ClawSweeper could not update the source PR branch directly, so it opened a writable replacement PR instead.
• Automerge requested by: @Takhoffman

• Repair fallback: GitHub rejected the repair branch push because it updates workflow files and the ClawSweeper app token does not have workflows permission

Inherited issue-closing references from the source PR: Closes #84041

Co-author credit kept:

• @NianJiuZst: Co-authored-by: NianJiu [email protected]

fish notes: model gpt-5.5, reasoning high; reviewed against f9f503add0ab.

Changed files

• CHANGELOG.md (modified, +1/-0)
• ui/src/ui/app-chat.test.ts (modified, +15/-0)
• ui/src/ui/app-chat.ts (modified, +2/-1)
• ui/src/ui/chat/run-lifecycle.ts (modified, +2/-1)
• ui/src/ui/controllers/sessions.test.ts (modified, +1/-1)
• ui/src/ui/session-run-state.ts (added, +13/-0)
• ui/src/ui/views/chat.test.ts (modified, +45/-0)
• ui/src/ui/views/chat.ts (modified, +9/-4)
• ui/src/ui/views/sessions.test.ts (modified, +50/-1)
• ui/src/ui/views/sessions.ts (modified, +7/-3)