Add minimum access level as a setting for the symbol graph extraction

Author: cmcgee1024

Sources/SWBCore/Settings/BuiltinMacros.swift

@@ -1099,6 +1099,7 @@ public final class BuiltinMacros {
     public static let DOCC_ARCHIVE_PATH = BuiltinMacros.declareStringMacro("DOCC_ARCHIVE_PATH")
     public static let DOCC_PRETTY_PRINT = BuiltinMacros.declareBooleanMacro("DOCC_PRETTY_PRINT")
     public static let DOCC_EXTRACT_SPI_DOCUMENTATION = BuiltinMacros.declareBooleanMacro("DOCC_EXTRACT_SPI_DOCUMENTATION")
+    public static let DOCC_MINIMUM_ACCESS_LEVEL = BuiltinMacros.declareEnumMacro("DOCC_MINIMUM_ACCESS_LEVEL") as EnumMacroDeclaration<DoccMinimumAccessLevel>
     public static let DOCC_SKIP_SYNTHESIZED_MEMBERS = BuiltinMacros.declareBooleanMacro("DOCC_SKIP_SYNTHESIZED_MEMBERS")
     public static let DOCC_EXTRACT_EXTENSION_SYMBOLS = BuiltinMacros.declareBooleanMacro("DOCC_EXTRACT_EXTENSION_SYMBOLS")
     public static let DOCC_EXTRACT_SWIFT_INFO_FOR_OBJC_SYMBOLS = BuiltinMacros.declareBooleanMacro("DOCC_EXTRACT_SWIFT_INFO_FOR_OBJC_SYMBOLS")
@@ -1650,6 +1651,7 @@ public final class BuiltinMacros {
         DOCC_ARCHIVE_PATH,
         DOCC_PRETTY_PRINT,
         DOCC_SKIP_SYNTHESIZED_MEMBERS,
+        DOCC_MINIMUM_ACCESS_LEVEL,
         DOCC_EXTRACT_SPI_DOCUMENTATION,
         DOCC_EXTRACT_EXTENSION_SYMBOLS,
         DOCC_EXTRACT_SWIFT_INFO_FOR_OBJC_SYMBOLS,
@@ -2757,6 +2759,18 @@ public enum LinkerDriverChoice: String, Equatable, Hashable, EnumerationMacroTyp
     case auto
 }
 
+public enum DoccMinimumAccessLevel: String, Equatable, Hashable, EnumerationMacroType {
+    public static let defaultValue = DoccMinimumAccessLevel.none
+
+    case none = ""
+    case `private` = "private"
+    case `fileprivate` = "fileprivate"
+    case `internal` = "internal"
+    case `package` = "package"
+    case `public` = "public"
+    case `open` = "open"
+}
+
 /// Enumeration macro type for the value of the `INFOPLIST_KEY_LSApplicationCategoryType` build setting.
 public enum ApplicationCategory: String, Equatable, Hashable, EnumerationMacroType {
     public static let defaultValue = ApplicationCategory.none

Sources/SWBCore/SpecImplementations/Tools/DocumentationCompiler.swift

@@ -78,7 +78,7 @@ final public class DocumentationCompilerSpec: GenericCompilerSpec, SpecIdentifie
         var additionalFlags = [String]()
 
         // Check if pretty print is requested
-        if cbc.scope.evaluate(BuiltinMacros.DOCC_PRETTY_PRINT) && swiftCompilerInfo.toolFeatures.has(.emitExtensionBlockSymbols) {
+        if cbc.scope.evaluate(BuiltinMacros.DOCC_PRETTY_PRINT) {
             additionalFlags.append("-symbol-graph-pretty-print")
         }
 
@@ -88,19 +88,34 @@ final public class DocumentationCompilerSpec: GenericCompilerSpec, SpecIdentifie
         }
 
         // Check if synthesized members should be skipped
-        if cbc.scope.evaluate(BuiltinMacros.DOCC_SKIP_SYNTHESIZED_MEMBERS) && swiftCompilerInfo.toolFeatures.has(.emitExtensionBlockSymbols) {
+        if cbc.scope.evaluate(BuiltinMacros.DOCC_SKIP_SYNTHESIZED_MEMBERS) {
             additionalFlags.append("-symbol-graph-skip-synthesized-members")
         }
 
-        switch DocumentationType(from: cbc) {
-        case .executable:
-            // When building executable types (like applications and command-line tools), include
-            // internal symbols in the generated symbol graph.
-            return additionalFlags.appending(contentsOf: ["-symbol-graph-minimum-access-level", "internal"])
-        case .framework, .none:
-            // For frameworks (and non-documentable types), just use the default behavior
-            // of the symbol graph tool.
-            return additionalFlags
+        switch cbc.scope.evaluate(BuiltinMacros.DOCC_MINIMUM_ACCESS_LEVEL) {
+        case .none:
+            switch DocumentationType(from: cbc) {
+            case .executable:
+                // When building executable types (like applications and command-line tools), include
+                // internal symbols in the generated symbol graph.
+                return additionalFlags.appending(contentsOf: ["-symbol-graph-minimum-access-level", "internal"])
+            case .framework, .none:
+                // For frameworks (and non-documentable types), just use the default behavior
+                // of the symbol graph tool.
+                return additionalFlags
+            }
+        case .private:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "private"])
+        case .fileprivate:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "fileprivate"])
+        case .internal:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "internal"])
+        case .package:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "package"])
+        case .public:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "public"])
+        case .open:
+            return additionalFlags.appending(contentsOf: ["symbol-graph-minimum-access-level", "open"])
         }
     }
 

Sources/SWBUniversalPlatform/Specs/Documentation.xcspec

@@ -175,6 +175,22 @@
                 DefaultValue = NO;
             },
 
+            // The minimum access level for API for the symbol graph extractor to generate
+            {
+                Name = DOCC_MINIMUM_ACCESS_LEVEL;
+                Type = Enumeration;
+                Values = (
+                    none,
+                    private,
+                    fileprivate,
+                    internal,
+                    package,
+                    public,
+                    open,
+                );
+                DefaultValue = none;
+            },
+
             {
                 Name = DOCC_EXTRACT_SWIFT_INFO_FOR_OBJC_SYMBOLS;
                 Type = bool;