Refactor Roslyn document retrieval and enhance completion trigger handling for non-workspace files

Author: lextm

src/SharpIDE.Application/Features/Analysis/RoslynAnalysis.cs

@@ -469,8 +469,12 @@ public async Task<ImmutableArray<SharpIdeDiagnostic>> GetDocumentAnalyzerDiagnos
 	private static async Task<Document> GetDocumentForSharpIdeFile(SharpIdeFile fileModel, CancellationToken cancellationToken = default)
 	{
 		var project = GetProjectForSharpIdeFile(fileModel);
-		var document = fileModel.IsCsharpFile ? project.Documents.SingleOrDefault(s => s.FilePath == fileModel.Path)
-				: await GetRazorSourceGeneratedDocumentInProjectForSharpIdeFile(project, fileModel, cancellationToken);
+		Document? document = fileModel switch
+		{
+			{ IsCsharpFile: true } => project.Documents.SingleOrDefault(s => s.FilePath == fileModel.Path),
+			{ IsRazorFile: true } => await GetRazorSourceGeneratedDocumentInProjectForSharpIdeFile(project, fileModel, cancellationToken),
+			_ => throw new InvalidOperationException($"Roslyn document lookup is not supported for file '{fileModel.Path}'.")
+		};
 		Guard.Against.Null(document, nameof(document));
 		return document;
 	}

src/SharpIDE.Godot/Features/CodeEditor/SharpIdeCodeEdit.cs

@@ -271,12 +271,18 @@ private void OnTextChanged()
 		{
 			var __ = SharpIdeOtel.Source.StartActivity($"{nameof(SharpIdeCodeEdit)}.{nameof(OnTextChanged)}");
 			_currentFile.IsDirty.Value = true;
-			await _fileChangedService.SharpIdeFileChanged(_currentFile, text, FileChangeType.IdeUnsavedChange);
-			if (pendingCompletionTrigger is not null)
-			{
-				_completionTrigger = pendingCompletionTrigger;
-				var linePosition = new LinePosition(cursorPosition.line, cursorPosition.col);
-				var shouldTriggerCompletion = await _roslynAnalysis.ShouldTriggerCompletionAsync(_currentFile, text, linePosition, _completionTrigger!.Value);
+				await _fileChangedService.SharpIdeFileChanged(_currentFile, text, FileChangeType.IdeUnsavedChange);
+				if (pendingCompletionTrigger is not null)
+				{
+					if (_currentFile.IsRoslynWorkspaceFile is false)
+					{
+						HighlightLog($"Skipping Roslyn completion trigger for non-workspace file '{_currentFile.Path}'");
+						return;
+					}
+
+					_completionTrigger = pendingCompletionTrigger;
+					var linePosition = new LinePosition(cursorPosition.line, cursorPosition.col);
+					var shouldTriggerCompletion = await _roslynAnalysis.ShouldTriggerCompletionAsync(_currentFile, text, linePosition, _completionTrigger!.Value);
 				GD.Print($"Code completion trigger typed: '{_completionTrigger.Value.Character}' at {linePosition.Line}:{linePosition.Character} should trigger: {shouldTriggerCompletion}");
 				if (shouldTriggerCompletion)
 				{