Implemented coverage for skipping empty imported file entries by adding multifile fixtures and a new BridgeJS codegen snapshot test to assert only non-empty import data is emitted (Plugins/BridgeJS/Tests/BridgeJSToolTests/Inputs/MacroSwift/Multifile/ImportedFunctions.swift, .../ExportedOnly.swift, BridgeJSCodegenTests.swift). Generated corresponding snapshots (__Snapshots__/BridgeJSCodegenTests/CrossFileSkipsEmptyImportedFiles.{json,swift}).

Tests:
- `UPDATE_SNAPSHOTS=1 swift test --package-path ./Plugins/BridgeJS --filter BridgeJSCodegenTests/codegenSkipsEmptyImportedFiles`
- `swift test --package-path ./Plugins/BridgeJS`

If you want full runtime coverage, next run `make unittest SWIFT_SDK_ID=DEVELOPMENT-SNAPSHOT-2025-11-03-a-wasm32-unknown-wasip1`.

Author: kateinoigakukun

Plugins/BridgeJS/Tests/BridgeJSToolTests/BridgeJSCodegenTests.swift

@@ -166,4 +166,21 @@ import Testing
         let skeleton = try swiftAPI.finalize()
         try snapshotCodegen(skeleton: skeleton, name: "CrossFileFunctionTypes.ReverseOrder")
     }
+
+    @Test
+    func codegenSkipsEmptyImportedFiles() throws {
+        let swiftAPI = SwiftToSkeleton(progress: .silent, moduleName: "TestModule", exposeToGlobal: false)
+        let importedURL = Self.multifileInputsDirectory.appendingPathComponent("ImportedFunctions.swift")
+        swiftAPI.addSourceFile(
+            Parser.parse(source: try String(contentsOf: importedURL, encoding: .utf8)),
+            inputFilePath: "ImportedFunctions.swift"
+        )
+        let exportedOnlyURL = Self.multifileInputsDirectory.appendingPathComponent("ExportedOnly.swift")
+        swiftAPI.addSourceFile(
+            Parser.parse(source: try String(contentsOf: exportedOnlyURL, encoding: .utf8)),
+            inputFilePath: "ExportedOnly.swift"
+        )
+        let skeleton = try swiftAPI.finalize()
+        try snapshotCodegen(skeleton: skeleton, name: "CrossFileSkipsEmptyImportedFiles")
+    }
 }

Plugins/BridgeJS/Tests/BridgeJSToolTests/Inputs/MacroSwift/Multifile/ExportedOnly.swift

@@ -0,0 +1,3 @@
+struct ExportedOnly {
+    let value: Int = 0
+}

Plugins/BridgeJS/Tests/BridgeJSToolTests/Inputs/MacroSwift/Multifile/ImportedFunctions.swift

@@ -0,0 +1 @@
+@JSFunction func fetchNumber() throws(JSException) -> Int

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/CrossFileSkipsEmptyImportedFiles.json

@@ -0,0 +1,43 @@
+{
+  "exported" : {
+    "classes" : [
+
+    ],
+    "enums" : [
+
+    ],
+    "exposeToGlobal" : false,
+    "functions" : [
+
+    ],
+    "protocols" : [
+
+    ],
+    "structs" : [
+
+    ]
+  },
+  "imported" : {
+    "children" : [
+      {
+        "functions" : [
+          {
+            "name" : "fetchNumber",
+            "parameters" : [
+
+            ],
+            "returnType" : {
+              "int" : {
+
+              }
+            }
+          }
+        ],
+        "types" : [
+
+        ]
+      }
+    ]
+  },
+  "moduleName" : "TestModule"
+}

Plugins/BridgeJS/Tests/BridgeJSToolTests/__Snapshots__/BridgeJSCodegenTests/CrossFileSkipsEmptyImportedFiles.swift

@@ -0,0 +1,16 @@
+#if arch(wasm32)
+@_extern(wasm, module: "TestModule", name: "bjs_fetchNumber")
+fileprivate func bjs_fetchNumber() -> Int32
+#else
+fileprivate func bjs_fetchNumber() -> Int32 {
+    fatalError("Only available on WebAssembly")
+}
+#endif
+
+func _$fetchNumber() throws(JSException) -> Int {
+    let ret = bjs_fetchNumber()
+    if let error = _swift_js_take_exception() {
+        throw error
+    }
+    return Int.bridgeJSLiftReturn(ret)
+}