asp/keep
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor: move terminal width calculation outside the loop

Browse Source 
Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) <aider@aider.chat>
This commit is contained in:
Andrew Phillips
2025-08-25 22:19:20 -03:00
parent 0a597d6263
commit 689f377865
1 changed files with 11 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
src/modes/list.rs
View File

@@ -60,6 +60,17 @@ pub fn mode_list(
return show_list_structured(items_with_meta, data_path, settings, output_format);
}
// Get terminal width once, default to 80 if not available
let columns_env = env::var("COLUMNS").ok();
debug!("COLUMNS environment variable: {:?}", columns_env);
let term_width = columns_env
.as_ref()
.and_then(|w| w.parse::<usize>().ok())
.unwrap_or(80);
debug!("Terminal width: {}", term_width);
let mut table = Table::new();
table.set_format(*prettytable::format::consts::FORMAT_CLEAN);
@@ -86,16 +97,6 @@ pub fn mode_list(
.unwrap_or_else(|_| panic!("Unknown column {:?}", column.name));
let mut meta_name: Option<&str> = None;
// Get terminal width, default to 80 if not available
let columns_env = env::var("COLUMNS").ok();
debug!("COLUMNS environment variable: {:?}", columns_env);
let term_width = columns_env
.as_ref()
.and_then(|w| w.parse::<usize>().ok())
.unwrap_or(80);
debug!("Terminal width: {}", term_width);
// Parse max_len, handling both numbers and percentages
let column_width = if let Some(max_len_str) = &column.max_len {