Expand code action range to entire document range when source actions are...

Expand code action range to entire document range when source actions are requested without any active selection

Fixes #53525

src/vs/editor/contrib/codeAction/codeAction.ts

@@ -28,6 +28,10 @@ export function getCodeActions(
 		trigger: trigger.type === 'manual' ? CodeActionTriggerKind.Manual : CodeActionTriggerKind.Automatic
 	};
 
+	if (filter.kind && CodeActionKind.Source.contains(filter.kind) && rangeOrSelection.isEmpty()) {
+		rangeOrSelection = model.getFullModelRange();
+	}
+
 	const promises = CodeActionProviderRegistry.all(model)
 		// Avoid calling providers that we know will not return code actions of interest
 		.filter(provider => {

src/vs/editor/contrib/codeAction/test/codeAction.test.ts

@@ -12,6 +12,7 @@ import { getCodeActions } from 'vs/editor/contrib/codeAction/codeAction';
 import { CodeActionKind } from 'vs/editor/contrib/codeAction/codeActionTrigger';
 import { IMarkerData, MarkerSeverity } from 'vs/platform/markers/common/markers';
 import { CancellationToken } from 'vs/base/common/cancellation';
+import { ITextModel } from 'vs/editor/common/model';
 
 suite('CodeAction', () => {
 
@@ -216,4 +217,49 @@ suite('CodeAction', () => {
 		assert.strictEqual(actions.length, 0);
 		assert.strictEqual(wasInvoked, false);
 	});
+
+	test('getCodeActions requests for source actions should expand source actions range to entire document #53525', async function () {
+		const provider = new class implements CodeActionProvider {
+			provideCodeActions(model: ITextModel, range: Range): CodeAction[] {
+				return [{
+					title: rangeToString(range),
+					kind: CodeActionKind.Source.value,
+				}];
+			}
+		};
+
+		disposables.push(CodeActionProviderRegistry.register('fooLang', provider));
+
+		{
+			const actions = await getCodeActions(model, new Range(1, 1, 1, 1), {
+				type: 'manual',
+				filter: {
+					kind: CodeActionKind.Source,
+					includeSourceActions: true,
+				}
+			}, CancellationToken.None);
+			assert.strictEqual(actions.length, 1);
+			assert.strictEqual(actions[0].title, rangeToString(model.getFullModelRange()));
+		}
+
+		{
+			const range = new Range(1, 1, 1, 2);
+
+			// But we should not expand for non-empty selections
+			const actions = await getCodeActions(model, range, {
+				type: 'manual',
+				filter: {
+					kind: CodeActionKind.Source,
+					includeSourceActions: true,
+				}
+			}, CancellationToken.None);
+			assert.strictEqual(actions.length, 1);
+			assert.strictEqual(actions[0].title, rangeToString(range));
+		}
+	});
 });
+
+function rangeToString(range: Range): string {
+	return `${range.startLineNumber},${range.startColumn} ${range.endLineNumber},${range.endColumn} `;
+}
+