perf(#3257): remove setup for renderer, log, appearance (#3307)

* perf(#3257): remove devicons setup

* perf(#3257): remove padding setup

* perf(#3257): remove appearance and log setup

* perf(#3257): remove appearance setup

* perf(#3257): remove rename-file setup

* perf(#3257): remove devicons setup

* perf(#3257): remove appearance setup

* perf(#3257): inline require legacy notify

* perf(#3257): inline require events notify

* perf(#3257): inline require log notify

Author: alex-courtis

lua/nvim-tree.lua

@@ -281,15 +281,14 @@ function M.setup(config_user)
 
   manage_netrw()
 
-  require("nvim-tree.log").setup(config.g)
+  log.start()
 
   if log.enabled("config") then
     log.line("config", "default config + user")
     log.raw("config", "%s\n", vim.inspect(config.g))
   end
 
-  require("nvim-tree.appearance").setup()
-  require("nvim-tree.renderer.components").setup(config.g)
+  require("nvim-tree.appearance").highlight()
 
   require("nvim-tree.view-state").initialize()
 

lua/nvim-tree/actions/fs/rename-file.lua

@@ -192,8 +192,4 @@ function M.rename_full(node)
   prompt_to_rename(node, ":p")
 end
 
-function M.setup(opts)
-  config.g.filesystem_watchers = opts.filesystem_watchers
-end
-
 return M

lua/nvim-tree/appearance/init.lua

@@ -182,7 +182,8 @@ M.LEGACY_LINKS = {
   NvimTreeDiagnosticHintFolderHL = "NvimTreeLspDiagnosticsHintFolderText",
 }
 
-function M.setup()
+---Create all highlight groups and links. Idempotent.
+function M.highlight()
   -- non-linked
   for _, g in ipairs(M.HIGHLIGHT_GROUPS) do
     if g.def then

lua/nvim-tree/config.lua

@@ -511,6 +511,21 @@ local function process_config(g)
   -- Open
   --
   g.actions.open_file.window_picker.chars = tostring(g.actions.open_file.window_picker.chars):upper()
+
+  --
+  -- Padding
+  --
+  if g.renderer.indent_width < 1 then
+    g.renderer.indent_width = 1
+  end
+  for k, v in pairs(g.renderer.indent_markers.icons) do
+    if #v == 0 then
+      g.renderer.indent_markers.icons[k] = " "
+    else
+      -- return the first character from the UTF-8 encoded string; we may use utf8.codes from Lua 5.3 when available
+      g.renderer.indent_markers.icons[k] = v:match("[%z\1-\127\194-\244][\128-\191]*")
+    end
+  end
 end
 
 ---Validate user config and migrate legacy.

lua/nvim-tree/events.lua

@@ -1,4 +1,3 @@
-local notify = require("nvim-tree.notify")
 local Event = require("nvim-tree._meta.api.events").Event
 
 local M = {}
@@ -25,7 +24,7 @@ local function dispatch(event_name, payload)
   for _, handler in pairs(get_handlers(event_name)) do
     local success, error = pcall(handler, payload)
     if not success then
-      notify.error("Handler for event " .. event_name .. " errored. " .. vim.inspect(error))
+      require("nvim-tree.notify").error("Handler for event " .. event_name .. " errored. " .. vim.inspect(error))
     end
   end
 end

lua/nvim-tree/explorer/init.lua

@@ -88,7 +88,7 @@ function Explorer:create_autocmds()
   vim.api.nvim_create_autocmd("ColorScheme", {
     group = self.augroup_id,
     callback = function()
-      appearance.setup()
+      appearance.highlight()
       view.reset_winhl()
       self.renderer:draw()
     end,

lua/nvim-tree/legacy.lua

@@ -1,5 +1,3 @@
-local notify = require("nvim-tree.notify")
-
 local M = {}
 
 --- Create empty sub-tables if not present
@@ -120,25 +118,32 @@ end
 ---@param u nvim_tree.config user supplied subset of config
 local function deprecated_config(u)
   if type(u.view) == "table" and u.view.hide_root_folder then
-    notify.info("view.hide_root_folder is deprecated, please set renderer.root_folder_label = false")
+    require("nvim-tree.notify").info(
+      "view.hide_root_folder is deprecated, please set renderer.root_folder_label = false"
+    )
   end
 end
 
 ---@param u nvim_tree.config user supplied subset of config
 local function removed_config(u)
   if u.auto_close then
-    notify.warn("auto close feature has been removed: https://github.com/nvim-tree/nvim-tree.lua/wiki/Auto-Close")
+    require("nvim-tree.notify").warn(
+      "auto close feature has been removed: https://github.com/nvim-tree/nvim-tree.lua/wiki/Auto-Close"
+    )
     u["auto_close"] = nil
   end
 
   if u.focus_empty_on_setup then
-    notify.warn("focus_empty_on_setup has been removed: https://github.com/nvim-tree/nvim-tree.lua/wiki/Open-At-Startup")
+    require("nvim-tree.notify").warn(
+      "focus_empty_on_setup has been removed: https://github.com/nvim-tree/nvim-tree.lua/wiki/Open-At-Startup"
+    )
     u["focus_empty_on_setup"] = nil
   end
 
   if u.create_in_closed_folder then
-    notify.warn(
-      "create_in_closed_folder has been removed and is now the default behaviour. You may use api.fs.create to add a file under your desired node.")
+    require("nvim-tree.notify").warn(
+      "create_in_closed_folder has been removed and is now the default behaviour. You may use api.fs.create to add a file under your desired node."
+    )
   end
   u["create_in_closed_folder"] = nil
 end

lua/nvim-tree/log.lua

@@ -1,3 +1,5 @@
+local config = require("nvim-tree.config")
+
 ---@alias LogTypes "all" | "config" | "copy_paste" | "dev" | "diagnostics" | "git" | "profile" | "watcher"
 
 ---@type table<LogTypes, boolean>
@@ -110,14 +112,17 @@ function M.enabled(typ)
   return file_path ~= nil and (types[typ] or types.all)
 end
 
-function M.setup(opts)
-  if opts.log and opts.log.enable and opts.log.types then
-    types = opts.log.types
+--- Create the log file and enable logging, if globally configured
+function M.start()
+  if config.g.log and config.g.log.enable and config.g.log.types then
+    types = config.g.log.types
     file_path = string.format("%s/nvim-tree.log", vim.fn.stdpath("log"), os.date("%H:%M:%S"), vim.env.USER)
-    if opts.log.truncate then
+    if config.g.log.truncate then
       os.remove(file_path)
     end
-    require("nvim-tree.notify").debug("nvim-tree.lua logging to " .. file_path)
+    if config.g.notify.threshold <= vim.log.levels.DEBUG then
+      require("nvim-tree.notify").debug("nvim-tree.lua logging to " .. file_path)
+    end
   end
 end
 

lua/nvim-tree/renderer/components/devicons.lua

@@ -1,3 +1,5 @@
+local config = require("nvim-tree.config")
+
 ---@alias devicons_get_icon fun(name: string, ext: string?, opts: table?): string?, string?
 ---@alias devicons_setup fun(opts: table?)
 
@@ -6,11 +8,18 @@
 ---@field get_icon devicons_get_icon
 local devicons
 
+--One shot lazy discovery and setup done
+local initialized = false
+
 local M = {}
 
 ---Wrapper around nvim-web-devicons, nils if devicons not available
 ---@type devicons_get_icon
 function M.get_icon(name, ext, opts)
+  if not initialized then
+    M.initialize()
+  end
+
   if devicons then
     return devicons.get_icon(name, ext, opts)
   else
@@ -19,9 +28,8 @@ function M.get_icon(name, ext, opts)
 end
 
 ---Attempt to use nvim-web-devicons if present and enabled for file or folder
----@param opts table
-function M.setup(opts)
-  if opts.renderer.icons.show.file or opts.renderer.icons.show.folder then
+function M.initialize()
+  if config.g.renderer.icons.show.file or config.g.renderer.icons.show.folder then
     local ok, di = pcall(require, "nvim-web-devicons")
     if ok then
       devicons = di --[[@as DevIcons]]
@@ -30,6 +38,7 @@ function M.setup(opts)
       devicons.setup()
     end
   end
+  initialized = true
 end
 
 return M