fix: use component name as description fallback

Incendo · Jan 14, 2024 · 8e1bb7d · 8e1bb7d
1 parent 3612320
commit 8e1bb7d
Showing 1 changed file with 18 additions and 3 deletions.
diff --git a/...d-common/src/main/java/org/incendo/cloud/discord/slash/StandardDiscordCommandFactory.java b/...d-common/src/main/java/org/incendo/cloud/discord/slash/StandardDiscordCommandFactory.java
@@ -145,9 +145,16 @@ public <T extends Number, P extends ArgumentParser<C, T>> void registerRangeMapp
             }
         }
 
+        final String description;
+        if (component.description().isEmpty()) {
+            description = component.name();
+        } else {
+            description = component.description().textDescription();
+        }
+
         return DiscordCommand.<C>builder()
                 .name(component.name())
-                .description(node.component().description().textDescription())
+                .description(description)
                 .addAllOptions(options)
                 .build();
     }
@@ -156,6 +163,13 @@ public <T extends Number, P extends ArgumentParser<C, T>> void registerRangeMapp
     private @NonNull DiscordOption<C> createOption(final @NonNull CommandNode<C> node) {
         final CommandComponent<C> component = node.component();
 
+        final String description;
+        if (component.description().isEmpty()) {
+            description = component.name();
+        } else {
+            description = component.description().textDescription();
+        }
+
         if (component.type() == CommandComponent.ComponentType.LITERAL) {
             // We need to determine whether to flatten the children into a sub-command
             // or whether to recursively extract the arguments.
@@ -182,7 +196,7 @@ public <T extends Number, P extends ArgumentParser<C, T>> void registerRangeMapp
 
             return ImmutableSubCommand.<C>builder()
                     .name(component.name())
-                    .description(component.description().textDescription())
+                    .description(description)
                     .addAllOptions(children)
                     .build();
         }
@@ -198,7 +212,8 @@ public <T extends Number, P extends ArgumentParser<C, T>> void registerRangeMapp
             autoComplete = false;
         }
 
-        return ImmutableVariable.<Object>builder().name(component.name()).description(component.description().textDescription())
+        return ImmutableVariable.<Object>builder().name(component.name())
+                .description(description)
                 .type(optionType)
                 .required(component.required())
                 .autocomplete(autoComplete)