rg --path-separator option not working

[doongjohn]

my config:

require 'grug-far'.setup {
  engines = {
    ripgrep = {
      extraArgs =
          '--no-heading -.ni --path-separator=/'
          .. ' -g !.git'
          .. ' -g !.github'
          .. ' -g !*cache'
          .. ' -g !obj'
          .. ' -g !.objs'
          .. ' -g !.deps'
          .. ' -g !bin'
          .. ' -g !out'
          .. ' -g !build'
          .. ' -g !target'
          .. ' -g !vendor'
          .. ' -g !dist'
          .. ' -g !node_modules'
          .. ' -g !.svelte-kit'
          .. ' -g !zig-out'
          .. ' -g !.godot',
    },
  },
  windowCreationCommand = 'vert topleft split',
  wrap = false,
}

I'm on windows but I want to set path separator to / because of this issue: nvim-telescope/telescope.nvim#2446

However, it still uses \ even if I set --path-separator to /.

[MagicDuck]

hmm, I think it's ripgrep --json option itself that does not respect --path-separator (I am on mac where default is forward slash).

Are you seeing an actual bug when triggering the Open action? I do wrap it with fnameescape() here:

grug-far.nvim/lua/grug-far/actions/openLocation.lua

Line 80 in 190c03d

vim.fn.win_execute(targetWin, 'e! ' .. vim.fn.fnameescape(location.filename), true)

[doongjohn]

If I open a file src\routes\(csr)\+layout.ts with the open action (,o) the file is empty.

If I do <c-g> in that empty file, the path is incorrect:

[MagicDuck]

hi @doongjohn, I have incorporated logic from neo-tree.nvim when escaping the file paths for opening.
They seem to have gone through a long history of fixes in this area and apparently replacing \ with / is not always ideal since it can confuse other plugins that rely on the right separator for the platform. Although doing that is always an option if it ends up not working.

I can't easily test on windows myself. Can you grab the latest version and check that opening is working fine now?

[doongjohn]

It's working really well! Thank you for the quick fix!

[MagicDuck]

Great! Thanks for checking!