remove unncessary function performEditsAndUpdateInternal() (#2277)

## Checklist

- [ ] I have added
[tests](https://www.cursorless.org/docs/contributing/test-case-recorder/)
- [ ] I have updated the
[docs](https://github.com/cursorless-dev/cursorless/tree/main/docs) and
[cheatsheet](https://github.com/cursorless-dev/cursorless/tree/main/cursorless-talon/src/cheatsheet)
- [ ] I have not broken the cheatsheet

All tests passing. ` 7211 passing (4m)`

Co-authored-by: Cedric Halbronn <[email protected]>

Author: saidelike

packages/cursorless-engine/src/core/updateSelections/updateSelections.ts

@@ -258,7 +258,7 @@ export async function performEditsAndUpdateSelections(
     document,
     originalSelections,
   );
-  return performEditsAndUpdateInternal(
+  return performEditsAndUpdateFullSelectionInfos(
     rangeUpdater,
     editor,
     edits,
@@ -307,27 +307,12 @@ export async function performEditsAndUpdateRanges(
 ): Promise<Range[][]> {
   const document = editor.document;
   const selectionInfoMatrix = rangesToSelectionInfos(document, originalRanges);
-  return performEditsAndUpdateInternal(
-    rangeUpdater,
-    editor,
-    edits,
-    selectionInfoMatrix,
-  );
-}
-
-async function performEditsAndUpdateInternal(
-  rangeUpdater: RangeUpdater,
-  editor: EditableTextEditor,
-  edits: Edit[],
-  selectionInfoMatrix: FullSelectionInfo[][],
-) {
-  await performEditsAndUpdateFullSelectionInfos(
+  return performEditsAndUpdateFullSelectionInfos(
     rangeUpdater,
     editor,
     edits,
     selectionInfoMatrix,
   );
-  return selectionInfosToSelections(selectionInfoMatrix);
 }
 
 // FIXME: Remove this function if we don't end up using it for the next couple use cases, eg `that` mark and cursor history