refactor(fs): use fs.ls in readdir implementation, prefer fs_stat over fs_fstat (#2101)

1 files changed, 13 insertions, 36 deletions

diff --git a/lua/mason-core/fs.lua b/lua/mason-core/fs.lua
index 597ba3fa..2f620a49 100644
--- a/lua/mason-core/fs.lua
+++ b/lua/mason-core/fs.lua
@@ -7,32 +7,29 @@ local function make_module(uv)
     local M = {}
 
     ---@param path string
-    function M.fstat(path)
-        log.trace("fs: fstat", path)
-        local fd = uv.fs_open(path, "r", 438)
-        local fstat = uv.fs_fstat(fd)
-        uv.fs_close(fd)
-        return fstat
+    function M.stat(path)
+        log.trace("fs: stat", path)
+        return assert(uv.fs_stat(path))
     end
 
     ---@param path string
     function M.file_exists(path)
         log.trace("fs: file_exists", path)
-        local ok, fstat = pcall(M.fstat, path)
+        local ok, stat = pcall(M.stat, path)
         if not ok then
             return false
         end
-        return fstat.type == "file"
+        return stat.type == "file"
     end
 
     ---@param path string
     function M.dir_exists(path)
         log.trace("fs: dir_exists", path)
-        local ok, fstat = pcall(M.fstat, path)
+        local ok, stat = pcall(M.stat, path)
         if not ok then
             return false
         end
-        return fstat.type == "directory"
+        return stat.type == "directory"
     end
 
     ---@param path string
@@ -188,33 +185,13 @@ local function make_module(uv)
     ---@param path string: The full path to the directory to read.
     ---@return ReaddirEntry[]
     function M.readdir(path)
-        log.trace("fs: fs_opendir", path)
-        local dir = assert(vim.loop.fs_opendir(path, nil, 25))
         local all_entries = {}
-        local exhausted = false
-
-        repeat
-            local entries = uv.fs_readdir(dir)
-            log.trace("fs: fs_readdir", path, entries)
-            if entries and #entries > 0 then
-                for i = 1, #entries do
-                    if entries[i].name and not entries[i].type then
-                        -- See https://github.com/luvit/luv/issues/660
-                        local full_path = Path.concat { path, entries[i].name }
-                        log.trace("fs: fs_readdir falling back to fs_stat to find type", full_path)
-                        local stat = uv.fs_stat(full_path)
-                        entries[i].type = stat.type
-                    end
-                    all_entries[#all_entries + 1] = entries[i]
-                end
-            else
-                log.trace("fs: fs_readdir exhausted scan", path)
-                exhausted = true
-            end
-        until exhausted
-
-        uv.fs_closedir(dir)
-
+        M.ls(path, function(_, entry, type)
+            all_entries[#all_entries + 1] = {
+                name = entry,
+                type = type,
+            }
+        end)
         return all_entries
     end