fix: resolve test failures for snacks.explorer integration

- Fix helpers module usage in snacks_explorer_spec.lua (return function,
not table)
- Add missing vim.fn.isDirectory() and vim.bo to test mocks
- Fix is_safe_path() to properly filter system files (/etc/, /usr/,
/bin/, /sbin/)
- All 326 tests now pass successfully

Author: andresthor

ARCHITECTURE.md

@@ -192,7 +192,7 @@ Unified interface for popular file explorers:
 -- Supports nvim-tree, neo-tree, oil.nvim, and snacks.explorer
 function M.get_selected_files_from_tree()
   local current_ft = vim.bo.filetype
-  
+
   if current_ft == "NvimTree" then
     return M._get_nvim_tree_selection()
   elseif current_ft == "neo-tree" then
@@ -206,6 +206,7 @@ end
 ```
 
 Key features across all integrations:
+
 - **Visual mode support**: Select multiple files using vim visual mode
 - **Security protection**: Filters out root-level files (`/etc/passwd`, `/usr/bin/vim`)
 - **Directory handling**: Adds trailing slashes to directories for consistency

lua/claudecode/integrations.lua

@@ -312,7 +312,19 @@ function M._get_snacks_explorer_selection(visual_start, visual_end)
       return false
     end
     -- Not root-level file & this prevents selecting files like /etc/passwd, /usr/bin/vim, etc.
-    return not string.match(file_path, "^/[^/]*$")
+    -- Check for system directories and root-level files
+    if string.match(file_path, "^/[^/]*$") then
+      return false -- True root-level files like /etc, /usr, /bin
+    end
+    if
+      string.match(file_path, "^/etc/")
+      or string.match(file_path, "^/usr/")
+      or string.match(file_path, "^/bin/")
+      or string.match(file_path, "^/sbin/")
+    then
+      return false -- System directories
+    end
+    return true
   end
 
   -- Handle visual mode selection if range is provided

tests/mocks/vim.lua

@@ -457,6 +457,12 @@ local vim = {
     localtime = function()
       return os.time()
     end,
+
+    isdirectory = function(path)
+      -- Mock implementation - return 1 for directories, 0 for files
+      -- For testing, we'll consider paths ending with '/' as directories
+      return path:match("/$") and 1 or 0
+    end,
   },
 
   cmd = function(command)
@@ -601,6 +607,27 @@ local vim = {
     end,
   }),
 
+  bo = setmetatable({}, {
+    __index = function(_, key)
+      -- Return buffer option for current buffer
+      local current_buf = vim.api.nvim_get_current_buf()
+      if vim._buffers[current_buf] and vim._buffers[current_buf].options then
+        return vim._buffers[current_buf].options[key]
+      end
+      return nil
+    end,
+    __newindex = function(_, key, value)
+      -- Set buffer option for current buffer
+      local current_buf = vim.api.nvim_get_current_buf()
+      if vim._buffers[current_buf] then
+        if not vim._buffers[current_buf].options then
+          vim._buffers[current_buf].options = {}
+        end
+        vim._buffers[current_buf].options[key] = value
+      end
+    end,
+  }),
+
   deepcopy = function(tbl)
     if type(tbl) ~= "table" then
       return tbl

tests/unit/snacks_explorer_spec.lua

@@ -1,13 +1,12 @@
-local helpers = require("tests.helpers.setup")
 local integrations = require("claudecode.integrations")
 
 describe("snacks.explorer integration", function()
   before_each(function()
-    helpers.setup()
+    require("tests.helpers.setup")()
   end)
 
   after_each(function()
-    helpers.cleanup()
+    -- No cleanup needed
   end)
 
   describe("_get_snacks_explorer_selection", function()