griptape-nodes 0.52.1__py3-none-any.whl → 0.54.0__py3-none-any.whl

griptape_nodes/retained_mode/managers/node_manager.py

@@ -203,6 +203,7 @@ class NodeManager:
         event_manager.assign_manager_to_request_type(DuplicateSelectedNodesRequest, self.on_duplicate_selected_nodes)
         event_manager.assign_manager_to_request_type(SetLockNodeStateRequest, self.on_toggle_lock_node_request)
         event_manager.assign_manager_to_request_type(GetFlowForNodeRequest, self.on_get_flow_for_node_request)
+        event_manager.assign_manager_to_request_type(SendNodeMessageRequest, self.on_send_node_message_request)
 
     def handle_node_rename(self, old_name: str, new_name: str) -> None:
         # Get the node itself
@@ -253,7 +254,6 @@ class NodeManager:
                 details = (
                     "Attempted to create Node in the Current Context. Failed because the Current Context was empty."
                 )
-                logger.error(details)
                 return CreateNodeResultFailure(result_details=details)
             parent_flow = GriptapeNodes.ContextManager().get_current_flow()
             parent_flow_name = parent_flow.name
@@ -265,7 +265,6 @@ class NodeManager:
                 parent_flow = flow_mgr.get_flow_by_name(parent_flow_name)
             except KeyError as err:
                 details = f"Attempted to create Node of type '{request.node_type}'. Failed when attempting to find the parent Flow. Error: {err}"
-                logger.error(details)
                 return CreateNodeResultFailure(result_details=details)
 
         # Now ensure that we're giving a valid name.
@@ -278,7 +277,6 @@ class NodeManager:
                 )
             except KeyError as err:
                 details = f"Attempted to create Node of type '{request.node_type}'. Failed when attempting to find the library this node type was in. Error: {err}"
-                logger.error(details)
                 return CreateNodeResultFailure(result_details=details)
 
             node_metadata = dest_library.get_node_metadata(request.node_type)
@@ -308,7 +306,6 @@ class NodeManager:
 
             traceback.print_exc()
             details = f"Could not create Node '{final_node_name}' of type '{request.node_type}': {err}"
-            logger.error(details)
 
             # Check if we should create an Error Proxy node instead of failing
             if request.create_error_proxy_on_failure:
@@ -329,7 +326,6 @@ class NodeManager:
                     )
                 except Exception as proxy_err:
                     details = f"Failed to create Error Proxy (placeholder) node: {proxy_err}"
-                    logger.error(details)
                     return CreateNodeResultFailure(result_details=details)
             else:
                 return CreateNodeResultFailure(result_details=details)
@@ -366,8 +362,6 @@ class NodeManager:
             log_level = logging.WARNING
             details = f"{details}. WARNING: Had to rename from original node name requested '{request.node_name}' as an object with this name already existed."
 
-        logger.log(level=log_level, msg=details)
-
         # Special handling for paired classes (e.g., create a Start node and it automatically creates a corresponding End node already connected).
         if isinstance(node, StartLoopNode) and not request.initial_setup:
             # If it's StartLoop, create an EndLoop and connect it to the StartLoop.
@@ -381,7 +375,7 @@ class NodeManager:
             # Check and see if the class exists
             libraries_with_node_type = LibraryRegistry.get_libraries_with_node_type(end_class_name)
             if not libraries_with_node_type:
-                msg = f"Attempted to create a paried set of nodes for Node '{final_node_name}'. Failed because paired class '{end_class_name}' does not exist for start class '{node_class_name}'. The corresponding node will have to be created by hand and attached manually."
+                msg = f"Attempted to create a paired set of nodes for Node '{final_node_name}'. Failed because paired class '{end_class_name}' does not exist for start class '{node_class_name}'. The corresponding node will have to be created by hand and attached manually."
                 logger.error(msg)  # while this is bad, it's not unsalvageable, so we'll consider this a success.
             else:
                 # Create the EndNode
