Update Kotlin core dependencies

gradle/libs.versions.toml

@@ -1,12 +1,12 @@
 [versions]
-kotlin = "1.9.10"
+kotlin = "1.9.22"
 jetbrains-annotations = "24.0.1"
 save-cli = "0.3.10"
 ktor = "2.3.6"
 okio = "3.3.0"
-serialization = "1.6.0"
-kotlinx-datetime = "0.4.1"
-kotlinx-coroutines = "1.7.3"
+serialization = "1.6.3"
+kotlinx-datetime = "0.5.0"
+kotlinx-coroutines = "1.8.0"
 kotlin-wrappers = "1.0.0-pre.634"
 spring-boot = "2.7.17"
 spring-cloud = "3.1.9"

save-agent/src/commonMain/kotlin/com/saveourtool/save/agent/SaveAgent.kt

@@ -51,12 +51,12 @@ class SaveAgent(
     /**
      * The current [AgentState] of this agent. Initial value corresponds to the period when agent needs to finish its configuration.
      */
-    val state = GenericAtomicReference(AgentState.BUSY)
+    val state = createGenericAtomicReference(AgentState.BUSY)
 
     // fixme (limitation of old MM): can't use atomic reference to Instant here, because when using `Clock.System.now()` as an assigned value
     // Kotlin throws `kotlin.native.concurrent.InvalidMutabilityException: mutation attempt of frozen kotlinx.datetime.Instant...`
-    private val executionStartSeconds = AtomicLong(0L)
-    private val saveProcessJob: GenericAtomicReference<Job?> = GenericAtomicReference(null)
+    private val executionStartSeconds = createAtomicLong(0L)
+    private val saveProcessJob: GenericAtomicReference<Job?> = createGenericAtomicReference(null)
     private val backgroundContext = newSingleThreadContext("background")
     private val saveProcessContext = newSingleThreadContext("save-process")
     private val reportFormat = Json {

save-agent/src/commonMain/kotlin/com/saveourtool/save/agent/utils/HttpUtils.kt

@@ -8,7 +8,7 @@ import com.saveourtool.save.agent.AgentState
 import com.saveourtool.save.agent.SaveAgent
 import com.saveourtool.save.core.logging.logWarn
 import com.saveourtool.save.core.utils.runIf
-import com.saveourtool.save.utils.AtomicLong
+import com.saveourtool.save.utils.createAtomicLong
 import com.saveourtool.save.utils.failureOrNotOk
 import com.saveourtool.save.utils.fs
 import com.saveourtool.save.utils.notOk
@@ -82,7 +82,7 @@ internal suspend fun HttpClient.download(url: String, file: Path): Result<HttpRe
         .execute { httpResponse ->
             if (httpResponse.status.isSuccess()) {
                 val channel: ByteReadChannel = httpResponse.body()
-                val totalBytes = AtomicLong(0L)
+                val totalBytes = createAtomicLong(0L)
                 while (!channel.isClosedForRead) {
                     val packet = channel.readRemaining(DEFAULT_HTTP_BUFFER_SIZE.toLong())
                     while (!packet.isEmpty) {

save-cloud-common/src/commonMain/kotlin/com/saveourtool/save/utils/PlatformUtils.kt

@@ -5,7 +5,6 @@
 package com.saveourtool.save.utils
 
 import com.saveourtool.save.core.logging.logDebug
-import com.saveourtool.save.core.utils.GenericAtomicReference
 
 import kotlin.time.Duration
 import kotlin.time.DurationUnit
@@ -14,7 +13,7 @@
 /**
  * Atomic values
  */
-expect class AtomicLong(value: Long) {
+interface AtomicLong {
     /**
      * @return value
      */
@@ -33,12 +32,15 @@
 }
 
 /**
- *  Class that holds value and shares atomic reference to the value
- *
- *  @param valueToStore value to store
+ * @param value
+ * @return [AtomicLong] with initial value [value]
+ */
+expect fun createAtomicLong(value: Long): AtomicLong
+
+/**
+ * Class that holds value and shares atomic reference to the value
  */
-@Suppress("USE_DATA_CLASS")
-expect class GenericAtomicReference<T>(valueToStore: T) {
+interface GenericAtomicReference<T> {
     /**
      * @return stored value
      */
@@ -50,6 +52,13 @@
     fun set(newValue: T)
 }
 
+
+/**
+ * @param valueToStore
+ * @return create [GenericAtomicReference] with initial value [valueToStore]
+ */
+expect fun <T> createGenericAtomicReference(valueToStore: T): GenericAtomicReference<T>
+
 /**
  * A wrapper around a value of type [T] that caches it for [expirationTimeSeconds] and then recalculates
  * using [valueGetter]
@@ -62,8 +71,8 @@
     private val valueGetter: () -> T,
 ) {
     private val expirationTimeSeconds = expirationTime.toLong(DurationUnit.SECONDS)
-    private val lastUpdateTimeSeconds = AtomicLong(0)
-    private val value: GenericAtomicReference<T> = GenericAtomicReference(valueGetter())
+    private val lastUpdateTimeSeconds = createAtomicLong(0)
+    private val value: GenericAtomicReference<T> = createGenericAtomicReference(valueGetter())
 
     /**
      * @return cached value or refreshes the value and returns it

save-cloud-common/src/jsMain/kotlin/com/saveourtool/save/utils/PlatformUtils.kt

@@ -4,22 +4,18 @@
 
 package com.saveourtool.save.utils
 
-actual class AtomicLong actual constructor(value: Long) {
-    actual fun get(): Long = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
+actual fun createAtomicLong(value: Long): AtomicLong = object : AtomicLong {
+    override fun get(): Long = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
 
-    actual fun set(newValue: Long) {
-        throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
-    }
+    override fun set(newValue: Long) = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
 
-    actual fun addAndGet(delta: Long): Long = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
+    override fun addAndGet(delta: Long): Long = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
 }
 
-@Suppress("USE_DATA_CLASS")
-actual class GenericAtomicReference<T> actual constructor(valueToStore: T) {
-    actual fun get(): T = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
-    actual fun set(newValue: T) {
-        throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
-    }
+actual fun <T> createGenericAtomicReference(valueToStore: T): GenericAtomicReference<T> = object : GenericAtomicReference<T> {
+    override fun get(): T = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
+
+    override fun set(newValue: T) = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)
 }
 
 actual fun getenv(envName: String): String? = throw NotImplementedError(NOT_IMPLEMENTED_ON_JS)

save-cloud-common/src/jvmMain/kotlin/com/saveourtool/save/utils/PlatformUtils.kt

@@ -7,13 +7,19 @@
 
 package com.saveourtool.save.utils
 
-actual typealias AtomicLong = java.util.concurrent.atomic.AtomicLong
+actual fun createAtomicLong(value: Long): AtomicLong = object : AtomicLong {
+    private val holder = java.util.concurrent.atomic.AtomicLong(value)
+    override fun get(): Long = holder.get()
 
-@Suppress("USE_DATA_CLASS")
-actual class GenericAtomicReference<T> actual constructor(valueToStore: T) {
+    override fun set(newValue: Long) = holder.set(newValue)
+
+    override fun addAndGet(delta: Long): Long = holder.addAndGet(delta)
+}
+
+actual fun <T> createGenericAtomicReference(valueToStore: T): GenericAtomicReference<T> = object : GenericAtomicReference<T> {
     private val holder: java.util.concurrent.atomic.AtomicReference<T> = java.util.concurrent.atomic.AtomicReference(valueToStore)
-    actual fun get(): T = holder.get()
-    actual fun set(newValue: T) {
+    override fun get(): T = holder.get()
+    override fun set(newValue: T) {
         holder.set(newValue)
     }
 }

save-cloud-common/src/nativeMain/kotlin/com/saveourtool/save/utils/PlatformUtils.kt

@@ -7,23 +7,22 @@ package com.saveourtool.save.utils
 import kotlinx.cinterop.ExperimentalForeignApi
 import kotlinx.cinterop.toKString
 
-actual class AtomicLong actual constructor(value: Long) {
-    private val kotlinAtomicLong = kotlin.native.concurrent.AtomicLong(value)
+actual fun createAtomicLong(value: Long): AtomicLong = object : AtomicLong {
+    private val kotlinAtomicLong = kotlin.concurrent.AtomicLong(value)
 
-    actual fun get(): Long = kotlinAtomicLong.value
+    override fun get(): Long = kotlinAtomicLong.value
 
-    actual fun set(newValue: Long) {
+    override fun set(newValue: Long) {
         kotlinAtomicLong.value = newValue
     }
 
-    actual fun addAndGet(delta: Long): Long = kotlinAtomicLong.addAndGet(delta)
+    override fun addAndGet(delta: Long): Long = kotlinAtomicLong.addAndGet(delta)
 }
 
-@Suppress("USE_DATA_CLASS")
-actual class GenericAtomicReference<T> actual constructor(valueToStore: T) {
+actual fun <T> createGenericAtomicReference(valueToStore: T): GenericAtomicReference<T> = object : GenericAtomicReference<T> {
     private val holder: kotlin.concurrent.AtomicReference<T> = kotlin.concurrent.AtomicReference(valueToStore)
-    actual fun get(): T = holder.value
-    actual fun set(newValue: T) {
+    override fun get(): T = holder.value
+    override fun set(newValue: T) {
         holder.value = newValue
     }
 }