letschatty 0.4.485__tar.gz → 0.4.487__tar.gz

{letschatty-0.4.485 → letschatty-0.4.487}/PKG-INFO

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

{letschatty-0.4.485 → letschatty-0.4.487}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "letschatty"
-version = "0.4.485"
+version = "0.4.487"
 description = "Models and custom classes to work across the Chattyverse"
 authors = ["Axel <axel@letschatty.com>"]
 license = "MIT"

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/chat/chat.py

@@ -50,6 +50,7 @@ class Chat(CompanyAssetModel):
     suggested_messages: List[MessageDraft] = Field(default_factory=list)
     topics_timeline: List[TopicTimelineEntry] = Field(default_factory=list, description="Timeline of conversation topics throughout the conversation")
     active_funnel: Optional[ActiveFunnel] = Field(default=None, description="Current active funnel for this chat")
+    proximo_recordatorio_at: Optional[datetime] = Field(default=None, frozen=True, description="FROZEN A PROPOSITO: lo escribe SOLO el $set del recalculo, nunca un save del chat. Sin frozen, cualquier guardado de documento completo (`update` mergea {**actuales, **entrantes}) lo pisaria con el valor que tuviera en memoria ese flujo, que puede ser anterior al recalculo — el badge desapareceria sin que nadie toque un recordatorio. Vencimiento del recordatorio SIN RESOLVER mas cercano de este chat, o None si no hay ninguno. Espejo del minimo sobre la coleccion `recordatorios`, para que la bandeja pueda mostrar el badge, filtrar y ordenar sin un join contra otra coleccion. Es una FECHA y no un booleano a proposito: 'vencido' se evalua al momento de la query (`<= ahora`), asi que el que se mueve es el reloj y no el dato, y no hace falta ningun barrido que lo actualice.")
     model_config = ConfigDict(extra="ignore")
 
     @property

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/chat/preview.py

@@ -68,6 +68,7 @@ class ChatPreview(BaseModel):
     scheduled_message : Optional[ScheduledMessageStatus] = Field(default=None)
     tags : Optional[List[TagPreview]] = Field(default=[])
     chatty_ai_agent_in_chat_status: Optional[ChattyAiAgentStatusPreview] = Field(default=None)
+    proximo_recordatorio_at: Optional[datetime] = Field(default=None, description="Vencimiento del recordatorio sin resolver mas cercano del chat. El frontend compara contra su propio reloj para decidir si va badge y de que color: el backend no decide que es hoy, porque el huso que importa es el del agente.")
     @property
     def assigned_agent_id(self) -> Optional[StrObjectId]:
         if self.area_status != Area.WITH_AGENT:

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/company/assets/media/__init__.py

@@ -1,2 +1,2 @@
 from .file import MediaFile, MediaType
-from .media_asset import MediaAsset, MediaAssetPreview, MediaOrigin
+from .media_asset import MediaAsset, MediaAssetPreview, MediaOrigin, MetaCompatStatus

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/company/assets/media/media_asset.py

@@ -20,6 +20,18 @@ class MediaOrigin(StrEnum):
     SYSTEM = "system"       # reserved (logos, avatars)
 
 
+class MetaCompatStatus(StrEnum):
+    """Whether this asset is known to satisfy Meta's media constraints.
+
+    Only videos are ever normalized, so this stays None for every other type — and
+    for every row written before the normalizer shipped. None means "not checked",
+    NOT "incompatible": the send path re-probes anything that isn't OK.
+    """
+    OK = "ok"                  # probed and compliant (either natively or after normalization)
+    OPTIMIZING = "optimizing"  # handed off to the video-normalizer lambda; not sendable yet
+    FAILED = "failed"          # normalization could not produce a compliant file
+
+
 class MediaAssetPreview(ChattyAssetPreview):
     type: MediaType
     url: str
@@ -28,6 +40,8 @@ class MediaAssetPreview(ChattyAssetPreview):
     is_starred: bool = False
     width: Optional[int] = None
     height: Optional[int] = None
+    # The picker needs this to grey out a video that is still being normalized.
+    meta_compat: Optional["MetaCompatStatus"] = None
 
     @classmethod
     def get_projection(cls) -> dict[str, Any]:
@@ -39,6 +53,7 @@ class MediaAssetPreview(ChattyAssetPreview):
             "is_starred": 1,
             "width": 1,
             "height": 1,
+            "meta_compat": 1,
         }
 
     @classmethod
@@ -57,6 +72,7 @@ class MediaAssetPreview(ChattyAssetPreview):
             is_starred=asset.is_starred,
             width=asset.width,
             height=asset.height,
+            meta_compat=asset.meta_compat,
         )
 
 
@@ -83,6 +99,18 @@ class MediaAsset(CompanyAssetModel):
     height: Optional[int] = None
     duration_ms: Optional[int] = None
 
+    # Meta compatibility — videos only (see MetaCompatStatus). None on every other
+    # type and on every row predating the normalizer.
+    meta_compat: Optional[MetaCompatStatus] = None
+    meta_compat_started_at: Optional[datetime] = Field(
+        default=None,
+        description=(
+            "When OPTIMIZING was stamped. The lambda can die without ever calling back, "
+            "so readers treat a row stuck here past the lambda timeout as FAILED rather "
+            "than waiting forever."
+        ),
+    )
+
     # provenance
     origin: MediaOrigin = MediaOrigin.UPLOADED
     uploaded_by: Optional[str] = Field(default=None, description="user_id of uploader/promoter; None for seeded rows")

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/messages/meta_message_model/schema/values/value_errors.py

