[codex] TUI: simplify stale response notice#77120
Conversation
|
Codex review: needs maintainer review before merge. Summary Reproducibility: not applicable. this is a cleanup PR for user-facing copy, not a bug report. Source inspection confirms current main still emits the old internal watchdog notice on the timeout path. Next step before merge Security Review detailsBest possible solution: Land a reviewed version of this copy-only TUI change after draft/mergeability is resolved and the focused TUI event-handler test passes. Do we have a high-confidence way to reproduce the issue? Not applicable: this is a cleanup PR for user-facing copy, not a bug report. Source inspection confirms current main still emits the old internal watchdog notice on the timeout path. Is this the best way to solve the issue? Yes for the stated goal: replacing the string at the existing watchdog system-notice point while preserving timeout and reconnect behavior is the narrowest maintainable solution. Maintainers should still approve the final wording before landing. What I checked:
Likely related people:
Remaining risk / open question:
Codex review notes: model gpt-5.5, reasoning high; reviewed against e2f4aa461700. |
6d63b89 to
dd62232
Compare
#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
openclaw#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
openclaw#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
openclaw#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
openclaw#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
openclaw#77120 added user-facing TUI copy that replaces the stale-response watchdog notice with plain language, but the entry landed without contributor attribution. Add the merging PR ref and credit the human contributor @davemorin per CLAUDE.md changelog-attribution rules.
Summary
Why
The previous message exposed implementation details like "streaming watchdog", "backend", "run", and "resync". That made a recoverable stale-response state sound like an internal failure report. The new copy tells users only what matters: the response is taking longer than expected, and they can send another message to continue.
Testing
pnpm test src/tui/tui-event-handlers.test.ts