Skip to content

Commit 9cdb7af

Browse files
committed
chore: removed unused code
1 parent e7e6cd1 commit 9cdb7af

File tree

6 files changed

+14
-245
lines changed

6 files changed

+14
-245
lines changed

lua/copilot-lsp/nes/init.lua

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,4 +113,10 @@ function M.apply_pending_nes(bufnr)
113113
return true
114114
end
115115

116+
---@param bufnr? integer
117+
function M.clear_suggestion(bufnr)
118+
bufnr = bufnr and bufnr > 0 and bufnr or vim.api.nvim_get_current_buf()
119+
nes_ui.clear_suggestion(bufnr, nes_ns)
120+
end
121+
116122
return M

lua/copilot-lsp/nes/state.lua

Whitespace-only changes.

lua/copilot-lsp/nes/ui.lua

Lines changed: 6 additions & 63 deletions
Original file line numberDiff line numberDiff line change
@@ -14,79 +14,21 @@ function M.clear_suggestion(bufnr, ns_id)
1414
vim.b[bufnr].nes_jump = false
1515
return
1616
end
17-
vim.api.nvim_buf_clear_namespace(bufnr, ns_id, 0, -1)
17+
_dismiss_suggestion(bufnr, ns_id)
1818
---@type copilotlsp.InlineEdit
1919
local state = vim.b[bufnr].nes_state
2020
if not state then
2121
return
2222
end
2323

24-
_dismiss_suggestion(bufnr, ns_id)
2524
vim.b[bufnr].nes_state = nil
2625
end
2726

28-
local function trim_end(s)
29-
return s:gsub("%s+$", "")
30-
end
31-
3227
---@private
33-
---@param suggestion copilotlsp.InlineEdit
34-
---@return copilotlsp.nes.LineCalculationResult
35-
function M._calculate_lines(suggestion)
36-
local deleted_lines_count = suggestion.range["end"].line - suggestion.range.start.line
37-
local added_lines = vim.split(trim_end(suggestion.newText), "\n")
38-
local added_lines_count = suggestion.newText == "" and 0 or #added_lines
39-
local same_line = false
40-
41-
if deleted_lines_count == 0 and added_lines_count == 1 then
42-
---changing within line
43-
deleted_lines_count = 1
44-
same_line = true
45-
end
46-
47-
-- if
48-
-- suggestion.range.start.line == suggestion.range["end"].line
49-
-- and suggestion.range.start.character == suggestion.range["end"].character
50-
-- then
51-
-- --add only
52-
-- TODO: Do we need to position specifically for add only?
53-
-- UI tests seem to say no
54-
-- end
55-
56-
-- Calculate positions for delete highlight extmark
57-
---@type copilotlsp.nes.DeleteExtmark
58-
local delete_extmark = {
59-
row = suggestion.range.start.line,
60-
end_row = (
61-
suggestion.range["end"].character ~= 0 and suggestion.range["end"].line + 1
62-
or suggestion.range["end"].line
63-
),
64-
}
65-
66-
-- Calculate positions for virtual lines extmark
67-
---@type copilotlsp.nes.AddExtmark
68-
local virt_lines_extmark = {
69-
row = (
70-
suggestion.range["end"].character ~= 0 and suggestion.range["end"].line
71-
or suggestion.range["end"].line - 1
72-
),
73-
virt_lines_count = added_lines_count,
74-
}
75-
76-
return {
77-
deleted_lines_count = deleted_lines_count,
78-
added_lines = added_lines,
79-
added_lines_count = added_lines_count,
80-
same_line = same_line,
81-
delete_extmark = delete_extmark,
82-
virt_lines_extmark = virt_lines_extmark,
83-
}
84-
end
85-
8628
---@param bufnr integer
8729
---@param edit lsp.TextEdit
8830
---@return copilotlsp.nes.InlineEditPreview
89-
function M.caculate_preview(bufnr, edit)
31+
function M._calculate_preview(bufnr, edit)
9032
local text = edit.newText
9133
local range = edit.range
9234
local start_line = range.start.line
@@ -175,10 +117,11 @@ function M.caculate_preview(bufnr, edit)
175117
}
176118
end
177119

120+
---@private
178121
---@param bufnr integer
179122
---@param ns_id integer
180123
---@param preview copilotlsp.nes.InlineEditPreview
181-
function M.display_inline_edit_preview(bufnr, ns_id, preview)
124+
function M._display_preview(bufnr, ns_id, preview)
182125
if preview.deletion then
183126
local range = preview.deletion.range
184127
vim.api.nvim_buf_set_extmark(bufnr, ns_id, range.start.line, range.start.character, {
@@ -222,8 +165,8 @@ function M._display_next_suggestion(bufnr, ns_id, edits)
222165

223166
local suggestion = edits[1]
224167

225-
local preview = M.caculate_preview(bufnr, suggestion)
226-
M.display_inline_edit_preview(bufnr, ns_id, preview)
168+
local preview = M._calculate_preview(bufnr, suggestion)
169+
M._display_preview(bufnr, ns_id, preview)
227170

228171
vim.b[bufnr].nes_state = suggestion
229172

lua/copilot-lsp/types.lua

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -6,28 +6,6 @@
66
---@class copilotlsp.copilotInlineEditResponse
77
---@field edits copilotlsp.InlineEdit[]
88

9-
---@class copilotlsp.nes.EditSuggestionUI
10-
---@field preview_winnr? integer
11-
12-
---@class copilotlsp.nes.DeleteExtmark
13-
--- Holds row information for delete highlight extmark.
14-
---@field row number
15-
---@field end_row number
16-
17-
---@class copilotlsp.nes.AddExtmark
18-
-- Holds row and virtual lines count for virtual lines extmark.
19-
---@field row number
20-
---@field virt_lines_count number
21-
22-
---@class copilotlsp.nes.LineCalculationResult
23-
--- The result of calculating lines for inline suggestion UI.
24-
---@field deleted_lines_count number
25-
---@field added_lines string[]
26-
---@field added_lines_count number
27-
---@field same_line boolean
28-
---@field delete_extmark copilotlsp.nes.DeleteExtmark
29-
---@field virt_lines_extmark copilotlsp.nes.AddExtmark
30-
319
---@class copilotlsp.nes.TextDeletion
3210
---@field range lsp.Range
3311

tests/nes/test_ui.lua

Lines changed: 0 additions & 158 deletions
This file was deleted.

tests/nes/test_ui_preview.lua

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -236,14 +236,14 @@ do
236236

237237
child.g.inline_edit = case.edit
238238
local preview = child.lua_func(function()
239-
return require("copilot-lsp.nes.ui").caculate_preview(0, vim.g.inline_edit)
239+
return require("copilot-lsp.nes.ui")._calculate_preview(0, vim.g.inline_edit)
240240
end)
241241
eq(preview, case.preview)
242242

243243
child.g.inline_preview = preview
244244
child.lua_func(function()
245245
local ns_id = vim.api.nvim_create_namespace("nes")
246-
require("copilot-lsp.nes.ui").display_inline_edit_preview(0, ns_id, vim.g.inline_preview)
246+
require("copilot-lsp.nes.ui")._display_preview(0, ns_id, vim.g.inline_preview)
247247
end)
248248
ref(child.get_screenshot())
249249

0 commit comments

Comments
 (0)