letschatty 0.4.552__tar.gz → 0.4.554__tar.gz

{letschatty-0.4.552 → letschatty-0.4.554}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: letschatty
-Version: 0.4.552
+Version: 0.4.554
 Summary: Models and custom classes to work across the Chattyverse
 License: MIT
 Author: Axel

{letschatty-0.4.552 → letschatty-0.4.554}/pyproject.toml

@@ -1,7 +1,7 @@
 [tool.poetry]
 name = "letschatty"
 
-version = "0.4.552"
+version = "0.4.554"
 
 description = "Models and custom classes to work across the Chattyverse"
 authors = ["Axel <axel@letschatty.com>"]

{letschatty-0.4.552 → letschatty-0.4.554}/src/letschatty/models/chat/chat.py

@@ -279,8 +279,13 @@ class Chat(CompanyAssetModel):
 
     @property
     def bought_product_ids(self) -> List[StrObjectId]:
-        """Get all sale ids in the chat"""
-        return [sale.product_id for sale in self.sales]
+        """Los productos que compró el cliente: todos los de cada venta, sin las ventas sin productos.
+
+        Una entrada por producto y por venta (dos ventas del mismo producto lo repiten), como
+        siempre. Antes de las líneas había un producto por venta; ahora una venta aporta
+        todos los suyos.
+        """
+        return [product_id for sale in self.sales for product_id in sale.productos_vendidos()]
 
     @property
     def products(self) -> List[AssignedAssetToChat]:

{letschatty-0.4.552 → letschatty-0.4.554}/src/letschatty/models/chat/query_builder.py

@@ -134,7 +134,17 @@ class QueryBuilder:
             query["client.sales.0"] = {"$exists": True}
 
         if filters.sales_products:
-            query["client.sales.product_id"] = {"$in": filters.sales_products}
+            # Una venta nueva lista todos sus productos en `product_ids`; las referencias
+            # guardadas antes de las líneas sólo tienen `product_id`. Se pregunta por los
+            # dos dentro de la misma venta, así no hay que migrar los chats (que no se barren).
+            query["client.sales"] = {
+                "$elemMatch": {
+                    "$or": [
+                        {"product_id": {"$in": filters.sales_products}},
+                        {"product_ids": {"$in": filters.sales_products}},
+                    ]
+                }
+            }
 
         if filters.sales_date_range:
             query["client.sales.assigned_at"] = {

{letschatty-0.4.552 → letschatty-0.4.554}/src/letschatty/models/company/assets/chat_assets.py

@@ -9,9 +9,10 @@ from datetime import datetime
 from zoneinfo import ZoneInfo
 from bson import ObjectId
 import json
-from typing import Dict, Any, Optional, TYPE_CHECKING
+from typing import Dict, Any, List, Optional, TYPE_CHECKING
 from letschatty.models.utils.types.serializer_type import SerializerType
 from letschatty.models.company.assets.ai_agents_v2.chatty_ai_mode import ChattyAIMode
+from letschatty.models.company.assets.sin_producto import SIN_PRODUCTO
 
 if TYPE_CHECKING:
     from letschatty.models.company.assets.sale import Sale
@@ -88,10 +89,20 @@ class SaleAssignedToChat(AssignedAssetToChat):
     arman las IAs con `client.sales`. Hasta que existió `sold_at` las dos fechas eran la
     misma, así que las referencias ya guardadas cumplen esto sin migrar nada.
 
+    `product_id` es el de la primera línea de la venta, o `SIN_PRODUCTO` si no tiene
+    productos: nunca `null`, porque 35 servicios reconstruyen el chat con una versión del
+    paquete donde es obligatorio (ver `sin_producto.py`). `product_ids` lista todos los
+    productos de la venta; las referencias guardadas antes no lo tienen, y por eso quien
+    pregunta por productos mira los dos campos (`productos_vendidos`).
+
     Se construye y se re-sincroniza siempre desde la venta (`from_sale`, `synced_with`):
-    la fecha y el producto de la referencia no se escriben a mano.
+    la fecha y los productos de la referencia no se escriben a mano.
     """
     product_id: StrObjectId = Field(frozen=True)
+    product_ids: List[StrObjectId] = Field(
+        default_factory=list, frozen=True,
+        description="Todos los productos de la venta. Vacío en las referencias guardadas antes de las líneas.",
+    )
 
     @classmethod
     def from_sale(cls, sale: "Sale", assigned_by: StrObjectId) -> "SaleAssignedToChat":
@@ -101,8 +112,14 @@ class SaleAssignedToChat(AssignedAssetToChat):
             assigned_at=sale.sold_at,
             assigned_by=assigned_by,
             product_id=sale.product_id,
+            product_ids=sale.ids_de_productos(),
         )
 
+    def productos_vendidos(self) -> List[StrObjectId]:
+        """Los productos reales de la venta, venga la referencia de antes o de después de las líneas."""
+        ids = self.product_ids or [self.product_id]
+        return [product_id for product_id in ids if product_id != SIN_PRODUCTO]
+
     def synced_with(self, sale: "Sale", previous: "Sale") -> "SaleAssignedToChat":
         """La misma referencia (id, quién la cargó) al día con `sale`, la venta ya editada.
 
@@ -110,11 +127,15 @@ class SaleAssignedToChat(AssignedAssetToChat):
         de editar). Hay referencias viejas cuya fecha no coincide con el registro de su venta
         —medido en prod: 12 de 299, corridas exactamente 3 horas por un bug de 2024 que guardó
         hora local como UTC— y una edición que no toca la fecha no tiene por qué moverla.
+        Lo mismo `product_ids`: una referencia vieja no lo tiene, y editar el monto de su
+        venta no es razón para reescribirla.
 
         Devuelve una copia en vez de asignar porque los campos son frozen: la referencia es
         un valor, y un valor que cambia se reemplaza.
         """
-        cambios = {"product_id": sale.product_id}
+        cambios: Dict[str, Any] = {"product_id": sale.product_id}
+        if sale.ids_de_productos() != previous.ids_de_productos():
+            cambios["product_ids"] = sale.ids_de_productos()
         if sale.sold_at != previous.sold_at:
             cambios["assigned_at"] = sale.sold_at
         return self.model_copy(update=cambios)

{letschatty-0.4.552 → letschatty-0.4.554}/src/letschatty/models/company/assets/product.py

@@ -6,11 +6,12 @@ from .product_parameter_embed_config import (
     ProductParameterEmbedFormat,
 )
 from typing import Dict, Any, Optional, ClassVar
-from pydantic import Field, field_validator, BaseModel
+from pydantic import Field, field_validator, model_validator, BaseModel
 import pycountry
 from datetime import datetime
 from zoneinfo import ZoneInfo
 import json
+from .sin_producto import SIN_PRODUCTO
 
 class ProductPreview(ChattyAssetPreview):
     color: str = Field(default="#000000")
@@ -43,6 +44,13 @@ class Product(CompanyAssetModel):
     preview_class: ClassVar[type[ProductPreview]] = ProductPreview
     color: str = Field(default="#000000")
 
+    @model_validator(mode='after')
+    def _no_usa_el_id_de_las_ventas_sin_productos(self) -> 'Product':
+        """`SIN_PRODUCTO` es el `product_id` de una venta sin productos: no puede ser un producto."""
+        if self.id == SIN_PRODUCTO:
+            raise ValueError(f"El id {SIN_PRODUCTO} está reservado para las ventas sin productos")
+        return self
+
     @field_validator('price', mode='before')
     def validate_price(cls, v: Dict[str, float]):
         if v is None: