Merged
Conversation
lostmygithubaccount
force-pushed
the
cody/ws-toolsv1otto
branch
from
d1fb5d3 to
674b990
Compare
Full-screen ratatui + crossterm terminal interface for multi-turn conversations with Otto. Includes Vi/Emacs keybindings, streaming with spinner, input history, markdown rendering, slash commands, clipboard support, and scrollable chat with scrollbar. New ascend-tools-tui crate + `otto tui` CLI subcommand. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
lostmygithubaccount
force-pushed
the
cody/ws-toolsv1tui
branch
from
89c80db to
c36f689
Compare
There was a problem hiding this comment.
Pull request overview
Adds a new ascend-tools-tui crate that provides an interactive full-screen terminal UI for Otto chat, and wires it into the ascend-tools otto tui CLI subcommand.
Changes:
- Introduces
ascend-tools-tui(ratatui + crossterm) with streaming chat, history, markdown rendering, slash commands, and clipboard support. - Adds
otto tuisubcommand toascend-tools-cliand documents it in CLI docs. - Updates workspace membership and dependencies to include the new crate.
Reviewed changes
Copilot reviewed 8 out of 9 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| src/ascend_tools/ascend-tools-tui/src/lib.rs | Implements the TUI application loop, input handling (Vi/Emacs), streaming display, history, and markdown rendering. |
| src/ascend_tools/ascend-tools-tui/README.md | Documents TUI features, usage, and slash commands. |
| src/ascend_tools/ascend-tools-tui/Cargo.toml | Declares the new crate and its UI/clipboard dependencies. |
| src/ascend_tools/ascend-tools-cli/src/skill-cli.md | Adds otto tui to the embedded CLI skill documentation. |
| src/ascend_tools/ascend-tools-cli/src/otto.rs | Adds the Tui subcommand and calls into ascend_tools_tui::run_tui. |
| src/ascend_tools/ascend-tools-cli/Cargo.toml | Adds ascend-tools-tui dependency. |
| docs/cli.md | Documents the new interactive chat command. |
| Cargo.toml | Adds ascend-tools-tui to the workspace members. |
| Cargo.lock | Locks new dependencies introduced by the TUI crate. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Set ascend-tools-tui version to 0.5.0 (was incorrectly 1.0.0) - Add experimental warning banner to splash screen - Fix byte/char bug in vi paste (p/P) that could panic on non-ASCII text - Remove process::exit(0) from library crate; use cancel flag for clean shutdown - Replace .lock().unwrap() with poison-tolerant .unwrap_or_else() - Fix O(n*m) handle_paste with Vec::splice - Document scroll field semantics Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- check-js: run npm install before build/test - Update experimental banner text Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
Author
|
will need follow up work for real use, but generally well-contained; could gate behind a feature and not ship, but I think it's fine to release given how early |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacks on: PR 3 (Otto chat)
Summary
ascend-tools-tuicrate:ratatui+crosstermfull-screen chat interface~/.ascend-tools/history), markdown rendering, slash commands, clipboardstop_threadstd::thread::scopeto borrow&AscendClientwithoutArcotto tuisubcommandPR 4 of 6 for v1.0.
🤖 Generated with Claude Code