@@ -419,7 +413,10 @@ class NodeManager:
                         end_node.start_node = node
 
         return CreateNodeResultSuccess(
-            node_name=node.name, node_type=node.__class__.__name__, specific_library_name=request.specific_library_name
+            node_name=node.name,
+            node_type=node.__class__.__name__,
+            specific_library_name=request.specific_library_name,
+            result_details=ResultDetails(message=details, level=log_level),
         )
 
     def cancel_conditionally(
@@ -448,35 +445,32 @@ class NodeManager:
             # get the current node executing / resolving
             # if it's in connected nodes, cancel flow.
             # otherwise, leave it.
-            control_node_name, resolving_node_name = GriptapeNodes.FlowManager().flow_state(parent_flow)
+            control_node_names, resolving_node_names = GriptapeNodes.FlowManager().flow_state(parent_flow)
             connected_nodes = parent_flow.get_all_connected_nodes(node)
             cancelled = False
-            if control_node_name is not None:
-                control_node = GriptapeNodes.ObjectManager().get_object_by_name(control_node_name)
-                if control_node in connected_nodes:
-                    result = GriptapeNodes.handle_request(CancelFlowRequest(flow_name=parent_flow_name))
-                    cancelled = True
-                    if not result.succeeded():
-                        details = (
-                            f"Attempted to delete a Node '{node.name}'. Failed because running flow could not cancel."
-                        )
-                        logger.error(details)
-                        return DeleteNodeResultFailure(result_details=details)
-            if resolving_node_name is not None and not cancelled:
-                resolving_node = GriptapeNodes.ObjectManager().get_object_by_name(resolving_node_name)
-                if resolving_node in connected_nodes:
-                    result = GriptapeNodes.handle_request(CancelFlowRequest(flow_name=parent_flow_name))
-                    if not result.succeeded():
-                        details = (
-                            f"Attempted to delete a Node '{node.name}'. Failed because running flow could not cancel."
-                        )
-                        logger.error(details)
-                        return DeleteNodeResultFailure(result_details=details)
+            if control_node_names is not None:
+                for control_node_name in control_node_names:
+                    control_node = GriptapeNodes.ObjectManager().get_object_by_name(control_node_name)
+                    if control_node in connected_nodes:
+                        result = GriptapeNodes.handle_request(CancelFlowRequest(flow_name=parent_flow_name))
+                        cancelled = True
+                        if not result.succeeded():
+                            details = f"Attempted to delete a Node '{node.name}'. Failed because running flow could not cancel."
+                            return DeleteNodeResultFailure(result_details=details)
+            if resolving_node_names is not None and not cancelled:
+                for resolving_node_name in resolving_node_names:
+                    resolving_node = GriptapeNodes.ObjectManager().get_object_by_name(resolving_node_name)
+                    if resolving_node in connected_nodes:
+                        result = GriptapeNodes.handle_request(CancelFlowRequest(flow_name=parent_flow_name))
+                        if not result.succeeded():
+                            details = f"Attempted to delete a Node '{node.name}'. Failed because running flow could not cancel."
+                            return DeleteNodeResultFailure(result_details=details)
+                        break  # Only need to cancel once
             # Clear the execution queue, because we don't want to hit this node eventually.
             parent_flow.clear_execution_queue()
         return None
 
-    def on_delete_node_request(self, request: DeleteNodeRequest) -> ResultPayload:  # noqa: C901, PLR0911, PLR0912, PLR0915 (complex logic, lots of edge cases)
+    def on_delete_node_request(self, request: DeleteNodeRequest) -> ResultPayload:  # noqa: C901, PLR0911, PLR0912 (complex logic, lots of edge cases)
         node_name = request.node_name
         node = None
         if node_name is None:
@@ -485,7 +479,6 @@ class NodeManager:
                 details = (
                     "Attempted to delete a Node from the Current Context. Failed because the Current Context is empty."
                 )
-                logger.error(details)
                 return DeleteNodeResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -494,7 +487,6 @@ class NodeManager:
             node = GriptapeNodes.ObjectManager().attempt_get_object_by_name_as_type(node_name, BaseNode)
         if node is None:
             details = f"Attempted to delete a Node '{node_name}', but no such Node was found."
-            logger.error(details)
             return DeleteNodeResultFailure(result_details=details)
 
         with GriptapeNodes.ContextManager().node(node=node):
@@ -503,7 +495,6 @@ class NodeManager:
                 parent_flow = GriptapeNodes.FlowManager().get_flow_by_name(parent_flow_name)
             except KeyError as err:
                 details = f"Attempted to delete a Node '{node_name}'. Error: {err}"
-                logger.error(details)
                 return DeleteNodeResultFailure(result_details=details)
 
             cancel_result = self.cancel_conditionally(parent_flow, parent_flow_name, node)
@@ -519,7 +510,6 @@ class NodeManager:
                 list_connections_result = GriptapeNodes.handle_request(request=list_node_connections_request)
                 if not isinstance(list_connections_result, ListConnectionsForNodeResultSuccess):
                     details = f"Attempted to delete a Node '{node_name}'. Failed because it could not gather Connections to the Node."
-                    logger.error(details)
                     return DeleteNodeResultFailure(result_details=details)
 
                 # Check incoming connections
@@ -537,7 +527,6 @@ class NodeManager:
                         details = (
                             f"Attempted to delete a Node '{node_name}'. Failed when attempting to delete Connection."
                         )
-                        logger.error(details)
                         return DeleteNodeResultFailure(result_details=details)
                     continue  # Refresh connection list after cascading deletions
 
@@ -556,7 +545,6 @@ class NodeManager:
                         details = (
                             f"Attempted to delete a Node '{node_name}'. Failed when attempting to delete Connection."
                         )
-                        logger.error(details)
                         return DeleteNodeResultFailure(result_details=details)
 
         # Remove from the owning Flow
@@ -571,9 +559,7 @@ class NodeManager:
             GriptapeNodes.ContextManager().pop_node()
 
         details = f"Successfully deleted Node '{node_name}'."
-        logger.debug(details)
-
-        return DeleteNodeResultSuccess()
+        return DeleteNodeResultSuccess(result_details=details)
 
     def on_get_node_resolution_state_request(self, request: GetNodeResolutionStateRequest) -> ResultPayload:
         node_name = request.node_name
@@ -582,7 +568,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to get resolution state for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return GetNodeResolutionStateResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -594,18 +579,13 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to get resolution state for a Node '{node_name}', but no such Node was found."
-                logger.error(details)
                 result = GetNodeResolutionStateResultFailure(result_details=details)
                 return result
 
         node_state = node.state
 
         details = f"Successfully got resolution state for Node '{node_name}'."
-        logger.debug(details)
-
-        result = GetNodeResolutionStateResultSuccess(
-            state=node_state.name,
-        )
+        result = GetNodeResolutionStateResultSuccess(state=node_state.name, result_details=details)
         return result
 
     def on_get_node_metadata_request(self, request: GetNodeMetadataRequest) -> ResultPayload:
@@ -615,7 +595,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to get metadata for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return GetNodeMetadataResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -628,18 +607,13 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to get metadata for a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = GetNodeMetadataResultFailure(result_details=details)
                 return result
 
         metadata = node.metadata
         details = f"Successfully retrieved metadata for a Node '{node_name}'."
-        logger.debug(details)
-
-        result = GetNodeMetadataResultSuccess(
-            metadata=metadata,
-        )
+        result = GetNodeMetadataResultSuccess(metadata=metadata, result_details=details)
         return result
 
     def on_set_node_metadata_request(self, request: SetNodeMetadataRequest) -> ResultPayload:
@@ -649,7 +623,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to set metadata for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return SetNodeMetadataResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -662,7 +635,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to set metadata for a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = SetNodeMetadataResultFailure(result_details=details)
                 return result
@@ -671,9 +643,7 @@ class NodeManager:
         for key, value in request.metadata.items():
             node.metadata[key] = value
         details = f"Successfully set metadata for a Node '{node_name}'."
-        logger.debug(details)
-
-        result = SetNodeMetadataResultSuccess()
+        result = SetNodeMetadataResultSuccess(result_details=details)
         return result
 
     def on_batch_set_node_metadata_request(self, request: BatchSetNodeMetadataRequest) -> ResultPayload:
@@ -714,7 +684,11 @@ class NodeManager:
                 result_details=f"Failed to update any nodes. Failed nodes: {failed_nodes}"
             )
 
-        return BatchSetNodeMetadataResultSuccess(updated_nodes=updated_nodes, failed_nodes=failed_nodes)
+        return BatchSetNodeMetadataResultSuccess(
+            updated_nodes=updated_nodes,
+            failed_nodes=failed_nodes,
+            result_details=f"Successfully updated metadata for {len(updated_nodes)} nodes.",
+        )
 
     def on_list_connections_for_node_request(self, request: ListConnectionsForNodeRequest) -> ResultPayload:
         node_name = request.node_name
@@ -723,7 +697,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to list Connections for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return ListConnectionsForNodeResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -736,7 +709,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to list Connections for a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = ListConnectionsForNodeResultFailure(result_details=details)
                 return result
@@ -746,7 +718,6 @@ class NodeManager:
             GriptapeNodes.FlowManager().get_flow_by_name(parent_flow_name)
         except KeyError as err:
             details = f"Attempted to list Connections for a Node '{node_name}'. Error: {err}"
-            logger.error(details)
 
             result = ListConnectionsForNodeResultFailure(result_details=details)
             return result
@@ -783,11 +754,10 @@ class NodeManager:
             ]
 
         details = f"Successfully listed all Connections to and from Node '{node_name}'."
-        logger.debug(details)
-
         result = ListConnectionsForNodeResultSuccess(
             incoming_connections=incoming_connections_list,
             outgoing_connections=outgoing_connections_list,
+            result_details=details,
         )
         return result
 
@@ -799,7 +769,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to list Parameters for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return ListParametersOnNodeResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -811,7 +780,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to list Parameters for a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = ListParametersOnNodeResultFailure(result_details=details)
                 return result
@@ -819,11 +787,7 @@ class NodeManager:
         ret_list = [param.name for param in node.parameters]
 
         details = f"Successfully listed Parameters for Node '{node_name}'."
-        logger.debug(details)
-
-        result = ListParametersOnNodeResultSuccess(
-            parameter_names=ret_list,
-        )
+        result = ListParametersOnNodeResultSuccess(parameter_names=ret_list, result_details=details)
         return result
 
     def generate_unique_parameter_name(self, node: BaseNode, base_name: str) -> str:
@@ -852,7 +816,6 @@ class NodeManager:
             # Get from the current context.
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to add Parameter to a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return AddParameterToNodeResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -864,7 +827,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to add Parameter '{request.parameter_name}' to a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = AddParameterToNodeResultFailure(result_details=details)
                 return result
@@ -872,7 +834,6 @@ class NodeManager:
         # Check if node is locked
         if node.lock:
             details = f"Attempted to add Parameter '{request.parameter_name}' to Node '{node_name}'. Failed because the Node was locked."
-            logger.error(details)
             result = AddParameterToNodeResultFailure(result_details=details)
             return result
 
@@ -880,12 +841,10 @@ class NodeManager:
             parameter = node.get_parameter_by_name(request.parent_container_name)
             if parameter is None:
                 details = f"Attempted to add Parameter to Container Parameter '{request.parent_container_name}' in node '{node_name}'. Failed because parameter didn't exist."
-                logger.error(details)
                 result = AddParameterToNodeResultFailure(result_details=details)
                 return result
             if not isinstance(parameter, ParameterContainer):
                 details = f"Attempted to add Parameter to Container Parameter '{request.parent_container_name}' in node '{node_name}'. Failed because parameter wasn't a container."
-                logger.error(details)
                 result = AddParameterToNodeResultFailure(result_details=details)
                 return result
             try:
@@ -897,11 +856,13 @@ class NodeManager:
                 return result
 
             return AddParameterToNodeResultSuccess(
-                parameter_name=new_param.name, type=new_param.type, node_name=node_name
+                parameter_name=new_param.name,
+                type=new_param.type,
+                node_name=node_name,
+                result_details=f"Successfully added parameter '{new_param.name}' to container parameter '{request.parent_container_name}' in node '{node_name}'.",
             )
         if request.parameter_name is None or request.tooltip is None:
             details = f"Attempted to add Parameter to node '{node_name}'. Failed because default_value, tooltip, or parameter_name was not defined."
