discord.js - Imagine an app•8mo ago

TS Error

export interface Command {
  name?: string; // Command names are taken from file names
  description: string;
  type: ApplicationCommandType;
  defaultPermissions?: PermissionResolvable;
  options?: ApplicationCommandOptionData[];
  run: (interaction: ECICInteraction) => void;
  autocomplete?: (interaction: AutocompleteInteraction) => void;
}

public appCommands: Command[] = [];

public refreshCommands(): Promise<Collection<string, ApplicationCommand<{
    guild: GuildResolvable;
  }>>> {
    return this.application.commands.set(this.appCommands.map(cmdObj => {
      const command: ApplicationCommandDataResolvable = {
        name: cmdObj.name as string,
        type: cmdObj.type, // Type 'ApplicationCommandType' is not assignable to type 'ApplicationCommandType.ChatInput'.
        defaultMemberPermissions: cmdObj.defaultPermissions || null,
        dmPermission: false
      };

      if(cmdObj.type === ApplicationCommandType.ChatInput) {
        command.description = cmdObj.description; // Property 'description' does not exist on type 'UserApplicationCommandData'.
        command.options = cmdObj.options; // Property 'options' does not exist on type 'UserApplicationCommandData'.
      }

      return command;
    }));
  }

export interface Command {
  name?: string; // Command names are taken from file names
  description: string;
  type: ApplicationCommandType;
  defaultPermissions?: PermissionResolvable;
  options?: ApplicationCommandOptionData[];
  run: (interaction: ECICInteraction) => void;
  autocomplete?: (interaction: AutocompleteInteraction) => void;
}

public appCommands: Command[] = [];

public refreshCommands(): Promise<Collection<string, ApplicationCommand<{
    guild: GuildResolvable;
  }>>> {
    return this.application.commands.set(this.appCommands.map(cmdObj => {
      const command: ApplicationCommandDataResolvable = {
        name: cmdObj.name as string,
        type: cmdObj.type, // Type 'ApplicationCommandType' is not assignable to type 'ApplicationCommandType.ChatInput'.
        defaultMemberPermissions: cmdObj.defaultPermissions || null,
        dmPermission: false
      };

      if(cmdObj.type === ApplicationCommandType.ChatInput) {
        command.description = cmdObj.description; // Property 'description' does not exist on type 'UserApplicationCommandData'.
        command.options = cmdObj.options; // Property 'options' does not exist on type 'UserApplicationCommandData'.
      }

      return command;
    }));
  }

15 Replies

d.js toolkit•8mo ago

- What's your exact discord.js npm list discord.js and node node -v version? - Not a discord.js issue? Check out #other-js-ts. - Consider reading #how-to-get-help to improve your question! - Explain what exactly your issue is. - Post the full error stack trace, not just the top part! - Show your code! - Issue solved? Press the button!

Syjalo•8mo ago

Just extend your Command interface with ApplicationCommandData and don't do that unnecessary mappings

ethan<333OP•8mo ago

wdym by unnecessary mappings?

Syjalo•8mo ago

By extending with ApplicationCommandData you will be able to pass the appCommands array directly without mapping it.

ethan<333OP•8mo ago

ah, how do i extend my interface with ApplicationCommandData? sorry im not the best with typescript haha

Syjalo•8mo ago

Ok, you can't extend it, since ApplicationCommandData is a type, but you can do

type Command = ApplicationCommandData & {
    run: () => void;
    autocomplete?: () => void;
}

type Command = ApplicationCommandData & {
    run: () => void;
    autocomplete?: () => void;
}

ethan<333OP•8mo ago

so i use this instead of importing Command from my types? i dont see why something so simple is this difficult, is it like this for everyone?

Syjalo•8mo ago

You can export this type and use it in your command files

Syjalo•8mo ago

And you'll be able to pass your array directly to set

Syjalo•8mo ago

You just need to understate how strict typing works

ethan<333OP•8mo ago

im still getting these errors

Syjalo•8mo ago

Again, no need to map your commmands

ethan<333OP•8mo ago

so i just return this.application.commands.set(this.appCommands)?

Syjalo•8mo ago

Yes

ethan<333OP•8mo ago

oh there's no errors now, why was i doing that mapping in the first place?

Gaming

Programming

TS Error