fiddlerwoaroof/improvise
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

refactor: execute_command owns its own mode transitions

Browse Source 
Removed the post-execution mode reset from the caller. execute_command
now sets mode = Normal at the top as the default; commands that open
a new mode (ImportWizard, Quit) simply override it. The caller no
longer needs a special-case exclusion list.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Ed L
2026-03-24 12:09:10 -07:00
parent f4978a9fd4
commit acc890764b
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/ui/app.rs
View File

@ -358,9 +358,6 @@ impl App {
buffer.clone() buffer.clone()
} else { return Ok(()); }; } else { return Ok(()); };
self.execute_command(&buf)?; self.execute_command(&buf)?;
if !matches!(self.mode, AppMode::Quit | AppMode::ImportWizard) {
self.mode = AppMode::Normal;
}
} }
KeyCode::Char(c) => { KeyCode::Char(c) => {
if let AppMode::CommandMode { buffer } = &mut self.mode { if let AppMode::CommandMode { buffer } = &mut self.mode {
@ -387,6 +384,9 @@ impl App {
.map(|(c, r)| (c, r.trim())) .map(|(c, r)| (c, r.trim()))
.unwrap_or((raw, "")); .unwrap_or((raw, ""));
// Default: return to Normal. Commands that open a new mode override this.
self.mode = AppMode::Normal;
match cmd_name { match cmd_name {
"q" | "quit" => { "q" | "quit" => {
if self.dirty { if self.dirty {