Drop Embedded Swift with 6.1 and earlier and remove JAVASCRIPTKIT_EXPERIMENTAL_EMBEDDED_WASM

Author: kateinoigakukun

Package.swift

@@ -4,7 +4,6 @@ import CompilerPluginSupport
 import PackageDescription
 
 // NOTE: needed for embedded customizations, ideally this will not be necessary at all in the future, or can be replaced with traits
-let shouldBuildForEmbedded = Context.environment["JAVASCRIPTKIT_EXPERIMENTAL_EMBEDDED_WASM"].flatMap(Bool.init) ?? false
 let useLegacyResourceBundling =
     Context.environment["JAVASCRIPTKIT_USE_LEGACY_RESOURCE_BUNDLING"].flatMap(Bool.init) ?? false
 
@@ -45,18 +44,9 @@ let package = Package(
             dependencies: ["_CJavaScriptKit", "BridgeJSMacros"],
             exclude: useLegacyResourceBundling ? [] : ["Runtime"],
             resources: useLegacyResourceBundling ? [.copy("Runtime")] : [],
-            cSettings: shouldBuildForEmbedded
-                ? [
-                    .unsafeFlags(["-fdeclspec"])
-                ] : nil,
             swiftSettings: [
                 .enableExperimentalFeature("Extern")
             ]
-                + (shouldBuildForEmbedded
-                    ? [
-                        .enableExperimentalFeature("Embedded"),
-                        .unsafeFlags(["-Xfrontend", "-emit-empty-object-file"]),
-                    ] : [])
         ),
         .target(name: "_CJavaScriptKit"),
         .macro(
@@ -78,12 +68,7 @@ let package = Package(
 
         .target(
             name: "JavaScriptBigIntSupport",
-            dependencies: ["_CJavaScriptBigIntSupport", "JavaScriptKit"],
-            swiftSettings: shouldBuildForEmbedded
-                ? [
-                    .enableExperimentalFeature("Embedded"),
-                    .unsafeFlags(["-Xfrontend", "-emit-empty-object-file"]),
-                ] : []
+            dependencies: ["_CJavaScriptBigIntSupport", "JavaScriptKit"]
         ),
         .target(name: "_CJavaScriptBigIntSupport", dependencies: ["_CJavaScriptKit"]),
         .testTarget(
@@ -94,12 +79,7 @@ let package = Package(
 
         .target(
             name: "JavaScriptEventLoop",
-            dependencies: ["JavaScriptKit", "_CJavaScriptEventLoop"],
-            swiftSettings: shouldBuildForEmbedded
-                ? [
-                    .enableExperimentalFeature("Embedded"),
-                    .unsafeFlags(["-Xfrontend", "-emit-empty-object-file"]),
-                ] : []
+            dependencies: ["JavaScriptKit", "_CJavaScriptEventLoop"]
         ),
         .target(name: "_CJavaScriptEventLoop"),
         .testTarget(

Plugins/PackageToJS/Sources/PackageToJSPlugin.swift

@@ -374,11 +374,6 @@ struct PackageToJSPlugin: CommandPlugin {
 
     /// Check if the build is for embedded WebAssembly
     private func isBuildingForEmbedded(selfPackage: Package) -> Bool {
-        if let rawValue = ProcessInfo.processInfo.environment["JAVASCRIPTKIT_EXPERIMENTAL_EMBEDDED_WASM"],
-            let value = Bool(rawValue), value
-        {
-            return true
-        }
         let coreTarget = selfPackage.targets.first { $0.name == "JavaScriptKit" }
         guard let swiftTarget = coreTarget as? SwiftSourceModuleTarget else {
             return false

Plugins/PackageToJS/Tests/ExampleTests.swift

@@ -352,21 +352,15 @@ extension Trait where Self == ConditionTrait {
         }
     }
 
-    // FIXME: This test fails on the current main snapshot
-    #if !compiler(>=6.3)
     @Test(.requireEmbeddedSwiftInSwiftSDK())
     func embeddedWasmUnknownWasi() throws {
         let swiftSDKID = try #require(Self.getEmbeddedSwiftSDKID())
         try withPackage(at: "Examples/Embedded") { packageDir, _, runSwift in
             try runSwift(
-                ["package", "--swift-sdk", swiftSDKID, "js", "-c", "release"],
-                [
-                    "JAVASCRIPTKIT_EXPERIMENTAL_EMBEDDED_WASM": "true"
-                ]
+                ["package", "--swift-sdk", swiftSDKID, "js", "-c", "release"], [:]
             )
         }
     }
-    #endif  // compiler(>=6.3)
 
     @Test(.requireSwiftSDK)
     func continuationLeakInTest_XCTest() throws {