Skip to content

feat: Add multi-window console logging capability #1235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
tangcent merged 1 commit into from
Apr 14, 2025
Merged

feat: Add multi-window console logging capability #1235

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion gradle.properties
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@ plugin_version=2.8.0.212.0
kotlin.code.style=official
kotlin_version=2.1.0
junit_version=5.9.2
itangcent_intellij_version=1.7.55
itangcent_intellij_version=1.7.7
9 changes: 4 additions & 5 deletions idea-plugin/src/main/kotlin/com/itangcent/debug/LoggerCollector.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,18 +1,17 @@
package com.itangcent.debug

import com.itangcent.debug.LoggerCollector.Companion.buffer
import com.itangcent.intellij.logger.AbstractLogger
import com.itangcent.intellij.logger.Level
import com.itangcent.intellij.logger.Logger

/**
*
* This interface represents [Logger] which collect all appended logs to a buffer.
*/
class LoggerCollector : AbstractLogger() {
class LoggerCollector : Logger {

override fun processLog(logData: String?) {
buffer.append(logData)
.appendLine()
override fun log(level: Level, msg: String) {
buffer.append("[${level.name}]\t$msg").appendLine()
}

companion object {
Expand Down
7 changes: 3 additions & 4 deletions idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/actions/BasicAnAction.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,18 +6,18 @@ import com.itangcent.common.spi.Setup
import com.itangcent.common.spi.SpiUtils
import com.itangcent.idea.config.CachedResourceResolver
import com.itangcent.idea.plugin.Initializer
import com.itangcent.idea.utils.ConfigurableLogger
import com.itangcent.intellij.actions.KotlinAnAction
import com.itangcent.intellij.config.resource.ResourceResolver
import com.itangcent.intellij.context.ActionContext
import com.itangcent.intellij.context.ActionContextBuilder
import com.itangcent.intellij.extend.guice.singleton
import com.itangcent.intellij.extend.guice.with
import com.itangcent.intellij.jvm.kotlin.KotlinAutoInject
import com.itangcent.intellij.logger.IdeaConsoleLogger
import com.itangcent.intellij.logger.Logger
import com.itangcent.intellij.logger.MultiWindowConsoleLogger
import com.itangcent.intellij.spi.IdeaAutoInject
import com.itangcent.intellij.tip.OnlyOnceInContextTipSetup
import com.itangcent.logger.LoggerProvider
import javax.swing.Icon

abstract class BasicAnAction : KotlinAnAction {
Expand All @@ -34,8 +34,7 @@ abstract class BasicAnAction : KotlinAnAction {

super.onBuildActionContext(event, builder)

builder.bind(Logger::class) { it.with(ConfigurableLogger::class).singleton() }
builder.bind(Logger::class, "delegate.logger") { it.with(IdeaConsoleLogger::class).singleton() }
builder.bind(Logger::class) { it.toProvider(LoggerProvider::class.java).singleton() }

builder.bind(ResourceResolver::class) { it.with(CachedResourceResolver::class).singleton() }

Expand Down
2 changes: 1 addition & 1 deletion ...-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/core/RequestClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -125,7 +125,7 @@ abstract class RequestClassExporter : ClassExporter {
}

shouldIgnore(cls) -> {
logger.info("ignore class:$clsQualifiedName")
logger.debug("ignore class:$clsQualifiedName")
return true
}
}
Expand Down
2 changes: 1 addition & 1 deletion ...main/kotlin/com/itangcent/idea/plugin/api/export/generic/GenericMethodDocClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ open class GenericMethodDocClassExporter : ClassExporter {
return false
}
shouldIgnore(cls) -> {
logger.info("ignore class: $clsQualifiedName")
logger.debug("ignore class: $clsQualifiedName")
disposable()
return true
}
Expand Down
8 changes: 4 additions & 4 deletions ...otlin/com/itangcent/idea/plugin/api/export/generic/SimpleGenericMethodDocClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ open class SimpleGenericMethodDocClassExporter : ClassExporter {
}

@Inject
private val logger: Logger? = null
private lateinit var logger: Logger

@Inject
protected val settingBinder: SettingBinder? = null
Expand Down Expand Up @@ -72,12 +72,12 @@ open class SimpleGenericMethodDocClassExporter : ClassExporter {
}

shouldIgnore(cls) -> {
logger!!.info("ignore class: $clsQualifiedName")
logger.debug("ignore class: $clsQualifiedName")
return true
}

else -> {
logger!!.info("search api from: $clsQualifiedName")
logger.info("search api from: $clsQualifiedName")

val fields = linkedMapOf<String, Any?>()

Expand All @@ -91,7 +91,7 @@ open class SimpleGenericMethodDocClassExporter : ClassExporter {
}
}
} catch (e: Exception) {
logger!!.traceError(e)
logger.traceError(e)
}

return true
Expand Down
8 changes: 4 additions & 4 deletions .../kotlin/com/itangcent/idea/plugin/api/export/generic/SimpleGenericRequestClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ open class SimpleGenericRequestClassExporter : ClassExporter {
}

@Inject
private val logger: Logger? = null
private lateinit var logger: Logger

@Inject
private lateinit var ruleComputer: RuleComputer
Expand All @@ -68,19 +68,19 @@ open class SimpleGenericRequestClassExporter : ClassExporter {
return false
}
shouldIgnore(cls) -> {
logger!!.info("ignore class: $clsQualifiedName")
logger.debug("ignore class: $clsQualifiedName")
return true
}
else -> {
logger!!.info("search api from: $clsQualifiedName")
logger.info("search api from: $clsQualifiedName")

classApiExporterHelper.foreachPsiMethod(cls) { method ->
exportMethodApi(cls, method, docHandle)
}
}
}
} catch (e: Exception) {
logger!!.traceError(e)
logger.traceError(e)
}
return true
}
Expand Down
8 changes: 4 additions & 4 deletions ...main/kotlin/com/itangcent/idea/plugin/api/export/jaxrs/SimpleJAXRSRequestClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ open class SimpleJAXRSRequestClassExporter : ClassExporter {
}

@Inject
private val logger: Logger? = null
private lateinit var logger: Logger

@Inject
private lateinit var ruleComputer: RuleComputer
Expand All @@ -74,18 +74,18 @@ open class SimpleJAXRSRequestClassExporter : ClassExporter {
return false
}
shouldIgnore(cls) -> {
logger!!.info("ignore class: $clsQualifiedName")
logger.debug("ignore class: $clsQualifiedName")
return true
}
else -> {
logger!!.info("search api from: $clsQualifiedName")
logger.info("search api from: $clsQualifiedName")
classApiExporterHelper.foreachPsiMethod(cls) { method ->
exportMethodApi(cls, method, docHandle)
}
}
}
} catch (e: Exception) {
logger!!.traceError(e)
logger.traceError(e)
}
return true
}
Expand Down
8 changes: 4 additions & 4 deletions ...in/kotlin/com/itangcent/idea/plugin/api/export/spring/SimpleSpringRequestClassExporter.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ open class SimpleSpringRequestClassExporter : ClassExporter {
}

@Inject
private val logger: Logger? = null
private lateinit var logger: Logger

@Inject
protected lateinit var ruleComputer: RuleComputer
Expand All @@ -80,12 +80,12 @@ open class SimpleSpringRequestClassExporter : ClassExporter {
}

shouldIgnore(cls) -> {
logger!!.info("ignore class: $clsQualifiedName")
logger.debug("ignore class: $clsQualifiedName")
return true
}

else -> {
logger!!.info("search api from: $clsQualifiedName")
logger.info("search api from: $clsQualifiedName")


classApiExporterHelper.foreachPsiMethod(cls) { method ->
Expand All @@ -94,7 +94,7 @@ open class SimpleSpringRequestClassExporter : ClassExporter {
}
}
} catch (e: Exception) {
logger!!.traceError(e)
logger.traceError(e)
}
return true
}
Expand Down
Loading
Loading