@@ -52,6 +52,11 @@ def get_error_details(code: int) -> Optional[Dict[str, str]]:
 FRIENDLY_AGENT_TEXT_ES: Dict[int, str] = {
     131047: "El cliente no escribió en las últimas 24 hs. Enviá una plantilla para retomar la conversación.",
     131051: "Tipo de mensaje no soportado para este envío.",
+    # Media (/media endpoint). Without these two, EVERY media rejection — bad codec,
+    # oversized, corrupt — collapsed into the generic _FRIENDLY_DEFAULT, which is why
+    # video failures were misdiagnosed as a size problem for months.
+    131052: "WhatsApp no pudo descargar el archivo. Volvé a subirlo.",
+    131053: "WhatsApp rechazó el archivo por su formato. Los videos tienen que ser MP4 con video H.264 y audio AAC.",
     131026: "El número no puede recibir mensajes de WhatsApp.",
     131021: "No se puede enviar un mensaje a este número.",
     131049: "WhatsApp no entregó el mensaje por políticas de Meta.",
@@ -68,6 +73,10 @@ FRIENDLY_AGENT_TEXT_ES: Dict[int, str] = {
 }
 
 _FRIENDLY_DEFAULT = "No se pudo enviar el mensaje."
+_FRIENDLY_MEDIA_REJECTED = (
+    "WhatsApp rechazó el archivo. Los videos tienen que ser MP4 con video H.264 y audio AAC, "
+    "y pesar menos de 16 MB."
+)
 _FRIENDLY_AUTH = "La conexión con WhatsApp expiró. Reconectá la cuenta."
 _FRIENDLY_TRANSIENT = "Error temporal de WhatsApp. Se reintentó sin éxito, probá de nuevo en unos minutos."
 _FRIENDLY_4XX = "WhatsApp rechazó el mensaje."
@@ -75,15 +84,24 @@ _FRIENDLY_4XX = "WhatsApp rechazó el mensaje."
 
 def friendly_failure_reason(*, meta_code: Optional[int] = None,
                             http_status: Optional[int] = None,
-                            transient: bool = False) -> str:
+                            transient: bool = False,
+                            media_upload: bool = False,
+                            meta_message: Optional[str] = None) -> str:
     """Map a failed outbound send to a short, agent-facing Spanish reason.
 
-    Precedence: a known Meta error code wins; then HTTP 401 (auth/expired token);
-    then transient/5xx; then any other 4xx; else a generic fallback. Pure function,
-    no side effects — safe to call anywhere and trivial to unit-test.
+    Precedence: a known Meta error code wins; then a refused media upload; then HTTP 401
+    (auth/expired token); then transient/5xx; then any other 4xx; else a generic fallback.
+    Pure function, no side effects — safe to call anywhere and trivial to unit-test.
     """
     if meta_code is not None and meta_code in FRIENDLY_AGENT_TEXT_ES:
         return FRIENDLY_AGENT_TEXT_ES[meta_code]
+    if media_upload:
+        # A file Meta refused. It answers /media with generic codes far more often than
+        # the documented 131052/131053, so keying only on the code would drop the agent
+        # back to "WhatsApp rechazó el mensaje." — which is exactly the blind spot that
+        # made video failures unreadable. Say it was the file, and quote Meta verbatim.
+        base = _FRIENDLY_MEDIA_REJECTED
+        return f"{base} (WhatsApp: {meta_message})" if meta_message else base
     if http_status == 401 or meta_code == 190:
         return _FRIENDLY_AUTH
     if transient or (http_status is not None and 500 <= http_status < 600):

{letschatty-0.4.485 → letschatty-0.4.487}/src/letschatty/models/utils/custom_exceptions/custom_exceptions.py

@@ -52,13 +52,21 @@ class UnauthorizedOrigin(CustomException):
         super().__init__(message, status_code=status_code, **context_data)
 
 class WhatsAppAPIError(CustomException):
-    def __init__(self, message="WhatsApp API error", status_code=400, meta_code: Optional[int] = None, transient: bool = False, **context_data):
+    def __init__(self, message="WhatsApp API error", status_code=400, meta_code: Optional[int] = None, transient: bool = False, media_upload: bool = False, meta_message: Optional[str] = None, **context_data):
         super().__init__(message, status_code=status_code, **context_data)
         # meta_code: Meta's numeric error code parsed from the response body (when present).
         # transient: True for retryable transport failures (clean 5xx). Both default-safe so
         # existing call sites that don't pass them keep working.
         self.meta_code = meta_code
         self.transient = transient
+        # media_upload: the failure came from POST /media (the file itself was refused),
+        # not from POST /messages. Meta answers those with generic codes (100 "Invalid
+        # parameter") much more often than the 131052/131053 the async webhook uses, so
+        # the code alone cannot tell an agent the FILE was the problem — this flag can.
+        self.media_upload = media_upload
+        # meta_message: Meta's own explanation, kept verbatim. friendly_failure_reason
+        # only maps known codes, so without this the specific text was always discarded.
+        self.meta_message = meta_message
 
 class WhatsAppPayloadValidationError(Exception):
     def __init__(self, message="WhatsApp payload validation error", status_code=400, **context_data):