Bump ruby/setup-ruby from 1.308.0 to 1.310.0

Bumps [ruby/setup-ruby](https://github.com/ruby/setup-ruby) from 1.308.0 to 1.310.0.
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb)
- [Commits](https://github.com/ruby/setup-ruby/compare/97ecb7b512899eb71ab1bf2310a624c6f1589ac6...afeafc3d1ab54a631816aba4c914a0081c12ff2f)

---
updated-dependencies:
- dependency-name: ruby/setup-ruby
  dependency-version: 1.310.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

.github/workflows/linux.yml

@@ -28,7 +28,7 @@ jobs:
         go-version: "1.23"
 
     - name: Setup Ruby
-      uses: ruby/setup-ruby@97ecb7b512899eb71ab1bf2310a624c6f1589ac6 # v1
+      uses: ruby/setup-ruby@afeafc3d1ab54a631816aba4c914a0081c12ff2f # v1
       with:
         ruby-version: 3.4.6
 

.github/workflows/macos.yml

@@ -25,7 +25,7 @@ jobs:
         go-version: "1.23"
 
     - name: Setup Ruby
-      uses: ruby/setup-ruby@97ecb7b512899eb71ab1bf2310a624c6f1589ac6 # v1
+      uses: ruby/setup-ruby@afeafc3d1ab54a631816aba4c914a0081c12ff2f # v1
       with:
         ruby-version: 3.0.0
 

Dockerfile

@@ -1,15 +1,5 @@
 FROM rubylang/ruby:3.4.1-noble
-RUN apt-get update && apt-get install -y git make golang zsh fish tmux
-
-# https://www.nushell.sh/book/installation.html
-RUN <<EOF
-    set -ex
-    apt-get install -y wget gnupg
-    wget -qO- https://apt.fury.io/nushell/gpg.key | gpg --dearmor -o /etc/apt/keyrings/fury-nushell.gpg
-    echo "deb [signed-by=/etc/apt/keyrings/fury-nushell.gpg] https://apt.fury.io/nushell/ /" | tee /etc/apt/sources.list.d/fury-nushell.list
-    apt-get update
-    apt-get install -y nushell
-EOF
+RUN apt-get update -y && apt install -y git make golang zsh fish tmux
 RUN gem install --no-document -v 5.22.3 minitest
 RUN echo '. /usr/share/bash-completion/completions/git' >> ~/.bashrc
 RUN echo '. ~/.bashrc' >> ~/.bash_profile

install

@@ -227,13 +227,13 @@ fi
 
 for s in $shells; do
   bin=$s
-  [[ $s == nushell ]] && bin=nu
+  [[ "$s" = nushell ]] && bin=nu
   if ! command -v "$bin" > /dev/null; then
     shells=${shells/$s/}
   fi
 done
 
-if [[ -z ${shells// /} ]]; then
+if [[ ${#shells} -lt 3 ]]; then
   echo "No shell configuration to be updated."
   exit 0
 fi
@@ -252,10 +252,10 @@ fi
 
 echo
 for shell in $shells; do
-  [[ $shell == nushell ]] && continue
   fzf_completion="source \"$fzf_base/shell/completion.${shell}\""
   fzf_key_bindings="source \"$fzf_base/shell/key-bindings.${shell}\""
   [[ $shell == fish ]] && continue
+  [[ $shell == nushell ]] && continue
   src=${prefix_expand}.${shell}
   echo -n "Generate $src ... "
 
@@ -442,7 +442,7 @@ if [[ $shells =~ fish ]]; then
   fi
 fi
 
-if [[ $shells =~ nushell ]]; then
+if [[ "$shells" =~ nushell ]]; then
   if [[ $key_bindings -eq 1 || $auto_completion -eq 1 ]]; then
     echo "Setting up Nushell integration ..."
     nushell_autoload_dir=$(nu -c '$nu.user-autoload-dirs | first')

shell/key-bindings.zsh

@@ -110,14 +110,8 @@ fzf-cd-widget() {
     zle redisplay
     return 0
   fi
-  # Use subshell expansion to get the absolute PWD of the target dir.
-  # This allows the recorded shell history to be reused even from a different
-  # working directory.
-  # If failed, fallback to the unexpanded path to surface the error to the user.
-  # NOTE: Don't use the `:a` modifier as it resolves symlinks like `pwd -P`.
-  dir=$(builtin cd >/dev/null -- "${dir}" && echo "${PWD}" || echo "${dir}")
   zle push-line # Clear buffer. Auto-restored on next prompt.
-  BUFFER="builtin cd -- ${(q)dir}"
+  BUFFER="builtin cd -- ${(q)dir:a}"
   zle accept-line
   local ret=$?
   unset dir # ensure this doesn't end up appearing in prompt expansion

src/options.go

@@ -10,6 +10,7 @@ import (
 	"strconv"
 	"strings"
 	"time"
+	"unicode"
 
 	"github.com/junegunn/fzf/src/algo"
 	"github.com/junegunn/fzf/src/tui"
@@ -1733,10 +1734,10 @@ Loop:
 	return masked
 }
 
-func parseSingleActionList(str string, putAllowed bool) ([]*action, error) {
+func parseSingleActionList(str string) ([]*action, error) {
 	// We prepend a colon to satisfy argActionRegexp and remove it later
 	masked := maskActionContents(":" + str)[1:]
-	return parseActionList(masked, str, []*action{}, putAllowed)
+	return parseActionList(masked, str, []*action{}, false)
 }
 
 func parseActionList(masked string, original string, prevActions []*action, putAllowed bool) ([]*action, error) {
@@ -2042,7 +2043,8 @@ func parseKeymap(keymap map[tui.Event][]*action, str string) error {
 				}
 				key = firstKey(keys)
 			}
-			keymap[key], err = parseActionList(pair[1], origPairStr[len(pair[0])+1:], keymap[key], key.Printable())
+			putAllowed := key.Type == tui.Rune && unicode.IsGraphic(key.Char)
+			keymap[key], err = parseActionList(pair[1], origPairStr[len(pair[0])+1:], keymap[key], putAllowed)
 			if err != nil {
 				return err
 			}

src/options_test.go

@@ -572,7 +572,7 @@ func TestValidateSign(t *testing.T) {
 }
 
 func TestParseSingleActionList(t *testing.T) {
-	actions, _ := parseSingleActionList("Execute@foo+bar,baz@+up+up+reload:down+down", false)
+	actions, _ := parseSingleActionList("Execute@foo+bar,baz@+up+up+reload:down+down")
 	if len(actions) != 4 {
 		t.Errorf("Invalid number of actions parsed:%d", len(actions))
 	}
@@ -588,7 +588,7 @@ func TestParseSingleActionList(t *testing.T) {
 }
 
 func TestParseSingleActionListError(t *testing.T) {
-	_, err := parseSingleActionList("change-query(foobar)baz", false)
+	_, err := parseSingleActionList("change-query(foobar)baz")
 	if err == nil {
 		t.Errorf("Failed to detect error")
 	}

src/result.go

@@ -198,10 +198,7 @@ func (result *Result) colorOffsets(matchOffsets []Offset, nthOffsets []Offset, t
 	start := 0
 	ansiToColorPair := func(ansi ansiOffset, base tui.ColorPair) tui.ColorPair {
 		if !theme.Colored {
-			// Ignore ANSI colors but keep the attributes. Retain the base
-			// colors (e.g. an overridden input-bg or list-bg) instead of
-			// resetting to the terminal default.
-			return tui.NewColorPair(base.Fg(), base.Bg(), ansi.color.attr).MergeAttr(base)
+			return tui.NewColorPair(-1, -1, ansi.color.attr).MergeAttr(base)
 		}
 		// fd --color always | fzf --ansi --delimiter / --nth -1 --color fg:dim:strip,nth:regular
 		if base.ShouldStripColors() {

src/server.go

@@ -240,7 +240,7 @@ Loop:
 	}
 	body = body[:contentLength]
 
-	actions, err := parseSingleActionList(strings.Trim(string(body), "\r\n"), false)
+	actions, err := parseSingleActionList(strings.Trim(string(body), "\r\n"))
 	if err != nil {
 		return bad(err.Error())
 	}

src/terminal.go

@@ -6973,8 +6973,7 @@ func (t *Terminal) Loop() error {
 				})
 			case actTransform, actBgTransform:
 				capture(false, func(body string) {
-					// Allow 'put' if the triggering key is a printable character
-					if actions, err := parseSingleActionList(strings.Trim(body, "\r\n"), event.Printable()); err == nil {
+					if actions, err := parseSingleActionList(strings.Trim(body, "\r\n")); err == nil {
 						// NOTE: We're not properly passing the return value here
 						doActions(actions)
 					}

src/tui/tui.go

@@ -4,7 +4,6 @@ import (
 	"strconv"
 	"strings"
 	"time"
-	"unicode"
 
 	"github.com/junegunn/fzf/src/util"
 	"github.com/rivo/uniseg"
@@ -253,12 +252,6 @@ func (e Event) Comparable() Event {
 	return Event{e.Type, e.Char, nil}
 }
 
-// Printable returns true if the event is a printable character that can be
-// inserted into the query (e.g. via the 'put' action).
-func (e Event) Printable() bool {
-	return e.Type == Rune && unicode.IsGraphic(e.Char)
-}
-
 func (e Event) KeyName() string {
 	if me := e.MouseEvent; me != nil {
 		return me.Name()
@@ -1002,56 +995,51 @@ func init() {
 	undefined := ColorAttr{colUndefined, AttrUndefined}
 
 	NoColorTheme = &ColorTheme{
-		Colored: false,
-		// Root colors. Everything else is left undefined so that overriding a
-		// root (e.g. --color bw,bg:blue) propagates to the derived colors,
-		// just like in the colored base themes.
-		Input:       defaultColor,
-		Fg:          defaultColor,
-		Bg:          defaultColor,
-		DarkBg:      defaultColor,
-		Prompt:      defaultColor,
-		Match:       defaultColor,
-		Spinner:     defaultColor,
-		Info:        defaultColor,
-		Pointer:     defaultColor,
-		Marker:      defaultColor,
-		Header:      defaultColor,
-		Footer:      defaultColor,
-		BorderLabel: defaultColor,
-		// Derived colors. Left undefined so they inherit from a root.
-		ListFg:           undefined,
-		ListBg:           undefined,
+		Colored:          false,
+		Input:            defaultColor,
+		Fg:               defaultColor,
+		Bg:               defaultColor,
+		ListFg:           defaultColor,
+		ListBg:           defaultColor,
 		AltBg:            undefined,
-		SelectedFg:       undefined,
-		SelectedBg:       undefined,
-		SelectedMatch:    undefined,
+		SelectedFg:       defaultColor,
+		SelectedBg:       defaultColor,
+		SelectedMatch:    defaultColor,
+		DarkBg:           defaultColor,
+		Prompt:           defaultColor,
+		Match:            defaultColor,
 		Current:          undefined,
 		CurrentMatch:     undefined,
+		Spinner:          defaultColor,
+		Info:             defaultColor,
+		Pointer:          defaultColor,
+		Marker:           defaultColor,
+		Header:           defaultColor,
 		Border:           undefined,
+		BorderLabel:      defaultColor,
 		Ghost:            undefined,
-		Disabled:         undefined,
-		PreviewFg:        undefined,
-		PreviewBg:        undefined,
+		Disabled:         defaultColor,
+		PreviewFg:        defaultColor,
+		PreviewBg:        defaultColor,
 		Gutter:           undefined,
 		AltGutter:        undefined,
-		PreviewBorder:    undefined,
-		PreviewScrollbar: undefined,
-		PreviewLabel:     undefined,
-		ListLabel:        undefined,
-		ListBorder:       undefined,
-		Separator:        undefined,
-		Scrollbar:        undefined,
-		InputBg:          undefined,
-		InputBorder:      undefined,
-		InputLabel:       undefined,
-		HeaderBg:         undefined,
-		HeaderBorder:     undefined,
-		HeaderLabel:      undefined,
-		FooterBg:         undefined,
-		FooterBorder:     undefined,
-		FooterLabel:      undefined,
-		GapLine:          undefined,
+		PreviewBorder:    defaultColor,
+		PreviewScrollbar: defaultColor,
+		PreviewLabel:     defaultColor,
+		ListLabel:        defaultColor,
+		ListBorder:       defaultColor,
+		Separator:        defaultColor,
+		Scrollbar:        defaultColor,
+		InputBg:          defaultColor,
+		InputBorder:      defaultColor,
+		InputLabel:       defaultColor,
+		HeaderBg:         defaultColor,
+		HeaderBorder:     defaultColor,
+		HeaderLabel:      defaultColor,
+		FooterBg:         defaultColor,
+		FooterBorder:     defaultColor,
+		FooterLabel:      defaultColor,
+		GapLine:          defaultColor,
 		Nth:              undefined,
 		Nomatch:          undefined,
 	}

test/lib/common.rb

@@ -136,7 +136,6 @@ class Tmux
       rescue Minitest::Assertion
         retries += 1
         raise if retries > 5
-
         retry
       end
       send_keys 'clear', :Enter
@@ -296,7 +295,7 @@ class Tmux
       if @shell == :nushell
         message = "Prepare[#{tries}]"
         send_keys 'C-u', 'C-l'
-        sleep(0.2)
+        sleep 0.2
         send_keys ' ', 'C-u', :Enter, message
         self.until { |lines| lines[-1] == message }
       else

test/test_core.rb

@@ -971,24 +971,6 @@ class TestCore < TestInteractive
     tmux.until { |lines| assert_includes lines[1], ' aabravo/aabravo' }
   end
 
-  def test_transform_put
-    tmux.send_keys %(seq 1000 | #{FZF} --bind 'a:transform:echo put'), :Enter
-    tmux.until { |lines| assert_equal 1000, lines.match_count }
-    tmux.send_keys :a
-    tmux.until { |lines| assert_equal '> a', lines.last }
-    tmux.send_keys :b
-    tmux.until { |lines| assert_equal '> ab', lines.last }
-  end
-
-  # The async callback runs in a later iteration, but 'put' must still insert
-  # the key that triggered the bg-transform (snapshot of the scheduling event).
-  def test_bg_transform_put
-    tmux.send_keys %(seq 1000 | #{FZF} --bind 'a:bg-transform:sleep 0.5; echo put'), :Enter
-    tmux.until { |lines| assert_equal 1000, lines.match_count }
-    tmux.send_keys 'ab'
-    tmux.until { |lines| assert_equal '> ba', lines.last }
-  end
-
   def test_accept_non_empty
     tmux.send_keys %(seq 1000 | #{fzf('--print-query --bind enter:accept-non-empty')}), :Enter
     tmux.until { |lines| assert_equal 1000, lines.match_count }

test/test_shell_integration.rb

@@ -100,47 +100,6 @@ module TestShell
     tmux.until { |lines| assert_equal '/tmp', lines[-1] }
   end
 
-  def test_alt_c_symlink
-    base = '/tmp/fzf-test-alt-c-symlink'
-    FileUtils.rm_rf(base)
-    FileUtils.mkdir_p("#{base}/real/subdir")
-    FileUtils.ln_s("#{base}/real", "#{base}/link")
-
-    tmux.prepare
-    tmux.send_keys "cd #{base}/link", :Enter
-    tmux.prepare
-    tmux.send_keys :Escape, :c
-    tmux.until { |lines| assert_operator lines.match_count, :>, 0 }
-    tmux.send_keys 'subdir'
-    tmux.until { |lines| assert_equal 1, lines.match_count }
-    tmux.send_keys :Enter
-    tmux.prepare
-    tmux.send_keys :pwd, :Enter
-    tmux.until { |lines| assert_equal "#{base}/link/subdir", lines[-1] }
-  ensure
-    FileUtils.rm_rf(base)
-  end
-
-  def test_alt_c_absolute_cmd
-    base = '/tmp/fzf-test-alt-c-absolute'
-    FileUtils.rm_rf(base)
-    FileUtils.mkdir_p(base)
-
-    set_var('FZF_ALT_C_COMMAND', "echo #{base}")
-
-    tmux.prepare
-    tmux.send_keys 'cd /tmp', :Enter
-    tmux.prepare
-    tmux.send_keys :Escape, :c
-    tmux.until { |lines| assert_equal 1, lines.match_count }
-    tmux.send_keys :Enter
-    tmux.prepare
-    tmux.send_keys :pwd, :Enter
-    tmux.until { |lines| assert_equal base, lines[-1] }
-  ensure
-    FileUtils.rm_rf(base)
-  end
-
   def test_ctrl_r
     tmux.prepare
     tmux.send_keys 'echo 1st', :Enter