Kotlin 1.9.20 update (#230)

* Bumped up the Kotlin version to 1.9.2
* Configure animalsniffer only when the java-base is applied
* Suppress actual/expect classes warning
* Get rid of default parameters from exception expect's ctors
* Setup wasmJs source sets to use old wasm directory
* Updated NodeJs version to v21.0.0-canary (the one with V8 supporting required GC features)

Author: fzhinkin

build-logic/src/main/kotlin/kotlinx/io/conventions/kotlinx-io-android-compat.gradle.kts

@@ -5,7 +5,7 @@
 
 import ru.vyarus.gradle.plugin.animalsniffer.AnimalSnifferExtension
 
-pluginManager.withPlugin("java") {
+pluginManager.withPlugin("org.gradle.java-base") {
     apply(plugin = "ru.vyarus.animalsniffer")
 
     configure<AnimalSnifferExtension> {

build-logic/src/main/kotlin/kotlinx/io/conventions/kotlinx-io-multiplatform.gradle.kts

@@ -6,6 +6,7 @@
 import org.jetbrains.kotlin.gradle.dsl.KotlinMultiplatformExtension
 import org.jetbrains.kotlin.gradle.plugin.KotlinSourceSet
 import org.jetbrains.kotlin.gradle.targets.js.nodejs.NodeJsRootExtension
+import org.jetbrains.kotlin.gradle.targets.js.npm.tasks.KotlinNpmInstallTask
 import kotlin.jvm.optionals.getOrNull
 
 plugins {
@@ -38,7 +39,7 @@ kotlin {
     }
 
     @OptIn(org.jetbrains.kotlin.gradle.targets.js.dsl.ExperimentalWasmDsl::class)
-    wasm {
+    wasmJs {
         nodejs()
         //  Disabled because we can't exclude some tests: https://youtrack.jetbrains.com/issue/KT-58291
         // browser()
@@ -89,6 +90,12 @@ kotlin {
         createSourceSet("linuxTest", parent = unixTest, children = linuxTargets())
         createSourceSet("androidMain", parent = unixMain, children = androidTargets())
         createSourceSet("androidTest", parent = unixTest, children = androidTargets())
+        getByName("wasmJsMain") {
+            kotlin.srcDir(File(File(projectDir, "wasm"), "src"))
+        }
+        getByName("wasmJsTest") {
+            kotlin.srcDir(File(File(projectDir, "wasm"), "test"))
+        }
     }
 }
 
@@ -192,5 +199,10 @@ fun androidTargets() = listOf(
 )
 
 rootProject.the<NodeJsRootExtension>().apply {
-    nodeVersion = "20.4.0"
+    nodeVersion = "21.0.0-v8-canary202310177990572111"
+    nodeDownloadBaseUrl = "https://nodejs.org/download/v8-canary"
+}
+
+rootProject.tasks.withType<KotlinNpmInstallTask>().configureEach {
+    args.add("--ignore-engines")
 }

build.gradle.kts

@@ -4,6 +4,7 @@
  */
 
 import kotlinx.kover.gradle.plugin.dsl.MetricType
+import org.jetbrains.kotlin.gradle.dsl.KotlinJsCompile
 import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
 import org.jetbrains.kotlin.gradle.tasks.KotlinNativeCompile
 
@@ -27,11 +28,19 @@ subprojects {
         kotlinOptions {
             allWarningsAsErrors = true
             freeCompilerArgs += "-Xjvm-default=all"
+            freeCompilerArgs += "-Xexpect-actual-classes"
         }
     }
     tasks.withType<KotlinNativeCompile>().configureEach {
         kotlinOptions {
             allWarningsAsErrors = true
+            freeCompilerArgs += "-Xexpect-actual-classes"
+        }
+    }
+    tasks.withType<KotlinJsCompile>().configureEach {
+        kotlinOptions {
+            allWarningsAsErrors = true
+            freeCompilerArgs += "-Xexpect-actual-classes"
         }
     }
 }

core/build.gradle.kts

@@ -32,7 +32,7 @@ kotlin {
     }
 
     @OptIn(org.jetbrains.kotlin.gradle.targets.js.dsl.ExperimentalWasmDsl::class)
-    wasm {
+    wasmJs {
         nodejs {
             testTask(Action {
                 useMocha {

core/common/src/-CommonPlatform.kt

@@ -27,13 +27,16 @@ internal expect fun String.asUtf8ToByteArray(): ByteArray
  * Signals about a general issue occurred during I/O operation.
  */
 public expect open class IOException(message: String?, cause: Throwable?) : Exception {
-    public constructor(message: String? = null)
+    public constructor()
+    public constructor(message: String?)
 }
 
 /**
  * Signals that the end of the file or stream was reached unexpectedly during an input operation.
  */
-public expect open class EOFException(message: String? = null) : IOException
+public expect open class EOFException(message: String?) : IOException {
+    public constructor()
+}
 
 
 // There is no actual AutoCloseable on JVM (https://youtrack.jetbrains.com/issue/KT-55777),

core/js/src/-PlatfromJs.kt

@@ -14,6 +14,10 @@ public actual open class IOException actual constructor(
     cause: Throwable?
 ) : Exception(message, cause) {
     public actual constructor(message: String?) : this(message, null)
+
+    public actual constructor() : this(null)
 }
 
-public actual open class EOFException actual constructor(message: String?) : IOException(message)
+public actual open class EOFException actual constructor(message: String?) : IOException(message) {
+    public actual constructor() : this(null)
+}

core/jvm/src/RawSink.kt

@@ -21,8 +21,6 @@
 package kotlinx.io
 
 import java.io.Flushable
-
-@OptIn(ExperimentalStdlibApi::class)
 public actual interface RawSink : AutoCloseableAlias, Flushable {
     public actual fun write(source: Buffer, byteCount: Long)
 

core/native/src/-NonJvmPlatform.kt

@@ -29,6 +29,10 @@ public actual open class IOException actual constructor(
     cause: Throwable?
 ) : Exception(message, cause) {
     public actual constructor(message: String?) : this(message, null)
+
+    public actual constructor() : this(null)
 }
 
-public actual open class EOFException actual constructor(message: String?) : IOException(message)
+public actual open class EOFException actual constructor(message: String?) : IOException(message) {
+    public actual constructor() : this(null)
+}

core/wasm/src/-PlatformWasm.kt

@@ -14,6 +14,10 @@ public actual open class IOException actual constructor(
     cause: Throwable?
 ) : Exception(message, cause) {
     public actual constructor(message: String?) : this(message, null)
+
+    public actual constructor() : this(null)
 }
 
-public actual open class EOFException actual constructor(message: String?) : IOException(message)
+public actual open class EOFException actual constructor(message: String?) : IOException(message) {
+    public actual constructor() : this(null)
+}

gradle.properties

@@ -8,3 +8,4 @@ version=0.3.1-SNAPSHOT
 kotlin.code.style=official
 org.gradle.jvmargs=-Xmx4G
 nativeBenchmarksEnabled=false
+kotlin.mpp.applyDefaultHierarchyTemplate=false