Add coil3 extension

[lightsummer233]

[Copilot]

Pull request overview

This PR adds a new kermit-coil3 extension module so Coil 3’s logging can be routed into Kermit, plus the necessary Gradle wiring to include/publish the new extension.

Changes:

• Register the new :kermit-coil3 Gradle module and add Coil 3 to the version catalog.
• Introduce KermitCoil3Logger implementing Coil 3’s Logger, plus level/Severity conversion extensions.
• Add module build configuration, public API dumps, and end-user README documentation.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
settings.gradle.kts	Includes the new :kermit-coil3 project and maps it to extensions/kermit-coil3.
gradle/libs.versions.toml	Adds Coil 3 version and coil3-core library alias for dependency management.
extensions/kermit-coil3/src/commonMain/kotlin/co/touchlab/kermit/coil3/KermitCoil3Logger.kt	Adds a Coil 3 Logger implementation backed by a Kermit Logger.
extensions/kermit-coil3/src/commonMain/kotlin/co/touchlab/kermit/coil3/Extensions.kt	Adds public conversion extensions between Coil 3 Logger.Level and Kermit Severity.
extensions/kermit-coil3/README.md	Documents how to wire KermitCoil3Logger into a Coil 3 ImageLoader.
extensions/kermit-coil3/build.gradle.kts	Adds the new KMP extension module build/publish configuration and targets.
extensions/kermit-coil3/api/jvm/kermit-coil3.api	Adds JVM API dump for binary compatibility tracking.
extensions/kermit-coil3/api/android/kermit-coil3.api	Adds Android API dump for binary compatibility tracking.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[faogustavo]

Thanks for the PR. This looks good. Just adding some minor requests. Let us know if you can work on those.

[faogustavo]

LGTM. @KevinSchildhorn can we have a second pair of eyes here? :D

[lightsummer233]

I've been thinking, do we need to add a separator parameter in the constructor for joining the Kermit tag and Coil tag? Right now the / is hardcoded as part of the Kermit tag itself (e.g. "Coil3/"), so when Coil's tag is empty, we end up with an awkward trailing separator. That said, I think those throwables from Coil without a tag could still benefit from having some tag. So maybe we should expose the separator as a constructor parameter instead of hardcoding it?

[KevinSchildhorn]

Can we rename this to KermitCoilLogger, we don't need to specify the version. Same with the package name and module. Places where it says coil3 it should just say coil. Otherwise it looks good.

[KevinSchildhorn]

I've been thinking, do we need to add a separator parameter in the constructor for joining the Kermit tag and Coil tag? Right now the / is hardcoded as part of the Kermit tag itself (e.g. "Coil3/"), so when Coil's tag is empty, we end up with an awkward trailing separator. That said, I think those throwables from Coil without a tag could still benefit from having some tag. So maybe we should expose the separator as a constructor parameter instead of hardcoding it?

That sounds alright to me, or maybe a default coil tag. I agree it could be an awkward separator.

[lightsummer233]

Can we rename this to KermitCoilLogger, we don't need to specify the version. Same with the package name and module. Places where it says coil3 it should just say coil. Otherwise it looks good.

Done

[lightsummer233]

By the way, it looks like org.jetbrains.kotlinx.binary-compatibility-validator doesn’t support AGP 9 yet (see Kotlin/binary-compatibility-validator#315), so I haven’t been able to dump the Android API for now.