refactor: remove voice hooks

The voice hooks were added to have a channel only visible when people
joined a voice channel. For a year or so, this has been a native feature
in Discord, so this voice role is actually not needed anymore.

docs/development/hooks.md

@@ -125,4 +125,3 @@ Esta es una lista de las funciones y los prototipos de referencia:
 * `onGuildMemberUpdate(prev, next)`: Se ejecuta cuando un miembro actualiza algo sobre ello. Depende mucho del evento que dispara la API de Discord. Te proporciona el estado anterior del usuario y el nuevo estado del usuario y ya es cosa tuya hacer el diff y detectar qué ha cambiado.
 * `onGuildMemberLeave(member)`: Se ejecuta cuando un miembro abandona el servidor. El `member` podría ser parcial. Cuidado con hacer un `fetch()`, porque el evento se puede recibir en cuentas que se eliminan (por ejemplo, el usuario borra su cuenta de Discord, o se trata de una cuenta temporal que Discord borra). En ese caso, hacer un `fetch()` provocará un error HTTP 404 en la API de Discord.
 * `onBuildMemberBan(ban)`: se ejecuta cuando un miembro es baneado. La información está disponible a través del objeto `ban`.
-* `onVoiceStateUpdate(oldState, newState)`: se ejecuta cuando un miembro interactúa con un canal de voz: conectándose, desconectándose, cambiando de canal, actualizando su estado (mute, deafen...). Cuando se conecta a un canal de voz, `oldState` vale null. Cuando se desconecta de cualquier canal de voz, `newState` vale null. Cuando salta de un canal a otro, puedes comparar el viejo canal con el nuevo canal mirando ambas variables para ver de dónde ha saltado. Lo mismo si apaga o enciende su micrófono o si silencia sus auriculares o quita el silencio.

package.json

@@ -21,7 +21,6 @@
     "datetoken": "^1.1.1",
     "discord-api-types": "^0.32.0",
     "discord.js": "^13.6.0",
-    "discordjs-voicerole": "^3.0.0",
     "dotenv": "^16.0.3",
     "express": "^4.18.2",
     "follow-redirects": ">=1.15.1",

src/Makibot.ts

@@ -41,7 +41,6 @@ export default class Makibot extends Client {
         Intents.FLAGS.GUILD_MESSAGES,
         Intents.FLAGS.GUILD_PRESENCES,
         Intents.FLAGS.GUILD_MESSAGE_REACTIONS,
-        Intents.FLAGS.GUILD_VOICE_STATES,
       ],
       partials: ["MESSAGE", "REACTION", "GUILD_MEMBER"],
     });

src/lib/hook.ts

@@ -8,7 +8,6 @@ import {
   PartialMessageReaction,
   PartialUser,
   User,
-  VoiceState,
 } from "discord.js";
 import Makibot from "../Makibot";
 import logger from "./logger";
@@ -44,8 +43,6 @@ export interface Hook {
   onGuildMemberUpdate?: (prev: GuildMember, next: GuildMember) => Promise<void>;
   onGuildMemberLeave?: (member: GuildMember | PartialGuildMember) => Promise<void>;
   onGuildMemberBan?: (ban: GuildBan) => Promise<void>;
-
-  onVoiceStateUpdate?: (oldStatus: VoiceState, newStatus: VoiceState) => Promise<void>;
 }
 
 type HookConstructor = {
@@ -113,7 +110,6 @@ export class HookManager {
     client.on("guildMemberRemove", this.onGuildMemberRemove.bind(this));
     client.on("guildMemberUpdate", this.onGuildMemberUpdate.bind(this));
     client.on("guildBanAdd", this.onGuildMemberBan.bind(this));
-    client.on("voiceStateUpdate", this.onVoiceStateUpdate.bind(this));
   }
 
   /**
@@ -252,14 +248,6 @@ export class HookManager {
     }
   }
 
-  async onVoiceStateUpdate(oldState: VoiceState, newState: VoiceState): Promise<void> {
-    const handlers = this.filterServices("onVoiceStateUpdate");
-    for (const handler of handlers) {
-      logger.debug(`[hooks] processing ${handler.name}(onVoiceStateUpdate)`);
-      await handler.onVoiceStateUpdate(oldState, newState);
-    }
-  }
-
   restart(name: string): void {
     logger.debug(`[hooks] restarting service ${name}...`);
     this.watchdog[name]?.restart();

src/lib/http/middlewares/guild.ts

@@ -8,7 +8,6 @@ import memberMiddleware from "./member";
 import providerMiddleware from "./provider";
 import threadOnlyChannels from "./thread";
 import trustedRoles from "./trusted";
-import voiceRoleMiddleware from "./voiceroles";
 
 export interface MiddlewareLocals {
   guild: Guild;
@@ -36,7 +35,6 @@ export default function guildMiddleware(makibot: Makibot): express.Router {
   });
 
   router.use("/members/:member", memberMiddleware(makibot));
-  router.use("/roles/voices", voiceRoleMiddleware(makibot));
   router.use("/roles/trusted", trustedRoles());
 
   router.use("/channels/threadonly", threadOnlyChannels());