-            logger.error(details)
             result = AddParameterToNodeResultFailure(result_details=details)
             return result
 
@@ -937,7 +898,6 @@ class NodeManager:
 
         if has_control_type and has_non_control_types:
             details = f"Attempted to add Parameter '{request.parameter_name}' to Node '{node_name}'. Failed because it had 'ParameterControlType' AND at least one other non-control type. If a Parameter is intended for control, it must only accept that type."
-            logger.error(details)
 
             result = AddParameterToNodeResultFailure(result_details=details)
             return result
@@ -976,7 +936,6 @@ class NodeManager:
                 node.add_parameter(new_param)
         except Exception as e:
             details = f"Couldn't add parameter with name {request.parameter_name} to Node '{node_name}'. Error: {e}"
-            logger.error(details)
             return AddParameterToNodeResultFailure(result_details=details)
 
         details = f"Successfully added Parameter '{final_param_name}' to Node '{node_name}'."
@@ -988,7 +947,7 @@ class NodeManager:
         logger.log(level=log_level, msg=details)
 
         result = AddParameterToNodeResultSuccess(
-            parameter_name=new_param.name, type=new_param.type, node_name=node_name
+            parameter_name=new_param.name, type=new_param.type, node_name=node_name, result_details=details
         )
         return result
 
@@ -1000,7 +959,6 @@ class NodeManager:
             # Get the Current Context
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to remove Parameter '{request.parameter_name}' from a Node, but no Current Context was found."
-                logger.error(details)
 
                 result = RemoveParameterFromNodeResultFailure(result_details=details)
                 return result
@@ -1014,14 +972,12 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to remove Parameter '{request.parameter_name}' from a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = RemoveParameterFromNodeResultFailure(result_details=details)
                 return result
         # Check if the node is locked
         if node.lock:
             details = f"Attempted to remove Element '{request.parameter_name}' from Node '{node_name}'. Failed because the Node was locked."
-            logger.error(details)
 
             result = RemoveParameterFromNodeResultFailure(result_details=details)
             return result
@@ -1029,7 +985,6 @@ class NodeManager:
         element = node.get_element_by_name_and_type(request.parameter_name)
         if element is None:
             details = f"Attempted to remove Element '{request.parameter_name}' from Node '{node_name}'. Failed because it didn't have an Element with that name on it."
-            logger.error(details)
 
             result = RemoveParameterFromNodeResultFailure(result_details=details)
             return result
@@ -1040,13 +995,14 @@ class NodeManager:
                 GriptapeNodes.handle_request(RemoveParameterFromNodeRequest(child.name, node_name))
             node.remove_parameter_element_by_name(request.parameter_name)
 
-            return RemoveParameterFromNodeResultSuccess()
+            return RemoveParameterFromNodeResultSuccess(
+                result_details=f"Successfully removed parameter group '{request.parameter_name}' and all its children from node '{node_name}'."
+            )
 
         # No tricky stuff, users!
         # if user_defined doesn't exist, or is false, then it's not user-defined
         if not getattr(element, "user_defined", False):
             details = f"Attempted to remove Element '{request.parameter_name}' from Node '{node_name}'. Failed because the Element was not user-defined (i.e., critical to the Node implementation). Only user-defined Elements can be removed from a Node."
-            logger.error(details)
 
             result = RemoveParameterFromNodeResultFailure(result_details=details)
             return result
@@ -1057,7 +1013,6 @@ class NodeManager:
             list_connections_result = GriptapeNodes.handle_request(request=list_node_connections_request)
             if not isinstance(list_connections_result, ListConnectionsForNodeResultSuccess):
                 details = f"Attempted to remove Parameter '{request.parameter_name}' from Node '{node_name}'. Failed because we were unable to get a list of Connections for the Parameter's Node."
-                logger.error(details)
 
                 result = RemoveParameterFromNodeResultFailure(result_details=details)
                 return result
@@ -1076,7 +1031,6 @@ class NodeManager:
                     delete_result = GriptapeNodes.handle_request(delete_request)
                     if isinstance(delete_result, DeleteConnectionResultFailure):
                         details = f"Attempted to remove Parameter '{request.parameter_name}' from Node '{node_name}'. Failed because we were unable to delete a Connection for that Parameter."
-                        logger.error(details)
 
                         result = RemoveParameterFromNodeResultFailure(result_details=details)
 
@@ -1092,7 +1046,6 @@ class NodeManager:
                     delete_result = GriptapeNodes.handle_request(delete_request)
                     if isinstance(delete_result, DeleteConnectionResultFailure):
                         details = f"Attempted to remove Parameter '{request.parameter_name}' from Node '{node_name}'. Failed because we were unable to delete a Connection for that Parameter."
-                        logger.error(details)
 
                         result = RemoveParameterFromNodeResultFailure(result_details=details)
 
@@ -1101,14 +1054,11 @@ class NodeManager:
             node.remove_parameter_element(element)
         else:
             details = f"Attempted to remove Element '{request.parameter_name}' from Node '{node_name}'. Failed because element didn't exist."
-            logger.error(details)
 
             result = RemoveParameterFromNodeResultFailure(result_details=details)
 
         details = f"Successfully removed Element '{request.parameter_name}' from Node '{node_name}'."
-        logger.debug(details)
-
-        result = RemoveParameterFromNodeResultSuccess()
+        result = RemoveParameterFromNodeResultSuccess(result_details=details)
         return result
 
     def on_get_parameter_details_request(self, request: GetParameterDetailsRequest) -> ResultPayload:
@@ -1118,7 +1068,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to get details for Parameter '{request.parameter_name}' from a Node, but no Current Context was found."
-                logger.error(details)
 
                 result = GetParameterDetailsResultFailure(result_details=details)
                 return result
@@ -1131,7 +1080,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to get details for Parameter '{request.parameter_name}' from a Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 result = GetParameterDetailsResultFailure(result_details=details)
                 return result
@@ -1141,7 +1089,6 @@ class NodeManager:
 
         if element is None:
             details = f"Attempted to get details for Element '{request.parameter_name}' from Node '{node_name}'. Failed because it didn't have an Element with that name on it."
-            logger.error(details)
             return GetParameterDetailsResultFailure(result_details=details)
 
         # Let's bundle up the details.
@@ -1155,9 +1102,7 @@ class NodeManager:
             allows_property = ParameterMode.PROPERTY in modes_allowed
             allows_output = ParameterMode.OUTPUT in modes_allowed
 
-            details = f"Successfully got details for Element '{request.parameter_name}' from Node '{node_name}'."
-            logger.debug(details)
-
+        details = f"Successfully got details for Element '{request.parameter_name}' from Node '{node_name}'."
         result = GetParameterDetailsResultSuccess(
             element_id=element.element_id,
             type=getattr(element, "type", ""),
@@ -1174,6 +1119,7 @@ class NodeManager:
             is_user_defined=getattr(element, "user_defined", False),
             settable=getattr(element, "settable", None),
             ui_options=getattr(element, "ui_options", None),
+            result_details=details,
         )
         return result
 
@@ -1184,7 +1130,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to get element details for element '{request.specific_element_id}` from a Node, but no Current Context was found."
-                logger.error(details)
 
                 return GetNodeElementDetailsResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -1196,7 +1141,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to get element details for Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 return GetNodeElementDetailsResultFailure(result_details=details)
 
@@ -1208,7 +1152,6 @@ class NodeManager:
             element = node.root_ui_element.find_element_by_id(request.specific_element_id)
             if element is None:
                 details = f"Attempted to get element details for element '{request.specific_element_id}' from Node '{node_name}'. Failed because it didn't have an element with that ID on it."
-                logger.error(details)
 
                 return GetNodeElementDetailsResultFailure(result_details=details)
 
@@ -1219,8 +1162,7 @@ class NodeManager:
         if param_to_value:
             element_details["element_id_to_value"] = param_to_value
         details = f"Successfully got element details for Node '{node_name}'."
