Address latest CodeRabbit review comments

- Fix test assertion: use assert.is_nil instead of assert.is_false for pattern matching
- Fix autocommand collision: use unique group names per buffer instance
- Remove trailing whitespace from test file

These changes ensure multi-instance autocommands don't interfere with each other
and fix the test assertion logic for buffer name sanitization validation.

Author: ahdzib-maya

lua/claude-code/keymaps.lua

@@ -94,7 +94,7 @@ function M.setup_terminal_navigation(claude_code, config)
   local buf = current_instance and claude_code.claude_code.instances[current_instance]
   if buf and vim.api.nvim_buf_is_valid(buf) then
     -- Create autocommand to enter insert mode when the terminal window gets focus
-    local augroup = vim.api.nvim_create_augroup('ClaudeCodeTerminalFocus', { clear = true })
+    local augroup = vim.api.nvim_create_augroup('ClaudeCodeTerminalFocus_' .. buf, { clear = true })
 
     -- Set up multiple events for more reliable focus detection
     vim.api.nvim_create_autocmd(

tests/spec/terminal_spec.lua

@@ -132,7 +132,7 @@ describe('terminal module', function()
 
       -- Current instance should be set
       assert.is_not_nil(claude_code.claude_code.current_instance, 'Current instance should be set')
-      
+
       -- Instance should be created in instances table
       local current_instance = claude_code.claude_code.current_instance
       assert.is_not_nil(claude_code.claude_code.instances[current_instance], 'Instance buffer should be set')
@@ -217,7 +217,7 @@ describe('terminal module', function()
       -- Use an instance ID with special characters
       config.git.use_git_root = false
       config.git.multi_instance = true
-      
+
       -- Mock getcwd to return path with special characters
       _G.vim.fn.getcwd = function()
         return '/test/path with spaces/and-symbols!'
@@ -232,7 +232,7 @@ describe('terminal module', function()
         if cmd:match('file claude%-code%-.*') then
           file_cmd_found = true
           -- Ensure no special characters remain
-          assert.is_false(cmd:match('[^%w%-_]'), 'Buffer name should not contain special characters')
+          assert.is_nil(cmd:match('[^%w%-_]'), 'Buffer name should not contain special characters')
           break
         end
       end
@@ -244,7 +244,7 @@ describe('terminal module', function()
       -- Setup invalid buffer in instances
       local instance_id = '/test/git/root'
       claude_code.claude_code.instances[instance_id] = 999 -- Invalid buffer number
-      
+
       -- Mock nvim_buf_is_valid to return false for this buffer
       _G.vim.api.nvim_buf_is_valid = function(bufnr)
         return bufnr ~= 999