Fix Luacheck linting issues

- Remove trailing whitespace from selection.lua and test files
- Convert whitespace-only lines to empty lines
- Ensures CI passes by meeting code style requirements

Co-authored-by: ThomasK33 <[email protected]>

Author: blink-so[bot]

lua/claudecode/selection.lua

@@ -585,10 +585,10 @@ function M.get_range_selection(line1, line2)
 
   local current_buf = vim.api.nvim_get_current_buf()
   local file_path = vim.api.nvim_buf_get_name(current_buf)
-  
+
   -- Get the total number of lines in the buffer
   local total_lines = vim.api.nvim_buf_line_count(current_buf)
-  
+
   -- Ensure line2 doesn't exceed buffer bounds
   if line2 > total_lines then
     line2 = total_lines

tests/selection_test.lua

@@ -466,7 +466,7 @@ describe("Range Selection Tests", function()
         name = "/test/file.lua",
         lines = {
           "line 1",
-          "line 2", 
+          "line 2",
           "line 3",
           "line 4",
           "line 5",
@@ -498,7 +498,7 @@ describe("Range Selection Tests", function()
   describe("get_range_selection", function()
     it("should return valid selection for valid range", function()
       local result = selection.get_range_selection(2, 4)
-      
+
       assert(result ~= nil)
       assert(result.text == "line 2\nline 3\nline 4")
       assert(result.filePath == "/test/file.lua")
@@ -512,7 +512,7 @@ describe("Range Selection Tests", function()
 
     it("should return valid selection for single line range", function()
       local result = selection.get_range_selection(3, 3)
-      
+
       assert(result ~= nil)
       assert(result.text == "line 3")
       assert(result.selection.start.line == 2) -- 0-indexed
@@ -522,7 +522,7 @@ describe("Range Selection Tests", function()
 
     it("should handle range that exceeds buffer bounds", function()
       local result = selection.get_range_selection(8, 15) -- buffer only has 10 lines
-      
+
       assert(result ~= nil)
       assert(result.text == "line 8\nline 9\nline 10")
       assert(result.selection.start.line == 7) -- 0-indexed
@@ -548,7 +548,7 @@ describe("Range Selection Tests", function()
       local result1 = selection.get_range_selection(nil, 3)
       local result2 = selection.get_range_selection(2, nil)
       local result3 = selection.get_range_selection(nil, nil)
-      
+
       assert(result1 == nil)
       assert(result2 == nil)
       assert(result3 == nil)
@@ -574,14 +574,14 @@ describe("Range Selection Tests", function()
           return true
         end
       }
-      
+
       selection.state.tracking_enabled = true
       selection.server = mock_server
     end)
 
     it("should send range selection successfully", function()
       local result = selection.send_at_mention_for_visual_selection(2, 4)
-      
+
       assert(result == true)
       assert(mock_server.last_broadcast ~= nil)
       assert(mock_server.last_broadcast.event == "at_mentioned")
@@ -607,9 +607,9 @@ describe("Range Selection Tests", function()
           isEmpty = false
         }
       }
-      
+
       local result = selection.send_at_mention_for_visual_selection()
-      
+
       assert(result == true)
       assert(mock_server.last_broadcast.params.lineStart == 0)
       assert(mock_server.last_broadcast.params.lineEnd == 0)

tests/unit/claudecode_send_command_spec.lua

@@ -34,7 +34,7 @@ describe("ClaudeCodeSend Command Range Functionality", function()
       notify = spy.new(function() end),
       log = { levels = { ERROR = 1, WARN = 2, INFO = 3 } },
       deepcopy = function(t) return t end,
-      tbl_deep_extend = function(behavior, ...) 
+      tbl_deep_extend = function(behavior, ...)
         local result = {}
         for _, tbl in ipairs({...}) do
           for k, v in pairs(tbl) do
@@ -74,7 +74,7 @@ describe("ClaudeCodeSend Command Range Functionality", function()
     }
 
     local mock_config = {
-      apply = function(opts) 
+      apply = function(opts)
         return {
           auto_start = false,
           track_selection = true,
@@ -114,7 +114,7 @@ describe("ClaudeCodeSend Command Range Functionality", function()
     -- Load and setup claudecode
     claudecode = require("claudecode")
     claudecode.setup({})
-    
+
     -- Manually set server state for testing
     claudecode.state.server = mock_server
     claudecode.state.port = 12345
@@ -128,7 +128,7 @@ describe("ClaudeCodeSend Command Range Functionality", function()
   describe("ClaudeCodeSend command", function()
     it("should be registered with range support", function()
       assert.spy(_G.vim.api.nvim_create_user_command).was_called()
-      
+
       -- Find the ClaudeCodeSend command call
       local calls = _G.vim.api.nvim_create_user_command.calls
       local claudecode_send_call = nil
@@ -138,107 +138,107 @@ describe("ClaudeCodeSend Command Range Functionality", function()
           break
         end
       end
-      
+
       assert(claudecode_send_call ~= nil, "ClaudeCodeSend command should be registered")
       assert(claudecode_send_call.vals[3].range == true, "ClaudeCodeSend should support ranges")
     end)
 
     it("should pass range information to selection module when range is provided", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       -- Simulate command called with range
       local opts = {
         range = 2,
         line1 = 5,
         line2 = 8
       }
-      
+
       command_callback(opts)
-      
+
       assert.spy(mock_selection_module.send_at_mention_for_visual_selection).was_called()
       assert(mock_selection_module.last_call.line1 == 5)
       assert(mock_selection_module.last_call.line2 == 8)
     end)
 
     it("should not pass range information when range is 0", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       -- Simulate command called without range
       local opts = {
         range = 0,
         line1 = 1,
         line2 = 1
       }
-      
+
       command_callback(opts)
-      
+
       assert.spy(mock_selection_module.send_at_mention_for_visual_selection).was_called()
       assert(mock_selection_module.last_call.line1 == nil)
       assert(mock_selection_module.last_call.line2 == nil)
     end)
 
     it("should not pass range information when range is nil", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       -- Simulate command called without range
       local opts = {}
-      
+
       command_callback(opts)
-      
+
       assert.spy(mock_selection_module.send_at_mention_for_visual_selection).was_called()
       assert(mock_selection_module.last_call.line1 == nil)
       assert(mock_selection_module.last_call.line2 == nil)
     end)
 
     it("should exit visual mode and focus terminal on successful send", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       local opts = {
         range = 2,
         line1 = 5,
         line2 = 8
       }
-      
+
       command_callback(opts)
-      
+
       assert.spy(_G.vim.api.nvim_feedkeys).was_called()
       assert.spy(mock_terminal.open).was_called()
     end)
 
     it("should handle server not running", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       -- Simulate server not running
       claudecode.state.server = nil
-      
+
       local opts = {
         range = 2,
         line1 = 5,
         line2 = 8
       }
-      
+
       command_callback(opts)
-      
+
       assert.spy(_G.vim.notify).was_called()
       assert.spy(mock_selection_module.send_at_mention_for_visual_selection).was_not_called()
     end)
 
     it("should handle selection module failure", function()
       assert(command_callback ~= nil, "Command callback should be set")
-      
+
       -- Mock selection module to return false
       mock_selection_module.send_at_mention_for_visual_selection = spy.new(function()
         return false
       end)
-      
+
       local opts = {
         range = 2,
         line1 = 5,
         line2 = 8
       }
-      
+
       command_callback(opts)
-      
+
       assert.spy(mock_selection_module.send_at_mention_for_visual_selection).was_called()
       -- Should not exit visual mode or focus terminal on failure
       assert.spy(_G.vim.api.nvim_feedkeys).was_not_called()