fix: required is always true

Author: davidabou

src/utils.ts

@@ -36,7 +36,7 @@ export const slashToJSON = (cmd: HarmonixCommand<true, ArgsDef>) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
 
             if (arg.metadata?.minLength) {
               opt.setMinLength(arg.metadata.minLength)
@@ -59,7 +59,7 @@ export const slashToJSON = (cmd: HarmonixCommand<true, ArgsDef>) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
 
             if (arg.metadata?.minValue) {
               opt.setMinValue(arg.metadata.minValue)
@@ -82,23 +82,23 @@ export const slashToJSON = (cmd: HarmonixCommand<true, ArgsDef>) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
           )
           break
         case 'User':
           builder.addUserOption((opt) =>
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
           )
           break
         case 'Channel':
           builder.addChannelOption((opt) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
 
             if (arg.metadata?.channelTypes) {
               for (const type of arg.metadata.channelTypes) {
@@ -114,15 +114,15 @@ export const slashToJSON = (cmd: HarmonixCommand<true, ArgsDef>) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
           )
           break
         case 'Number':
           builder.addNumberOption((opt) => {
             opt
               .setName(name)
               .setDescription(arg.description ?? 'No description provided')
-              .setRequired(arg.required || true)
+              .setRequired(arg.required ?? true)
 
             if (arg.metadata?.minValue) {
               opt.setMinValue(arg.metadata.minValue)