Skip to content

fix(oclite): fix TUI - render messages, clear screen, command feedback (#148)#149

Merged
randomm merged 1 commit intodevfrom
fix/issue-148-tui-broken
Feb 5, 2026
Merged

fix(oclite): fix TUI - render messages, clear screen, command feedback (#148)#149
randomm merged 1 commit intodevfrom
fix/issue-148-tui-broken

Conversation

@randomm
Copy link
Owner

@randomm randomm commented Feb 5, 2026

Summary

Fixes critical TUI issues where messages disappeared and commands appeared broken.

Changes

  • MessageList rendering: App.tsx now imports and renders MessageList component - messages no longer disappear after MESSAGE_COMPLETE
  • Idle clearing fix: Removed CLEAR_STREAMING from session.status idle handler to prevent race condition
  • /clear command: Now clears entire terminal with ANSI escape codes (cross-platform)
  • Command feedback: All handlers now show success/error messages to user

Root Causes Fixed

  1. MessageList component existed but was never rendered
  2. session.status idle event could arrive before message.updated, clearing content
  3. /clear only cleared React state, not terminal
  4. Commands had silent failures with no user feedback

Testing

  • All 1307 tests passing
  • TypeScript typecheck passes
  • All 4 fixes approved by adversarial review

Fixes #148

#148)

- Add MessageList component to App.tsx to render completed messages
- Fix session.status idle handler to not clear streaming content prematurely
- Fix /clear command to clear entire terminal with ANSI codes
- Add user feedback to all command handlers (model, submodel, agent, session)
- Update tests for new behavior
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@randomm randomm merged commit 7721c73 into dev Feb 5, 2026
2 of 3 checks passed
@randomm randomm deleted the fix/issue-148-tui-broken branch February 5, 2026 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant