detekt report

Total: 12

complexity: 2

ComplexCondition: 2 Complex conditions should be simplified and extracted into well-named methods if necessary.

mokoki/darwin/main/kotlin/com/javiersc/mokoki/internal/LoggerNames.darwin.kt:46:17This condition is too complex (4). Defined complexity threshold for conditions is set to '4'

  43             val methodName = line?.substringAfter("#")?.substringBefore("(")
  44             val lineNumber = line?.substringAfterLast(".kt:")?.substringBefore(":")
  45 
  46             if (fileName != null && className != null && methodName != null && lineNumber != null) {
  47                 StackTraceElement(fileName, className, methodName, lineNumber)
  48             } else {
  49                 null

mokoki/mingw/main/kotlin/com/javiersc/mokoki/internal/LoggerNames.mingw.kt:48:17This condition is too complex (4). Defined complexity threshold for conditions is set to '4'

  45                 line?.substringBefore("\$FUNCTION_REFERENCE")?.substringAfterLast('$')
  46             val lineNumber = "1"
  47 
  48             if (fileName != null && className != null && methodName != null && lineNumber != null) {
  49                 StackTraceElement(fileName, className, methodName, lineNumber)
  50             } else {
  51                 null

exceptions: 5

SwallowedException: 4 The caught exception is swallowed. The original exception could be lost.

Documentation

mokoki-serialization/common/main/kotlin/com/javiersc/mokoki/serialization/internal/BuildMokokiSerialization.kt:21:18The caught exception is swallowed. The original exception could be lost.

  18     val message =
  19         try {
  20             json.encodeToString(json.parseToJsonElement(jsonToPrint))
  21         } catch (exception: SerializationException) {
  22             jsonToPrint
  23         }
  24     return buildMokokiMessage(priority, tag, message)

mokoki-serialization/common/main/kotlin/com/javiersc/mokoki/serialization/internal/BuildMokokiSerialization.kt:38:18The caught exception is swallowed. The original exception could be lost.

  35     val message =
  36         try {
  37             json.encodeToString(serializer, data)
  38         } catch (exception: SerializationException) {
  39             data.toString()
  40         }
  41     return buildMokokiMessage(priority, tag, message)

mokoki-test/jvmAndAndroid/main/kotlin/com/javiersc/mokoki/test/internal/LoggerNames.jvmAndAndroid.kt:12:18The caught exception is swallowed. The original exception could be lost.

   9             val trace = Thread.currentThread().stackTrace
  10             val index = trace.indexOfLast { it.isTestClass }
  11             trace[index]
  12         } catch (throwable: Throwable) {
  13             println("Mokoki has not been able to get the StackTrace")
  14             null
  15         }

mokoki/jvmAndAndroid/main/kotlin/com/javiersc/mokoki/internal/LoggerNames.jvmAndAndroid.kt:24:18The caught exception is swallowed. The original exception could be lost.

  21             val trace = Thread.currentThread().stackTrace
  22             val index = trace.indexOfLast { it.isLogFunction }
  23             trace[index + 1]
  24         } catch (throwable: Throwable) {
  25             println("Mokoki has not been able to get the StackTrace")
  26             null
  27         }

TooGenericExceptionCaught: 1 The caught exception is too generic. Prefer catching specific exceptions to the case that is currently handled.

Documentation

mokoki/jvmAndAndroid/main/kotlin/com/javiersc/mokoki/internal/LoggerNames.jvmAndAndroid.kt:24:18The caught exception is too generic. Prefer catching specific exceptions to the case that is currently handled.

  21             val trace = Thread.currentThread().stackTrace
  22             val index = trace.indexOfLast { it.isLogFunction }
  23             trace[index + 1]
  24         } catch (throwable: Throwable) {
  25             println("Mokoki has not been able to get the StackTrace")
  26             null
  27         }

naming: 4

InvalidPackageDeclaration: 3 Kotlin source files should be stored in the directory corresponding to its package statement.

Documentation

samples/android/android-core/main/kotlin/MainActivity.kt:3:1The package declaration does not match the actual file location.

   1 @file:Suppress("MagicNumber")
   2 
   3 package com.javiersc.mokoki.core
   4 
   5 import android.os.Bundle
   6 import androidx.appcompat.app.AppCompatActivity

samples/jvm/jvm-core/main/kotlin/Main.kt:1:1The package declaration does not match the actual file location.

   1 package com.javiersc.mokoki.jvm.core
   2 
   3 import com.javiersc.mokoki.LoggerSeparator
   4 import com.javiersc.mokoki.MokokiLogger

samples/jvm/jvm-serialization/main/kotlin/Main.kt:3:1The package declaration does not match the actual file location.

   1 @file:Suppress("MagicNumber")
   2 
   3 package com.javiersc.mokoki.jvm.serialization
   4 
   5 import com.javiersc.mokoki.LoggerSeparator
   6 import com.javiersc.mokoki.MokokiLogger

MatchingDeclarationName: 1 If a source file contains only a single non-private top-level class or object, the file name should reflect the case-sensitive name plus the .kt extension.

Documentation

mokoki/mingw/test/kotlin/com/javiersc/mokoki/MokokiMingGWTest.mingw.kt:9:7The file name 'MokokiMingGWTest.mingw' does not match the name of the single top-level declaration 'MokokiMingGWTest'.

   6 import kotlin.test.BeforeTest
   7 import kotlin.test.Test
   8 
   9 class MokokiMingGWTest {
  10 
  11     private val testLogger = TestMokokiLogger(minPriority = Priority.VERBOSE)
  12

style: 1

ForbiddenComment: 1 Flags a forbidden comment.

Documentation

mokoki/darwin/main/kotlin/com/javiersc/mokoki/DefaultMokokiLogger.darwin.kt:3:1This comment contains 'TODO:' that has been defined as forbidden in detekt.

   1 package com.javiersc.mokoki
   2 
   3 // TODO: test print or native platform logger
   4 public actual typealias DefaultMokokiLogger = PrintMokokiLogger
   5

Metrics

Complexity Report

Findings

complexity: 2

exceptions: 5

naming: 4

style: 1