Improve the logic to infer command names (#168)

ajalt · Apr 30, 2020 · 97a0292 · 97a0292
1 parent 76e4942
commit 97a0292
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/clikt/src/commonMain/kotlin/com/github/ajalt/clikt/core/CliktCommand.kt b/clikt/src/commonMain/kotlin/com/github/ajalt/clikt/core/CliktCommand.kt
@@ -51,7 +51,7 @@ abstract class CliktCommand(
         private val autoCompleteEnvvar: String? = "",
         internal val allowMultipleSubcommands: Boolean = false
 ) : ParameterHolder {
-    val commandName = name ?: classSimpleName().toLowerCase()
+    val commandName = name ?: inferCommandName(classSimpleName())
     val commandHelp = help
     val commandHelpEpilog = epilog
     internal var _subcommands: List<CliktCommand> = emptyList()
@@ -371,3 +371,8 @@ fun <T : CliktCommand> T.context(block: Context.Builder.() -> Unit): T = apply {
 }
 
 private fun Any.classSimpleName(): String = this::class.simpleName.orEmpty().split("$").last()
+
+internal fun inferCommandName(name: String) =
+    name.removeSuffix("Command").replace(Regex("([a-z])([A-Z])")) {
+        "${it.groupValues[1]}-${it.groupValues[2]}"
+    }.toLowerCase()
diff --git a/clikt/src/commonTest/kotlin/com/github/ajalt/clikt/core/CliktCommandTest.kt b/clikt/src/commonTest/kotlin/com/github/ajalt/clikt/core/CliktCommandTest.kt
@@ -25,6 +25,14 @@ import kotlin.test.Test
 
 @Suppress("unused")
 class CliktCommandTest {
+    @Test
+    @JsName("commandNameInferred")
+    fun `command name inferred`() {
+        inferCommandName("ListAllValuesCommand") shouldBe "list-all-values"
+        inferCommandName("LGTMMeansLookingGoodToMe") shouldBe "lgtmmeans-looking-good-to-me"
+        inferCommandName("nothing-to-change") shouldBe "nothing-to-change"
+    }
+
     @Test
     @JsName("invokeWithoutSubcommand_false")
     fun `invokeWithoutSubcommand=false`() {