griptape-nodes 0.64.10__py3-none-any.whl → 0.65.0__py3-none-any.whl

griptape_nodes/retained_mode/retained_mode.py

@@ -928,7 +928,7 @@ class RetainedMode:
         )
         result = GriptapeNodes().handle_request(request)
 
-        if not result.succeeded():
+        if result.failed():
             return False, result
 
         # Navigate through indices
@@ -983,7 +983,7 @@ class RetainedMode:
         )
         result = GriptapeNodes().handle_request(request)
 
-        if not result.succeeded():
+        if result.failed():
             return result
 
         # Navigate to the proper location and set the value
@@ -1169,7 +1169,7 @@ class RetainedMode:
             )
             result = GriptapeNodes().handle_request(request)
 
-            if not result.succeeded():
+            if result.failed():
                 logger.error(
                     'set_value failed for "%s.%s", failed to get value for container "%s".',
                     node,

griptape_nodes/traits/button.py

@@ -104,6 +104,7 @@ class Button(Trait):
     loading_icon: str | None = None
     loading_icon_class: str | None = None
     tooltip: str | None = None
+    button_link: str | None = None
 
     element_id: str = field(default_factory=lambda: "Button")
     on_click_callback: OnClickCallback | None = field(default=None, init=False)
@@ -124,6 +125,7 @@ class Button(Trait):
         loading_icon: str | None = None,
         loading_icon_class: str | None = None,
         tooltip: str | None = None,
+        button_link: str | None = None,
         on_click: OnClickCallback | None = None,
         get_button_state: GetButtonStateCallback | None = None,
     ) -> None:
@@ -140,9 +142,42 @@ class Button(Trait):
         self.loading_icon = loading_icon
         self.loading_icon_class = loading_icon_class
         self.tooltip = tooltip
-        self.on_click_callback = on_click
+        self.button_link = button_link
+
+        # Validate that both button_link and on_click are not provided simultaneously
+        if button_link is not None and on_click is not None:
+            error_msg = (
+                "Cannot specify both 'button_link' and 'on_click' for Button. "
+                "Use 'button_link' for simple URL navigation or 'on_click' for custom behavior."
+            )
+            raise ValueError(error_msg)
+
+        # If button_link is provided and no custom on_click handler, create a default handler
+        if button_link is not None:
+            self.on_click_callback = self._create_button_link_handler(button_link)
+        else:
+            self.on_click_callback = on_click
         self.get_button_state_callback = get_button_state
 
+    def _create_button_link_handler(self, url: str) -> OnClickCallback:
+        """Create a default handler for button_link URLs."""
+
+        def handler(
+            button: Button,  # noqa: ARG001
+            button_details: ButtonDetailsMessagePayload,
+        ) -> NodeMessageResult:
+            return NodeMessageResult(
+                success=True,
+                details="Opening URL",
+                response=OnClickMessageResultPayload(
+                    button_details=button_details,
+                    href=url,
+                ),
+                altered_workflow_state=False,
+            )
+
+        return handler
+
     @classmethod
     def get_trait_keys(cls) -> list[str]:
         return ["button", "addbutton"]
@@ -195,7 +230,7 @@ class Button(Trait):
 
         return options
 
