Skip to content

Conversation

aibrahim-oai
Copy link
Collaborator

Screen.Recording.2025-08-18.at.2.09.26.PM.mov

Building up on #2431 #2428

Copy link
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@easong-openai or @edward-bayes do you want to weigh in?

@@ -13,8 +16,6 @@ pub enum ReasoningEffort {
#[default]
Medium,
High,
/// Option to disable reasoning.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need to update config.md to reflect this change.

} => self.move_down(),
KeyEvent {
code: KeyCode::Esc, ..
} => self.accept(),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cancel?

}

fn render(&self, area: Rect, buf: &mut Buffer) {
let rows: Vec<GenericDisplayRow> = self
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add a title.

Copy link
Collaborator

@easong-openai easong-openai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree on adding a title, otherwise LGTM

@aibrahim-oai aibrahim-oai enabled auto-merge (squash) August 19, 2025 17:37
@aibrahim-oai aibrahim-oai merged commit e91c3d6 into main Aug 19, 2025
15 checks passed
@aibrahim-oai aibrahim-oai deleted the reasoning-effort-tui branch August 19, 2025 17:55
@github-actions github-actions bot locked and limited conversation to collaborators Aug 19, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants