Restore textDocument/documentSymbol handler

Co-authored-by: Adam Tao <[email protected]>
razzmatazz · Feb 24, 2024 · 36bcc29 · 36bcc29
1 parent 1328fa9
commit 36bcc29
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/CSharpLanguageServer/Handlers/DocumentSymbol.fs b/src/CSharpLanguageServer/Handlers/DocumentSymbol.fs
@@ -14,7 +14,9 @@ module DocumentSymbol =
     let provider (clientCapabilities: ClientCapabilities option) : U2<bool,DocumentSymbolOptions> option =
         true |> U2.First |> Some
 
-    let handle (scope: ServerRequestScope) (p: DocumentSymbolParams): AsyncLspResult<DocumentSymbol [] option> = async {
+    let handle (scope: ServerRequestScope)
+               (p: DocumentSymbolParams)
+            : AsyncLspResult<U2<SymbolInformation [], DocumentSymbol []> option> = async {
         let canEmitDocSymbolHierarchy =
             scope.ClientCapabilities
             |> Option.bind (fun cc -> cc.TextDocument)
@@ -35,6 +37,7 @@ module DocumentSymbol =
             collector.Visit(syntaxTree.GetRoot())
 
             return collector.GetDocumentSymbols(canEmitDocSymbolHierarchy)
+                   |> U2.Second
                    |> Some
                    |> LspResult.success
 

diff --git a/src/CSharpLanguageServer/Lsp/Server.fs b/src/CSharpLanguageServer/Lsp/Server.fs
@@ -240,7 +240,8 @@ type CSharpLspServer(lspClient: CSharpLspClient, settings: ServerSettings) =
         override this.TextDocumentOnTypeFormatting(p) =
             p |> withReadOnlyScope DocumentOnTypeFormatting.handle
 
-        override this.TextDocumentDocumentSymbol(p) = notImplemented
+        override this.TextDocumentDocumentSymbol(p) =
+            p |> withReadOnlyScope DocumentSymbol.handle
 
         override __.WorkspaceDidChangeWatchedFiles(p) =
             p |> withReadWriteScope (Workspace.didChangeWatchedFiles logMessage diagnostics.Post)