Unify the naming of request and notification types

Request types should always have the suffix Request and notifications
should end with Notification.

Also moved all request and notification types into separate folders to
reduce the number of files in the LanguageServerProtocol folder.

Author: ahoppen

Sources/LanguageServerProtocol/Messages.swift

@@ -26,9 +26,9 @@ public let builtinRequests: [_RequestType.Type] = [
   ImplementationRequest.self,
   ReferencesRequest.self,
   DocumentHighlightRequest.self,
-  DocumentFormatting.self,
-  DocumentRangeFormatting.self,
-  DocumentOnTypeFormatting.self,
+  DocumentFormattingRequest.self,
+  DocumentRangeFormattingRequest.self,
+  DocumentOnTypeFormattingRequest.self,
   FoldingRangeRequest.self,
   DocumentSymbolRequest.self,
   DocumentColorRequest.self,
@@ -39,7 +39,7 @@ public let builtinRequests: [_RequestType.Type] = [
   // MARK: LSP Extension Requests
 
   SymbolInfoRequest.self,
-  PollIndex.self,
+  PollIndexRequest.self,
 ]
 
 /// The set of known notifications.
@@ -49,17 +49,17 @@ public let builtinRequests: [_RequestType.Type] = [
 /// `MessageRegistry._register()` which allows you to avoid bloating the real server implementation.
 public let builtinNotifications: [NotificationType.Type] = [
   InitializedNotification.self,
-  Exit.self,
-  CancelRequest.self,
-  LogMessage.self,
-  DidChangeConfiguration.self,
-  DidChangeWorkspaceFolders.self,
-  DidOpenTextDocument.self,
-  DidCloseTextDocument.self,
-  DidChangeTextDocument.self,
-  DidSaveTextDocument.self,
-  WillSaveTextDocument.self,
-  PublishDiagnostics.self,
+  ExitNotification.self,
+  CancelRequestNotification.self,
+  LogMessageNotification.self,
+  DidChangeConfigurationNotification.self,
+  DidChangeWorkspaceFoldersNotification.self,
+  DidOpenTextDocumentNotification.self,
+  DidCloseTextDocumentNotification.self,
+  DidChangeTextDocumentNotification.self,
+  DidSaveTextDocumentNotification.self,
+  WillSaveTextDocumentNotification.self,
+  PublishDiagnosticsNotification.self,
 ]
 
 // MARK: Miscellaneous Message Types

Sources/LanguageServerProtocol/CancelRequest.swift renamed to Sources/LanguageServerProtocol/Notifications/CancelRequestNotification.swift

@@ -18,7 +18,7 @@
 /// As with any `$` requests, the server is free to ignore this notification.
 ///
 /// - Parameter id: The request to cancel.
-public struct CancelRequest: NotificationType, Hashable {
+public struct CancelRequestNotification: NotificationType, Hashable {
   public static let method: String = "$/cancelRequest"
 
   /// The request to cancel.

Sources/LanguageServerProtocol/Configuration.swift renamed to Sources/LanguageServerProtocol/Notifications/ConfigurationNotification.swift

@@ -15,7 +15,7 @@
 /// - Note: the format of the settings is implementation-defined.
 ///
 /// - Parameter settings: The changed workspace settings.
-public struct DidChangeConfiguration: NotificationType {
+public struct DidChangeConfigurationNotification: NotificationType {
   public static let method: String = "workspace/didChangeConfiguration"
 
   /// The changed workspace settings.

Sources/LanguageServerProtocol/WorkspaceFolders.swift renamed to Sources/LanguageServerProtocol/Notifications/DidChangeWorkspaceFoldersNotification.swift

@@ -10,24 +10,12 @@
 //
 //===----------------------------------------------------------------------===//
 
-/// Request from the server for the set of currently open workspace folders.
-///
-///
-/// Clients that support workspace folders should set the `workspaceFolders` client capability.
-///
-/// - Returns: The set of currently open workspace folders. Returns nil if only a single file is
-///   open. Returns an empty array if a workspace is open but no folders are configured.
-public struct WorkspaceFoldersRequest: RequestType, Hashable {
-    public static let method: String = "workspace/workspaceFolders"
-    public typealias Response = [WorkspaceFolder]?
-}
-
 /// Notification from the client that the set of open workspace folders has changed.
 ///
 /// - Parameter event: The set of changes.
 ///
 /// Requires the `workspaceFolders` capability on both the client and server.
-public struct DidChangeWorkspaceFolders: NotificationType {
+public struct DidChangeWorkspaceFoldersNotification: NotificationType {
     public static let method: String = "workspace/didChangeWorkspaceFolders"
 
     /// The set of changes.

Sources/LanguageServerProtocol/Notifications/ExtiNotification.swift

@@ -0,0 +1,19 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2014 - 2019 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+//
+//===----------------------------------------------------------------------===//
+
+/// Notification that the server process should exit.
+///
+/// This notification will come after the shutdown request finishes.
+public struct ExitNotification: NotificationType, Hashable {
+  public static let method: String = "exit"
+  public init() { }
+}

Sources/LanguageServerProtocol/Notifications/InitializedNotification.swift

@@ -0,0 +1,18 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2014 - 2019 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+//
+//===----------------------------------------------------------------------===//
+
+/// Notification from the client that its own initialization of the language server has finished.
+public struct InitializedNotification: NotificationType, Hashable {
+  public static let method: String = "initialized"
+
+  public init() {}
+}

Sources/LanguageServerProtocol/LogMessage.swift renamed to Sources/LanguageServerProtocol/Notifications/LogMessageNotification.swift

@@ -15,7 +15,7 @@
 /// - Parameters:
 ///   - type: The kind of log message.
 ///   - message: The contents of the message.
-public struct LogMessage: NotificationType, Hashable {
+public struct LogMessageNotification: NotificationType, Hashable {
   public static let method: String = "window/logMessage"
 
   /// The kind of log message.
@@ -28,4 +28,4 @@ public struct LogMessage: NotificationType, Hashable {
     self.type = type
     self.message = message
   }
-}
+}

Sources/LanguageServerProtocol/PublishDiagnostics.swift renamed to Sources/LanguageServerProtocol/Notifications/PublishDiagnosticsNotification.swift

@@ -24,7 +24,7 @@
 /// - Parameters:
 ///   - uri: The document in which the diagnostics should be shown.
 ///   - diagnostics: The complete list of diagnostics in the document, if any.
-public struct PublishDiagnostics: NotificationType, Hashable, Codable {
+public struct PublishDiagnosticsNotification: NotificationType, Hashable, Codable {
   public static let method: String = "textDocument/publishDiagnostics"
 
   /// The document in which the diagnostics should be shown.

Sources/LanguageServerProtocol/Notifications/ShowMessageNotification.swift

@@ -0,0 +1,31 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2014 - 2019 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+//
+//===----------------------------------------------------------------------===//
+
+/// Notification from the server containing a message for the client to display.
+///
+/// - Parameters:
+///   - type: The kind of message.
+///   - message: The contents of the message.
+public struct ShowMessageNotification: NotificationType, Hashable {
+  public static let method: String = "window/showMessage"
+
+  /// The kind of message.
+  public var type: WindowMessageType
+
+  /// The contents of the message.
+  public var message: String
+
+  public init(type: WindowMessageType, message: String) {
+    self.type = type
+    self.message = message
+  }
+}

Sources/LanguageServerProtocol/TextSynchronization.swift renamed to Sources/LanguageServerProtocol/Notifications/TextSynchronizationNotifications.swift

@@ -26,7 +26,7 @@
 /// capability.
 ///
 /// - Parameter textDocument: The document identifier and initial contents.
-public struct DidOpenTextDocument: NotificationType, Hashable {
+public struct DidOpenTextDocumentNotification: NotificationType, Hashable {
   public static let method: String = "textDocument/didOpen"
 
   /// The document identifier and initial contents.
@@ -44,7 +44,7 @@ public struct DidOpenTextDocument: NotificationType, Hashable {
 /// management of the document contents to disk, if appropriate.
 ///
 /// - Parameter textDocument: The document to close, which must be currently open.
-public struct DidCloseTextDocument: NotificationType, Hashable {
+public struct DidCloseTextDocumentNotification: NotificationType, Hashable {
   public static let method: String = "textDocument/didClose"
 
   /// The document to close, which must be currently open.
@@ -65,7 +65,7 @@ public struct DidCloseTextDocument: NotificationType, Hashable {
 /// - Parameters:
 ///   - textDocument: The document to change and its current version identifier.
 ///   - contentChanges: Edits to the document.
-public struct DidChangeTextDocument: NotificationType, Hashable {
+public struct DidChangeTextDocumentNotification: NotificationType, Hashable {
   public static let method: String = "textDocument/didChange"
 
   /// The document to change and its current version identifier.
@@ -90,7 +90,7 @@ public struct DidChangeTextDocument: NotificationType, Hashable {
 ///   - reason: Whether this was user-initiated, auto-saved, etc.
 ///
 /// Servers that support willSave should set the `willSave` text document sync option.
-public struct WillSaveTextDocument: NotificationType, Hashable {
+public struct WillSaveTextDocumentNotification: NotificationType, Hashable {
   public static let method: String = "textDocument/willSave"
 
   /// The document that will be saved.
@@ -107,7 +107,7 @@ public struct WillSaveTextDocument: NotificationType, Hashable {
 ///   - text: The content of the document at the time of save.
 ///
 /// Servers that support didSave should set the `save` text document sync option.
-public struct DidSaveTextDocument: NotificationType, Hashable {
+public struct DidSaveTextDocumentNotification: NotificationType, Hashable {
   public static let method: String = "textDocument/didSave"
 
   /// The document that was saved.

Sources/LanguageServerProtocol/ApplyEdit.swift renamed to Sources/LanguageServerProtocol/Requests/ApplyEditRequest.swift

@@ -111,7 +111,7 @@ public struct CodeActionContext: Codable, Hashable {
   }
 }
 
-public struct CodeAction: Codable, Equatable, ResponseType {
+public struct CodeAction: Codable, Equatable {
 
   /// A short, human-readable, title for this code action.
   public var title: String

Sources/LanguageServerProtocol/CodeAction.swift renamed to Sources/LanguageServerProtocol/Requests/CodeActionRequest.swift

@@ -62,7 +62,7 @@ public enum DocumentSymbolResponse: ResponseType, Hashable {
 /// Represents programming constructs like variables, classes, interfaces etc. that appear 
 /// in a document. Document symbols can be hierarchical and they have two ranges: one that encloses
 /// its definition and one that points to its most interesting range, e.g. the range of an identifier.
-public struct DocumentSymbol: Hashable, Codable, ResponseType {
+public struct DocumentSymbol: Hashable, Codable {
 
   /// The name of this symbol. Will be displayed in the user interface and therefore must not be
   /// an empty string or a string only consisting of white spaces.

Sources/LanguageServerProtocol/ColorPresentationRequest.swift renamed to Sources/LanguageServerProtocol/Requests/ColorPresentationRequest.swift

@@ -19,7 +19,7 @@
 ///   - options: Options to customize the formatting.
 ///
 /// - Returns: An array of of text edits describing the formatting changes to the document, if any.
-public struct DocumentFormatting: TextDocumentRequest, Hashable {
+public struct DocumentFormattingRequest: TextDocumentRequest, Hashable {
   public static let method: String = "textDocument/formatting"
   public typealias Response = [TextEdit]?
 
@@ -41,7 +41,7 @@ public struct DocumentFormatting: TextDocumentRequest, Hashable {
 ///   - options: Options to customize the formatting.
 ///
 /// - Returns: An array of of text edits describing the formatting changes to the document, if any.
-public struct DocumentRangeFormatting: TextDocumentRequest, Hashable {
+public struct DocumentRangeFormattingRequest: TextDocumentRequest, Hashable {
   public static let method: String = "textDocument/rangeFormatting"
   public typealias Response = [TextEdit]?
 
@@ -72,7 +72,7 @@ public struct DocumentRangeFormatting: TextDocumentRequest, Hashable {
 ///   - options: Options to customize the formatting.
 ///
 /// - Returns: An array of of text edits describing the formatting changes to the document, if any.
-public struct DocumentOnTypeFormatting: TextDocumentRequest, Hashable {
+public struct DocumentOnTypeFormattingRequest: TextDocumentRequest, Hashable {
   public static let method: String = "textDocument/onTypeFormatting"
   public typealias Response = [TextEdit]?
 

Sources/LanguageServerProtocol/Completion.swift renamed to Sources/LanguageServerProtocol/Requests/CompletionRequest.swift

@@ -106,10 +106,3 @@ public struct InitializeResult: ResponseType, Hashable {
     self.capabilities = capabilities
   }
 }
-
-/// Notification from the client that its own initialization of the language server has finished.
-public struct InitializedNotification: NotificationType, Hashable {
-  public static let method: String = "initialized"
-
-  public init() {}
-}

Sources/LanguageServerProtocol/DefinitionRequest.swift renamed to Sources/LanguageServerProtocol/Requests/DefinitionRequest.swift

@@ -15,7 +15,7 @@
 ///
 /// Users of PollIndex should set `"initializationOptions": { "listenToUnitEvents": false }` during
 /// the `initialize` request.
-public struct PollIndex: RequestType {
+public struct PollIndexRequest: RequestType {
   public static var method: String = "workspace/_pollIndex"
   public typealias Response = VoidResponse
 }

Sources/LanguageServerProtocol/DocumentColor.swift renamed to Sources/LanguageServerProtocol/Requests/DocumentColorRequest.swift

@@ -52,23 +52,3 @@ public struct MessageActionItem: ResponseType, Hashable {
     self.title = title
   }
 }
-
-/// Notification from the server containing a message for the client to display.
-///
-/// - Parameters:
-///   - type: The kind of message.
-///   - message: The contents of the message.
-public struct ShowMessage: NotificationType, Hashable {
-  public static let method: String = "window/showMessage"
-
-  /// The kind of message.
-  public var type: WindowMessageType
-
-  /// The contents of the message.
-  public var message: String
-
-  public init(type: WindowMessageType, message: String) {
-    self.type = type
-    self.message = message
-  }
-}

Sources/LanguageServerProtocol/DocumentHighlight.swift renamed to Sources/LanguageServerProtocol/Requests/DocumentHighlightRequest.swift

@@ -22,11 +22,3 @@ public struct Shutdown: RequestType, Hashable {
 
   public init() { }
 }
-
-/// Notification that the server process should exit.
-///
-/// This notification will come after the shutdown request finishes.
-public struct Exit: NotificationType, Hashable {
-  public static let method: String = "exit"
-  public init() { }
-}

Sources/LanguageServerProtocol/DocumentSymbol.swift renamed to Sources/LanguageServerProtocol/Requests/DocumentSymbolRequest.swift

@@ -0,0 +1,23 @@
+//===----------------------------------------------------------------------===//
+//
+// This source file is part of the Swift.org open source project
+//
+// Copyright (c) 2014 - 2019 Apple Inc. and the Swift project authors
+// Licensed under Apache License v2.0 with Runtime Library Exception
+//
+// See https://swift.org/LICENSE.txt for license information
+// See https://swift.org/CONTRIBUTORS.txt for the list of Swift project authors
+//
+//===----------------------------------------------------------------------===//
+
+/// Request from the server for the set of currently open workspace folders.
+///
+///
+/// Clients that support workspace folders should set the `workspaceFolders` client capability.
+///
+/// - Returns: The set of currently open workspace folders. Returns nil if only a single file is
+///   open. Returns an empty array if a workspace is open but no folders are configured.
+public struct WorkspaceFoldersRequest: RequestType, Hashable {
+    public static let method: String = "workspace/workspaceFolders"
+    public typealias Response = [WorkspaceFolder]?
+}

Sources/LanguageServerProtocol/ExecuteCommand.swift renamed to Sources/LanguageServerProtocol/Requests/ExecuteCommandRequest.swift

@@ -120,7 +120,7 @@ extension SKSwiftPMTestWorkspace {
 
 extension SKSwiftPMTestWorkspace {
   public func openDocument(_ url: URL, language: Language) throws {
-    sk.send(DidOpenTextDocument(textDocument: TextDocumentItem(
+    sk.send(DidOpenTextDocumentNotification(textDocument: TextDocumentItem(
       uri: DocumentURI(url),
       language: language,
       version: 1,

Sources/LanguageServerProtocol/FoldingRange.swift renamed to Sources/LanguageServerProtocol/Requests/FoldingRangeRequest.swift

@@ -83,7 +83,7 @@ extension SKTibsTestWorkspace {
 
 extension SKTibsTestWorkspace {
   public func openDocument(_ url: URL, language: Language) throws {
-    sk.send(DidOpenTextDocument(textDocument: TextDocumentItem(
+    sk.send(DidOpenTextDocumentNotification(textDocument: TextDocumentItem(
       uri: DocumentURI(url),
       language: language,
       version: 1,