Revert snippet text cursor positioning but keep sort order changes

Co-authored-by: RyanCavanaugh <[email protected]>

Author: Copilot

src/services/completions.ts

@@ -1731,14 +1731,7 @@ function createCompletionEntry(
     // We should only have needsConvertPropertyAccess if there's a property access to convert. But see #21790.
     // Somehow there was a global with a non-identifier name. Hopefully someone will complain about getting a "foo bar" global completion and provide a repro.
     else if ((useBraces || insertQuestionDot) && propertyAccessToConvert) {
-        if (useBraces && preferences.includeCompletionsWithSnippetText) {
-            // For symbol completions, position cursor inside brackets for better UX
-            insertText = needsConvertPropertyAccess ? `[${quotePropertyName(sourceFile, preferences, name)}$0]` : `[${name}$0]`;
-            isSnippet = true;
-        }
-        else {
-            insertText = useBraces ? needsConvertPropertyAccess ? `[${quotePropertyName(sourceFile, preferences, name)}]` : `[${name}]` : name;
-        }
+        insertText = useBraces ? needsConvertPropertyAccess ? `[${quotePropertyName(sourceFile, preferences, name)}]` : `[${name}]` : name;
         if (insertQuestionDot || propertyAccessToConvert.questionDotToken) {
             insertText = `?.${insertText}`;
         }