Changes for pull request comments

Author: zwiora

build.gradle.kts

@@ -49,13 +49,13 @@ dependencies {
     implementation(libs.kotlin.reflect)
     implementation(libs.bundles.kotlin.stdlib)
     implementation(libs.kotlin.script.runtime)
+    implementation(libs.kotlin.build.tools.api)
+    implementation(libs.kotlin.build.tools.impl)
+    implementation(libs.kotlin.compiler.embeddable)
+    implementation(libs.kotlin.tooling.core)
     implementation(project(":executors", configuration = "default"))
     implementation(project(":common", configuration = "default"))
     implementation(project(":dependencies"))
-    implementation("org.jetbrains.kotlin:kotlin-build-tools-api:${libs.versions.kotlin.get()}")
-    implementation("org.jetbrains.kotlin:kotlin-build-tools-impl:${libs.versions.kotlin.get()}")
-    implementation("org.jetbrains.kotlin:kotlin-compiler-embeddable:${libs.versions.kotlin.get()}")
-    implementation("org.jetbrains.kotlin:kotlin-tooling-core:${libs.versions.kotlin.get()}")
 
     testImplementation(libs.kotlin.test)
     testImplementation("org.springframework.boot:spring-boot-starter-test") {

common/build.gradle.kts

@@ -3,5 +3,5 @@ plugins {
 }
 
 dependencies {
-    implementation("org.jetbrains.kotlin:kotlin-compiler-embeddable:${libs.versions.kotlin.get()}")
+    implementation(libs.kotlin.compiler.embeddable)
 }

gradle/libs.versions.toml

@@ -27,7 +27,9 @@ kotlin-stdlib-js = { group = "org.jetbrains.kotlin", name = "kotlin-stdlib-js",
 kotlin-stdlib-wasm-js = { group = "org.jetbrains.kotlin", name = "kotlin-stdlib-wasm-js", version.ref = "kotlin" }
 kotlin-test = { group = "org.jetbrains.kotlin", name = "kotlin-test", version.ref = "kotlin" }
 kotlin-test-junit = { group = "org.jetbrains.kotlin", name = "kotlin-test-junit", version.ref = "kotlin" }
-kotlin-compiler = { group = "org.jetbrains.kotlin", name = "kotlin-compiler", version.ref = "kotlin" }
+kotlin-build-tools-api = { group = "org.jetbrains.kotlin", name = "kotlin-build-tools-api", version.ref = "kotlin"}
+kotlin-build-tools-impl = { group = "org.jetbrains.kotlin", name = "kotlin-build-tools-impl", version.ref = "kotlin"}
+kotlin-compiler-embeddable = { group = "org.jetbrains.kotlin", name = "kotlin-compiler-embeddable", version.ref = "kotlin" }
 kotlin-tooling-core = { group = "org.jetbrains.kotlin", name = "kotlin-tooling-core", version.ref = "kotlin" }
 kotlin-compiler-arguments-description = { group = "org.jetbrains.kotlin", name = "kotlin-compiler-arguments-description", version.ref = "kotlin" }
 kotlin-script-runtime = { group = "org.jetbrains.kotlin", name = "kotlin-script-runtime", version.ref = "kotlin" }

indexation/build.gradle.kts

@@ -5,18 +5,38 @@ import com.compiler.server.model.ProjectSeveriry
 import com.compiler.server.model.TextInterval
 import org.jetbrains.kotlin.buildtools.api.KotlinLogger
 
+
+/**
+ * CompilationLogger is used for collecting logs from compilation with kotlin-build-tools-api.
+ * It is passed as an argument while executing a compilation operation. It collects logs returned
+ * during compilation, interprets returned strings, and saves logs in the map based on their severity.
+ * The map is later used to provide information about warnings and errors in the user's code.
+ *
+ * KotlinLogger interface will be changed in the future to contain more log details.
+ * Implementation of the CompilationLogger should be therefore updated after KT-80963 is implemented.
+ *
+ * @property isDebugEnabled A flag to indicate whether debug-level logging is enabled for the logger.
+ *                          If true, all messages are printed to the standard output.
+ */
 class CompilationLogger(
     override val isDebugEnabled: Boolean = false,
 ) : KotlinLogger {
 
+    /**
+     * Stores a collection of compilation logs organized by file paths.
+     *
+     * The map keys represent file paths as strings, and the associated values are mutable lists of
+     * `ErrorDescriptor` objects containing details about compilation issues, such as error messages,
+     * intervals, severity, and optional class names.
+     */
     var compilationLogs: Map<String, MutableList<ErrorDescriptor>> = emptyMap()
 
     override fun debug(msg: String) {
         if (isDebugEnabled) println("[DEBUG] $msg")
     }
 
     override fun error(msg: String, throwable: Throwable?) {
-        if (isDebugEnabled) System.err.println("[ERROR] $msg" + (throwable?.let { ": ${it.message}" } ?: ""))
+        if (isDebugEnabled) println("[ERROR] $msg" + (throwable?.let { ": ${it.message}" } ?: ""))
         try {
             addCompilationLog(msg, ProjectSeveriry.ERROR, classNameOverride = null)
         } catch (_: Exception) {}
@@ -31,12 +51,22 @@ class CompilationLogger(
     }
 
     override fun warn(msg: String, throwable: Throwable?) {
-        if (isDebugEnabled) System.err.println("[WARN] $msg" + (throwable?.let { ": ${it.message}" } ?: ""))
+        if (isDebugEnabled) println("[WARN] $msg" + (throwable?.let { ": ${it.message}" } ?: ""))
         try {
             addCompilationLog(msg, ProjectSeveriry.WARNING, classNameOverride = "WARNING")
         } catch (_: Exception) {}
     }
 
+
+    /**
+     * Adds a compilation log entry to the `compilationLogs` map based on the sting log.
+     *
+     * @param msg The raw log message containing information about the compilation event,
+     *            including the file path and error details.
+     * @param severity The severity level of the compilation event, represented by the `ProjectSeveriry` enum.
+     * @param classNameOverride An optional override for the class name that will be recorded in the log.
+     *                          If null, it will be derived from the file path in the message.
+     */
     private fun addCompilationLog(msg: String, severity: ProjectSeveriry, classNameOverride: String?) {
         val path = msg.split(" ")[0]
         val className = path.split("/").last().split(".").first()