Skip to content

fix various code completion text edits#2621

Merged
Techatrix merged 4 commits intomasterfrom
techatrix/completion-text-edit
Feb 21, 2026
Merged

fix various code completion text edits#2621
Techatrix merged 4 commits intomasterfrom
techatrix/completion-text-edit

Conversation

@Techatrix
Copy link
Member

// selecting the "src" completion item will add another "/"
const foo = @import("src<cursor>/main.zig");
// Completion items for modules will contain invalid text edits.
// The observable behavior varies between editors.
const known_folders = @import("<cursor>");
// The "foo" completion items will contain invalid text edits.
// The observable behavior varies between editors.
const foo = 5;
const bar = foo(<cursor>);

Techatrix and others added 4 commits February 21, 2026 23:05
The existing checks where only executed on tests made with `testCompletionTextEdit`.
They also didn't cover all the requirements made by the LSP spec.
Co-Authored-By: xdBronch <51252236+xdBronch@users.noreply.github.com>
@Techatrix Techatrix merged commit 3817d1e into master Feb 21, 2026
6 checks passed
@Techatrix Techatrix deleted the techatrix/completion-text-edit branch February 21, 2026 22:22
@Techatrix Techatrix mentioned this pull request Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant