Merge branch 'dev'

Author: NLGS2907

.github/workflows/releases.yml

@@ -0,0 +1,20 @@
+name: Release
+
+on:
+  push:
+    tags:
+      - v[0-9]+.[0-9]+.[0-9]+
+
+jobs:
+  release:
+    name: Create release with tag '${{ github.ref_name }}'
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Create GitHub Release
+        env:
+          GITHUB_TOKEN: ${{ github.token }}
+        run: >
+          gh release create '${{ github.ref_name }}'
+          --repo '${{ github.repository }}'
+          --generate-notes

CHANGELOG.md

@@ -4,6 +4,7 @@ Historial con detalles de cambios entre versiones del asistente.
 
 |                  |                  |                  |                  |      Version     |
 |:----------------:|:----------------:|:----------------:|:----------------:|:----------------:|
+|                  |                  |                  |                  |**[2.0.1](#v201)**|
 |                  |                  |                  |  [1.4.1](#v141)  |**[2.0.0](#v200)**|
 |                  |                  |                  |                  |**[1.4.0](#v140)**|
 |                  |                  |                  |  [1.2.1](#v121)  |**[1.3.0](#v130)**|
@@ -13,6 +14,12 @@ Historial con detalles de cambios entre versiones del asistente.
 
 <hr/>
 
+### v2.0.1
+
+* _fixes_ para bugs de parámetros de búsqueda en `/random`
+
+<hr style="height:2px" />
+
 ## v2.0.0
 
 * **El bot ahora se llama *"Asistente de Fundamentos"*, ya no *"Lector de Ejercicios"***. Esto es para ser más conforme con futuras características.

README.md

@@ -6,7 +6,7 @@
 
 ![status](https://dcbadge.limes.pink/api/shield/889312376036425810?bot=true?logoColor=presence&theme=discord)
 
-![version](https://img.shields.io/badge/version-2.0.0-brightgreen)
+![version](https://img.shields.io/badge/version-2.0.1-brightgreen)
 ![estrellas](https://img.shields.io/github/stars/NLGS2907/Asistente-Fundamentos?label=Estrellas&style=social)
 ![forks](https://img.shields.io/github/forks/NLGS2907/Asistente-Fundamentos?style=social)
 ![Tests](https://github.com/NLGS2907/Asistente-Fundamentos/actions/workflows/tests.yml/badge.svg)

asistente/bot/asistente.py

@@ -47,7 +47,7 @@ def version() -> tuple[int, int, int]:
         indicando una revisión importante, un parche mayor, o un parche menor respectivamente.
         """
 
-        return (2, 0, 0)
+        return (2, 0, 1)
 
 
     @staticmethod

asistente/cogs/comandos/ejercicios.py

@@ -212,11 +212,17 @@ async def ejercicio_al_azar(self,
         """
 
         guia = get_guia_por_sv(interaccion.guild_id)
+        guia.pop("version")
+
+        if isinstance(sentido, Choice):
+            sentido_parseado = sentido.value
+        else: # str
+            sentido_parseado = sentido
 
         unidad_posible_real = (unidad_posible if unidad_posible is None else unidad_posible.value)
-        unidad_pivote = (unidad_posible_real
-                         if CogEjercicios.existe_unidad(unidad_posible_real, guia)
-                         else choice(list(guia.keys())))
+        unidad_pivote = int(unidad_posible_real
+                            if CogEjercicios.existe_unidad(unidad_posible_real, guia)
+                            else choice(list(guia.keys())))
         unidad_elegida = ''
         ejercicio_elegido = ''
 
@@ -226,29 +232,44 @@ def _expresion_busqueda(unidad: int) -> bool:
             incluirlo entre los candidatos.
             """
 
-            match sentido:
+            match sentido_parseado:
                 case '=':
-                    return unidad == int(unidad_pivote)
+                    return unidad == unidad_pivote
 
                 case '<':
-                    return unidad < int(unidad_pivote)
+                    return unidad < unidad_pivote
 
                 case "<=":
-                    return unidad <= int(unidad_pivote)
+                    return unidad <= unidad_pivote
 
                 case '>':
-                    return unidad > int(unidad_pivote)
+                    return unidad > unidad_pivote
 
                 case ">=":
-                    return unidad >= int(unidad_pivote)
+                    return unidad >= unidad_pivote
 
                 case _:
                     # Viene de un Choice, así que nunca va a ocurrir. Pero por si acaso, suponemos
                     # que una expresión de 'sentido' inválida, hace incorrectas a todos los números.
                     return False
 
-        unidad_elegida = choice([unidad for unidad in list(guia.keys())[1:]
-                                if _expresion_busqueda(int(unidad))])
+        candidatos = [unidad for unidad in guia.keys() if _expresion_busqueda(int(unidad))]
+
+        # El usuario eligió justo un parámetro de búsqueda que no retorna ningún candidato viable:
+        # Un ejemplo sería si se elige la unidad 1 de pivote pero con parámetro "Antes de".
+        # Naturalmente, no va a retornar nada.
+        if not candidatos:
+            await interaccion.response.send_message(
+                content="No hay unidades de la guía que coincidan con los parámetros de búsqueda "
+                        f"dados y la unidad pivote `{unidad_pivote}. "
+                        f"{guia[str(unidad_pivote)]['titulo']}`.",
+                ephemeral=True
+            )
+            return
+
+        unidad_elegida = choice(candidatos)
+        # para que no agarre el titulo ni por casualidad, lo sacamos
+        guia[unidad_elegida].pop("titulo")
         ejercicio_elegido = choice(list(guia[unidad_elegida].keys()))
 
         await self.mandar_ejercicio(interaccion,