@danhachuel/thunderbolt 0.5.17 → 0.5.19

package/hermes_ui/canva_mcp_workflow.py

@@ -71,24 +71,50 @@ def _first_id(value: Any, keys: tuple[str, ...]) -> str:
     return ""
 
 
-def _first_text_element(value: Any) -> str:
+def _first_text_element(value: Any, *, text_context: bool = False) -> str:
     if isinstance(value, Mapping):
-        for key in ("element_id", "id"):
+        # The MCP returns rich text elements with slightly different ID
+        # names depending on the design type/version of the server.
+        text_markers = ("text", "richtext", "rich_text", "content", "paragraph")
+        mapping_text = str(value).lower()
+        is_text_mapping = text_context or any(
+            marker in str(key).lower() or marker in mapping_text
+            for key in value
+            for marker in text_markers
+        )
+        for key in ("element_id", "elementId", "richtext_id", "richTextId", "text_id", "textId", "id"):
             candidate = str(value.get(key) or "").strip()
-            if candidate and any(token in str(value).lower() for token in ("text", "richtext", "content")):
+            if candidate and is_text_mapping:
                 return candidate
-        for child in value.values():
-            found = _first_text_element(child)
+        for key, child in value.items():
+            child_context = text_context or any(marker in str(key).lower() for marker in text_markers)
+            found = _first_text_element(child, text_context=child_context)
             if found:
                 return found
     elif isinstance(value, list):
         for child in value:
-            found = _first_text_element(child)
+            found = _first_text_element(child, text_context=text_context)
             if found:
                 return found
     return ""
 
 
+def _has_richtext(value: Any) -> bool:
+    """Return whether a Canva payload contains at least one richtext item."""
+    if isinstance(value, Mapping):
+        for key, child in value.items():
+            if str(key).lower() in {"richtexts", "rich_texts", "text_elements", "textelements"}:
+                if isinstance(child, list) and any(isinstance(item, Mapping) for item in child):
+                    return True
+                if isinstance(child, Mapping) and child:
+                    return True
+            if _has_richtext(child):
+                return True
+    elif isinstance(value, list):
+        return any(_has_richtext(child) for child in value)
+    return False
+
+
 def _first_page(value: Any) -> tuple[str, list[dict[str, Any]]]:
     pages: list[dict[str, Any]] = []
     if isinstance(value, Mapping) and isinstance(value.get("pages"), list):
@@ -122,17 +148,27 @@ def run_direct_canva_thumbnail(
         candidates = _items(result)
         if not candidates:
             raise CanvaMCPError("Canva não encontrou designs para as palavras-chave da thumbnail.")
-        selected = next((candidate for candidate in candidates if _design_id(candidate)), candidates[0])
-        design_id = _design_id(selected)
-        if not design_id:
+        candidates = [candidate for candidate in candidates if _design_id(candidate)]
+        if not candidates:
             raise CanvaMCPError("A pesquisa Canva devolveu um design sem ID.")
 
         get_name = "get-design-content" if "get-design-content" in available else "get_design_content"
-        content = _payload(client.call(get_name, {
-            "design_id": design_id,
-            "content_types": ["richtexts"],
-            "user_intent": "Inspect the selected Canva design before applying the thumbnail edit.",
-        }))
+        selected = None
+        content: Any = {}
+        for candidate in candidates:
+            candidate_id = _design_id(candidate)
+            candidate_content = _payload(client.call(get_name, {
+                "design_id": candidate_id,
+                "content_types": ["richtexts"],
+                "user_intent": "Inspect the selected Canva design before applying the thumbnail edit.",
+            }))
+            if _has_richtext(candidate_content):
+                selected = candidate
+                content = candidate_content
+                break
+        if selected is None:
+            raise CanvaMCPError("Os designs Canva encontrados não têm elementos de texto editáveis.")
+        design_id = _design_id(selected)
         start_name = "start-editing-transaction" if "start-editing-transaction" in available else "start_editing_transaction"
         started = _payload(client.call(start_name, {
             "design_id": design_id,
@@ -144,6 +180,12 @@ def run_direct_canva_thumbnail(
         page_id, pages = _first_page(started)
         element_id = _first_text_element(started) or _first_text_element(content)
         if not element_id:
+            cancel_name = "cancel-editing-transaction" if "cancel-editing-transaction" in available else "cancel_editing_transaction"
+            if cancel_name in available:
+                client.call(cancel_name, {
+                    "transaction_id": transaction_id,
+                    "user_intent": "Discard the Canva editing transaction because no editable text element was returned.",
+                })
             raise CanvaMCPError("O design Canva não devolveu um elemento de texto editável.")
         perform_name = "perform-editing-operations" if "perform-editing-operations" in available else "perform_editing_operations"
         operations = [{"type": "replace_text", "element_id": element_id, "text": title[:255]}]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danhachuel/thunderbolt",
-  "version": "0.5.17",
+  "version": "0.5.19",
   "description": "Thunderbolt — interface local para operação de canais faceless e motor MoneyPrinterTurbo",
   "license": "MIT",
   "main": "scripts/cli.mjs",