HaruStar/discord-bot

View on GitHub

Showing 18 of 32 total issues

Function exports has a Cognitive Complexity of 38 (exceeds 5 allowed). Consider refactoring.
Open

module.exports = (client) => {

  /*
  PERMISSION LEVEL FUNCTION
  This is a very basic permission system for commands which uses "levels"
Severity: Minor
Found in modules/functions.js - About 5 hrs to fix

Cognitive Complexity

Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

A method's cognitive complexity is based on a few simple rules:

  • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
  • Code is considered more complex for each "break in the linear flow of the code"
  • Code is considered more complex when "flow breaking structures are nested"

Further reading

Function exports has 97 lines of code (exceeds 25 allowed). Consider refactoring.
Open

module.exports = (client) => {

  /*
  PERMISSION LEVEL FUNCTION
  This is a very basic permission system for commands which uses "levels"
Severity: Major
Found in modules/functions.js - About 3 hrs to fix

    Function run has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.
    Open

    exports.run = (client, message, args, level) => {
      // If no specific command is called, show all filtered commands.
      if (!args[0]) {
        // Filter all commands by which are available for the user's level, using the <Collection>.filter() method.
        const myCommands = message.guild ? client.commands.filter(cmd => client.levelCache[cmd.conf.permLevel] <= level) : client.commands.filter(cmd => client.levelCache[cmd.conf.permLevel] <= level &&  cmd.conf.guildOnly !== true);
    Severity: Minor
    Found in commands/help.js - About 2 hrs to fix

    Cognitive Complexity

    Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

    A method's cognitive complexity is based on a few simple rules:

    • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
    • Code is considered more complex for each "break in the linear flow of the code"
    • Code is considered more complex when "flow breaking structures are nested"

    Further reading

    Function run has 58 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

    exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
    
      if (args.length === 0){ 
        message.reply("You cannot take an action with air");
        return;
    Severity: Major
    Found in commands/action.js - About 2 hrs to fix

      Function run has 47 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

      exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
        if (args.length === 0){ 
          message.reply("You cannot wear the air");
          return;
        }
      Severity: Minor
      Found in commands/wear.js - About 1 hr to fix

        Function run has 44 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

        exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
          const foods = Array(
            ":bug:",
            ":ant:",
            ":bee:",
        Severity: Minor
        Found in commands/feed.js - About 1 hr to fix

          Function exports has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.
          Open

          module.exports = async (client, message) => {
            // It's good practice to ignore other bots. This also makes your bot ignore itself
            // and not get into a spam loop (we call that "botception").
            if (message.author.bot) return;
          
          
          Severity: Minor
          Found in events/message.js - About 1 hr to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function exports has 31 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

          module.exports = async (client, message) => {
            // It's good practice to ignore other bots. This also makes your bot ignore itself
            // and not get into a spam loop (we call that "botception").
            if (message.author.bot) return;
          
          
          Severity: Minor
          Found in events/message.js - About 1 hr to fix

            Function run has 29 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

            exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
              if (!args.length) {
                return;
              } else if (args == "blood"){
              //get infos
            Severity: Minor
            Found in commands/element.js - About 1 hr to fix

              Function run has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.
              Open

              exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
                if (args.length === 0){ 
                  message.reply("You cannot wear the air");
                  return;
                }
              Severity: Minor
              Found in commands/wear.js - About 1 hr to fix

              Cognitive Complexity

              Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

              A method's cognitive complexity is based on a few simple rules:

              • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
              • Code is considered more complex for each "break in the linear flow of the code"
              • Code is considered more complex when "flow breaking structures are nested"

              Further reading

              Function run has 26 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

              exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
                const items = Array("Ok",
                  ":eyes:",
                  "What's up?",
                  "I've no idea",
              Severity: Minor
              Found in commands/speak.js - About 1 hr to fix

                Function run has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
                Open

                exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
                
                  if (args.length === 0){ 
                    message.reply("You cannot take an action with air");
                    return;
                Severity: Minor
                Found in commands/action.js - About 35 mins to fix

                Cognitive Complexity

                Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                A method's cognitive complexity is based on a few simple rules:

                • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                • Code is considered more complex for each "break in the linear flow of the code"
                • Code is considered more complex when "flow breaking structures are nested"

                Further reading

                Function run has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
                Open

                exports.run = async (client, message, args, level) => { // eslint-disable-line no-unused-vars
                  const foods = Array(
                    ":bug:",
                    ":ant:",
                    ":bee:",
                Severity: Minor
                Found in commands/feed.js - About 35 mins to fix

                Cognitive Complexity

                Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                A method's cognitive complexity is based on a few simple rules:

                • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                • Code is considered more complex for each "break in the linear flow of the code"
                • Code is considered more complex when "flow breaking structures are nested"

                Further reading

                Avoid too many return statements within this function.
                Open

                      return console.log(`${timestamp} ${chalk.black.bgWhite(type.toUpperCase())} ${content}`);
                Severity: Major
                Found in modules/Logger.js - About 30 mins to fix

                  Avoid too many return statements within this function.
                  Open

                        return console.log(`${timestamp} ${chalk.black.bgGreen(type.toUpperCase())} ${content}`);
                  Severity: Major
                  Found in modules/Logger.js - About 30 mins to fix

                    Avoid too many return statements within this function.
                    Open

                          return;
                    Severity: Major
                    Found in events/message.js - About 30 mins to fix

                      Avoid too many return statements within this function.
                      Open

                          return message.channel.send("This command is unavailable via private message. Please run this command in a guild.");
                      Severity: Major
                      Found in events/message.js - About 30 mins to fix

                        Avoid too many return statements within this function.
                        Open

                              return message.channel.send(`You do not have permission to use this command.
                          Your permission level is ${level} (${client.config.permLevels.find(l => l.level === level).name})
                          This command requires level ${client.levelCache[cmd.conf.permLevel]} (${cmd.conf.permLevel})`);
                        Severity: Major
                        Found in events/message.js - About 30 mins to fix
                          Severity
                          Category
                          Status
                          Source
                          Language