Limpieza: código muerto, SOLID y duplicación

[Isma-L154]

Cierra #31.

Limpieza del árbol completo: código muerto, responsabilidades mezcladas y las
duplicaciones que ya habían empezado a desincronizarse. Cero cambios de
comportamiento salvo dos correcciones deliberadas, listadas abajo.

269 líneas añadidas, 338 eliminadas. 293 tests pasando (antes 279), sin
relajar ninguna aserción.

Código muerto eliminado

Qué	Por qué estaba muerto
utils.checks.bot_in_voice()	Ningún cog la llamaba; sus únicas referencias eran los dos tests escritos para ella.
track["spotify_url"] (2 sitios)	Spotify se eliminó del proyecto; _build_track nunca escribía esa clave.
title.split(" — ") en !lyrics	Deshacía el formato "Canción — Artista" de Spotify. Sin Spotify, truncaba títulos legítimos de YouTube que llevaran un guion largo.
track["stream"]	Se fijaba a None en cada track y no se leía en ninguna parte.
config.COOKIES_PATH sin dueño	Se definía pero nadie la importaba: services/media.py leía la variable de entorno por su cuenta. Un ajuste, dos dueños.

Responsabilidades separadas

MusicPlayer era a la vez motor de audio, maquetador de embeds y emisor de
mensajes. Los dos últimos trabajos se van a utils/announcer.py. Cambiar la
redacción de un aviso ya no obliga a tocar el núcleo de reproducción.

MusicPlayer ↔ players era un ciclo. destroy() llamaba al singleton
global que contenía al propio player. Ahora recibe un callback on_destroy. El
efecto más visible está en el fixture de tests:

 def player(fake_bot, fake_guild):
-    p = MusicPlayer(fake_bot, fake_guild, MagicMock())
-    yield p
-    # Keep the module-level singleton clean between tests.
-    players.discard(fake_guild.id)
+    return MusicPlayer(fake_bot, fake_guild, MagicMock())

Construir un player ya no ensucia estado de todo el proceso.

PlayerManager deja de leer player._destroyed (atributo privado ajeno) y
usa is_destroyed. Y recibe un guild y un channel en vez de un Context
entero de discord.py del que solo necesitaba dos campos.

Añadir un efecto costaba tres ediciones — el filtro en EFFECTS, el texto
en _LABELS, y un método de comando casi idéntico a los otros ocho. Dos dicts
con las mismas claves que podían desincronizarse en silencio. Ahora es una
entrada:

EFFECTS = {
    "echo": Effect("aecho=0.8:0.88:60:0.4", "Echo effect applied 🔔"),
}

Los nombres de comando salen de las claves del registro. CogMeta recolecta
los comandos al ejecutar el cuerpo de la clase, así que no se puede registrar
uno por efecto a posteriori; un único handler con cada nombre como alias da el
mismo resultado desde una sola definición, y ctx.invoked_with dice cuál se
tecleó. Un test nuevo recorre EFFECTS y falla si una clave no es alcanzable
como comando.

!help se genera del registro real de comandos. La lista escrita a mano ya
había derivado: !effect existía y no aparecía ni en la ayuda ni en el
README, y nada fallaba. tests/test_help.py ahora falla si cualquier comando
o alias queda sin documentar. main.py pasa de 96 a 68 líneas.

Otros

• utils/checks._error era idéntica a utils.embeds.error_embed. Ahora reusa.
• services/lyrics_api usaba print() — no llegaba a journalctl con nivel ni
  timestamp, contra la regla de logging del CLAUDE.md. Ahora usa logging, y
  reutiliza un solo cliente de Genius en vez de construir uno por consulta.
• Se elimina el handler @lyrics.error. Hacía ctx.send(str(error)), lo que
  exponía texto crudo de excepción al usuario y, como errors.handle se rinde
  ante cualquier comando con handler propio, desactivaba el manejo central de
  errores para todo el cog de letras — justo lo que construyó el Answer every wrong command invocation #16.

Comentarios

Se borran los que repiten su propia línea y los banners decorativos. Se
conservan los que explican el porqué: el orden de _PLAYER_CLIENTS, por qué
el stderr de AudioStream es un archivo y no un pipe, por qué el aresample
inicial es load-bearing. Cada uno documenta un bug que ya costó un arreglo
(#17, #25, #28, #30); borrarlos invita a reintroducirlos.

deploy/launch_ec2.sh se queda: la EC2 está detenida, no terminada, y
deploy/README.md la documenta como camino de despliegue alterno.

Cómo probarlo

pip install -r requirements.txt -r requirements-dev.txt
pytest                  # 293 pasando

Los tests de efectos renderizan audio con FFmpeg real y se saltan solos si no
está instalado.

En Discord, lo que conviene mirar a mano:

1. !help — debe listar !effect, y los 8 efectos en una sola línea.
2. !play <algo> → !bass → !nightcore dentro de 3 s — el segundo debe
   responder "wait 3s", no aplicarse.
3. !play → !bass a mitad de canción — debe seguir reanudando en la misma
   posición, no volver a 0:00.
4. !lyrics sin nada sonando, y !lyrics Bohemian Rhapsody - Queen.
5. !skip con la cola vacía, !remove 99, !move 1 99 — cada uno responde.
6. !stop y luego !play otra vez — el player se recrea; con is_destroyed
   es el mismo camino que antes.

Fuera de alcance

El bot murió el 2026-09-11 a las 06:18 UTC con ClientConnectorDNSError
durante un unattended-upgrades que actualizó glibc y reinició el resolver.
bot.start() no tiene reintento, así que cualquier hipo de DNS en el arranque
mata el proceso. Restart=on-failure lo recuperó al tercer intento. Es un
cambio de comportamiento y va en su propio issue, no mezclado aquí.