feat: Added a TUI component to initially implement interaction #31
Conversation
… experience - Updated `packages/ui/src/index.ts` to `packages/ui/src/index.tsx` for React compatibility. - Implemented interactive and once-only modes in the CLI for better user engagement. - Introduced session management and logging improvements with JSONL format. - Added new components for rendering user and assistant messages, status updates, and command handling. - Enhanced command parsing to support help, exit, clear, and tools listing. - Created a structured UI with components for header, input prompt, main content, and token usage display. - Improved session closing logic to prevent multiple closures. - Added type definitions and utility functions for better type safety and code clarity.
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
minorcell
changed the title
![linearb[bot]](https://avatars.githubusercontent.com/in/1658443?s=60&v=4){kind=small}
There was a problem hiding this comment.
✨ PR Review
The PR implements a comprehensive UI overhaul with TUI interface and language localization from Chinese to English. However, there are significant breaking changes in tool call parsing and history storage that could impact existing functionality.
2 issues detected:
🧹 Maintainability - Removing support for existing API format without deprecation could break existing integrations
Details: The removal of legacy action format support (nested "action" object) is a breaking change that could cause existing prompts or models using the old format to fail silently.
File:packages/core/src/utils/utils.ts🐞 Bug - Changing the fundamental storage format of conversation history without considering backward compatibility
Details: The assistant message history storage format has changed significantly - now storing JSON for tool calls instead of raw assistant text. This could break history replay, token counting accuracy, and any code expecting the original assistant output format.
File:packages/core/src/runtime/session.ts
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how
✨ PR Description
Purpose: Internationalize memo-cli codebase by replacing Chinese text with English across system prompts, error messages, and documentation to improve accessibility for global developers.
Main changes:
Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how