Merge pull request #1936 from ahoppen/flush-nondarwin-logger-shutdown

ahoppen · web-flow · commit 12c225c00bab · 2025-01-21T15:23:00.000-08:00
Flush the `NonDarwinLogger` before shutting down `SourceKitLSPServer`
diff --git a/Sources/SourceKitLSP/SourceKitLSPServer.swift b/Sources/SourceKitLSP/SourceKitLSPServer.swift
@@ -1181,6 +1181,8 @@ extension SourceKitLSPServer {
       await workspace.buildSystemManager.setMainFilesProvider(nil)
       workspace.closeIndex()
     }
+    // Make sure we emit all pending log messages. When we're not using `NonDarwinLogger` this is a no-op.
+    await NonDarwinLogger.flush()
   }
 
   func shutdown(_ request: ShutdownRequest) async throws -> ShutdownRequest.Response {

Original file line number	Diff line number	Diff line change
`@@ -1181,6 +1181,8 @@ extension SourceKitLSPServer {`
`1181`	`1181`	`await workspace.buildSystemManager.setMainFilesProvider(nil)`
`1182`	`1182`	`workspace.closeIndex()`
`1183`	`1183`	`}`
	`1184`	+ // Make sure we emit all pending log messages. When we're not using `NonDarwinLogger` this is a no-op.
	`1185`	`+ await NonDarwinLogger.flush()`
`1184`	`1186`	`}`
`1185`	`1187`
`1186`	`1188`	`func shutdown(_ request: ShutdownRequest) async throws -> ShutdownRequest.Response {`