-        logger.debug(details)
-        result = GetNodeElementDetailsResultSuccess(element_details=element_details)
+        result = GetNodeElementDetailsResultSuccess(element_details=element_details, result_details=details)
         return result
 
     def _set_param_to_value(self, node: BaseNode, element: BaseNodeElement, param_to_value: dict) -> None:
@@ -1326,7 +1268,6 @@ class NodeManager:
                     )
                     if isinstance(delete_result, ResultPayloadFailure):
                         details = f"Failed to delete incompatible incoming connection from {conn.source_node_name}.{conn.source_parameter_name} to {node_name}.{request.parameter_name}: {delete_result}"
-                        logger.error(details)
                         return AlterParameterDetailsResultFailure(result_details=details)
 
         # Check and break invalid outgoing connections
@@ -1345,7 +1286,6 @@ class NodeManager:
                     )
                     if isinstance(delete_result, ResultPayloadFailure):
                         details = f"Failed to delete incompatible outgoing connection from {node_name}.{request.parameter_name} to {conn.target_node_name}.{conn.target_parameter_name}: {delete_result}"
-                        logger.error(details)
                         return AlterParameterDetailsResultFailure(result_details=details)
 
         return None
@@ -1357,7 +1297,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to alter details for Parameter '{request.parameter_name}' from node in the Current Context. Failed because there was no such Node."
-                logger.error(details)
 
                 return AlterParameterDetailsResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -1369,14 +1308,12 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to alter details for Parameter '{request.parameter_name}' from Node '{node_name}', but no such Node was found."
-                logger.error(details)
 
                 return AlterParameterDetailsResultFailure(result_details=details)
 
         # Is the node locked?
         if node.lock:
             details = f"Attempted to alter details for Parameter '{request.parameter_name}' from Node '{node_name}'. Failed because the Node was locked."
-            logger.error(details)
             return AlterParameterDetailsResultFailure(result_details=details)
 
         # Handle ErrorProxyNode parameter alteration requests
@@ -1387,21 +1324,18 @@ class NodeManager:
 
                 # Early return with warning - we're just preserving the original changes
                 details = f"Parameter '{request.parameter_name}' alteration recorded for ErrorProxyNode '{node_name}'. Original node '{node.original_node_type}' had loading errors - preserving changes for correct recreation when dependency '{node.original_library_name}' is resolved."
-                logger.warning(details)
 
-                result_details = ResultDetails(message=details, level="WARNING")
+                result_details = ResultDetails(message=details, level=logging.WARNING)
                 return AlterParameterDetailsResultSuccess(result_details=result_details)
 
             # Reject runtime parameter alterations on ErrorProxy
             details = f"Cannot modify parameter '{request.parameter_name}' on placeholder node '{node_name}'. This placeholder preserves your workflow structure but doesn't allow parameter modifications, as they could cause issues when the original node is restored."
-            logger.error(details)
             return AlterParameterDetailsResultFailure(result_details=details)
 
         # Does the Element actually exist on the Node?
         element = node.get_element_by_name_and_type(request.parameter_name)
         if element is None:
             details = f"Attempted to alter details for Element '{request.parameter_name}' from Node '{node_name}'. Failed because it didn't have an Element with that name on it."
-            logger.error(details)
             return AlterParameterDetailsResultFailure(result_details=details)
         if request.ui_options is not None:
             element.ui_options = request.ui_options  # type: ignore[attr-defined]
@@ -1423,8 +1357,9 @@ class NodeManager:
             if hasattr(element, "user_defined") and element.user_defined is False and request.request_id:  # type: ignore[attr-defined]
                 # TODO: https://github.com/griptape-ai/griptape-nodes/issues/826
                 details = f"Attempted to alter details for Element '{request.parameter_name}' from Node '{node_name}'. Could only alter some values because the Element was not user-defined (i.e., critical to the Node implementation). Only user-defined Elements can be totally modified from a Node."
-                logger.warning(details)
-                return AlterParameterDetailsResultSuccess()
+                return AlterParameterDetailsResultSuccess(
+                    result_details=ResultDetails(message=details, level=logging.WARNING)
+                )
             self.modify_key_parameter_fields(request, element)
 
         # This field requires the node as well
@@ -1433,9 +1368,7 @@ class NodeManager:
             node.parameter_values[request.parameter_name] = request.default_value
 
         details = f"Successfully altered details for Element '{request.parameter_name}' from Node '{node_name}'."
-        logger.debug(details)
-
-        result = AlterParameterDetailsResultSuccess()
+        result = AlterParameterDetailsResultSuccess(result_details=details)
         return result
 
     # For C901 (too complex): Need to give customers explicit reasons for failure on each case.
@@ -1446,7 +1379,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to get value for Parameter '{request.parameter_name}' from node in the Current Context. Failed because there was no such Node."
-                logger.error(details)
 
                 return GetParameterValueResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -1461,14 +1393,12 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f'"{node_name}" not found'
-                logger.error(details)
                 return GetParameterValueResultFailure(result_details=details)
 
         # Does the Parameter actually exist on the Node?
         parameter = node.get_parameter_by_name(param_name)
         if parameter is None:
             details = f'"{node_name}.{param_name}" not found'
-            logger.error(details)
             return GetParameterValueResultFailure(result_details=details)
 
         # Values are actually stored on the NODE, so let's ask them.
@@ -1484,13 +1414,12 @@ class NodeManager:
 
         # Cool.
         details = f"{node_name}.{request.parameter_name} = {data_value}"
-        logger.debug(details)
-
         result = GetParameterValueResultSuccess(
             input_types=parameter.input_types,
             type=parameter.type,
             output_type=parameter.output_type,
             value=TypeValidator.safe_serialize(data_value),
+            result_details=details,
         )
         return result
 
@@ -1508,7 +1437,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = f"Attempted to set parameter '{request.parameter_name}' value. Failed because no Node was found in the Current Context."
-                logger.error(details)
                 return SetParameterValueResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
             node_name = node.name
@@ -1522,13 +1450,11 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to set parameter '{param_name}' value on node '{node_name}'. Failed because no such Node could be found."
-                logger.error(details)
                 return SetParameterValueResultFailure(result_details=details)
 
         # Is the node locked?
         if node.lock:
             details = f"Attempted to set parameter '{param_name}' value on node '{node_name}'. Failed because the Node was locked."
-            logger.error(details)
             return SetParameterValueResultFailure(result_details=details)
 
         # Handle ErrorProxyNode parameter value requests
@@ -1544,14 +1470,12 @@ class NodeManager:
             else:
                 # Reject runtime parameter value changes on ErrorProxy
                 details = f"Cannot set parameter '{param_name}' on placeholder node '{node_name}'. This placeholder preserves your workflow structure but doesn't allow parameter changes, as they could cause issues when the original node is restored."
-                logger.error(details)
                 return SetParameterValueResultFailure(result_details=details)
 
         # Does the Parameter actually exist on the Node?
         parameter = node.get_parameter_by_name(param_name)
         if parameter is None:
             details = f"Attempted to set parameter value for '{node_name}.{param_name}'. Failed because no parameter with that name could be found."
-            logger.error(details)
 
             result = SetParameterValueResultFailure(result_details=details)
             return result
@@ -1561,7 +1485,6 @@ class NodeManager:
         incoming_param_set = request.incoming_connection_source_parameter_name is not None
         if incoming_node_set != incoming_param_set:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because incoming connection source fields must both be None or both be set. Got incoming_connection_source_node_name={request.incoming_connection_source_node_name}, incoming_connection_source_parameter_name={request.incoming_connection_source_parameter_name}."
-            logger.error(details)
             result = SetParameterValueResultFailure(result_details=details)
             return result
 
@@ -1583,7 +1506,6 @@ class NodeManager:
                 if param_connections:  # Has incoming connections
                     # TODO: https://github.com/griptape-ai/griptape-nodes/issues/1965 Consider emitting UI events when parameters become settable/unsettable due to connection changes
                     details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because this parameter has incoming connections and cannot be set as a property while connected."
