Fix language availability for articles See Alsos

rdar://90159299

Author: franklinsch

Sources/SwiftDocC/Model/Rendering/RenderNodeTranslator.swift

@@ -698,29 +698,44 @@ public struct RenderNodeTranslator: SemanticVisitor {
         }
         node.metadata.role = contentRenderer.roleForArticle(article, nodeKind: documentationNode.kind).rawValue
 
-        // Authored See Also section
-        if let seeAlso = article.seeAlso, !seeAlso.taskGroups.isEmpty {
-            node.seeAlsoSections.append(
-                contentsOf: renderGroups(
-                    seeAlso,
-                    allowExternalLinks: true,
-                    allowedTraits: documentationNode.availableVariantTraits,
-                    contentCompiler: &contentCompiler
+        node.seeAlsoSectionsVariants = VariantCollection<[TaskGroupRenderSection]>(
+            from: documentationNode.availableVariantTraits,
+            fallbackDefaultValue: []
+        ) { trait in
+            var seeAlsoSections = [TaskGroupRenderSection]()
+            
+            // Authored See Also section
+            if let seeAlso = article.seeAlso, !seeAlso.taskGroups.isEmpty {
+                seeAlsoSections.append(
+                    contentsOf: renderGroups(
+                        seeAlso,
+                        allowExternalLinks: true,
+                        allowedTraits: [trait],
+                        contentCompiler: &contentCompiler
+                    )
                 )
-            )
-        }
-        
-        // Automatic See Also section
-        if let seeAlso = try! AutomaticCuration.seeAlso(for: documentationNode, context: context, bundle: bundle, renderContext: renderContext, renderer: contentRenderer) {
-            contentCompiler.collectedTopicReferences.append(contentsOf: seeAlso.references)
-            node.seeAlsoSections.append(TaskGroupRenderSection(
-                title: seeAlso.title,
-                abstract: nil,
-                discussion: nil,
-                identifiers: seeAlso.references.map { $0.absoluteString },
-                generated: true
-            ))
-        }
+            }
+            
+            // Automatic See Also section
+            if let seeAlso = try! AutomaticCuration.seeAlso(
+                for: documentationNode,
+                context: context,
+                bundle: bundle,
+                renderContext: renderContext,
+                renderer: contentRenderer
+            ) {
+                contentCompiler.collectedTopicReferences.append(contentsOf: seeAlso.references)
+                seeAlsoSections.append(TaskGroupRenderSection(
+                    title: seeAlso.title,
+                    abstract: nil,
+                    discussion: nil,
+                    identifiers: seeAlso.references.map { $0.absoluteString },
+                    generated: true
+                ))
+            }
+            
+            return seeAlsoSections
+        } ?? .init(defaultValue: [])
 
         collectedTopicReferences.append(contentsOf: contentCompiler.collectedTopicReferences)
         node.references = createTopicRenderReferences()

Tests/SwiftDocCTests/Model/SemaToRenderNodeMultiLanguageTests.swift

@@ -468,6 +468,10 @@ class SemaToRenderNodeMixedLanguageTests: ExperimentalObjectiveCTestCase {
             topicSectionIdentifiers: [
                 "doc://org.swift.MixedLanguageFramework/documentation/MixedLanguageFramework/SwiftOnlyStruct"
             ],
+            seeAlsoSectionIdentifiers: [
+                "doc://org.swift.MixedLanguageFramework/documentation/MixedLanguageFramework/SwiftOnlyStruct",
+                "doc://org.swift.MixedLanguageFramework/documentation/MixedLanguageFramework/Article",
+            ],
             referenceTitles: [
                 "Article",
                 "MixedLanguageFramework",
@@ -495,6 +499,9 @@ class SemaToRenderNodeMixedLanguageTests: ExperimentalObjectiveCTestCase {
             topicSectionIdentifiers: [
                 "doc://org.swift.MixedLanguageFramework/documentation/MixedLanguageFramework/_MixedLanguageFrameworkVersionNumber"
             ],
+            seeAlsoSectionIdentifiers: [
+                "doc://org.swift.MixedLanguageFramework/documentation/MixedLanguageFramework/Article",
+            ],
             referenceTitles: [
                 "Article",
                 "MixedLanguageFramework",
@@ -520,6 +527,7 @@ class SemaToRenderNodeMixedLanguageTests: ExperimentalObjectiveCTestCase {
         declarationTokens expectedDeclarationTokens: [String]?,
         discussionSection expectedDiscussionSection: [String]?,
         topicSectionIdentifiers expectedTopicSectionIdentifiers: [String],
+        seeAlsoSectionIdentifiers expectedSeeAlsoSectionIdentifiers: [String]? = nil,
         referenceTitles expectedReferenceTitles: [String],
         referenceFragments expectedReferenceFragments: [String],
         failureMessage failureMessageForField: (_ field: String) -> String,
@@ -593,6 +601,16 @@ class SemaToRenderNodeMixedLanguageTests: ExperimentalObjectiveCTestCase {
             line: line
         )
 
+        if let expectedSeeAlsoSectionIdentifiers = expectedSeeAlsoSectionIdentifiers {
+            XCTAssertEqual(
+                renderNode.seeAlsoSections.flatMap(\.identifiers),
+                expectedSeeAlsoSectionIdentifiers,
+                failureMessageForField("see also sections identifiers"),
+                file: file,
+                line: line
+            )
+        }
+        
         XCTAssertEqual(
             renderNode.references.map(\.value).compactMap { reference in
                 (reference as? TopicRenderReference)?.title

Tests/SwiftDocCTests/Test Bundles/MixedLanguageFramework.docc/APICollection.md

@@ -12,4 +12,8 @@ This is an API collection.
 
 - <doc:_MixedLanguageFrameworkVersionNumber>
 
+## See Also
+
+- ``SwiftOnlyStruct``
+
 <!-- Copyright (c) 2022 Apple Inc and the Swift Project authors. All Rights Reserved. -->