feat(platform): enhance Human-in-the-Loop block with improved review UX

- Backend: Fix message handling and unify API structure
  - Fix HITL block to properly yield review messages for both approved and rejected reviews
  - Unify review API structure with single `reviews` array using `approved` boolean field
  - Remove separate approved_reviews/rejected_review_ids in favor of cleaner unified approach

- Frontend: Complete UI/UX overhaul for review interface
  - Replace plain JSON textarea with type-aware input components matching run dialog styling
  - Add "Approve All" and "Reject All" buttons with smart disabled states
  - Show rejection reason input only when excluding items (simplified UX)
  - Fix Reviews tab auto-population when execution status changes to REVIEW
  - Add proper local state management for real-time input updates
  - Use design system Input components for consistent rounded styling

Key improvements:
- No more JSON syntax errors for string inputs
- Professional appearance matching platform standards
- Intuitive workflow with conditional UI elements
- Type-safe unified API structure
- Real-time input updates with proper state management

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: majdyz

autogpt_platform/backend/backend/blocks/human_in_the_loop.py

@@ -148,11 +148,13 @@ async def run(
                 node_exec_id=node_exec_id, processed=True
             )
 
-            # Yield the results
             if result.status == ReviewStatus.APPROVED:
-                yield "reviewed_data", result.data
                 yield "status", "approved"
-                yield "review_message", result.message
+                yield "reviewed_data", result.data
+                if result.message:
+                    yield "review_message", result.message
+
             elif result.status == ReviewStatus.REJECTED:
                 yield "status", "rejected"
-                yield "review_message", result.message
+                if result.message:
+                    yield "review_message", result.message

autogpt_platform/backend/backend/server/v2/executions/review/model.py

@@ -98,10 +98,15 @@ class ReviewItem(BaseModel):
     """Single review item for processing."""
 
     node_exec_id: str = Field(description="Node execution ID to review")
+    approved: bool = Field(
+        description="Whether this review is approved (True) or rejected (False)"
+    )
     message: str | None = Field(
         None, description="Optional review message", max_length=2000
     )
-    reviewed_data: SafeJsonData | None = Field(None, description="Optional edited data")
+    reviewed_data: SafeJsonData | None = Field(
+        None, description="Optional edited data (ignored if approved=False)"
+    )
 
     @field_validator("reviewed_data")
     @classmethod
@@ -168,31 +173,24 @@ class ReviewRequest(BaseModel):
 
     This request must include ALL pending reviews for a graph execution.
     Each review will be either approved (with optional data modifications)
