Bump dependencies and add Writer skill#622
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 20a6eca5cd
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (this is KotlinJvmCompilerOptions) { | ||
| jvmDefault.set(JvmDefaultMode.ENABLE) | ||
| } |
There was a problem hiding this comment.
Apply jvmDefault to KMP JVM compiler options
setFreeCompilerArgs() now sets JVM default mode only when this is KotlinJvmCompilerOptions, but in buildSrc/src/main/kotlin/kmp-module.gradle.kts this helper is invoked from the top-level compilerOptions {} block (common options) and not in the jvm { compilerOptions {} } block. As a result, multiplatform JVM compilations lose the previous -Xjvm-default=all behavior and will emit different interface-method bytecode, which can break binary compatibility for published KMP JVM artifacts consumed from Java/Kotlin.
Useful? React with 👍 / 👎.
This PR updates the local and external dependencies, brining latest
buildSrcchanges from Base Libraries. Also, the PR adds the Writer agent skill recently adopted in Validation.