Public/nushell
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Respect case sensitivity when sorting by fuzzy score

Browse Source
This commit is contained in:
ysthakur 2024-07-06 11:58:25 -04:00
parent 224b16ce31
commit e22d8fa6c7
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
crates/nu-cli/src/completions/completion_common.rs
View File

@ -280,7 +280,7 @@ pub fn sort_suggestions(
} }
/// # Arguments /// # Arguments
/// * `prefix` - What the user's typed, for sorting by Levenshtein distance /// * `prefix` - What the user's typed, for sorting by fuzzy matcher score
pub fn sort_completions<T>( pub fn sort_completions<T>(
prefix: &str, prefix: &str,
mut items: Vec<T>, mut items: Vec<T>,
@ -289,8 +289,13 @@ pub fn sort_completions<T>(
) -> Vec<T> { ) -> Vec<T> {
// Sort items // Sort items
if options.sort == CompletionSort::Default && options.match_algorithm == MatchAlgorithm::Fuzzy { if options.sort == CompletionSort::Default && options.match_algorithm == MatchAlgorithm::Fuzzy {
let mut matcher = SkimMatcherV2::default();
if options.case_sensitive {
matcher = matcher.respect_case();
} else {
matcher = matcher.ignore_case();
};
items.sort_by(|a, b| { items.sort_by(|a, b| {
let matcher = SkimMatcherV2::default();
let a_str = get_value(a); let a_str = get_value(a);
let b_str = get_value(b); let b_str = get_value(b);
let a_score = matcher.fuzzy_match(a_str, prefix).unwrap_or_default(); let a_score = matcher.fuzzy_match(a_str, prefix).unwrap_or_default();