[Macros] Introduce CompilerPluginMessageListener

Separate message listening logic from 'CompilerPluginMessgeHandler' to
'CompilerPluginMessageListener', so 'CompilerPluginMessageHandler' can be
used independently without 'MessageConnection'

'CompilerPluginMessageHandler.handleMessage()' now returns a response
object instead of sending it to the peer.

Author: rintaro

Sources/SwiftCompilerPlugin/CompilerPlugin.swift

@@ -143,7 +143,7 @@ extension CompilerPlugin {
     // Handle messages from the host until the input stream is closed,
     // indicating that we're done.
     let provider = MacroProviderAdapter(plugin: Self())
-    let impl = CompilerPluginMessageHandler(connection: connection, provider: provider)
+    let impl = CompilerPluginMessageListener(connection: connection, provider: provider)
     do {
       try impl.main()
     } catch {

Sources/SwiftCompilerPluginMessageHandling/CompilerPluginMessageHandler.swift

@@ -57,29 +57,22 @@ struct HostCapability {
   var hasExpandMacroResult: Bool { protocolVersion >= 5 }
 }
 
-/// 'CompilerPluginMessageHandler' is a type that listens to the message
-/// connection and dispatches them to the actual plugin provider, then send back
+/// 'CompilerPluginMessageListener' is a type that listens to the message
+/// connection, delegate them to the message handler, then send back
 /// the response.
 ///
-/// The low level connection and the provider is injected by the client.
-public class CompilerPluginMessageHandler<Connection: MessageConnection, Provider: PluginProvider> {
+/// The low level connection and the plugin provider is injected by the client.
+public class CompilerPluginMessageListener<Connection: MessageConnection, Provider: PluginProvider> {
   /// Message channel for bidirectional communication with the plugin host.
   let connection: Connection
 
-  /// Object to provide actual plugin functions.
-  let provider: Provider
-
-  /// Plugin host capability
-  var hostCapability: HostCapability
+  let handler: CompilerPluginMessageHandler<Provider>
 
   public init(connection: Connection, provider: Provider) {
     self.connection = connection
-    self.provider = provider
-    self.hostCapability = HostCapability()
+    self.handler = CompilerPluginMessageHandler(provider: provider)
   }
-}
 
-extension CompilerPluginMessageHandler {
   func sendMessage(_ message: PluginToHostMessage) throws {
     try connection.sendMessage(message)
   }
@@ -94,12 +87,31 @@ extension CompilerPluginMessageHandler {
   /// to serialize/deserialize the message.
   public func main() throws {
     while let message = try self.waitForNextMessage() {
-      try handleMessage(message)
+      let result = handler.handleMessage(message)
+      try connection.sendMessage(result)
     }
   }
+}
+
+/// 'CompilerPluginMessageHandler' is a type that handle a message and do the
+/// corresponding operation.
+@_spi(Compiler)
+public class CompilerPluginMessageHandler<Provider: PluginProvider> {
+  /// Object to provide actual plugin functions.
+  let provider: Provider
+
+  /// Plugin host capability
+  var hostCapability: HostCapability
+
+  public init(provider: Provider) {
+    self.provider = provider
+    self.hostCapability = HostCapability()
+  }
+}
 
+extension CompilerPluginMessageHandler {
   /// Handles a single message received from the plugin host.
-  fileprivate func handleMessage(_ message: HostToPluginMessage) throws {
+  public func handleMessage(_ message: HostToPluginMessage) -> PluginToHostMessage {
     switch message {
     case .getCapability(let hostCapability):
       // Remember the peer capability if provided.
@@ -112,10 +124,10 @@ extension CompilerPluginMessageHandler {
         protocolVersion: PluginMessage.PROTOCOL_VERSION_NUMBER,
         features: provider.features.map({ $0.rawValue })
       )
-      try self.sendMessage(.getCapabilityResult(capability: capability))
+      return .getCapabilityResult(capability: capability)
 
     case .expandFreestandingMacro(let macro, let macroRole, let discriminator, let expandingSyntax):
-      try expandFreestandingMacro(
+      return expandFreestandingMacro(
         macro: macro,
         macroRole: macroRole,
         discriminator: discriminator,
@@ -132,7 +144,7 @@ extension CompilerPluginMessageHandler {
       let extendedTypeSyntax,
       let conformanceListSyntax
     ):
-      try expandAttachedMacro(
+      return expandAttachedMacro(
         macro: macro,
         macroRole: macroRole,
         discriminator: discriminator,
@@ -159,7 +171,7 @@ extension CompilerPluginMessageHandler {
           )
         )
       }
-      try self.sendMessage(.loadPluginLibraryResult(loaded: diags.isEmpty, diagnostics: diags));
+      return .loadPluginLibraryResult(loaded: diags.isEmpty, diagnostics: diags)
     }
   }
 }

Sources/SwiftCompilerPluginMessageHandling/Macros.swift

@@ -29,7 +29,7 @@ extension CompilerPluginMessageHandler {
     macroRole pluginMacroRole: PluginMessage.MacroRole?,
     discriminator: String,
     expandingSyntax: PluginMessage.Syntax
-  ) throws {
+  ) -> PluginToHostMessage {
     let sourceManager = SourceManager()
     let syntax = sourceManager.add(expandingSyntax, foldingWith: .standardOperators)
 
@@ -73,7 +73,7 @@ extension CompilerPluginMessageHandler {
       // TODO: Remove this  when all compilers have 'hasExpandMacroResult'.
       response = .expandFreestandingMacroResult(expandedSource: expandedSource, diagnostics: diagnostics)
     }
-    try self.sendMessage(response)
+    return response
   }
 
   /// Expand `@attached(XXX)` macros.
@@ -86,7 +86,7 @@ extension CompilerPluginMessageHandler {
     parentDeclSyntax: PluginMessage.Syntax?,
     extendedTypeSyntax: PluginMessage.Syntax?,
     conformanceListSyntax: PluginMessage.Syntax?
-  ) throws {
+  ) -> PluginToHostMessage {
     let sourceManager = SourceManager()
     let context = PluginMacroExpansionContext(
       sourceManager: sourceManager,
@@ -150,7 +150,7 @@ extension CompilerPluginMessageHandler {
     } else {
       response = .expandAttachedMacroResult(expandedSources: expandedSources, diagnostics: diagnostics)
     }
-    try self.sendMessage(response)
+    return response
   }
 }