Added isEmpty to ExportedSkeleton and now omit exported skeletons entirely when they contain no declarations (Plugins/BridgeJS/Sources/BridgeJSSkeleton/BridgeJSSkeleton.swift, Plugins/BridgeJS/Sources/BridgeJSCore/SwiftToSkeleton.swift). This prevents empty exported blocks from being emitted.

kateinoigakukun · kateinoigakukun · commit 177f74fa8273 · 2026-02-04T18:25:11.000+09:00
Updated snapshots to match the new behavior (exported blocks disappear when empty), including the imported-only scenarios such as `CrossFileSkipsEmptyImportedFiles`.

Tests/snapshots run:
- `UPDATE_SNAPSHOTS=1 swift test --package-path ./Plugins/BridgeJS`
- `swift test --package-path ./Plugins/BridgeJS`

If you’d like runtime coverage too, run `make unittest SWIFT_SDK_ID=DEVELOPMENT-SNAPSHOT-2025-11-03-a-wasm32-unknown-wasip1`.
diff --git a/Plugins/BridgeJS/Sources/BridgeJSCore/SwiftToSkeleton.swift b/Plugins/BridgeJS/Sources/BridgeJSCore/SwiftToSkeleton.swift
@@ -91,7 +91,8 @@ public final class SwiftToSkeleton {
             return module
         }()
 
-        return BridgeJSSkeleton(moduleName: moduleName, exported: exported, imported: importedSkeleton)
+        let exportedSkeleton: ExportedSkeleton? = exported.isEmpty ? nil : exported
+        return BridgeJSSkeleton(moduleName: moduleName, exported: exportedSkeleton, imported: importedSkeleton)
     }
 
     func lookupType(for type: TypeSyntax, errors: inout [DiagnosticError]) -> BridgeType? {
diff --git a/Plugins/BridgeJS/Sources/BridgeJSSkeleton/BridgeJSSkeleton.swift b/Plugins/BridgeJS/Sources/BridgeJSSkeleton/BridgeJSSkeleton.swift
@@ -602,6 +602,10 @@ public struct ExportedSkeleton: Codable {
         self.protocols.append(contentsOf: other.protocols)
         assert(self.exposeToGlobal == other.exposeToGlobal)
     }
+
+    public var isEmpty: Bool {
+        functions.isEmpty && classes.isEmpty && enums.isEmpty && structs.isEmpty && protocols.isEmpty
+    }
 }
 
 // MARK: - Imported Skeleton
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/CrossFileSkipsEmptyImportedFiles.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/CrossFileSkipsEmptyImportedFiles.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/GlobalGetter.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/GlobalGetter.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/GlobalThisImports.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/GlobalThisImports.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/InvalidPropertyNames.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/InvalidPropertyNames.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/JSClass.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/JSClass.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {
diff --git a/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/SwiftClosureImports.json b/Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/SwiftClosureImports.json
@@ -1,22 +1,4 @@
 {
-  "exported" : {
-    "classes" : [
-
-    ],
-    "enums" : [
-
-    ],
-    "exposeToGlobal" : false,
-    "functions" : [
-
-    ],
-    "protocols" : [
-
-    ],
-    "structs" : [
-
-    ]
-  },
   "imported" : {
     "children" : [
       {

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,8 @@ public final class SwiftToSkeleton {`
`91`	`91`	`return module`
`92`	`92`	`}()`
`93`	`93`
`94`		`- return BridgeJSSkeleton(moduleName: moduleName, exported: exported, imported: importedSkeleton)`
	`94`	`+ let exportedSkeleton: ExportedSkeleton? = exported.isEmpty ? nil : exported`
	`95`	`+ return BridgeJSSkeleton(moduleName: moduleName, exported: exportedSkeleton, imported: importedSkeleton)`
`95`	`96`	`}`
`96`	`97`
`97`	`98`	`func lookupType(for type: TypeSyntax, errors: inout [DiagnosticError]) -> BridgeType? {`
Original file line number	Diff line number	Diff line change
`@@ -602,6 +602,10 @@ public struct ExportedSkeleton: Codable {`
`602`	`602`	`self.protocols.append(contentsOf: other.protocols)`
`603`	`603`	`assert(self.exposeToGlobal == other.exposeToGlobal)`
`604`	`604`	`}`
	`605`	`+`
	`606`	`+ public var isEmpty: Bool {`
	`607`	`+ functions.isEmpty && classes.isEmpty && enums.isEmpty && structs.isEmpty && protocols.isEmpty`
	`608`	`+ }`
`605`	`609`	`}`
`606`	`610`
`607`	`611`	`// MARK: - Imported Skeleton`