cleanup

autogpt_platform/backend/backend/api/features/executions/review/routes.py

@@ -202,9 +202,6 @@ async def process_review_action(
                     user_id=user_id, graph_id=first_review.graph_id
                 )
 
-                # Create execution context with settings
-                # Note: auto-approval is now handled via database lookup in
-                # check_auto_approval(), no need to pass auto_approved_node_ids
                 execution_context = ExecutionContext(
                     human_in_the_loop_safe_mode=settings.human_in_the_loop_safe_mode,
                     sensitive_action_safe_mode=settings.sensitive_action_safe_mode,

autogpt_platform/backend/backend/blocks/helpers/review.py

@@ -9,7 +9,7 @@ from typing import Any, Optional
 from prisma.enums import ReviewStatus
 from pydantic import BaseModel
 
-from backend.data.execution import ExecutionContext, ExecutionStatus
+from backend.data.execution import ExecutionStatus
 from backend.data.human_review import ReviewResult, check_auto_approval
 from backend.executor.manager import async_update_node_execution_status
 from backend.util.clients import get_database_manager_async_client
@@ -60,7 +60,6 @@ class HITLReviewHelper:
         graph_exec_id: str,
         graph_id: str,
         graph_version: int,
-        execution_context: ExecutionContext,
         block_name: str = "Block",
         editable: bool = False,
     ) -> Optional[ReviewResult]:
@@ -75,7 +74,6 @@ class HITLReviewHelper:
             graph_exec_id: ID of the graph execution
             graph_id: ID of the graph
             graph_version: Version of the graph
-            execution_context: Current execution context
             block_name: Name of the block requesting review
             editable: Whether the reviewer can edit the data
 
@@ -92,11 +90,10 @@ class HITLReviewHelper:
         # This function only handles auto-approval for specific nodes.
 
         # Check if this node has been auto-approved in a previous review
-        auto_approval = await check_auto_approval(
+        if await check_auto_approval(
             graph_exec_id=graph_exec_id,
             node_id=node_id,
-        )
-        if auto_approval:
+        ):
             logger.info(
                 f"Block {block_name} skipping review for node {node_exec_id} - "
                 f"node {node_id} has auto-approval from previous review"
@@ -148,7 +145,6 @@ class HITLReviewHelper:
         graph_exec_id: str,
         graph_id: str,
         graph_version: int,
-        execution_context: ExecutionContext,
         block_name: str = "Block",
         editable: bool = False,
     ) -> Optional[ReviewDecision]:
@@ -163,7 +159,6 @@ class HITLReviewHelper:
             graph_exec_id: ID of the graph execution
             graph_id: ID of the graph
             graph_version: Version of the graph
-            execution_context: Current execution context
             block_name: Name of the block requesting review
             editable: Whether the reviewer can edit the data
 
@@ -179,7 +174,6 @@ class HITLReviewHelper:
             graph_exec_id=graph_exec_id,
             graph_id=graph_id,
             graph_version=graph_version,
-            execution_context=execution_context,
             block_name=block_name,
             editable=editable,
         )

autogpt_platform/backend/backend/blocks/human_in_the_loop.py

@@ -121,7 +121,6 @@ class HumanInTheLoopBlock(Block):
             graph_exec_id=graph_exec_id,
             graph_id=graph_id,
             graph_version=graph_version,
-            execution_context=execution_context,
             block_name=self.name,
             editable=input_data.editable,
         )

autogpt_platform/backend/backend/data/block.py

@@ -655,7 +655,6 @@ class Block(ABC, Generic[BlockSchemaInputType, BlockSchemaOutputType]):
             graph_exec_id=graph_exec_id,
             graph_id=graph_id,
             graph_version=graph_version,
-            execution_context=execution_context,
             block_name=self.name,
             editable=True,
         )