-    or rejected. The execution will resume only after ALL reviews are processed.
+    or rejected (data ignored). The execution will resume only after ALL reviews are processed.
     """
 
-    approved_reviews: List[ReviewItem] = Field(
-        default=[], description="Reviews to approve with their data and messages"
-    )
-    rejected_review_ids: List[str] = Field(
-        default=[], description="Node execution IDs of reviews to reject"
+    reviews: List[ReviewItem] = Field(
+        description="All reviews with their approval status, data, and messages"
     )
 
     @model_validator(mode="after")
     def validate_review_completeness(self):
-        """Validate that we have at least one review to process and no overlaps."""
-        if not self.approved_reviews and not self.rejected_review_ids:
+        """Validate that we have at least one review to process and no duplicates."""
+        if not self.reviews:
             raise ValueError("At least one review must be provided")
 
-        # Ensure no duplicate node_exec_ids between approved and rejected
-        approved_ids = {review.node_exec_id for review in self.approved_reviews}
-        rejected_ids = set(self.rejected_review_ids)
-
-        overlap = approved_ids & rejected_ids
-        if overlap:
-            raise ValueError(
-                f"Review IDs cannot be both approved and rejected: {', '.join(overlap)}"
-            )
+        # Ensure no duplicate node_exec_ids
+        node_ids = [review.node_exec_id for review in self.reviews]
+        if len(node_ids) != len(set(node_ids)):
+            duplicates = [nid for nid in set(node_ids) if node_ids.count(nid) > 1]
+            raise ValueError(f"Duplicate review IDs found: {', '.join(duplicates)}")
 
         return self
 

autogpt_platform/backend/backend/server/v2/executions/review/routes.py

@@ -139,13 +139,7 @@ async def process_review_action(
     """Process reviews with approve or reject actions."""
 
     # Collect all node exec IDs from the request
-    all_request_node_ids = set()
-    if request.approved_reviews:
-        all_request_node_ids.update(
-            review.node_exec_id for review in request.approved_reviews
-        )
-    if request.rejected_review_ids:
-        all_request_node_ids.update(request.rejected_review_ids)
+    all_request_node_ids = {review.node_exec_id for review in request.reviews}
 
     if not all_request_node_ids:
         raise HTTPException(
@@ -156,18 +150,19 @@ async def process_review_action(
     try:
         # Build review decisions map
         review_decisions = {}
-        for review in request.approved_reviews:
-            review_decisions[review.node_exec_id] = (
-                ReviewStatus.APPROVED,
-                review.reviewed_data,
-                review.message,
-            )
-        for node_id in request.rejected_review_ids:
-            review_decisions[node_id] = (
-                ReviewStatus.REJECTED,
-                None,
-                "Rejected by user",
-            )
+        for review in request.reviews:
+            if review.approved:
+                review_decisions[review.node_exec_id] = (
+                    ReviewStatus.APPROVED,
+                    review.reviewed_data,
+                    review.message,
+                )
+            else:
+                review_decisions[review.node_exec_id] = (
+                    ReviewStatus.REJECTED,
+                    None,
+                    review.message,
+                )
 
         # Process all reviews
         updated_reviews = await process_all_reviews_for_execution(

autogpt_platform/frontend/src/app/(platform)/library/agents/[id]/components/AgentRunsView/components/SelectedRunView/SelectedRunView.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import React from "react";
+import React, { useEffect } from "react";
 import {
   TabsLine,
   TabsLineContent,
@@ -50,6 +50,12 @@ export function SelectedRunView({
     parseAsString.withDefault("output"),
   );
 
+  useEffect(() => {
+    if (run?.status === AgentExecutionStatus.REVIEW && runId) {
+      refetchReviews();
+    }
+  }, [run?.status, runId, refetchReviews]);
+
   if (responseError || httpError) {
     return (
       <ErrorCard
@@ -114,22 +120,25 @@ export function SelectedRunView({
           </RunDetailCard>
         </TabsLineContent>
 
-        {pendingReviews.length > 0 &&
-          run?.status === AgentExecutionStatus.REVIEW && (
-            <TabsLineContent value="reviews">
-              <RunDetailCard>
-                {reviewsLoading ? (
-                  <div className="text-neutral-500">Loading reviews…</div>
-                ) : (
-                  <PendingReviewsList
-                    reviews={pendingReviews}
-                    onReviewComplete={refetchReviews}
-                    emptyMessage="No pending reviews for this execution"
-                  />
-                )}
-              </RunDetailCard>
-            </TabsLineContent>
-          )}
+        {run?.status === AgentExecutionStatus.REVIEW && (
+          <TabsLineContent value="reviews">
+            <RunDetailCard>
+              {reviewsLoading ? (
+                <div className="text-neutral-500">Loading reviews…</div>
+              ) : pendingReviews.length > 0 ? (
+                <PendingReviewsList
+                  reviews={pendingReviews}
+                  onReviewComplete={refetchReviews}
+                  emptyMessage="No pending reviews for this execution"
+                />
+              ) : (
+                <div className="text-neutral-600">
+                  No pending reviews for this execution
+                </div>
+              )}
+            </RunDetailCard>
+          </TabsLineContent>
+        )}
       </TabsLine>
     </div>
   );

autogpt_platform/frontend/src/app/(platform)/library/agents/[id]/components/OldAgentLibraryView/components/agent-run-details-view.tsx

@@ -1,6 +1,6 @@
 "use client";
 import moment from "moment";
-import React, { useCallback, useMemo } from "react";
+import React, { useCallback, useMemo, useEffect } from "react";
 
 import {
   Graph,
@@ -77,6 +77,13 @@ export function AgentRunDetailsView({
 
   const toastOnFail = useToastOnFail();
 
+  // Refetch pending reviews when execution status changes to REVIEW
+  useEffect(() => {
+    if (runStatus === "review" && run.id) {
+      refetchReviews();
+    }
+  }, [runStatus, run.id, refetchReviews]);
+
   const infoStats: { label: string; value: React.ReactNode }[] = useMemo(() => {
     if (!run) return [];
     return [
@@ -382,7 +389,7 @@ export function AgentRunDetailsView({
         )}
 
         {/* Pending Reviews Section */}
-        {pendingReviews.length > 0 && (
+        {runStatus === "review" && (
           <Card className="agpt-box">
             <CardHeader>
               <CardTitle className="font-poppins text-lg">
@@ -392,12 +399,16 @@ export function AgentRunDetailsView({
             <CardContent>
               {reviewsLoading ? (
                 <LoadingBox spinnerSize={12} className="h-24" />
-              ) : (
+              ) : pendingReviews.length > 0 ? (
                 <PendingReviewsList
                   reviews={pendingReviews}
                   onReviewComplete={refetchReviews}
                   emptyMessage="No pending reviews for this execution"
                 />
+              ) : (
+                <div className="py-4 text-neutral-600">
+                  No pending reviews for this execution
+                </div>
               )}
             </CardContent>
           </Card>

autogpt_platform/frontend/src/app/api/openapi.json

@@ -2988,7 +2988,7 @@
       "delete": {
         "tags": ["v2", "store", "private"],
         "summary": "Delete store submission",
-        "description": "Delete a store listing submission.\n\nArgs:\n    user_id (str): ID of the authenticated user\n    submission_id (str): ID of the submission to be deleted\n\nReturns:\n    StoreSubmission: The deleted submission object",
+        "description": "Delete a store listing submission.\n\nArgs:\n    user_id (str): ID of the authenticated user\n    submission_id (str): ID of the submission to be deleted\n\nReturns:\n    bool: True if the submission was successfully deleted, False otherwise",
         "operationId": "deleteV2Delete store submission",
         "security": [{ "HTTPBearerJWT": [] }],
         "parameters": [
@@ -3001,15 +3001,16 @@
         ],
         "responses": {
           "200": {
-            "description": "Submission deleted successfully",
+            "description": "Successful Response",
             "content": {
               "application/json": {
-                "schema": { "$ref": "#/components/schemas/StoreSubmission" }
+                "schema": {
+                  "type": "boolean",
+                  "title": "Response Deletev2Delete Store Submission"
+                }
               }
             }
           },
-          "404": { "description": "Submission not found" },
-          "500": { "description": "Server error" },
           "422": {
             "description": "Validation Error",
             "content": {
@@ -8119,6 +8120,11 @@
             "title": "Node Exec Id",
             "description": "Node execution ID to review"
           },
+          "approved": {
+            "type": "boolean",
+            "title": "Approved",
+            "description": "Whether this review is approved (True) or rejected (False)"
+          },
           "message": {
             "anyOf": [
               { "type": "string", "maxLength": 2000 },
@@ -8138,34 +8144,27 @@
               { "type": "null" }
             ],
             "title": "Reviewed Data",
-            "description": "Optional edited data"
+            "description": "Optional edited data (ignored if approved=False)"
           }
         },
         "type": "object",
-        "required": ["node_exec_id"],
+        "required": ["node_exec_id", "approved"],
         "title": "ReviewItem",
         "description": "Single review item for processing."
       },
       "ReviewRequest": {
         "properties": {
-          "approved_reviews": {
+          "reviews": {
             "items": { "$ref": "#/components/schemas/ReviewItem" },
             "type": "array",
-            "title": "Approved Reviews",
-            "description": "Reviews to approve with their data and messages",
-            "default": []
-          },
-          "rejected_review_ids": {
-            "items": { "type": "string" },
-            "type": "array",
-            "title": "Rejected Review Ids",
-            "description": "Node execution IDs of reviews to reject",
-            "default": []
+            "title": "Reviews",
+            "description": "All reviews with their approval status, data, and messages"
           }
         },
         "type": "object",
+        "required": ["reviews"],
         "title": "ReviewRequest",
-        "description": "Request model for processing ALL pending reviews for an execution.\n\nThis request must include ALL pending reviews for a graph execution.\nEach review will be either approved (with optional data modifications)\nor rejected. The execution will resume only after ALL reviews are processed."
+        "description": "Request model for processing ALL pending reviews for an execution.\n\nThis request must include ALL pending reviews for a graph execution.\nEach review will be either approved (with optional data modifications)\nor rejected (data ignored). The execution will resume only after ALL reviews are processed."
       },
       "ReviewResponse": {
         "properties": {