Add noHistory argument to cursorMove command to skip navigation history

src/vs/editor/browser/coreCommands.ts

@@ -31,6 +31,7 @@ import { IViewModel } from '../common/viewModel.js';
 import { ISelection } from '../common/core/selection.js';
 import { getActiveElement, isEditableElement } from '../../base/browser/dom.js';
 import { EnterOperation } from '../common/cursor/cursorTypeEditOperations.js';
+import { TextEditorSelectionSource } from '../../platform/editor/common/editor.js';
 
 const CORE_WEIGHT = KeybindingWeight.EditorCore;
 
@@ -604,13 +605,16 @@ export namespace CoreNavigationCommands {
 		}
 
 		private _runCursorMove(viewModel: IViewModel, source: string | null | undefined, args: CursorMove_.ParsedArguments): void {
+			// If noHistory is true, use PROGRAMMATIC source to prevent adding to navigation history
+			const effectiveSource = args.noHistory ? TextEditorSelectionSource.PROGRAMMATIC : source;
+
 			viewModel.model.pushStackElement();
 			viewModel.setCursorStates(
-				source,
+				effectiveSource,
 				CursorChangeReason.Explicit,
 				CursorMoveImpl._move(viewModel, viewModel.getCursorStates(), args)
 			);
-			viewModel.revealAllCursors(source, true);
+			viewModel.revealAllCursors(effectiveSource, true);
 		}
 
 		private static _move(viewModel: IViewModel, cursors: CursorState[], args: CursorMove_.ParsedArguments): PartialCursorState[] | null {

src/vs/editor/common/cursor/cursorMoveCommands.ts

@@ -604,6 +604,10 @@ export namespace CursorMove {
 			return false;
 		}
 
+		if (!types.isUndefined(cursorMoveArg.noHistory) && !types.isBoolean(cursorMoveArg.noHistory)) {
+			return false;
+		}
+
 		return true;
 	};
 
@@ -626,6 +630,7 @@ export namespace CursorMove {
 						\`\`\`
 					* 'value': Number of units to move. Default is '1'.
 					* 'select': If 'true' makes the selection. Default is 'false'.
+					* 'noHistory': If 'true' does not add the movement to navigation history. Default is 'false'.
 				`,
 				constraint: isCursorMoveArgs,
 				schema: {
@@ -647,6 +652,10 @@ export namespace CursorMove {
 						'select': {
 							'type': 'boolean',
 							'default': false
+						},
+						'noHistory': {
+							'type': 'boolean',
+							'default': false
 						}
 					}
 				}
@@ -697,6 +706,7 @@ export namespace CursorMove {
 		select?: boolean;
 		by?: string;
 		value?: number;
+		noHistory?: boolean;
 	}
 
 	export function parse(args: Partial<RawArguments>): ParsedArguments | null {
@@ -777,7 +787,8 @@ export namespace CursorMove {
 			direction: direction,
 			unit: unit,
 			select: (!!args.select),
-			value: (args.value || 1)
+			value: (args.value || 1),
+			noHistory: (!!args.noHistory)
 		};
 	}
 
@@ -786,6 +797,7 @@ export namespace CursorMove {
 		unit: Unit;
 		select: boolean;
 		value: number;
+		noHistory: boolean;
 	}
 
 	export interface SimpleMoveArguments {