-                    logger.error(details)
                     result = SetParameterValueResultFailure(result_details=details)
                     return result
 
@@ -1594,15 +1516,14 @@ class NodeManager:
                 request.value = modified_value
         except Exception as err:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because before_value_set hook raised exception: {err}"
-            logger.error(details)
             result = SetParameterValueResultFailure(result_details=details)
             return result
 
         # Validate that parameters can be set at all (note: we want the value to be set during initial setup, but not after)
-        # This check comes after before_value_set to allow nodes to temporarily modify settable state
-        if not parameter.settable and not request.initial_setup:
+        # We skip this if it's a passthru from a connection or if we're on initial setup; those always trump settable.
+        # This check comes *AFTER* before_value_set() to allow nodes to temporarily modify settable state
+        if not parameter.settable and not incoming_node_set and not request.initial_setup:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because that Parameter was flagged as not settable."
-            logger.error(details)
             result = SetParameterValueResultFailure(result_details=details)
             return result
 
@@ -1611,7 +1532,6 @@ class NodeManager:
         # Is this value kosher for the types allowed?
         if not parameter.is_incoming_type_allowed(object_type):
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because the value's type of '{object_type}' was not in the Parameter's list of allowed types: {parameter.input_types}."
-            logger.error(details)
 
             result = SetParameterValueResultFailure(result_details=details)
             return result
@@ -1620,27 +1540,23 @@ class NodeManager:
             parent_flow_name = self.get_node_parent_flow_by_name(node.name)
         except KeyError:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because the node's parent flow does not exist. Could not unresolve future nodes."
-            logger.error(details)
             return SetParameterValueResultFailure(result_details=details)
 
         obj_mgr = GriptapeNodes.ObjectManager()
         parent_flow = obj_mgr.attempt_get_object_by_name_as_type(parent_flow_name, ControlFlow)
         if not parent_flow:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because the node's parent flow does not exist. Could not unresolve future nodes."
-            logger.error(details)
             return SetParameterValueResultFailure(result_details=details)
         try:
             finalized_value, modified = self._set_and_pass_through_values(request, node)
         except Exception as err:
             details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because Exception: {err}"
-            logger.error(details)
             return SetParameterValueResultFailure(result_details=details)
         if not request.initial_setup and modified:
             try:
                 GriptapeNodes.FlowManager().get_connections().unresolve_future_nodes(node)
             except Exception as err:
                 details = f"Attempted to set parameter value for '{node_name}.{request.parameter_name}'. Failed because Exception: {err}"
-                logger.error(details)
                 return SetParameterValueResultFailure(result_details=details)
         if request.initial_setup is False and not request.is_output and modified:
             # Mark node as unresolved, broadcast an event
@@ -1666,9 +1582,9 @@ class NodeManager:
 
         # Cool.
         details = f"Successfully set value on Node '{node_name}' Parameter '{request.parameter_name}'."
-        logger.debug(details)
-
-        result = SetParameterValueResultSuccess(finalized_value=finalized_value, data_type=parameter.type)
+        result = SetParameterValueResultSuccess(
+            finalized_value=finalized_value, data_type=parameter.type, result_details=details
+        )
         return result
 
     def _set_and_pass_through_values(self, request: SetParameterValueRequest, node: BaseNode) -> ModifiedReturnValue:
@@ -1703,7 +1619,7 @@ class NodeManager:
     # want to give clear reasoning for each failure.
     # For PLR0915 (too many statements): very little reusable code here, want to be explicit and
     # make debugger use friendly.
-    def on_get_all_node_info_request(self, request: GetAllNodeInfoRequest) -> ResultPayload:  # noqa: C901, PLR0911, PLR0915
+    def on_get_all_node_info_request(self, request: GetAllNodeInfoRequest) -> ResultPayload:  # noqa: C901, PLR0911
         node_name = request.node_name
         node = None
 
@@ -1711,7 +1627,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to get all info for a Node from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return GetAllNodeInfoResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -1723,14 +1638,12 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to get all info for Node named '{node_name}', but no such Node was found."
-                logger.error(details)
                 return GetAllNodeInfoResultFailure(result_details=details)
 
         get_metadata_request = GetNodeMetadataRequest(node_name=node_name)
         get_metadata_result = self.on_get_node_metadata_request(get_metadata_request)
         if not get_metadata_result.succeeded():
             details = f"Attempted to get all info for Node named '{node_name}', but failed getting the metadata."
-            logger.error(details)
             return GetAllNodeInfoResultFailure(result_details=details)
 
         get_resolution_state_request = GetNodeResolutionStateRequest(node_name=node_name)
@@ -1739,7 +1652,6 @@ class NodeManager:
             details = (
                 f"Attempted to get all info for Node named '{node_name}', but failed getting the resolution state."
             )
-            logger.error(details)
             return GetAllNodeInfoResultFailure(result_details=details)
 
         list_connections_request = ListConnectionsForNodeRequest(node_name=node_name)
@@ -1748,7 +1660,6 @@ class NodeManager:
             details = (
                 f"Attempted to get all info for Node named '{node_name}', but failed listing all connections for it."
             )
-            logger.error(details)
 
             return GetAllNodeInfoResultFailure(result_details=details)
         # Cast everything to get the linter off our back.
@@ -1758,7 +1669,6 @@ class NodeManager:
             list_connections_success = cast("ListConnectionsForNodeResultSuccess", list_connections_result)
         except Exception as err:
             details = f"Attempted to get all info for Node named '{node_name}'. Failed due to error: {err}."
-            logger.error(details)
 
             return GetAllNodeInfoResultFailure(result_details=details)
         get_node_elements_request = GetNodeElementDetailsRequest(node_name=node_name)
@@ -1767,13 +1677,11 @@ class NodeManager:
             details = (
                 f"Attempted to get all info for Node named '{node_name}', but failed getting details for elements."
             )
-            logger.error(details)
             return GetAllNodeInfoResultFailure(result_details=details)
         try:
             get_element_details_success = cast("GetNodeElementDetailsResultSuccess", get_node_elements_result)
         except Exception as err:
             details = f"Attempted to get all info for Node named '{node_name}'. Failed due to error: {err}."
-            logger.error(details)
             return GetAllNodeInfoResultFailure(result_details=details)
 
         # this will return the node element and the value
@@ -1784,7 +1692,6 @@ class NodeManager:
         else:
             element_id_to_value = {}
         details = f"Successfully got all node info for node '{node_name}'."
-        logger.debug(details)
         result = GetAllNodeInfoResultSuccess(
             metadata=get_metadata_success.metadata,
             node_resolution_state=get_resolution_state_success.state,
@@ -1792,6 +1699,7 @@ class NodeManager:
             connections=list_connections_success,
             element_id_to_value=element_id_to_value,
             root_node_element=element_details,
+            result_details=details,
         )
         return result
 
@@ -1802,7 +1710,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to get compatible parameters for node, but no current node was found."
-                logger.error(details)
                 return GetCompatibleParametersResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
             node_name = node.name
@@ -1815,14 +1722,12 @@ class NodeManager:
                 details = (
                     f"Attempted to get compatible parameters for node '{node_name}', but that node does not exist."
                 )
-                logger.error(details)
                 return GetCompatibleParametersResultFailure(result_details=details)
 
         # Vet the parameter.
         request_param = node.get_parameter_by_name(request.parameter_name)
         if request_param is None:
             details = f"Attempted to get compatible parameters for '{node_name}.{request.parameter_name}', but that no Parameter with that name could not be found."
-            logger.error(details)
             return GetCompatibleParametersResultFailure(result_details=details)
 
         # Figure out the mode we're going for, and if this parameter supports the mode.
@@ -1830,7 +1735,6 @@ class NodeManager:
         # Does this parameter support that?
         if request_mode not in request_param.allowed_modes:
             details = f"Attempted to get compatible parameters for '{node_name}.{request.parameter_name}' as '{request_mode}', but the Parameter didn't support that type of input/output."
-            logger.error(details)
             return GetCompatibleParametersResultFailure(result_details=details)
 
         # Get the parent flows.
