fix: nil-check opts.cmd #2188

Author: phanen

lua/fzf-lua/actions.lua

@@ -1031,7 +1031,7 @@ M.toggle_flag = function(_, opts)
   local o = vim.tbl_deep_extend("keep", {
     -- grep|live_grep sets `opts._cmd` to the original
     -- command without the search argument
-    cmd = utils.toggle_cmd_flag(opts._cmd or opts.cmd, opts.toggle_flag),
+    cmd = utils.toggle_cmd_flag(assert(opts._cmd or opts.cmd), assert(opts.toggle_flag)),
     resume = true
   }, opts.__call_opts)
   opts.__call_fn(o)

lua/fzf-lua/make_entry.lua

@@ -245,7 +245,7 @@ M.preprocess = function(opts)
       end)
   end
 
-  if utils.__IS_WINDOWS and opts.cmd:match("!") then
+  if utils.__IS_WINDOWS and opts.cmd and opts.cmd:match("!") then
     -- https://ss64.com/nt/syntax-esc.html
     -- This changes slightly if you are running with DelayedExpansion of variables:
     -- if any part of the command line includes an '!' then CMD will escape a second

lua/fzf-lua/providers/files.lua

@@ -8,6 +8,8 @@ local make_entry = require "fzf-lua.make_entry"
 
 local M = {}
 
+---@param opts table
+---@return string
 local get_files_cmd = function(opts)
   if opts.raw_cmd and #opts.raw_cmd > 0 then
     return opts.raw_cmd

lua/fzf-lua/utils.lua

@@ -1442,12 +1442,12 @@ function M.termopen(cmd, opts)
   end
 end
 
+---@param cmd string
+---@param flag string
+---@param enabled boolean?
+---@param append boolean?
+---@return string
 function M.toggle_cmd_flag(cmd, flag, enabled, append)
-  if not flag then
-    M.err("'toggle_flag' not set")
-    return
-  end
-
   -- flag must be preceded by whitespace
   if not flag:match("^%s") then flag = " " .. flag end