Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion crates/tui/src/commands/skills.rs
Original file line number Diff line number Diff line change
Expand Up @@ -441,7 +441,7 @@ fn sync_skills(app: &mut App) -> CommandResult {
}
SkillSyncOutcome::Denied { name, host } => {
failed += 1;
let _ = writeln!(out, " [x] {name} — network denied ({host})");
let _ = writeln!(out, " [] {name} — network denied ({host})");
}
SkillSyncOutcome::NeedsApproval { name, host } => {
failed += 1;
Expand Down
4 changes: 2 additions & 2 deletions crates/tui/src/cycle_manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -284,7 +284,7 @@ impl StructuredState {
let marker = match item.status {
crate::tools::todo::TodoStatus::Pending => "[ ]",
crate::tools::todo::TodoStatus::InProgress => "[~]",
crate::tools::todo::TodoStatus::Completed => "[x]",
crate::tools::todo::TodoStatus::Completed => "[]",
};
out.push_str(&format!("- {marker} {}\n", item.content));
}
Expand All @@ -299,7 +299,7 @@ impl StructuredState {
let marker = match item.status {
crate::tools::plan::StepStatus::Pending => "[ ]",
crate::tools::plan::StepStatus::InProgress => "[~]",
crate::tools::plan::StepStatus::Completed => "[x]",
crate::tools::plan::StepStatus::Completed => "[]",
};
out.push_str(&format!("- {marker} {}\n", item.step));
}
Expand Down
20 changes: 10 additions & 10 deletions crates/tui/src/tui/sidebar.rs
Original file line number Diff line number Diff line change
Expand Up @@ -410,7 +410,7 @@ fn push_work_checklist_lines(
let (prefix, color) = match item.status {
TodoStatus::Pending => ("[ ]", palette::TEXT_MUTED),
TodoStatus::InProgress => ("[~]", palette::STATUS_WARNING),
TodoStatus::Completed => ("[x]", palette::STATUS_SUCCESS),
TodoStatus::Completed => ("[]", palette::STATUS_SUCCESS),
};
let text = format!("{prefix} #{} {}", item.id, item.content);
lines.push(Line::from(Span::styled(
Expand Down Expand Up @@ -501,7 +501,7 @@ fn push_work_strategy_lines(
let (prefix, color) = match step.status {
StepStatus::Pending => ("[ ]", theme.plan_pending_color),
StepStatus::InProgress => ("[~]", theme.plan_in_progress_color),
StepStatus::Completed => ("[x]", theme.plan_completed_color),
StepStatus::Completed => ("[]", theme.plan_completed_color),
};
let mut text = format!("{prefix} {}", step.text);
if !step.elapsed.is_empty() {
Expand Down Expand Up @@ -1304,7 +1304,7 @@ fn first_nonempty_line(text: &str) -> &str {
fn tool_status_marker(status: ToolStatus) -> (&'static str, ratatui::style::Color) {
match status {
ToolStatus::Running => ("[~]", palette::STATUS_WARNING),
ToolStatus::Success => ("[x]", palette::STATUS_SUCCESS),
ToolStatus::Success => ("[]", palette::STATUS_SUCCESS),
ToolStatus::Failed => ("[!]", palette::STATUS_ERROR),
}
}
Expand Down Expand Up @@ -1593,7 +1593,7 @@ pub fn subagent_panel_lines(
fn agent_status_marker(status: &str) -> (&'static str, ratatui::style::Color) {
match status {
"running" => ("[~]", palette::STATUS_WARNING),
"done" => ("[x]", palette::STATUS_SUCCESS),
"done" => ("[]", palette::STATUS_SUCCESS),
"failed" => ("[!]", palette::STATUS_ERROR),
"canceled" | "interrupted" => ("[-]", palette::TEXT_MUTED),
_ => ("[ ]", palette::TEXT_MUTED),
Expand Down Expand Up @@ -2066,7 +2066,7 @@ mod tests {
"recent section missing: {text:?}"
);
assert!(
text.iter().any(|line| line.contains("[x] read_file")),
text.iter().any(|line| line.contains("[] read_file")),
"recent read_file row missing: {text:?}"
);
}
Expand Down Expand Up @@ -2095,7 +2095,7 @@ mod tests {
let text = lines_to_text(&task_panel_lines(&app, 64, 8));

assert!(
!text.iter().any(|line| line.contains("[x] read_file")),
!text.iter().any(|line| line.contains("[] read_file")),
"expired completed active row should leave the sidebar: {text:?}"
);
}
Expand Down Expand Up @@ -2133,7 +2133,7 @@ mod tests {
let text = lines_to_text(&task_panel_lines(&app, 64, 8));

assert!(
text.iter().any(|line| line.contains("[x] read_file")),
text.iter().any(|line| line.contains("[] read_file")),
"fresh completed active row should linger briefly: {text:?}"
);
}
Expand Down Expand Up @@ -2262,7 +2262,7 @@ mod tests {
.expect("failed grep row should stay visible");
let read_group_index = text
.iter()
.position(|line| line.contains("[x] read_file x3"))
.position(|line| line.contains("[] read_file x3"))
.expect("repeated read_file rows should collapse");

assert!(
Expand All @@ -2271,7 +2271,7 @@ mod tests {
);
assert_eq!(
text.iter()
.filter(|line| line.contains("[x] read_file"))
.filter(|line| line.contains("[] read_file"))
.count(),
1,
"read_file should render once after grouping: {text:?}"
Expand Down Expand Up @@ -2371,7 +2371,7 @@ mod tests {

assert!(
text.iter()
.any(|line| line.contains("[x] cargo check 1.2s")),
.any(|line| line.contains("[] cargo check 1.2s")),
"status marker and duration should stay in the row label: {text:?}"
);
assert!(
Expand Down
2 changes: 1 addition & 1 deletion crates/tui/src/tui/views/status_picker.rs
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ impl ModalView for StatusPickerView {
for (idx, item) in self.rows.iter().enumerate() {
let checked = *self.selected.get(idx).unwrap_or(&false);
let is_cursor = idx == self.cursor;
let mark = if checked { "[x]" } else { "[ ]" };
let mark = if checked { "[]" } else { "[ ]" };

let row_style = if is_cursor {
Style::default()
Expand Down