Merge pull request #159 from mrjones2014/mrj/158/non-zero-exit-code

fix(mux): Change `on_exit` to run on `VimLeavePre` instead of `VimLeave`

Also changes to use `jobstart` with `detach = true` for tmux

Author: mrjones2014

lua/smart-splits/mux/tmux.lua

@@ -20,7 +20,6 @@ end
 
 ---@param args (string|number)[]
 ---@param as_list boolean|nil
----@return nil
 local function tmux_exec(args, as_list)
   local socket = get_socket_path()
   if not socket then
@@ -165,10 +164,15 @@ function M.on_exit()
     log.warn('tmux init: could not detect pane ID!')
     return
   end
-  tmux_exec({ 'set-option', '-pt', pane_id, '@pane-is-vim', 0 })
-  if vim.v.shell_error ~= 0 then
-    log.warn('tmux init: failed to detect pane_id')
+  local socket = get_socket_path()
+  if not socket then
+    log.warn('on_exit: Could not find tmux socket')
+    return
   end
+  local args = { 'set-option', '-pt', pane_id, '@pane-is-vim', 0 }
+  local cmd = vim.list_extend({ 'tmux', '-S', socket }, args, 1, #args)
+
+  vim.fn.jobstart(cmd, { detach = true })
 end
 
 return M

lua/smart-splits/mux/utils.lua

@@ -23,7 +23,7 @@ end
 ---Initialization for mux capabilities.
 ---If selected mux has an `on_init` or `on_exit`,
 ---call `on_init` and set up autocmds to call `on_init` on `VimResume`
----and `on_exit` on `VimSuspend` and `VimLeave`.
+---and `on_exit` on `VimSuspend` and `VimLeavePre`.
 function M.startup()
   local mux = require('smart-splits.mux').get()
   if not mux then
@@ -38,7 +38,7 @@ function M.startup()
     })
   end
   if mux.on_exit then
-    vim.api.nvim_create_autocmd({ 'VimSuspend', 'VimLeave' }, {
+    vim.api.nvim_create_autocmd({ 'VimSuspend', 'VimLeavePre' }, {
       callback = function()
         mux.on_exit()
       end,