refactor: split open_cmd in float_cmd and float_term

lua/lazy/core/config.lua

@@ -72,21 +72,15 @@ M.defaults = {
 
       -- open lazygit log
       ["<localleader>l"] = function(plugin)
-        require("lazy.util").open_cmd({ "lazygit", "log" }, {
+        require("lazy.util").float_term({ "lazygit", "log" }, {
           cwd = plugin.dir,
-          terminal = true,
-          close_on_exit = true,
-          enter = true,
         })
       end,
 
       -- open a terminal for the plugin dir
       ["<localleader>t"] = function(plugin)
-        require("lazy.util").open_cmd({ vim.go.shell }, {
+        require("lazy.util").float_term(nil, {
           cwd = plugin.dir,
-          terminal = true,
-          close_on_exit = true,
-          enter = true,
         })
       end,
     },

lua/lazy/util.lua

@@ -93,27 +93,27 @@ function M.throttle(ms, fn)
   end
 end
 
----@class LazyCmdOpts
+---@class LazyCmdOptions: LazyFloatOptions
 ---@field cwd? string
 ---@field env? table<string,string>
 ---@field float? LazyFloatOptions
 
----@param cmd string[]
+-- Opens a floating terminal (interactive by default)
----@param opts? {cwd:string, filetype:string, terminal?:boolean, close_on_exit?:boolean, enter?:boolean, float?:LazyFloatOptions}
+---@param cmd? string[]|string
-function M.open_cmd(cmd, opts)
+---@param opts? LazyCmdOptions|{interactive?:boolean}
+function M.float_term(cmd, opts)
+  cmd = cmd or {}
+  if type(cmd) == "string" then
+    cmd = { cmd }
+  end
+  if #cmd == 0 then
+    cmd = { vim.env.SHELL or vim.o.shell }
+  end
   opts = opts or {}
-  local float = M.float(opts.float)
+  local float = M.float(opts)
-
+  vim.fn.termopen(cmd, vim.tbl_isempty(opts) and vim.empty_dict() or opts)
-  if opts.filetype then
+  if opts.interactive ~= false then
-    vim.bo[float.buf].filetype = opts.filetype
-  end
-  if opts.terminal then
-    opts.terminal = nil
-    vim.fn.termopen(cmd, opts)
-    if opts.enter then
     vim.cmd.startinsert()
-    end
-    if opts.close_on_exit then
     vim.api.nvim_create_autocmd("TermClose", {
       once = true,
       buffer = float.buf,
@@ -123,12 +123,23 @@ function M.open_cmd(cmd, opts)
       end,
     })
   end
-  else
+  return float
+end
+
+--- Runs the command and shows it in a floating window
+---@param cmd string[]
+---@param opts? LazyCmdOptions|{filetype?:string}
+function M.float_cmd(cmd, opts)
+  opts = opts or {}
+  local float = M.float(opts)
+  if opts.filetype then
+    vim.bo[float.buf].filetype = opts.filetype
+  end
   local Process = require("lazy.manage.process")
   local lines = Process.exec(cmd, { cwd = opts.cwd })
   vim.api.nvim_buf_set_lines(float.buf, 0, -1, false, lines)
   vim.bo[float.buf].modifiable = false
-  end
+  return float
 end
 
 ---@return string?

lua/lazy/view/diff.lua

@@ -43,7 +43,7 @@ M.handlers = {
       cmd[#cmd + 1] = diff.from
       cmd[#cmd + 1] = diff.to
     end
-    Util.open_cmd(cmd, { cwd = plugin.dir, filetype = "git" })
+    Util.float_cmd(cmd, { cwd = plugin.dir, filetype = "git" })
   end,
 
   ---@type LazyDiffFun
@@ -57,7 +57,7 @@ M.handlers = {
       cmd[#cmd + 1] = diff.from
       cmd[#cmd + 1] = diff.to
     end
-    Util.open_cmd(cmd, { cwd = plugin.dir, terminal = true, env = { PAGER = "cat" } })
+    Util.float_term(cmd, { cwd = plugin.dir, interactive = false, env = { PAGER = "cat" } })
   end,
 }