Remove a couple references to DocCDocumentation

Author: ahoppen

Package.swift

@@ -246,6 +246,7 @@ var targets: [Target] = [
       "DocCDocumentation",
       "LanguageServerProtocol",
       "SKUtilities",
+      "SourceKitLSP",
       "SemanticIndex",
       .product(name: "Markdown", package: "swift-markdown"),
     ],

Sources/ClangLanguageService/ClangLanguageService.swift

@@ -24,10 +24,6 @@ package import SwiftSyntax
 import TSCExtensions
 package import ToolchainRegistry
 
-#if canImport(DocCDocumentation)
-import DocCDocumentation
-#endif
-
 #if os(Windows)
 import WinSDK
 #endif
@@ -497,16 +493,16 @@ extension ClangLanguageService {
     return try await forwardRequestToClangd(req)
   }
 
-  #if canImport(DocCDocumentation)
   package func doccDocumentation(_ req: DoccDocumentationRequest) async throws -> DoccDocumentationResponse {
     guard let sourceKitLSPServer else {
       throw ResponseError.unknown("Connection to the editor closed")
     }
 
     let snapshot = try sourceKitLSPServer.documentManager.latestSnapshot(req.textDocument.uri)
-    throw ResponseError.requestFailed(doccDocumentationError: .unsupportedLanguage(snapshot.language))
+    throw ResponseError.requestFailed(
+      "Documentation preview is not available for \(snapshot.language.description) files"
+    )
   }
-  #endif
 
   package func symbolInfo(_ req: SymbolInfoRequest) async throws -> [SymbolDetails] {
     return try await forwardRequestToClangd(req)

Sources/LanguageServerProtocol/Error.swift

@@ -83,6 +83,9 @@ public struct ErrorCode: RawRepresentable, Codable, Hashable, Sendable {
 
   // MARK: SourceKit-LSP specific error codes
   public static let workspaceNotOpen: ErrorCode = ErrorCode(rawValue: -32003)
+
+  /// The method is not implemented in this `LanguageService`.
+  public static let requestNotImplemented: ErrorCode = ErrorCode(rawValue: -32004)
 }
 
 /// An error response represented by a code and message.
@@ -131,6 +134,10 @@ extension ResponseError {
   public static func internalError(_ message: String) -> ResponseError {
     return ResponseError(code: .internalError, message: message)
   }
+
+  public static func requestNotImplemented(_ method: any RequestType.Type) -> ResponseError {
+    return ResponseError(code: .requestNotImplemented, message: "request not implemented: \(method.method)")
+  }
 }
 
 /// An error during message decoding.

Sources/SourceKitLSP/LanguageService.swift

@@ -176,9 +176,7 @@ package protocol LanguageService: AnyObject, Sendable {
   func completion(_ req: CompletionRequest) async throws -> CompletionList
   func completionItemResolve(_ req: CompletionItemResolveRequest) async throws -> CompletionItem
   func hover(_ req: HoverRequest) async throws -> HoverResponse?
-  #if canImport(DocCDocumentation)
   func doccDocumentation(_ req: DoccDocumentationRequest) async throws -> DoccDocumentationResponse
-  #endif
   func symbolInfo(_ request: SymbolInfoRequest) async throws -> [SymbolDetails]
 
   /// Return the symbol graph at the given location for the contents of the document as they are on-disk (opposed to the

Sources/SourceKitLSP/SourceKitLSPServer.swift

@@ -757,10 +757,8 @@ extension SourceKitLSPServer: QueueBasedMessageHandler {
       await self.handleRequest(for: request, requestHandler: self.declaration)
     case let request as RequestAndReply<DefinitionRequest>:
       await self.handleRequest(for: request, requestHandler: self.definition)
-    #if canImport(DocCDocumentation)
     case let request as RequestAndReply<DoccDocumentationRequest>:
       await self.handleRequest(for: request, requestHandler: self.doccDocumentation)
-    #endif
     case let request as RequestAndReply<DocumentColorRequest>:
       await self.handleRequest(for: request, requestHandler: self.documentColor)
     case let request as RequestAndReply<DocumentDiagnosticsRequest>:
@@ -1573,15 +1571,13 @@ extension SourceKitLSPServer {
     return try await documentService(for: uri).completionItemResolve(request)
   }
 
-  #if canImport(DocCDocumentation)
   func doccDocumentation(
     _ req: DoccDocumentationRequest,
     workspace: Workspace,
     languageService: LanguageService
   ) async throws -> DoccDocumentationResponse {
     return try await languageService.doccDocumentation(req)
   }
-  #endif
 
   func hover(
     _ req: HoverRequest,

Sources/SwiftLanguageService/DoccDocumentation.swift

@@ -10,20 +10,23 @@
 //
 //===----------------------------------------------------------------------===//
 
-#if canImport(DocCDocumentation)
 import BuildServerIntegration
-import DocCDocumentation
 import Foundation
 import IndexStoreDB
 package import LanguageServerProtocol
-import SemanticIndex
 import SKLogging
+import SemanticIndex
+import SourceKitLSP
 import SwiftExtensions
 import SwiftSyntax
-import SourceKitLSP
+
+#if canImport(DocCDocumentation)
+import DocCDocumentation
+#endif
 
 extension SwiftLanguageService {
   package func doccDocumentation(_ req: DoccDocumentationRequest) async throws -> DoccDocumentationResponse {
+    #if canImport(DocCDocumentation)
     guard let sourceKitLSPServer else {
       throw ResponseError.internalError("SourceKit-LSP is shutting down")
     }
@@ -100,8 +103,12 @@ extension SwiftLanguageService {
       moduleName: moduleName,
       catalogURL: catalogURL
     )
+    #else
+    throw ResponseError.requestFailed("Documentation preview is not available in this build of SourceKit-LSP")
+    #endif
   }
 
+  #if canImport(DocCDocumentation)
   private func findMarkupExtensionFile(
     workspace: Workspace,
     documentationManager: DocCDocumentationManager,
@@ -127,6 +134,7 @@ extension SwiftLanguageService {
       language: .markdown
     )?.text
   }
+  #endif
 }
 
 private struct DocumentableSymbol {
@@ -216,4 +224,3 @@ private struct DocumentableSymbol {
     return nil
   }
 }
-#endif

Tests/SourceKitLSPTests/DoccDocumentationTests.swift

@@ -21,7 +21,6 @@ import SwiftDocC
 import XCTest
 
 final class DoccDocumentationTests: XCTestCase {
-
   func testUnsupportedLanguage() async throws {
     try await renderDocumentation(
       markedText: "1️⃣",

Tests/SourceKitLSPTests/DocumentationLanguageServiceTests.swift

@@ -10,6 +10,7 @@
 //
 //===----------------------------------------------------------------------===//
 
+#if canImport(DocCDocumentation)
 import LanguageServerProtocol
 import SKTestSupport
 import SourceKitLSP
@@ -39,3 +40,4 @@ private func assertHandles(language: Language) async throws {
   )
   XCTAssertEqual(completions, .init(isIncomplete: false, items: []))
 }
+#endif