Fixes #44870: use replace-move semantics to avoid expanding selection

src/vs/editor/contrib/format/formatCommand.ts

@@ -46,7 +46,7 @@ export class EditOperationsCommand implements editorCommon.ICommand {
 	static execute(editor: ICodeEditor, _edits: TextEdit[]) {
 		let edits = this._handleEolEdits(editor, _edits);
 		editor.pushUndoStop();
-		editor.executeEdits('formatEditsCommand', edits.map(edit => EditOperation.replace(Range.lift(edit.range), edit.text)));
+		editor.executeEdits('formatEditsCommand', edits.map(edit => EditOperation.replaceMove(Range.lift(edit.range), edit.text)));
 		editor.pushUndoStop();
 	}
 

src/vs/editor/contrib/format/test/formatCommand.test.ts

@@ -11,6 +11,7 @@ import { ISingleEditOperation } from 'vs/editor/common/model';
 import { TextModel } from 'vs/editor/common/model/textModel';
 import { EditOperationsCommand } from 'vs/editor/contrib/format/formatCommand';
 import { testCommand } from 'vs/editor/test/browser/testCommand';
+import { withTestCodeEditor } from 'vs/editor/test/browser/testCodeEditor';
 
 function editOp(startLineNumber: number, startColumn: number, endLineNumber: number, endColumn: number, text: string[]): ISingleEditOperation {
 	return {
@@ -310,4 +311,27 @@ suite('FormatCommand', () => {
 		);
 	});
 
+	test('issue #44870', () => {
+		const initialText = [
+			'[',
+			'    {},{',
+			'    }',
+			']',
+		];
+		withTestCodeEditor(initialText, {}, (editor) => {
+			editor.setSelection(new Selection(2, 8, 2, 8));
+			EditOperationsCommand.execute(editor, [
+				editOp(2, 8, 2, 8, ['', '    ']),
+				editOp(2, 9, 3, 5, ['']),
+			]);
+			assert.equal(editor.getValue(), [
+				'[',
+				'    {},',
+				'    {}',
+				']',
+			].join('\n'));
+			assert.deepEqual(editor.getSelection(), new Selection(3, 5, 3, 5));
+		});
+	});
+
 });