fix(tui): drop the redundant plan-step kind label from to-do rows
Every row in the to-do strip is a plan step, so suffixing each row with "plan step" said nothing while eating horizontal budget on narrow terminals (user report, 2026-07-23 session). Ready steps now carry no detail text at all, and non-ready steps show just their state (running, completed, blocked, ...). Non-step work-graph nodes keep the state · kind detail pair, and the inspector's "Current" section still names the node kind, where mixed node kinds actually appear. Verified: cargo test -p codewhale-tui --bin codewhale-tui 'tui::' — 2698 passed, 0 failed (work_surface 56/56).
This commit is contained in:
@@ -1320,8 +1320,16 @@ fn graph_node_row(snapshot: &WorkGraphSnapshot, node: &WorkNode) -> WorkRow {
|
||||
};
|
||||
let state = state_label(node);
|
||||
let kind = kind_label(node.kind);
|
||||
let detail = if node.state == NodeState::Ready && node.kind == NodeKind::PlanStep {
|
||||
kind.to_string()
|
||||
// To-do rows are self-evidently plan steps — the strip's checkbox marks
|
||||
// already say so, so the "plan step" kind label is pure noise there. A
|
||||
// ready step shows no detail at all; other states show just the state.
|
||||
// Non-step nodes keep the state · kind pair.
|
||||
let detail = if node.kind == NodeKind::PlanStep {
|
||||
if node.state == NodeState::Ready {
|
||||
String::new()
|
||||
} else {
|
||||
state.to_string()
|
||||
}
|
||||
} else {
|
||||
format!("{state} · {kind}")
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user