hexcore 9.7.2__tar.gz → 9.7.4__tar.gz

{hexcore-9.7.2/hexcore.egg-info → hexcore-9.7.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hexcore
-Version: 9.7.2
+Version: 9.7.4
 Summary: Reusable core for Python projects with hexagonal architecture, DDD, CQRS and background tasks. Ships the abstractions and the infrastructure: SQL session layer, FastAPI factories, worker runner, dynamic cron and identity.
 Author-email: "David Latosefki (Indroic)" <indroic@outlook.com>
 License-Expression: MIT
@@ -10,6 +10,16 @@ Project-URL: Repository, https://github.com/Indroic/HexCore
 Project-URL: Changelog, https://github.com/Indroic/HexCore/blob/master/CHANGELOG.md
 Project-URL: Issues, https://github.com/Indroic/HexCore/issues
 Keywords: hexagonal-architecture,ddd,cqrs,event-sourcing,fastapi,sqlalchemy,background-tasks,cron,authentication,identity
+Classifier: Development Status :: 5 - Production/Stable
+Classifier: Intended Audience :: Developers
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
+Classifier: Programming Language :: Python :: 3 :: Only
+Classifier: Framework :: FastAPI
+Classifier: Topic :: Software Development :: Libraries :: Application Frameworks
+Classifier: Typing :: Typed
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE

{hexcore-9.7.2 → hexcore-9.7.4}/hexcore/darwin/infrastructure/orms/sqlalchemy/repositories.py

@@ -28,6 +28,8 @@ from uuid import UUID
 
 from sqlalchemy import delete, select, update
 
+from hexcore.infrastructure.repositories.orms.sqlalchemy.utils import filas_afectadas
+
 from hexcore.darwin.domain.entities import (
     CREDENTIAL_PROVIDER,
     Account,
@@ -228,7 +230,7 @@ class SqlAlchemySessionRepository(_BaseIdentityRepository, AbstractSessionReposi
                 .values(revoked_at=at)
             )
             await session.commit()
-            return int(resultado.rowcount or 0)
+            return filas_afectadas(resultado)
 
     async def consume_for_rotation(
         self, session_id: UUID, *, at: datetime
@@ -272,7 +274,7 @@ class SqlAlchemySessionRepository(_BaseIdentityRepository, AbstractSessionReposi
                 delete(self._model).where(self._model.expires_at < before)
             )
             await session.commit()
-            return int(resultado.rowcount or 0)
+            return filas_afectadas(resultado)
 
 
 class SqlAlchemyAccountRepository(_BaseIdentityRepository, AbstractAccountRepository):
@@ -427,7 +429,7 @@ class SqlAlchemyVerificationRepository(
                 .values(consumed_at=at)
             )
             await session.commit()
-            return int(resultado.rowcount or 0)
+            return filas_afectadas(resultado)
 
     async def delete_expired(self, *, before: datetime) -> int:
         async with self._session_scope() as session:
@@ -435,7 +437,7 @@ class SqlAlchemyVerificationRepository(
                 delete(self._model).where(self._model.expires_at < before)
             )
             await session.commit()
-            return int(resultado.rowcount or 0)
+            return filas_afectadas(resultado)
 
 
 class SqlAlchemyAuditSink(AbstractAuditSink):

{hexcore-9.7.2 → hexcore-9.7.4}/hexcore/darwin/plugins/oauth/orms/sqlalchemy/repository.py

@@ -17,6 +17,8 @@ from datetime import UTC, datetime
 
 from sqlalchemy import delete, update
 
+from hexcore.infrastructure.repositories.orms.sqlalchemy.utils import filas_afectadas
+
 from hexcore.darwin.plugins.oauth.domain import (
     AbstractOAuthStateRepository,
     OAuthState,
@@ -111,7 +113,7 @@ class SqlAlchemyOAuthStateRepository(AbstractOAuthStateRepository):
                 delete(self._model).where(self._model.expires_at < before)
             )
             await session.commit()
-            return int(resultado.rowcount or 0)
+            return filas_afectadas(resultado)
 
 
 def _a_entidad(fila: t.Any) -> OAuthState:

{hexcore-9.7.2 → hexcore-9.7.4}/hexcore/infrastructure/repositories/orms/sqlalchemy/utils.py

@@ -6,6 +6,7 @@ from datetime import datetime
 from uuid import UUID
 
 from sqlalchemy import and_, cast, func, or_, select, String
+from sqlalchemy import Result
 from sqlalchemy.ext.asyncio import AsyncSession
 from sqlalchemy.orm import selectinload, RelationshipProperty
 
@@ -523,3 +524,22 @@ def ensure_framework_models_loaded() -> list[str]:
             continue
         importados.append(module_name)
     return importados
+
+
+def filas_afectadas(resultado: "Result[t.Any]") -> int:
+    """
+    Cuántas filas tocó un `DELETE` o un `UPDATE`, leído de forma que tipe.
+
+    `session.execute()` está tipado como que devuelve `Result[Any]`, pero con una sentencia DML
+    lo que devuelve en runtime es un `CursorResult`, que es la única de las dos clases que tiene
+    `rowcount`. Hasta SQLAlchemy 2.0.43 los stubs dejaban pasar `resultado.rowcount` igual;
+    2.0.54 afinó la jerarquía y pyright empezó a marcarlo — doce errores, todos este mismo
+    acceso repetido en cinco lugares.
+
+    El cast va acá y una sola vez, con el motivo escrito, en vez de cinco `# pyright: ignore`
+    sueltos que el próximo lector tendría que reconstruir de memoria. `rowcount` es `-1` cuando
+    el driver no lo sabe, y `or 0` lo aplana — que es lo que los repositorios ya hacían.
+    """
+    from sqlalchemy import CursorResult
+
+    return int(t.cast("CursorResult[t.Any]", resultado).rowcount or 0)

{hexcore-9.7.2 → hexcore-9.7.4/hexcore.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hexcore
-Version: 9.7.2
+Version: 9.7.4
 Summary: Reusable core for Python projects with hexagonal architecture, DDD, CQRS and background tasks. Ships the abstractions and the infrastructure: SQL session layer, FastAPI factories, worker runner, dynamic cron and identity.
 Author-email: "David Latosefki (Indroic)" <indroic@outlook.com>
 License-Expression: MIT
@@ -10,6 +10,16 @@ Project-URL: Repository, https://github.com/Indroic/HexCore
 Project-URL: Changelog, https://github.com/Indroic/HexCore/blob/master/CHANGELOG.md
 Project-URL: Issues, https://github.com/Indroic/HexCore/issues
 Keywords: hexagonal-architecture,ddd,cqrs,event-sourcing,fastapi,sqlalchemy,background-tasks,cron,authentication,identity
+Classifier: Development Status :: 5 - Production/Stable
+Classifier: Intended Audience :: Developers
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
+Classifier: Programming Language :: Python :: 3 :: Only
+Classifier: Framework :: FastAPI
+Classifier: Topic :: Software Development :: Libraries :: Application Frameworks
+Classifier: Typing :: Typed
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
 License-File: LICENSE

{hexcore-9.7.2 → hexcore-9.7.4}/pyproject.toml

@@ -10,7 +10,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "hexcore"
-version = "9.7.2"
+version = "9.7.4"
 authors = [
   { name="David Latosefki (Indroic)", email="indroic@outlook.com" },
 ]
@@ -31,6 +31,31 @@ keywords = [
     "sqlalchemy", "background-tasks", "cron", "authentication", "identity",
 ]
 
+# El badge de "Python" del README sale de acá y **no** de `requires-python`: shields.io lee los
+# trove classifiers `Programming Language :: Python :: X.Y` de la metadata de PyPI. Sin esta
+# tabla el badge renderizaba literalmente "missing", que es peor que no tenerlo — parecía que
+# el paquete no declaraba ninguna versión soportada.
+#
+# Cada versión declarada acá tiene su pata en la matriz de `pytest.yml`. Es la mitad que
+# importa: un classifier es una afirmación sobre en qué corre el paquete, y una afirmación que
+# nadie verifica es exactamente lo que este repo evita en todos lados. Si se agrega una versión
+# acá, va también a la matriz — y al revés.
+#
+# No va `License :: OSI Approved :: MIT License`: la licencia ya se declara arriba como
+# expresión SPDX (PEP 639), y setuptools >=77 rechaza tener las dos formas a la vez.
+classifiers = [
+    "Development Status :: 5 - Production/Stable",
+    "Intended Audience :: Developers",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
+    "Programming Language :: Python :: 3 :: Only",
+    "Framework :: FastAPI",
+    "Topic :: Software Development :: Libraries :: Application Frameworks",
+    "Typing :: Typed",
+]
+
 # La documentación **no** se empaqueta en la wheel (`MANIFEST.in` sólo incluye `*.pyi` y
 # `py.typed`), así que estas URLs son la única forma de llegar a ella desde PyPI. Apuntan a
 # `docs/` de la raíz del monorepo, que es donde vive desde la consolidación — un enlace

{hexcore-9.7.2 → hexcore-9.7.4}/tests/test_documentation_examples.py

@@ -477,6 +477,43 @@ def test_docs_facade_attributes_exist():
     )
 
 
+# ── Los classifiers de Python dicen la verdad ──────────────────────────────
+
+
+def test_cada_version_de_python_declarada_tiene_su_pata_en_la_matriz():
+    """
+    Los `Programming Language :: Python :: X.Y` de `pyproject.toml` y la matriz de
+    `pytest.yml` tienen que coincidir exactamente.
+
+    Un classifier no es decoración: es la respuesta a "¿corre en mi Python?", y es de donde
+    shields.io saca el badge del README. Prometer una versión que la CI no ejecuta es un verde
+    falso con la peor latencia posible — se descubre cuando alguien la instala.
+
+    Al revés también falla: una pata en la matriz que no esté declarada significa que se está
+    pagando el runner por una versión que el paquete no dice soportar.
+    """
+    tomllib = pytest.importorskip("tomllib")
+    yaml = pytest.importorskip("yaml")
+
+    with (REPO / "packages" / "hexcore" / "pyproject.toml").open("rb") as archivo:
+        classifiers = tomllib.load(archivo)["project"]["classifiers"]
+
+    prefijo = "Programming Language :: Python :: "
+    declaradas = {
+        c[len(prefijo) :]
+        for c in classifiers
+        if c.startswith(prefijo) and c[len(prefijo) :][:1].isdigit() and "." in c
+    }
+
+    flujo = yaml.safe_load((REPO / ".github" / "workflows" / "pytest.yml").read_text(encoding="utf-8"))
+    en_matriz = {str(v) for v in flujo["jobs"]["test"]["strategy"]["matrix"]["python"]}
+
+    assert declaradas == en_matriz, (
+        f"pyproject declara {sorted(declaradas)} y pytest.yml corre {sorted(en_matriz)}. "
+        "Agreg\u00e1 la versi\u00f3n que falta al otro lado, o sac\u00e1 la que sobra."
+    )
+
+
 # ── La skill de agente ─────────────────────────────────────────────────────────
 #
 # `skills/hexcore/` ya entra en los guardas de arriba por `ALL_DOC_FILES`: sus `from hexcore…
@@ -515,20 +552,37 @@ def _surface():
 
 def test_la_skill_tiene_frontmatter_utilizable():
     """
-    Sin `name` y `description` en el frontmatter, Claude Code no carga la skill y nadie se
-    entera: no hay error, simplemente el agente no la usa nunca. Un typo en el YAML la apaga
-    en silencio, que es el mismo modo de falla que la skill entera existe para documentar.
+    El frontmatter de `SKILL.md` tiene que **parsear como YAML**, no sólo contener las claves.
+
+    Chequear que el texto `description:` esté presente no alcanza, y no es una hipótesis: la
+    primera versión de este test hacía exactamente eso y dejó pasar una `description` con un
+    `: ` en el medio (`"...del monorepo: el framework Python..."`), que YAML lee como un mapping
+    anidado dentro de un mapping compacto. El parser de Claude Code es lo bastante tolerante
+    como para cargarla igual, así que acá todo se veía bien; el de `npx skills` no lo es y
+    rechazó la skill entera con "No valid skills found".
+
+    Ese es justo el modo de falla que esta skill documenta en otros lados: no levanta una
+    excepción donde está el error, simplemente el agente no la usa nunca.
     """
+    yaml = pytest.importorskip("yaml")
+
     contenido = (SKILL_DIR / "SKILL.md").read_text(encoding="utf-8")
 
     assert contenido.startswith("---\n"), "SKILL.md tiene que abrir con el frontmatter YAML"
     cierre = contenido.index("\n---\n", 3)
     frontmatter = contenido[4:cierre]
 
-    for clave in ("name:", "description:"):
-        assert clave in frontmatter, f"al frontmatter de SKILL.md le falta `{clave}`"
+    try:
+        metadatos = yaml.safe_load(frontmatter)
+    except yaml.YAMLError as error:  # pragma: no cover - el mensaje es el valor del test
+        pytest.fail(f"el frontmatter de SKILL.md no parsea como YAML: {error}")
+
+    assert isinstance(metadatos, dict), "el frontmatter tiene que ser un mapping YAML"
+
+    for clave in ("name", "description"):
+        assert metadatos.get(clave), f"al frontmatter de SKILL.md le falta `{clave}`"
 
-    assert "darwin-client" in frontmatter, (
+    assert "darwin-client" in metadatos["description"], (
         "la `description` es lo único que el modelo lee para decidir si activa la skill: si no "
         "nombra el cliente TypeScript, un prompt de frontend no la dispara"
     )

{hexcore-9.7.2 → hexcore-9.7.4}/tests/test_repositories_utils.py

@@ -248,8 +248,17 @@ def test_discover_sql_repositories_warns_for_abstract_repositories() -> None:
     assert "concretewarning" in discovered
     assert discovered["concretewarning"] is _ConcreteWarningRepo
     assert caught
-    warning_message = str(caught[0].message)
-    assert "_AbstractWarningRepo" in warning_message
+    # El warning se busca, no se asume que sea `caught[0]`. `catch_warnings(record=True)`
+    # captura **todos** los del bloque, incluidos los que no son de este código: en Python 3.14
+    # el recolector cierra las conexiones sqlite pendientes dentro de la ventana y mete un
+    # `ResourceWarning: unclosed database` que llega primero. El test fallaba comparando contra
+    # ese, con un mensaje que no nombraba la causa real.
+    mensajes = [str(w.message) for w in caught]
+    coincidencias = [m for m in mensajes if "_AbstractWarningRepo" in m]
+    assert coincidencias, (
+        "ning\u00fan warning menciona _AbstractWarningRepo; se capturaron: " + repr(mensajes)
+    )
+    warning_message = coincidencias[0]
     assert "custom_required" in warning_message