perf(#3253): short-circuit open_on_directory au

Author: alex-courtis

lua/nvim-tree.lua

@@ -45,16 +45,18 @@ local function setup_autocommands()
     })
   end
 
-  -- TODO this fires on setup
-  -- if config.g.hijack_directories.enable and (config.g.disable_netrw or config.g.hijack_netrw) then
-  --   vim.api.nvim_create_autocmd({ "BufEnter", "BufNewFile" }, {
-  --     group = augroup_id,
-  --     callback = function()
-  --       require("nvim-tree.actions.tree.open").open_on_directory()
-  --     end,
-  --     nested = true
-  --   })
-  -- end
+  if config.g.hijack_directories.enable and (config.g.disable_netrw or config.g.hijack_netrw) then
+    vim.api.nvim_create_autocmd({ "BufEnter", "BufNewFile" }, {
+      group = augroup_id,
+      nested = true,
+      callback = function(data)
+        local bufname = vim.api.nvim_buf_get_name(data.buf)
+        if vim.fn.isdirectory(bufname) == 1 then
+          require("nvim-tree.actions.tree.open").open_on_directory(bufname)
+        end
+      end,
+    })
+  end
 
   if config.g.view.centralize_selection then
     vim.api.nvim_create_autocmd("BufEnter", {

lua/nvim-tree/actions/tree/open.lua

@@ -51,26 +51,21 @@ function M.fn(opts)
   end
 end
 
-function M.open_on_directory()
+---@param dirname string absolute directory path
+function M.open_on_directory(dirname)
   local should_proceed = config.g.hijack_directories.auto_open or view.is_visible()
   if not should_proceed then
     return
   end
 
-  local buf = vim.api.nvim_get_current_buf()
-  local bufname = vim.api.nvim_buf_get_name(buf)
-  if vim.fn.isdirectory(bufname) ~= 1 then
-    return
-  end
-
   -- instantiate an explorer if there is not one
   if not core.get_explorer() then
-    core.init(bufname)
+    core.init(dirname)
   end
 
   local explorer = core.get_explorer()
   if explorer then
-    explorer:force_dirchange(bufname, true, false)
+    explorer:force_dirchange(dirname, true, false)
   end
 end