use BuildSystemManager.moduleName(for:in:) instead of BuildTarget.displayName

matthewbastien · matthewbastien · commit 74cb30e92811 · 2025-01-17T11:11:55.000-05:00
diff --git a/Sources/SourceKitLSP/Documentation/DocumentationManager.swift b/Sources/SourceKitLSP/Documentation/DocumentationManager.swift
@@ -46,7 +46,7 @@ package final actor DocumentationManager {
     let targetId = await workspace.buildSystemManager.canonicalTarget(for: documentURI)
     var moduleName: String? = nil
     if let targetId = targetId {
-      moduleName = await workspace.buildSystemManager.buildTarget(named: targetId)?.displayName
+      moduleName = await workspace.buildSystemManager.moduleName(for: documentURI, in: targetId)
     }
 
     var externalIDsToConvert: [String]?

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ package final actor DocumentationManager {`
`46`	`46`	`let targetId = await workspace.buildSystemManager.canonicalTarget(for: documentURI)`
`47`	`47`	`var moduleName: String? = nil`
`48`	`48`	`if let targetId = targetId {`
`49`		`- moduleName = await workspace.buildSystemManager.buildTarget(named: targetId)?.displayName`
	`49`	`+ moduleName = await workspace.buildSystemManager.moduleName(for: documentURI, in: targetId)`
`50`	`50`	`}`
`51`	`51`
`52`	`52`	`var externalIDsToConvert: [String]?`