YQL-19747: Set cursor shift at completion item

Set `TCandidate::Shift` for functions and generic
types. So now brackets are balanced and UI should
adopt it. YDB CLI is ready for the update and just
cut off symbols after an expected cursor position.
commit_hash:9efc1110869af7be618b841c6c132572b61046a1

Author: vityaman

yql/essentials/sql/v1/complete/sql_complete.cpp

@@ -209,42 +209,80 @@ namespace NSQLComplete {
                 if constexpr (std::is_base_of_v<TKeyword, T>) {
                     TVector<TString>& seq = context.Keywords[name.Content];
                     seq.insert(std::begin(seq), name.Content);
-                    return {ECandidateKind::Keyword, FormatKeywords(seq)};
+
+                    TCandidate candidate = {
+                        .Kind = ECandidateKind::Keyword,
+                        .Content = FormatKeywords(seq),
+                    };
+
+                    if (candidate.Content.EndsWith('(')) {
+                        candidate.Content += ')';
+                        candidate.CursorShift = 1;
+                    }
+
+                    return candidate;
                 }
 
                 if constexpr (std::is_base_of_v<TPragmaName, T>) {
                     return {ECandidateKind::PragmaName, std::move(name.Indentifier)};
                 }
 
                 if constexpr (std::is_base_of_v<TTypeName, T>) {
+                    TCandidate candidate = {
+                        .Kind = ECandidateKind::TypeName,
+                        .Content = std::move(name.Indentifier),
+                    };
+
                     switch (name.Kind) {
                         case TTypeName::EKind::Simple: {
                         } break;
                         case TTypeName::EKind::Container: {
-                            name.Indentifier += "<";
+                            candidate.Content += "<>";
+                            candidate.CursorShift = 1;
                         } break;
                         case TTypeName::EKind::Parameterized: {
-                            name.Indentifier += "(";
+                            candidate.Content += "()";
+                            candidate.CursorShift = 1;
                         } break;
                     }
-                    return {ECandidateKind::TypeName, std::move(name.Indentifier)};
+
+                    return candidate;
                 }
 
                 if constexpr (std::is_base_of_v<TFunctionName, T>) {
-                    name.Indentifier += "(";
-                    return {ECandidateKind::FunctionName, std::move(name.Indentifier)};
+                    TCandidate candidate = {
+                        .Kind = ECandidateKind::FunctionName,
+                        .Content = std::move(name.Indentifier),
+                    };
+
+                    candidate.Content += "()";
+                    candidate.CursorShift = 1;
+
+                    return candidate;
                 }
 
                 if constexpr (std::is_base_of_v<THintName, T>) {
                     return {ECandidateKind::HintName, std::move(name.Indentifier)};
                 }
 
                 if constexpr (std::is_base_of_v<TFolderName, T>) {
-                    name.Indentifier.append('/');
+                    TCandidate candidate = {
+                        .Kind = ECandidateKind::FolderName,
+                        .Content = std::move(name.Indentifier),
+                    };
+
                     if (!context.IsQuoted.AtLhs) {
-                        name.Indentifier.prepend('`');
+                        candidate.Content.prepend('`');
+                    }
+
+                    candidate.Content.append('/');
+
+                    if (!context.IsQuoted.AtRhs) {
+                        candidate.Content.append('`');
+                        candidate.CursorShift = 1;
                     }
-                    return {ECandidateKind::FolderName, std::move(name.Indentifier)};
+
+                    return candidate;
                 }
 
                 if constexpr (std::is_base_of_v<TTableName, T>) {
@@ -396,5 +434,9 @@ void Out<NSQLComplete::ECandidateKind>(IOutputStream& out, NSQLComplete::ECandid
 
 template <>
 void Out<NSQLComplete::TCandidate>(IOutputStream& out, const NSQLComplete::TCandidate& value) {
-    out << "{" << value.Kind << ", \"" << value.Content << "\"}";
+    out << "{" << value.Kind << ", \"" << value.Content << "\"";
+    if (value.CursorShift != 0) {
+        out << ", " << value.CursorShift;
+    }
+    out << "}";
 }