@@ -1838,7 +1742,6 @@ class NodeManager:
             flow_name = self.get_node_parent_flow_by_name(node_name)
         except KeyError as err:
             details = f"Attempted to get compatible parameters for '{node_name}.{request.parameter_name}', but the node's parent flow could not be found: {err}"
-            logger.error(details)
             return GetCompatibleParametersResultFailure(result_details=details)
 
         # Iterate through all nodes in this Flow (yes, this restriction still sucks)
@@ -1848,14 +1751,12 @@ class NodeManager:
         )
         if not list_nodes_in_flow_result.succeeded():
             details = f"Attempted to get compatible parameters for '{node_name}.{request.parameter_name}'. Failed due to inability to list nodes in parent flow '{flow_name}'."
-            logger.error(details)
             return GetCompatibleParametersResultFailure(result_details=details)
 
         try:
             list_nodes_in_flow_success = cast("ListNodesInFlowResultSuccess", list_nodes_in_flow_result)
         except Exception as err:
             details = f"Attempted to get compatible parameters for '{node_name}.{request.parameter_name}'. Failed due to {err}"
-            logger.error(details)
             return GetCompatibleParametersResultFailure(result_details=details)
 
         # Walk through all nodes that are NOT us to find compatible Parameters.
@@ -1867,7 +1768,6 @@ class NodeManager:
                     test_node = self.get_node_by_name(test_node_name)
                 except ValueError as err:
                     details = f"Attempted to get compatible parameters for node '{node_name}', and sought to test against {test_node_name}, but that node does not exist. Error: {err}."
-                    logger.error(details)
                     return GetCompatibleParametersResultFailure(result_details=details)
 
                 # Get Parameters from Node
@@ -1904,8 +1804,9 @@ class NodeManager:
                                 valid_parameters_by_node[test_node_name] = compatible_list
 
         details = f"Successfully got compatible parameters for '{node_name}.{request.parameter_name}'."
-        logger.debug(details)
-        return GetCompatibleParametersResultSuccess(valid_parameters_by_node=valid_parameters_by_node)
+        return GetCompatibleParametersResultSuccess(
+            valid_parameters_by_node=valid_parameters_by_node, result_details=details
+        )
 
     def get_node_by_name(self, name: str) -> BaseNode:
         obj_mgr = GriptapeNodes.ObjectManager()
@@ -1923,20 +1824,18 @@ class NodeManager:
             raise KeyError(msg)
         return self._name_to_parent_flow_name[node_name]
 
-    async def on_resolve_from_node_request(self, request: ResolveNodeRequest) -> ResultPayload:  # noqa: C901, PLR0911, PLR0915, PLR0912
+    async def on_resolve_from_node_request(self, request: ResolveNodeRequest) -> ResultPayload:  # noqa: C901, PLR0911, PLR0912
         node_name = request.node_name
         debug_mode = request.debug_mode
 
         if node_name is None:
             details = "No Node name was provided. Failed to resolve node."
-            logger.error(details)
 
             return ResolveNodeResultFailure(validation_exceptions=[], result_details=details)
         try:
             node = self.get_node_by_name(node_name)
         except ValueError as e:
             details = f'Resolve failure. "{node_name}" does not exist. {e}'
-            logger.error(details)
 
             return ResolveNodeResultFailure(validation_exceptions=[e], result_details=details)
         # try to get the flow parent of this node
@@ -1944,7 +1843,6 @@ class NodeManager:
             flow_name = self._name_to_parent_flow_name[node_name]
         except KeyError as e:
             details = f'Failed to fetch parent flow for "{node_name}": {e}'
-            logger.error(details)
 
             return ResolveNodeResultFailure(validation_exceptions=[e], result_details=details)
         try:
@@ -1952,30 +1850,25 @@ class NodeManager:
             flow = obj_mgr.attempt_get_object_by_name_as_type(flow_name, ControlFlow)
         except KeyError as e:
             details = f'Failed to fetch parent flow for "{node_name}": {e}'
-            logger.error(details)
 
             return ResolveNodeResultFailure(validation_exceptions=[e], result_details=details)
 
         if flow is None:
             details = f'Failed to fetch parent flow for "{node_name}"'
-            logger.error(details)
             return ResolveNodeResultFailure(validation_exceptions=[], result_details=details)
         if GriptapeNodes.FlowManager().check_for_existing_running_flow():
             details = f"Failed to resolve from node '{node_name}'. Flow is already running."
-            logger.error(details)
             return ResolveNodeResultFailure(validation_exceptions=[], result_details=details)
         try:
             GriptapeNodes.FlowManager().get_connections().unresolve_future_nodes(node)
         except Exception as e:
             details = f'Failed to mark future nodes dirty. Unable to kick off flow from "{node_name}": {e}'
-            logger.error(details)
             return ResolveNodeResultFailure(validation_exceptions=[e], result_details=details)
         # Validate here.
         result = self.on_validate_node_dependencies_request(ValidateNodeDependenciesRequest(node_name=node_name))
         try:
             if not result.succeeded():
                 details = f"Failed to resolve node '{node_name}'. Flow Validation Failed"
-                logger.error(details)
                 return StartFlowResultFailure(validation_exceptions=[], result_details=details)
             result = cast("ValidateNodeDependenciesResultSuccess", result)
 
@@ -1984,21 +1877,17 @@ class NodeManager:
                 if len(result.exceptions) > 0:
                     for exception in result.exceptions:
                         details = f"{details}\n\t{exception}"
-                logger.error(details)
                 return StartFlowResultFailure(validation_exceptions=result.exceptions, result_details=details)
         except Exception as e:
             details = f"Failed to resolve node '{node_name}'. Flow Validation Failed. Error: {e}"
-            logger.error(details)
             return StartFlowResultFailure(validation_exceptions=[e], result_details=details)
         try:
-            await GriptapeNodes.FlowManager().resolve_singular_node(flow, node, debug_mode)
+            await GriptapeNodes.FlowManager().resolve_singular_node(flow, node, debug_mode=debug_mode)
         except Exception as e:
             details = f'Failed to resolve "{node_name}".  Error: {e}'
-            logger.error(details)
             return ResolveNodeResultFailure(validation_exceptions=[e], result_details=details)
         details = f'Starting to resolve "{node_name}" in "{flow_name}"'
-        logger.debug(details)
-        return ResolveNodeResultSuccess()
+        return ResolveNodeResultSuccess(result_details=details)
 
     def on_validate_node_dependencies_request(self, request: ValidateNodeDependenciesRequest) -> ResultPayload:
         node_name = request.node_name
@@ -2006,18 +1895,15 @@ class NodeManager:
         node = obj_manager.attempt_get_object_by_name_as_type(node_name, BaseNode)
         if node is None:
             details = f'Failed to validate node dependencies. Node with "{node_name}" does not exist.'
-            logger.error(details)
             return ValidateNodeDependenciesResultFailure(result_details=details)
         try:
             flow_name = self.get_node_parent_flow_by_name(node_name)
         except Exception as e:
             details = f'Failed to validate node dependencies. Node with "{node_name}" has no parent flow. Error: {e}'
-            logger.error(details)
             return ValidateNodeDependenciesResultFailure(result_details=details)
         flow = GriptapeNodes.ObjectManager().attempt_get_object_by_name_as_type(flow_name, ControlFlow)
         if not flow:
             details = f'Failed to validate node dependencies. Flow with "{flow_name}" does not exist.'
-            logger.error(details)
             return ValidateNodeDependenciesResultFailure(result_details=details)
         # Gets all dependent nodes
         nodes = flow.get_node_dependencies(node)
@@ -2027,7 +1913,9 @@ class NodeManager:
             if exceptions:
                 all_exceptions = all_exceptions + exceptions
         return ValidateNodeDependenciesResultSuccess(
-            validation_succeeded=(len(all_exceptions) == 0), exceptions=all_exceptions
+            validation_succeeded=(len(all_exceptions) == 0),
+            exceptions=all_exceptions,
+            result_details=f"Successfully validated dependencies for node '{node_name}'. Found {len(all_exceptions)} validation issues.",
         )
 
     def on_serialize_node_to_commands(self, request: SerializeNodeToCommandsRequest) -> ResultPayload:  # noqa: C901, PLR0912, PLR0915
