refactor: improve import specificity and simplify checks for StructuralKind handling

Author: rasros

src/commonMain/kotlin/com/eignex/kencode/Base85.kt

@@ -1,6 +1,6 @@
 package com.eignex.kencode
 
-val ASCII85 =
+const val ASCII85 =
     "!\"#$%&'()*+,-./0123456789:;<=>?@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_`abcdefghijklmnopqrstu"
 
 /**
@@ -68,7 +68,7 @@ open class Base85(private val alphabet: CharArray) : ByteEncoding {
 
         val fullGroups = len / 5
         val rem = len % 5
-        if (rem == 1) throw IllegalArgumentException("Invalid ASCII85 length")
+        require(rem != 1) { "Invalid ASCII85 length" }
 
         val extraBytes = if (rem == 0) 0 else (rem - 1)
         val out = ByteArray(fullGroups * 4 + extraBytes)

src/commonMain/kotlin/com/eignex/kencode/PackedContext.kt

@@ -1,7 +1,9 @@
 package com.eignex.kencode
 
 import kotlinx.serialization.ExperimentalSerializationApi
-import kotlinx.serialization.descriptors.*
+import kotlinx.serialization.descriptors.PrimitiveKind
+import kotlinx.serialization.descriptors.SerialDescriptor
+import kotlinx.serialization.descriptors.StructureKind
 
 /**
  * Schema-derived boolean/nullable metadata for a single class descriptor.

src/commonMain/kotlin/com/eignex/kencode/PackedDecoder.kt

@@ -2,7 +2,10 @@ package com.eignex.kencode
 
 import kotlinx.serialization.DeserializationStrategy
 import kotlinx.serialization.ExperimentalSerializationApi
-import kotlinx.serialization.descriptors.*
+import kotlinx.serialization.descriptors.PolymorphicKind
+import kotlinx.serialization.descriptors.PrimitiveKind
+import kotlinx.serialization.descriptors.SerialDescriptor
+import kotlinx.serialization.descriptors.StructureKind
 import kotlinx.serialization.encoding.CompositeDecoder
 import kotlinx.serialization.encoding.Decoder
 import kotlinx.serialization.modules.EmptySerializersModule
@@ -334,8 +337,9 @@ class PackedDecoder internal constructor(
 
         val kind = deserializer.descriptor.kind
         val isInline = deserializer.descriptor.isInline
+        val isStructural = kind is StructureKind || kind is PolymorphicKind
 
-        if (isNullableCollection && deserializer.descriptor.isNullable && !isInline && (kind is StructureKind || kind is PolymorphicKind)) {
+        if (isNullableCollection && deserializer.descriptor.isNullable && !isInline && isStructural) {
             val isNotNull = decodeNotNullMark()
             if (!isNotNull) {
                 currentIndex = -1
@@ -351,7 +355,7 @@ class PackedDecoder internal constructor(
             return value
         }
 
-        if (!isInline && (kind is StructureKind || kind is PolymorphicKind)) {
+        if (!isInline && isStructural) {
             // Share the HeaderContext only when the current decoder is in merged mode
             // and the child is a non-nullable, non-inline CLASS/OBJECT in a non-collection context.
             val shouldShareCtx = shouldMergeChildCtx(

src/commonMain/kotlin/com/eignex/kencode/PackedEncoder.kt

@@ -2,7 +2,9 @@ package com.eignex.kencode
 
 import kotlinx.serialization.ExperimentalSerializationApi
 import kotlinx.serialization.SerializationStrategy
-import kotlinx.serialization.descriptors.*
+import kotlinx.serialization.descriptors.PrimitiveKind
+import kotlinx.serialization.descriptors.SerialDescriptor
+import kotlinx.serialization.descriptors.StructureKind
 import kotlinx.serialization.encoding.CompositeEncoder
 import kotlinx.serialization.encoding.Encoder
 import kotlinx.serialization.modules.EmptySerializersModule

src/commonMain/kotlin/com/eignex/kencode/PackedFormat.kt

@@ -1,6 +1,9 @@
 package com.eignex.kencode
 
-import kotlinx.serialization.*
+import kotlinx.serialization.BinaryFormat
+import kotlinx.serialization.DeserializationStrategy
+import kotlinx.serialization.ExperimentalSerializationApi
+import kotlinx.serialization.SerializationStrategy
 import kotlinx.serialization.modules.EmptySerializersModule
 import kotlinx.serialization.modules.SerializersModule
 

src/commonTest/kotlin/com/eignex/kencode/CrcTest.kt

@@ -1,6 +1,8 @@
 package com.eignex.kencode
 
-import kotlin.test.*
+import kotlin.test.Test
+import kotlin.test.assertContentEquals
+import kotlin.test.assertEquals
 
 class CrcTest {
 

src/commonTest/kotlin/com/eignex/kencode/PackedFormatTest.kt

@@ -3,10 +3,14 @@
 package com.eignex.kencode
 
 import kotlinx.serialization.*
-import kotlinx.serialization.builtins.*
+import kotlinx.serialization.builtins.ListSerializer
+import kotlinx.serialization.builtins.nullable
+import kotlinx.serialization.builtins.serializer
 import kotlinx.serialization.protobuf.ProtoIntegerType
 import kotlinx.serialization.protobuf.ProtoType
-import kotlin.test.*
+import kotlin.test.Test
+import kotlin.test.assertEquals
+import kotlin.test.assertFailsWith
 import kotlin.time.Duration.Companion.milliseconds
 import kotlin.time.Duration.Companion.minutes
 import kotlin.time.Duration.Companion.seconds

src/commonTest/kotlin/com/eignex/kencode/PackedUtilsTest.kt

@@ -2,7 +2,7 @@ package com.eignex.kencode
 
 import kotlin.test.*
 
-class BitPackingTest {
+class PackedUtilsTest {
 
     @Test
     fun `packFlags simple bit pattern`() {

src/commonTest/kotlin/com/eignex/kencode/TestClasses.kt

@@ -4,7 +4,9 @@ package com.eignex.kencode
 
 import kotlinx.serialization.Serializable
 import kotlin.jvm.JvmInline
-import kotlin.time.*
+import kotlin.time.Duration
+import kotlin.time.ExperimentalTime
+import kotlin.time.Instant
 
 @Serializable
 data class SimpleIntsAndBooleans(

src/jvmTest/kotlin/com/eignex/kencode/Examples.kt

@@ -1,6 +1,9 @@
 package com.eignex.kencode
 
-import kotlinx.serialization.*
+import kotlinx.serialization.ExperimentalSerializationApi
+import kotlinx.serialization.Serializable
+import kotlinx.serialization.decodeFromString
+import kotlinx.serialization.encodeToString
 import kotlinx.serialization.protobuf.ProtoBuf
 import org.bouncycastle.jce.provider.BouncyCastleProvider
 import java.security.SecureRandom
@@ -58,8 +61,7 @@ class Examples {
         // This encodes the object into the string:
         // 02waa1a8
 
-        val decodedState =
-            EncodedFormat.decodeFromString<JobState>(encodedState)
+        EncodedFormat.decodeFromString<JobState>(encodedState)
     }
 
     @Test