From 8a2c0766910856d501fb332feac7374e8b49f55d Mon Sep 17 00:00:00 2001
From: mlogclub <mlog1@qq.com>
Date: Mon, 22 Jun 2026 17:14:35 +0800
Subject: [PATCH] feat: add WorkflowValidationBadge component for improved
 validation feedback

---
 .../_components/workflow-editor.tsx           | 52 +++++++++++++++++--
 1 file changed, 47 insertions(+), 5 deletions(-)

diff --git a/web/app/dashboard/ai-workflows/_components/workflow-editor.tsx b/web/app/dashboard/ai-workflows/_components/workflow-editor.tsx
index e3805b8..b0026d5 100644
--- a/web/app/dashboard/ai-workflows/_components/workflow-editor.tsx
+++ b/web/app/dashboard/ai-workflows/_components/workflow-editor.tsx
@@ -19,6 +19,11 @@ import { useCallback, useEffect, useMemo, useState } from "react"
 
 import { Badge } from "@/components/ui/badge"
 import { Button } from "@/components/ui/button"
+import {
+  Popover,
+  PopoverContent,
+  PopoverTrigger,
+} from "@/components/ui/popover"
 import {
   ResizableHandle,
   ResizablePanel,
@@ -217,11 +222,7 @@ export function WorkflowEditor({
             <Controls />
             <MiniMap pannable zoomable />
           </ReactFlow>
-          <div className="absolute left-3 top-3 flex gap-2">
-            <Badge variant={validation.valid ? "default" : "destructive"}>
-              {validation.valid ? "Valid draft" : `${validation.errors.length} issues`}
-            </Badge>
-          </div>
+          <WorkflowValidationBadge errors={validation.errors} valid={validation.valid} />
         </section>
       </ResizablePanel>
       <ResizableHandle withHandle />
@@ -252,3 +253,44 @@ export function WorkflowEditor({
     </ResizablePanelGroup>
   )
 }
+
+function WorkflowValidationBadge({
+  errors,
+  valid,
+}: {
+  errors: string[]
+  valid: boolean
+}) {
+  return (
+    <div className="absolute left-3 top-3 flex gap-2">
+      {valid ? (
+        <Badge variant="default">Valid draft</Badge>
+      ) : (
+        <Popover>
+          <PopoverTrigger
+            render={
+              <button
+                type="button"
+                className="inline-flex rounded-full outline-none focus-visible:ring-2 focus-visible:ring-ring"
+              />
+            }
+          >
+            <Badge variant="destructive" className="cursor-pointer">
+              {errors.length} issues
+            </Badge>
+          </PopoverTrigger>
+          <PopoverContent side="bottom" align="start" className="w-80">
+            <div className="text-sm font-medium">Validation issues</div>
+            <ul className="mt-2 max-h-72 space-y-1 overflow-y-auto text-xs text-destructive">
+              {errors.map((error) => (
+                <li key={error} className="rounded-md bg-destructive/10 px-2 py-1.5">
+                  {error}
+                </li>
+              ))}
+            </ul>
+          </PopoverContent>
+        </Popover>
+      )}
+    </div>
+  )
+}