Mariano/agent 5 (#1921)

Marfuen · web-flow · commit ca79bc871be8 · 2025-12-14T23:42:35.000-05:00
* refactor(hooks): ensure onComplete is always called after execution
diff --git a/apps/app/src/app/(app)/[orgId]/tasks/[taskId]/hooks/useBrowserExecution.ts b/apps/app/src/app/(app)/[orgId]/tasks/[taskId]/hooks/useBrowserExecution.ts
@@ -68,8 +68,6 @@ export function useBrowserExecution({
         } else {
           toast.error(execRes.data?.error || 'Automation failed');
         }
-
-        onComplete();
       } catch (err) {
         toast.error(err instanceof Error ? err.message : 'Failed to run automation');
       } finally {
@@ -91,6 +89,10 @@ export function useBrowserExecution({
             // Ignore cleanup errors (don't block UI / don't mask original error)
           }
         }
+
+        // Always refresh after any run attempt (success/failure/exception) so the UI doesn't
+        // get stuck showing the old "running" run until a full page refresh.
+        onComplete();
       }
     },
     [organizationId, onNeedsReauth, onComplete],

Original file line number	Diff line number	Diff line change
`@@ -68,8 +68,6 @@ export function useBrowserExecution({`
`68`	`68`	`} else {`
`69`	`69`	`toast.error(execRes.data?.error \|\| 'Automation failed');`
`70`	`70`	`}`
`71`		`-`
`72`		`- onComplete();`
`73`	`71`	`} catch (err) {`
`74`	`72`	`toast.error(err instanceof Error ? err.message : 'Failed to run automation');`
`75`	`73`	`} finally {`
`@@ -91,6 +89,10 @@ export function useBrowserExecution({`
`91`	`89`	`// Ignore cleanup errors (don't block UI / don't mask original error)`
`92`	`90`	`}`
`93`	`91`	`}`
	`92`	`+`
	`93`	`+ // Always refresh after any run attempt (success/failure/exception) so the UI doesn't`
	`94`	`+ // get stuck showing the old "running" run until a full page refresh.`
	`95`	`+ onComplete();`
`94`	`96`	`}`
`95`	`97`	`},`
`96`	`98`	`[organizationId, onNeedsReauth, onComplete],`