Explicit api for all modules (and small changes on the way)

buildSrc/build.gradle.kts

@@ -9,7 +9,6 @@ repositories {
 }
 
 dependencies {
-    // We currently cannot use Kotlin 1.5.31: https://github.com/kordlib/kord/issues/399
     implementation(kotlin("gradle-plugin", version = "1.6.10"))
     implementation(kotlin("serialization", version = "1.6.10"))
     implementation("org.jetbrains.dokka", "dokka-gradle-plugin", "1.6.0")

buildSrc/src/main/kotlin/kord-module.gradle.kts

@@ -19,6 +19,10 @@ dependencies {
     testRuntimeOnly(kotlin("test-junit5"))
 }
 
+kotlin {
+    explicitApi()
+}
+
 tasks {
     tasks.getByName("apiCheck") {
         onlyIf { Library.isRelease }

common/src/main/kotlin/Color.kt

@@ -10,14 +10,14 @@ import kotlinx.serialization.encoding.Encoder
 
 
 @Serializable(with = Color.Serializer::class)
-class Color(rgb: Int) {
-    constructor(red: Int, green: Int, blue: Int) : this(rgb(red, green, blue))
+public class Color(rgb: Int) {
+    public constructor(red: Int, green: Int, blue: Int) : this(rgb(red, green, blue))
 
-    val rgb = rgb and 0xFFFFFF
+    public val rgb: Int = rgb and 0xFFFFFF
 
-    val red: Int get() = (rgb shr 16) and 0xFF
-    val green: Int get() = (rgb shr 8) and 0xFF
-    val blue: Int get() = (rgb shr 0) and 0xFF
+    public val red: Int get() = (rgb shr 16) and 0xFF
+    public val green: Int get() = (rgb shr 8) and 0xFF
+    public val blue: Int get() = (rgb shr 0) and 0xFF
 
     init {
         require(this.rgb in MIN_COLOR..MAX_COLOR) { "RGB should be in range of $MIN_COLOR..$MAX_COLOR but was ${this.rgb}" }
@@ -33,7 +33,7 @@ class Color(rgb: Int) {
         return color.rgb == rgb
     }
 
-    companion object {
+    public companion object {
         private const val MIN_COLOR = 0
         private const val MAX_COLOR = 0xFFFFFF
     }
@@ -61,4 +61,4 @@ private fun rgb(red: Int, green: Int, blue: Int): Int {
             (blue and 0xFF) shl 0
 }
 
-val java.awt.Color.kColor get() = Color(rgb)
+public val java.awt.Color.kColor: Color get() = Color(rgb)

common/src/main/kotlin/DiscordBitSet.kt

@@ -14,26 +14,27 @@ import kotlin.math.min
 
 private const val SAFE_LENGTH = 19
 private const val WIDTH = Byte.SIZE_BITS
-fun EmptyBitSet() = DiscordBitSet(0)
 
-@OptIn(ExperimentalUnsignedTypes::class)
+@Suppress("FunctionName")
+public fun EmptyBitSet(): DiscordBitSet = DiscordBitSet(0)
+
 @Serializable(with = DiscordBitSetSerializer::class)
-class DiscordBitSet(internal var data: LongArray) {
+public class DiscordBitSet(internal var data: LongArray) {
 
-    val isEmpty: Boolean
+    public val isEmpty: Boolean
         get() = data.all { it == 0L }
 
-    val value: String
+    public val value: String
         get() {
             val buffer = ByteBuffer.allocate(data.size * Long.SIZE_BYTES)
             buffer.asLongBuffer().put(data.reversedArray())
             return BigInteger(buffer.array()).toString()
         }
 
-    val size: Int
+    public val size: Int
         get() = data.size * WIDTH
 
-    val binary: String
+    public val binary: String
         get() = data.joinToString("") { it.toULong().toString(2) }.reversed().padEnd(8, '0')
 
     override fun equals(other: Any?): Boolean {
@@ -46,54 +47,54 @@ class DiscordBitSet(internal var data: LongArray) {
 
     private fun getOrZero(i: Int) = data.getOrNull(i) ?: 0L
 
-    operator fun get(index: Int): Boolean {
+    public operator fun get(index: Int): Boolean {
         if (index !in 0 until size) return false
         val indexOfWidth = index / WIDTH
         val bitIndex = index % WIDTH
         return data[indexOfWidth] and (1L shl bitIndex) != 0L
     }
 
-    operator fun contains(other: DiscordBitSet): Boolean {
+    public operator fun contains(other: DiscordBitSet): Boolean {
         if (other.size > size) return false
         for (i in other.data.indices) {
             if (data[i] and other.data[i] != other.data[i]) return false
         }
         return true
     }
 
-    operator fun set(index: Int, value: Boolean) {
+    public operator fun set(index: Int, value: Boolean) {
         if (index !in 0 until size) data.copyOf((63 + index) / WIDTH)
         val indexOfWidth = index / WIDTH
         val bitIndex = index % WIDTH
         val bit = if (value) 1L else 0L
         data[index] = data[indexOfWidth] or (bit shl bitIndex)
     }
 
-    operator fun plus(another: DiscordBitSet): DiscordBitSet {
+    public operator fun plus(another: DiscordBitSet): DiscordBitSet {
         val dist = LongArray(data.size)
         data.copyInto(dist)
         val copy = DiscordBitSet(dist)
         copy.add(another)
         return copy
     }
 
-    operator fun minus(another: DiscordBitSet): DiscordBitSet {
+    public operator fun minus(another: DiscordBitSet): DiscordBitSet {
         val dist = LongArray(data.size)
         data.copyInto(dist)
         val copy = DiscordBitSet(dist)
         copy.remove(another)
         return copy
     }
 
-    fun add(another: DiscordBitSet) {
+    public fun add(another: DiscordBitSet) {
         if (another.data.size > data.size) data = data.copyOf(another.data.size)
         for (i in another.data.indices) {
             data[i] = data[i] or another.data[i]
         }
     }
 
 
-    fun remove(another: DiscordBitSet) {
+    public fun remove(another: DiscordBitSet) {
         for (i in 0 until min(data.size, another.data.size)) {
             data[i] = data[i] xor (data[i] and another.data[i])
         }
@@ -111,11 +112,11 @@ class DiscordBitSet(internal var data: LongArray) {
 
 }
 
-fun DiscordBitSet(vararg widths: Long): DiscordBitSet {
+public fun DiscordBitSet(vararg widths: Long): DiscordBitSet {
     return DiscordBitSet(widths)
 }
 
-fun DiscordBitSet(value: String): DiscordBitSet {
+public fun DiscordBitSet(value: String): DiscordBitSet {
     if (value.length <= SAFE_LENGTH) {// fast path
         return DiscordBitSet(longArrayOf(value.toULong().toLong()))
     }
@@ -140,7 +141,7 @@ fun DiscordBitSet(value: String): DiscordBitSet {
 }
 
 
-object DiscordBitSetSerializer : KSerializer<DiscordBitSet> {
+public object DiscordBitSetSerializer : KSerializer<DiscordBitSet> {
     override val descriptor: SerialDescriptor
         get() = PrimitiveSerialDescriptor("DiscordBitSet", PrimitiveKind.STRING)
 
@@ -150,7 +151,5 @@ object DiscordBitSetSerializer : KSerializer<DiscordBitSet> {
 
     override fun serialize(encoder: Encoder, value: DiscordBitSet) {
         encoder.encodeString(value.value)
-
     }
 }
-

common/src/main/kotlin/DiscordTimestamp.kt

@@ -2,13 +2,13 @@ package dev.kord.common
 
 import kotlinx.datetime.Instant
 
-fun Instant.toMessageFormat(style: DiscordTimestampStyle = DiscordTimestampStyle.ShortDateTime) =
+public fun Instant.toMessageFormat(style: DiscordTimestampStyle = DiscordTimestampStyle.ShortDateTime): String =
     "<t:$epochSeconds:${style.style}>"
 
 /**
  * The class representing the [style of a timestamp](https://discord.com/developers/docs/reference#message-formatting-timestamp-styles)
  */
-enum class DiscordTimestampStyle(val style: String) {
+public enum class DiscordTimestampStyle(public val style: String) {
 
     /**
      * For example 16:20