fix(tui): handle sequence larger than entire buffer

(cherry picked from commit f64475a226)
This commit is contained in:
zeertzjq
2025-10-03 15:47:38 +08:00
committed by github-actions[bot]
parent 2655998a07
commit 07184aa973
2 changed files with 75 additions and 9 deletions

View File

@@ -2297,8 +2297,8 @@ describe('TUI', function()
]])
child_exec_lua([[
vim.api.nvim_buf_set_lines(0, 0, 0, true, {'Hello'})
local ns = vim.api.nvim_create_namespace('test')
vim.api.nvim_buf_set_extmark(0, ns, 0, 1, {
_G.NS = vim.api.nvim_create_namespace('test')
vim.api.nvim_buf_set_extmark(0, _G.NS, 0, 1, {
end_col = 3,
url = 'https://example.com',
})
@@ -2306,6 +2306,19 @@ describe('TUI', function()
retry(nil, 1000, function()
eq({ { id = 0xE1EA0000, url = 'https://example.com' } }, exec_lua([[return _G.urls]]))
end)
-- No crash with very long URL #30794
child_exec_lua([[
vim.api.nvim_buf_set_extmark(0, _G.NS, 0, 3, {
end_col = 5,
url = 'https://example.com/' .. ('a'):rep(65536),
})
]])
retry(nil, nil, function()
eq({
{ id = 0xE1EA0000, url = 'https://example.com' },
{ id = 0xE1EA0001, url = 'https://example.com/' .. ('a'):rep(65536) },
}, exec_lua([[return _G.urls]]))
end)
end)
it('TermResponse works with vim.wait() from another autocommand #32706', function()
@@ -3708,6 +3721,44 @@ describe('TUI client', function()
end
end)
it('does not crash or hang with a very long title', function()
local server, _, screen_client = start_headless_server_and_client()
local server_exec_lua = tt.make_lua_executor(server)
if not server_exec_lua('return pcall(require, "ffi")') then
pending('missing LuaJIT FFI')
end
local bufname = api.nvim_buf_get_name(0)
-- Normally a title cannot be longer than the 65535-byte buffer as maketitle()
-- limits it length. Use FFI to send a very long title directly.
server_exec_lua([=[
local ffi = require('ffi')
local cstr = ffi.typeof('char[?]')
local function to_cstr(string)
return cstr(#string + 1, string)
end
ffi.cdef([[
typedef struct { char *data; size_t size; } String;
void ui_call_set_title(String title);
]])
local len = 65536
local title = ffi.new('String', { data = to_cstr(('a'):rep(len)), size = len })
ffi.C.ui_call_set_title(title)
]=])
screen_client:expect_unchanged()
assert_log('TUI: escape sequence for ext%.set_title too long', testlog)
eq(bufname, api.nvim_buf_get_var(0, 'term_title'))
-- Following escape sequences are not affected.
server:request('nvim_set_option_value', 'title', true, {})
retry(nil, nil, function()
eq('[No Name] + - Nvim', api.nvim_buf_get_var(0, 'term_title'))
end)
end)
it('throws error when no server exists', function()
clear()
local screen = tt.setup_child_nvim({