fix completions replacing punctuation

[xdBronch]

basically the same as #2605, the (,[,{ would get replaced when completions were manually triggered
this affected a lot more than what I added tests for but it seems unlikely that any one in particular would uniquely regress so I just did some more likely cases to keep it small

[Techatrix]

This does not handle "replace" text edits correctly in certain cases:

const foo = 5;
const bar = foo(f<cursor>oo);

The "replace" text edit should change the line to the following:

const bar = foo(foo);

but instead it does the same as the "insert" text edit:

const bar = foo(foooo);

Test Case:

    try testCompletionTextEdit(.{
        .source =
        \\const foo = 5;
        \\const bar = foo(<cursor>foo);
        ,
        .label = "foo",
        .expected_insert_line = "const bar = foo(foofoo);",
        .expected_replace_line = "const bar = foo(foo);",
    });

Since I had some related fixes to completion text edits, I went ahead and fixed this in #2621.