-    def on_message_received(self, message_type: str, message: NodeMessagePayload | None) -> NodeMessageResult | None:  # noqa: PLR0911
+    def on_message_received(self, message_type: str, message: NodeMessagePayload | None) -> NodeMessageResult | None:  # noqa: C901, PLR0911, PLR0912
         """Handle messages sent to this button trait.
 
         Args:
@@ -211,6 +246,13 @@ class Button(Trait):
                     try:
                         # Pre-fill button details with current state and pass to callback
                         button_details = self.get_button_details()
+                        # Include original message's data if present (for payloadData support)
+                        if message is not None:
+                            # Handle both NodeMessagePayload objects and dict messages
+                            if isinstance(message, NodeMessagePayload) and message.data is not None:
+                                button_details.data = message.data
+                            elif isinstance(message, dict) and "data" in message:
+                                button_details.data = message["data"]
                         result = self.on_click_callback(self, button_details)
 
                         # If callback returns None, provide optimistic success result

griptape_nodes/traits/file_system_picker.py

@@ -16,7 +16,7 @@ class FileSystemPicker(Trait):
     include_patterns: list[str] = field(default_factory=list)
     max_file_size: int | None = None
     min_file_size: int | None = None
-    workspace_only: bool = True
+    workspace_only: bool = False
     initial_path: str | None = None
     allow_create: bool = False
     allow_rename: bool = False
@@ -34,7 +34,7 @@ class FileSystemPicker(Trait):
         include_patterns: list[str] | None = None,
         max_file_size: int | None = None,
         min_file_size: int | None = None,
-        workspace_only: bool = True,
+        workspace_only: bool = False,
         initial_path: str | None = None,
         allow_create: bool = False,
         allow_rename: bool = False,

griptape_nodes/utils/file_utils.py

@@ -0,0 +1,101 @@
+"""Utilities for file and directory operations."""
+
+from __future__ import annotations
+
+import logging
+import os
+from fnmatch import fnmatch
+from pathlib import Path
+
+logger = logging.getLogger(__name__)
+
+
+def find_file_in_directory(directory: Path, pattern: str) -> Path | None:
+    """Search directory recursively for a file matching the given pattern.
+
+    Args:
+        directory: Directory to search in
+        pattern: Glob pattern to match files against (e.g., '*.json', '*library*.json')
+
+    Returns:
+        Path to the first matching file if found, None otherwise.
+        Logs a warning if multiple files match the pattern.
+
+    Examples:
+        >>> find_file_in_directory(Path("/workspace"), "config.json")
+        Path("/workspace/subdir/config.json")
+        >>> find_file_in_directory(Path("/workspace"), "*library*.json")
+        Path("/workspace/libs/my_library.json")
+        >>> find_file_in_directory(Path("/empty"), "missing.txt")
+        None
+    """
+    if not directory.exists():
+        logger.debug("Directory does not exist: %s", directory)
+        return None
+
+    if not directory.is_dir():
+        logger.debug("Path is not a directory: %s", directory)
+        return None
+
+    matches = []
+    for root, _, files_found in os.walk(directory):
+        for file in files_found:
+            if fnmatch(file, pattern):
+                found_path = Path(root) / file
+                matches.append(found_path)
+
+    if not matches:
+        logger.debug("No files matching pattern '%s' found in directory: %s", pattern, directory)
+        return None
+
+    if len(matches) > 1:
+        for _match in matches:
+            pass
+        logger.warning(
+            "Found multiple files matching pattern '%s' in %s, using first one at %s",
+            pattern,
+            directory,
+            matches[0],
+        )
+
+    logger.debug("Found file matching pattern '%s' at: %s", pattern, matches[0])
+    return matches[0]
+
+
+def find_all_files_in_directory(directory: Path, pattern: str) -> list[Path]:
+    """Search directory recursively for all files matching the given pattern.
+
+    Args:
+        directory: Directory to search in
+        pattern: Glob pattern to match files against (e.g., '*.json', '*library*.json')
+
+    Returns:
+        List of all matching file paths. Returns empty list if none found.
+
+    Examples:
+        >>> find_all_files_in_directory(Path("/workspace"), "*.json")
+        [Path("/workspace/a.json"), Path("/workspace/sub/b.json")]
+        >>> find_all_files_in_directory(Path("/empty"), "*.txt")
+        []
+    """
+    if not directory.exists():
+        logger.debug("Directory does not exist: %s", directory)
+        return []
+
+    if not directory.is_dir():
+        logger.debug("Path is not a directory: %s", directory)
+        return []
+
+    matches = []
+    for root, _, files_found in os.walk(directory):
+        for file in files_found:
+            if fnmatch(file, pattern):
+                found_path = Path(root) / file
+                matches.append(found_path)
+
+    if not matches:
+        logger.debug("No files matching pattern '%s' found in directory: %s", pattern, directory)
+    else:
+        logger.debug("Found %d file(s) matching pattern '%s' in directory: %s", len(matches), pattern, directory)
+
+    return matches