Repository navigation
Conversation
Each tool call was a bare line with an absolute path and no result, the home screen did not fit a 24-row terminal, and the model selector showed one provider at a time. Tool calls now render as one row each with a result from the new ToolFinished event, grouped without blank lines, with paths relative to the workspace. The home screen shows a 5-row ghost, the model, the workspace, and start commands. The status line shows a state word, the model, and the workspace fitted to the width. The model selector lists every provider with its state next to the models.
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
This branch has not been deployed
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.
This PR completes the TUI redesign that #47 started. It changes three screens.
Tool calls. Each call used to be a bare
name(args)line with an absolute path, a blank line under it, and no result. Now each call is one row: a state mark, the tool, its arguments relative to the workspace, and a short result on the right.ToolFinished { ok, summary }event after each call.tool_result_summarybuilds the result from the tool output, such as12 entries,3 matches, orexit 1. A command with a non-zero exit code shows as failed.◌ runninguntil the call ends, then updates in place. A row still running when the turn ends showsstopped.UNSANDBOXED shell command in .:prefix. The approval panel already shows that warning.Home screen. The 11-row ghost and the 9-row title did not fit in a 24-row terminal. The new screen has a 5-row ghost, the version, the model, the workspace, and 4 start commands. The command descriptions come from the same list as
/help.Status line. It now starts with a state:
ready, the running tool and its time, orwaiting for you. Then come the model and the workspace. The key hints keep their full width. A long workspace path is shortened from the start, so its end stays visible.Model selector. It used to show one provider at a time. It now lists all providers in a column next to the models. Each provider has a state from the last model check: green
ready, rederror, or not checked yet. The keys do not change.No dependencies were added. The command docs describe the new status line, tool rows, and provider states.
Validation:
cargo fmt -- --checkcargo clippy --all-targets -- -D warningscargo test: 215 tests passed, with new tests for result summaries, tool row state and grouping, relative paths, one-line fitting, the home screen, and path shortening/model, the approval panel, a denied command, and an approved command. The tools ran on a sample workspace.bun run testandbun run build, for the docs changeCloud providers were not called. The selector does not mark the current model yet.