we were not correctly excluding system dependencies and executable targets

tayloraswift · tayloraswift · commit 2bfc1023fa50 · 2024-09-14T20:37:52.000Z
diff --git a/Sources/SymbolGraphBuilder/Builds/SSGC.PackageBuild.swift b/Sources/SymbolGraphBuilder/Builds/SSGC.PackageBuild.swift
@@ -89,7 +89,10 @@ extension SSGC.PackageBuild
                 default:    continue
                 }
 
-                modulesToDump[constituent.id] = include
+                if  constituent.module.type.hasSymbols
+                {
+                    modulesToDump[constituent.id] = include
+                }
             }
         }
 
diff --git a/Sources/SymbolGraphBuilder/Sources/SSGC.DocumentationSources.swift b/Sources/SymbolGraphBuilder/Sources/SSGC.DocumentationSources.swift
@@ -47,7 +47,8 @@ extension SSGC.DocumentationSources
             moduleIndexes = try moduleLayouts.map
             {
                 let id:Symbol.Module = $0.id
-                let constituents:[SSGC.ModuleLayout] = try self.constituents(of: $0)
+                let constituents:[SSGC.ModuleLayout] = try self.constituents(of: $0).filter(
+                    \.module.type.hasSymbols)
 
                 let symbols:
                 (

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,10 @@ extension SSGC.PackageBuild`
`89`	`89`	`default: continue`
`90`	`90`	`}`
`91`	`91`
`92`		`- modulesToDump[constituent.id] = include`
	`92`	`+ if constituent.module.type.hasSymbols`
	`93`	`+ {`
	`94`	`+ modulesToDump[constituent.id] = include`
	`95`	`+ }`
`93`	`96`	`}`
`94`	`97`	`}`
`95`	`98`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,8 @@ extension SSGC.DocumentationSources`
`47`	`47`	`moduleIndexes = try moduleLayouts.map`
`48`	`48`	`{`
`49`	`49`	`let id:Symbol.Module = $0.id`
`50`		`- let constituents:[SSGC.ModuleLayout] = try self.constituents(of: $0)`
	`50`	`+ let constituents:[SSGC.ModuleLayout] = try self.constituents(of: $0).filter(`
	`51`	`+ \.module.type.hasSymbols)`
`51`	`52`
`52`	`53`	`let symbols:`
`53`	`54`	`(`