From 62deb5244aac0ae49ae254fbf12a6732d2c2d09d Mon Sep 17 00:00:00 2001
From: Vedant Mamgain <mamgainvedant@gmail.com>
Date: Tue, 30 Sep 2025 21:25:19 +0530
Subject: [PATCH] [v3-1-test] Replace defaultValue with value in TaskTrySelect
 (#56141)

* Replace defaultValue with value in TaskTrySelect

Changed the Select component to use the controlled 'value' prop instead of 'defaultValue' for try number selection, ensuring the selected value updates correctly with state changes.

* fix: Reorder props in TaskTrySelect for linting compliance

Applied prop ordering fix as requested in code review.
Props are now alphabetically ordered in the Select.Root component.
(cherry picked from commit 3f7991f30e8c3f2dd85755ee02267b9ec882bcbe)

Co-authored-by: Vedant Mamgain <mamgainvedant@gmail.com>
---
 airflow-core/src/airflow/ui/src/components/TaskTrySelect.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/airflow-core/src/airflow/ui/src/components/TaskTrySelect.tsx b/airflow-core/src/airflow/ui/src/components/TaskTrySelect.tsx
index 1d5614ba04a84..bfb9f48c1cea5 100644
--- a/airflow-core/src/airflow/ui/src/components/TaskTrySelect.tsx
+++ b/airflow-core/src/airflow/ui/src/components/TaskTrySelect.tsx
@@ -89,7 +89,6 @@ export const TaskTrySelect = ({ onSelectTryNumber, selectedTryNumber, taskInstan
         <Select.Root
           collection={tryOptions}
           data-testid="select-task-try"
-          defaultValue={[selectedTryNumber?.toString() ?? finalTryNumber.toString()]}
           onValueChange={(details) => {
             if (onSelectTryNumber) {
               onSelectTryNumber(
@@ -97,6 +96,7 @@ export const TaskTrySelect = ({ onSelectTryNumber, selectedTryNumber, taskInstan
               );
             }
           }}
+          value={[selectedTryNumber?.toString() ?? finalTryNumber.toString()]}
           width="200px"
         >
           <Select.Trigger>