@@ -2037,7 +1925,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to serialize a Node to commands from the Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return SerializeNodeToCommandsResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
             node_name = node.name
@@ -2048,7 +1935,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to serialize Node '{node_name}' to commands. Failed because no Node with that name could be found."
-                logger.error(details)
                 return SerializeNodeToCommandsResultFailure(result_details=details)
 
         # This is our current dude.
@@ -2073,7 +1959,6 @@ class NodeManager:
                 else:
                     # For regular nodes, this is still an error
                     details = f"Attempted to serialize Node '{node_name}' to commands. Failed to get metadata for library '{library_used}'."
-                    logger.error(details)
                     return SerializeNodeToCommandsResultFailure(result_details=details)
             else:
                 library_version = library_metadata_result.metadata.library_version
@@ -2161,11 +2046,10 @@ class NodeManager:
                     node=node,
                     unique_parameter_uuid_to_values=request.unique_parameter_uuid_to_values,
                     serialized_parameter_value_tracker=request.serialized_parameter_value_tracker,
+                    create_node_request=create_node_request,
                 )
                 if set_param_value_requests is not None:
                     set_value_commands.extend(set_param_value_requests)
-                else:
-                    create_node_request.resolution = NodeResolutionState.UNRESOLVED.value
         # now check if locked
         if node.lock:
             lock_command = SetLockNodeStateRequest(node_name=None, lock=True)
@@ -2179,10 +2063,10 @@ class NodeManager:
             lock_node_command=lock_command,
         )
         details = f"Successfully serialized node '{node_name}' into commands."
-        logger.debug(details)
         result = SerializeNodeToCommandsResultSuccess(
             serialized_node_commands=serialized_node_commands,  # How to serialize this node
             set_parameter_value_commands=set_value_commands,  # The commands to serialize it with
+            result_details=details,
         )
         return result
 
@@ -2300,7 +2184,6 @@ class NodeManager:
         create_node_result = GriptapeNodes().handle_request(create_node_request)
         if not isinstance(create_node_result, CreateNodeResultSuccess):
             details = f"Attempted to deserialize a serialized set of Node Creation commands. Failed to create node '{create_node_request.node_name}'."
-            logger.error(details)
             return DeserializeNodeFromCommandsResultFailure(result_details=details)
 
         # Adopt the newly-created node as our current context.
@@ -2308,7 +2191,6 @@ class NodeManager:
         node = GriptapeNodes.ObjectManager().attempt_get_object_by_name_as_type(node_name, BaseNode)
         if node is None:
             details = f"Attempted to deserialize a serialized set of Node Creation commands. Failed to get node '{node_name}'."
-            logger.error(details)
             return DeserializeNodeFromCommandsResultFailure(result_details=details)
         with GriptapeNodes.ContextManager().node(node=node):
             for element_command in request.serialized_node_commands.element_modification_commands:
@@ -2319,11 +2201,9 @@ class NodeManager:
                 element_result = GriptapeNodes().handle_request(element_command)
                 if element_result.failed():
                     details = f"Attempted to deserialize a serialized set of Node Creation commands. Failed to execute an element command for node '{node_name}'."
-                    logger.error(details)
                     return DeserializeNodeFromCommandsResultFailure(result_details=details)
         details = f"Successfully deserialized a serialized set of Node Creation commands for node '{node_name}'."
-        logger.debug(details)
-        return DeserializeNodeFromCommandsResultSuccess(node_name=node_name)
+        return DeserializeNodeFromCommandsResultSuccess(node_name=node_name, result_details=details)
 
     def on_serialize_selected_nodes_to_commands(
         self, request: SerializeSelectedNodesToCommandsRequest
@@ -2355,7 +2235,6 @@ class NodeManager:
             )
             if not isinstance(result, SerializeNodeToCommandsResultSuccess):
                 details = f"Attempted to serialize a selection of Nodes. Failed to serialize {node_name}."
-                logger.error(details)
                 return SerializeNodeToCommandsResultFailure(result_details=details)
             node_commands[node_name] = result.serialized_node_commands
             node_name_to_uuid[node_name] = result.serialized_node_commands.node_uuid
@@ -2401,7 +2280,10 @@ class NodeManager:
         # Set everything in the clipboard!
         GriptapeNodes.ContextManager()._clipboard.node_commands = final_result
         GriptapeNodes.ContextManager()._clipboard.parameter_uuid_to_values = unique_uuid_to_values
-        return SerializeSelectedNodesToCommandsResultSuccess(final_result)
+        return SerializeSelectedNodesToCommandsResultSuccess(
+            final_result,
+            result_details=f"Successfully serialized {len(request.nodes_to_serialize)} selected nodes to commands.",
+        )
 
     def on_deserialize_selected_nodes_from_commands(  # noqa: C901, PLR0912
         self,
@@ -2431,13 +2313,11 @@ class NodeManager:
             )
             if not isinstance(result, DeserializeNodeFromCommandsResultSuccess):
                 details = "Attempted to deserialize node but ran into an error on node serialization."
-                logger.error(details)
                 return DeserializeSelectedNodesFromCommandsResultFailure(result_details=details)
             node_uuid_to_name[node_command.node_uuid] = result.node_name
             node = GriptapeNodes.ObjectManager().attempt_get_object_by_name_as_type(result.node_name, BaseNode)
             if node is None:
                 details = "Attempted to deserialize node but ran into an error on node serialization."
-                logger.error(details)
                 return DeserializeSelectedNodesFromCommandsResultFailure(result_details=details)
             with GriptapeNodes.ContextManager().node(node=node):
                 parameter_commands = commands.set_parameter_value_commands[node_command.node_uuid]
@@ -2480,7 +2360,10 @@ class NodeManager:
             if not result.succeeded():
                 details = f"Failed to create a connection between {connection_request.source_node_name} and {connection_request.target_node_name}"
                 logger.warning(details)
-        return DeserializeSelectedNodesFromCommandsResultSuccess(node_names=list(node_uuid_to_name.values()))
+        return DeserializeSelectedNodesFromCommandsResultSuccess(
+            node_names=list(node_uuid_to_name.values()),
+            result_details=f"Successfully deserialized {len(node_uuid_to_name)} nodes from commands.",
+        )
 
     def on_duplicate_selected_nodes(self, request: DuplicateSelectedNodesRequest) -> ResultPayload:
         result = GriptapeNodes.handle_request(
@@ -2488,12 +2371,10 @@ class NodeManager:
         )
         if not result.succeeded():
             details = "Failed to serialized selected nodes."
-            logger.error(details)
             return DuplicateSelectedNodesResultFailure(result_details=details)
         result = GriptapeNodes.handle_request(DeserializeSelectedNodesFromCommandsRequest(positions=request.positions))
         if not isinstance(result, DeserializeSelectedNodesFromCommandsResultSuccess):
             details = "Failed to deserialize selected nodes."
-            logger.error(details)
             return DuplicateSelectedNodesResultFailure(result_details=details)
 
         # Remake duplicate connections of node
@@ -2502,7 +2383,9 @@ class NodeManager:
         initial_nodes = [sublist[0] for sublist in request.nodes_to_duplicate]
 
         NodeManager.remake_connections(self, new_node_names=result.node_names, old_node_names=initial_nodes)
-        return DuplicateSelectedNodesResultSuccess(result.node_names)
+        return DuplicateSelectedNodesResultSuccess(
+            result.node_names, result_details=f"Successfully duplicated {len(initial_nodes)} nodes."
+        )
 
     @staticmethod
     def _manage_alter_details(parameter: Parameter, base_node_obj: BaseNode) -> dict:
@@ -2584,6 +2467,7 @@ class NodeManager:
         node: BaseNode,
         unique_parameter_uuid_to_values: dict[SerializedNodeCommands.UniqueParameterValueUUID, Any],
         serialized_parameter_value_tracker: SerializedParameterValueTracker,
