Insert mode not activated when snippet has default value in the placeholder of the insert node

[NicolasGB]

Make sure you have done the following

• I have updated to the latest version of blink.cmp
• I have read the README

Bug Description

When using snippets, in my case it's luasnip If an insert node has a placeholder value, i find myself in normal mode instead of insert mode.

Before the snippets refactor (at least from what i see in main, in the config) I found myself successfully in insert mode even if the snippets had a placeholder. (Up to this afternoon when I updated main and rewrite my config)

Here's a video of me having to go in insert mode:

insert.mode.mp4

Then if i remove the placeholder value from the same snippet:

insert.mode.2.mp4

Is this related to blink? Should i report it to blink_luasnip instead?

I'm also finding issues with expansion if the prefix is not a fullmatch but i've already open an issue

Relevant configuration

-- Snippets
        snippets = {
          expand = function(snippet)
            ls.lsp_expand(snippet)
          end,
          active = function(filter)
            if filter and filter.direction then
              return ls.jumpable(filter.direction)
            end
            return ls.in_snippet()
          end,
          jump = function(direction)
            ls.jump(direction)
          end,
        },
        sources = {
          -- add lazydev to sources
          completion = {
            enabled_providers = { "lsp", "path", "luasnip", "buffer", "lazydev" },
          },
          providers = {
            -- dont show LuaLS require statements when lazydev has items
            lsp = { fallback_for = { "lazydev" } },
            lazydev = { name = "LazyDev", module = "lazydev.integrations.blink" },
            -- Luasnip
            luasnip = {
              name = "luasnip",
              module = "blink_luasnip",

              score_offset = -1,

              opts = {
                use_show_condition = false,
                show_autosnippets = true,
              },
            },
          },
        }

neovim version

v0.11.0-dev-1247+g5897994cb7

blink.cmp version: branch, tag, or commit

main: 59add2d

[NicolasGB]

To add a bit more context, there might be an issue with the way expansion is handled.

After an exchange with @leiserfg I quote his response that may help you find the issue:

What I see happening, is that the trigger is being written to the buffer before calling execute, as if it were a regular text completion. And that messes up the logic to tell luasnip what region of the text to remove before expanding.

I know this is not exactly related to the placeholder part, let me know if you want me to open a different issue.