feat: 引数取得処理の共通化

.eslintrc.yml

@@ -17,5 +17,6 @@ rules:
   '@typescript-eslint/no-explicit-any': off
   '@typescript-eslint/ban-ts-comment': off
   'unicorn/prefer-top-level-await': off
+  'unicorn/prevent-abbreviations': off
   'unicorn/no-null': off
   'no-console': error

src/commands/bassline.ts

@@ -20,11 +20,14 @@ export class BasslineCommand implements BaseCommand {
     ].join('\n')
   }
 
-  async execute(_discord: Discord, message: Message<boolean>): Promise<void> {
-    const [, ...suspectName] = message.content.split(' ')
+  async execute(
+    _discord: Discord,
+    message: Message<boolean>,
+    args: string[]
+  ): Promise<void> {
     await message.channel.send(
-      suspectName.length > 0
-        ? this.getBasslineText(suspectName.join(' '))
+      args.length > 0
+        ? this.getBasslineText(args.join(' '))
         : this.getBasslineText()
     )
   }

src/commands/index.ts

@@ -19,12 +19,30 @@ interface PermissionPermission {
 export type Permission = UserPermission | RolePermission | PermissionPermission
 
 export abstract class BaseCommand {
-  /** 名前: コマンドの名前 */
+  /**
+   * 名前: コマンドの名前
+   *
+   * @returns {string} コマンドの名前
+   */
   abstract get name(): string
 
-  /** 権限: コマンドの実行に必要なユーザー・ロール・パーミッション。NULLが指定された場合はすべて許可 */
+  /**
+   * 権限: コマンドの実行に必要なユーザー・ロール・パーミッション。NULLが指定された場合はすべて許可
+   *
+   * @returns {Permission[] | null} コマンドの実行に必要なユーザー・ロール・パーミッション
+   */
   abstract get permissions(): Permission[] | null
 
-  /** 実行: コマンドの実行定義 */
-  abstract execute(discord: Discord, message: Message): Promise<void>
+  /**
+   * 実行: コマンドの実行定義
+   *
+   * @param {Discord} discord Discordインスタンス
+   * @param {Message} message Discordメッセージ
+   * @param {string[]} args コマンド引数
+   */
+  abstract execute(
+    discord: Discord,
+    message: Message,
+    args: string[]
+  ): Promise<void>
 }

src/discord.ts

@@ -63,14 +63,18 @@ export class Discord {
   }
 
   async onMessageCreate(message: Message) {
+    const logger = Logger.configure('Discord.onMessageCreate')
     const command = Discord.commands.find((command) =>
       message.content.startsWith(`/${command.name}`)
     )
     if (!command) {
       return
     }
 
-    command.execute(this, message)
+    logger.info(`👌 ${message.author.tag}: execute ${command.name}`)
+
+    const [, ...args] = message.content.split(' ')
+    command.execute(this, message, args)
   }
 
   waitReady() {