From 275d40713191e6c0012783ecf762a4faa138098b Mon Sep 17 00:00:00 2001
From: ohogb <147062435+ohogb@users.noreply.github.com>
Date: Tue, 7 Jan 2025 20:17:26 +0200
Subject: [PATCH] fix(signature): use `char_under_cursor` in `on_char_added`
 handler (#935)

`char` from the `on_char_added` handler, which was used before, doesn't
represent the character under the cursor in some situations, such as
when using an autopairs plugin.

Fixes #909
---
 lua/blink/cmp/signature/trigger.lua | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/lua/blink/cmp/signature/trigger.lua b/lua/blink/cmp/signature/trigger.lua
index ff2c248f..2d874c5f 100644
--- a/lua/blink/cmp/signature/trigger.lua
+++ b/lua/blink/cmp/signature/trigger.lua
@@ -45,15 +45,18 @@ function trigger.activate()
     has_context = function() return trigger.context ~= nil end,
   })
   trigger.buffer_events:listen({
-    on_char_added = function(char)
+    on_char_added = function()
+      local cursor_col = vim.api.nvim_win_get_cursor(0)[2]
+      local char_under_cursor = vim.api.nvim_get_current_line():sub(cursor_col, cursor_col)
+
       -- ignore if disabled
       if not require('blink.cmp.config').enabled() then
         return trigger.hide()
       -- character forces a trigger according to the sources, refresh the existing context if it exists
-      elseif trigger.is_trigger_character(char) then
-        return trigger.show({ trigger_character = char })
+      elseif trigger.is_trigger_character(char_under_cursor) then
+        return trigger.show({ trigger_character = char_under_cursor })
       -- character forces a re-trigger according to the sources, show if we have a context
-      elseif trigger.is_trigger_character(char, true) and trigger.context ~= nil then
+      elseif trigger.is_trigger_character(char_under_cursor, true) and trigger.context ~= nil then
         return trigger.show()
       end
     end,