+        create_node_request: CreateNodeRequest,
     ) -> list[SerializedNodeCommands.IndirectSetParameterValueCommand] | None:
         """Generates code to save a parameter value for a node in a Griptape workflow.
 
@@ -2600,6 +2484,7 @@ class NodeManager:
             node (BaseNode): The node object that contains the parameter
             unique_parameter_uuid_to_values (dict[SerializedNodeCommands.UniqueParameterValueUUID, Any]): Dictionary mapping unique value UUIDs to values
             serialized_parameter_value_tracker (SerializedParameterValueTracker): Object mapping maintaining value hashes to unique value UUIDs, and non-serializable values
+            create_node_request (CreateNodeRequest): The node creation request that will be modified if serialization fails
 
         Returns:
             None (if no value to be serialized) or an IndirectSetParameterValueCommand linking the value to the unique value map
@@ -2612,7 +2497,7 @@ class NodeManager:
         """
         output_value = None
         internal_value = None
-        if parameter.name in node.parameter_output_values and node.state == NodeResolutionState.RESOLVED:
+        if parameter.name in node.parameter_output_values:
             # Output values are more important.
             output_value = node.parameter_output_values[parameter.name]
         if parameter.name in node.parameter_values:
@@ -2634,6 +2519,8 @@ class NodeManager:
             if internal_command is None:
                 details = f"Attempted to serialize set value for parameter '{parameter.name}' on node '{node.name}'. The set value will not be restored in anything that attempts to deserialize or save this node. The value for this parameter was not serialized because it did not match Griptape Nodes' criteria for serializability. To remedy, either update the value's type to support serializability or mark the parameter as not serializable by setting serializable=False when creating the parameter."
                 logger.warning(details)
+                # Set node to unresolved when serialization fails
+                create_node_request.resolution = NodeResolutionState.UNRESOLVED.value
             else:
                 commands.append(internal_command)
         if output_value is not None:
@@ -2649,6 +2536,8 @@ class NodeManager:
             if output_command is None:
                 details = f"Attempted to serialize output value for parameter '{parameter.name}' on node '{node.name}'. The output value will not be restored in anything that attempts to deserialize or save this node. The value for this parameter was not serialized because it did not match Griptape Nodes' criteria for serializability. To remedy, either update the value's type to support serializability or mark the parameter as not serializable by setting serializable=False when creating the parameter."
                 logger.warning(details)
+                # Set node to unresolved when serialization fails
+                create_node_request.resolution = NodeResolutionState.UNRESOLVED.value
             else:
                 commands.append(output_command)
         return commands if commands else None
@@ -2667,7 +2556,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to rename Parameter in the Current Context. Failed because the Current Context was empty."
-                logger.error(details)
                 return RenameParameterResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
             node_name = node.name
@@ -2676,38 +2564,32 @@ class NodeManager:
                 node = self.get_node_by_name(node_name)
             except KeyError as err:
                 details = f"Attempted to rename Parameter '{request.parameter_name}' on Node '{node_name}'. Failed because the Node could not be found. Error: {err}"
-                logger.error(details)
                 return RenameParameterResultFailure(result_details=details)
 
         # Is the node locked?
         if node.lock:
             details = f"Attempted to rename Parameter '{request.parameter_name}' on Node '{node_name}'. Failed because the Node is locked."
-            logger.error(details)
             return RenameParameterResultFailure(result_details=details)
 
         # Get the parameter
         parameter = node.get_parameter_by_name(request.parameter_name)
         if parameter is None:
             details = f"Attempted to rename Parameter '{request.parameter_name}' on Node '{node_name}'. Failed because the Parameter could not be found."
-            logger.error(details)
             return RenameParameterResultFailure(result_details=details)
 
         # Only allow parameter rename for user-defined params
         if not parameter.user_defined:
             details = f"Attempted to rename Parameter '{request.parameter_name}' on Node '{node_name}'. Failed because the Parameter is not user-defined."
-            logger.error(details)
             return RenameParameterResultFailure(result_details=details)
 
         # Validate the new parameter name
         if any(char.isspace() for char in request.new_parameter_name):
             details = f"Failed to rename Parameter '{request.parameter_name}' to '{request.new_parameter_name}'. Parameter names cannot contain any whitespace characters."
-            logger.error(details)
             return RenameParameterResultFailure(result_details=details)
 
         # Check for duplicate names
         if node.does_name_exist(request.new_parameter_name):
             details = f"Failed to rename Parameter '{request.parameter_name}' to '{request.new_parameter_name}'. A Parameter with that name already exists."
-            logger.error(details)
             return RenameParameterResultFailure(result_details=details)
 
         # Get all connections for this node
@@ -2743,7 +2625,10 @@ class NodeManager:
             node.parameter_output_values[request.new_parameter_name] = node.parameter_output_values.pop(old_name)
 
         return RenameParameterResultSuccess(
-            old_parameter_name=old_name, new_parameter_name=request.new_parameter_name, node_name=node_name
+            old_parameter_name=old_name,
+            new_parameter_name=request.new_parameter_name,
+            node_name=node_name,
+            result_details=f"Successfully renamed parameter '{old_name}' to '{request.new_parameter_name}' on node '{node_name}'.",
         )
 
     def on_toggle_lock_node_request(self, request: SetLockNodeStateRequest) -> ResultPayload:
@@ -2751,7 +2636,6 @@ class NodeManager:
         if node_name is None:
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to lock node in the Current Context. Failed because the Current Context was empty."
-                logger.error(details)
                 return SetLockNodeStateResultFailure(result_details=details)
             node = GriptapeNodes.ContextManager().get_current_node()
             node_name = node.name
@@ -2760,10 +2644,13 @@ class NodeManager:
                 node = self.get_node_by_name(node_name)
             except ValueError as err:
                 details = f"Attempted to lock node '{request.node_name}'. Failed because the Node could not be found. Error: {err}"
-                logger.error(details)
                 return SetLockNodeStateResultFailure(result_details=details)
         node.lock = request.lock
-        return SetLockNodeStateResultSuccess(node_name=node_name, locked=node.lock)
+        return SetLockNodeStateResultSuccess(
+            node_name=node_name,
+            locked=node.lock,
+            result_details=f"Successfully set lock state to {node.lock} for node '{node_name}'.",
+        )
 
     def on_send_node_message_request(self, request: SendNodeMessageRequest) -> ResultPayload:
         """Handle a SendNodeMessageRequest by calling the node's message callback.
@@ -2781,7 +2668,6 @@ class NodeManager:
             # Get from the current context
             if not GriptapeNodes.ContextManager().has_current_node():
                 details = "Attempted to send message to Node from Current Context. Failed because the Current Context is empty."
-                logger.error(details)
                 return SendNodeMessageResultFailure(result_details=details)
 
             node = GriptapeNodes.ContextManager().get_current_node()
@@ -2793,7 +2679,6 @@ class NodeManager:
             node = obj_mgr.attempt_get_object_by_name_as_type(node_name, BaseNode)
             if node is None:
                 details = f"Attempted to send message to Node '{node_name}', but no such Node was found."
-                logger.error(details)
                 return SendNodeMessageResultFailure(result_details=details)
 
         # Validate optional_element_name if specified
@@ -2801,7 +2686,6 @@ class NodeManager:
             element = node.root_ui_element.find_element_by_name(request.optional_element_name)
             if element is None:
                 details = f"Attempted to send message to Node '{node_name}' with element '{request.optional_element_name}', but no such element was found."
-                logger.error(details)
                 return SendNodeMessageResultFailure(result_details=details, altered_workflow_state=False)
 
         # Call the node's message callback
@@ -2813,7 +2697,6 @@ class NodeManager:
 
         if not callback_result.success:
             details = f"Failed to handle message for Node '{node_name}': {callback_result.details}"
-            logger.warning(details)
             return SendNodeMessageResultFailure(
                 result_details=callback_result.details,
                 response=callback_result.response,
@@ -2821,7 +2704,6 @@ class NodeManager:
             )
 
         details = f"Successfully sent message to Node '{node_name}': {callback_result.details}"
-        logger.debug(details)
         return SendNodeMessageResultSuccess(
             result_details=callback_result.details,
             response=callback_result.response,