From 9abf2c8c9ca625f30ff0775316f51ad798a922f3 Mon Sep 17 00:00:00 2001 From: Junegunn Choi Date: Wed, 12 Feb 2025 20:50:01 +0900 Subject: [PATCH] Allow suffix match on --nth with custom --delimiter When --nth is used with a custom --delimiter, the last delimiter was included in the search scope, forcing you to write the delimiter in a suffix-match query. This commit removes the last delimiter from the search scope. # No need to write 'bar,$' echo foo,bar,baz | fzf --delimiter , --nth 2 --filter 'bar$' This can be seen as a breaking change, but I'm gonna say it's a bug fix. Fix #3983 --- src/pattern.go | 9 +++++++-- src/tokenizer.go | 5 +++++ test/test_filter.rb | 6 ++++++ 3 files changed, 18 insertions(+), 2 deletions(-) diff --git a/src/pattern.go b/src/pattern.go index 93640cb6..8e6966c3 100644 --- a/src/pattern.go +++ b/src/pattern.go @@ -432,8 +432,13 @@ func (p *Pattern) transformInput(item *Item) []Token { tokens := Tokenize(item.text.ToString(), p.delimiter) ret := Transform(tokens, p.nth) - // TODO: We could apply StripLastDelimiter to exclude the last delimiter from - // the search allowing suffix match with a string or a regex delimiter. + // Strip the last delimiter to allow suffix match + if len(ret) > 0 && !p.delimiter.IsAwk() { + chars := ret[len(ret)-1].text + stripped := StripLastDelimiter(chars.ToString(), p.delimiter) + newChars := util.ToChars(stringBytes(stripped)) + ret[len(ret)-1].text = &newChars + } item.transformed = &transformed{p.revision, ret} return ret } diff --git a/src/tokenizer.go b/src/tokenizer.go index aaddd17d..573a3576 100644 --- a/src/tokenizer.go +++ b/src/tokenizer.go @@ -78,6 +78,11 @@ type Delimiter struct { str *string } +// IsAwk returns true if the delimiter is an AWK-style delimiter +func (d Delimiter) IsAwk() bool { + return d.regex == nil && d.str == nil +} + // String returns the string representation of a Delimiter. func (d Delimiter) String() string { return fmt.Sprintf("Delimiter{regex: %v, str: &%q}", d.regex, *d.str) diff --git a/test/test_filter.rb b/test/test_filter.rb index 718c6e57..3e604412 100644 --- a/test/test_filter.rb +++ b/test/test_filter.rb @@ -52,6 +52,12 @@ class TestFilter < TestBase `find . -print0 | #{FZF} --read0 -e -f "^#{lines.last}$"`.chomp end + def test_nth_suffix_match + assert_equal \ + 'foo,bar,baz', + `echo foo,bar,baz | #{FZF} -d, -f'bar$' -n2`.chomp + end + def test_with_nth_basic writelines(['hello world ', 'byebye']) assert_equal \