Add support for Kotlin 2.2+ (#1387)

agluszak · web-flow · commit 89fd270f1e77 · 2025-11-14T11:37:19.000-06:00
* Add support for Kotlin 2.2+ with versioned artifact system Kotlin 2.2 removed trove4j and android-extensions jars from the distribution. This change introduces version-specific artifact definitions (artifacts_2.1.bzl and artifacts_2.2.bzl) to handle these differences, with automatic selection based on Kotlin version. Updates default Kotlin version to 2.2. Changes: - Created WriteKotlinArtifacts tool to generate artifact definitions - Added artifacts_2.1.bzl (includes trove4j) and artifacts_2.2.bzl - Updated compiler.bzl to use KOTLIN_STDLIBS from artifacts - Updated default Kotlin version to 2.2.21 - Added custom_compiler_version example to test backward compatibility - Updated jdeps plugin imports to use K2 FIR APIs Fixes #1373 * Fix * Remove artifacts.bzl generation * Fix docs * Fix * kotlinc_opts dictionary -> array * Fix RBE * Fix jetpack_compose * Downgrade rules_proto, upgrade kotlin-build-tools-impl
diff --git a/docs/kotlin.md b/docs/kotlin.md
@@ -600,8 +600,8 @@ Call this in the WORKSPACE file to setup the Kotlin rules.
 | <a id="kotlin_repositories-is_bzlmod"></a>is_bzlmod |  <p align="center"> - </p>   |  `False` |
 | <a id="kotlin_repositories-compiler_repository_name"></a>compiler_repository_name |  for the kotlinc compiler repository.   |  `"com_github_jetbrains_kotlin"` |
 | <a id="kotlin_repositories-ksp_repository_name"></a>ksp_repository_name |  <p align="center"> - </p>   |  `"com_github_google_ksp"` |
-| <a id="kotlin_repositories-compiler_release"></a>compiler_release |  version provider from versions.bzl.   |  `struct(sha256 = "1ba08a8b45da99339a0601134cc037b54cf85e9bc0edbe76dcbd27c2d684a977", url_templates = ["https://github.com/JetBrains/kotlin/releases/download/v{version}/kotlin-compiler-{version}.zip"], version = "2.1.21")` |
-| <a id="kotlin_repositories-ksp_compiler_release"></a>ksp_compiler_release |  (internal) version provider from versions.bzl.   |  `struct(sha256 = "44e965bb067b2bb5cd9184dab2c3dea6e3eab747d341c07645bb4c88f09e49c8", url_templates = ["https://github.com/google/ksp/releases/download/{version}/artifacts.zip"], version = "2.1.21-2.0.1")` |
+| <a id="kotlin_repositories-compiler_release"></a>compiler_release |  version provider from versions.bzl.   |  `struct(sha256 = "a623871f1cd9c938946948b70ef9170879f0758043885bbd30c32f024e511714", url_templates = ["https://github.com/JetBrains/kotlin/releases/download/v{version}/kotlin-compiler-{version}.zip"], version = "2.2.21")` |
+| <a id="kotlin_repositories-ksp_compiler_release"></a>ksp_compiler_release |  (internal) version provider from versions.bzl.   |  `struct(sha256 = "6550f1117d7c9590cc9a5075b92682a218c8e1df4093d7e683d73cc481733dd1", url_templates = ["https://github.com/google/ksp/releases/download/{version}/artifacts.zip"], version = "2.2.21-2.0.4")` |
 
 
 <a id="versions.use_repository"></a>
diff --git a/examples/jetpack_compose/MODULE.bazel b/examples/jetpack_compose/MODULE.bazel
@@ -37,7 +37,7 @@ maven.install(
     name = "maven_rules_kotlin_example",
     aar_import_bzl_label = "@rules_android//rules:rules.bzl",
     artifacts = [
-        "org.jetbrains.kotlin:kotlin-compose-compiler-plugin-embeddable:2.1.20",
+        "org.jetbrains.kotlin:kotlin-compose-compiler-plugin-embeddable:2.2.21",
         "androidx.core:core-ktx:1.15.0",
         "androidx.appcompat:appcompat:1.7.0",
         "androidx.activity:activity-compose:1.7.0",
diff --git a/examples/jetpack_compose/maven_install.json b/examples/jetpack_compose/maven_install.json
@@ -1,7 +1,7 @@
 {
   "__AUTOGENERATED_FILE_DO_NOT_MODIFY_THIS_FILE_MANUALLY": "THERE_IS_NO_DATA_ONLY_ZUUL",
-  "__INPUT_ARTIFACTS_HASH": -84190194,
-  "__RESOLVED_ARTIFACTS_HASH": 1828198228,
+  "__INPUT_ARTIFACTS_HASH": 546178286,
+  "__RESOLVED_ARTIFACTS_HASH": 981137442,
   "artifacts": {
     "androidx.activity:activity": {
       "shasums": {
@@ -958,10 +958,10 @@
     },
     "org.jetbrains.kotlin:kotlin-compose-compiler-plugin-embeddable": {
       "shasums": {
-        "jar": "cf87503abc96914f169c9ed61d30a097578c26b0da2666fdd172ec7cff2fac5d",
-        "sources": "b1cc37d9883f388e1e5abfa6405c9d524baf0408262bc63e1d0592c2c876d747"
+        "jar": "bd6b2db8bbf5ef1c12888ae885b72054e70cea7ef4d6e8bb5eca4faa0539393f",
+        "sources": "0c1a7ad3991be34d03555140c304f15c67748f5be71a8e79dc393b745e442202"
       },
-      "version": "2.1.20"
+      "version": "2.2.21"
     },
     "org.jetbrains.kotlin:kotlin-stdlib": {
       "shasums": {
diff --git a/kotlin/compiler/compiler.bzl b/kotlin/compiler/compiler.bzl
@@ -18,11 +18,11 @@ load("//kotlin/internal:defs.bzl", _KT_COMPILER_REPO = "KT_COMPILER_REPO")
 
 KOTLIN_STDLIBS = [
     "//kotlin/compiler:annotations",
+    "//kotlin/compiler:kotlin-reflect",
     "//kotlin/compiler:kotlin-stdlib",
     "//kotlin/compiler:kotlin-stdlib-jdk7",
     "//kotlin/compiler:kotlin-stdlib-jdk8",
     "//kotlin/compiler:kotlinx-coroutines-core-jvm",
-    "//kotlin/compiler:trove4j",
 ]
 
 def _import_artifacts(artifacts, rule_kind):
diff --git a/src/main/kotlin/BUILD.release.bazel b/src/main/kotlin/BUILD.release.bazel
@@ -12,6 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+load("@com_github_jetbrains_kotlin//:artifacts.bzl", "KOTLIN_STDLIBS")
 load("@rules_java//java:defs.bzl", "java_binary", "java_import")
 
 java_import(
@@ -78,14 +79,7 @@ java_binary(
 
 java_binary(
     name = "jdeps_merger",
-    data = [
-        "@com_github_jetbrains_kotlin//:annotations",
-        "@com_github_jetbrains_kotlin//:kotlin-stdlib",
-        "@com_github_jetbrains_kotlin//:kotlin-stdlib-jdk7",
-        "@com_github_jetbrains_kotlin//:kotlin-stdlib-jdk8",
-        "@com_github_jetbrains_kotlin//:kotlinx-coroutines-core-jvm",
-        "@com_github_jetbrains_kotlin//:trove4j",
-    ],
+    data = KOTLIN_STDLIBS,
     jvm_flags = [
         "-XX:-MaxFDLimit",
     ],
diff --git a/src/main/kotlin/bootstrap.bzl b/src/main/kotlin/bootstrap.bzl
@@ -16,7 +16,7 @@ load("//kotlin:lint.bzl", _ktlint_fix = "ktlint_fix", _ktlint_test = "ktlint_tes
 load("//src/main/starlark/core/compile:rules.bzl", "core_kt_jvm_library")
 load("//third_party:jarjar.bzl", "jar_jar")
 
-def kt_bootstrap_library(name, deps = [], neverlink_deps = [], srcs = [], visibility = [], **kwargs):
+def kt_bootstrap_library(name, deps = [], neverlink_deps = [], srcs = [], visibility = [], kotlinc_opts = [], **kwargs):
     """
     Simple compilation of a kotlin library using a non-persistent worker. The target is a JavaInfo provider.
 
@@ -34,6 +34,7 @@ def kt_bootstrap_library(name, deps = [], neverlink_deps = [], srcs = [], visibi
         srcs = srcs,
         visibility = visibility,
         deps = deps + ["%s_neverlink" % name],
+        kotlinc_opts = kotlinc_opts,
         **kwargs
     )
 
@@ -43,6 +44,7 @@ def kt_bootstrap_library(name, deps = [], neverlink_deps = [], srcs = [], visibi
         visibility = ["//visibility:private"],
         config = "//:ktlint_editorconfig",
         tags = ["no-ide", "ktlint"],
+        **kwargs
     )
 
     _ktlint_fix(
diff --git a/src/main/kotlin/io/bazel/kotlin/builder/cmd/BUILD.bazel b/src/main/kotlin/io/bazel/kotlin/builder/cmd/BUILD.bazel
@@ -19,7 +19,6 @@ kt_bootstrap_binary(
         "//kotlin/compiler:jvm-abi-gen",
         "//kotlin/compiler:kotlin-annotation-processing",
         "//kotlin/compiler:kotlin-compiler",
-        "//kotlin/compiler:kotlin-reflect",
         "//kotlin/compiler:symbol-processing-api",
         "//kotlin/compiler:symbol-processing-cmdline",
         "//src/main/kotlin:jdeps-gen",
diff --git a/src/main/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinJvmTaskExecutor.kt b/src/main/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinJvmTaskExecutor.kt
@@ -63,7 +63,7 @@ class KotlinJvmTaskExecutor
 
       context.execute("compile classes") {
         preprocessedTask.apply {
-          sequenceOf(
+          listOf(
             runCatching {
               context.execute("kotlinc") {
                 if (compileKotlin) {
@@ -80,6 +80,9 @@ class KotlinJvmTaskExecutor
                             inputs.directDependenciesList.forEach {
                               flag("direct_dependencies", it)
                             }
+                            inputs.classpathList.forEach {
+                              flag("full_classpath", it)
+                            }
                             flag("strict_kotlin_deps", info.strictKotlinDeps)
                           }
                         }.given(outputs.jar)
diff --git a/src/main/kotlin/io/bazel/kotlin/generate/BUILD.bazel b/src/main/kotlin/io/bazel/kotlin/generate/BUILD.bazel
@@ -1,19 +1,13 @@
 load("//kotlin:jvm.bzl", "kt_jvm_binary", "kt_jvm_import", "kt_jvm_library")
+load("//kotlin/compiler:compiler.bzl", _KOTLIN_STDLIBS = "KOTLIN_STDLIBS")
 
 # Work around neverlink.
 kt_jvm_import(
     name = "kotlinc_jar",
     jars = [
         "@com_github_jetbrains_kotlin//:kotlin-compiler",
     ],
-    deps = [
-        "//kotlin/compiler:annotations",
-        "//kotlin/compiler:kotlin-stdlib",
-        "//kotlin/compiler:kotlin-stdlib-jdk7",
-        "//kotlin/compiler:kotlin-stdlib-jdk8",
-        "//kotlin/compiler:kotlinx-coroutines-core-jvm",
-        "//kotlin/compiler:trove4j",
-    ],
+    deps = _KOTLIN_STDLIBS,
 )
 
 kt_jvm_library(
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/BUILD.bazel b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/BUILD.bazel
@@ -25,6 +25,9 @@ kt_bootstrap_library(
         "k2/checker/declaration/*.kt",
         "k2/checker/expression/*.kt",
     ]),
+    kotlinc_opts = [
+        "-Xcontext-parameters",
+    ],
     visibility = ["//src:__subpackages__"],
     deps = [
         "//kotlin/compiler:kotlin-compiler",
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/BaseJdepsGenExtension.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/BaseJdepsGenExtension.kt
@@ -16,23 +16,69 @@ abstract class BaseJdepsGenExtension(
   ) {
     val directDeps = configuration.getList(JdepsGenConfigurationKeys.DIRECT_DEPENDENCIES)
     val targetLabel = configuration.getNotNull(JdepsGenConfigurationKeys.TARGET_LABEL)
-    val explicitDeps = createDepsMap(explicitClassesCanonicalPaths)
+    val fullClasspath = configuration.getList(JdepsGenConfigurationKeys.FULL_CLASSPATH)
 
-    doWriteJdeps(directDeps, targetLabel, explicitDeps, implicitClassesCanonicalPaths)
+    // Create mapping from canonical paths to original classpath paths
+    val canonicalToClasspath = createCanonicalToClasspathMap(fullClasspath)
+
+    val explicitDeps = createDepsMap(explicitClassesCanonicalPaths, canonicalToClasspath)
+
+    doWriteJdeps(
+      directDeps,
+      targetLabel,
+      explicitDeps,
+      implicitClassesCanonicalPaths,
+      canonicalToClasspath,
+    )
 
     doStrictDeps(configuration, targetLabel, directDeps, explicitDeps)
   }
 
+  /**
+   * Creates a mapping from canonical filesystem paths to original classpath paths.
+   * This ensures jdeps uses the same paths that were on the compiler's classpath.
+   * When multiple classpath entries point to the same canonical file, prefer runfiles paths.
+   */
+  private fun createCanonicalToClasspathMap(classpathEntries: List<String>): Map<String, String> {
+    val mapping = mutableMapOf<String, String>()
+
+    // Create canonical -> original mapping
+    // Process entries and prefer runfiles paths over execroot paths
+    classpathEntries.forEach { classpathPath ->
+      try {
+        val canonicalPath = File(classpathPath).canonicalPath
+        val existing = mapping[canonicalPath]
+        // Only update if no existing entry, or if current path is a runfiles path
+        // and existing is not
+        if (existing == null ||
+          (classpathPath.contains("/runfiles/") && !existing.contains("/runfiles/"))
+        ) {
+          mapping[canonicalPath] = classpathPath
+        }
+      } catch (e: Exception) {
+        // If we can't resolve, just use the original path for both
+        mapping[classpathPath] = classpathPath
+      }
+    }
+    return mapping
+  }
+
   /**
    * Returns a map of jars to classes loaded from those jars.
+   * Uses the canonical-to-classpath mapping to preserve original classpath paths.
    */
-  private fun createDepsMap(classes: Set<String>): Map<String, List<String>> {
+  private fun createDepsMap(
+    classes: Set<String>,
+    canonicalToClasspath: Map<String, String>,
+  ): Map<String, List<String>> {
     val jarsToClasses = mutableMapOf<String, MutableList<String>>()
     classes.forEach {
       val parts = it.split("!/")
-      val jarPath = parts[0]
-      if (jarPath.endsWith(".jar")) {
-        jarsToClasses.computeIfAbsent(jarPath) { ArrayList() }.add(parts[1])
+      val canonicalJarPath = parts[0]
+      if (canonicalJarPath.endsWith(".jar")) {
+        // Map back to original classpath path
+        val classpathJarPath = canonicalToClasspath[canonicalJarPath] ?: canonicalJarPath
+        jarsToClasses.computeIfAbsent(classpathJarPath) { ArrayList() }.add(parts[1])
       }
     }
     return jarsToClasses
@@ -43,8 +89,9 @@ abstract class BaseJdepsGenExtension(
     targetLabel: String,
     explicitDeps: Map<String, List<String>>,
     implicitClassesCanonicalPaths: Set<String>,
+    canonicalToClasspath: Map<String, String>,
   ) {
-    val implicitDeps = createDepsMap(implicitClassesCanonicalPaths)
+    val implicitDeps = createDepsMap(implicitClassesCanonicalPaths, canonicalToClasspath)
 
     // Build and write out deps.proto
     val jdepsOutput = configuration.getNotNull(JdepsGenConfigurationKeys.OUTPUT_JDEPS)
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/JdepsGenCommandLineProcessor.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/JdepsGenCommandLineProcessor.kt
@@ -24,6 +24,14 @@ class JdepsGenCommandLineProcessor : CommandLineProcessor {
         required = false,
         allowMultipleOccurrences = true,
       )
+    val FULL_CLASSPATH_OPTION: CliOption =
+      CliOption(
+        "full_classpath",
+        "<List>",
+        "Full classpath used for compilation (includes all transitive dependencies and stdlib)",
+        required = false,
+        allowMultipleOccurrences = true,
+      )
     val STRICT_KOTLIN_DEPS_OPTION: CliOption =
       CliOption("strict_kotlin_deps", "<String>", "Report strict deps violations", required = true)
   }
@@ -36,6 +44,7 @@ class JdepsGenCommandLineProcessor : CommandLineProcessor {
         OUTPUT_JDEPS_FILE_OPTION,
         TARGET_LABEL_OPTION,
         DIRECT_DEPENDENCIES_OPTION,
+        FULL_CLASSPATH_OPTION,
         STRICT_KOTLIN_DEPS_OPTION,
       )
 
@@ -52,6 +61,11 @@ class JdepsGenCommandLineProcessor : CommandLineProcessor {
           JdepsGenConfigurationKeys.DIRECT_DEPENDENCIES,
           value,
         )
+      FULL_CLASSPATH_OPTION ->
+        configuration.appendList(
+          JdepsGenConfigurationKeys.FULL_CLASSPATH,
+          value,
+        )
       STRICT_KOTLIN_DEPS_OPTION ->
         configuration.put(
           JdepsGenConfigurationKeys.STRICT_KOTLIN_DEPS,
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/JdepsGenConfigurationKeys.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/JdepsGenConfigurationKeys.kt
@@ -32,4 +32,13 @@ object JdepsGenConfigurationKeys {
     CompilerConfigurationKey.create(
       JdepsGenCommandLineProcessor.DIRECT_DEPENDENCIES_OPTION.description,
     )
+
+  /**
+   * Full classpath used for compilation (includes all transitive dependencies and stdlib).
+   * Used for mapping canonical filesystem paths back to original classpath paths.
+   */
+  val FULL_CLASSPATH: CompilerConfigurationKey<List<String>> =
+    CompilerConfigurationKey.create(
+      JdepsGenCommandLineProcessor.FULL_CLASSPATH_OPTION.description,
+    )
 }
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/BasicDeclarationChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/BasicDeclarationChecker.kt
@@ -11,11 +11,8 @@ import org.jetbrains.kotlin.fir.declarations.toAnnotationClassLikeSymbol
 internal class BasicDeclarationChecker(
   private val classUsageRecorder: ClassUsageRecorder,
 ) : FirBasicDeclarationChecker(MppCheckerKind.Common) {
-  override fun check(
-    declaration: FirDeclaration,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(declaration: FirDeclaration) {
     declaration.annotations.forEach { annotation ->
       annotation.toAnnotationClassLikeSymbol(context.session)?.let {
         classUsageRecorder.recordClass(it, context)
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/CallableChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/CallableChecker.kt
@@ -16,11 +16,8 @@ internal class CallableChecker(
    * Tracks the return type & type parameters of a callable declaration. Function parameters are
    * tracked in [FunctionChecker].
    */
-  override fun check(
-    declaration: FirCallableDeclaration,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(declaration: FirCallableDeclaration) {
     // return type
     declaration.returnTypeRef.let { classUsageRecorder.recordTypeRef(it, context) }
 
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/ClassLikeChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/ClassLikeChecker.kt
@@ -11,11 +11,8 @@ import org.jetbrains.kotlin.fir.resolve.getSuperTypes
 internal class ClassLikeChecker(
   private val classUsageRecorder: ClassUsageRecorder,
 ) : FirClassLikeChecker(MppCheckerKind.Common) {
-  override fun check(
-    declaration: FirClassLikeDeclaration,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(declaration: FirClassLikeDeclaration) {
     declaration.symbol.let { classUsageRecorder.recordClass(it, context) }
     // [recordClass] also handles supertypes, but this marks direct supertypes as explicit
     declaration.symbol.getSuperTypes(context.session, recursive = false).forEach {
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/FileChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/FileChecker.kt
@@ -22,11 +22,8 @@ import org.jetbrains.kotlin.name.ClassId
 internal class FileChecker(
   private val classUsageRecorder: ClassUsageRecorder,
 ) : FirFileChecker(MppCheckerKind.Common) {
-  override fun check(
-    declaration: FirFile,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(declaration: FirFile) {
     declaration.imports.filterIsInstance<FirResolvedImport>().forEach { import ->
       // check for classlike import (class, interface, object, enum, annotation, etc)
       if (import.resolvesToClass(context)) {
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/FunctionChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/declaration/FunctionChecker.kt
@@ -14,11 +14,8 @@ internal class FunctionChecker(
    * Tracks the value parameters of a function declaration. Return type & type parameters are
    * tracked in [CallableChecker].
    */
-  override fun check(
-    declaration: FirFunction,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(declaration: FirFunction) {
     // function parameters
     declaration.valueParameters.forEach { valueParam ->
       valueParam.returnTypeRef.let { classUsageRecorder.recordTypeRef(it, context) }
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/expression/QualifiedAccessChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/expression/QualifiedAccessChecker.kt
@@ -19,11 +19,8 @@ import org.jetbrains.kotlin.fir.types.resolvedType
 internal class QualifiedAccessChecker(
   private val classUsageRecorder: ClassUsageRecorder,
 ) : FirQualifiedAccessExpressionChecker(MppCheckerKind.Common) {
-  override fun check(
-    expression: FirQualifiedAccessExpression,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(expression: FirQualifiedAccessExpression) {
     // track function's owning class
     val resolvedCallableSymbol = expression.toResolvedCallableSymbol()
     resolvedCallableSymbol?.containerSource?.binaryClass()?.let {
diff --git a/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/expression/ResolvedQualifierChecker.kt b/src/main/kotlin/io/bazel/kotlin/plugin/jdeps/k2/checker/expression/ResolvedQualifierChecker.kt
@@ -11,11 +11,8 @@ import org.jetbrains.kotlin.fir.expressions.FirResolvedQualifier
 internal class ResolvedQualifierChecker(
   private val classUsageRecorder: ClassUsageRecorder,
 ) : FirResolvedQualifierChecker(MppCheckerKind.Common) {
-  override fun check(
-    expression: FirResolvedQualifier,
-    context: CheckerContext,
-    reporter: DiagnosticReporter,
-  ) {
+  context(context: CheckerContext, reporter: DiagnosticReporter)
+  override fun check(expression: FirResolvedQualifier) {
     expression.symbol?.let {
       classUsageRecorder.recordClass(it, context)
     }
diff --git a/src/main/starlark/core/compile/cli/compile.bzl b/src/main/starlark/core/compile/cli/compile.bzl
diff --git a/src/main/starlark/core/compile/rules.bzl b/src/main/starlark/core/compile/rules.bzl
diff --git a/src/main/starlark/core/repositories/kotlin/artifacts.bzl b/src/main/starlark/core/repositories/kotlin/artifacts.bzl
diff --git a/src/main/starlark/core/repositories/setup.bzl b/src/main/starlark/core/repositories/setup.bzl
diff --git a/src/main/starlark/core/repositories/versions.bzl b/src/main/starlark/core/repositories/versions.bzl
diff --git a/src/test/kotlin/io/bazel/kotlin/builder/BUILD b/src/test/kotlin/io/bazel/kotlin/builder/BUILD
diff --git a/src/test/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinBuilderJvmBasicTest.java b/src/test/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinBuilderJvmBasicTest.java
diff --git a/src/test/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinBuilderJvmStrictDepsTest.kt b/src/test/kotlin/io/bazel/kotlin/builder/tasks/jvm/KotlinBuilderJvmStrictDepsTest.kt
